aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authortravis%sedsystems.ca <>2005-01-28 03:50:25 +0000
committertravis%sedsystems.ca <>2005-01-28 03:50:25 +0000
commite749f7333f0e2844f17d9b9011e1a13ed00ad121 (patch)
tree7eed9fa49be850251df9b8c6c2a632244854f02d
parent3b3cf885ca4101b679ff31fd9dc853424db062f4 (diff)
downloadbugs-e749f7333f0e2844f17d9b9011e1a13ed00ad121.tar
bugs-e749f7333f0e2844f17d9b9011e1a13ed00ad121.tar.gz
bugs-e749f7333f0e2844f17d9b9011e1a13ed00ad121.tar.bz2
bugs-e749f7333f0e2844f17d9b9011e1a13ed00ad121.tar.xz
bugs-e749f7333f0e2844f17d9b9011e1a13ed00ad121.zip
Bug 278650 : editversions.cgi still uses "auth_cant_edit_versions"
Patch by Frederic Buclin <LpSolit@gmail.com> r=vladd a=justdave
-rwxr-xr-xeditversions.cgi9
-rw-r--r--template/en/default/global/user-error.html.tmpl5
2 files changed, 4 insertions, 10 deletions
diff --git a/editversions.cgi b/editversions.cgi
index 613f2468a..e4cf47e2a 100755
--- a/editversions.cgi
+++ b/editversions.cgi
@@ -115,11 +115,10 @@ Bugzilla->login(LOGIN_REQUIRED);
print Bugzilla->cgi->header();
-unless (UserInGroup("editcomponents")) {
- ThrowUserError('auth_cant_edit_versions');
- exit;
-}
-
+UserInGroup("editcomponents")
+ || ThrowUserError("auth_failure", {group => "editcomponents",
+ action => "edit",
+ object => "versions"});
#
# often used variables
diff --git a/template/en/default/global/user-error.html.tmpl b/template/en/default/global/user-error.html.tmpl
index 5ea789a2f..e849e7827 100644
--- a/template/en/default/global/user-error.html.tmpl
+++ b/template/en/default/global/user-error.html.tmpl
@@ -218,11 +218,6 @@
Sorry, but you can not delete the default classification,
'[% name FILTER html %]'.
- [% ELSIF error == "auth_cant_edit_versions" %]
- [% title = "Access Denied" %]
- Sorry, you aren't a member of the 'editcomponents' group, and so
- you aren't allowed to add, modify or delete versions.
-
[% ELSIF error == "component_already_exists" %]
[% title = "Component Already Exists" %]
A component with the name '[% name FILTER html %]' already exists.