Should fix #18

Wasn’t catering for fields without an existing ID (new fields), now corrected.
nathanpitman · Sep 23, 2014 · 371836d · 371836d
1 parent a985112
commit 371836d
Showing 1 changed file with 10 additions and 8 deletions.
diff --git a/system/third_party/nf_categories_field/ft.nf_categories_field.php b/system/third_party/nf_categories_field/ft.nf_categories_field.php
@@ -225,14 +225,16 @@ function save_settings($settings)
         $settings['field_show_fmt'] = 'n';
         $settings['field_type'] = 'nf_categories_field';
 
-        $field_name = "field_id_".$settings['field_id'];
-
-        if ($settings['sync_cats']) {
-            ee()->db->where($field_name, '');
-            ee()->db->update('channel_data', array($field_name => 'EMPTY'));
-        } else {
-            ee()->db->where($field_name, 'EMPTY');
-            ee()->db->update('channel_data', array($field_name => ''));
+        // Only set the empty field value for existing fields
+        if(!empty($settings['field_id'])) {
+            $field_name = "field_id_".$settings['field_id'];
+            if ($settings['sync_cats']) {
+                ee()->db->where($field_name, '');
+                ee()->db->update('channel_data', array($field_name => 'EMPTY'));
+            } else {
+                ee()->db->where($field_name, 'EMPTY');
+                ee()->db->update('channel_data', array($field_name => ''));
+            }
         }
 
         return $settings;