Fixed
Project:
PhotoSwipe - Responsive JavaScript Modal Image Gallery
Version:
5.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
4 Sep 2026 at 17:40 UTC
Updated:
15 Sep 2026 at 13:53 UTC
Jump to comment: Most recent
Comments
Comment #2
anybodyComment #5
ressaI took a look at it @anybody, and the
phpcserrors, andcspellsuggestions I could fix.It said that a component in photoswipe/modules/photoswipe_dynamic_caption/photoswipe_dynamic_caption.module was no longer used:
... and it does seem use of
Xss::filterwas removed here:From "Improve caption value output"
https://git.drupalcode.org/project/photoswipe/-/commit/b70e9a47ca1174644...
But probably best to double check this change in the MR.
There are also some more complicated tasks, like warnings about converting \Drupal::service to a dependency injection, which is a bit beyond my basic coding skills:
It seems like the
phpcsandcspelltests are now green, but like you wrote, perhaps some more work is needed here?Comment #6
anybodyThank you very much! We should try to make eslint and phpstan also happy if possible or add ignores maybe?
Asking @grevil to take a short look if possible.
Comment #9
grevil commentedNot really a novice task, as the failures need some major refactoring.
Comment #10
grevil commentedAll green now!
Thanks ressa for your initial work! I created a seperate branch, since this needed some further work, and some changes weren't necessarily needed (e.g. README adjustments and the @codingStyleIgnoreX removals).
Please review @anybody!
Comment #11
ressaThank you @grevil! Nice to see all tests are now green :)
Comment #12
ressaSorry about the change in the "Assigned" field, I am not sure why that happened?
I only changed the Status to "Needs review", but maybe that affected that field as well? (and I cannot add @anybody)
Comment #13
anybodyThank you both very much! Merging!
Comment #14
anybody@grevil: As follow-up finally please see the both other tasks assigned to you, so we're finished here. Thanks!