API page: http://api.drupal.org/api/drupal/modules--node--node.module/function/nod...
Describe the problem you have found:
Return value
Current:
A fully-populated node object.
Expanded:
A fully-populated node object or false if the node is not found.
Comment | File | Size | Author |
---|---|---|---|
#9 | 1345480-d7-node_load_return_documentation.patch | 555 bytes | chris.leversuch |
#8 | 1345480-node_load_return_documentation-D7.patch | 555 bytes | chris.leversuch |
#5 | 1345480-node_load_return_documentation.patch | 588 bytes | chris.leversuch |
#3 | node_load_return_documentation-1345480.patch | 587 bytes | rbayliss |
#3 | node_load_return_documentation-1345480-D7.patch | 554 bytes | rbayliss |
Comments
Comment #1
jhodgdonThat sounds like a good change to make, in D8/7 documentation, and actually D6 too. Looks like a good project for a novice contributor...
Comment #2
jhodgdonComment #3
rbayliss CreditAttribution: rbayliss commentedPatches attached.
Comment #4
jhodgdonThanks! Can you add a comma before the "or"?
Also, I recommend just making the patch for D8 on the first go-around, and then after that patch has been accepted and committed, only then making a d7 or d6 version. This saves time and effort, since there are often changes reviewers point out that are needed in the d8 version -- might as well get that right before taking the time to port it to other versions of Drupal.
Comment #5
chris.leversuch CreditAttribution: chris.leversuch commentedD8 patch with , included
Comment #6
jhodgdonLooks good, thanks!
Comment #7
Dries CreditAttribution: Dries commentedCommitted to 8.x. Moving to 7.x.
Comment #8
chris.leversuch CreditAttribution: chris.leversuch commentedD7 version.
Comment #9
chris.leversuch CreditAttribution: chris.leversuch commentedDifferent file name fo testing
Comment #10
jhodgdonThanks, ready for D7 too. :)
Comment #11
webchickCommitted and pushed to 7.x. Thanks!