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-01 14:49) wrote:
1) Looks like some style definitions are missing from the patch...I guessed
as to what they should be, but maybe a diff against your screen.css would
be helpful.
2) The Ansel_Report stuff that your using for the "Claim" functionality is
broken...perhaps there are missing patches against Ansel_Report? The
issues I see are:
- Currently, Ansel_Report *always* sends to the configured report-to
address. It looks to me like the face code attempts to pass Ansel_Report
an identity object, that I assume it's to get an email address from
(assuming it's available). I'm assuming there is also some missing checks
for what to do if there is no valid email address in the user's identity?
- There is no Ansel_Report#setTitle method, but the image code
attempts to call it. Again, maybe from a missing patch?