Added optional support for other http methods (for #280)

prototypejs / prototype

Prototype JavaScript framework

http://prototypejs.org/

Other

3.53k stars 639 forks source link

Added optional support for other http methods (for #280) #321

Closed qsiebers closed 7 years ago

qsiebers commented 7 years ago

This is a patch I created for a product a while ago which can serve as a starting point for work on #280.

Added the allMethods option, which defaults to false for backward compatibility
The _method parameter is added when allMethods is false
The request body is only set when the method is not GET
The request content type is set when the method is not GET

savetheclocktower commented 7 years ago

Now that we've dropped support for old browsers I'm happy to be bold about this change and abandon the old behavior entirely. Thanks for doing the work on this.

savetheclocktower commented 7 years ago

Addressed this in 5140d0c8e1401f962e040d238e9f7510b9ed565b.