From e9c8107cc5ffa110df98ae5f15e7233b498f5cd2 Mon Sep 17 00:00:00 2001
From: Samuel Georges <sam@daftspunk.com>
Date: Sat, 6 Dec 2014 12:16:09 +1100
Subject: [PATCH] Checkbox list should support preview mode

---
 .../form/partials/_field_checkboxlist.htm     | 118 +++++++++++-------
 1 file changed, 75 insertions(+), 43 deletions(-)

diff --git a/modules/backend/widgets/form/partials/_field_checkboxlist.htm b/modules/backend/widgets/form/partials/_field_checkboxlist.htm
index 4c9c4fd83..6ebe62e1f 100644
--- a/modules/backend/widgets/form/partials/_field_checkboxlist.htm
+++ b/modules/backend/widgets/form/partials/_field_checkboxlist.htm
@@ -5,54 +5,86 @@
 <!-- Checkbox List -->
 <?php if (count($fieldOptions)): ?>
 
-    <?php if (count($fieldOptions) > 10): ?>
-        <!-- Quick selection -->
-        <small>
-            <?= e(trans('backend::lang.form.select')) ?>:
-            <a href="javascript:;" onclick="jQuery('#<?= $field->getId('scrollable') ?> input[type=checkbox]').prop('checked', true)"><?= e(trans('backend::lang.form.select_all')) ?></a>,
-            <a href="javascript:;" onclick="jQuery('#<?= $field->getId('scrollable') ?> input[type=checkbox]').prop('checked', false)"><?= e(trans('backend::lang.form.select_none')) ?></a>
-        </small>
+    <?php if ($this->previewMode): ?>
+        <!-- Read-only -->
 
-        <!-- Scrollable Checkbox list -->
-        <div class="field-checkboxlist-scrollable" id="<?= $field->getId('scrollable') ?>">
-            <div class="control-scrollbar" data-control="scrollbar">
-    <?php endif ?>
+        <?php $index = 0; foreach ($fieldOptions as $value => $option): ?>
+            <?php
+                $index++;
+                $checkboxId = 'checkbox_'.$field->getId().'_'.$index;
+                if (!in_array($value, $checkedValues)) continue;
+                if (is_string($option)) $option = [$option];
+            ?>
+            <div class="checkbox custom-checkbox">
+                <input
+                    type="checkbox"
+                    id="<?= $checkboxId ?>"
+                    name="<?= $field->getName() ?>[]"
+                    value="<?= $value ?>"
+                    disabled="disabled"
+                    checked="checked">
 
-    <input
-        type="hidden"
-        name="<?= $field->getName() ?>"
-        value="0"
-        <?= $this->previewMode ? 'disabled="disabled"' : '' ?>>
-
-    <?php $index = 0; foreach ($fieldOptions as $value => $option): ?>
-        <?php
-            $index++;
-            $checkboxId = 'checkbox_'.$field->getId().'_'.$index;
-            if (is_string($option)) $option = [$option];
-        ?>
-        <div class="checkbox custom-checkbox">
-            <input
-                type="checkbox"
-                id="<?= $checkboxId ?>"
-                name="<?= $field->getName() ?>[]"
-                value="<?= $value ?>"
-                <?= $this->previewMode ? 'readonly="readonly"' : '' ?>
-                <?= in_array($value, $checkedValues) ? 'checked="checked"' : '' ?>>
-
-            <label for="<?= $checkboxId ?>">
-                <?= e(trans($option[0])) ?>
-            </label>
-            <?php if (isset($option[1])): ?>
-                <p class="help-block"><?= e(trans($option[1])) ?></p>
-            <?php endif ?>
-        </div>
-    <?php endforeach ?>
-
-    <?php if (count($fieldOptions) > 10): ?>
+                <label for="<?= $checkboxId ?>">
+                    <?= e(trans($option[0])) ?>
+                </label>
+                <?php if (isset($option[1])): ?>
+                    <p class="help-block"><?= e(trans($option[1])) ?></p>
+                <?php endif ?>
             </div>
-        </div>
+        <?php endforeach ?>
+
+    <?php else: ?>
+        <!-- Editable -->
+
+        <?php if (count($fieldOptions) > 10): ?>
+            <!-- Quick selection -->
+            <small>
+                <?= e(trans('backend::lang.form.select')) ?>:
+                <a href="javascript:;" onclick="jQuery('#<?= $field->getId('scrollable') ?> input[type=checkbox]').prop('checked', true)"><?= e(trans('backend::lang.form.select_all')) ?></a>,
+                <a href="javascript:;" onclick="jQuery('#<?= $field->getId('scrollable') ?> input[type=checkbox]').prop('checked', false)"><?= e(trans('backend::lang.form.select_none')) ?></a>
+            </small>
+
+            <!-- Scrollable Checkbox list -->
+            <div class="field-checkboxlist-scrollable" id="<?= $field->getId('scrollable') ?>">
+                <div class="control-scrollbar" data-control="scrollbar">
+        <?php endif ?>
+
+        <input
+            type="hidden"
+            name="<?= $field->getName() ?>"
+            value="0" />
+
+        <?php $index = 0; foreach ($fieldOptions as $value => $option): ?>
+            <?php
+                $index++;
+                $checkboxId = 'checkbox_'.$field->getId().'_'.$index;
+                if (is_string($option)) $option = [$option];
+            ?>
+            <div class="checkbox custom-checkbox">
+                <input
+                    type="checkbox"
+                    id="<?= $checkboxId ?>"
+                    name="<?= $field->getName() ?>[]"
+                    value="<?= $value ?>"
+                    <?= in_array($value, $checkedValues) ? 'checked="checked"' : '' ?>>
+
+                <label for="<?= $checkboxId ?>">
+                    <?= e(trans($option[0])) ?>
+                </label>
+                <?php if (isset($option[1])): ?>
+                    <p class="help-block"><?= e(trans($option[1])) ?></p>
+                <?php endif ?>
+            </div>
+        <?php endforeach ?>
+
+        <?php if (count($fieldOptions) > 10): ?>
+                </div>
+            </div>
+        <?php endif ?>
+
     <?php endif ?>
 
+
 <?php else: ?>
 
     <!-- No options specified -->