This would make the user interface more consistent (A Good Thing).. clicks on terms would always go to faceted searches.

For example, I currently have to rewrite links in my .tpl file like this:

[...]
<?php if ($terms): ?>
    <div class="terms"><?=t('Categories')?>: <?
        print str_replace("/taxonomy/term/", "/faceted_search/results/taxonomy%3A", $terms);
    ?></div>
  <?php endif; ?>
[...]

It would be nice if faceted_search would go into the ... theming(?)... function, and alter these links by itself. This way Tagadelic and others would lead the user into a faceted search if we so desired to.

Thanks for a great module.

Comments

David Lesieur’s picture

David Lesieur commented

Actually, there is a hook_term_path() function that modules can implement for this purpose. Taxonomy Redirect does that, but would need some changes to work for Faceted Search paths. I'd prefer to recommend another module rather than duplicating this functionality in Faceted Search.

It certainly would be a great way to glue Faceted Search and other modules together.

David Lesieur’s picture

David Lesieur commented

As always, help is welcome on the Taxonomy Redirect issue, if anyone can contribute patches or ideas. :)

janusman’s picture

janusman commented

Hmm; out of curiosity I installed Taxonomy Redirect, and it does work, I just set my vocabularies to point to:

faceted_search/results/taxonomy%3A!tid

And voilá. I didn't expect clicks on terms to further limit the search, I just expected them to launch a new search on that term... so then... this is "fixed"? =) Or how do you think this should work?

Perhaps the deeper question is: how do users expect it to work? Maybe I'm breaking the common mental model by launching a new search from these terms instead of restricting a current search.

For the working example see: http://enlinea.mty.itesm.mx

David Lesieur’s picture

David Lesieur commented

Glad to read this! Just tried it too... I previously had the wrong version of Taxonomy Redirect...

You're asking the right question about what users expect. The issue may or may not be fixed depending on that. :)

Contrary to what I just said in comment #1, it could make sense for Faceted Search to implement hook_term_path() itself... That way it could intelligently add the term to the current search criteria, rather than launch a new search. I don't see how external modules such as Taxonomy Redirect could do that without more intimate knowledge of Faceted Search's path format.

David Lesieur’s picture

David Lesieur commented

There is also a small issue with term hierarchies... Taxonomy Facets expects the full list of ancestors of the term to be specified on the URL (this ensures disambiguation for terms with multiple parents), but with Taxonomy Redirect we can't provide this list of ancestors. Without it Faceted Search results are still accurate, but the Current search and the Guided search blocks don't show the term's parents.

There's a patch that might help, but the term separator would have to be customizable (Taxonomy Facets uses '.' rather than '/' to separate ancestor term IDs).

Christefano-oldaccount’s picture

Christefano-oldaccount commented

Great discussion here. My thoughts are that Taxonomy Facets and Taxonomy Redirect are perfect together.

Perhaps the deeper question is: how do users expect it to work?

If someone clicks a taxonomy term and is redirected to the Faceted Search results, that results page simply needs to inform the user that the context has changed to a search. I need to use Faceted Search some more to have a better sense but don't think that Faceted Search does that very well on its own.

David Lesieur’s picture

David Lesieur commented

Currently, Faceted Search pages have their own title that says "Search" (with the current "dev" version admins may enter any title), and there is also a "Current search" block that shows the current search criteria and emphasizes the fact that a search is taking place. These elements could further be accentuated through theming. Any suggestions on how to improve on this?

Christefano-oldaccount’s picture

Christefano-oldaccount commented

Sorry, I made a typo that changed the message my comment. I meant to say that the results page needs to inform the user why the context has changed to a search.

I'll get the latest dev version. Being able to set the title to something other than "Search" should take care of my concern. Meanwhile, thank you for making Drupal search so much more elegant and powerful.

David Lesieur’s picture

David Lesieur commented
Status: Active » Fixed
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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