Closed (fixed)
Project:
Conference Organizing Distribution Support Modules
Version:
7.x-1.x-dev
Component:
cod_events
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
25 May 2013 at 14:38 UTC
Updated:
16 Jun 2014 at 19:20 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
aburrows commentedJust uploaded patch, includes new hook for form, flag and views. Off onto plane to London now, so can amend anything when I'm back
Comment #2
japerryGreat work! We're almost there, however this patch doesn't allow you to select one default event. The expected functionality is when one event is selected as default, the other event (previously selected as default) will be unselected.
I've re-rolled the patch to apply cleanly against the latest head. We should be testing against this patch since #1 doesn't apply anymore.
Comment #3
ezra-g commentedSlightly more descriptive title.
Comment #4
aburrows commentedPatch that uses previous patch and additional rules to fix issue. Please test
Comment #5
aburrows commentedComment #6
japerryadded! well sorta--
After looking at how the flag worked, we decided that it'd be better to make it a configuration option. So now you can configure a default event. If there is only one event, it'll just use that by default instead.
Either way, now when creating a node, you'll not see the group on creation if you have a default set. Of course if you have more than one event, or zero events, you can set the default to be -none-
Comment #7
aburrows commentedHas this functionality been done, or do you want a patch for it?
Comment #8
chrisguindon commentedAfter pulling-in the new changes in my sandbox site, I started to see a long list of php errors on some of my views.
Notice: Undefined property: stdClass::$nid in cod_events_views_pre_render() (line 6 of /profiles/cod/modules/contrib/cod_support/cod_events/cod_events.views.inc).
I suggest, we verify the name of the view before we the loop to add '!event (default event)'.
http://drupalcode.org/project/cod_support.git/blob/d957530c599b848ef5b67...
Attached is a patch that does that.
Comment #9
jyee commented#8 was committed, but I think we're still not quite fixed on this.
Comment #10
jyee commentedMarking as fixed. Changing events via the admin interface is confirmed to work and japerry is currently making changes to set the default event during install.