Closed (outdated)
Project:
Security Kit
Version:
2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
4 Jan 2021 at 11:56 UTC
Updated:
1 Oct 2026 at 18:02 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
suzymasriPatch attached.
Comment #4
suzymasriUpdated the patch to cover the below errors:
Comment #5
Pooja Ganjage commentedHi,
Creating a patch for raised issue in #2 comment patch and as well follow to solve deprecated code as per the below link mentioned
https://www.drupal.org/node/3168858
Please review the patch.
Thanks.
Comment #6
suzymasriThanks @Pooja Ganjage, there aren't any changes between patch #4 and #5, interdiff was empty.
Fixed a missing class issue, patch and interdiff attached.
Comment #7
Pooja Ganjage commentedComment #8
Pooja Ganjage commentedComment #10
suzymasri@Pooja Ganjage a minor text change was overridden by your patch #7, it would make it a lot easier for everyone to provide interdiff so the changes are clear, thanks!
Patch updated with text fixes.
Comment #11
timmillwoodLooks like the failing test above is on 9.1.0 due to #3186443: PHPUnit 9.5 Call to undefined method ::getAnnotations() which has been committed to 9.1.x and will be released in 9.1.1. The tests are passing fine in 9.1.x and 9.0.10.
Therefore, looks good to me, great to get these deprecations fixed early.
Comment #12
damienmckennaComment #13
mcdruid commentedThere are a handful of these wide-ranging patches in the queue which all unfortunately require a reroll after some other fixes were committed.
Comment #14
mcdruid commentedComment #15
sourabhjainComment #16
sourabhjainCreated a latest patch. Please review.
Comment #17
silvi.addweb commentedI've tested the #16 patch and there are no problems found.
Comment #18
the_g_bomb commentedI suspect all deprecations listed here were fixed in #3289537: Automated Drupal 10 compatibility fixes. Patch #16 no longer applies. Marking "Closed (outdated)".