🇺🇸United States @i.mcbride

Are you running any other modules which use linkit's API? I ran into this issue with the media_entity_file_redirect module 🐛 Update MediaFileRedirect::getUrl() to match Linkit Needs review today because the function signature of MediaFileRedirect::getUrl() changed in linkit 6.1.5.

i.mcbride → created an issue.

greggles → credited i.mcbride → .

The previous patch had a malformed character on line 44. Here is a corrected version.

Here is a patch which implements the proposed solution.

i.mcbride → created an issue.

A patch with the suggested change is attached.

i.mcbride → created an issue.

i.mcbride → created an issue.

Here is a version which will apply against 2.1.x. I've changed the name from removeContent() to removeRelationship() to conform to function names in the 2.1.x branch.

i.mcbride → made their first commit to this issue’s fork.

i.mcbride → made their first commit to this issue’s fork.

i.mcbride → made their first commit to this issue’s fork.

i.mcbride → made their first commit to this issue’s fork.

i.mcbride → made their first commit to this issue’s fork.

Reopening as requested. I've installed security_review 2.0.1 and notice the error reported in this issue running drush secrev with Drupal 9.5.4 on PHP 8.1. Applying the patch in #6 resolves the issue. The issue only affected CLI execution of the checks. The web UI worked without issue.