search

spox / soap4r-spox

Modified soap4r library to run on Ruby 1.9
Other
56 stars 100 forks source link

lib/soap/property.rb stomps inject in ruby 1.9 #3

Open tribalvibes opened 14 years ago

tribalvibes commented 14 years ago

line 323 unless Enumerable.instance_methods.include?('inject') should be :inject

As is it replaces Enumerable.inject, it stomps on the ruby 1.9 behavior which should be [].inject => nil rather than Argument Error (0 for 1)

rubyjedi commented 14 years ago

Truth be told, because that was part of a monkey-patch for Ruby 1.6, I just commented out the whole monkey patch in my fork (rubyjedi-soap4r). I'll probably remove it in a later revision.

tribalvibes commented 14 years ago

Sounds right. Took some head scratching to track it down as it was causing plugin loading which depends on active_support Enumerable#sum to bonk in a rails app.