Tickets :: [#8523] Photo attribute is corrupted after a move or a copy

6.0.0-beta6

4/10/26

Summary	Photo attribute is corrupted after a move or a copy
Queue	Turba
Queue Version	2.3.1
Type	Bug
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	ryu (at) ryux (dot) org
Created	8/24/09 (6073 days ago)
Due
Updated	1/13/10 (5931 days ago)
Assigned	9/30/09 (6036 days ago)
Resolved	10/1/09 (6035 days ago)
Github Issue Link
Github Pull Request
Milestone	2.3.3
Patch	No

01/13/2010 12:12:55 AM	CVS Commit	Comment #15	Reply to this comment
Changes have been made in Git for this ticket: MFB: Avoid notices (~~Bug #8523~~). http://git.horde.org/diff.php/turba/lib/View/Browse.php?rt=horde-git&r1=31c0700959386a1218a8e1f4d88641197fada63f&r2=7957144af4f605e35d7a2a0bd835c937b4a0a0b6

01/13/2010 12:12:54 AM	CVS Commit	Comment #14	Reply to this comment
Changes have been made in Git for this ticket: Fix ~~bug #8523~~ - Photo attribute is corrupted after a move or a copy Signed-off-by: Jan Schneider <jan@horde.org> http://git.horde.org/diff.php/turba/lib/View/Browse.php?rt=horde-git&r1=0a70ce52cdd66d0213a84aa542dfe81eecb42fd3&r2=31c0700959386a1218a8e1f4d88641197fada63f

10/01/2009 03:04:25 PM	CVS Commit	Comment #13	Reply to this comment
Changes have been made in Git for this ticket: MFB: Avoid notices (~~Bug #8523~~). http://git.horde.org/diff.php/turba/lib/View/Browse.php?rt=horde-hatchery&r1=31c0700959386a1218a8e1f4d88641197fada63f&r2=7957144af4f605e35d7a2a0bd835c937b4a0a0b6

10/01/2009 03:04:24 PM	CVS Commit	Comment #12	Reply to this comment
Changes have been made in Git for this ticket: Fix ~~bug #8523~~ - Photo attribute is corrupted after a move or a copy Signed-off-by: Jan Schneider <jan@horde.org> http://git.horde.org/diff.php/turba/lib/View/Browse.php?rt=horde-hatchery&r1=0a70ce52cdd66d0213a84aa542dfe81eecb42fd3&r2=31c0700959386a1218a8e1f4d88641197fada63f

10/01/2009 10:02:46 AM	Jan Schneider	Milestone ⇒ 2.3.3

10/01/2009 10:02:34 AM	Jan Schneider	Comment #11 State ⇒ Resolved	Reply to this comment
Committed, thanks for the patch!

10/01/2009 10:01:54 AM	CVS Commit	Comment #10	Reply to this comment
Changes have been made in Git for this ticket: MFB: Avoid notices (~~Bug #8523~~). http://git.horde.org/diff.php/turba/lib/View/Browse.php?rt=horde-hatchery&r1=31c0700959386a1218a8e1f4d88641197fada63f&r2=7957144af4f605e35d7a2a0bd835c937b4a0a0b6

10/01/2009 09:57:35 AM	CVS Commit	Comment #9	Reply to this comment
Changes have been made in CVS for this ticket: Avoid notices (~~Bug #8523~~). http://cvs.horde.org/diff.php/turba/lib/Views/Browse.php?rt=horde&r1=1.12.2.10&r2=1.12.2.11&ty=u

10/01/2009 09:53:42 AM	CVS Commit	Comment #8	Reply to this comment
Changes have been made in CVS for this ticket: MFH: Fix ~~bug #8523~~ - Photo attribute is corrupted after a move or a copy http://cvs.horde.org/diff.php/turba/lib/Views/Browse.php?rt=horde&r1=1.12.2.9&r2=1.12.2.10&ty=u

10/01/2009 09:50:49 AM	CVS Commit	Comment #7	Reply to this comment
Changes have been made in Git for this ticket: Fix ~~bug #8523~~ - Photo attribute is corrupted after a move or a copy Signed-off-by: Jan Schneider <jan@horde.org> http://git.horde.org/diff.php/turba/lib/View/Browse.php?rt=horde-hatchery&r1=0a70ce52cdd66d0213a84aa542dfe81eecb42fd3&r2=31c0700959386a1218a8e1f4d88641197fada63f

10/01/2009 09:46:50 AM	Jan Schneider	Deleted Original Message

09/30/2009 07:56:15 PM	ryu (at) ryux (dot) org	Comment #6 New Attachment: 0002-Fix-bug-8523-Photo-attribute-is-corrupted-after-a.patch	Reply to this comment
This is my new patch. WIth only Browser that changes. It cancel & replace the previous. Hope is ok !

09/30/2009 03:03:57 PM	ryu (at) ryux (dot) org	Comment #5	Reply to this comment
Please change the patch so that instead of introducing a new parameter to getValue(), it checks whether the attribute is of type image in Browser.php, and uses the 'data' value in that case. Ok, I see. I do that, and I repost ! Thanks

09/30/2009 10:18:03 AM	Jan Schneider	Comment #4 Taken from Horde Developers State ⇒ Feedback	Reply to this comment
Please change the patch so that instead of introducing a new parameter to getValue(), it checks whether the attribute is of type image in Browser.php, and uses the 'data' value in that case.

09/30/2009 07:03:39 AM	ryu (at) ryux (dot) org	Comment #3 New Attachment: 0001-Fix-bug-8523-Photo-attribute-is-corrupted-after-a.patch	Reply to this comment
Ok, I finally found time to search where was the problem, and I found it ! It's the getValue() function which return an array for 'image' type attribute. But, if we want to copy this attribute, we have to return the raw data. So I join a patch to fix this thicket in this post ! (This patch is for git version) Thanks for comments.

08/26/2009 03:27:45 PM	Jan Schneider	Comment #2 State ⇒ Assigned Assigned to Horde Developers Assigned to Jan Schneider	Reply to this comment
Confirmed, not limited to LDAP.

08/24/2009 07:03:16 AM	ryu (at) ryux (dot) org	Comment #1 Priority ⇒ 1. Low State ⇒ Unconfirmed Patch ⇒ No Milestone ⇒ Summary ⇒ Photo attribute is corrupted after a move or a copy Type ⇒ Bug Queue ⇒ Turba	Reply to this comment
How to reproduce : - Create a contact in a ldap addessbook 1 - Add a photo (jpegPhoto) to this contact => Photo OK - Move this contact to the ldap addressbook 2 => Photo doesn't display, file is corrupted ! I try to find the problem, with no success until now...