# Updating user status **URL:** \ **Category:** Rails\ **Created:** [January 11, 2010, 6:49am UTC](https://www.ruby-forum.com/t/updating-user-status/181377 "2010-01-11T06:49:26Z")\ **Posts on this page:** 5\ **Page:** 1
**Author:** ![tom-mac](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/t/bc8723/32.png) [@tom-mac](https://www.ruby-forum.com/u/tom-mac)\ **Post date:** [January 11, 2010, 6:49am UTC](https://www.ruby-forum.com/t/updating-user-status/181377/1 "2010-01-11T06:49:26Z")
Hi I have the models User belongs\_to user\_status and UserStatus ``` user_statuses is a master table having values ``` id name 1 Pending 2 Approved 3 Deleted Now when ever a user is created for the first time his user\_status\_id is 1 (ie pending). Administrator when click on Approve button status changes to “Approved”. My question is implementing this. Is it right to update like In user controller update action user = User.find(params[:id]) user.update\_attributes(:user\_status\_id =\> 2) ``` Rather than hard coding user_status_id = 2, is there any other ``` approach? Thanks in advance Tom ---
**Author:** ![tom-mac](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/t/bc8723/32.png) [@tom-mac](https://www.ruby-forum.com/u/tom-mac)\ **Post date:** [January 11, 2010, 7:02am UTC](https://www.ruby-forum.com/t/updating-user-status/181377/2 "2010-01-11T07:02:49Z")
Hi, Maybe you could try acts\_as\_state\_machine ---
**Author:** ![tom-mac](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/t/bc8723/32.png) [@tom-mac](https://www.ruby-forum.com/u/tom-mac)\ **Post date:** [January 12, 2010, 12:33pm UTC](https://www.ruby-forum.com/t/updating-user-status/181377/3 "2010-01-12T12:33:29Z")
> ``` > user_statuses is a master table having values > > ``` > > id name > 1 Pending > 2 Approved > 3 Deleted My first question is why do you have a table for user\_status? Are administrators of your app going to be adding new statuses via a web interface? if not, why not just have constants: class User STATUS\_PENDING = 1 STATUS\_APPROVED = 2 STATUS\_DELETE = 3 … end > user = User.find(params[:id]) > user.update\_attributes(:user\_status\_id =\> 2) > > ``` > Rather than hard coding user_status_id = 2, is there any other > > ``` > > approach? user.update\_attributes(user\_status =\> User::STATUS\_APPROVED) It sounds like you’re making it more complex than it needs to be. Alternatively if you wanted to keep it your way you could do: user.update\_attributes(:user\_status\_id =\> UserStatus.find\_by\_name(“Approved”).try(:id)) This finds the UserStatus record with the name Approved and uses it’s id (if it’s found) Cheers, Andy ---
**Author:** ![tom-mac](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/t/bc8723/32.png) [@tom-mac](https://www.ruby-forum.com/u/tom-mac)\ **Post date:** [January 12, 2010, 9:58pm UTC](https://www.ruby-forum.com/t/updating-user-status/181377/4 "2010-01-12T21:58:22Z")
I recommend keeping the table. Let me explain why and try to explain how to set it up better. This type of information is known at the outset of the application and it sounds like it is unlikely to change significantly over time. As such it could be pulled into a domain table, similar to states and zipcodes for example. Some benefits: 1. allows you to ensure referential integrity at the database level. This will ensure that your rails app, the pgsql or mysql prompt, another app, etc can only enter valid user roles. 2. changes can be made in one central location in the database 3. more importantly it scales the app may require additional information. For example, if you ever want to add more data associated with a role you can do this in an appropriate table. Take an address model and zipcode field as an example. Since zipcodes are known at the outset of an application they can be pulled into a domain table that the address table references. Then if you add latitude and longitude info to do distace calculations, you have a place to put this information and not repeat it many times within the address table. The same may be true for your user roles as the app evolves. Drawbacks this is a bit more work upfront but I argue that it is well worth it in the long run. Setting up a solid data layer is the most important part of any application. The data will likely outlive the app built atop it. Best to use solid db techniques. Make a migration for your user roles and a AR::base model. Make your constants there. For example in UserRole \< AR::Base define ADMIN = 1, etc. Then in the code for a user set @user.user\_role\_id = UserRoles::ADMIN. You can also write a function to create and cache these automatically based on the name you provide. I will leave that part to you. Just my thoughts Andrew ---
**Author:** ![tom-mac](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/t/bc8723/32.png) [@tom-mac](https://www.ruby-forum.com/u/tom-mac)\ **Post date:** [January 13, 2010, 4:35am UTC](https://www.ruby-forum.com/t/updating-user-status/181377/5 "2010-01-13T04:35:17Z")
Hi Thanks for all your help Tom