Bug 114172 - add link to /ccm/admin/ on user admin tab
Summary: add link to /ccm/admin/ on user admin tab
Alias: None
Product: Red Hat Enterprise CMS
Classification: Retired
Component: ui   
(Show other bugs)
Version: nightly
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Vadim Nasardinov
QA Contact: Jon Orris
Depends On:
Blocks: 106481
TreeView+ depends on / blocked
Reported: 2004-01-23 16:37 UTC by Richard Li
Modified: 2007-04-18 17:02 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2004-01-28 19:23:47 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

Description Richard Li 2004-01-23 16:37:22 UTC
The user admin tab currently only enables the admin to set the user's
home folder. The name of the tab implies greater functionality.

We can partially meet this goal by adding a link to /ccm/admin/ so
that all the user admin functionality is at least linked from a single

Comment 1 Vadim Nasardinov 2004-01-26 22:51:18 UTC
The user admin tab is displayed conditionally based on whether or not
the logged in user has the SecurityConstants.STAFF_ADMIN privilege on
the current content section.  This is done in generateXML in

The /ccm/admin/ UI is backed by
It checks if the user has the PrivilegeDescriptor.ADMIN privilege on
the current site node.  (The actual check is performed in the

So, these two pages require two different kinds of permission.  Thus,
our options are:

  1. Display the "/ccm/admin/" link on the content section's user
     admin tab regardless of whether the user can access /ccm/admin/
     or not.

  2. Display the link conditionally by checking first if the user will
     be able to actually view /ccm/admin/.  This can be done by
     copying and pasting the permission check from SubsiteDispatcher
     into UserAdminPane.

Which option should I pursue?

Comment 2 Richard Li 2004-01-27 00:02:54 UTC
I think the latter is more user friendly.

Comment 3 Vadim Nasardinov 2004-01-27 22:20:39 UTC
Fixed on the trunk in change 39810.

In the process of working on this, discovered and eliminated
a particularly egregious instance of copy-and-paste programming.
See change 39792.

Comment 4 Jon Orris 2004-01-28 19:23:47 UTC

Note You need to log in before you can comment on or make changes to this bug.