--- contribs/weblinks_checker/weblinks_checker.admin.inc orig 2009-07-11 18:37:33.000000000 +0100 +++ contribs/weblinks_checker/weblinks_checker.admin.inc 2009-07-28 08:22:17.000000000 +0100 @@ -276,14 +276,14 @@ function weblinks_checker_settings() { */ function weblinks_checker_settings_validate($form, &$form_state) { // Check that the number of links to include is in the range 1 to 50. - $limit = $form_state['#values']['weblinks_checker_limit']; - if (($limit < 1 || $limit > 50) && $form_state['#values']['weblinks_checker_enabled'] <> 0) { + $limit = $form_state['values']['weblinks_checker_limit']; + if (($limit < 1 || $limit > 50) && $form_state['values']['weblinks_checker_enabled'] <> 0) { form_set_error('weblinks_checker_limit', t("You entered @limit for 'links to include'. Please use a number in the range 1 to 50.", array('@limit' => ($limit == '' ? t('nothing') : $limit)))); } // If unpublishing links in error, check that the value of N days or N crons is in the range 1 to 99. - $unpublish_after = $form_state['#values']['weblinks_error_handling']['weblinks_checker_unpublish_after']; - if ($form_state['#values']['weblinks_error_handling']['weblinks_checker_unpublish'] <> 0 && ($unpublish_after < 1 || $unpublish_after > 99)) { + $unpublish_after = $form_state['values']['weblinks_checker_unpublish_after']; + if ($form_state['values']['weblinks_checker_unpublish'] <> 0 && ($unpublish_after < 1 || $unpublish_after > 99)) { form_set_error('weblinks_checker_unpublish_after', t("You entered @unpublish_after for 'unpublish after'. Please use a number in the range 1 to 99.", array('@unpublish_after' => ($unpublish_after == '' ? t('nothing') : $unpublish_after)))); } @@ -291,7 +291,8 @@ function weblinks_checker_settings_valid return FALSE; } - // If the 'N days' or 'N crons' option has been changed, remove the existing last_status_info values where a value has been set (as they would now give wrong results) + // If the 'N days' or 'N crons' option has been changed, reset the existing last_status_info + // values to zero where a value has been set, as they would now give wrong results. $unpublish = $form['weblinks_error_handling']['weblinks_checker_unpublish']; if ($unpublish['#value'] != $unpublish['#default_value']) { db_query('UPDATE {weblinks} SET last_status_info=0 WHERE last_status_info > 0');