node_show return doc links to nonexistent function [#1402620]

API page: http://api.drupal.org/api/drupal/modules--node--node.module/function/nod...

Describe the problem you have found:

The text
"Return value

A $page element suitable for use by drupal_page_render()."

should be replaced by
"Return value

A $page element suitable for use by drupal_render()."

The function drupal_page_render does not exist

Comment	File	Size	Author
#5	drupal-7.x-1402620-node-show-return-api-doc.patch	557 bytes	jwlogemann
#5
#3	1402620-node-show-return-api-doc.patch	458 bytes	jeckman
#3
#2	1402620-node-show-return-api-doc.patch	458 bytes	jeckman
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

jhodgdon

she/her

English

CreditAttribution: jhodgdon commented 12 January 2012 at 17:20

Title:	Documentation problem with node_show	» node_show return doc links to nonexistent function
Version:	7.x-dev	» 8.x-dev
Issue tags:		+Novice, +Needs backport to D7

Good catch, thanks for reporting! Good Novice project too.

Comment #2

jeckman CreditAttribution: jeckman commented 12 January 2012 at 19:48

File	Size
1402620-node-show-return-api-doc.patch	458 bytes

Here's my first attempt - adding a patch. Let's see if I've done it correctly.

Comment #3

jeckman CreditAttribution: jeckman commented 12 January 2012 at 19:50

Status:

Active

» Needs review

File	Size
1402620-node-show-return-api-doc.patch	458 bytes

Guess i need to set it to needs review to get testing bot, yes?

Comment #4

jhodgdon

she/her

English

CreditAttribution: jhodgdon commented 13 January 2012 at 00:12

Status:

Needs review

» Reviewed & tested by the community

Looks fine, thanks! Congratulations on your first patch. :)

For future reference, you don't need to upload the patch again when setting to "needs review". The test bot will look for the latest uploaded patch automatically.

Comment #5

jwlogemann CreditAttribution: jwlogemann commented 16 January 2012 at 12:27

File	Size
drupal-7.x-1402620-node-show-return-api-doc.patch	557 bytes

I don't know if this is a duplicate of what jeckman did, but I've created a patch for 7.x. Can someone double check what I did here (first patch as well)