Closed (fixed)
Project:
Rules
Version:
7.x-2.x-dev
Component:
User interface
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
16 Aug 2011 at 11:39 UTC
Updated:
4 Jan 2014 at 01:11 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
fago>+ if (isset($value_state['provides'])) {
$value_state isn't a fitting variable name ;) Let's just use $values, or $form_values.
> // Enable AJAX once #756762 is fixed.
- // '#ajax' => rules_ui_form_default_ajax('none'),
+ '#ajax'
Does that work? IF yes, remove the comment too.
>Returns the right form values array from the $form_state.
Let's shortly describe that this is necessary if form is embedded somewhere.
Else the patch looks good to me.
Comment #2
sepgil commentedCorrected the issues.
Comment #3
sepgil commentedComment #4
fagoI fixed the comment to be like that.
Also, I've done some improvements like directly passing $form['settings'] and fixing the function when called during $form generation + I fixed a validation problem.
Committed it, thanks.
Comment #5
sepgil commentedAdded GSoC tag.