Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
tribalvibes opened this issue Sep 12, 2010 · 3 comments
Open

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

tribalvibes opened this issue Sep 12, 2010 · 3 comments

Comments

@tribalvibes
Copy link

No description provided.

@tribalvibes
Copy link
Author

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
Copy link

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
Copy link
Author

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.

swalterd added a commit to swalterd/soap4r that referenced this issue Dec 2, 2013
small fix in mimemessage.rb.  Changing each to each_line to add support for Ruby 1.9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants