Feeling Lucky
?
[#13358] - Load system plugins in Smart Search indexer
- Fixed in Code Base
- 27 Dec 2016
- Medium
- Build: staging
- # 13358
- Diff
- chrisdavenport:ss-cli-compat
- 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:
As noted in #13338 the Smart Search CLI indexer does not load the system plugins so event handlers not in "finder" plugins will not be called. This PR loads the system plugins as well as the finder plugins.
Summary of Changes
Loads system plugins. Also defines the JDEBUG constant which was causing some spurious warning messages.
Testing Instructions
Testing would require creating a system plugin containing a finder event handler. I doubt anyone will go to that much trouble so I suggest this can be simply merged on review.
Documentation Changes Required
None.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | CLI |
bembelimen
- comment
- 23 Dec 2016
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13358.
| Status | Pending | ⇒ | Ready to Commit |
jeckodevelopment
- comment
- 24 Dec 2016
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13358.
| Milestone |
Added: |
||
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-12-27 21:51:20 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
|
||
I have tested this item✅ successfully on 7e0794b
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13358.