Added error checking for blank names and values

Author: amyreese

Snippets/Snippets.php

@@ -26,6 +26,13 @@ public function config() {
 		);
 	}
 
+	public function errors() {
+		return array(
+			"name_empty" => plugin_lang_get("error_name_empty"),
+			"value_empty" => plugin_lang_get("error_value_empty"),
+		);
+	}
+
 	public function hooks() {
 		return array(
 			"EVENT_MENU_ACCOUNT" => "menu_account",

Snippets/lang/strings_english.txt

@@ -8,6 +8,9 @@ $s_plugin_Snippets_name = "Snippets";
 $s_plugin_Snippets_description = "Allow users to save blocks of text";
 $s_plugin_Snippets_version = "0.2";
 
+$s_plugin_Snippets_error_name_empty = "The snippet name is empty.";
+$s_plugin_Snippets_error_value_empty = "The snippet value is empty.";
+
 $s_plugin_Snippets_select_label = "Snippets: ";
 $s_plugin_Snippets_select_default = "--";
 

Snippets/pages/snippet_create.php

@@ -16,6 +16,13 @@
 $name = gpc_get_string("name");
 $value = gpc_get_string("value");
 
+if (is_blank($name)) {
+	plugin_error("name_empty");
+}
+if (is_blank($value)) {
+	plugin_error("value_empty");
+}
+
 $snippet = new Snippet(0, $name, $value, $user_id);
 $snippet->save();
 

Snippets/pages/snippet_list_action.php

@@ -95,8 +95,12 @@ function array_object_properties($arr, $prop) {
 		if ($snippet->name != $new_name
 			|| $snippet->value != $new_value)
 		{
-			$snippet->name = $new_name;
-			$snippet->value = $new_value;
+			if (!is_blank($new_name)) {
+				$snippet->name = $new_name;
+			}
+			if (!is_blank($new_value)) {
+				$snippet->value = $new_value;
+			}
 
 			$snippet->save();
 		}