? ? Pending

User tests: Successful: Unsuccessful:

avatar Quy
Quy
16 Feb 2018

Pull Request for Issue #19478.

Summary of Changes

Only super users have access to Global Configuration and System Information. This PR hides these 2 menu items from administrators.

Use identical operators for minor performance improvement.

Testing Instructions

Log in as an Administrator.
View System menu.

Expected result

No Global Configuration and System Information

Actual result

Displays Global Configuration and System Information

Documentation Changes Required

none

avatar Quy Quy - open - 16 Feb 2018
avatar Quy Quy - change - 16 Feb 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 Feb 2018
Category Modules Administration
avatar dautrich
dautrich - comment - 16 Feb 2018

I have tested this item successfully on 506a66c

Logged in as Administrator: Menu items Global Configuration and System Information not shown

Logged in as Super User: Menu items Global Configuration and System Information shown


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19697.

avatar dautrich dautrich - test_item - 16 Feb 2018 - Tested successfully
avatar Quy Quy - change - 16 Feb 2018
Labels Added: ?
avatar Quy Quy - change - 16 Feb 2018
The description was changed
avatar Quy Quy - edited - 16 Feb 2018
avatar alikon
alikon - comment - 17 Feb 2018

I have tested this item successfully on dde0967


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19697.

avatar alikon alikon - test_item - 17 Feb 2018 - Tested successfully
avatar alikon
alikon - comment - 17 Feb 2018

however i've noticed a small inconsistence (with or without this pr)
as an administrator
on: system information http://localhost/test/administrator/index.php?option=com_admin&view=sysinfo you got
screenshot from 2018-02-17 09-51-29

throw new JAccessExceptionNotallowed(JText::_('JERROR_ALERTNOAUTHOR'), 403);

on: configuration http://localhost/test/administrator/index.php?option=com_config

screenshot from 2018-02-17 09-55-28

$this->app->enqueueMessage(JText::_('JERROR_ALERTNOAUTHOR'));

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Feb 2018

@alikon can you please open a new Issue for the small inconsistence you reported above?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Feb 2018

@dautrich can you please retest?

avatar dautrich
dautrich - comment - 17 Feb 2018

I have tested this item successfully on dde0967


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19697.

avatar dautrich dautrich - test_item - 17 Feb 2018 - Tested successfully
avatar dautrich
dautrich - comment - 17 Feb 2018

Done


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19697.

avatar Quy Quy - change - 17 Feb 2018
Status Pending Ready to Commit
avatar Quy
Quy - comment - 17 Feb 2018

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19697.

avatar mbabker mbabker - change - 26 Feb 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-02-26 03:13:00
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 26 Feb 2018
avatar mbabker mbabker - merge - 26 Feb 2018

Add a Comment

Login with GitHub to post a comment