-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[3.10] - make com_search work on J4 #35095
Conversation
I thought com_search was removed ? |
it is removed from new installation , but if you upgrade from a j3 site ..... |
@@ -94,7 +94,7 @@ public function display($tpl = null) | |||
$lists['searchphrase'] = JHtml::_('select.radiolist', $searchphrases, 'searchphrase', '', 'value', 'text', $state->get('match')); | |||
|
|||
// Log the search | |||
\Joomla\CMS\Helper\SearchHelper::logSearch($searchWord, 'com_search'); | |||
SearchHelper::oldlogSearch($searchWord, 'com_search'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like a bad change - your not importing the namespace either. so this will be a invalid class.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
probably you forgot this remove
https://github.com/joomla/joomla-cms/blob/4.0-dev/administrator/components/com_admin/script.php#L3857
The intention was that people should update to the decoupled com_search here https://github.com/joomla-extensions/search/tree/4.0-dev similar to how we did weblinks. |
Yes the only thing we would to do in an ideal world would be to add the update server into 3.10 and make sure it is detected as 3rd party extension at the time of the upgrade checks. |
or even better add the update xml into 4.0 as the upgrade works and the new version will be 4.x only anyway. That would mean 3.10 -> 4.0 -> com_search update like all other 3rd party extensions that are no longer 3.x compatible would go, |
@wilsonge We did weblinks? You mean it is ready for J4? If so then I missed something. To me the weblinks repository seems to be abandoned. No PR gets merged there, CI checks broken since months. |
Hmm when the code provided here makes com_search work I would say the correct way forward would be to do a PR against: https://github.com/joomla-extensions/search/tree/4.0-dev/src Once merged there do a release and add the new update server on updating from 3.10. So after updating to 4.0 the update shows up and the package can be used with the package that works with 4.0 |
that's easy, simply test it
there is no need to do this pr there, it's a complete new rewrite in j4 style already this pr just give us the time to do:
things that hardly can be done until 17 |
Hi, this PR is the base for the minimal version of search running on 4.0 that we have called and published as 4.0.0-dev here: https://github.com/joomla-extensions/search/releases/tag/4.0.0-dev to be used until a more final package of search has been build. Thanks @alikon for providing this PR 👍 |
Summary of Changes
make com_search work on J4
Testing Instructions
apply pr
upgrade to j4
Actual result BEFORE applying this Pull Request
com_search doesn't work
Expected result AFTER applying this Pull Request
com_search works in j4
Documentation Changes Required
?