[Tickets #6418] Re: Face recognition

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/6418
-----------------------------------------------------------------------
 Ticket             | 6418
 Updated By         | Michael Rubinsky <mrubinsk at horde.org>
 Summary            | Face recognition
 Queue              | Ansel
 Type               | Enhancement
 State              | Assigned
 Priority           | 1. Low
 Milestone          |
 Patch              |
 Owners             | Michael Rubinsky
-----------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2008-06-06 20:50) wrote:

For  getRecentImages() we must allow the galleries selection in a
> direct SQL query. Of course is impossible to run a SQL query with
> thousands of gallery_id in the IN(...) clause. I attaching a patch,
> maybe a little hacky but is all I can do for now.

It might be a little hacky, but I don't think it's all that bad...we
already have a few other places in Ansel_Storage where we directly access
the ansel_shares table either because it's necessary or for performance
reasons.  Of course, we'll also have to remove the code in the beginning of
that method that fills the galleries array if it's empty...or your new code
will never execute ;)

Thanks.