As part of Acquia's maintenance of COD and to include better integration with Acquia Search, Insight and other services.

Comments

ezra-g’s picture

Status: Active » Needs work
StatusFileSize
new2.1 KB

This checkbox needs to "actually do something" (tm).

ezra-g’s picture

Next step is to make the checkbox function and add apchesolr_search (dependency of acquia search) and Acquia Connector modules to the .make file.

greggles’s picture

Status: Needs work » Needs review
StatusFileSize
new5.05 KB

Which part of this relies on solr?

I wasn't sure so I left that out.

Only changes
* the += style array merging wasn't working, so I changed it to be a real array_merge
* tried to make the .make files more similar and added acquia_connector to them

Seems close.

ezra-g’s picture

Assigned: Unassigned » ezra-g
StatusFileSize
new5.16 KB

Thanks.

Which part of this relies on solr?

I wasn't sure so I left that out.

I forgot that the D6 version of Acquia search module is a separate project -http://drupal.org/project/acquia_search. In the D7 version of Acquia_connector this is integrated into the Acquia network connector.

In my testing the #3 enables Acquia connector whether or not the checkbox is checked.

This patch adds acquia_search and apachesolr modules and attempts to correct the checkbox bug. I plan to test shortly.

Also, we should be updating drupal-org.make. I'll look into whether we can eliminate cod.make so we don't have to maintain duplicate make files and we can avoid forgetting to make changes in both places.

ezra-g’s picture

Status: Needs review » Fixed

I tested #4 by installing with and without the Acquia connector. http://drupal.org/commitlog/commit/3734/4ce4ebaa30c7e4108b4b416283fbd6a4...

This is committed - Thanks!

Status: Fixed » Closed (fixed)

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