Needs review
Project:
Dynamic dependent fields
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
20 Feb 2014 at 08:46 UTC
Updated:
12 Apr 2017 at 13:38 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
maximpodorov commentedWhat did you mean?
Comment #2
Konstantin Komelin commentedHi Max,
I'm experiencing the same issue. It seems like ddf settings are not exportable through Features module.
Do you have plans to implement Features module support?
Thanks,
Konstantin
Comment #3
maximpodorov commentedThey are normal field settings. Why won't they be exported?
Comment #4
maximpodorov commentedOh, no, I forget about the additional settings stored in the DB. Could someone provide the patch for this?
Comment #5
Konstantin Komelin commentedHi Max,
Please give us an idea which setting exactly we need to save additionally.
Thanks,
Konstantin
Comment #6
Konstantin Komelin commentedOh, I've found it, the ddf table contains necessary settings.
Comment #7
Konstantin Komelin commentedThe following code will add necessary setting to the database:
I've added this code to my hook_update_N as a workaround.
Comment #8
yseki commentedHi,
Here is one patch me make DDF features exportable.
Comment #9
yseki commentedComment #11
yseki commentedPath for retest
Comment #12
yseki commentedComment #14
yseki commentedOh!
Different folder.
Comment #16
Konstantin Komelin commented@yuriseki
Advanced patch contributor guide may help.
Also, make sure the number (N) in hook_update_N is correct.
Comment #17
yseki commentedThanks Konstantin,
I followed the guide, hope it works this time :-)
Comment #18
azinck commentedThanks for the patch. Just a couple of little issues with #17
Fixed patch attached.
Comment #19
mortona2k commentedOn a clean install with a ddf field in a feature, the dynamic field doesn't update properly. Saving the field settings fixes it. It seems that the feature revert isn't saving the settings to the db? It could also be that it uses a field id to target ddf, which can be different depending on when features are enabled.
Comment #20
azinck commentedAh, good catch mortona2k...I didn't look at this carefully. Needed to refactor ddf to use a consistent load function and use Ctools to load our exportables. Patch attached.
Comment #22
azinck commentedComment #24
azinck commentedComment #25
maximpodorov commentedMaybe it's better to get rid of a dedicated DB table and just store the ddf settings in field instance settings.
Comment #26
Konstantin Komelin commentedMaxim, in that case we'd need an upgrade mechanism.
Comment #27
Leiph commentedI agree with #25. Storing the settings in field instance settings will make the export more robust.
/Leiph