Mike Breen's profile

Mike Breen commented at 8:13 PM
@Mislav - Thanks for the feedback. I guess it would be better to handle anything that has a to_a:
```
ids.inject([]) {|m,o| m << (o.respond_to? :to_a ? o.to_a : o)...
```
Mike Breen created the ticket at 5:58 PM
This patch will allow you to pass a Range of IDs to find:
```
Person.find(1..5)
Person.find(1, 3..8)
```

1211 reset_counter_cache was updated in Ruby on Rails

- Tag changed from active_record, counter_cache, patch to active_record, bugmash, counter_cache, patch
by Mike Breen at 2:15 PM

Mike Breen commented at 12:35 AM
@eloy Is there a reason we wouldn't put this in master also?

Mike Breen commented at 12:24 AM
Is there a reason we can't get this patch into 2-3-stable. It applies cleanly.

Mike Breen commented at 1:24 AM

sorry, this does not apply cleanly to master:

error: patch failed: actionpack/test/template/form_options_helper_test.rb:681
error: actionpack/test/template/form...
- Assigned user set to Pratik
- Tag changed from actionview, enhancement, formoptionshelper, patch to actionview, bugmash, enhancement, formoptionshelper, patch
by Mike Breen at 1:12 AM
+1 for this change and I have verified that this patch applies cleanly to master.

Mike Breen commented at 1:18 AM
+1 verified Elad's patch applies cleanly to master. all tests are green