#4603 [PATCH] Allow to set dataType in jquery-ujs - Ruby on Rails

Type	To find
responsible:me	tickets assigned to you
tagged:"@high"	tickets tagged @high
milestone:next	tickets in the upcoming milestone
state:invalid	tickets with the state invalid
created:"last week"	tickets created last week
sort:number, importance, updated	tickets sorted by #, importance or updated
Combine keywords for powerful searching.
Use advanced searching »

This project is archived and is in readonly mode.

#4603 ✓invalid

[PATCH] Allow to set dataType in jquery-ujs

Reported by Piotr Sarnacki | May 15th, 2010 @ 04:52 PM

Currently there is no way to set dataType in jquery-ujs, it's set to 'script' by default. It would be nice to have ability to do that by setting data-type HTML attribute with default set to 'script'.

allow-to-set-datatype-with-data-type-html-attribute.patch 1.3 KB

Comments and changes to this ticket

Santiago Pastorino May 16th, 2010 @ 12:14 AM
- Tag changed from “jquery-ujs patch” to “jquery-ujs patch, bugmash”
Santiago Pastorino May 16th, 2010 @ 12:16 AM
- Milestone cleared.
- Tag changed from “jquery-ujs patch, bugmash” to “jquery-ujs patch”
- State changed from “new” to “open”
+1
Santiago Pastorino May 16th, 2010 @ 12:18 AM
Hehe, +1 for the idea the patch is wrong
Piotr Sarnacki May 16th, 2010 @ 12:26 AM
What's wrong with it? I've used it in my app, works great :)
Santiago Pastorino May 16th, 2010 @ 12:27 AM
- Milestone cleared.
- State changed from “open” to “invalid”
Sorry i should be a bit asleep this is for http://github.com/rails/jquery-ujs
You should report this kind of issues in this project issue page.
Also is already reported http://github.com/rails/jquery-ujs/issues#issue/3 you can provide your patch there.
Thanks.
Santiago Pastorino May 16th, 2010 @ 12:30 AM
I've added your patch to the comments there
Piotr Sarnacki May 16th, 2010 @ 08:54 AM
I think I was a bit asleep too ;-) I haven't noticed issue 3 and added my own: http://github.com/rails/jquery-ujs/issues#issue/7, but then I asked Jose Valim about where to put such tickets. He said that here. Anyway, sorry for the confusion :)