Bart ten Brinke's profile

Bart ten Brinke commented at 12:07 AM
Doh :) Repackaged. More tests + added an ActiveResource.enable_delfate option to set the headers for you.

Bart ten Brinke commented at 10:13 PM
Is zlib packed with Rails 2.3? Because that will allow for some simplification of the patch. Too bad it's to late for 2.3.
Bart ten Brinke commented at 10:09 PM

Last line of the ticket:

To enable add this to your ActiveResource base class via a Mixin: base.headers['Accept-Encoding'] = 'deflate'

Bart ten Brinke commented at 9:25 PM
Anyone got a nice idea how to implement this in the active support rake file?
```
if gem 'libxml-ruby', '=0.9.7'
  XmlMini.backend = 'LibXML'
  run_entire_testsuit...
```
Bart ten Brinke commented at 9:12 PM
Ok, but the behavior is still different :). I didn't look into the Nokogiri implementation. I do agree with you that this test needs to be automated somehow, fo...
Bart ten Brinke commented at 9:00 PM
If you use XmlMini.backend = 'LibXML' you will only get 1 failure: test_default_is_rexml, which is obvious.
Bart ten Brinke commented at 8:54 PM
Well, if you enable the nokogiri gem as default:
```
XmlMini.backend = 'Nokogiri'
```
and then run the rails test, you'll see a lot of issues where your nokogiri imp...
Bart ten Brinke commented at 8:27 PM
If you run the entire rails test suite you'll get even more errors. Looks like the behaviour is different from Rexml. The LibXML implementation does not have th...

Bart ten Brinke commented at 8:24 PM

I'm getting failures if I actually enable nokogiri:

1) Error: test_array_with_multiple_entries_from_xml(HashToXmlTest): TypeError: can't convert String into Int...

- Tag changed from activeresource, patch to activeresource, patch, verified
by Bart ten Brinke at 7:12 PM
It would be really nice if this went in 2.3, together with the XML parser upgrades. It would really make activeresource usable for a lot of people.