On line 304 of agreement.module the agreement form is declared,
function agreement_form($form, $text, $status = 0, $uid = -1) {
But the FAPI docs say that the first argument in a form function should be $form_state, so line 304 should be,
function agreement_form($form_state, $text, $status = 0, $uid = -1) {
It caused some problems for me when using drupal_execute and luckily a coworker noticed the difference.
Comments
Comment #1
pagaille commentedThanks for catching that - will fix in next release.
Comment #2
pagaille commentedThis has been fixed in release 6.x-2.2.