Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.