Closed (fixed)
Project:
Agreement
Version:
6.x-2.0-rc2
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Nov 2011 at 15:54 UTC
Updated:
1 Aug 2012 at 23:49 UTC
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.