- Issue created by @larowlan
- Status changed to Needs review
almost 2 years ago 3:25am 7 February 2023 - ๐ฎ๐ณIndia noorulshameera
Adding a patch to remove PathProcessor class file.
- Status changed to RTBC
almost 2 years ago 5:52am 7 February 2023 - ๐ฆ๐บAustralia larowlan ๐ฆ๐บ๐.au GMT+10
Thanks, before I opened this I did a search for any other usages and found none - e.g. service modifiers etc.
- First commit to issue fork.
- @gxleano opened merge request.
- ๐ช๐ธSpain gxleano Cรกceres
Hi @noorulshameera!
Thanks for the patch, but instead of it, we should create a proper MR in order to make things easier for maintainers.
I've created a MR with the change and remove the issue tag "novice".
- ๐ฆ๐บAustralia larowlan ๐ฆ๐บ๐.au GMT+10
@gxleano please stop creating extra work for people
There is a patch, and it is RTBC
Removing credit and closing the MR
- ๐ณ๐ฟNew Zealand quietone
I checked core for usages in comments (I used PHPStorm) and did not find anything.
This file was added in #1976180: url_alter_test module uses an event subscriber instead of a path processor โ and it should have been removed in #1888424: Make Drupal's URL generation logic available to HttpKernel, and minimize code repetition/divergence โ .
I intend to commit this in the next 24 hours.
-
quietone โ
committed ba4f933c on 10.1.x
Issue #3339770 by noorulshameera, larowlan: Remove dead class...
-
quietone โ
committed ba4f933c on 10.1.x
- Status changed to Fixed
almost 2 years ago 1:09am 11 February 2023 Automatically closed - issue fixed for 2 weeks with no activity.