[#8651] - storePermissions should use existing API - Fixes #8650
- Closed
- 25 Jun 2016
- Medium
- Build: staging
- # 8651
- Diff
- klas:#80650
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Problem is described in #8650 .
As this is pure code improvement there isn't much to test - check behaviour of permissions tab before and after the patch, they should stay the same.
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
?
|
||
Hello @klas
Thank you for your contribution.
P;ease provide clear test instructions to be able to test / reproduce this issue.
If no reply is received within 4 weeks we will close this issue.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8651.
wilsonge
- | Labels |
Removed:
?
|
||
brianteeman
- | Rel_Number | 0 | ⇒ | 8650 |
| Relation Type | ⇒ | Pull Request for |
| Status | Pending | ⇒ | Information Required |
| Status | Information Required | ⇒ | Pending |
| Category | ⇒ | ACL |
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-06-25 11:01:09 |
| Closed_By | ⇒ | roland-d |
@klas I am going to close this PR since there has been no more activity and there have been a number of other PRs in regards to ACL fixes. Feel free to re-open this if you want to work further on it. Thank you for your contribution.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8651.
Sorry, but no wish to work further on something that is not going to be merged, it was a simple code change to get this part of code in line with existing API, code like it is should not be allowed in by release manager or whoever is in charge of code quality for new features.
Please add more information to your issue. Without test instructions and/or any description we will close this issue within 4 weeks. Thanks.
This is an automated message from the J!Tracker Application.