Remaining:
#2346097: Use fullcalendar.permissions.yml
#2827930: "Undefined index: display" errors after changing view's format
#2827936: Bad URL in AJAX requests
#2827939: Error: Call to undefined method Drupal\Core\Access\AccessResultAllowed::cachePerRole()
#2827950: Call to undefined function l() in AJAX Update controller
#2828939: Restore AJAX functionality
#2853942: Libraries are not attached for plugins

Original post:
It has been a little time since this module was updated --- core is constantly changing.

This is just a parent issue for a bunch of narrowly focused issue digging through change records and restoring basic functionality.

I can see 2 issue that need solving ... and that was within the first 5 mins of looking so I expect more.

1) We are still using String::checkPlain - when SafeMarkup::checkPlain is the well established replacement

2) drush en fullcalendar identifies compatibility issues with Drupal\fullcalendar\Form\SettingsForm

3) The l() function has been removed from core.

4) submitForm method needs work.

Nice to have:

While not strictly essential to restoring basic functionality. From experience whenever I am working on these types of projects
with coupled javascript I have found converting all the legacy javascript to using strict CAN resolve lots of funnies ahead of time.

Comments

martin107’s picture

Issue summary: View changes

added the "use strict" nice to have.

martin107’s picture

Issue summary: View changes

updated issue summary with l() issue

martin107’s picture

Issue summary: View changes

added submitForm() issue

martin107’s picture

Assigned: martin107 » Unassigned
tacituseu’s picture

Assigned: Unassigned » tacituseu
tacituseu’s picture

Issue summary: View changes
tacituseu’s picture

Category: Bug report » Task
dakala’s picture

@tacituseu: Thanks for bringing this issue to light. On a cursory look, I don't think most of the points are still valid. anyhow, let's focus on the code here https://github.com/dakala/fullcalendar which is same as https://git.drupal.org/project/fullcalendar.git and takes things from there.

tacituseu’s picture

It is based on latest https://git.drupal.org/project/fullcalendar.git, added my issues under 'Remaining' and as child issues.

tacituseu’s picture

Issue summary: View changes

Changed issue summary, created new child issue with working patch #2828939: Restore AJAX functionality.

tacituseu’s picture

Assigned: tacituseu » Unassigned
tacituseu’s picture

Issue summary: View changes