Closed (fixed)
Project:
Date
Version:
7.x-2.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
15 May 2012 at 14:43 UTC
Updated:
4 Jan 2014 at 20:50 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jmuzz commentedIt seems to be a problem related to #field_parents and #parents in date_combo_validate . I don't know if this is the right answer as I don't know the theory and I don't have time to test right now, but I changed the following lines at 454 in date_elements.inc from
to
And the dates seem to be getting saved now.
Comment #2
jmuzz commentedI am more confident about this change now. I have looked more at the values of #parents and #field_parents and it looks like #parents always includes the elements from #field_parents plus a few more that identify the field itself.
The current code works on dates directly in the top level of an entity since #field_parents is empty and #parents just has info about the field, but when a date is inside a field collection it breaks because it assumes that #parents will still only have info about the field itself when it actually contains all the info from #field_parents as well.
I have tested this fix on a date inside a field collection and a date nested a level deeper than that inside another field collection.
Comment #4
jmuzz commentedComment #5
jmuzz commented2: date-parents_fix-1580032-2.patch queued for re-testing.
Comment #6
podarok#2 looks good for me
good to see here test
Comment #7
vijaycs85Committed c4c0522 and pushed to 7.x-2.x. lets add test cases now.
Comment #8
krystlc commented#2 worked for me too~
Comment #9
vijaycs85Lets add testcase as sepate issue then