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.

Files: 
CommentFileSizeAuthor
#3 PathSubscriber-2105803-3.patch1.36 KBStephaneQ
PASSED: [[SimpleTest]]: [MySQL] 58,556 pass(es).
[ View ]
#1 PathSubscriber-2105803-1.patch570 bytesStephaneQ
PASSED: [[SimpleTest]]: [MySQL] 58,559 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new570 bytes
PASSED: [[SimpleTest]]: [MySQL] 58,559 pass(es).
[ View ]

Here is a patch.

Should we fix the RedirectResponseSubscriber class in the same time, I think it's the same problem.

Status:Needs review» Needs work

Thanks! Yes, let's fix both.

Also, a small grammatical problem: which -> that.

Status:Needs work» Needs review
StatusFileSize
new1.36 KB
PASSED: [[SimpleTest]]: [MySQL] 58,556 pass(es).
[ View ]

I also changed the file docblock, should be "Contains" according to https://drupal.org/node/1354#file

Title:PathSubscriber documentation is totally wrongSubscriber class documentation is lacking in a couple of places

Looks good to me! Thanks for the extra doc fixes. In a small patch like this, they are always a welcome addition. Updating title. :)

Status:Needs review» Fixed

Thanks again! Committed to 8.x.

Automatically closed -- issue fixed for 2 weeks with no activity.