Problem/Motivation
The Smart Date module stores values as timestamps, so would need a little work for those values to display in a Calendar view.
Proposed resolution
Issues like #2820803: Support Recurring Dates Field in Calendar 8.x-1.x patch Calendar to work with other contrib modules. Could Calendar create a processor or some other kind of plugin class that other modules could implement in order to define their own integration? For example, Fullcalendar View has the FullcalendarViewProcessorBase that modules can extend to ensure the data is passed in with the correct data structure.
API changes
Provide some method for contrib modules to define their own compatibility with Calendar.
Note that I have also created #3177760: Support Calendar module specifically to address the work for Smart Date to implement whatever integration method is available.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 3177761-6.calendar.Support-for-Smart-Date.patch | 710 bytes | grahamc |
Issue fork calendar-3177761
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
mandclu commentedComment #3
mandclu commentedComment #4
derekb commentedFollowing - I had to switch to FullCalendar view because it supported SmartDate but Calendar was a lot easier to work with and theme.
Comment #5
sseto commentedI'm using SmartDate with FullCalendar view and it's really destroying my server because of loading all the historical events at once. I really hope SmartDate will work with Calendar.
Comment #6
grahamcThe required changes for Smart Date are actually way smaller than the patch targeting date_recur...
I have it up and running with the attached one-line diff, in conjunction with Smart Date patch to add granularity arguments.
(Yes, calling out specific module names is icky! - either an interface or otherwise inferring correctly from the field definitions would still be preferable)
Comment #7
mandclu commentedAdded some comments to #3177760: Support Calendar module, not sure if any of them really apply more to this issue.
Comment #8
apkwilson commentedI've just set this up using #6 and the patch in #3177760-13: Support Calendar module. Aside from having to add the date contextual filter before selecting the calendar format, the experience was seamless.
Agree that referencing
smartdateisn't the best, but what would be required to avoid it? A new plugin type?Comment #9
daniel korteI think this is fixed by #3238239: Allow base field definitions to render