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
features_integration-1213898-4.patch1.37 KBfrippuz

Comments

restyler’s picture

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 TN, USA
commented

Subscribe.

czigor’s picture

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

czigor’s picture

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

kalabro’s picture

kalabro
she/her
commented
Status: Fixed » Needs work

So maybe status is not fixed?

bforchhammer’s picture

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 commented
Status: Needs work » Needs review
StatusFileSize
new 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 commented

Works for me so far. Thanks.

czigor’s picture

czigor commented
Status: Needs review » Fixed

Committed, thanks!

bforchhammer’s picture

bforchhammer commented
Status: Fixed » Needs work

Great, thank you!

bforchhammer’s picture

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 🇨🇦
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 🇨🇦
commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new 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 🇨🇦
commented
StatusFileSize
new cnr-backport_exportables_patch-1213898-18.patch6.41 KB

Fixed whitespace error