rails4: replace update_attributes with the new update

Braulio Bhavamitra
1 parent b71858da
Showing 60 changed files with 98 additions and 98 deletions Show diff stats
app/controllers/admin/admin_panel_controller.rb
app/controllers/admin/categories_controller.rb
app/controllers/admin/features_controller.rb
app/controllers/admin/licenses_controller.rb
app/controllers/admin/plugins_controller.rb
app/controllers/admin/role_controller.rb
app/controllers/box_organizer_controller.rb
app/controllers/my_profile/cms_controller.rb
app/controllers/my_profile/enterprise_validation_controller.rb
app/controllers/my_profile/manage_products_controller.rb
app/controllers/my_profile/maps_controller.rb
app/controllers/my_profile/profile_editor_controller.rb
app/controllers/my_profile/profile_roles_controller.rb
app/controllers/my_profile/tasks_controller.rb
app/controllers/public/account_controller.rb
app/controllers/public/chat_controller.rb
app/controllers/public/comment_controller.rb
app/models/certifier.rb
app/models/create_community.rb
app/models/profile.rb
@@ -12,7 +12,7 @@ class AdminPanelController &lt; AdminController
       if params[:environment][:languages]
         params[:environment][:languages] = params[:environment][:languages].map {|lang, value| lang if value=='true'}.compact
       end
-      if @environment.update_attributes(params[:environment])
+      if @environment.update(params[:environment])
         session[:notice] = _('Environment settings updated')
         redirect_to :action => 'index'
       end
@@ -65,7 +65,7 @@ class AdminPanelController &lt; AdminController
  
   def set_portal_news_amount
     if request.post?
-      if @environment.update_attributes(params[:environment])
+      if @environment.update(params[:environment])
         session[:notice] = _('Saved the number of news on folders')
         redirect_to :action => 'index'
       end
@@ -43,7 +43,7 @@ class CategoriesController &lt; AdminController
     begin
       @category = environment.categories.find(params[:id])
       if request.post?
-        @category.update_attributes!(params[:category])
+        @category.update!(params[:category])
         @saved = true
         session[:notice] = _("Category %s saved." % @category.name)
         redirect_to :action => 'index'
@@ -7,7 +7,7 @@ class FeaturesController &lt; AdminController
  
   post_only :update
   def update
-    if @environment.update_attributes(params[:environment])
+    if @environment.update(params[:environment])
       session[:notice] = _('Features updated successfully.')
       redirect_to :action => 'index'
     else
@@ -23,7 +23,7 @@ class LicensesController &lt; AdminController
     @license = environment.licenses.find(params[:license_id])
     if request.post?
       begin
-        @license.update_attributes!(params[:license])
+        @license.update!(params[:license])
         session[:notice] = _('License updated')
         redirect_to :action => 'index'
       rescue
@@ -8,7 +8,7 @@ class PluginsController &lt; AdminController
   post_only :update
   def update
     params[:environment][:enabled_plugins].delete('')
-    if @environment.update_attributes(params[:environment])
+    if @environment.update(params[:environment])
       session[:notice] = _('Plugins updated successfully.')
     else
       session[:error] = _('Plugins were not updated successfully.')
@@ -29,7 +29,7 @@ class RoleController &lt; AdminController
  
   def update
     @role = environment.roles.find(params[:id])
-    if @role.update_attributes(params[:role])
+    if @role.update(params[:role])
       redirect_to :action => 'show', :id => @role
     else
       session[:notice] = _('Failed to edit role')
@@ -98,7 +98,7 @@ class BoxOrganizerController &lt; ApplicationController
  
   def save
     @block = boxes_holder.blocks.find(params[:id])
-    @block.update_attributes(params[:block])
+    @block.update(params[:block])
     redirect_to :action => 'index'
   end
  
@@ -93,7 +93,7 @@ class CmsController &lt; MyProfileController
         @article.image.save!
       end
       @article.last_changed_by = user
-      if @article.update_attributes(params[:article])
+      if @article.update(params[:article])
         if !continue
           if @article.content_type.nil? || @article.image?
             success_redirect
@@ -54,7 +54,7 @@ class EnterpriseValidationController &lt; MyProfileController
   def edit_validation_info
     @info = profile.validation_info
     if request.post?
-      if @info.update_attributes(params[:info])
+      if @info.update(params[:info])
         redirect_to :action => 'index'
       end
     end
@@ -69,7 +69,7 @@ class ManageProductsController &lt; ApplicationController
     field = params[:field]
     if request.post?
       begin
-        @product.update_attributes!(params[:product])
+        @product.update!(params[:product])
         render :partial => "display_#{field}", :locals => {:product => @product}
       rescue Exception => e
         render :partial => "edit_#{field}", :locals => {:product => @product, :errors => true}
@@ -86,7 +86,7 @@ class ManageProductsController &lt; ApplicationController
     @edit = true
     @level = @category.level
     if request.post?
-      if @product.update_attributes({:product_category_id => params[:selected_category_id]}, :without_protection => true)
+      if @product.update({:product_category_id => params[:selected_category_id]}, :without_protection => true)
         render :partial => 'shared/redirect_via_javascript',
           :locals => { :url => url_for(:controller => 'manage_products', :action => 'show', :id => @product) }
       else
