Hi,
In views_slideshow_cycle_views_slideshow_options_form_validate() the fieldname used as a key for the form_error() is not the good one so the wrong field is highlighted when the validation fails.

if (!is_numeric($form_state['values']['style_options']['views_slideshow_cycle']['timeout'])) {
  form_error($form['views_slideshow_cycle']['speed'], t('!setting must be numeric!', array('timeout')));
}

should be

if (!is_numeric($form_state['values']['style_options']['views_slideshow_cycle']['timeout'])) {
  form_error($form['views_slideshow_cycle']['timeout'], t('!setting must be numeric!', array('timeout')));
}

Patch to follow.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vbouchet’s picture

Status: Active » Needs review
FileSize
1.09 KB
vbouchet’s picture

Status: Needs review » Fixed

This has been fixed via the following commit.

Status: Fixed » Closed (fixed)

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