Tickets :: [#5739] patch for ulaform 'set' field

6.0.0-beta1

11/9/25

Summary	patch for ulaform 'set' field
Queue	Ulaform
Type	Bug
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	vilius (at) lnk (dot) lt
Created	09/25/2007 (6620 days ago)
Due
Updated	10/07/2007 (6608 days ago)
Assigned	09/26/2007 (6619 days ago)
Resolved	10/07/2007 (6608 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	Yes

10/07/2007 12:39:45 PM	Jan Schneider	Comment #8 Assigned to Jan Schneider State ⇒ Resolved	Reply to this comment
On the other thought, Jan's comment about comma in a field is irrelevant because you cannot use commas in any of fields. Commas are used to separate values during creation of the checkbox field. You are right. I committed you patch unchanged.

10/03/2007 09:25:51 PM	vilius (at) lnk (dot) lt	Comment #7	Reply to this comment
On the other thought, Jan's comment about comma in a field is irrelevant because you cannot use commas in any of fields. Commas are used to separate values during creation of the checkbox field.

10/03/2007 03:49:24 PM	Chuck Hagenbuch	Comment #6	Reply to this comment
Seems to me like serializing or imploding with an argument should be options on the field. Can you provide an updated patch?

10/01/2007 01:02:50 PM	Jan Schneider	Comment #5	Reply to this comment
That sounds like a good compromise.

10/01/2007 06:42:42 AM	vilius (at) lnk (dot) lt	Comment #4	Reply to this comment
What about imploding with \n ?

09/27/2007 05:28:32 AM	vilius (at) lnk (dot) lt	Comment #3	Reply to this comment
Well, in my case after everyone filled the form I just export MySQL table (through phpMyAdmin) to Excel format and send to responsible person. So serializing is not an option too. I think data in table should be as clean as possible.

09/26/2007 10:40:36 PM	Jan Schneider	Comment #2 State ⇒ Feedback	Reply to this comment
This won't work as soon as one of the fields contains a comma. Wouldn't it be better to serialize the array?

09/25/2007 08:04:59 AM	vilius (at) lnk (dot) lt	Comment #1 Priority ⇒ 1. Low State ⇒ Unconfirmed New Attachment: sql.php.patch Queue ⇒ Ulaform Summary ⇒ patch for ulaform 'set' field Type ⇒ Bug	Reply to this comment
'set' field returns an array, so implode to allow data to be stored in one database column. Also give more bytes.