API page: http://api.drupal.org/api/drupal/modules%21node%21node.pages.inc/functio...

Enter a descriptive title (above) relating to node_add_page, then describe the problem you have found:

Seems to be missing a docblock on D7. Has one on D8.

Files: 
CommentFileSizeAuthor
#25 core-node_add_page-doc-backport-1664070-21.patch473 bytesmarcin.wosinek
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]
#21 core-node_add_page doc backport-1664070-11.patch473 bytesmarcin.wosinek
FAILED: [[SimpleTest]]: [MySQL] Fetch test patch: failed to retrieve [core-node_add_page doc backport-1664070-11.patch] from [drupal.org].
[ View ]
#11 drupal-1664070-11.patch576 bytestim.plunkett
PASSED: [[SimpleTest]]: [MySQL] 39,236 pass(es).
[ View ]
#9 node-add-page-doc-1664070-9.patch478 bytesmjonesdinero
PASSED: [[SimpleTest]]: [MySQL] 37,057 pass(es).
[ View ]
#7 node-add-page-doc-1664070-7.patch472 bytesmjonesdinero
PASSED: [[SimpleTest]]: [MySQL] 37,018 pass(es).
[ View ]
#5 node-add-page-doc-1664070-5.patch380 bytesmjonesdinero
PASSED: [[SimpleTest]]: [MySQL] 37,018 pass(es).
[ View ]
#3 node-add-page-doc-1664070-3.patch477 bytesmjonesdinero
PASSED: [[SimpleTest]]: [MySQL] 37,003 pass(es).
[ View ]

Comments

Though it should be noted that the D8 doc header is wrong: this is not a form.

Title:node_add_page() has no docblock?node_add_page() documentation is wrong in D8 and missing in D7
Version:7.x-dev» 8.x-dev
Issue tags:+needs backport to D7

The D8 header does not say it is a form. It says it is a page callback that presents a form... but you are right, there isn't a form there at all. OK, let's document this function correctly in D8 and then backport this correct doc to D7. Thanks for the report and clarification!

Assigned:Unassigned» mjonesdinero
Status:Active» Needs review
StatusFileSize
new477 bytes
PASSED: [[SimpleTest]]: [MySQL] 37,003 pass(es).
[ View ]

assigning to me for further feedback.

attach is a patch for my documentation of the function

@jhodgdon
if feedback is past 6pm Philippines time, will continue to work on this on Monday morning in Philippines time also

Thanks

Status:Needs review» Needs work

Good try! But what we need to do is replace the existing line that says "Presents the node add form" with a more accurate description, not add text before-hand. Also, the first line of any function documentation needs to be a one-line sentence of less than 80 characters:
http://drupal.org/node/1354#functions
and specifically for page callbacks:
http://drupal.org/node/1354#menu-callback

Status:Needs work» Needs review
StatusFileSize
new380 bytes
PASSED: [[SimpleTest]]: [MySQL] 37,018 pass(es).
[ View ]

update the patch

Status:Needs review» Needs work

Thanks for the 2nd try! It's a bit better, but this still isn't quite right:
- Verb tense is wrong: http://drupal.org/node/1354#menu-callback
- The text is still inaccurate. The function normally makes a list of links to add different types of content (if your system has multiple content types), and alteratively goes directly to the node/add/[type] page if you only have one content type. This text does not say that at all, and it needs to.

Status:Needs work» Needs review
StatusFileSize
new472 bytes
PASSED: [[SimpleTest]]: [MySQL] 37,018 pass(es).
[ View ]

hope this is okie now..

Status:Needs review» Needs work

Much better! Maybe though the first line should say:
Displays a list of add content links for available content types.
I make this suggestion because (a) it's not displaying just a list of the content types, but a list of links to add content, and (b) in the other line you added, they're correctly referred to as "content types" rather than "node types". I'm not sure if this will fit in 80 characters... hopefully?

Status:Needs work» Needs review
StatusFileSize
new478 bytes
PASSED: [[SimpleTest]]: [MySQL] 37,057 pass(es).
[ View ]

HI jhodgdon.

Displays a list of add content links for available content types.
i change it to
Displays add content links for available content types.

so that it fit for 80lines..

Version:8.x-dev» 7.x-dev
Status:Needs review» Patch (to be ported)

Great! I love that choice of words. Simple, concise, and gets the point across. :)

I've committed the patch in #9 to Drupal 8.x. Time to backport to 7.x, where the function doesn't even have documentation. Thanks!

Status:Patch (to be ported)» Needs review
StatusFileSize
new576 bytes
PASSED: [[SimpleTest]]: [MySQL] 39,236 pass(es).
[ View ]

Rerolled.

Status:Needs review» Reviewed & tested by the community

Looks good to me.

Thanks! I'll get it committed shortly.

Status:Reviewed & tested by the community» Fixed

Committed to 7.x -- thanks all!

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Version:7.x-dev» 6.x-dev
Assigned:mjonesdinero» Unassigned
Status:Closed (fixed)» Patch (to be ported)

Shall we backport this to 6.x too?

Sounds like a good idea. The function node_add_page() appears to be completely undocumented in D6.

A direct port of the page doesn't seem like the right thing to do, though, since it looks like in D6 the function always displays a list (even if there is only one type of node the person can add).

Version:7.14» 6.x-dev
Component:page.module» documentation

Title:captcha not working in contact formnode_add_page() documentation is wrong in D8 and missing in D7

StatusFileSize
new473 bytes
FAILED: [[SimpleTest]]: [MySQL] Fetch test patch: failed to retrieve [core-node_add_page doc backport-1664070-11.patch] from [drupal.org].
[ View ]

Backport to 6.

Status:Patch (to be ported)» Needs review

Status:Needs review» Reviewed & tested by the community

Thanks! The patch looks good for D6. By the way, you need to set the issue status to "needs review" when you upload a patch. :)

Status:Reviewed & tested by the community» Needs work

The last submitted patch, core-node_add_page doc backport-1664070-11.patch, failed testing.

Status:Needs work» Needs review
StatusFileSize
new473 bytes
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]

File name fixed.

Status:Needs review» Reviewed & tested by the community

That previous error looks like a test bot glitch... Still RTBC.

Oh I see, the file name had spaces in it. Ugh.

Status:Reviewed & tested by the community» Fixed

Thanks again -- committed to 6.x.

Automatically closed -- issue fixed for 2 weeks with no activity.