API page: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21EventSubs...
Enter a descriptive title (above) relating to class PathSubscriber, then describe the problem you have found:
The PathSubscriber class has only one line of documentation and it appears to have been copy/pasted from another class. It's totally wrong.
It needs to be replaced with a line documentation that says what it actually does, which is to translate path aliases.
I think this is actually a good novice project.
Comment | File | Size | Author |
---|---|---|---|
#3 | PathSubscriber-2105803-3.patch | 1.36 KB | StephaneQ |
#1 | PathSubscriber-2105803-1.patch | 570 bytes | StephaneQ |
Comments
Comment #1
StephaneQHere is a patch.
Should we fix the RedirectResponseSubscriber class in the same time, I think it's the same problem.
Comment #2
jhodgdonThanks! Yes, let's fix both.
Also, a small grammatical problem: which -> that.
Comment #3
StephaneQI also changed the file docblock, should be "Contains" according to https://drupal.org/node/1354#file
Comment #4
jhodgdonLooks good to me! Thanks for the extra doc fixes. In a small patch like this, they are always a welcome addition. Updating title. :)
Comment #5
jhodgdonThanks again! Committed to 8.x.