Warning: missing argument on the rules scheduler page

me too had the same error when iam on my rules scheduler page
warning: Missing argument 3 for rules_data_type_date::get_default_input_form(), called in /var/www/cart/modules/rules/rules_scheduler/rules_scheduler.admin.inc on line 54 and defined in /var/www/cart/modules/rules/rules/modules/rules.rules.inc on line 88.

Same for me !

subscribe

Subscribing and confirmed.

This is easy to reproduce:

- Add a new rule set with no arguments (you don't have to add any rules)
- Goto the scheduling page at rules/rule_sets/scheduling

The function "get_default_input_form" on line 88 of rules.rules.inc defines the method in object "rules_data_type_date". It specifies three arguments:
$info (an array used to describe the form),
$value (the initial value of the date field),
$form_state (which is not used within the function).

The call to this function (method) from line 54 of "rules_scheduler.admin.inc" provides only the first two arguments.

The solution is to either modify the call in line 54 to provide a third argument (it is passed by reference, so a new variable needs to be created and supplied), or to change the function definition on line 88 to remove the unused argument of $form_state.

While either change should work, my preference would be to remove the redundant unused argument from the function definition. However, I note that other class definitions in the file "rules.rules.inc" also include redundant $form_state arguments that could usefully be removed as well.

There are only two calls to any of these functions (methods) from within the rules code. One of these calls supplies the $form_state argument, and the other does not.

Maybe the safest patch is to modify line 88 of "rules.rules.inc" file to specify a default for a missing argument ("&$form_state = NULL").

The patch in #5 is working fine for me.

confirmed.

$form_state may be required by other modules that override the data type classes, so the invocation in rules_scheduler must be fixed. Patch attached.

#8: 940820-rules-scheduler-warning.patch queued for re-testing.

#8: 940820-rules-scheduler-warning.patch queued for re-testing.

Fails due to #996512: D6 contrib project tests: [MySQL] Drupal installation failed..

The patch looks good though. I've committed it, thanks.