The documentation for arg() doesn't report that it returns the array containing all the components of the current path, when it is called without arguments; that it's clear looking at the code, but it should be better if the documentation would point out that.

The same is true for the documentation page for Drupal 6; the function behaved differently, in Drupal 5.

CommentFileSizeAuthor
#9 937508-D6.patch511 bytesamateescu
#5 937508-2.patch553 bytesamateescu
#3 937508.patch552 bytesamateescu

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

Gracious! This should definitely be fixed.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Issue tags: +Novice

Good project for a novice doc contributor

amateescu’s picture

amateescu commented
Status: Active » Needs review
StatusFileSize
new 937508.patch552 bytes

The novice contributor strikes again :)

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Needs work 
+ *   the components of the current path

It misses a period.

amateescu’s picture

amateescu commented
Status: Needs work » Needs review
StatusFileSize
new 937508-2.patch553 bytes

Right, thanks!

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

That's better, thanks!

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Nice.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Version: 7.x-dev » 6.x-dev
Status: Fixed » Patch (to be ported)

We should port this fix to Drupal 6 as well.

amateescu’s picture

amateescu commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new 937508-D6.patch511 bytes

Here is a patch for D6.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Thanks, the D6 patch looks good too. amateescu -- your recent work is much appreciated!

amateescu’s picture

amateescu commented

Thank you :) I've been using drupal for a few months and i thought it was time to start helping this incredible community.

Right now i'm working through the novice queue but in a little while i have plans to start/maintain some modules and.. who knows, maybe some serious work for D8 :)

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed. Looking forward for your future contributions, keep it up!

Status: Fixed » Closed (fixed)
Issue tags: -Novice

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

Issue tags: +Novice

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.