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

pagaille’s picture

Version: 6.x-1.2 » 6.x-2.0-rc2

Thanks for catching that - will fix in next release.

pagaille’s picture

Status: Active » Closed (fixed)

This has been fixed in release 6.x-2.2.