Closed (won't fix)
Project:
Drupal core
Version:
4.6.x-dev
Component:
documentation
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Mar 2005 at 16:52 UTC
Updated:
7 Mar 2008 at 19:17 UTC
I'm a newbie, so if this is innappropriate, please be polite...
In drupaldocs.org, in the form_textfield page, I would request changing the doc info for the name param to (emphasis added for new text):
$name The internal name used to refer to the field. The value is automatically enclosed in square brackets before it's used.
You can figure it out from the code at the end of the doc page, but documentation is supposed to help by letting you skip having to read the code.
Ben in DC
PublicMailbox at benslade dot com
(append 030516 to the subj line to bypass spam filters)
Comments
Comment #1
dtan commentedmoving to documentation since forms is wrong module.
-1 since a themed functioned is being called.
Comment #2
rivena commentedThis page is now: http://api.drupal.org/api/4.6/function/form_textfield
I have no comment on anything else. ^.^
Anisa.
Comment #3
dries commentedThe only way to fix API documentation, is by submitting a patch against Drupal core. This is not an issue for the documentation team so feel free to move it to the Drupal project instead.
Comment #4
rivena commentedAre you really sure? I seem to recall some people think the other way around. If that's the verdict, I'll happily shift the whole lot, but I want to double check.
Anisa.
Comment #5
Richard Archer commentedThe content on api.drupal.org is automatically generated from the phpdoc comments in the source. So you need to get a patch containing the change into core, and then it will propagate to api.
Comment #6
jstarbow commentedmoved from the doc issue queue - still relevant?
Comment #7
scoutbaker commentedThe reference for form_textfield only shows up in Drupal 4.6. There is no mention of form_textfield for Drupal 4.7, 5, or 6.
Since Drupal 4.x is no longer being developed, I'm setting this to "won't fix."
Comment #8
scoutbaker commentedSorry. Meant to set it back to 4.6 as well.