Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Simply expands on their documentation and makes it more understandable, with more examples.
Comment | File | Size | Author |
---|---|---|---|
#6 | more352712.patch | 835 bytes | jastraat |
document-menu_get_object-arg-better.patch | 3.3 KB | dmitrig01 | |
Comments
Comment #1
moshe weitzman CreditAttribution: moshe weitzman commentedComment #2
Dries CreditAttribution: Dries commentedI've committed this to CVS HEAD. Thanks Dmitri.
The patch failed to apply against DRUPAL-6 so I'm marking this 'fixed' for now. If this needs to be backported, feel free to re-open this issue.
Comment #3
ksenzeeThis isn't this patch's fault, but reading it I noticed that there's a mismatch here. If the path is now "admin/build/types", arg(1) returns "build", not "content". I'm not on my dev machine or I'd provide an actual patch -- sorry about that.
Comment #4
jhodgdonRE #3 - The arg() doc header has been changed again, due to more changes in core paths, and now says
When viewing a page at the path "admin/structure/types", for example, arg(0) returns "admin", arg(1) returns "content", and arg(2) returns "types".
http://api.drupal.org/api/function/arg/7
This does need to be fixed. Good project for a novice doc contributor.
Comment #5
jastraat CreditAttribution: jastraat commentedsorry - I didn't think this had been assigned and I was looking for a good practice issue.
Comment #6
jastraat CreditAttribution: jastraat commentedComment #7
jastraat CreditAttribution: jastraat commentedComment #8
jhodgdonjastraat: You cannot normally mark your own patch as "reviewed and tested by the community". You need to wait for someone else to review it.
I've just reviewed it. It's fine. :)
Comment #9
jhodgdonComment #10
jastraat CreditAttribution: jastraat commentedGenerally no - but it was a very, very tiny string change :)
Comment #11
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks.