seastar-ss / rest-client

Automatically exported from code.google.com/p/rest-client
Apache License 2.0
0 stars 0 forks source link

RESTView Patch: Treat +xml media type suffix as XML & invoke URL on Enter #124

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I've patched RESTView so it does the following:

(1) Invoke URL after hitting <ENTER> in URL combo box. After hitting <ENTER> in 
the URL combo box, it'll invoke the HTTP request. It seems that the code was 
there but was commented out because it didn't work correctly. I think I've 
fixed all the bugs associated with it but I'm sure subwiz can tell me if 
there's something else that needs fixing :-)

(2) Treat responses that have a content type suffix of "+xml" as XML. Per 
RFC3023, a content type that has a "+xml" suffix should be treated as a XML 
document.

Please feel free to patch the SVN code with this. The contributed code is 
licensed under the Apache Software License 2.0.

Original issue reported on code.google.com by json.dow...@gmail.com on 23 Jan 2011 at 8:03

Attachments:

GoogleCodeExporter commented 9 years ago
Wow! Thanks for the effort! I will test and incorporate it today evening.

Original comment by subwiz on 24 Jan 2011 at 3:18

GoogleCodeExporter commented 9 years ago
I incorporated the (2) of your change in trunk. (1) gives me 
NullPointerException when I opened from a saved request.

Original comment by subwiz on 26 Jan 2011 at 3:47

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Thanks for letting me know about the problem. I approached the problem from a 
slightly different angle this time. Hopefully, this time it's bug free.

Original comment by json.dow...@gmail.com on 26 Jan 2011 at 9:15

Attachments:

GoogleCodeExporter commented 9 years ago
Patch incorporated. Thanks once again for the contribution :-)

Original comment by subwiz on 30 Jan 2011 at 6:02

GoogleCodeExporter commented 9 years ago
svn r488 has the fix.

Original comment by subwiz on 30 Jan 2011 at 6:29