Hello there is a small bug which disable validation if website is not in english :

on search_config.module at line 285,

replace :

if ($form_state['values']['form_id'] == 'search_admin_settings' && $form_state['values']['op'] == 'Save configuration') {

by :

if ($form_state['values']['form_id'] == 'search_admin_settings' && $form_state['values']['op'] == t('Save configuration')) {

CommentFileSizeAuthor
#6 search_config-565232-6.patch1.53 KBjbomb
#2 565232-search_config.patch785 bytesjweowu

Comments

dmitrit’s picture

dmitrit commented

Also re-index doesn't work:

 function search_config_validate($form, &$form_state) {
-  if ($form_state['values']['form_id'] == 'search_admin_settings' && $form_state['values']['op'] == 'Save configuration') {
+  if ($form_state['values']['form_id'] == 'search_admin_settings' && $form_state['values']['op'] == t('Re-index site')) {
+    drupal_goto('admin/settings/search/wipe');
+  }
+  elseif ($form_state['values']['form_id'] == 'search_admin_settings' && $form_state['values']['op'] == t('Save configuration')) {
     $post_values = $form_state['values'];
     $node_types = node_get_types('names');
jweowu’s picture

jweowu commented
Status: Active » Needs review
StatusFileSize
new 565232-search_config.patch785 bytes

Attaching a patch for #1

jweowu’s picture

jweowu commented
Status: Needs review » Reviewed & tested by the community

I'm going to mark this RTBC, as I've confirmed both of the fixes in dt_brume's solution, and my patch is identical to that.

ckng’s picture

ckng
Primary language English
commented

Patch confirmed works for both problems.

jbomb’s picture

jbomb commented

Using drupal_goto could prevent other modules from performing validation. The following patch moves the validation function from hook_search() to hook_form_alter().

search_config-703322.patch

jbomb’s picture

jbomb commented
StatusFileSize
new search_config-565232-6.patch1.53 KB

Re-rolling to include the solution from #1.

crutch’s picture

crutch commented

Works for me, thank you!

jbomb’s picture

jbomb commented
Status: Reviewed & tested by the community » Fixed

Committed to 6.x branch.

Status: Fixed » Closed (fixed)

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