Small bug:

--- apachesolr_search.module	27 Aug 2009 20:07:02 -0000	1.1.2.6.2.111.2.10
+++ apachesolr_search.module	28 Aug 2009 17:44:18 -0000
@@ -991,7 +991,7 @@
 function apachesolr_search_make_default_submit($form, &$form_state) {
   // We use variable_get() instead of the form values so as to also handle reset to defaults.
   if ($form_state['values']['apachesolr_search_default_previous'] != variable_get('apachesolr_search_make_default', 0) ||
-      $form_state['values']['apachesolr_search_taxonomy_previous'] != variable_get('apachesolr_search_make_default', 0)) {
+      $form_state['values']['apachesolr_search_taxonomy_previous'] != variable_get('apachesolr_search_taxonomy_links', 0)) {
     // Take account of path changes
     menu_rebuild();
   }

Comments

robertdouglass’s picture

Looking. Want Peter to confirm.

janusman’s picture

Bumping this... IMO it just fixes an obvious typo =)

robertdouglass’s picture

Yeah, I misread the patch before. Dumb typo.

robertdouglass’s picture

Version: 6.x-2.x-dev » 5.x-2.x-dev
Status: Needs review » Patch (to be ported)

Committed to 6.1 and 6.2. Marking to be ported for Claudiu.

janusman’s picture

yay! I wonder if we could get a diff formatter into drupal.org so that changes within lines are highlighted? Maybe this could potentially save thousands of man-hours =)

claudiu.cristea’s picture

Status: Patch (to be ported) » Fixed
StatusFileSize
new1.26 KB

Fixed in #317572.

Attached the patch.

Status: Fixed » Closed (fixed)

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