Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hello,
in your documentation you said that there is a $search['submit'] key. However with a print_r i found that there is a $search['actions'] key.
Thank you.
apanag
Comment | File | Size | Author |
---|---|---|---|
#8 | 1008022-8.patch | 2.08 KB | jhodgdon |
#1 | search.patch | 595 bytes | droplet |
Comments
Comment #1
droplet CreditAttribution: droplet commentedyeah, because this changes:
http://drupal.org/update/modules/6/7#actions_buttons
Comment #2
aspilicious CreditAttribution: aspilicious commentedD7:
Patch:
Shouldn't this be: "$search['actions']['submit']: Form submit button." ???
Comment #3
droplet CreditAttribution: droplet commentedcan you provide a patch ? I have no idea :(. actions return a grouped set value.
Comment #4
aspilicious CreditAttribution: aspilicious commentedMe neither but the original text tries to access a specific button "Form submit button". If it still tries to access that one button it needs to be $search['actions']['submit'];
If not, the text "Form submit button" is wrong :).
I don't write a patch before I know what to do :)
Comment #5
jhodgdonI think it should say something like:
$search['actions']: Buttons for the search form, including $search['actions']['submit'] (the submit button).
Comment #6
droplet CreditAttribution: droplet commentedIt's not exactly "including $search['actions']['submit']" ??
$search['actions'] is not an array and contains rendered array elements.
maybe:
$search['actions']: Buttons for the search form.
or
$search['actions']: Search form buttons ready for print.
if change $search['actions'] into an array requires API change.
Comment #7
jhodgdonOK, thanks for the correction. Still needs a patch.
Comment #8
jhodgdonHere's a patch, which also brings the tpl header up to standards.
Comment #9
droplet CreditAttribution: droplet commentedpatch seems ok.
** where can I see the docs about "tpl header up to standards", thanks.
Comment #10
jhodgdonhttp://drupal.org/node/1354 has all of our doc header standards
Comment #11
webchickCommitted to HEAD. Thanks!