From 50dbcc4e7a38642856cbeeef88d2b3a4a884b5e1 Mon Sep 17 00:00:00 2001 From: "mkanat%bugzilla.org" <> Date: Fri, 8 Aug 2008 06:26:33 +0000 Subject: Bug 442031: Make Bugzilla::User::groups return an arrayref of Bugzilla::Group objects (instead of a hashref of group ids and names). --- .../en/default/account/prefs/permissions.html.tmpl | 4 ++-- template/en/default/admin/admin.html.tmpl | 20 ++++++++++---------- .../admin/fieldvalues/confirm-delete.html.tmpl | 2 +- .../en/default/admin/users/confirm-delete.html.tmpl | 4 ++-- template/en/default/admin/users/userdata.html.tmpl | 2 +- template/en/default/attachment/edit.html.tmpl | 2 +- template/en/default/bug/dependency-tree.html.tmpl | 2 +- template/en/default/global/common-links.html.tmpl | 8 ++++---- template/en/default/global/site-navigation.html.tmpl | 16 ++++++++-------- template/en/default/global/user-error.html.tmpl | 2 +- template/en/default/list/quips.html.tmpl | 4 ++-- template/en/default/pages/sudo.html.tmpl | 4 ++-- template/en/default/sidebar.xul.tmpl | 16 ++++++++-------- 13 files changed, 43 insertions(+), 43 deletions(-) (limited to 'template') diff --git a/template/en/default/account/prefs/permissions.html.tmpl b/template/en/default/account/prefs/permissions.html.tmpl index 4a097e2f2..5e8dc9ca2 100644 --- a/template/en/default/account/prefs/permissions.html.tmpl +++ b/template/en/default/account/prefs/permissions.html.tmpl @@ -65,7 +65,7 @@ There are no permission bits set on your account. [% END %] - [% IF user.groups.editusers %] + [% IF user.in_group('editusers') %]
You have editusers privileges. You can turn on and off all permissions for all users. @@ -83,7 +83,7 @@ [% END %] - [% IF user.groups.bz_sudoers %] + [% IF user.in_group('bz_sudoers') %]
You are a member of the bz_sudoers group, so you can impersonate someone else. diff --git a/template/en/default/admin/admin.html.tmpl b/template/en/default/admin/admin.html.tmpl index c681767b5..15f126ba5 100644 --- a/template/en/default/admin/admin.html.tmpl +++ b/template/en/default/admin/admin.html.tmpl @@ -36,7 +36,7 @@
- [% class = user.groups.tweakparams ? "" : "forbidden" %] + [% class = user.in_group('tweakparams') ? "" : "forbidden" %]
Parameters
Set core parameters of the installation. That's the place where you specify the URL to access this installation, determine how @@ -49,25 +49,25 @@ which will be used by default for all users. Users will be able to edit their own preferences from the Preferences.
- [% class = user.groups.editcomponents ? "" : "forbidden" %] + [% class = user.in_group('editcomponents') ? "" : "forbidden" %]
Sanity Check
Run sanity checks to locate problems in your database. This may take several tens of minutes depending on the size of your installation. You can also automate this check by running sanitycheck.pl from a cron job. A notification will be sent per email to the specified user if errors are detected.
- [% class = (user.groups.editusers || user.can_bless) ? "" : "forbidden" %] + [% class = (user.in_group('editusers') || user.can_bless) ? "" : "forbidden" %]
Users
Create new user accounts or edit existing ones. You can also add and remove users from groups (also known as "user privileges").
- [% class = (Param('useclassification') && user.groups.editclassifications) ? "" : "forbidden" %] + [% class = (Param('useclassification') && user.in_group('editclassifications')) ? "" : "forbidden" %]
Classifications
If your installation has to manage many products at once, it's a good idea to group these products into distinct categories. This lets users find information more easily when doing searches or when filing new [% terms.bugs %].
- [% class = (user.groups.editcomponents + [% class = (user.in_group('editcomponents') || user.get_products_by_permission("editcomponents").size) ? "" : "forbidden" %]
Products
Edit all aspects of products, including group restrictions @@ -76,7 +76,7 @@ components, versions and milestones directly.
- [% class = user.groups.editcomponents ? "" : "forbidden" %] + [% class = user.in_group('editcomponents') ? "" : "forbidden" %]
Flags
A flag is a custom 4-states attribute of [% terms.bugs %] and/or attachments. These states are: granted, denied, requested and undefined. @@ -87,7 +87,7 @@
- [% class = user.groups.admin ? "" : "forbidden" %] + [% class = user.in_group('admin') ? "" : "forbidden" %]
Custom Fields
[% terms.Bugzilla %] lets you define fields which are not implemented by default, based on your local and specific requirements. @@ -107,18 +107,18 @@ statuses available on [% terms.bug %] creation and allowed [% terms.bug %] status transitions when editing existing [% terms.bugs %].
- [% class = user.groups.creategroups ? "" : "forbidden" %] + [% class = user.in_group('creategroups') ? "" : "forbidden" %]
Groups
Define groups which will be used in the installation. They can either be used to define new user privileges or to restrict the access to some [% terms.bugs %].
- [% class = user.groups.editkeywords ? "" : "forbidden" %] + [% class = user.in_group('editkeywords') ? "" : "forbidden" %]
Keywords
Set keywords to be used with [% terms.bugs %]. Keywords are an easy way to "tag" [% terms.bugs %] to let you find them more easily later.
- [% class = user.groups.bz_canusewhines ? "" : "forbidden" %] + [% class = user.in_group('bz_canusewhines') ? "" : "forbidden" %]
Whining
Set queries which will be run at some specified date and time, and get the result of these queries directly per email. This is a diff --git a/template/en/default/admin/fieldvalues/confirm-delete.html.tmpl b/template/en/default/admin/fieldvalues/confirm-delete.html.tmpl index 3320ae48f..d18e03a53 100644 --- a/template/en/default/admin/fieldvalues/confirm-delete.html.tmpl +++ b/template/en/default/admin/fieldvalues/confirm-delete.html.tmpl @@ -71,7 +71,7 @@ [% IF param_name.defined && Param(param_name) == value %]
  • '[% value FILTER html %]' is the default value for the '[% field.description FILTER html %]' field. - [% IF user.groups.tweakparams %] + [% IF user.in_group('tweakparams') %] You first have to change the default value for this field before you can delete this value. diff --git a/template/en/default/admin/users/confirm-delete.html.tmpl b/template/en/default/admin/users/confirm-delete.html.tmpl index 218775dca..77d6772a6 100644 --- a/template/en/default/admin/users/confirm-delete.html.tmpl +++ b/template/en/default/admin/users/confirm-delete.html.tmpl @@ -69,8 +69,8 @@ [% IF otheruser.groups.size %]
      - [% FOREACH group = otheruser.groups.keys %] -
    • [% group FILTER html %]
      • + [% FOREACH group = otheruser.groups %] +
    • [% group.name FILTER html %]
      • [% END %]
    [% ELSE %] diff --git a/template/en/default/admin/users/userdata.html.tmpl b/template/en/default/admin/users/userdata.html.tmpl index feee4c5d6..f23aa1b85 100644 --- a/template/en/default/admin/users/userdata.html.tmpl +++ b/template/en/default/admin/users/userdata.html.tmpl @@ -27,7 +27,7 @@ [% IF editform %] - [% IF !otheruser.groups.bz_sudo_protect %] + [% IF !otheruser.in_group('bz_sudo_protect') %]
    Impersonate this user diff --git a/template/en/default/attachment/edit.html.tmpl b/template/en/default/attachment/edit.html.tmpl index 7989dc7c9..49c28aa64 100644 --- a/template/en/default/attachment/edit.html.tmpl +++ b/template/en/default/attachment/edit.html.tmpl @@ -274,7 +274,7 @@ | Diff [% END %] [% IF Param("allow_attachment_deletion") - && user.groups.admin + && user.in_group('admin') && attachment.datasize > 0 %] | Delete [% END %] diff --git a/template/en/default/bug/dependency-tree.html.tmpl b/template/en/default/bug/dependency-tree.html.tmpl index cc49d2fa4..adabf8ea2 100644 --- a/template/en/default/bug/dependency-tree.html.tmpl +++ b/template/en/default/bug/dependency-tree.html.tmpl @@ -83,7 +83,7 @@ [% IF ids.size %] ([% IF maxdepth -%]Up to [% maxdepth %] level[% "s" IF maxdepth > 1 %] deep | [% END -%] view as [% terms.bug %] list - [% IF user.groups.editbugs && ids.size > 1 %] + [% IF user.in_group('editbugs') && ids.size > 1 %] | change several [% END %])
      diff --git a/template/en/default/global/common-links.html.tmpl b/template/en/default/global/common-links.html.tmpl index bfd18af4d..4690d177a 100644 --- a/template/en/default/global/common-links.html.tmpl +++ b/template/en/default/global/common-links.html.tmpl @@ -56,10 +56,10 @@ [% IF user.login %]
    • | Preferences
      • - [% IF user.groups.tweakparams || user.groups.editusers || user.can_bless - || (Param('useclassification') && user.groups.editclassifications) - || user.groups.editcomponents || user.groups.admin || user.groups.creategroups - || user.groups.editkeywords || user.groups.bz_canusewhines + [% IF user.in_group('tweakparams') || user.in_group('editusers') || user.can_bless + || (Param('useclassification') && user.in_group('editclassifications')) + || user.in_group('editcomponents') || user.in_group('admin') || user.in_group('creategroups') + || user.in_group('editkeywords') || user.in_group('bz_canusewhines') || user.get_products_by_permission("editcomponents").size %]
    • | Administration
      • [% END %] diff --git a/template/en/default/global/site-navigation.html.tmpl b/template/en/default/global/site-navigation.html.tmpl index 2acbcf44d..5440fe1f8 100644 --- a/template/en/default/global/site-navigation.html.tmpl +++ b/template/en/default/global/site-navigation.html.tmpl @@ -108,20 +108,20 @@ [%# *** Bugzilla Administration Tools *** %] [% IF user.login %] [% '' IF user.groups.tweakparams %] + href="editparams.cgi">' IF user.in_group('tweakparams') %] [% '' IF user.groups.editusers %] + href="editusers.cgi">' IF user.in_group('editusers') %] [% '' - IF user.groups.editcomponents || user.get_products_by_permission("editcomponents").size %] + IF user.in_group('editcomponents') || user.get_products_by_permission("editcomponents").size %] [% '' IF user.groups.editcomponents %] + href="editflagtypes.cgi">' IF user.in_group('editcomponents') %] [% '' IF user.groups.creategroups %] + href="editgroups.cgi">' IF user.in_group('creategroups') %] [% '' IF user.groups.editkeywords %] + href="editkeywords.cgi">' IF user.in_group('editkeywords') %] [% '' IF user.groups.bz_canusewhines %] + href="editwhines.cgi">' IF user.in_group('bz_canusewhines') %] [% '' IF user.groups.editcomponents %] + href="sanitycheck.cgi">' IF user.in_group('editcomponents') %] [% END %] [% END %] diff --git a/template/en/default/global/user-error.html.tmpl b/template/en/default/global/user-error.html.tmpl index 09d40272c..b04c9e7dc 100644 --- a/template/en/default/global/user-error.html.tmpl +++ b/template/en/default/global/user-error.html.tmpl @@ -443,7 +443,7 @@ [% title = "Specified Field Value Is Default" %] '[% value FILTER html %]' is the default value for the '[% field.description FILTER html %]' field and cannot be deleted. - [% IF user.groups.tweakparams %] + [% IF user.in_group('tweakparams') %] You have to change the default value first. [% END %] diff --git a/template/en/default/list/quips.html.tmpl b/template/en/default/list/quips.html.tmpl index 14cecb26e..b8359ffc6 100644 --- a/template/en/default/list/quips.html.tmpl +++ b/template/en/default/list/quips.html.tmpl @@ -37,7 +37,7 @@

      Your quip '[% added_quip FILTER html %]' has been added. - [% IF Param("quip_list_entry_control") == "moderated" AND !user.groups.admin %] + [% IF Param("quip_list_entry_control") == "moderated" AND !user.in_group('admin') %] It will be used as soon as it gets approved. [% END %] @@ -66,7 +66,7 @@

      You can extend the quip list. Type in something clever or funny or boring (but not obscene or offensive, please) and bonk on the button. - [% IF Param("quip_list_entry_control") == "moderated" AND !user.groups.admin %] + [% IF Param("quip_list_entry_control") == "moderated" AND !user.in_group('admin') %] Note that your quip has to be approved before it is used. [% END %]

      diff --git a/template/en/default/pages/sudo.html.tmpl b/template/en/default/pages/sudo.html.tmpl index dff2d7d87..c790ff1ab 100644 --- a/template/en/default/pages/sudo.html.tmpl +++ b/template/en/default/pages/sudo.html.tmpl @@ -51,14 +51,14 @@ doing the impersonating has the appropriate privileges.

      - [% IF user.groups.bz_sudoers %] + [% IF user.in_group('bz_sudoers') %] You are a member of the bz_sudoers group. You may use this feature to impersonate others. [% ELSE %] You are not a member of an appropriate group. You may not use this feature. [% END %] - [% IF user.groups.bz_sudo_protect %] + [% IF user.in_group('bz_sudo_protect') %]
      You are a member of the bz_sudo_protect group. Other people will not be able to use this feature to impersonate you. diff --git a/template/en/default/sidebar.xul.tmpl b/template/en/default/sidebar.xul.tmpl index 31c14729f..8035c8298 100644 --- a/template/en/default/sidebar.xul.tmpl +++ b/template/en/default/sidebar.xul.tmpl @@ -69,31 +69,31 @@ function normal_keypress_handler( aEvent ) { [% IF user.id %] - [%- IF user.groups.tweakparams %] + [%- IF user.in_group('tweakparams') %] [%- END %] - [%- IF user.groups.editusers || user.can_bless %] + [%- IF user.in_group('editusers') || user.can_bless %] [%- END %] - [%- IF Param('useclassification') && user.groups.editclassifications %] + [%- IF Param('useclassification') && user.in_group('editclassifications') %] [%- END %] - [%- IF user.groups.editcomponents %] + [%- IF user.in_group('editcomponents') %] [%- END %] - [%- IF user.groups.creategroups %] + [%- IF user.in_group('creategroups') %] [%- END %] - [%- IF user.groups.editkeywords %] + [%- IF user.in_group('editkeywords') %] [%- END %] - [%- IF user.groups.bz_canusewhines %] + [%- IF user.in_group('bz_canusewhines') %] [%- END %] - [%- IF user.groups.editcomponents %] + [%- IF user.in_group('editcomponents') %] [%- END %] [%- IF user.authorizer.can_logout %] -- cgit v1.2.3-24-g4f1b