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

Gracious! This should definitely be fixed.

Issue tags:+Novice

Good project for a novice doc contributor

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

The novice contributor strikes again :)

Status:Needs review» Needs work

+ *   the components of the current path

It misses a period.

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

Right, thanks!

Status:Needs review» Reviewed & tested by the community

That's better, thanks!

Status:Reviewed & tested by the community» Fixed

Committed to CVS HEAD. Nice.

Version:7.x-dev» 6.x-dev
Status:Fixed» Patch (to be ported)

We should port this fix to Drupal 6 as well.

Status:Patch (to be ported)» Needs review
StatusFileSize
new511 bytes

Here is a patch for D6.

Status:Needs review» Reviewed & tested by the community

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

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 :)

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.