Opigno calendar event limit to members functionality disallows global events [#3181738]

Problem/Motivation

opigno_calendar has added a new field_calendar_event_members which requires members to be added to an event for it to show up on the calendar.

This means that any previous global events (i.e. before this field was added) will fail to show for any users, because they don't have any members populated, also means it's not possible to add new global events.

Relatedly, opigno_calendar_event_opigno_calendar_event_form_validate should be in opigno_calendar_event module, but is actually in opigno_calendar_module

Steps to reproduce

Proposed resolution

Would suggest either making this behaviour configurable, or moving it to a new calendar bundle.

Remaining tasks

User interface changes

API changes

Data model changes

Comments

Comment #1

10 November 2020 at 16:56

catch created an issue. See original summary.

Comment #2

catch

he/him

English

commented 10 November 2020 at 17:01

It looks like opigno_calendar_form_alter() already checks if the field exists, but the validation logic and hook_views_query_alter() implementations don't. So adding a field existence check in both of those places would make the entire logic optional.