simple_bridge icon indicating copy to clipboard operation
simple_bridge copied to clipboard

form input tag's name attribute should handle all of CDATA

Open jbothma opened this issue 13 years ago • 2 comments
trafficstars

A form input tag's name attribute should be able to contain any CDATA characters. At least according to http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd

Currently quite a limited (albeit the vastly most common) set of characters are supported by https://github.com/nitrogen/simple_bridge/blob/master/src/simple_bridge_request_wrapper.erl#L97 but it breaks when you want to parse an array of things like Chicago Boss model IDs like the following because of the hyphen

<input type... name="items[word-3]" />
<input type... name="items[word-5]" />

Of course there are many workarounds and I'm not even providing a patch, but I thought I'd log the issue until I can provide a patch ans see if there's any interest in such a fix anyway.

jbothma avatar Aug 21 '12 14:08 jbothma

A solid recommendation. If it's breaking the standard, it should definitely be fixed.

This should be an easy fix of the regex, and if you want to do it so that you're included in the repo history, go right ahead and submit a pull request.

choptastic avatar Aug 26 '12 17:08 choptastic

Cool. I won't get to it for a few weeks (that project's on the backburner) so go ahead if you have time, otherwise I'll be back when I can.

jbothma avatar Aug 26 '12 18:08 jbothma