Feeling Lucky
?
[#12019] - Remove duplicate enqueueMessages
- Closed
- 5 Oct 2016
- Medium
- Build: 3.7.x
- # 12019
- Diff
- C-Lodder:message
- Success continuous-integration/drone the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Pending JTracker/HumanTestResults Human Test Results: 1 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #12017
Summary of Changes
The conditional statements determining which message should be enqueued were incorrect and thus displaying multiple. In addition to that, the link to the plugin pages was resulting in a bad gateway (administrator/%s) because no link was being pushed through sprintf, which this PR also fixes.
Testing Instructions
Apply the patch.
- com_redirect
- Only 1 message telling you the status of the Redirect plugin will be shown.
- The "Redirect System Plugin" link will work as expected.
- com_finder
- Only 1 message telling you the status of the Smart Search plugin will be shown.
- The "Smart Search Content Plugin" link will work as expected.
- com_search
- Only 1 message telling you the status of the Smart Search plugin will be shown.
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Administration Components |
| Title |
|
||||||
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-10-05 19:36:07 |
| Closed_By | ⇒ | zero-24 |
I have tested this item✅ successfully on
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/12019.