[#5211] Importing event with category name > 80 char fails
Summary Importing event with category name > 80 char fails
Queue Kronolith
Queue Version HEAD
Type Bug
State Resolved
Priority 1. Low
Owners Jan Schneider <jan (at) horde (dot) org>
Requester patrick (dot) abiven (at) apitech (dot) fr
Created 04/04/2007 (403 days ago)
Due
Updated 04/02/2008 (39 days ago)
Assigned 03/14/2008 (58 days ago)
Resolved 04/02/2008 (39 days ago)
Attachments
Milestone
Patch

History
04/02/2008 Jan Schneider Comment #6
State ⇒ Resolved
Assigned to Jan Schneider
Reply to this comment
Alright, we cut them off at 80 chars now.
03/15/2008 Chuck Hagenbuch Comment #5 Reply to this comment
I would guess single digits.
03/14/2008 Jan Schneider Comment #4 Reply to this comment
I'm a bit hesitating to hardcode the limit inside the SQL code. It would of course be possible to cut off the categories at 80 chars. OTOH, how many people really do change the default SQL scheme that we provide with Kronolith?
03/14/2008 patrick (dot) abiven (at) apitech (dot) fr Comment #3 Reply to this comment
PostGreSQL

> Which database do you use? MySQL seems to truncate the category
> automatically.
03/14/2008 Jan Schneider Comment #2
State ⇒ Feedback
Taken from Karsten Fourmont
Reply to this comment
Which database do you use? MySQL seems to truncate the category automatically.
03/14/2008 Jan Schneider Queue ⇒ Kronolith
 
04/11/2007 Jan Schneider Assigned to Karsten Fourmont
State ⇒ Assigned
 
04/04/2007 patrick (dot) abiven (at) apitech (dot) fr Comment #1
Type ⇒ Bug
State ⇒ Unconfirmed
Priority ⇒ 1. Low
Summary ⇒ Importing event with category name > 80 char fails
Queue ⇒ SyncML
Reply to this comment
Hello
In the kronolith_events table, the field event_category is defined as VARCHAR(80). But in kronolith/lib/Driver.php, the setCategory() function doesn't check the size of $category. So importing or synchronizing events having a category name greater than 80 char fails (DB Error: unknown error).
A workaround is to reduce the $category to 80 char.

Regards
Patrick