Posted by joncup on March 19, 2008 at 6:21am
| Project: | Splash |
| Version: | 6.x-2.x-dev |
| Component: | User interface |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
at admin/settings/splash/how
when i select Display mode: Redirect, i get a warning message: Display mode field is required.
Comments
#1
I have the same problem... No solution?
#2
i have the same problem, too.
#3
in the 6.x-2.3 module, search for the next array
$form['splash_how']['mode'] = array('#type' => 'radios',
'#required' => TRUE,
'#title' => t('Display mode'),
'#options' => $options,
'#default_value' => $splash_how['mode'],
'#description' => t('Redirect to the splash page, open it in a new window or in a fancy !thickbox.', array('!thickbox' => l(t('ThickBox'), 'http://jquery.com/demo/thickbox/'))),
);
you can comment out the line that says #required like this
$form['splash_how']['mode'] = array('#type' => 'radios',
// '#required' => TRUE,
'#title' => t('Display mode'),
'#options' => $options,
'#default_value' => $splash_how['mode'],
'#description' => t('Redirect to the splash page, open it in a new window or in a fancy !thickbox.', array('!thickbox' => l(t('ThickBox'), 'http://jquery.com/demo/thickbox/'))),
);
that will fix it.
#4
I had the same problem and it went back to redirect when i clicked the "Reset to Defaults" button. That seemed to have solved it.
#5
Problem is up a couple lines where it lists the options. Could you please do me a favor and test the attached patch?
#6
works for me, after the patch was successfully able to save with the first option.
That said, the redirect option doesn't seem to actually do anything, I could only get Splash working by choosing the Thickbox method.
#7
Committed to HEAD, will be in the next release.
#8
Automatically closed -- issue fixed for 2 weeks with no activity.