?
avatar virgilbugnariu
virgilbugnariu
27 Apr 2016

Steps to reproduce the issue

Updating permissions from the ACL permissions form will improperly update the permission saved in the assets table. This only happens when the action name value contains an "_" (underscore). For example:

<action name="avchat3.allow_user_interface" [...]

Expected result

In the assets table in the rules column, it should be saved as

{"avchat3.allow_user_interface":{"1":0,"2":1} [...]

Actual result

However, the action name seems to stop at the first encountered "_", additionally adding the second word (after the underline) as the user role ID, so the rules cell looks like:

{"avchat3.allow":{"user":1 [...]

System information (as much as possible)

Fresh Joomla 3.5.1 installation running on Apache, PHP 5.6

Additional comments

I can confirm that on Joomla! 3.4.8 everything worked as expected.

avatar virgilbugnariu virgilbugnariu - open - 27 Apr 2016
avatar brianteeman
brianteeman - comment - 27 Apr 2016

Is this resolved by #9648 that has already been merged into staging?

On 27 April 2016 at 13:51, Virgil B. notifications@github.com wrote:

Steps to reproduce the issue

Updating permissions from the ACL permissions form will improperly update
the permission saved in the assets table. This only happens when the
action name value contains an "_" (underscore). For example:

<action name="avchat3.allow_user_interface" [...]

Expected result

In the assets table in the rules column, it should be saved as

{"avchat3.allow_user_interface":{"1":0,"2":1} [...]

Actual result

However, the action name seems to stop at the first encountered "_",
additionally adding the second word (after the underline) as the user role
ID, so the rules cell looks like:

{"avchat3.allow":{"user":1 [...]

System information (as much as possible)

Fresh Joomla 3.5.1 installation running on Apache, PHP 5.6
Additional comments

I can confirm that on Joomla! 3.4.8 everything worked as expected.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#10107

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar brianteeman brianteeman - change - 27 Apr 2016
Status New Information Required
avatar virgilbugnariu
virgilbugnariu - comment - 27 Apr 2016

Hi Brian,

Sorry, I must have missed the issue posted on GitHub.
I updated the permissions.js and permissions.min.js files and the permissions are saved properly.

You can close the issue.
Thank you for your help.


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

avatar brianteeman
brianteeman - comment - 27 Apr 2016

Closed

avatar brianteeman brianteeman - change - 27 Apr 2016
Status Information Required Closed
Closed_Date 0000-00-00 00:00:00 2016-04-27 13:28:39
Closed_By brianteeman
avatar brianteeman brianteeman - close - 27 Apr 2016

Add a Comment

Login with GitHub to post a comment