From de3f8142fbcb65584484e323536bc39541ddc001 Mon Sep 17 00:00:00 2001
From: Luke Towers <luke@luketowers.ca>
Date: Thu, 29 Aug 2019 17:51:52 -0600
Subject: [PATCH] Support multiple instances of the PermissionEditor on a
 single page

---
 .../partials/_permissioneditor.htm            | 20 +++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/modules/backend/formwidgets/permissioneditor/partials/_permissioneditor.htm b/modules/backend/formwidgets/permissioneditor/partials/_permissioneditor.htm
index 4a6377650..1284a6c59 100644
--- a/modules/backend/formwidgets/permissioneditor/partials/_permissioneditor.htm
+++ b/modules/backend/formwidgets/permissioneditor/partials/_permissioneditor.htm
@@ -32,6 +32,10 @@
                     else {
                         $isChecked = array_key_exists($permission->code, $permissionsData);
                     }
+
+                    $allowId = $this->getId('permission-' . $globalIndex . '-allow');
+                    $inheritId = $this->getId('permission-' . $globalIndex . '-inherit');
+                    $denyId = $this->getId('permission-' . $globalIndex . '-deny');
                 ?>
                 <tr class="<?= $lastIndex == $index ? 'last-section-row' : '' ?>
                         <?= $checkboxMode ? 'mode-checkbox' : 'mode-radio' ?>
@@ -47,7 +51,7 @@
                         <?php if (!$checkboxMode): ?>
                             <div class="radio custom-radio">
                                  <input
-                                    id="<?= 'permission-'.$globalIndex.'-allow' ?>"
+                                    id="<?= $allowId ?>"
                                     name="<?= e($baseFieldName) ?>[<?= e($permission->code) ?>]"
                                     value="1"
                                     type="radio"
@@ -55,19 +59,19 @@
                                     data-radio-color="green"
                                 >
 
-                                <label for="<?= 'permission-'.$globalIndex.'-allow' ?>"><span>Allow</span></label>
+                                <label for="<?= $allowId ?>"><span>Allow</span></label>
                             </div>
                         <?php else: ?>
                             <div class="checkbox custom-checkbox">
                                  <input
-                                    id="<?= 'permission-'.$globalIndex.'-allow' ?>"
+                                    id="<?= $allowId ?>"
                                     name="<?= e($baseFieldName) ?>[<?= e($permission->code) ?>]"
                                     value="1"
                                     type="checkbox"
                                     <?= $isChecked ? 'checked="checked"' : '' ?>
                                 >
 
-                                <label for="<?= 'permission-'.$globalIndex.'-allow' ?>"><span>Allow</span></label>
+                                <label for="<?= $allowId ?>"><span>Allow</span></label>
                             </div>
                         <?php endif?>
                     </td>
@@ -76,20 +80,20 @@
                         <td class="permission-value">
                             <div class="radio custom-radio">
                                  <input
-                                    id="<?= 'permission-'.$globalIndex.'-inherit' ?>"
+                                    id="<?= $inheritId ?>"
                                     name="<?= e($baseFieldName) ?>[<?= e($permission->code) ?>]"
                                     value="0"
                                     <?= $permissionValue == 0 ? 'checked="checked"' : '' ?>
                                     type="radio"
                                 >
 
-                                <label for="<?= 'permission-'.$globalIndex.'-inherit' ?>"><span>Inherit</span></label>
+                                <label for="<?= $inheritId ?>"><span>Inherit</span></label>
                             </div>
                         </td>
                         <td class="permission-value">
                             <div class="radio custom-radio">
                                  <input
-                                    id="<?= 'permission-'.$globalIndex.'-deny' ?>"
+                                    id="<?= $denyId ?>"
                                     name="<?= e($baseFieldName) ?>[<?= e($permission->code) ?>]"
                                     value="-1"
                                     <?= $permissionValue == -1 ? 'checked="checked"' : '' ?>
@@ -97,7 +101,7 @@
                                     data-radio-color="red"
                                 >
 
-                                <label for="<?= 'permission-'.$globalIndex.'-deny' ?>"><span>Deny</span></label>
+                                <label for="<?= $denyId ?>"><span>Deny</span></label>
                             </div>
                         </td>
                     <?php endif ?>