"config.model" raises "table doesn't exist" exception if database is dropped

[krisleech]

initializers/rails_admin.rb

config.model Competition do
end

then

rake db:drop
rake db:create
rake db:migrate

In fact any rake task (which loads the environment) whic is run after dropping the database fails. This would appear to be because "config.model" needs to call Competition, I assume to get a list of database fields, and because the competitions table does not exist ActiveRecord raises an exception.

The solution I am using so far is to wrap the config.model block like so:

if defined? Competition
  config.model Competition
  end
end

Ruby 1.9.2
Rails 3.0.3
rails_admin HEAD

[sferik]

Thanks for the bug report. This needs to be fixed before we officially release RailsAdmin as a gem.

[krisleech]

Agreed, the workaround I proposed does not work in all situations either. Competition is always defined, and soon as it is inspected the 'no table' exception is raised.

The other workaround I have tried is wrapping the config.model block in an rescue and silently ignoring the exception. Maybe this could be done internally, rescue just the exception in question, and log to the database.

[sdrew]

If it helps, here's the way I'm working around it:

if Competition.table_exists?
  config.model Competition
  end
end

[niedfelj]

Yes, but even if the table exists, if you add columns and define them in a single push to a site like Heroku, you will still have problems when you try to run the rake task to migrate. For example, I added a :body field to a table and put in the rails admin config to have that field show the ckeditor. After pushing to Heroku, I was unable to run the rake task because it would fail on the rails admin config since the field didn't exist yet that rails admin was trying to configure.

[spint]

This workaround does not work for me (same error: table does not exist during migration):

if Competition.table_exists?

It does seem to work with:

if defined? Competition

ruby 1.9.2
rails 3.0.9

[netmute]

if Competition.table_exists?

works for me.

if defined? Competition

does not.

ruby 1.9.2
rails 3.0.7

This is a serious showstopper :)

[kaapa]

Would it be a sensible solution to move from using an initializer to an active record class method for model specific configuration such as:

in app/models/competition.rb

class Competition
  rails_admin do
    # What ever was done in an initializer with config.model method
  end
end

In that case the Competition model wouldn't get loaded due to RailsAdmin, right?

[sferik]

@kaapa 👍 Sounds good to me. :)

[bbenezech]

👍 I second that. Is it possible to make it work with class reload to true?

[kaapa]

@bbenezech: I've added a configuration reset on Railtie to_prepare hook which is run per request in development and on startup in other environments. Is that what you mean?

[bbenezech]

It's perfect!

[gunn]

Looks good. Is it possible to have multiple rails_admin blocks in the same model that would add to each other?

The advantage to this is that it would let us use modules to mixin admin behaviour.