Account created on 26 June 2009, about 15 years ago
#

Merge Requests

Recent comments

πŸ‡ΊπŸ‡ΈUnited States i.mcbride

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

πŸ‡ΊπŸ‡ΈUnited States i.mcbride

Here is a patch which implements the proposed solution.

πŸ‡ΊπŸ‡ΈUnited States i.mcbride

A patch with the suggested change is attached.

πŸ‡ΊπŸ‡ΈUnited States i.mcbride

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.

πŸ‡ΊπŸ‡ΈUnited States i.mcbride

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

πŸ‡ΊπŸ‡ΈUnited States i.mcbride

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

πŸ‡ΊπŸ‡ΈUnited States i.mcbride

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

πŸ‡ΊπŸ‡ΈUnited States i.mcbride

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

πŸ‡ΊπŸ‡ΈUnited States i.mcbride

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

πŸ‡ΊπŸ‡ΈUnited States i.mcbride

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.

Production build 0.69.0 2024