Feeling Lucky
?
?
[#22697] - Use new user instance
- Fixed in Code Base
- 20 Jul 2019
- Medium
- Build: staging
- # 22697
- Diff
- SharkyKZ:plgSystemActionlogs_2
- Success continuous-integration/appveyor/pr AppVeyor build succeeded Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
Create new user instance before clearing access rights cache so as not to clear it globally.
Testing Instructions
Code review.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
franz-wohlkoenig
- | Title |
|
||||||
| Title |
|
||||||
alikon
- comment
- 2 Jul 2019
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22697.
| Status | Pending | ⇒ | Ready to Commit |
alikon
- comment
- 2 Jul 2019
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22697.
SharkyKZ
- comment
- 3 Jul 2019
It was a mistake on my part. We need to check new user access rights in this plugin but we shouldn't clear them on the global user instance as this could lead to undesirable behavior in code and other plugins that run after this one.
HLeithner
- comment
- 3 Jul 2019
ok thx
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-07-20 17:17:20 |
| Closed_By | ⇒ | HLeithner | |
| Labels |
Added:
?
|
||
HLeithner
- comment
- 20 Jul 2019
thx
I have tested this item✅ successfully on 56f7bc0
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22697.