? ? Pending

User tests: Successful: Unsuccessful:

avatar Anu1601CS
Anu1601CS
11 Mar 2018

Pull Request for Issue #19427

Summary of Changes

  1. Move the code to the custom element
  2. Fix the icon position

Testing Instructions

  1. Go to Admin Global Configuration and test the permissions
  2. Change some of the permissions

Expected result

Working as before

Actual result

Working as before

Documentation Changes Required

No

@dgt41 @C-Lodder PR closed in joomla-es6 repo. reopen in main repo.

e10946f 28 Feb 2018 avatar Anu1601CS CS
avatar Anu1601CS Anu1601CS - open - 11 Mar 2018
avatar Anu1601CS Anu1601CS - change - 11 Mar 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Mar 2018
Category Administration com_config JavaScript Repository Layout Unit Tests
avatar Anu1601CS Anu1601CS - change - 11 Mar 2018
Labels Added: ? ?
avatar Anu1601CS
Anu1601CS - comment - 16 Mar 2018

@astridx can you test this?
Thanks

avatar Anu1601CS
Anu1601CS - comment - 19 Mar 2018

@franz-wohlkoenig can you test this ?

avatar astridx
astridx - comment - 19 Mar 2018

I have tested this item successfully on 88b6319

I applied this path and saw, that the permissions are now in the webcomponent <joomla-field-permissions ..>

I changed some permissions in global configuration. The changed permissions are saved and the inheritance to components works fine. For example, in the global configuration, I set the create permission to denied for the author. In the components com_content I see now that the create = allow is no longer inherited. And I was not able to apply the permisson.

I have created a new user group under registered. The permission have been set correctly (like registered) in the webcomponent <joomla-field-permissions ..>.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19889.
avatar astridx astridx - test_item - 19 Mar 2018 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 19 Mar 2018

@Anu1601CS i didn't test anymore as my lack of Knowledge generates the Risk of false positive Results.

avatar Anu1601CS
Anu1601CS - comment - 19 Mar 2018
avatar astridx
astridx - comment - 19 Mar 2018

@franz-wohlkoenig I do not understand this. I think it's good to have a lot of tests and each test brings something possitive.
It is important that we write down what we have tested, so those who are merge can judge this.

avatar C-Lodder
C-Lodder - comment - 19 Mar 2018

@franz-wohlkoenig

  1. Go to permissions view.
  2. Select an option from the dropdown
  3. A loading/spinner icon should appear whilst the request is being executed
  4. The label/badge should change to red/green depending on the value selected
  5. Refresh the page and ensure the value is the same just to make sure it was properly saved (sanity check)
avatar Anu1601CS
Anu1601CS - comment - 20 Mar 2018

Dropped some extra files. Please test again.
Thanks

avatar Anu1601CS
Anu1601CS - comment - 1 Apr 2018

@astridx @franz-wohlkoenig Can you please test again?
Thanks

avatar laoneo
laoneo - comment - 3 Apr 2018

@dgt41 can you give here the final go to get it merged.

avatar dgt41
dgt41 - comment - 3 Apr 2018

@laoneo go for it

avatar Anu1601CS
Anu1601CS - comment - 3 Apr 2018

@laoneo Done!

avatar laoneo laoneo - change - 3 Apr 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-04-03 12:58:06
Closed_By laoneo
avatar laoneo laoneo - close - 3 Apr 2018
avatar laoneo laoneo - merge - 3 Apr 2018
avatar laoneo
laoneo - comment - 3 Apr 2018

Thanks. Good work.

avatar Anu1601CS
Anu1601CS - comment - 3 Apr 2018

@laoneo Your welcome.

Add a Comment

Login with GitHub to post a comment