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.

Files: 
CommentFileSizeAuthor
#9 937508-D6.patch511 bytesamateescu
#5 937508-2.patch553 bytesamateescu
PASSED: [[SimpleTest]]: [MySQL] 26,565 pass(es). View
#3 937508.patch552 bytesamateescu
PASSED: [[SimpleTest]]: [MySQL] 26,498 pass(es). View

Comments

jhodgdon’s picture

Gracious! This should definitely be fixed.

jhodgdon’s picture

Issue tags: +Novice

Good project for a novice doc contributor

amateescu’s picture

Status: Active » Needs review
FileSize
552 bytes
PASSED: [[SimpleTest]]: [MySQL] 26,498 pass(es). View

The novice contributor strikes again :)

kiamlaluno’s picture

Status: Needs review » Needs work
+ *   the components of the current path

It misses a period.

amateescu’s picture

Status: Needs work » Needs review
FileSize
553 bytes
PASSED: [[SimpleTest]]: [MySQL] 26,565 pass(es). View

Right, thanks!

jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

That's better, thanks!

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Nice.

jhodgdon’s picture

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

Status: Patch (to be ported) » Needs review
FileSize
511 bytes

Here is a patch for D6.

jhodgdon’s picture

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

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

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.