From 9ad253a2bbd76048934c304c54239b94f45b3809 Mon Sep 17 00:00:00 2001
From: Bharat Mediratta <bharat@menalto.com>
Date: Wed, 31 Dec 2008 04:21:38 +0000
Subject: Switch from checkboxes to simple anchors so that we can reflect
 multiple states better

---
 core/views/permission_edit.html.php | 22 ++++++++++------------
 1 file changed, 10 insertions(+), 12 deletions(-)

(limited to 'core')

diff --git a/core/views/permission_edit.html.php b/core/views/permission_edit.html.php
index 9e65b864..332d31ab 100644
--- a/core/views/permission_edit.html.php
+++ b/core/views/permission_edit.html.php
@@ -16,18 +16,16 @@
         <td> <?= _($permission->display_name) ?> </td>
         <? foreach ($groups as $group): ?>
         <td>
-          <? $locks = access::locking_items($group, $permission->name, $item) ?>
-          <input type="checkbox"
-                 name="<?= "{$permission->name}_$group->id" ?>"
-                 value="1"
-                 <? if (access::group_can($group, $permission->name, $item)): ?> checked="checked" <? endif ?>
-            <? if ($locks): ?> disabled="disabled" <? endif ?>
-            />
-          <? if ($locks): ?>
-          Locked by: <!-- Not internationalized because its hard and this is prob. the wrong UI anyway -->
-          <? foreach ($locks as $lock): ?>
-          <a href="<?= url::site("{$lock->type}s/$lock->id") ?>"><?= $lock->title ?></a>
-          <? endforeach ?>
+          <? $locked = access::locking_items($group, $permission->name, $item) ?>
+          <? $allowed = access::group_can($group, $permission->name, $item) ?>
+          <? if ($locked && $allowed): ?>
+          allowed <a href="#">locked</a>
+          <? elseif ($locked && !$allowed): ?>
+          denied <a href="#">locked</a>
+          <? elseif ($allowed): ?>
+          <a href="#">allowed</a>
+          <? elseif (!$allowed): ?>
+          <a href="#">denied</a>
           <? endif ?>
         </td>
         <? endforeach ?>
-- 
cgit v1.2.3