@@ -115,7 +115,7 @@ class ManageProductsController &lt; ApplicationController
     @categories = ProductCategory.top_level_for(environment)
     @level = 0
     if request.post?
-      if @input.update_attributes(:product_category_id => params[:selected_category_id])
+      if @input.update(:product_category_id => params[:selected_category_id])
         @inputs = @product.inputs
         render :partial => 'display_inputs'
       else
@@ -172,7 +172,7 @@ class ManageProductsController &lt; ApplicationController
       @input = @profile.inputs.find_by_id(params[:id])
       if @input
         if request.post?
-          if @input.update_attributes(params[:input])
+          if @input.update(params[:input])
             render :partial => 'display_input', :locals => {:input => @input}
           else
             render :partial => 'edit_input'
@@ -15,7 +15,7 @@ class MapsController &lt; MyProfileController
         end
  
         Profile.transaction do
-          if profile.update_attributes!(params[:profile_data])
+          if profile.update!(params[:profile_data])
             BlockSweeper.expire_blocks profile.blocks.select{ |b| b.class == LocationBlock }
             session[:notice] = _('Address was updated successfully!')
             redirect_to :action => 'edit_location'
@@ -24,7 +24,7 @@ class ProfileEditorController &lt; MyProfileController
         Image.transaction do
           begin
             @plugins.dispatch(:profile_editor_transaction_extras)
-            @profile_data.update_attributes!(params[:profile_data])
+            @profile_data.update!(params[:profile_data])
             redirect_to :action => 'index', :profile => profile.identifier
           rescue Exception => ex
             profile.identifier = params[:profile] if profile.identifier.blank?
@@ -94,7 +94,7 @@ class ProfileEditorController &lt; MyProfileController
     @welcome_page = profile.welcome_page || TinyMceArticle.new(:name => 'Welcome Page', :profile => profile, :published => false)
     if request.post?
       begin
-        @welcome_page.update_attributes!(params[:welcome_page])
+        @welcome_page.update!(params[:welcome_page])
         profile.welcome_page = @welcome_page
         profile.save!
         session[:notice] = _('Welcome page saved successfully.')
@@ -58,7 +58,7 @@ class ProfileRolesController &lt; MyProfileController
  
   def update
     @role = environment.roles.find(params[:id])
-    if @role.update_attributes(params[:role])
+    if @role.update(params[:role])
       redirect_to :action => 'show', :id => @role
     else
       session[:notice] = _('Failed to edit role')
@@ -48,7 +48,7 @@ class TasksController &lt; MyProfileController
         if request.post? && VALID_DECISIONS.include?(decision) && id && decision != 'skip'
           task = profile.find_in_all_tasks(id)
           begin
-            task.update_attributes(value[:task])
+            task.update(value[:task])
             task.send(decision, current_person)
           rescue Exception => ex
             message = "#{task.title} (#{task.requestor ? task.requestor.name : task.author_name})"
@@ -119,7 +119,7 @@ class AccountController &lt; ApplicationController
           @user.person.affiliate(@user.person, [owner_role]) if owner_role
           invitation = Task.from_code(@invitation_code).first
           if invitation
-            invitation.update_attributes! friend: @user.person
+            invitation.update! friend: @user.person
             invitation.finish
           end
  
@@ -218,7 +218,7 @@ class AccountController &lt; ApplicationController
  
     if request.post?
       begin
-        @change_password.update_attributes!(params[:change_password])
+        @change_password.update!(params[:change_password])
         @change_password.finish
         render :action => 'new_password_ok'
       rescue ActiveRecord::RecordInvalid => e
@@ -487,7 +487,7 @@ class AccountController &lt; ApplicationController
   def check_redirection
     unless params[:redirection].blank?
       session[:return_to] = @user.return_to
-      @user.update_attributes(:return_to => nil)
+      @user.update(:return_to => nil)
     end
   end
  
@@ -67,7 +67,7 @@ class ChatController &lt; PublicController
  
   def update_presence_status
     if request.xhr?
-      current_user.update_attributes({:chat_status_at => DateTime.now}.merge(params[:status] || {}))
+      current_user.update({:chat_status_at => DateTime.now}.merge(params[:status] || {}))
     end
     render :nothing => true
   end
@@ -78,7 +78,7 @@ class CommentController &lt; ApplicationController
     respond_to do |format|
       format.js do
         comment_to_render = @comment.comment_root
