From 0a06582ce7766cbf016c568c62b9f35b4b85aaea Mon Sep 17 00:00:00 2001
From: Samuel Georges <sam@daftspunk.com>
Date: Sat, 22 Jul 2017 21:32:16 +1000
Subject: [PATCH] Layouts can be reset to default

---
 modules/system/controllers/MailLayouts.php    | 12 +++++++++
 modules/system/controllers/MailPartials.php   |  5 ++++
 .../system/controllers/maillayouts/update.htm | 11 +++++++-
 .../controllers/mailpartials/update.htm       | 16 +++++-------
 modules/system/models/MailLayout.php          | 26 +++++++++----------
 5 files changed, 47 insertions(+), 23 deletions(-)

diff --git a/modules/system/controllers/MailLayouts.php b/modules/system/controllers/MailLayouts.php
index d142cc4ec..da913c2be 100644
--- a/modules/system/controllers/MailLayouts.php
+++ b/modules/system/controllers/MailLayouts.php
@@ -35,4 +35,16 @@ class MailLayouts extends Controller
         BackendMenu::setContext('October.System', 'system', 'settings');
         SettingsManager::setContext('October.System', 'mail_templates');
     }
+
+    public function update_onResetDefault($recordId)
+    {
+        $model = $this->formFindModelObject($recordId);
+
+        $model->fillFromCode();
+        $model->save();
+
+        Flash::success(Lang::get('backend::lang.form.reset_success'));
+
+        return Redirect::refresh();
+    }
 }
diff --git a/modules/system/controllers/MailPartials.php b/modules/system/controllers/MailPartials.php
index 4e0606d52..b7692d031 100644
--- a/modules/system/controllers/MailPartials.php
+++ b/modules/system/controllers/MailPartials.php
@@ -35,4 +35,9 @@ class MailPartials extends Controller
         BackendMenu::setContext('October.System', 'system', 'settings');
         SettingsManager::setContext('October.System', 'mail_templates');
     }
+
+    public function formBeforeSave($model)
+    {
+        $model->is_custom = 1;
+    }
 }
diff --git a/modules/system/controllers/maillayouts/update.htm b/modules/system/controllers/maillayouts/update.htm
index 62bab8bed..5a403ca0c 100644
--- a/modules/system/controllers/maillayouts/update.htm
+++ b/modules/system/controllers/maillayouts/update.htm
@@ -44,7 +44,16 @@
                     class="btn btn-default">
                     <?= e(trans('backend::lang.form.save_and_close')) ?>
                 </button>
-                <?php if (!$formModel->is_locked): ?>
+                <?php if ($formModel->is_locked): ?>
+                    <button
+                        type="button"
+                        class="btn btn-danger pull-right"
+                        data-request="onResetDefault"
+                        data-load-indicator="<?= e(trans('backend::lang.form.resetting')) ?>"
+                        data-request-confirm="<?= e(trans('backend::lang.form.action_confirm')) ?>">
+                        <?= e(trans('backend::lang.form.reset_default')) ?>
+                    </button>
+                <?php else: ?>
                     <button
                         type="button"
                         class="oc-icon-trash-o btn-icon danger pull-right"
diff --git a/modules/system/controllers/mailpartials/update.htm b/modules/system/controllers/mailpartials/update.htm
index b5e5ea706..f328f378f 100644
--- a/modules/system/controllers/mailpartials/update.htm
+++ b/modules/system/controllers/mailpartials/update.htm
@@ -44,15 +44,13 @@
                     class="btn btn-default">
                     <?= e(trans('backend::lang.form.save_and_close')) ?>
                 </button>
-                <?php if (!$formModel->is_locked): ?>
-                    <button
-                        type="button"
-                        class="oc-icon-trash-o btn-icon danger pull-right"
-                        data-request="onDelete"
-                        data-load-indicator="<?= e(trans('system::lang.mail_templates.deleting_layout')) ?>"
-                        data-request-confirm="<?= e(trans('system::lang.mail_templates.delete_layout_confirm')) ?>">
-                    </button>
-                <?php endif ?>
+                <button
+                    type="button"
+                    class="oc-icon-trash-o btn-icon danger pull-right"
+                    data-request="onDelete"
+                    data-load-indicator="<?= e(trans('system::lang.mail_templates.deleting_layout')) ?>"
+                    data-request-confirm="<?= e(trans('system::lang.mail_templates.delete_layout_confirm')) ?>">
+                </button>
                 <span class="btn-text">
                     <?= e(trans('backend::lang.form.or')) ?> <a href="<?= Backend::url('system/mailtemplates/index/partials') ?>"><?= e(trans('backend::lang.form.cancel')) ?></a>
                 </span>
diff --git a/modules/system/models/MailLayout.php b/modules/system/models/MailLayout.php
index ef108d8ec..df7065459 100644
--- a/modules/system/models/MailLayout.php
+++ b/modules/system/models/MailLayout.php
@@ -107,21 +107,21 @@ class MailLayout extends Model
         $sections = self::getTemplateSections($path);
 
         $css = '
-            @media only screen and (max-width: 600px) {
-                .inner-body {
-                    width: 100% !important;
-                }
+@media only screen and (max-width: 600px) {
+    .inner-body {
+        width: 100% !important;
+    }
 
-                .footer {
-                    width: 100% !important;
-                }
-            }
+    .footer {
+        width: 100% !important;
+    }
+}
 
-            @media only screen and (max-width: 500px) {
-                .button {
-                    width: 100% !important;
-                }
-            }
+@media only screen and (max-width: 500px) {
+    .button {
+        width: 100% !important;
+    }
+}
         ';
 
         $this->name = array_get($sections, 'settings.name', '???');