6.0.0-RC7
6/27/26

[#5739] patch for ulaform 'set' field
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 9/25/07 (6850 days ago)
Due
Updated 10/7/07 (6838 days ago)
Assigned 9/26/07 (6849 days ago)
Resolved 10/7/07 (6838 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
4512 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.
519 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.
243 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?
501 Jan Schneider Comment #5 Reply to this comment
That sounds like a good compromise.
426 vilius (at) lnk (dot) lt Comment #4 Reply to this comment
What about imploding with \n ?
325 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.
3610 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?
598 vilius (at) lnk (dot) lt Comment #1
Priority ⇒ 1. Low
State ⇒ Unconfirmed
New Attachment: sql.php.patch Download
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.

Saved Queries