Port Core searches to 6.x

yngens - June 17, 2008 - 00:43
Project:Core searches
Version:5.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

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?

#1

robertDouglass - June 17, 2008 - 13:02

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.

#2

aufumy - July 10, 2008 - 00:49
Status:active» patch (code needs review)

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

AttachmentSize
coresearches_drupal6.patch6.96 KB
DRUPAL-6-3.patch19.74 KB

#3

aufumy - July 11, 2008 - 21:25

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()

AttachmentSize
coresearches_drupal6_jul11.patch19.97 KB

#4

robertDouglass - July 12, 2008 - 01:36
Status:patch (code needs review)» fixed

Thanks. Committed.

#5

robertDouglass - July 12, 2008 - 01:37

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

#6

aufumy - July 23, 2008 - 17:52

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().

AttachmentSize
drupal6_jul23.patch9.97 KB

#7

aufumy - July 23, 2008 - 17:52
Status:fixed» patch (code needs review)

#8

xiong - July 31, 2008 - 02:58
Status:patch (code needs review)» patch (reviewed & tested by the community)

tested the july23 patch, it works fine.

#9

David Lesieur - August 24, 2008 - 04:26
Status:patch (reviewed & tested by the community)» fixed

Looks good to me too. Committed. Thanks!

temp

Anonymous (not verified) - September 7, 2008 - 04:36

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

temp

Anonymous (not verified) - September 7, 2008 - 04:43

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

temp

Anonymous (not verified) - September 7, 2008 - 04:54

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

temp

Anonymous (not verified) - September 7, 2008 - 05:04

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

temp

Anonymous (not verified) - September 7, 2008 - 05:14

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

temp

Anonymous (not verified) - September 7, 2008 - 05:24

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

temp

Anonymous (not verified) - September 7, 2008 - 05:35

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

temp

Anonymous (not verified) - September 7, 2008 - 05:47

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

temp

Anonymous (not verified) - September 7, 2008 - 05:54

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

temp

Anonymous (not verified) - September 7, 2008 - 06:03

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

#10

Anonymous (not verified) - September 7, 2008 - 06:33
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.