7

I'm just starting with Rails and decided to make a small app to learn with something practical.

I have a user class which has a user group integer field. I want to add to the migration a :default value using a constant.

In my user model I defined the different groups with constants so that I can later easily check "admin?" etc. 

t.integer :user_group, :default => USER

I get the following error on db:migrate

rake aborted! Expected [...]/app/models/user.rb to define USER

However in the user model I have this:

ADMIN = 1
USER = 2

Any ideas what I'm doing wrong?

zbrox
  • 2,683
  • 3
  • 25
  • 29

3 Answers3

4

You need to include your class name when referencing your constant. If your class is named User, try this:

t.integer :user_group, :default => User::USER

or 

t.integer :user_group, :default => User::ADMIN
Brett Bender
  • 19,388
  • 2
  • 38
  • 46
  • Thanks, I'm new to this and I wasn't sure if I had to include the class name. I even tried to use a dot conotation :) – zbrox Dec 24 '10 at 17:30
  • Yes, constants use that syntax, it's the namespace resolution operator. You can find more info on it (if you're curious) at this question: http://stackoverflow.com/questions/3009477/what-is-rubys-double-colon-all-about – Brett Bender Dec 24 '10 at 18:02
3

You shouldn't use a constant in a migration, since the migration should represent an independent point in time. A migration should not be coupled to a codebase which could change over time, since the migration then would change depending on when you run it. If you or someone else changes the value of the constant in the codebase (later on), it would impact the migration. It might not be realistic that you would actually ever need to change the constant value in the code, but this is merely an argument from principle.

If you want to change the default value in the DB at a later point in time, then just make a new migration then, with a new value.

Magne
  • 16,401
  • 10
  • 68
  • 88
0

I think you can also write :

t.integer :User, :user_group, :default => ADMIN

Am i wrong ?

Taryn
  • 242,637
  • 56
  • 362
  • 405
babttz
  • 446
  • 4
  • 11