-        render :json => { 
+        render :json => {
             :render_target => comment_to_render.anchor,
             :html => render_to_string(:partial => 'comment', :locals => {:comment => comment_to_render, :display_link => true}),
             :msg => _('Comment successfully created.')
@@ -114,7 +114,7 @@ class CommentController &lt; ApplicationController
   end
  
   def update
-    if @comment.update_attributes(params[:comment])
+    if @comment.update(params[:comment])
       @plugins.dispatch(:process_extra_comment_params, [@comment,params])
  
       respond_to do |format|
@@ -20,7 +20,7 @@ class Certifier &lt; ActiveRecord::Base
   validates_presence_of :name
  
   def destroy
-    product_qualifiers.each { |pq| pq.update_attributes! :certifier => nil }
+    product_qualifiers.each { |pq| pq.update! :certifier => nil }
     super
   end
  
@@ -30,7 +30,7 @@ class CreateCommunity &lt; Task
       ! DATA_FIELDS.include?(key.to_s)
     end
  
-    community.update_attributes(community_data)
+    community.update(community_data)
     community.image = image if image
     community.environment = self.environment
     community.save!
@@ -720,7 +720,7 @@ private :generate_url, :url_options
         num = num + 1
         new_name = original_article.name + ' ' + num.to_s
       end
-      original_article.update_attributes!(:name => new_name)
+      original_article.update!(:name => new_name)
     end
     article_copy = article.copy(:profile => self, :parent => parent, :advertise => false)
     if article.profile.home_page == article
@@ -15,12 +15,12 @@ Given /^the following users?$/ do |table|
 end
  
 Given /^"(.+)" is (invisible|visible)$/ do |user, visibility|
-  User.find_by_login(user).person.update_attributes({:visible => (visibility == 'visible')}, :without_protection => true)
+  User.find_by_login(user).person.update({:visible => (visibility == 'visible')}, :without_protection => true)
 end
  
 Given /^"(.+)" is (online|offline|busy) in chat$/ do |user, status|
   status = {'online' => 'chat', 'offline' => '', 'busy' => 'dnd'}[status]
-  User.find_by_login(user).update_attributes(:chat_status => status, :chat_status_at => DateTime.now)
+  User.find_by_login(user).update(:chat_status => status, :chat_status_at => DateTime.now)
 end
  
 Given /^the following (community|communities|enterprises?|organizations?)$/ do |kind,table|
@@ -680,7 +680,7 @@ end
 Given /^the article "([^\"]*)" is updated with$/ do |article, table|
   a = Article.find_by_name article
   row = table.hashes.first
-  a.update_attributes(row)
+  a.update(row)
 end
  
 Given /^the cache is turned (on|off)$/ do |state|
@@ -28,7 +28,7 @@ module ActsAsHavingPosts
   def feed=(attrs)
     if attrs
       if self.feed
-        self.feed.update_attributes(attrs)
+        self.feed.update(attrs)
       else
         self.feed_attrs = attrs
       end
@@ -140,7 +140,7 @@ class BscPluginMyprofileController &lt; MyProfileController
       session[:notice] = _('Could not edit such contract.')
       redirect_to :action => 'manage_contracts'
     end
-    if request.post? && @contract.update_attributes(params[:contract])
+    if request.post? && @contract.update(params[:contract])
  
       # updating associated enterprises
       enterprises_ids = params[:enterprises] || ''
@@ -159,7 +159,7 @@ class BscPluginMyprofileController &lt; MyProfileController
  
       to_keep.each do |sale_attrs|
         sale = @contract.sales.find_by_product_id(sale_attrs[:product_id])
-        sale.update_attributes!(sale_attrs)
+        sale.update!(sale_attrs)
         sales.delete(sale_attrs)
       end
  
@@ -27,7 +27,7 @@ class CommentClassificationPluginLabelsController &lt; AdminController
     @colors = CommentClassificationPlugin::Label::COLORS
     if request.post?
       begin
-        @label.update_attributes!(params[:label])
+        @label.update!(params[:label])
         session[:notice] = _('Label updated')
         redirect_to :action => :index
       rescue
@@ -25,7 +25,7 @@ class CommentClassificationPluginStatusController &lt; AdminController
     @status = CommentClassificationPlugin::Status.find(params[:id])
     if request.post?
       begin
-        @status.update_attributes!(params[:status])
+        @status.update!(params[:status])
         session[:notice] = _('Status updated')
         redirect_to :action => :index
       rescue
@@ -13,7 +13,7 @@ class CommentClassificationPluginMyprofileController &lt; MyProfileController
     @statuses = CommentClassificationPlugin::Status.enabled
     @status = CommentClassificationPlugin::CommentStatusUser.new(:profile => user, :comment => @comment)
     if request.post? && params[:status]
-      @status.update_attributes(params[:status])
+      @status.update(params[:status])
       @status.save
     end
   end
@@ -15,7 +15,7 @@ class DrivenSignupPlugin::AdminController &lt; AdminController
   def edit
     @auth = environment.driven_signup_auths.where(id: params[:id]).first
     @auth ||= environment.driven_signup_auths.build
-    @auth.update_attributes params[:auth]
+    @auth.update params[:auth]
   end
  
   def destroy
@@ -68,7 +68,7 @@ class FbAppPluginPageTabController &lt; FbAppPluginController
     if request.put? and @page_id.present?
       create_page_tabs if @page_tab.nil?
  
-      @page_tab.update_attributes! params[:page_tab]
+      @page_tab.update! params[:page_tab]
  
       respond_to do |format|
         format.js{ render action: 'admin' }
@@ -27,7 +27,7 @@ class FbAppPlugin::PageTab &lt; ActiveRecord::Base
     page_ids.map do |page_id|
       page_tab = FbAppPlugin::PageTab.where(page_id: page_id).first
       page_tab ||= FbAppPlugin::PageTab.new page_id: page_id
-      page_tab.update_attributes! attrs
+      page_tab.update! attrs
       page_tab
     end
   end
@@ -129,7 +129,7 @@ class LattesCurriculumPlugin &lt; Noosfero::Plugin
  
   def academic_info_transaction
     AcademicInfo.transaction do
-      context.profile.academic_info.update_attributes!(context.params[:academic_infos])
+      context.profile.academic_info.update!(context.params[:academic_infos])
     end
   end
  
@@ -6,7 +6,7 @@ class LdapPluginAdminController &lt; PluginAdminController
   end
  
   def update
-    if @environment.update_attributes(params[:environment])
+    if @environment.update(params[:environment])
       session[:notice] = _('Ldap configuration updated successfully.')
     else
       session[:notice] = _('Ldap configuration could not be saved.')
@@ -16,7 +16,7 @@ class OauthClientPluginAdminController &lt; AdminController
   def edit
     @provider = params[:id] ? environment.oauth_providers.find(params[:id]) : environment.oauth_providers.new
     if request.post?
-      if @provider.update_attributes(params['oauth_client_plugin_provider'])
+      if @provider.update(params['oauth_client_plugin_provider'])
         session[:notice] = _('Saved!')
       else
         session[:notice] = _('Error!')
@@ -17,7 +17,7 @@ class OpenGraphPlugin::MyprofileController &lt; MyProfileController
   end
  
   def track_config
-    profile.update_attributes! params[:profile_data]
+    profile.update! params[:profile_data]
     render partial: 'track_form', locals: {context: context, reload: true}
   end
  
@@ -25,7 +25,7 @@ class OpenGraphPlugin::PublisherTest &lt; ActiveSupport::TestCase
  
     @community = @actor.environment.communities.create! name: 'comm', identifier: 'comm', closed: false
  
-    @actor.update_attributes!({
+    @actor.update!({
       open_graph_settings: {
         activity_track_enabled: "true",
         enterprise_track_enabled: "true",
@@ -43,7 +43,7 @@ class OpenGraphPlugin::PublisherTest &lt; ActiveSupport::TestCase
       open_graph_enterprise_profiles_ids: "#{@enterprise.id}",
       open_graph_community_profiles_ids: "#{@community.id}",
     })
-    @other_actor.update_attributes! open_graph_settings: { activity_track_enabled: "true", },
+    @other_actor.update! open_graph_settings: { activity_track_enabled: "true", },
       open_graph_activity_track_configs_attributes: { 0 => { tracker_id: @other_actor.id, object_type: 'friend', }, }
  
     # active
@@ -8,7 +8,7 @@ class OrganizationRatingsPluginAdminController &lt; PluginAdminController
   end
  
   def update
-    if env_organization_ratings_config.update_attributes(params[:organization_ratings_config])
+    if env_organization_ratings_config.update(params[:organization_ratings_config])
       session[:notice] = _('Configuration updated successfully.')
     else
       session[:notice] = _('Configuration could not be saved.')
@@ -16,4 +16,4 @@ class OrganizationRatingsPluginAdminController &lt; PluginAdminController
     render :action => 'index'
   end
  
-end
 \ No newline at end of file
+end
@@ -4,7 +4,7 @@ class PiwikPluginAdminController &lt; PluginAdminController
  
   def index
     if request.post?
-      if @environment.update_attributes(params[:environment])
+      if @environment.update(params[:environment])
         session[:notice] = _('Piwik plugin settings updated successfully.')
       else
         session[:notice] = _('Piwik plugin settings could not be saved.')
@@ -3,7 +3,7 @@ class SendEmailPluginAdminController &lt; PluginsController
   def index
     @environment = environment
     if request.post?
-      if environment.update_attributes(params[:environment])
+      if environment.update(params[:environment])
         session[:notice] = _('Configurations was saved')
         redirect_to :controller => 'plugins'
       else
@@ -9,7 +9,7 @@ class SnifferPluginMyprofileController &lt; MyProfileController
   def edit
     if request.post?
       begin
-        @sniffer_profile.update_attributes(params[:sniffer_plugin_profile])
+        @sniffer_profile.update(params[:sniffer_plugin_profile])
         @sniffer_profile.enabled = true
         @sniffer_profile.save!
         session[:notice] = _('Consumer interests updated')
@@ -5,7 +5,7 @@ class ToleranceTimePluginMyprofileController &lt; MyProfileController
     if request.post?
       begin
         convert_params
-        @tolerance.update_attributes!(params[:tolerance])
+        @tolerance.update!(params[:tolerance])
         convert_values
         session[:notice] = _('Tolerance updated')
       rescue
@@ -46,7 +46,7 @@ when &#39;active-enterprise&#39;
   active_enterprises.each do |enterprise|
     old_home = enterprise.home_page
     enterprise.apply_template(env.enterprise_template)
-    enterprise.home_page.update_attributes!(:body => old_home.body)
+    enterprise.home_page.update!(:body => old_home.body)
     enterprise.save!
   end
 when 'community'
@@ -73,14 +73,14 @@ class ChatControllerTest &lt; ActionController::TestCase
   end
  
   should 'not update presence status from non-ajax requests' do
-    @person.user.expects(:update_attributes).never
+    @person.user.expects(:update).never
     @controller.stubs(:current_user).returns(@person.user)
     get :update_presence_status
     assert_template nil
   end
  
   should 'update presence status from ajax requests' do
-    @person.user.expects(:update_attributes).once
+    @person.user.expects(:update).once
     @controller.stubs(:current_user).returns(@person.user)
     @request.stubs(:xhr?).returns(true)
     get :update_presence_status
@@ -144,7 +144,7 @@ class ContentViewerControllerTest &lt; ActionController::TestCase
  
   should 'not display forbidden articles' do
     profile.articles.create!(:name => 'test')
-    profile.update_attributes!({:public_content => false}, :without_protection => true)
+    profile.update!({:public_content => false}, :without_protection => true)
  
     Article.any_instance.expects(:display_to?).with(anything).returns(false)
     get :view_page, :profile => profile.identifier, :page => [ 'test' ]
@@ -153,7 +153,7 @@ class ContentViewerControllerTest &lt; ActionController::TestCase
  
   should 'display allowed articles' do
     profile.articles.create!(:name => 'test')
-    profile.update_attributes!({:public_content => false}, :without_protection => true)
+    profile.update!({:public_content => false}, :without_protection => true)
  
     Article.any_instance.expects(:display_to?).with(anything).returns(true)
     get :view_page, :profile => profile.identifier, :page => [ 'test' ]
@@ -66,7 +66,7 @@ class FeaturesControllerTest &lt; ActionController::TestCase
  
   def test_should_mark_current_organization_approval_method_in_view
     uses_host 'anhetegua.net'
-    Environment.find(environments(:anhetegua_net).id).update_attributes(:organization_approval_method => :region)
+    Environment.find(environments(:anhetegua_net).id).update(:organization_approval_method => :region)
  
     post :index
  
@@ -60,7 +60,7 @@ class MailconfControllerTest &lt; ActionController::TestCase
  
   should 'display correctly the state false of e-mail enable/disable' do
     login_as('ze')
-    user.update_attributes!(:enable_email => false)
+    user.update!(:enable_email => false)
     get :index, :profile => 'ze'
     assert_tag :tag => 'a', :content => 'Enable e-Mail'
     assert_no_tag :tag => 'a', :content => 'Disable e-Mail', :attributes => { :href => '/myprofile/ze/mailconf/disable' }
@@ -51,7 +51,7 @@ class MapsControllerTest &lt; ActionController::TestCase
   end
  
   should 'back when update address fail' do
-    Profile.any_instance.stubs(:update_attributes!).returns(false)
+    Profile.any_instance.stubs(:update!).returns(false)
     post :edit_location, :profile => profile.identifier, :profile_data => { 'address' => 'new address' }
     assert_nil profile.address
     assert_template 'edit_location'
@@ -196,7 +196,7 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase
   end
  
   should 'display properly that the profile is non-public' do
-    profile.update_attributes!(:public_profile => false)
+    profile.update!(:public_profile => false)
     get :edit, :profile => profile.identifier
     assert_tag :tag => 'input', :attributes => { :type => 'radio', :checked => 'checked', :name => 'profile_data[public_profile]', :value => 'false' }
     assert_tag :tag => 'input', :attributes => { :type => 'radio', :name => 'profile_data[public_profile]', :value => 'true' }
@@ -227,7 +227,7 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase
  
   should 'back when update community info fail' do
     org = fast_create(Community)
-    Community.any_instance.expects(:update_attributes!).raises(ActiveRecord::RecordInvalid)
+    Community.any_instance.expects(:update!).raises(ActiveRecord::RecordInvalid)
     post :edit, :profile => org.identifier
  
     assert_template 'edit'
@@ -237,7 +237,7 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase
   should 'back when update enterprise info fail' do
     org = fast_create(Enterprise)
  
-    Enterprise.any_instance.expects(:update_attributes!).raises(ActiveRecord::RecordInvalid)
+    Enterprise.any_instance.expects(:update!).raises(ActiveRecord::RecordInvalid)
     post :edit, :profile => org.identifier
     assert_template 'edit'
     assert_response :success
@@ -801,7 +801,7 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase
   end
  
   should 'show profile nickname on title' do
-    profile.update_attributes(:nickname => 'my nick')
+    profile.update(:nickname => 'my nick')
     get :index, :profile => profile.identifier
     assert_tag :tag => 'h1', :attributes => { :class => 'block-title'}, :descendant => {
       :tag => 'span', :attributes => { :class => 'control-panel-title' }, :content => 'my nick'
@@ -123,7 +123,7 @@ class TasksControllerTest &lt; ActionController::TestCase
  
     post :close, :tasks => {t.id => {:decision => 'finish', :task => {}}}
     t.reload
-    
+
     ok('task should be finished') { t.status == Task::Status::FINISHED }
  
     c.reload
@@ -190,7 +190,7 @@ class TasksControllerTest &lt; ActionController::TestCase
  
   should 'create article with reference_article after finish approve article task' do
     c = fast_create(Community)
-    c.update_attributes(:moderated_articles => false)
+    c.update(:moderated_articles => false)
     @controller.stubs(:profile).returns(c)
     c.affiliate(profile, Profile::Roles.all_roles(profile.environment.id))
     article = profile.articles.create!(:name => 'something interesting', :body => 'ruby on rails')
@@ -202,7 +202,7 @@ class TasksControllerTest &lt; ActionController::TestCase
  
   should 'create published article in folder after finish approve article task' do
     c = fast_create(Community)
-    c.update_attributes(:moderated_articles => false)
+    c.update(:moderated_articles => false)
     @controller.stubs(:profile).returns(c)
     folder = create(Folder, :profile => c, :name => 'test folder')
     c.affiliate(profile, Profile::Roles.all_roles(profile.environment.id))
@@ -215,7 +215,7 @@ class TasksControllerTest &lt; ActionController::TestCase
  
   should 'be highlighted if asked when approving a published article' do
     c = fast_create(Community)
-    c.update_attributes(:moderated_articles => false)
+    c.update(:moderated_articles => false)
     @controller.stubs(:profile).returns(c)
     folder = create(Article, :profile => c, :name => 'test folder', :type => 'Folder')
     c.affiliate(profile, Profile::Roles.all_roles(profile.environment.id))
@@ -228,7 +228,7 @@ class TasksControllerTest &lt; ActionController::TestCase
  
   should 'create article of same class after choosing root folder on approve article task' do
     c = fast_create(Community)
-    c.update_attributes(:moderated_articles => false)
+    c.update(:moderated_articles => false)
     @controller.stubs(:profile).returns(c)
     c.affiliate(profile, Profile::Roles.all_roles(profile.environment.id))
     article = profile.articles.create!(:name => 'something interesting', :body => 'ruby on rails')
@@ -24,7 +24,7 @@ class SignupTest &lt; ActionDispatch::IntegrationTest
  
   def test_should_require_acceptance_of_terms_for_signup
     env = Environment.default
-    env.update_attributes(:terms_of_use => 'You agree to not be annoying.')
+    env.update(:terms_of_use => 'You agree to not be annoying.')
     env.min_signup_delay = 0
     env.save!
  
@@ -136,7 +136,7 @@ class BlockTest &lt; ActiveSupport::TestCase
     user = create_user('testinguser').person
     box = fast_create(Box, :owner_id => user.id, :owner_type => 'Profile')
     block = create(Block, :display => 'never', :box_id => box.id)
-    assert block.update_attributes!(:display => 'always')
+    assert block.update!(:display => 'always')
     block.reload
     assert_equal 'always', block.display
   end
@@ -62,7 +62,7 @@ class CertifierTest &lt; ActiveSupport::TestCase
   should 'set qualifier as self-certified when destroyed' do
     pq = mock
     Certifier.any_instance.stubs(:product_qualifiers).returns([pq])
-    pq.expects(:update_attributes!).with(:certifier => nil)
+    pq.expects(:update!).with(:certifier => nil)
     cert = fast_create(Certifier)
     cert.destroy
   end
@@ -63,7 +63,7 @@ class HighlightsBlockTest &lt; ActiveSupport::TestCase
     block = HighlightsBlock.create!(:display => 'never').tap do |b|
       b.box = box
     end
-    assert block.update_attributes!(:display => 'always')
+    assert block.update!(:display => 'always')
     block.reload
     assert_equal 'always', block.display
   end
@@ -10,7 +10,7 @@ class LinkListBlockTest &lt; ActiveSupport::TestCase
     l = LinkListBlock.new
     assert_respond_to l, :links
   end
-  
+
   should 'default value of links' do
     l = LinkListBlock.new
     assert_equal [], l.links
@@ -117,7 +117,7 @@ class LinkListBlockTest &lt; ActiveSupport::TestCase
     block = LinkListBlock.new(:display => 'never').tap do |b|
       b.box = box
     end
-    assert block.update_attributes!(:display => 'always')
+    assert block.update!(:display => 'always')
     block.reload
     assert_equal 'always', block.display
   end
@@ -31,7 +31,7 @@ class MyNetworkBlockTest &lt; ActiveSupport::TestCase
     user = create_user('testinguser').person
     box = fast_create(Box, :owner_id => user.id)
     block = MyNetworkBlock.create!(:display => 'never', :box => box)
-    assert block.update_attributes!(:display => 'always')
+    assert block.update!(:display => 'always')
     block.reload
     assert_equal 'always', block.display
   end
@@ -153,7 +153,7 @@ class PersonTest &lt; ActiveSupport::TestCase
     person = create_user('just_another_person').person
     env.affiliate(person, role)
     refute  person.is_admin?(env)
-    role.update_attributes(:permissions => ['view_environment_admin_panel'])
+    role.update(:permissions => ['view_environment_admin_panel'])
     person = Person.find(person.id)
     assert person.is_admin?(env)
   end
@@ -164,7 +164,7 @@ class PersonTest &lt; ActiveSupport::TestCase
  
     # role is an admin role
     role = create(Role, :name => 'just_another_admin_role')
-    role.update_attributes(:permissions => ['view_environment_admin_panel'])
+    role.update(:permissions => ['view_environment_admin_panel'])
  
     # user is admin of env1, but not of env2
     person = create_user('just_another_person').person
@@ -84,7 +84,7 @@ class RecentDocumentsBlockTest &lt; ActiveSupport::TestCase
   end
  
   should 'be able to update display setting' do
-    assert @block.update_attributes!(:display => 'always')
+    assert @block.update!(:display => 'always')
     @block.reload
     assert_equal 'always', @block.display
   end
@@ -255,7 +255,7 @@ class RssFeedTest &lt; ActiveSupport::TestCase
   should 'include only posts from some language' do
     profile = create_user('testuser').person
     blog = create(Blog, :name => 'blog-test', :profile => profile)
-    blog.feed.update_attributes! :language => 'es'
+    blog.feed.update! :language => 'es'
     blog.posts << en_post = fast_create(TextArticle, :name => "English", :profile_id => profile.id, :parent_id => blog.id, :published => true, :language => 'en')
     blog.posts << es_post = fast_create(TextArticle, :name => "Spanish", :profile_id => profile.id, :parent_id => blog.id, :published => true, :language => 'es')
  
@@ -43,12 +43,12 @@ class UserTest &lt; ActiveSupport::TestCase
   end
  
   def test_should_reset_password
-    users(:johndoe).update_attributes(:password => 'new password', :password_confirmation => 'new password')
+    users(:johndoe).update(:password => 'new password', :password_confirmation => 'new password')
     assert_equal users(:johndoe), User.authenticate('johndoe', 'new password')
   end
  
   def test_should_not_rehash_password
-    users(:johndoe).update_attributes(:login => 'johndoe2')
+    users(:johndoe).update(:login => 'johndoe2')
     assert_equal users(:johndoe), User.authenticate('johndoe2', 'test')
   end
  
@@ -37,7 +37,7 @@ module ActionTracker
       target_hash = params[:target].nil? ? {} : {:target_type => params[:target].class.base_class.to_s, :target_id => params[:target].id}
       conditions = { :user_id => u.id, :user_type => u.class.base_class.to_s, :verb => params[:verb].to_s }.merge(target_hash)
       l = where(conditions).last
-      ( !l.nil? and Time.now - l.updated_at < ActionTrackerConfig.timeout ) ? l.update_attributes(params.merge({ :updated_at => Time.now })) : l = new(params)
+      ( !l.nil? and Time.now - l.updated_at < ActionTrackerConfig.timeout ) ? l.update(params.merge({ :updated_at => Time.now })) : l = new(params)
       l
     end
  
@@ -48,7 +48,7 @@ module ActionTracker
       l = where({user_id: u.id, user_type: u.class.base_class.to_s, verb: params[:verb].to_s}.merge target_hash).last
       if !l.nil? and Time.now - l.created_at < ActionTrackerConfig.timeout
         params[:params].clone.each { |key, value| params[:params][key] = l.params[key].clone.push(value) }
-        l.update_attributes params
+        l.update params
       else
         params[:params].clone.each { |key, value| params[:params][key] = [value] }
         l = new params
@@ -42,7 +42,7 @@ class VoteablesController &lt; ApplicationController
   end
  
   # GET /users/:id/voteables/1/edit
-  def edit 
+  def edit
     @voteable ||= Voteable.find(params[:id])
   end
  
@@ -51,7 +51,7 @@ class VoteablesController &lt; ApplicationController
   def create
     @voteable = Voteable.new(params[:voteable])
     @voteable.user = current_user
-    
+
     respond_to do |format|
       if @voteable.save
         flash[:notice] = 'Voteable was successfully saved.'
@@ -68,9 +68,9 @@ class VoteablesController &lt; ApplicationController
   # PUT /users/:id/voteable/1.xml
   def update
     @voteable = Voteable.find(params[:id])
-    
+
     respond_to do |format|
-      if @quote.update_attributes(params[:voteable])
+      if @quote.update(params[:voteable])
         flash[:notice] = 'Voteable was successfully updated.'
         format.html { redirect_to([@user, @voteable]) }
         format.xml  { head :ok }
@@ -92,17 +92,17 @@ class VoteablesController &lt; ApplicationController
       format.xml  { head :ok }
     end
   end
-  
+
   private
-  def find_user 
+  def find_user
     @user = User.find(params[:user_id])
   end
-    
+
   def must_own_voteable
     @voteable ||= Voteable.find(params[:id])
     @voteable.user == current_user || ownership_violation
   end
-  
+
   def ownership_violation
     respond_to do |format|
       flash[:notice] = 'You cannot edit or delete voteable that you do not own!'
@@ -111,7 +111,7 @@ class VoteablesController &lt; ApplicationController
       end
     end
   end
-  
-  
-  
+
+
+
 end
 class VotesController < ApplicationController
   # First, figure out our nested scope. User or vote?
 #  before_filter :find_my_scope
-  
+
 #  before_filter :find_user
-    
+
 #  before_filter :login_required, :only => [:new, :edit, :destroy, :create, :update]
 #  before_filter :must_own_vote,  :only => [:edit, :destroy, :update]
  
@@ -70,7 +70,7 @@ class VotesController &lt; ApplicationController
       @vote = Vote.find(params[:id])
  
       respond_to do |format|
-        if @vote.update_attributes(params[:vote])
+        if @vote.update(params[:vote])
           flash[:notice] = 'Vote was successfully updated.'
           format.html { redirect_to([@user, @vote]) }
           format.xml  { head :ok }
...	...	@@ -12,7 +12,7 @@ class AdminPanelController < AdminController
12	12	if params[:environment][:languages]
13	13	params[:environment][:languages] = params[:environment][:languages].map {\|lang, value\| lang if value=='true'}.compact
14	14	end
15		- if @environment.update_attributes(params[:environment])
	15	+ if @environment.update(params[:environment])
16	16	session[:notice] = _('Environment settings updated')
17	17	redirect_to :action => 'index'
18	18	end
...	...	@@ -65,7 +65,7 @@ class AdminPanelController < AdminController
65	65
66	66	def set_portal_news_amount
67	67	if request.post?
68		- if @environment.update_attributes(params[:environment])
	68	+ if @environment.update(params[:environment])
69	69	session[:notice] = _('Saved the number of news on folders')
70	70	redirect_to :action => 'index'
71	71	end
...	...
...	...	@@ -43,7 +43,7 @@ class CategoriesController < AdminController
43	43	begin
44	44	@category = environment.categories.find(params[:id])
45	45	if request.post?
46		- @category.update_attributes!(params[:category])
	46	+ @category.update!(params[:category])
47	47	@saved = true
48	48	session[:notice] = _("Category %s saved." % @category.name)
49	49	redirect_to :action => 'index'
...	...
...	...	@@ -7,7 +7,7 @@ class FeaturesController < AdminController
7	7
8	8	post_only :update
9	9	def update
10		- if @environment.update_attributes(params[:environment])
	10	+ if @environment.update(params[:environment])
11	11	session[:notice] = _('Features updated successfully.')
12	12	redirect_to :action => 'index'
13	13	else
...	...
...	...	@@ -23,7 +23,7 @@ class LicensesController < AdminController
23	23	@license = environment.licenses.find(params[:license_id])
24	24	if request.post?
25	25	begin
26		- @license.update_attributes!(params[:license])
	26	+ @license.update!(params[:license])
27	27	session[:notice] = _('License updated')
28	28	redirect_to :action => 'index'
29	29	rescue
...	...
...	...	@@ -8,7 +8,7 @@ class PluginsController < AdminController
8	8	post_only :update
9	9	def update
10	10	params[:environment][:enabled_plugins].delete('')
11		- if @environment.update_attributes(params[:environment])
	11	+ if @environment.update(params[:environment])
12	12	session[:notice] = _('Plugins updated successfully.')
13	13	else
14	14	session[:error] = _('Plugins were not updated successfully.')
...	...
...	...	@@ -29,7 +29,7 @@ class RoleController < AdminController
29	29
30	30	def update
31	31	@role = environment.roles.find(params[:id])
32		- if @role.update_attributes(params[:role])
	32	+ if @role.update(params[:role])
33	33	redirect_to :action => 'show', :id => @role
34	34	else
35	35	session[:notice] = _('Failed to edit role')
...	...
...	...	@@ -98,7 +98,7 @@ class BoxOrganizerController < ApplicationController
98	98
99	99	def save
100	100	@block = boxes_holder.blocks.find(params[:id])
101		- @block.update_attributes(params[:block])
	101	+ @block.update(params[:block])
102	102	redirect_to :action => 'index'
103	103	end
104	104
...	...
...	...	@@ -93,7 +93,7 @@ class CmsController < MyProfileController
93	93	@article.image.save!
94	94	end
95	95	@article.last_changed_by = user
96		- if @article.update_attributes(params[:article])
	96	+ if @article.update(params[:article])
97	97	if !continue
98	98	if @article.content_type.nil? \|\| @article.image?
99	99	success_redirect
...	...
...	...	@@ -54,7 +54,7 @@ class EnterpriseValidationController < MyProfileController
54	54	def edit_validation_info
55	55	@info = profile.validation_info
56	56	if request.post?
57		- if @info.update_attributes(params[:info])
	57	+ if @info.update(params[:info])
58	58	redirect_to :action => 'index'
59	59	end
60	60	end
...	...
...	...	@@ -69,7 +69,7 @@ class ManageProductsController < ApplicationController
69	69	field = params[:field]
70	70	if request.post?
71	71	begin
72		- @product.update_attributes!(params[:product])
	72	+ @product.update!(params[:product])
73	73	render :partial => "display_#{field}", :locals => {:product => @product}
74	74	rescue Exception => e
75	75	render :partial => "edit_#{field}", :locals => {:product => @product, :errors => true}
...	...	@@ -86,7 +86,7 @@ class ManageProductsController < ApplicationController
86	86	@edit = true
87	87	@level = @category.level
88	88	if request.post?
89		- if @product.update_attributes({:product_category_id => params[:selected_category_id]}, :without_protection => true)
	89	+ if @product.update({:product_category_id => params[:selected_category_id]}, :without_protection => true)
90	90	render :partial => 'shared/redirect_via_javascript',
91	91	:locals => { :url => url_for(:controller => 'manage_products', :action => 'show', :id => @product) }
92	92	else
...	...	@@ -115,7 +115,7 @@ class ManageProductsController < ApplicationController
115	115	@categories = ProductCategory.top_level_for(environment)
116	116	@level = 0
117	117	if request.post?
118		- if @input.update_attributes(:product_category_id => params[:selected_category_id])
	118	+ if @input.update(:product_category_id => params[:selected_category_id])
119	119	@inputs = @product.inputs
120	120	render :partial => 'display_inputs'
121	121	else
...	...	@@ -172,7 +172,7 @@ class ManageProductsController < ApplicationController
172	172	@input = @profile.inputs.find_by_id(params[:id])
173	173	if @input
174	174	if request.post?
175		- if @input.update_attributes(params[:input])
	175	+ if @input.update(params[:input])
176	176	render :partial => 'display_input', :locals => {:input => @input}
177	177	else
178	178	render :partial => 'edit_input'
...	...