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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented

Gracious! This should definitely be fixed.

jhodgdon’s picture

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

Good project for a novice doc contributor

amateescu’s picture

amateescu CreditAttribution: amateescu commented
Status: Active » Needs review
FileSize
937508.patch552 bytes

The novice contributor strikes again :)

apaderno’s picture

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

It misses a period.

amateescu’s picture

amateescu CreditAttribution: amateescu commented
Status: Needs work » Needs review
FileSize
937508-2.patch553 bytes

Right, thanks!

jhodgdon’s picture

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

That's better, thanks!

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Nice.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon 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 CreditAttribution: amateescu commented
Status: Patch (to be ported) » Needs review
FileSize
937508-D6.patch511 bytes

Here is a patch for D6.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon 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 CreditAttribution: 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
CreditAttribution: Gábor Hojtsy 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.