6.0.0-beta1
7/6/25

[#8925] image_faces missing in lib/Ansel.php
Summary image_faces missing in lib/Ansel.php
Queue Ansel
Queue Version FRAMEWORK_3
Type Bug
State Resolved
Priority 1. Low
Owners mrubinsk (at) horde (dot) org
Requester horde (at) albasoft (dot) com
Created 03/19/2010 (5588 days ago)
Due
Updated 03/25/2010 (5582 days ago)
Assigned 03/19/2010 (5588 days ago)
Resolved 03/25/2010 (5582 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
03/25/2010 12:37:31 AM Michael Rubinsky Comment #3
State ⇒ Resolved
Reply to this comment
Fixed by changing the schema to add a default value of zero.

This value is always directly changed by Ansel_Faces code, and never 
by the Ansel_Image class, so writing it in Ansel_Image#save results in 
wasted resources.

Thanks for the catch, though :)
03/19/2010 05:11:28 PM Jan Schneider State ⇒ Assigned
Assigned to Michael Rubinsky
 
03/19/2010 02:34:07 PM horde (at) albasoft (dot) com Comment #1
Priority ⇒ 1. Low
New Attachment: ansel-h3-1.1-image_faces.patch Download
Patch ⇒ Yes
Milestone ⇒
Summary ⇒ image_faces missing in lib/Ansel.php
Type ⇒ Bug
State ⇒ Unconfirmed
Queue ⇒ Ansel
Reply to this comment
When using database backend, ansel defines 'image_faces' on 
'ansel_images' table as not null (ansel/scripts/sql/ansel.sql), but it 
is missing at lib/Ansel.php 'save' function. It results in errors when 
inserting a new image.

Something like the attached patch solves the problem.

Saved Queries