Bug 114172 - add link to /ccm/admin/ on user admin tab
Summary: add link to /ccm/admin/ on user admin tab
Status: CLOSED RAWHIDE
Alias: None
Product: Red Hat Enterprise CMS
Classification: Retired
Component: ui   
(Show other bugs)
Version: nightly
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Vadim Nasardinov
QA Contact: Jon Orris
URL:
Whiteboard:
Keywords:
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:
Environment:
Last Closed: 2004-01-28 19:23:47 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
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
tab.

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
//cms/dev/src/com/arsdigita/cms/ui/UserAdminPane.java

The /ccm/admin/ UI is backed by
//core-platform/dev/src/com/arsdigita/ui/admin/AdminDispatcher.java.
It checks if the user has the PrivilegeDescriptor.ADMIN privilege on
the current site node.  (The actual check is performed in the
superclass
//core-platform/dev/src/com/arsdigita/ui/admin/SubsiteDispatcher.java.)

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
verified


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