Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#12796] - [ACL] [com_banners banners view] Remove invalid check

? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
6 Nov 2016

Summary of Changes

There is an ACL check $user->authorise('core.edit.state', 'com_banners.category') in this view.
This is an invalid ACL check. For ACL check categories we need to check with the category id (example: 'com_banners.category.99'.)

So this PR removes this ACL check, since it is doing nothing and is bad code.

Testing Instructions

Mainly code review, but you can test if banners view continues to work.
1. Apply patch
2. Banners view still work.

Documentation Changes Required

None.

avatar andrepereiradasilva andrepereiradasilva - open - 6 Nov 2016
avatar andrepereiradasilva andrepereiradasilva - change - 6 Nov 2016
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Nov 2016
Category ⇒ Administration Components Templates (admin)
avatar andrepereiradasilva andrepereiradasilva - change - 7 Nov 2016
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - change - 7 Nov 2016
Title
…
[com_banners banners view] Remove invalid ACL check
[ACL] [com_banners banners view] Remove invalid check
avatar andrepereiradasilva andrepereiradasilva - change - 7 Nov 2016
Title
…
[com_banners banners view] Remove invalid ACL check
[ACL] [com_banners banners view] Remove invalid check
avatar andrepereiradasilva andrepereiradasilva - edited - 7 Nov 2016
avatar andrepereiradasilva
andrepereiradasilva - comment - 11 Nov 2016

reverted change as discussed in #12799

now the only change is corrected the asset name from com_banners.category to com_banners.

avatar andrepereiradasilva
andrepereiradasilva - comment - 11 Nov 2016

closed will be replace by one more integrated PR

avatar andrepereiradasilva andrepereiradasilva - change - 11 Nov 2016
Status Pending ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2016-11-11 15:21:31
Closed_By ⇒ andrepereiradasilva
avatar andrepereiradasilva andrepereiradasilva - close - 11 Nov 2016
avatar joomla-cms-bot joomla-cms-bot - change - 11 Nov 2016
Category Administration Components Templates (admin) ⇒ Administration com_banners Templates (admin) Components

Add a Comment

Login with GitHub to post a comment