Apache Solr Search Integration module (http://drupal.org/project/apachesolr) for Drupal 6.x, which I wanted to install on my site, references this module. I wonder if the module's maintainers are planning to port it to 6.x soon?

CommentFileSizeAuthor
#6 drupal6_jul23.patch9.97 KBaufumy
#3 coresearches_drupal6_jul11.patch19.97 KBaufumy
#2 DRUPAL-6-3.patch19.74 KBaufumy
#2 coresearches_drupal6.patch6.96 KBaufumy

Comments

robertdouglass’s picture

robertdouglass commented

The module maintainer welcomes help porting it. As far as I know nobody is working on it yet. If the reader of this is working on the port, or intends to, please let us know.

aufumy’s picture

aufumy commented
Status: Active » Needs review
StatusFileSize
new coresearches_drupal6.patch6.96 KB
new DRUPAL-6-3.patch19.74 KB

Here is an initial cut of the drupal 6 port. It includes the latest cvs changes that Robert has worked on of the DRUPAL-5 branch.

Audrey

aufumy’s picture

aufumy commented
StatusFileSize
new coresearches_drupal6_jul11.patch19.97 KB

Included latest DRUPAL-6-3 changes from
* user.module - user_search()
* node.module - node_update_index()
* node.module - node_form_alter()
* node.module - node_search()

robertdouglass’s picture

robertdouglass commented
Status: Needs review » Fixed

Thanks. Committed.

robertdouglass’s picture

robertdouglass commented

Please test, report back, and if everything is cool I'll make the current dev snapshot into a release.

aufumy’s picture

aufumy commented
StatusFileSize
new drupal6_jul23.patch9.97 KB

Some of the differences between the jul 11th and jul 10th patch are not in the cvs tree.

This is the patch for the remainder, which includes the changes to node_search from revision 1.900 of node.module (issue #146466) as well as the renaming of user_search() in usersearch.module to usersearch_search().

aufumy’s picture

aufumy commented
Status: Fixed » Needs review
xiong’s picture

xiong commented
Status: Needs review » Reviewed & tested by the community

tested the july23 patch, it works fine.

David Lesieur’s picture

David Lesieur commented
Status: Reviewed & tested by the community » Fixed

Looks good to me too. Committed. Thanks!

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.