Looks like in event_calendar_data(), when the $nid object is added to the array, it is added by reference. This makes subsequent changes to $nid->event_state affect all instances. As a result, for events spanning more than one day, all parts are marked as "end".

This patch adds a drupal_clone() call which fixes the problem.

CommentFileSizeAuthor
event-module-state-clash-fix.patch945 bytesumonkey

Comments

umonkey’s picture

umonkey commented
Version: 5.x-2.x-dev » 5.x-1.x-dev
Status: Needs review » Reviewed & tested by the community

http://drupal.org/node/63239
http://drupal.org/node/85972
http://drupal.org/node/118760

Will anybody please fix this bug? First reported in May 2006, with patches and discussions, what's the problem?

killes@www.drop.org’s picture

killes@www.drop.org commented

the problem is that I don't understand what the problem is. ;(

killes@www.drop.org’s picture

killes@www.drop.org commented
Status: Reviewed & tested by the community » Fixed

modified patch applied.

umonkey’s picture

umonkey commented

Thank you!

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)