[cvs] commit: ansel/lib Tags.php ansel/lib/Views Results.php

[Michael Rubinsky]

mrubinsk    2007-05-08 13:24:49 EDT

  Modified files:
    lib                  Tags.php 
    lib/Views            Results.php 
  Log:
  - Add a Ansel_Tags_Search::save() method and require explicit saving of selected tags to the session.
  - Add a check to Ansel_Tags_Search::getRelatedTags() so we don't return related tags that would return zero search results due to no permissions.
  Note: This is the only way I could come up with to prevent "no results" tags from being displayed, but this could be a performance issue due to the multiple queries that need to be run. I'm thinking maybe limiting the number of related tags that are returned (pre-permissions check) to something like the top 5 to at least lessen the potential impact. Thoughts?
  
  Revision  Changes    Path
  1.30      +31 -9     ansel/lib/Tags.php
  1.6       +3 -1      ansel/lib/Views/Results.php

  Chora Links:
  http://cvs.horde.org/diff.php/ansel/lib/Tags.php?r1=1.29&r2=1.30&ty=u
  http://cvs.horde.org/diff.php/ansel/lib/Views/Results.php?r1=1.5&r2=1.6&ty=u