Hi,

i'm using CNR and EVA for a restaurant site.
I set it up so the editor can create a new "menu" node, pick from a list of node referenced "dishes", and so publishing a custom menu without touching the views UI.

I packed it all in a feature, but the problem is that CNR correspondence is not included.

Do you have a solution for this?

Thank you
Simone

CommentFileSizeAuthor
#18 cnr-backport_exportables_patch-1213898-18.patch6.41 KBergonlogic
#17 cnr-backport_exportables_patch-1213898-17.patch6.41 KBergonlogic
#10 features-integration-1213898-10.patch4.55 KBbforchhammer
#4 features_integration-1213898-4.patch1.37 KBFrippuz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

restyler’s picture

restyler CreditAttribution: restyler commented

+1 for feature integration. I think this can easily be achieved by exposing cnr table to ctools exportable http://drupal.org/node/928026

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

Subscribe.

czigor’s picture

czigor CreditAttribution: czigor commented
Status: Active » Fixed

It's in the dev, please test. Might need to disable and enable the module (but to uninstall definetely not).

Frippuz’s picture

Frippuz CreditAttribution: Frippuz commented
FileSize
features_integration-1213898-4.patch1.37 KB

I get the error:

Notice: Undefined index: api in _ctools_export_get_defaults() (line 625 of /srv/www/mysite/web/profiles/myprofile/modules/ctools/includes/export.inc).

Skipping the presets part but to make it work the following is needed:
- a couple of new lines in the install file
- a ctools plugin api hook

Patch is provided.

Exporting doesn't work without these.

czigor’s picture

czigor CreditAttribution: czigor commented
Version: 7.x-4.1 » 7.x-4.x-dev

Sorry, my bad. Thanks for the patch, it's committed.

kalabro’s picture

kalabro
she/her
CreditAttribution: kalabro commented

Unfortunately I can't see how it works. I've created feature with one enabled CNR preset but all what it done is deleting record from CNR table.
Ctools 7.x-1.0-rc1
CNR 7.x-4.x-dev

Frippuz’s picture

Frippuz CreditAttribution: Frippuz commented

The values are exported to the feature. Perhaps there is something wrong with the export making it impossible to import.
I'll better take a look at it one more time then :-)

kalabro’s picture

kalabro
she/her
CreditAttribution: kalabro commented
Status: Fixed » Needs work

So maybe status is not fixed?

bforchhammer’s picture

bforchhammer CreditAttribution: bforchhammer commented

I can confirm that this isn't working properly. CNR settings are exported to the feature fine, but when "reverting" the feature it simply deletes all settings...

The ways ctools exportables work, the "deleting settings from the database" part is probably fine, if the module loads settings from code instead. Maybe that part is missing?

bforchhammer’s picture

bforchhammer CreditAttribution: bforchhammer commented
Status: Needs work » Needs review
FileSize
features-integration-1213898-10.patch4.55 KB

Attached patch moves all configuration-related loading/saving over to ctools. This fixes feature reverting etc. for me.

Matthew Davidson’s picture

Matthew Davidson CreditAttribution: Matthew Davidson commented

Works for me so far. Thanks.

czigor’s picture

czigor CreditAttribution: czigor commented
Status: Needs review » Fixed

Committed, thanks!

bforchhammer’s picture

bforchhammer CreditAttribution: bforchhammer commented
Status: Fixed » Needs work

Great, thank you!

bforchhammer’s picture

bforchhammer CreditAttribution: bforchhammer commented
Status: Needs work » Fixed

Didn't mean to change the status.

Status: Fixed » Closed (fixed)

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

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic commented
Version: 7.x-4.x-dev » 6.x-4.x-dev
Component: Miscellaneous » Code
Status: Closed (fixed) » Patch (to be ported)

It would be great to see features support in the 6.x branch :)

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic commented
Status: Patch (to be ported) » Needs review
FileSize
cnr-backport_exportables_patch-1213898-17.patch6.41 KB

The attached patch works for me.

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic commented
FileSize
cnr-backport_exportables_patch-1213898-18.patch6.41 KB

Fixed whitespace error