This patch corrects the following issues:
- $may_cache is no longer used in hook_menu()
- the $form_id, $form_values parameters have gone away in favour of $form and &$form_state (the data previously in $form_values is now stored $form_state['values'])
- The submit form has been changed to use a submit button instead of system_settings_form() as the module has its own submit function that sets the variables

Misc. other coding style changes as per recommendations by the Code Review module
- Eliminated use of tabs in favour of two spaces
- Empty lines should not be indented

CommentFileSizeAuthor
#1 fix-d5-leftovers-1376892-1.patch11.42 KBstewart.adam

Comments

stewart.adam’s picture

stewart.adam commented
StatusFileSize
new fix-d5-leftovers-1376892-1.patch11.42 KB
mattcasey’s picture

mattcasey commented

Thanks for this. There's one typo, an extra '$' I found in your patch:

+    else if ($$form_state['values']['reset'] == $form_state['values']['op'])
stewart.adam’s picture

stewart.adam commented

Oops, sorry about that!

simon georges’s picture

simon georges commented
Status: Active » Needs review

Changing status, as there is a patch.