From 4d4352c2f89d39c3dd5b010b7249745dd1f16d03 Mon Sep 17 00:00:00 2001
From: Michael M Slusarz <slusarz@curecanti.org>
Date: Thu, 21 Oct 2010 00:04:33 -0600
Subject: [PATCH] Bug #4480: always show mailbox list, even on ACL list error

---
 imp/lib/Prefs/Ui.php         | 10 ++--------
 imp/templates/prefs/acl.html |  4 ++--
 2 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/imp/lib/Prefs/Ui.php b/imp/lib/Prefs/Ui.php
index 1ee767d41..630792567 100644
--- a/imp/lib/Prefs/Ui.php
+++ b/imp/lib/Prefs/Ui.php
@@ -701,10 +701,9 @@ class IMP_Prefs_Ui
             $curr_acl = $acl->getACL($folder);
         } catch (IMP_Exception $e) {
             $GLOBALS['notification']->push($e);
-            return '';
+            $curr_acl = array();
         }
         $canEdit = $acl->canEdit($folder, $GLOBALS['registry']->getAuth());
-
         $canEdit = false;
 
         if (!$canEdit) {
@@ -722,9 +721,8 @@ class IMP_Prefs_Ui
         $t->set('current', sprintf(_("Current access to %s"), IMP::displayFolder($folder)));
         $t->set('folder', IMP::formMbox($folder, true));
         $t->set('hasacl', count($curr_acl));
-        $t->set('maxrule', 1);
 
-        if (!$t->get('noacl')) {
+        if (!$t->get('hasacl')) {
             $i = 0;
             $cval = array();
             $protected = $acl->getProtected();
@@ -750,10 +748,6 @@ class IMP_Prefs_Ui
              }
 
              $t->set('curr_acl', $cval);
-             $t->set('maxval', count($curr_acl));
-
-             /* Number of individual ACL options, for table rendering. */
-             $t->set('maxrule', count($rights));
         }
 
         $t->set('canedit', $canEdit);
diff --git a/imp/templates/prefs/acl.html b/imp/templates/prefs/acl.html
index 5e6bb30da..c1cc15cbc 100644
--- a/imp/templates/prefs/acl.html
+++ b/imp/templates/prefs/acl.html
@@ -1,11 +1,11 @@
 <input type="hidden" name="acl_folder" value="<tag:folder />" />
 
+<if:hasacl>
 <div>
  <h3><tag:current /></h3>
 </div>
 
 <table class="prefsAclTable">
-<if:hasacl>
 <tr>
  <th class="item" width="<tag:width />">
   <h4><strong><gettext>User</gettext></strong></h4>
@@ -50,8 +50,8 @@
 </loop:rights>
 </tr>
 </if:canedit>
-</if:hasacl>
 </table>
+</if:hasacl>
 
 <div>
  <span class="folderImg"></span>
-- 
2.11.0