-
-
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
Smart Search - Corrects the ordering of loaded JS files ... #5336
Smart Search - Corrects the ordering of loaded JS files ... #5336
Conversation
…ding of Mootools (loaded automatically by progressbar.js call)
@Kubik-Rubik May I suggest to move all the calls to tmpl/default.php. I think this is the proper place to call any needed scripts |
@dgt41 Let us first optimize the loading ordering and in a second step move them in the template files because it has to be done in many places (components). Agree? |
@Kubik-Rubik but this way is kinda double work, I guess? Anyways if this is the way you think is gonna be easier to test/merge no real objection here. Test results coming soon... |
@dgt41 Yeah, but I don't want to mix the PRs. Thank you for testing! :-) |
@test success |
Guys, this MUST be merged: without this com_finder indexing is broken with the current staging code... |
@smanzi @Kubik-Rubik The same goes for #5099 |
@dgt41 I will test your PR #5099 today and give feedback! We have 2 successful tests -> RTC |
Smart Search - Corrects the ordering of loaded JS files ...
... and removes the explicit loading of Mootools (loaded automatically by progressbar.js call)
Jquery has to be loaded before indexer.js and Mootools doesn't have to be loaded explicitly because it is loaded automatically. (We should remove the dependency of Mootools in progressbar.js in another PR)
How to test?
Apply this PR and run the indexer in the finder component (Smart Search). Everything should run properly and the items indexed properly.