Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Adding export Features for DDF
Comment | File | Size | Author |
---|---|---|---|
#20 | interdiff-18-20.txt | 5.82 KB | azinck |
#20 | ddf-features_exportable-2201097-20.patch | 9.32 KB | azinck |
#18 | interdiff-17-18.txt | 832 bytes | azinck |
#18 | ddf-features_exportable-2201097-18.patch | 3.46 KB | azinck |
#17 | ddf-features_exportable-2201097-17.patch | 3.42 KB | yseki |
Comments
Comment #1
maximpodorov CreditAttribution: maximpodorov commentedWhat did you mean?
Comment #2
Konstantin Komelin CreditAttribution: Konstantin Komelin as a volunteer 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 CreditAttribution: maximpodorov commentedThey are normal field settings. Why won't they be exported?
Comment #4
maximpodorov CreditAttribution: maximpodorov commentedOh, no, I forget about the additional settings stored in the DB. Could someone provide the patch for this?
Comment #5
Konstantin Komelin CreditAttribution: Konstantin Komelin as a volunteer commentedHi Max,
Please give us an idea which setting exactly we need to save additionally.
Thanks,
Konstantin
Comment #6
Konstantin Komelin CreditAttribution: Konstantin Komelin as a volunteer commentedOh, I've found it, the ddf table contains necessary settings.
Comment #7
Konstantin Komelin CreditAttribution: Konstantin Komelin as a volunteer 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 CreditAttribution: yseki as a volunteer and commentedHi,
Here is one patch me make DDF features exportable.
Comment #9
yseki CreditAttribution: yseki as a volunteer and commentedComment #11
yseki CreditAttribution: yseki as a volunteer and commentedPath for retest
Comment #12
yseki CreditAttribution: yseki as a volunteer and commentedComment #14
yseki CreditAttribution: yseki as a volunteer and commentedOh!
Different folder.
Comment #16
Konstantin Komelin CreditAttribution: Konstantin Komelin as a volunteer commented@yuriseki
Advanced patch contributor guide may help.
Also, make sure the number (N) in hook_update_N is correct.
Comment #17
yseki CreditAttribution: yseki as a volunteer and commentedThanks Konstantin,
I followed the guide, hope it works this time :-)
Comment #18
azinck CreditAttribution: azinck commentedThanks for the patch. Just a couple of little issues with #17
Fixed patch attached.
Comment #19
mortona2k CreditAttribution: mortona2k at Forum One 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 CreditAttribution: 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 CreditAttribution: azinck commentedComment #24
azinck CreditAttribution: azinck commentedComment #25
maximpodorov CreditAttribution: 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 CreditAttribution: Konstantin Komelin as a volunteer commentedMaxim, in that case we'd need an upgrade mechanism.
Comment #27
Leiph CreditAttribution: Leiph as a volunteer commentedI agree with #25. Storing the settings in field instance settings will make the export more robust.
/Leiph