[#12763] - Fix updatenotificaltion plugin
- Fixed in Code Base
- 20 Nov 2016
- Medium
- Build: staging
- # 12763
- Diff
- zero-24:updatenotification
- Success continuous-integration/drone the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #12614
Summary of Changes
The method to get the super users have a problem in getting the super user groups
Testing Instructions
place this code
$rootId = JTable::getInstance('Asset', 'JTable')->getRootId();
$rules = JAccess::getAssetRules($rootId)->getData();
$rawGroups = $rules['core.admin'];
$rawGroups_new = $rules['core.admin']->getData();
$rawGroups_old = $rules['core.admin'];
echo 'Old Result:<br>';
print_r($rawGroups_old);
echo '<br>New result:<br>';
print_r($rawGroups_new);
In the index.php of your template (you can use the template manager for that)
Result:
Documentation Changes Required
None it is a PHP Problem
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
| Title |
|
||||||
| Labels |
Added:
?
|
||||||
| Title |
|
||||||
| Milestone |
Added: |
||
Correct as the patch fix the code in the update plugin which i can't find a way to test. There fore i have created the snipped which shows the old / wrong value and the new value
Confused here too:
I get the same result before and after patch.
Is there something missing in your test instructions?
Please look at the code ;)
It should just show that the new code produce the correct result. I have not found a way to exactly test the method in the plugin. So i have added here the code that shows the result before and after. Applying the patch does not change anything as this is just the patch to have the new value in the plugin method.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12763.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12763.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12763.
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-11-20 09:46:11 |
| Closed_By | ⇒ | rdeutz |
@zero-24 you say to place some code in the template. Is that result supposed to be before or after the PR or are you saying that it should be the same before AND after the PR
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12763.