[Rt-commit] rt branch, 4.0/group-dashboards, updated. rt-4.0.1-133-g2e85388

Alex Vandiver alexmv at bestpractical.com
Mon Jul 18 17:21:27 EDT 2011


The branch, 4.0/group-dashboards has been updated
       via  2e85388e291dffa14b9817d0bb1305d1b4be5b0f (commit)
      from  f6cb772b25445c69f3024ede7a020d3657134d1d (commit)

Summary of changes:
 share/html/Elements/Tabs |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

- Log -----------------------------------------------------------------
commit 2e85388e291dffa14b9817d0bb1305d1b4be5b0f
Author: Alex Vandiver <alexmv at bestpractical.com>
Date:   Mon Jul 18 17:21:24 2011 -0400

    Explain why we're skipping superusers in the menu

diff --git a/share/html/Elements/Tabs b/share/html/Elements/Tabs
index ee02c32..5db9bc3 100755
--- a/share/html/Elements/Tabs
+++ b/share/html/Elements/Tabs
@@ -61,6 +61,10 @@ my $query_string = sub {
 my $build_main_nav = sub {
 
     my $home = Menu->child( home => title => loc('Homepage'), path => '/' );
+    # We explicitly exclude superusers; otherwise the dashboards for
+    # groups you're not in (but can see the dashboards of by dint of
+    # being a superuser) would push the useful ones from the groups
+    # you're actually in off of the stack.
     my @dashboards = $m->comp("/Dashboards/Elements/ListOfDashboards", IncludeSuperusers => 0);
     my $limit      = 7;
 

-----------------------------------------------------------------------


More information about the Rt-commit mailing list