Tickets :: Comment on [#6418] Face recognition

* Your Email Address
* Spam protection	Enter the letters below: . ..___. ..__..__ \|_/ [__ \| \|\| \|\| \ \| \\| \|__\|\|__\\|__/
Comment	> 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?
Attachment
Watch this ticket