Merge pull request #29 from tattersoftware/parent-null

Template Bugs

Author: MGatner

src/Controllers/Templates.php

@@ -59,7 +59,7 @@ public function new($id = null): string
      */
     public function create(): RedirectResponse
     {
-        if ($this->model->insert($this->request->getPost())) {
+        if ($this->model->insert($this->getData())) {
             return redirect()->to(site_url('emails/templates'))->with('success', 'Email template created.');
         }
 
@@ -115,7 +115,7 @@ public function update($id = null): RedirectResponse
     {
         $template = $this->getTemplate($id);
 
-        if ($this->model->update($template->id, $this->request->getPost())) {
+        if ($this->model->update($template->id, $this->getData())) {
             return redirect()->back()->with('success', 'Email template updated.');
         }
 
@@ -181,4 +181,18 @@ public function send_commit($id = null): RedirectResponse
 
         return redirect()->back()->withInput()->with('error', $email->printDebugger([]));
     }
+
+    /**
+     * Retrieves POST data safe for the database.
+     */
+    protected function getData(): array
+    {
+        $data = $this->request->getPost();
+
+        if (array_key_exists('parent_id', $data)) {
+            $data['parent_id'] = empty($data['parent_id']) ? null : $data['parent_id'];
+        }
+
+        return $data;
+    }
 }

src/Entities/Template.php

@@ -16,6 +16,7 @@ class Template extends Entity
         'deleted_at',
     ];
     protected $casts = [
+        'id'        => '?int',
         'parent_id' => '?int',
     ];