Closed (fixed)
Project:
Field collection
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Apr 2013 at 06:55 UTC
Updated:
24 Oct 2013 at 12:20 UTC
Jump to comment: Most recent file
Comments
Comment #1
wizonesolutionsI have the same problem. Did you ever find a workaround in the end?
I'm going to dare to raise this to major and hopefully get some attention on it. Also, I'll see if it's a quick fix, e.g. what they did in Features itself to make this work with nodes.
Comment #2
wizonesolutionsAlright, I have found a solution. It's really simple. The function
field_collection_features_pipe_field_alterinfield_collection.moduleneeds to be renamed tofield_collection_features_pipe_field_instance_alter.It appears Features doesn't call the former anymore.
My version of Field Collection is already patched, so I can't provide a patch quite yet, but I will do some Git magic and post one soon.
Comment #3
wizonesolutionsOh, forgot one thing. Change:
to:
Comment #4
wizonesolutionsHere's the patch.
Comment #6
technicalknockout commentedLooks like you might have built the patch in a repository other than the module's - just learning how to make these patches myself so I figured I'd give re-rolling it a try.
Comment #7
damienmckennaThis needs to support both the old and new APIs, for sites that have not upgraded to Features v2 yet.
Comment #8
damienmckennaThis patch adds field_collection_features_pipe_field_instance_alter() instead of replacing field_collection_features_pipe_field_alter(). It also adds a check to see if the new field_instance functionality is available before field_collection_features_pipe_field_alter() returns anything, to avoid unnecessarily causing problems by defining both the new 'field_instances' and the legacy 'field' values. Lastly, the docbook comments have been made more verbose to explain what is going on.
Comment #9
wizonesolutionsThanks for the rerolls :)
Comment #10
pcoucke commentedThis patch works for me on field collection 7.x-1.0-beta5 with features 7.x-2.0-rc1.
Comment #11
acrollet commentedPatch in #8 also works nicely for me.
Comment #12
fagoThanks, committed.