Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
What was done :
- Blocks were removed that can now be added using facetapi-6.x-3.x and apachesolr_sort
- Some legacy code has been removed and commented out. We should revisit those
- module is using proper apachesolr-6.x-3.x apis
- path was changed from /search/apachesolr_search to /search/site
new branch of apachesolr_sort was created so the legacy code in this module could disappear.
Check out the branch apachesolr_sort-6.x-3.x
Comment | File | Size | Author |
---|---|---|---|
#14 | 1684952-14.patch | 21.75 KB | Nick_vh |
#10 | 1684952-10.patch | 21.56 KB | Nick_vh |
#9 | 1684952-9.patch | 8.29 KB | Nick_vh |
#8 | 1684952-7.patch | 7.38 KB | Nick_vh |
#7 | 1684952-7.patch | 176 bytes | Nick_vh |
Comments
Comment #1
Nick_vhComment #2
Nick_vhComment #3
drummDoes anything handle redirecting
/search/apachesolr_search
to/search/site
?I'm guessing the new blocks have to be themed to match Drupal.org?
Comment #4
Nick_vhthere's no automatic redirect from apachesolr_search to search/site but that might be something a redirect module can handle?
What would be a sensible approach here?
Comment #5
drummSince this is a wildcard redirect, I don't think path_redirect can do it. I'm not eager to install a new module just for this. We can either do a tiny menu callback or put it in our .htaccess. I'd prefer the menu callback since PHP is nicer than Apache config.
Comment #6
Nick_vhI'm perfectly ok with a menu callback. Just need to be sure that the http code is a redirect code, so we don't get penalized by crawlers. But perhaps the search paths are already excluded in robots.txt?
Comment #7
Nick_vhJust posting the latest diff that is currently there.
Comment #8
Nick_vhComment #9
Nick_vhCould someone already commit this patch. We'll do all the following issues in new issue nodes such as #1792104: Handle redirecting /search/apachesolr_search to /search/site
This is to prevent that we end up in monolithic issues and no structure is left
Comment #10
Nick_vhSmall update. This goes hand in hand with #1549496-14: Upgrade project to support 6.x-3.x version of Apachesolr (prep for d.o. upgrade to D7)
Comment #11
Nick_vhdon't use tid here
Comment #12
drummThe deployment issue is #1793968: Deploy Apache Solr Search Integration 6.x-3.x.
Comment #13
drummI went ahead and committed #10 to 1548064-support-new-apachesolr. Leaving this open for #11.
Comment #14
Nick_vhAdding defaults and update/upgrade script
Comment #15
drummComment #16
drummCommitted #14 to the branch with some cleanup:
Still leaving open for #11.
Comment #17
drummtaking this
Comment #18
drummhttp://drupalcode.org/project/drupalorg.git/commitdiff/d254bbc?hp=68603f...
Comment #19.0
(not verified) CreditAttribution: commentedAdding info about apachesolr_sort