Merge pull request #1729 from CachetHQ/fix-removal-customizations

Fixes removal of Customization values

Author: GrahamCampbell

app/Config/Repository.php

@@ -70,6 +70,20 @@ public function set($name, $value)
         }
     }
 
+    /**
+     * Deletes a setting.
+     *
+     * @param string $name
+     *
+     * @return void
+     */
+    public function delete($name)
+    {
+        $this->stale = true;
+
+        $this->model->where('name', $name)->delete();
+    }
+
     /**
      * Is the config state stale?
      *

app/Http/Controllers/Dashboard/SettingsController.php

@@ -226,12 +226,22 @@ public function postSettings()
             $setting->set('app_banner', null);
         }
 
-        if ($header = Binput::get('header', null, false, false)) {
-            $setting->set('header', $header);
+        $parameters = Binput::all();
+
+        if (isset($parameters['header'])) {
+            if ($header = Binput::get('header', null, false, false)) {
+                $setting->set('header', $header);
+            } else {
+                $setting->delete('header');
+            }
         }
 
-        if ($footer = Binput::get('footer', null, false, false)) {
-            $setting->set('footer', $footer);
+        if (isset($parameters['footer'])) {
+            if ($footer = Binput::get('footer', null, false, false)) {
+                $setting->set('footer', $footer);
+            } else {
+                $setting->delete('footer');
+            }
         }
 
         if (Binput::hasFile('app_banner')) {