Display mode: Redirect... error; Display mode field is required.

joncup - March 19, 2008 - 06:21
Project:Splash
Version:6.x-2.x-dev
Component:User interface
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

at admin/settings/splash/how
when i select Display mode: Redirect, i get a warning message: Display mode field is required.

#1

ionuts71 - April 7, 2008 - 22:56

I have the same problem... No solution?

#2

ricsch - December 9, 2008 - 21:35

i have the same problem, too.

#3

heronog - December 14, 2008 - 00:18

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

nivelula - May 27, 2009 - 21:08

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

seanr - June 5, 2009 - 20:24
Status:active» needs review

Problem is up a couple lines where it lists the options. Could you please do me a favor and test the attached patch?

AttachmentSize
splash.admin_.inc_.patch 579 bytes

#6

kscheirer - July 3, 2009 - 21:05
Status:needs review» reviewed & tested by the community

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

seanr - July 6, 2009 - 21:55
Status:reviewed & tested by the community» fixed

Committed to HEAD, will be in the next release.

#8

System Message - July 20, 2009 - 22:00
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.