I just tried 1.3 and I still see the category name of which I have selected the terms.

tried the following: re-enable the module, turn off the settings and back on, cleared cache.

Thanks

CommentFileSizeAuthor
#6 search_config_5x.patch1.51 KBNaX

Comments

canen’s picture

Know issue. If all terms it a category are chosen not be displayed the empty form shows up. I'll try and do a fix soon.

nathanraft’s picture

Great thanks!

Lityi’s picture

HI canen,

could you please be so kind to give us an time estimate ?

Thank you for your great work!

Best regards,
Tibor

NaX’s picture

Status: Active » Needs review

I fixed this in the 5.x-1.0 (search_config.module,v 1.8) with the following.

Change line 106 - 124 to in search_config_form_alter() function

            // FIXME: What about multiple hierarchy categories?
            foreach ($taxonomy as $vocab => $term) {
              foreach ($term as $k => $v) {
                if(in_array($k, $terms)) {
                  unset($taxonomy[$vocab][$k]);
                }
              }
              // Remove empty vocabs
              if (count($taxonomy[$vocab]) == 0) {
                unset($taxonomy[$vocab]);
              }
            }

            if (count($taxonomy) == 0) {
              unset($form['advanced']['category']);
            }
            else {
              // Taxonomy box:
              $form['advanced']['category'] = array(
                '#type' => 'select',
                '#title' => t('Only in the category(s)'),
                '#prefix' => '<div class="criterion">',
                '#size' => 10,
                '#suffix' => '</div>',
                '#options' => $taxonomy,
                '#multiple' => TRUE,
              );
            }
canen’s picture

NaX ,

Thanks. Would it be too much to ask for a patch? If it is I can just make the fix manually.

Thanks again.

NaX’s picture

StatusFileSize
new1.51 KB

no problem

canen’s picture

New release http://drupal.org/node/145818. Thanks again for the patch.

canen’s picture

canen’s picture

Status: Needs review » Closed (fixed)
canen’s picture

I'll actually understand the release system one of these days but for now botched releases all around. Ignore the last two links and use the two latest releases. These actually have some changes in them.