Automated Drupal 10 compatibility fixes

Same patch, except using DI.

Re-rolled patch#2 with check on null for strip_tags.

Deprecated function: strip_tags(): Passing null to parameter #1 ($string) of type string is deprecated in Drupal\addtocalendar\Plugin\Field\FieldFormatter\AddToCalendar->getProperValue() (line 233 of modules/contrib/addtocalendar/src/Plugin/Field/FieldFormatter/AddToCalendar.php).

Fixed my previous patch from comment #4.

I really like the updates made in #3. It would have been great to take that as the starting point for patches #4 and #6 instead of starting back at #2.

Here's a patch that starts with #3 and adds null coalesce operations to all the strip_tags calls to protect against null values.

When manually testing #8, I got an exception about not enough arguments being passed to AddToCalendarApiWidget::_construct()

I think we could add an empty update hook to rebuild the container, but I'd rather just add a simple BC Layout. That's what this patch does.

Patch #9 appears to be working well on our site.

I noticed I missed one BC layer in the patch in #9. This patch adds it.

I just became a maintainer and was able to commit our patch! I'm going to do a quick final test before doing a D10-compatible release. Thanks!

We did it!

https://www.drupal.org/project/addtocalendar/releases/8.x-3.3 →

Automatically closed - issue fixed for 2 weeks with no activity.