The documentation for search_form() says the return value is 'An HTML string containing the search form' but it actually returns a form array.


dawehner’s picture

It would be possible to use the d7 documentation "A Form API array for the search form."

jhodgdon’s picture

Yes, the D7 version should be used, for consistency. Thanks for reporting.

jhodgdon’s picture

Title:Documentation problem with search_form» search_form doc has wrong return value information
Issue tags:+Novice

good project for a novice doc contributor

daniels220’s picture

Status:Active» Needs review
new492 bytes

Have a patch.

jhodgdon’s picture

Status:Needs review» Needs work

This patch doesn't apply - badly formatted at the top with "search_old.module".

Other than that, looks good...

daniels220’s picture

Status:Needs work» Needs review
new515 bytes

I hope the two extra lines Git created at the top don't mess things up. New patch attached.

jhodgdon’s picture

Status:Needs review» Reviewed & tested by the community

Looks good, thanks!

Gábor Hojtsy’s picture

Status:Reviewed & tested by the community» Fixed

Thanks, committed.

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

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