Tom Ward's profile

Tom Ward commented at 3:14 PM
I've encountered this issue too. Giving a warning is the least I'd expect. The patch looks good to me, so +1.

Tom Ward commented at 11:20 AM
According to http://stackoverflow.com/questio... it appears this may be caused by a VERSION environment variable already being set.
- Tag changed from activerecord, migration to activerecord, migration
by Tom Ward at 10:51 AM

Rollo,

Could you tell me what OS/Database/Version of Ruby you are using? It will help in trying to recreate the issue.

Tom

Tom Ward commented at 11:28 PM
Coercing blank strings to nil for non-string columns makes a whole load of sense to me. A definite +1.

- Tag changed from dependencies, eager_loading to dependencies, eager_loading, patch
by Tom Ward at 1:29 PM
I've written a patch (with tests) which sorts the globbed files. Having thought about this in some depth, I'm convinced that eager loading should load parents ...
Tom Ward created the ticket at 12:12 PM
The problem

I have the following files:

app/models/zoo.rb
```
class Zoo < ActiveRecord::Base
  include HippoManangement
end
```
app/models/zoo/hippo_management.rb
...

Tom Ward commented at 2:52 PM
Might it be nicer to automatically provide a helper method/instance variable to indicate whether the first or last records in the collection are being passed?

- Tag set to activerecord, patch, tiny
by Tom Ward at 3:00 PM
I've created a patch with Michalis' change (for which he should get the credit), and tested against sqlite3 and mysql. As this fixes a problem in a non-core ad...

Tom Ward commented at 10:55 AM
Can you provide a failing test case? The solution is pretty simple (select 1 rather than select primary_key), but in what circumstance would you actually have...