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

The documentation for search_box() makes a reference to theme_search_box_form() in the "see also" section, but the function is not found in Drupal 6.

#11 search_box-1098556-11.patch481 bytesrocket_nova
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]
#2 theme_search_box_form-1098556.patch446 bytesbarbi
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]


Title:The documentation for search_box() contains a reference to theme_search_box_form(), but the function doesn't exist in Drupal 6search_box() doc refers to nonexistent theme_search_box_form()
Issue tags:+Novice

Yeah, that @see should be removed. I can't see anything to replace it with either. Good project for a novice doc contributor.

Assigned:Unassigned» barbi
Status:Active» Needs review
new446 bytes
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]

I think it's supposed to link to theme_search_block_form instead of theme_search_box_form.

If we change the function name in @see (rather than removing the reference) then we should also add a reference for theme_search_theme_form.

Why is theme_search_block_form() relevant to this function? I don't think it's called from this function or very related? Besides which, theme_search_block_form() is not a real function. It's a dummy that shouldn't even exist on the API site, since it's just saying "this is really search-block-form.tpl.php.

Same with theme_search_theme_form() -- why relevant and isn't really a function.

It's relevant for someone looking to theme the search box form. I'm ok with removing the references though, I was just curious where the current reference to theme_search_box_form came from.

Status:Needs review» Needs work

Status was never changed.

This should just link to search-block-form.tpl.php as it does in Drupal 7/8. Good project for a novice patch contributor. Oh, it's already tagged. :)

Assigned:barbi» Unassigned

un-assigning since barbi hasn't worked on this in quite a long time.

This is a 6.x-only issue, so I don't think the proposed removal of search-block-form.tpl.php in 8.x affects us here. But hopefully that issue's patch will also remove the link in the doc header for 8.x...

Assigned:Unassigned» rocket_nova

Working on this at BADCamp.

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

Removed reference to theme_search_box_form() and added reference to search-block-form.tpl.php.

Status:Needs review» Reviewed & tested by the community

Looks good, thanks!

Status:Reviewed & tested by the community» Fixed

Thanks, committed, pushed.

Status:Fixed» Closed (fixed)
Issue tags:-Novice

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