[Tickets #6982] Re: Optimized Tags

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/6982
------------------------------------------------------------------------------
  Ticket             | 6982
  Updated By         | Michael Rubinsky <mrubinsk at horde.org>
  Summary            | Optimized Tags
  Queue              | Ansel
  Type               | Enhancement
  State              | Feedback
  Priority           | 1. Low
  Milestone          | 1.0
  Patch              |
  Owners             | Michael Rubinsky
------------------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2008-07-02 16:49) wrote:

> Actually we don't need to know the actual number of images/galleries
> tag count. Is used only as a flag if the gallery/image has tags or
> not to save later queries. So this can be a boolean value.

Huh?  Your saving the actual number of tags in ansel_tags.tag_count  
and then your using the value of that field directly in  
Ansel_Tags::readTags() and Ansel_Tags::listTagInfo().  With your  
changes, once any tags are added/removed from an image or gallery, the  
tag counts will never be correct because the value in the  
ansel_tags.tag_count field is only taking into account the number of  
times that tag is used in _either_ the images table *or* the galleries  
table, when it needs to be both.