diff options
| author | webchick <webchick@24967.no-reply.drupal.org> | 2012-04-28 23:20:21 -0700 |
|---|---|---|
| committer | webchick <webchick@24967.no-reply.drupal.org> | 2012-04-28 23:20:21 -0700 |
| commit | 9afbc13d6bb62ee93e4ceabc6d9e22b61dcd7e91 (patch) | |
| tree | 781fed4c8c5d10f7423e8c31aae41a5e979a45f9 /modules/search | |
| parent | a7d43f3460a97ad3c9b1cb23f7d374a4994837a3 (diff) | |
| download | brdo-9afbc13d6bb62ee93e4ceabc6d9e22b61dcd7e91.tar.gz brdo-9afbc13d6bb62ee93e4ceabc6d9e22b61dcd7e91.tar.bz2 | |
Issue #1527988 by Niklas Fiekas: Fixed Missing or legacy number validation.
Diffstat (limited to 'modules/search')
| -rw-r--r-- | modules/search/search.admin.inc | 3 | ||||
| -rw-r--r-- | modules/search/search.test | 11 |
2 files changed, 13 insertions, 1 deletions
diff --git a/modules/search/search.admin.inc b/modules/search/search.admin.inc index fda14ee7b..a609485ac 100644 --- a/modules/search/search.admin.inc +++ b/modules/search/search.admin.inc @@ -95,7 +95,8 @@ function search_admin_settings($form) { '#default_value' => variable_get('minimum_word_size', 3), '#size' => 5, '#maxlength' => 3, - '#description' => t('The number of characters a word has to be to be indexed. A lower setting means better search result ranking, but also a larger database. Each search query must contain at least one keyword that is this size (or longer).') + '#description' => t('The number of characters a word has to be to be indexed. A lower setting means better search result ranking, but also a larger database. Each search query must contain at least one keyword that is this size (or longer).'), + '#element_validate' => array('element_validate_integer_positive'), ); $form['indexing_settings']['overlap_cjk'] = array( '#type' => 'checkbox', diff --git a/modules/search/search.test b/modules/search/search.test index 26c663e71..eeb6bf21b 100644 --- a/modules/search/search.test +++ b/modules/search/search.test @@ -1461,6 +1461,17 @@ class SearchConfigSettingsForm extends DrupalWebTestCase { $this->assertText(t('The index will be rebuilt')); $this->drupalGet('admin/config/search/settings'); $this->assertText(t('There is 1 item left to index.')); + + // Test that the form saves with the default values. + $this->drupalPost('admin/config/search/settings', array(), t('Save configuration')); + $this->assertText(t('The configuration options have been saved.'), 'Form saves with the default values.'); + + // Test that the form does not save with an invalid word length. + $edit = array( + 'minimum_word_size' => $this->randomName(3), + ); + $this->drupalPost('admin/config/search/settings', $edit, t('Save configuration')); + $this->assertNoText(t('The configuration options have been saved.'), 'Form does not save with an invalid word length.'); } /** |