It would be great to be able to wrap ApacheSolr's servers and settings up in a features module.

CommentFileSizeAuthor
#13 1134610-features_exportables_ctools-13.patch1.3 KBpcambra
#9 ctools_export_1134610_5.patch666 bytesRavi.J
#7 ctools_export_1134610_4.patch674 bytesRavi.J
#3 ctools-export-1134610-3.patch1003 bytesJoeMcGuire
#1 ctools-export-1134610.patch665 bytesJoeMcGuire

Comments

JoeMcGuire’s picture

JoeMcGuire commented
Status: Active » Needs work
StatusFileSize
new ctools-export-1134610.patch665 bytes

Patch attached to make the apachesolr_server table available for export.
The apachesolr_server_variable table has multiple primary keys which isn't supported by ctools yet. See #924236

This patch is for 7.x-1.0-beta5. I see 7.x-1.x has different table names.

pwolanin’s picture

pwolanin commented
Version: 7.x-1.0-beta5 » 7.x-1.x-dev

The dev code is changed a bit, including renaming that table.

Can you point to a reference for the exportables info?

Also, the variables table is essentially linked to this, so I'd prefer to export/import the 2 together.

JoeMcGuire’s picture

JoeMcGuire commented
StatusFileSize
new ctools-export-1134610-3.patch1003 bytes

I found the ctools documentation is in its module by using the advanced help module.

Attached is an amended patch because I wasn't having any luck having the previous one working with drush make.

pwolanin’s picture

pwolanin commented

Still needs work - you are patching an outdated codebase.

JoeMcGuire’s picture

JoeMcGuire commented

Let us know when you finished tweaking the table names and schemas.
I've just started with a patch for the latest tag as I wanted it to work with Drush Make.

pwolanin’s picture

pwolanin commented

There won't be a new tag until there is a new release.

If you want to develop, you'll need to use git to pull the latest code.

Ravi.J’s picture

Ravi.J commented
StatusFileSize
new ctools_export_1134610_4.patch674 bytes

Resubmitting patch for 7.x-dev

JoeMcGuire’s picture

JoeMcGuire commented

Hi Ravi,

Thanks - I don't think my initial patch was enough to reimport the exported data. I think you need to look into a loader method mentioned in the Advanced Help documentation.

We've moved out projects over to the Search API which does support features and exportables.
http://drupal.org/project/search_api

Ravi.J’s picture

Ravi.J commented
StatusFileSize
new ctools_export_1134610_5.patch666 bytes

Redid the patch, could not get previous patch work with Drush Make

JoeMcGuire’s picture

JoeMcGuire commented

I was having that issue yesterday it's to do with different strip -p0 -p1 options see #745224: Apply patches from git diff and git format-patch

alexander allen’s picture

alexander allen commented

Subscribing, interested as well. It would definitely be nice to simultaneously propagate settings such as apachesolr_enabled_facets to multiple dev and prod servers via Features and features revert. As soon as I investigate how to implement ctool exportables I should be able to contribute some code for exporting apachesolr_enabled_facets.

alexander allen’s picture

alexander allen commented
Status: Needs work » Active

Regarding facets, the latest version of the Facet API module is allowing facet settings to be exported via Features.

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Active » Needs review
StatusFileSize
new 1134610-features_exportables_ctools-13.patch1.3 KB

Added also enviroment variable table and rolled against latest -dev

Edit: hadn't read #1 variables table is not supported by ctools, forget about this patch.

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented

comment #9 patch is still valid. Can anyone confirm this is working well for them? pcambra?
This way we can change this status and commit it

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented

I can confirm #9 is working for enviroment table, but I had to implement my own hook_install for the enviroment variables table.

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Needs review » Reviewed & tested by the community
pwolanin’s picture

pwolanin commented
Status: Reviewed & tested by the community » Fixed

committed, thanks.

Obviously we need a follow-up for import (needs to be optionally available when ctools is enabled).

Status: Fixed » Closed (fixed)

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

andymantell’s picture

andymantell commented

Has this functionality been finished? I exported my solr settings but they don't seem to be getting pulled in again... I assume that is the follow up that pwolanin was referring to? Is there a ticket already?

Bußmeyer’s picture

Bußmeyer commented

No, it hasn't been finished. Here you can find the new issue for the import task: #1357588: follow-up for import (needs to be optionally available when ctools is enabled)