\Drupal calls should be avoided in classes, use dependency injection instead

Thanks @Maithri shetty.

I am aware that the patch is based on branch 6.x. As there are a lots of work still needed for branch 6.x, I am not sure when we can have a stable release for 6.x in the near future.

Currently, the default main branch is 5.x. If you have time, could you create another patch for branch 5.x as well?

Thanks @Maithri.

Merged.

This merge breaks the functionality. If FullcalendarViewPreprocess is a service then the dependencies should be added directly into the services.yml file, not with create method.

Something like this:

# View preprocess service.
fullcalendar_view.view_preprocess:
class: Drupal\fullcalendar_view\FullcalendarViewPreprocess
arguments: ['@language_manager', '@csrf_token', '@entity_type.manager', '@fullcalendar_view.timezone_conversion_service']

and the create method is not needed.

Thanks @cslevy.

Good point.

I believe the hot fix suggested by @cslevy from #15 patch the bug.

Please feel free to reopen it if it is still an issue.

Yes, the dev branch has fixed the issue! Thank you!