Merge branch 'rails4'

Noosfero now uses Rails 4.2, and requires Debian Jessie.

Merge branch 'rails4'
Noosfero now uses Rails 4.2, and requires Debian Jessie.
Antonio Terceiro
2 parents d877a56d f9a04729
Showing 1143 changed files with 15359 additions and 31589 deletions Show diff stats
.travis.yml
Gemfile
Vagrantfile
app/controllers/admin/admin_panel_controller.rb
app/controllers/admin/categories_controller.rb
app/controllers/admin/environment_role_manager_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/admin/users_controller.rb
app/controllers/application.rb
app/controllers/application_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_members_controller.rb
@@ -8,8 +8,7 @@ notifications:
  
 language: ruby
 rvm:
-# for 2.2 support we need to upgrade the pg gem
-  - 2.1.6
+  - 2.2.3
  
 sudo: false
 addons:
@@ -26,19 +25,16 @@ addons:
       - libsqlite3-dev
       - libxslt1-dev
  
-before_install:
-  - gem env
-
 # workaround for https://github.com/travis-ci/travis-ci/issues/4536
 before_install:
-  - export GEM_HOME=$PWD/vendor/bundle/ruby/2.1.0
+  - export GEM_HOME=$PWD/vendor/bundle/ruby/2.2.0
   - gem install bundler
 cache: bundler
  
 before_script:
   - mkdir -p tmp/{pids,cache} log
   - script/noosfero-plugins disableall
-  - bundle exec rake makemo &>/dev/null
+  #- bundle exec rake makemo &>/dev/null
 # database
   - cp config/database.yml.travis config/database.yml
   - psql -c 'create database myapp_test;' -U postgres
@@ -49,7 +45,7 @@ env:
   - TASK=test:units
   - TASK=test:functionals
   - TASK=test:integration
-  - TASK=cucumber
+  - TASK=cucumber LANG=en
   - TASK=selenium
   - TASK=test:noosfero_plugins BUNDLE_OPTS=install
  
 source "https://rubygems.org"
-gem 'rails',                    '~> 3.2.22'
-gem 'minitest',                 '~> 3.2.0'
-gem 'fast_gettext',             '~> 0.6.8'
-gem 'acts-as-taggable-on',      '~> 3.4.2'
+gem 'rails',                    '~> 4.2.4'
+gem 'fast_gettext',             '~> 0.9'
+gem 'acts-as-taggable-on',      '~> 3.5'
 gem 'rails_autolink',           '~> 1.1.5'
-gem 'pg',                       '~> 0.13.2'
-gem 'rmagick',                  '~> 2.13.1'
-gem 'RedCloth',                 '~> 4.2.9'
-gem 'will_paginate',            '~> 3.0.3'
+gem 'pg',                       '~> 0.17'
+gem 'rmagick',                  '~> 2.13'
+gem 'RedCloth',                 '~> 4.2'
 gem 'ruby-feedparser',          '~> 0.7'
-gem 'daemons',                  '~> 1.1.5'
+gem 'daemons',                  '~> 1.1'
 gem 'unicorn',                  '~> 4.8'
-gem 'nokogiri',                 '~> 1.5.5'
+gem 'nokogiri',                 '~> 1.6.0'
+gem 'will_paginate'
+gem 'pothoven-attachment_fu',   '~> 3.2.16'
+gem 'delayed_job'
+gem 'delayed_job_active_record'
 gem 'rake', :require => false
-gem 'rest-client',              '~> 1.6.7'
+gem 'rest-client',              '~> 1.6'
 gem 'exception_notification',   '~> 4.0.1'
-gem 'gettext',                  '~> 2.2.1', :require => false
-gem 'locale',                   '~> 2.0.5'
+gem 'gettext',                  '~> 3.1', :require => false
+gem 'locale',                   '~> 2.1'
 gem 'whenever', :require => false
-gem 'eita-jrails', '~> 0.9.5', require: 'jrails'
+gem 'eita-jrails', '~> 0.10.0', require: 'jrails'
+gem 'diffy',                    '~> 3.0'
  
 # API dependencies
 gem 'grape',                    '~> 0.12'
@@ -30,30 +33,55 @@ gem &#39;rack-contrib&#39;
 # asset pipeline
 gem 'uglifier', '>= 1.0.3'
 gem 'sass-rails'
-gem 'sass', '~> 3.1.19'
+
+# gems to enable rails3 behaviour
+gem 'protected_attributes'
+gem 'rails-observers'
+gem 'actionpack-page_caching'
+gem 'actionpack-action_caching'
+gem 'activerecord-session_store'
+gem 'activerecord-deprecated_finders', require: 'active_record/deprecated_finders'
  
 group :production do
   gem 'dalli', '~> 2.7.0'
 end
  
+group :development do
+end
+
+group :development, :test do
+  gem 'spring'
+end
+
 group :test do
-  gem 'rspec',                  '~> 2.14.0'
-  gem 'rspec-rails',            '~> 2.14.1'
+  gem 'rspec',                  '~> 3.3'
+  gem 'rspec-rails',            '~> 3.2'
   gem 'mocha',                  '~> 1.1.0', :require => false
   gem 'test-unit' if RUBY_VERSION >= '2.2.0'
+  gem 'minitest'
+  gem 'minitest-reporters'
 end
  
 group :cucumber do
-  gem 'cucumber-rails',         '~> 1.0.6', :require => false
-  gem 'capybara',               '~> 2.1.0'
-  gem 'cucumber',               '~> 1.0.6'
-  gem 'database_cleaner',       '~> 1.2.0'
-  gem 'selenium-webdriver',     '~> 2.47.0'
+  gem 'capybara',               '~> 2.2'
+  gem 'launchy'
+  gem 'cucumber'
+  gem 'cucumber-rails',         '~> 1.4.2', :require => false
+  gem 'database_cleaner',       '~> 1.3'
+  gem 'selenium-webdriver'
 end
  
 # Requires custom dependencies
 eval(File.read('config/Gemfile'), binding) rescue nil
  
+vendor = Dir.glob('vendor/{,plugins/}*') - ['vendor/plugins']
+vendor.each do |dir|
+  plugin = File.basename dir
+  version = if Dir.glob("#{dir}/*.gemspec").length > 0 then '> 0.0.0' else '0.0.0' end
+
+  gem plugin, version, path: dir
+end
+
 # include gemfiles from enabled plugins
 # plugins in baseplugins/ are not included on purpose. They should not have any
 # dependencies.
@@ -3,7 +3,7 @@
  
 VAGRANTFILE_API_VERSION = "2"
 Vagrant.configure(VAGRANTFILE_API_VERSION) do |config|
-  config.vm.box = ENV.fetch('VAGRANT_BOX', "debian-wheezy")
+  config.vm.box = ENV.fetch('VAGRANT_BOX', "debian/jessie64")
   config.vm.network :forwarded_port, host: 3000, guest: 3000
   config.vm.provision :shell do |shell|
     shell.inline = 'su vagrant -c /vagrant/script/vagrant'
@@ -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
@@ -54,7 +54,7 @@ class AdminPanelController &lt; AdminController
  
      if request.post?
        env = environment
-       folders = params[:folders].map{|fid| Folder.find(:first, :conditions => {:profile_id => env.portal_community, :id => fid})} if params[:folders]
+       folders = env.portal_community.folders.where(id: params[:folders]).order(params[:folders].reverse.map{ |f| "id=#{f}" }) if params[:folders]
        env.portal_folders = folders
        if env.save
          session[:notice] = _('Saved the portal folders')
@@ -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
 class CategoriesController < AdminController
  
   protect 'manage_environment_categories', :environment
-  
+
   helper :categories
  
   def index
-    @categories = environment.categories.find(:all, :conditions => "parent_id is null AND type is null")
-    @regions = environment.regions.find(:all, :conditions => {:parent_id => nil})
-    @product_categories = environment.product_categories.find(:all, :conditions => {:parent_id => nil})
+    @categories = environment.categories.where("parent_id is null AND type is null")
+    @regions = environment.regions.where(:parent_id => nil)
+    @product_categories = environment.product_categories.where(:parent_id => nil)
   end
  
   def get_children
@@ -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'
@@ -2,7 +2,7 @@ class EnvironmentRoleManagerController &lt; AdminController
   protect 'manage_environment_roles', :environment
  
   def index
-    @admins = Person.find(:all, :conditions => ['role_assignments.resource_type = ?', 'Environment'], :include => :role_assignments )
+    @admins = Person.where('role_assignments.resource_type = ?', 'Environment').includes(:role_assignments)
   end
  
   def change_roles
 class FeaturesController < AdminController
   protect 'edit_environment_features', :environment
-  
+
   def index
     @features = Environment.available_features.sort_by{|k,v|v}
   end
  
   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
@@ -53,7 +53,7 @@ class FeaturesController &lt; AdminController
  
   def search_members
     arg = params[:q].downcase
-    result = environment.people.find(:all, :conditions => ['LOWER(name) LIKE ? OR identifier LIKE ?', "%#{arg}%", "%#{arg}%"])
+    result = environment.people.where('LOWER(name) LIKE ? OR identifier LIKE ?', "%#{arg}%", "%#{arg}%")
     render :text => prepare_to_token_input(result).to_json
   end
  
@@ -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')
@@ -63,7 +63,7 @@ class UsersController &lt; AdminController
     respond_to do |format|
       format.html
       format.xml do
-        users = User.find(:all, :conditions => {:environment_id => environment.id}, :include => [:person])
+        users = User.where(:environment_id => environment.id).includes(:person)
         send_data users.to_xml(
             :skip_types => true,
             :only => %w[email login created_at updated_at],
@@ -1 +0,0 @@
-require 'application_controller'
@@ -8,7 +8,7 @@ class ApplicationController &lt; ActionController::Base
   before_filter :init_noosfero_plugins
   before_filter :allow_cross_domain_access
  
-  before_filter :login_from_cookie
+  include AuthenticatedSystem
   before_filter :require_login_for_environment, :if => :private_environment?
  
   before_filter :verify_members_whitelist, :if => [:private_environment?, :user]
@@ -45,7 +45,7 @@ class ApplicationController &lt; ActionController::Base
   include ApplicationHelper
   layout :get_layout
   def get_layout
-    return nil if request.format == :js or request.xhr?
+    return false if request.format == :js or request.xhr?
  
     theme_layout = theme_option(:layout)
     if theme_layout
@@ -69,15 +69,12 @@ class ApplicationController &lt; ActionController::Base
   helper :language
  
   include DesignHelper
-
-  # Be sure to include AuthenticationSystem in Application Controller instead
-  include AuthenticatedSystem
   include PermissionCheck
  
   before_filter :set_locale
   def set_locale
     FastGettext.available_locales = environment.available_locales
-    FastGettext.default_locale = environment.default_locale
+    FastGettext.default_locale = environment.default_locale || 'en'
     FastGettext.locale = (params[:lang] || session[:lang] || environment.default_locale || request.env['HTTP_ACCEPT_LANGUAGE'] || 'en')
     I18n.locale = FastGettext.locale.to_s.gsub '_', '-'
     I18n.default_locale = FastGettext.default_locale.to_s.gsub '_', '-'
@@ -148,7 +145,7 @@ class ApplicationController &lt; ActionController::Base
       # Check if the requested profile belongs to another domain
       if @profile && !params[:profile].blank? && params[:profile] != @profile.identifier
         @profile = @environment.profiles.find_by_identifier params[:profile]
-        redirect_to params.merge(:host => @profile.default_hostname)
+        redirect_to url_for(params.merge host: @profile.default_hostname)
       end
     end
   end
@@ -163,7 +160,8 @@ class ApplicationController &lt; ActionController::Base
   def render_not_found(path = nil)
     @no_design_blocks = true
     @path ||= request.path
-    render :template => 'shared/not_found.html.erb', :status => 404, :layout => get_layout
+    # force html template even if the browser asked for a image
+    render template: 'shared/not_found', status: 404, layout: get_layout, formats: [:html]
   end
   alias :render_404 :render_not_found
  
@@ -171,7 +169,8 @@ class ApplicationController &lt; ActionController::Base
     @no_design_blocks = true
     @message = message
     @title = title
-    render :template => 'shared/access_denied.html.erb', :status => 403
+    # force html template even if the browser asked for a image
+    render template: 'shared/access_denied', status: 403, formats: [:html]
   end
  
   def load_category
@@ -204,7 +203,7 @@ class ApplicationController &lt; ActionController::Base
   def redirect_to_current_user
     if params[:profile] == '~'
       if logged_in?
-        redirect_to params.merge(:profile => user.identifier)
+        redirect_to url_for(params.merge profile: user.identifier)
       else
         render_not_found
       end
@@ -84,9 +84,9 @@ class BoxOrganizerController &lt; ApplicationController
     if request.xhr? and params[:query]
       search = params[:query]
       path_list = if boxes_holder.is_a?(Environment) && boxes_holder.enabled?('use_portal_community') && boxes_holder.portal_community
-        boxes_holder.portal_community.articles.find(:all, :conditions=>"name ILIKE '%#{search}%' or path ILIKE '%#{search}%'", :limit=>20).map { |content| "/{portal}/"+content.path }
+                    boxes_holder.portal_community.articles.where("name ILIKE ? OR path ILIKE ?", "%#{search}%", "%#{search}%").limit(20).map { |content| "/{portal}/"+content.path }
       elsif boxes_holder.is_a?(Profile)
-        boxes_holder.articles.find(:all, :conditions=>"name ILIKE '%#{search}%' or path ILIKE '%#{search}%'", :limit=>20).map { |content| "/{profile}/"+content.path }
+        boxes_holder.articles.where("name ILIKE ? OR path ILIKE ?", "%#{search}%", "%#{search}%").limit(20).map { |content| "/{profile}/"+content.path }
       else
         []
       end
@@ -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
  
@@ -6,7 +6,7 @@ class CmsController &lt; MyProfileController
  
   def search_tags
     arg = params[:term].downcase
-    result = ActsAsTaggableOn::Tag.where('name ILIKE ?', "%#{arg}%").limit(10)
+    result = Tag.where('name ILIKE ?', "%#{arg}%").limit(10)
     render :text => prepare_to_token_input_by_label(result).to_json, :content_type => 'application/json'
   end
  
@@ -50,16 +50,9 @@ class CmsController &lt; MyProfileController
  
   def view
     @article = profile.articles.find(params[:id])
-    conditions = []
-    if @article.has_posts?
-      conditions = ['type != ?', 'RssFeed']
-    end
-
-    @articles = @article.children.reorder("case when type = 'Folder' then 0 when type ='Blog' then 1 else 2 end, updated_at DESC, name").paginate(
-      :conditions => conditions,
-      :per_page => per_page,
-      :page => params[:npage]
-    )
+    @articles = @article.children.reorder("case when type = 'Folder' then 0 when type ='Blog' then 1 else 2 end, updated_at DESC, name")
+    @articles = @articles.where "type <> ?", 'RssFeed' if @article.has_posts?
+    @articles = @articles.paginate per_page: per_page, page: params[:npage]
   end
  
   def index
@@ -99,7 +92,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
@@ -230,7 +223,7 @@ class CmsController &lt; MyProfileController
       else
         session[:notice] = _('File(s) successfully uploaded')
         if @back_to
-          redirect_to @back_to
+          redirect_to url_for(@back_to)
         elsif @parent
           redirect_to :action => 'view', :id => @parent.id
         else
@@ -381,7 +374,7 @@ class CmsController &lt; MyProfileController
  
   def search_article_privacy_exceptions
     arg = params[:q].downcase
-    result = profile.members.find(:all, :conditions => ['LOWER(name) LIKE ?', "%#{arg}%"])
+    result = profile.members.where('LOWER(name) LIKE ?', "%#{arg}%")
     render :text => prepare_to_token_input(result).to_json
   end
  
@@ -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'
@@ -27,7 +27,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?
@@ -83,7 +83,7 @@ class ProfileEditorController &lt; MyProfileController
       if @profile.destroy
         session[:notice] = _('The profile was deleted.')
         if(params[:return_to])
-          redirect_to params[:return_to]
+          redirect_to url_for(params[:return_to])
         else
           redirect_to :controller => 'home'
         end
@@ -97,7 +97,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.')
@@ -47,7 +47,7 @@ class ProfileMembersController &lt; MyProfileController
   end
  
   def remove_role
-    @association = RoleAssignment.find(:all, :conditions => {:id => params[:id], :target_id => profile.id})
+    @association = RoleAssignment.where(:id => params[:id], :target_id => profile.id)
     if @association.destroy
       session[:notice] = 'Member succefully unassociated'
     else
@@ -120,7 +120,7 @@ class ProfileMembersController &lt; MyProfileController
  
   def search_user
     role = Role.find(params[:role])
-    render :text => environment.people.find(:all, :conditions => ['LOWER(name) LIKE ? OR LOWER(identifier) LIKE ?', "%#{params['q_'+role.key]}%", "%#{params['q_'+role.key]}%"]).
+    render :text => environment.people.where('LOWER(name) LIKE ? OR LOWER(identifier) LIKE ?', "%#{params['q_'+role.key]}%", "%#{params['q_'+role.key]}%").
       select { |person| !profile.members_by_role(role).include?(person) }.
       map {|person| {:id => person.id, :name => person.name} }.
       to_json
@@ -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})"
@@ -86,7 +86,7 @@ class TasksController &lt; MyProfileController
   end
  
   def ticket_details
-    @ticket = Ticket.find(:first, :conditions => ['(requestor_id = ? or target_id = ?) and id = ?', profile.id, profile.id, params[:id]])
+    @ticket = Ticket.where('(requestor_id = ? or target_id = ?) and id = ?', profile.id, profile.id, params[:id]).first
   end
  
 end
@@ -117,9 +117,9 @@ class AccountController &lt; ApplicationController
           @user.signup!
           owner_role = Role.find_by_name('owner')
           @user.person.affiliate(@user.person, [owner_role]) if owner_role
-          invitation = Task.find_by_code(@invitation_code)
+          invitation = Task.from_code(@invitation_code).first
           if invitation
-            invitation.update_attributes!({:friend => @user.person})
+            invitation.update! friend: @user.person
             invitation.finish
           end
  
@@ -209,7 +209,7 @@ class AccountController &lt; ApplicationController
   #
   # Posts back.
   def new_password
-    @change_password = ChangePassword.find_by_code(params[:code])
+    @change_password = ChangePassword.from_code(params[:code]).first
  
     unless @change_password
       render :action => 'invalid_change_password_code', :status => 403
@@ -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
@@ -402,7 +402,7 @@ class AccountController &lt; ApplicationController
   end
  
   def load_enterprise_activation
-    @enterprise_activation ||= EnterpriseActivation.find_by_code(params[:enterprise_code])
+    @enterprise_activation ||= EnterpriseActivation.from_code(params[:enterprise_code]).first
   end
  
   def load_enterprise
@@ -426,7 +426,7 @@ class AccountController &lt; ApplicationController
  
   def go_to_initial_page
     if params[:return_to]
-      redirect_to params[:return_to]
+      redirect_to url_for(params[:return_to])
     elsif environment.enabled?('allow_change_of_redirection_after_login')
       check_redirection_options(user, user.preferred_login_redirection, user.admin_url)
     else
@@ -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|
@@ -4,7 +4,7 @@ class EnterpriseRegistrationController &lt; ApplicationController
   before_filter :login_required
  
   # Just go to the first step.
-  # 
+  #
   # FIXME: shouldn't this action present some sort of welcome message and point
   # to the first step explicitly?
   def index
@@ -15,7 +15,7 @@ class EnterpriseRegistrationController &lt; ApplicationController
         @create_enterprise.target = Profile.find(params[:create_enterprise][:target_id])
       end
     elsif @validation == :admin || @validation == :none
-        @create_enterprise.target = environment
+      @create_enterprise.target = environment
     end
     @create_enterprise.requestor = user
     the_action =
@@ -40,7 +40,7 @@ class EnterpriseRegistrationController &lt; ApplicationController
  
   protected
  
-  # Fill in the form and select your Region. 
+  # Fill in the form and select your Region.
   #
   # Posts back.
   def basic_information
@@ -37,7 +37,7 @@ class ProfileController &lt; PublicController
  
   def tag_feed
     @tag = params[:id]
-    tagged = profile.articles.paginate(:per_page => 20, :page => 1, :order => 'published_at DESC', :include => :tags, :conditions => ['tags.name LIKE ?', @tag])
+    tagged = profile.articles.paginate(:per_page => 20, :page => 1).order('published_at DESC').joins(:tags).where('tags.name LIKE ?', @tag)
     feed_writer = FeedWriter.new
     data = feed_writer.write(
       tagged,
@@ -205,7 +205,7 @@ class ProfileController &lt; PublicController
  
   def more_comments
     profile_filter = @profile.person? ? {:user_id => @profile} : {:target_id => @profile}
-    activity = ActionTracker::Record.find(:first, :conditions => {:id => params[:activity]}.merge(profile_filter))
+    activity = ActionTracker::Record.where({:id => params[:activity]}.merge profile_filter).first
     comments_count = activity.comments.count
     comment_page = (params[:comment_page] || 1).to_i
     comments_per_page = 5
@@ -225,7 +225,7 @@ class ProfileController &lt; PublicController
   end
  
   def more_replies
-    activity = Scrap.find(:first, :conditions => {:id => params[:activity], :receiver_id => @profile, :scrap_id => nil})
+    activity = Scrap.where(:id => params[:activity], :receiver_id => @profile, :scrap_id => nil).first
     comments_count = activity.replies.count
     comment_page = (params[:comment_page] || 1).to_i
     comments_per_page = 5
@@ -272,7 +272,7 @@ class ProfileController &lt; PublicController
   def remove_notification
     begin
       raise if !can_edit_profile
-      notification = ActionTrackerNotification.find(:first, :conditions => {:profile_id => profile.id, :action_tracker_id => params[:activity_id]})
+      notification = ActionTrackerNotification.where(profile_id: profile.id, action_tracker_id: params[:activity_id]).first
       notification.destroy
       render :text => _('Notification successfully removed.')
     rescue
@@ -14,7 +14,7 @@ class SearchController &lt; PublicController
   # Backwards compatibility with old URLs
   def redirect_asset_param
     return unless params.has_key?(:asset)
-    redirect_to params.merge(:action => params.delete(:asset))
+    redirect_to url_for(params.merge action: params.delete(:asset))
   end
  
   no_design_blocks
@@ -62,12 +62,12 @@ class SearchController &lt; PublicController
   end
  
   def articles
-    @scope = @environment.articles.public
+    @scope = @environment.articles.is_public
     full_text_search
   end
  
   def contents
-    redirect_to params.merge(:action => :articles)
+    redirect_to url_for(params.merge action: :articles)
   end
  
   def people
@@ -122,7 +122,7 @@ class SearchController &lt; PublicController
   # keep old URLs workings
   def assets
     params[:action] = params[:asset].is_a?(Array) ? :index : params.delete(:asset)
-    redirect_to params
+    redirect_to url_for(params)
   end
  
   def tags
@@ -8,6 +8,10 @@ module ApplicationHelper
  
   include PermissionNameHelper
  
+  include UrlHelper
+
+  include PartialsHelper
+
   include ModalHelper
  
   include BoxesHelper
@@ -281,36 +285,6 @@ module ApplicationHelper
     concat(content_tag('div', capture(&block).to_s + tag('br', :style => 'clear: left;'), options))
   end
  
-
-  def partial_for_class_in_view_path(klass, view_path, prefix = nil, suffix = nil)
-    return nil if klass.nil?
-    name = [prefix, klass.name.underscore, suffix].compact.map(&:to_s).join('_')
-
-    search_name = String.new(name)
-    if search_name.include?("/")
-      search_name.gsub!(/(\/)([^\/]*)$/,'\1_\2')
-      name = File.join(params[:controller], name) if defined?(params) && params[:controller]
-    else
-      search_name = "_" + search_name
-    end
-
-    path = defined?(params) && params[:controller] ? File.join(view_path, params[:controller], search_name + '.html.erb') : File.join(view_path, search_name + '.html.erb')
-    return name if File.exists?(File.join(path))
-
-    partial_for_class_in_view_path(klass.superclass, view_path, prefix, suffix)
-  end
-
-  def partial_for_class(klass, prefix=nil, suffix=nil)
-    raise ArgumentError, 'No partial for object. Is there a partial for any class in the inheritance hierarchy?' if klass.nil?
-    name = klass.name.underscore
-    controller.view_paths.each do |view_path|
-      partial = partial_for_class_in_view_path(klass, view_path, prefix, suffix)
-      return partial if partial
-    end
-
-    raise ArgumentError, 'No partial for object. Is there a partial for any class in the inheritance hierarchy?'
-  end
-
   def render_profile_actions klass
     name = klass.to_s.underscore
     begin
@@ -907,8 +881,14 @@ module ApplicationHelper
   end
   alias :top_url :base_url
  
+  class View < ActionView::Base
+    def url_for *args
+      self.controller.url_for *args
+    end
+  end
+
   def helper_for_article(article)
-    article_helper = ActionView::Base.new
+    article_helper = View.new
     article_helper.controller = controller
     article_helper.extend ArticleHelper
     article_helper.extend Rails.application.routes.url_helpers
@@ -19,7 +19,7 @@ module BlockHelper
         content_tag('span', _('Title')) +
         text_field_tag('block[images][][title]', image[:title], :class => 'highlight-title', :size => 45)
       }</label></td>
-      <td>#{button_without_text(:delete, _('Remove'), '#', class: 'delete-highlight', :confirm=>_('Are you sure you want to remove this highlight'))}</td>
+      <td>#{button_without_text(:delete, _('Remove'), '#', class: 'delete-highlight', data: {confirm: _('Are you sure you want to remove this highlight')})}</td>
     </tr>
     "
   end
@@ -236,7 +236,7 @@ module BoxesHelper
     end
  
     if movable?(block) && !block.main?
-      buttons << icon_button(:delete, _('Remove block'), { :action => 'remove', :id => block.id }, { :method => 'post', :confirm => _('Are you sure you want to remove this block?')})
+      buttons << icon_button(:delete, _('Remove block'), { action: 'remove', id: block.id }, method: 'post', data: {confirm: _('Are you sure you want to remove this block?')})
       buttons << icon_button(:clone, _('Clone'), { :action => 'clone_block', :id => block.id }, { :method => 'post' })
     end
  
@@ -34,7 +34,7 @@ module CmsHelper
   end
  
   def display_delete_button(article)
-    expirable_button article, :delete, _('Delete'), { :action => 'destroy', :id => article.id }, :method => :post, :confirm => delete_article_message(article)
+    expirable_button article, :delete, _('Delete'), { :action => 'destroy', :id => article.id }, :method => :post, 'data-confirm' => delete_article_message(article)
   end
  
   def expirable_button(content, action, title, url, options = {})
@@ -42,7 +42,7 @@ module CmsHelper
     if reason.present?
       options[:class] = (options[:class] || '') + ' disabled'
       options[:disabled] = 'disabled'
-      options.delete(:confirm)
+      options.delete('data-confirm')
       options.delete(:method)
       title = reason
     end
@@ -24,7 +24,7 @@ module EventsHelper
   end
  
   def populate_calendar(selected_date, events)
-    events.reject! {|event| !event.display_to?(user)}
+    events = events.reject{ |event| !event.display_to? user }
     calendar = Event.date_range(selected_date.year, selected_date.month).map do |date|
       [
         # the day itself
@@ -5,13 +5,11 @@ module FolderHelper
   def list_contents(configure={})
     configure[:recursive] ||= false
     configure[:list_type] ||= :folder
-    if !configure[:contents].blank?
-      configure[:contents] = configure[:contents].paginate(
-        :order => "name ASC",
-        :per_page => 30,
-        :page => params[:npage]
-      )
-
+    contents = configure[:contents]
+    contents = contents.order('name ASC') unless contents.is_a? Array
+    contents = contents.paginate per_page: 30, page: params[:npage]
+    configure[:contents] = contents
+    if contents.present?
       render :file => 'shared/content_list', :locals => configure
     else
       content_tag('em', _('(empty folder)'))
@@ -0,0 +1,43 @@
+module PartialsHelper
+
+  def partial_for_class_in_view_path(klass, view_path, prefix = nil, suffix = nil)
+    return nil if klass.nil?
+    name = [prefix, klass.name.underscore, suffix].compact.map(&:to_s).join('_')
+
+    search_name = String.new(name)
+    if search_name.include?("/")
+      search_name.gsub!(/(\/)([^\/]*)$/,'\1_\2')
+      name = File.join(params[:controller], name) if defined?(params) && params[:controller]
+    else
+      search_name = "_" + search_name
+    end
+
+    path = defined?(params) && params[:controller] ? File.join(view_path, params[:controller], search_name + '.html.erb') : File.join(view_path, search_name + '.html.erb')
+    return name if File.exists?(File.join(path))
+
+    partial_for_class_in_view_path(klass.superclass, view_path, prefix, suffix)
+  end
+
+  def partial_for_class(klass, prefix=nil, suffix=nil)
+    raise ArgumentError, 'No partial for object. Is there a partial for any class in the inheritance hierarchy?' if klass.nil?
+    name = klass.name.underscore
+    controller.view_paths.each do |view_path|
+      partial = partial_for_class_in_view_path(klass, view_path, prefix, suffix)
+      return partial if partial
+    end
+
+    raise ArgumentError, 'No partial for object. Is there a partial for any class in the inheritance hierarchy?'
+  end
+
+  def render_partial_for_class klass, *args
+    raise ArgumentError, 'No partial for object. Is there a partial for any class in the inheritance hierarchy?' if klass.nil?
+    begin
+      partial = klass.name.underscore
+      partial = "#{params[:controller]}/#{partial}" if params[:controller] and partial.index '/'
+      return render partial, *args
+    rescue ActionView::MissingTemplate
+      return render_partial_for_class klass.superclass, *args
+    end
+  end
+
+end
@@ -0,0 +1,7 @@
+module UrlHelper
+
+  def back_url
+    'javascript:history.back()'
+  end
+
+end
@@ -0,0 +1,18 @@
+require_relative '../helpers/application_helper'
+
+class ApplicationMailer < ActionMailer::Base
+
+  include AuthenticatedSystem
+
+  helper ApplicationHelper
+
+  attr_accessor :environment
+
+  def default_url_options options = nil
+    options ||= {}
+    options[:host] = environment.default_hostname if environment
+    options
+  end
+
+end
+
-class CommentNotifier < ActionMailer::Base
+class CommentNotifier < ApplicationMailer
+
   def notification(comment)
     profile = comment.article.profile
+    self.environment = profile.environment
     @recipient = profile.nickname || profile.name
     @sender = comment.author_name
     @sender_link = comment.author_link
@@ -8,7 +10,6 @@ class CommentNotifier &lt; ActionMailer::Base
     @comment_url = comment.url
     @comment_title = comment.title
     @comment_body = comment.body
-    @environment = profile.environment.name
     @url = profile.environment.top_url
  
     mail(
@@ -20,6 +21,8 @@ class CommentNotifier &lt; ActionMailer::Base
  
   def mail_to_followers(comment, emails)
     profile = comment.article.profile
+    self.environment = profile.environment
+
     @recipient = profile.nickname || profile.name
     @sender = comment.author_name
     @sender_link = comment.author_link
@@ -28,7 +31,6 @@ class CommentNotifier &lt; ActionMailer::Base
     @unsubscribe_url = comment.article.view_url.merge({:unfollow => true})
     @comment_title = comment.title
     @comment_body = comment.body
-    @environment = profile.environment.name
     @url = profile.environment.top_url
  
     mail(
@@ -30,14 +30,16 @@ class Contact
     Contact::Sender.notification(self).deliver
   end
  
-  class Sender < ActionMailer::Base
+  class Sender < ApplicationMailer
+
     def notification(contact)
+      self.environment = contact.dest.environment
+
       @name = contact.name
       @email = contact.email
       @city = contact.city
       @state = contact.state
       @message = contact.message
-      @environment = contact.dest.environment.name
       @url = url_for(:host => contact.dest.environment.default_hostname, :controller => 'home')
       @target = contact.dest.name
  
 class EnvironmentMailing < Mailing
  
   def recipients(offset=0, limit=100)
-   source.people.all(:order => :id, :offset => offset, :limit => limit, :joins => "LEFT OUTER JOIN mailing_sents m ON (m.mailing_id = #{id} AND m.person_id = profiles.id)", :conditions => { "m.person_id" => nil })
+    source.people.order(:id).offset(offset).limit(limit)
+      .joins("LEFT OUTER JOIN mailing_sents m ON (m.mailing_id = #{id} AND m.person_id = profiles.id)")
+      .where("m.person_id" => nil)
   end
  
   def each_recipient
@@ -46,7 +46,8 @@ class Mailing &lt; ActiveRecord::Base
     end
   end
  
-  class Sender < ActionMailer::Base
+  class Sender < ApplicationMailer
+
     def notification(mailing, recipient)
       @message = mailing.body
       @signature_message = mailing.signature_message
@@ -5,7 +5,9 @@ class OrganizationMailing &lt; Mailing
   end
  
   def recipients(offset=0, limit=100)
-    source.members.all(:order => :id, :offset => offset, :limit => limit, :joins => "LEFT OUTER JOIN mailing_sents m ON (m.mailing_id = #{id} AND m.person_id = profiles.id)", :conditions => { "m.person_id" => nil })
+    source.members.order(:id).offset(offset).limit(limit)
+      .joins("LEFT OUTER JOIN mailing_sents m ON (m.mailing_id = #{id} AND m.person_id = profiles.id)")
+      .where("m.person_id" => nil)
   end
  
   def each_recipient
-class PendingTaskNotifier < ActionMailer::Base
+class PendingTaskNotifier < ApplicationMailer
  
   def notification(person)
+    self.environment = person.environment
+
     @person = person
     @tasks = person.tasks.pending
     @organizations_with_pending_tasks = person.organizations_with_pending_tasks
-    @environment = person.environment.name
     @url = url_for(:host => person.environment.default_hostname, :controller => 'home')
     @default_hostname = person.environment.default_hostname
     @url_for_pending_tasks = url_for(:host => person.environment.default_hostname, :controller => 'tasks', :profile => person.identifier)
-class ScrapNotifier < ActionMailer::Base
+class ScrapNotifier < ApplicationMailer
+
   def notification(scrap)
     sender, receiver = scrap.sender, scrap.receiver
+    self.environment = sender.environment
     # for tests
     return unless receiver.email
  
@@ -9,7 +11,6 @@ class ScrapNotifier &lt; ActionMailer::Base
     @sender_link = sender.url
     @scrap_content = scrap.content
     @wall_url = scrap.scrap_wall_url
-    @environment = sender.environment.name
     @url = sender.environment.top_url
     mail(
       to: receiver.email,
-class TaskMailer < ActionMailer::Base
+class TaskMailer < ApplicationMailer
  
   def target_notification(task, message)
+    self.environment = task.environment
+
     @message = extract_message(message)
     @target = task.target.name
-    @environment = task.environment.name
     @url = generate_environment_url(task, :controller => 'home')
     url_for_tasks_list = task.target.kind_of?(Environment) ? '' : url_for(task.target.tasks_url.merge(:script_name => Noosfero.root('/')))
     @tasks_url = url_for_tasks_list
@@ -16,6 +17,8 @@ class TaskMailer &lt; ActionMailer::Base
   end
  
   def invitation_notification(task)
+    self.environment = task.requestor.environment
+
     msg = task.expanded_message
     @message = msg.gsub /<url>/, generate_environment_url(task, :controller => 'account', :action => 'signup', :invitation_code => task.code)
  
@@ -27,11 +30,12 @@ class TaskMailer &lt; ActionMailer::Base
   end
  
   def generic_message(name, task)
+    self.environment = task.requestor.environment
+
     return if !task.respond_to?("#{name}_message")
  
     @message = extract_message(task.send("#{name}_message"))
     @requestor = task.requestor.name
-    @environment = task.requestor.environment.name
     @url = url_for(:host => task.requestor.environment.default_hostname, :controller => 'home')
  
     mail(
-class UserMailer < ActionMailer::Base
+class UserMailer < ApplicationMailer
+
   def activation_email_notify(user)
+    self.environment = user.environment
+
     user_email = "#{user.login}@#{user.email_domain}"
     @name = user.name
     @email = user_email
     @webmail = MailConf.webmail_url(user.login, user.email_domain)
-    @environment = user.environment.name
     @url = url_for(:host => user.environment.default_hostname, :controller => 'home')
  
     mail(
@@ -15,9 +17,10 @@ class UserMailer &lt; ActionMailer::Base
   end
  
   def activation_code(user)
+    self.environment = user.environment
+
     @recipient = user.name
     @activation_code = user.activation_code
-    @environment = user.environment.name
     @url = user.environment.top_url
     @redirection = (true if user.return_to)
     @join = (user.community_to_join if user.community_to_join)
@@ -30,6 +33,8 @@ class UserMailer &lt; ActionMailer::Base
   end
  
   def signup_welcome_email(user)
+    self.environment = user.environment
+
     @body = user.environment.signup_welcome_text_body.gsub('{user_name}', user.name)
     email_subject = user.environment.signup_welcome_text_subject
     mail(
@@ -42,8 +47,9 @@ class UserMailer &lt; ActionMailer::Base
   end
  
   def profiles_suggestions_email(user)
+    self.environment = user.environment
+
     @recipient = user.name
-    @environment = user.environment.name
     @url = user.environment.top_url
     @people_suggestions_url = user.people_suggestions_url
     @people_suggestions = user.suggested_people.sample(3)
@@ -11,7 +11,7 @@ class AddMember &lt; Task
   alias :organization :target
   alias :organization= :target=
  
-  settings_items :roles
+  settings_items :roles, type: Array
  
   after_create do |task|
     remove_from_suggestion_list(task)
@@ -8,7 +8,7 @@ class ApproveComment &lt; Task
   def comment
     unless @comment || self.comment_attributes.nil?
       @comment = Comment.new
-      @comment.assign_attributes(ActiveSupport::JSON.decode(self.comment_attributes), :without_protection => true)
+      @comment.assign_attributes(ActiveSupport::JSON.decode(self.comment_attributes.to_s), :without_protection => true)
     end
     @comment
   end
@@ -79,7 +79,7 @@ class Article &lt; ActiveRecord::Base
  
   has_many :comments, :class_name => 'Comment', :foreign_key => 'source_id', :dependent => :destroy, :order => 'created_at asc'
  
-  has_many :article_categorizations, :conditions => [ 'articles_categories.virtual = ?', false ]
+  has_many :article_categorizations, -> { where 'articles_categories.virtual = ?', false }
   has_many :categories, :through => :article_categorizations
  
   has_many :article_categorizations_including_virtual, :class_name => 'ArticleCategorization'
@@ -131,17 +131,15 @@ class Article &lt; ActiveRecord::Base
  
   xss_terminate :only => [ :name ], :on => 'validation', :with => 'white_list'
  
-  scope :in_category, lambda { |category|
-    {:include => 'categories_including_virtual', :conditions => { 'categories.id' => category.id }}
+  scope :in_category, -> category {
+    includes('categories_including_virtual').where('categories.id' => category.id)
   }
  
   include TimeScopes
  
-  scope :by_range, lambda { |range| {
-    :conditions => [
-      'articles.published_at BETWEEN :start_date AND :end_date', { :start_date => range.first, :end_date => range.last }
-    ]
-  }}
+  scope :by_range, -> range {
+    where 'articles.published_at BETWEEN :start_date AND :end_date', { start_date: range.first, end_date: range.last }
+  }
  
   URL_FORMAT = /\A(http|https):\/\/[a-z0-9]+([\-\.]{1}[a-z0-9]+)*\.[a-z]{2,5}(([0-9]{1,5})?\/.*)?\Z/ix
  
@@ -263,19 +261,21 @@ class Article &lt; ActiveRecord::Base
  
   # retrieves all articles belonging to the given +profile+ that are not
   # sub-articles of any other article.
-  scope :top_level_for, lambda { |profile|
-    {:conditions => [ 'parent_id is null and profile_id = ?', profile.id ]}
+  scope :top_level_for, -> profile {
+    where 'parent_id is null and profile_id = ?', profile.id
   }
  
-  scope :public,
-    :conditions => [ "advertise = ? AND published = ? AND profiles.visible = ? AND profiles.public_profile = ?", true, true, true, true ], :joins => [:profile]
+  scope :is_public, -> {
+    joins(:profile).
+    where("articles.advertise = ? AND articles.published = ? AND profiles.visible = ? AND profiles.public_profile = ?", true, true, true, true)
+  }
  
-  scope :more_recent,
-    :conditions => [ "advertise = ? AND published = ? AND profiles.visible = ? AND profiles.public_profile = ? AND
-      ((articles.type != ?) OR articles.type is NULL)",
-      true, true, true, true, 'RssFeed'
-    ],
-    :order => 'articles.published_at desc, articles.id desc'
+  scope :more_recent, -> {
+    order('articles.published_at desc, articles.id desc')
+    .where("articles.advertise = ? AND articles.published = ? AND profiles.visible = ? AND profiles.public_profile = ? AND
+    ((articles.type != ?) OR articles.type is NULL)",
+    true, true, true, true, 'RssFeed')
+  }
  
   # retrives the most commented articles, sorted by the comment count (largest
   # first)
@@ -283,12 +283,14 @@ class Article &lt; ActiveRecord::Base
     paginate(:order => 'comments_count DESC', :page => 1, :per_page => limit)
   end
  
-  scope :more_popular, :order => 'hits DESC'
-  scope :relevant_as_recent, :conditions => ["(articles.type != 'UploadedFile' and articles.type != 'RssFeed' and articles.type != 'Blog') OR articles.type is NULL"]
+  scope :more_popular, -> { order 'hits DESC' }
+  scope :relevant_as_recent, -> {
+   where "(articles.type != 'UploadedFile' and articles.type != 'RssFeed' and articles.type != 'Blog') OR articles.type is NULL"
+  }
  
   def self.recent(limit = nil, extra_conditions = {}, pagination = true)
     result = scoped({:conditions => extra_conditions}).
-      public.
+      is_public.
       relevant_as_recent.
       limit(limit).
       order(['articles.published_at desc', 'articles.id desc'])
@@ -413,7 +415,7 @@ class Article &lt; ActiveRecord::Base
     self.translations.map(&:language)
   end
  
-  scope :native_translations, :conditions => { :translation_of_id => nil }
+  scope :native_translations, -> { where :translation_of_id => nil }
  
   def translatable?
     false
@@ -455,7 +457,7 @@ class Article &lt; ActiveRecord::Base
  
   def rotate_translations
     unless self.translations.empty?
-      rotate = self.translations
+      rotate = self.translations.all
       root = rotate.shift
       root.update_attribute(:translation_of_id, nil)
       root.translations = rotate
@@ -468,7 +470,7 @@ class Article &lt; ActiveRecord::Base
     elsif self.native_translation.language == locale
       self.native_translation
     else
-      self.native_translation.translations.first(:conditions => { :language => locale })
+      self.native_translation.translations.where(:language => locale).first
     end
   end
  
@@ -492,19 +494,19 @@ class Article &lt; ActiveRecord::Base
     ['TextArticle', 'TextileArticle', 'TinyMceArticle']
   end
  
-  scope :published, :conditions => ['articles.published = ?', true]
-  scope :folders, lambda {|profile|{:conditions => ['articles.type IN (?)', profile.folder_types] }}
-  scope :no_folders, lambda {|profile|{:conditions => ['articles.type NOT IN (?)', profile.folder_types]}}
-  scope :galleries, :conditions => [ "articles.type IN ('Gallery')" ]
-  scope :images, :conditions => { :is_image => true }
-  scope :no_images, :conditions => { :is_image => false }
-  scope :text_articles, :conditions => [ 'articles.type IN (?)', text_article_types ]
-  scope :files, :conditions => { :type => 'UploadedFile' }
-  scope :with_types, lambda { |types| { :conditions => [ 'articles.type IN (?)', types ] } }
+  scope :published, -> { where 'articles.published = ?', true }
+  scope :folders, -> profile { where 'articles.type IN (?)', profile.folder_types }
+  scope :no_folders, -> profile { where 'articles.type NOT IN (?)', profile.folder_types }
+  scope :galleries, -> { where "articles.type IN ('Gallery')" }
+  scope :images, -> { where :is_image => true }
+  scope :no_images, -> { where :is_image => false }
+  scope :text_articles, -> { where 'articles.type IN (?)', text_article_types }
+  scope :files, -> { where :type => 'UploadedFile' }
+  scope :with_types, -> types { where 'articles.type IN (?)', types }
  
-  scope :more_popular, :order => 'hits DESC'
-  scope :more_comments, :order => "comments_count DESC"
-  scope :more_recent, :order => "created_at DESC"
+  scope :more_popular, -> { order 'hits DESC' }
+  scope :more_comments, -> { order "comments_count DESC" }
+  scope :more_recent, -> { order "created_at DESC" }
  
   scope :display_filter, lambda {|user, profile|
     return published if (user.nil? && profile && profile.public?)
@@ -626,7 +628,7 @@ class Article &lt; ActiveRecord::Base
   ]
  
   def self.find_by_old_path(old_path)
-    find(:first, :include => :versions, :conditions => ['article_versions.path = ?', old_path], :order => 'article_versions.id desc')
+    self.includes(:versions).where('article_versions.path = ?', old_path).order('article_versions.id DESC').first
   end
  
   def hit
@@ -704,11 +706,11 @@ class Article &lt; ActiveRecord::Base
   end
  
   def get_version(version_number = nil)
-    version_number ? versions.find(:first, :order => 'version', :offset => version_number - 1) : versions.earliest
+    if version_number then self.versions.order('version').offset(version_number - 1).first else self.versions.earliest end
   end
  
   def author_by_version(version_number = nil)
-    version_number ? profile.environment.people.find_by_id(get_version(version_number).author_id) : author
+    if version_number then profile.environment.people.where(id: get_version(version_number).author_id).first else author end
   end
  
   def author_name(version_number = nil)
@@ -804,7 +806,7 @@ class Article &lt; ActiveRecord::Base
   end
  
   def activity
-    ActionTracker::Record.find_by_target_type_and_target_id 'Article', self.id
+    ActionTracker::Record.where(target_type: 'Article', target_id: self.id).first
   end
  
   def create_activity
 class ArticleCategorization < ActiveRecord::Base
-  set_table_name :articles_categories
+  self.table_name = :articles_categories
+
   belongs_to :article
   belongs_to :category
  
@@ -20,7 +20,7 @@ class Block &lt; ActiveRecord::Base
  
   acts_as_having_settings
  
-  scope :enabled, :conditions => { :enabled => true }
+  scope :enabled, -> { where :enabled => true }
  
   after_save do |block|
     if block.owner.kind_of?(Profile) && block.owner.is_template? && block.mirror?
@@ -15,7 +15,7 @@ class BlogArchivesBlock &lt; Block
     _('Blog posts')
   end
  
-  settings_items :blog_id, Integer
+  settings_items :blog_id, type: Integer
  
   def blog
     blog_id && owner.blogs.exists?(blog_id) ? owner.blogs.find(blog_id) : owner.blog
@@ -33,10 +33,10 @@ class BlogArchivesBlock &lt; Block
     results = ''
     posts = visible_posts(args[:person])
     posts.except(:order).count(:all, :group => 'EXTRACT(YEAR FROM published_at)').sort_by {|year, count| -year.to_i}.each do |year, count|
-      results << content_tag('li', content_tag('strong', "#{year} (#{count})"))
-      results << "<ul class='#{year}-archive'>"
-      posts.except(:order).count(:all, :conditions => ['EXTRACT(YEAR FROM published_at)=?', year], :group => 'EXTRACT(MONTH FROM published_at)').sort_by {|month, count| -month.to_i}.each do |month, count|
-        results << content_tag('li', link_to("#{month_name(month.to_i)} (#{count})", owner_blog.url.merge(:year => year, :month => month)))
+      results << content_tag('li', content_tag('strong', "#{year.to_i} (#{count})"))
+      results << "<ul class='#{year.to_i}-archive'>"
+      posts.except(:order).where('EXTRACT(YEAR FROM published_at)=?', year.to_i).group('EXTRACT(MONTH FROM published_at)').count.sort_by {|month, count| -month.to_i}.each do |month, count|
+        results << content_tag('li', link_to("#{month_name(month.to_i)} (#{count})", owner_blog.url.merge(year: year.to_i, month: month.to_i)))
       end
       results << "</ul>"
     end
@@ -7,7 +7,7 @@ class Box &lt; ActiveRecord::Base
  
   include Noosfero::Plugin::HotSpot
  
-  scope :with_position, :conditions => ['boxes.position > 0']
+  scope :with_position, -> { where 'boxes.position > 0' }
  
   def environment
     owner ? (owner.kind_of?(Environment) ? owner : owner.environment) : nil
 module Categorization
  
   def add_category_to_object(category, object)
-    if !find(:first, :conditions => {object_id_column => object, :category_id => category} )
+    if !self.where(object_id_column => object, :category_id => category).first
       connection.execute("insert into #{table_name} (category_id, #{object_id_column}) values(#{category.id}, #{object.id})")
  
       c = category.parent
-      while !c.nil? && !self.find(:first, :conditions => {object_id_column => object, :category_id => c})
+      while !c.nil? && !self.where(object_id_column => object, :category_id => c).first
         connection.execute("insert into #{table_name} (category_id, #{object_id_column}, virtual) values(#{c.id}, #{object.id}, 1>0)")
         c = c.parent
       end
@@ -14,12 +14,12 @@ class Category &lt; ActiveRecord::Base
   validates_uniqueness_of :slug,:scope => [ :environment_id, :parent_id ], :message => N_('{fn} is already being used by another category.').fix_i18n
   belongs_to :environment
  
-  # Finds all top level categories for a given environment. 
-  scope :top_level_for, lambda { |environment|
-    {:conditions => ['parent_id is null and environment_id = ?', environment.id ]}
+  # Finds all top level categories for a given environment.
+  scope :top_level_for, -> environment {
+    where 'parent_id is null and environment_id = ?', environment.id
   }
  
-  scope :on_level, lambda { |parent| {:conditions => {:parent_id => parent}} }
+  scope :on_level, -> parent { where :parent_id => parent }
  
   acts_as_filesystem
  
@@ -46,26 +46,26 @@ class Category &lt; ActiveRecord::Base
     display_color = nil if display_color.blank?
   end
  
-  scope :from_types, lambda { |types|
-    types.select{ |t| t.blank? }.empty? ?
-      { :conditions => { :type => types } } :
-      { :conditions => [ "type IN (?) OR type IS NULL", types.reject{ |t| t.blank? } ] }
+  scope :from_types, -> types {
+    if types.select{ |t| t.blank? }.empty? then
+      where(type: types) else
+      where("type IN (?) OR type IS NULL", types.reject{ |t| t.blank? }) end
   }
  
   def recent_people(limit = 10)
-    self.people.paginate(:order => 'created_at DESC, id DESC', :page => 1, :per_page => limit)
+    self.people.reorder('created_at DESC, id DESC').paginate(page: 1, per_page: limit)
   end
  
   def recent_enterprises(limit = 10)
-    self.enterprises.paginate(:order => 'created_at DESC, id DESC', :page => 1, :per_page => limit)
+    self.enterprises.reorder('created_at DESC, id DESC').paginate(page: 1, per_page: limit)
   end
  
   def recent_communities(limit = 10)
-    self.communities.paginate(:order => 'created_at DESC, id DESC', :page => 1, :per_page => limit)
+    self.communities.reorder('created_at DESC, id DESC').paginate(page: 1, per_page: limit)
   end
  
   def recent_products(limit = 10)
-    self.products.paginate(:order => 'created_at DESC, id DESC', :page => 1, :per_page => limit)
+    self.products.reorder('created_at DESC, id DESC').paginate(page: 1, per_page: limit)
   end
  
   def recent_articles(limit = 10)
@@ -73,7 +73,7 @@ class Category &lt; ActiveRecord::Base
   end
  
   def recent_comments(limit = 10)
-    comments.paginate(:order => 'created_at DESC, comments.id DESC', :page => 1, :per_page => limit)
+    self.comments.reorder('created_at DESC, comments.id DESC').paginate(page: 1, per_page: limit)
   end
  
   def most_commented_articles(limit = 10)
@@ -81,7 +81,7 @@ class Category &lt; ActiveRecord::Base
   end
  
   def upcoming_events(limit = 10)
-    self.events.paginate(:conditions => [ 'start_date >= ?', DateTime.now.beginning_of_day ], :order => 'start_date', :page => 1, :per_page => limit)
+    self.events.where('start_date >= ?', DateTime.now.beginning_of_day).order('start_date').paginate(page: 1, per_page: limit)
   end
  
   def display_in_menu?
@@ -90,11 +90,11 @@ class Category &lt; ActiveRecord::Base
  
   def children_for_menu
     results = []
-    pending = children.find(:all, :conditions => { :display_in_menu => true})
-    while !pending.empty?
+    pending = children.where(display_in_menu: true).all
+    while pending.present?
       cat = pending.shift
       results << cat
-      pending += cat.children.find(:all, :conditions => { :display_in_menu => true} )
+      pending += cat.children.where :display_in_menu => true
     end
  
     results
@@ -102,7 +102,7 @@ class Category &lt; ActiveRecord::Base
  
   def is_leaf_displayable_in_menu?
     return false if self.display_in_menu == false
-    self.children.find(:all, :conditions => {:display_in_menu => true}).empty?
+    self.children.where(:display_in_menu => true).empty?
   end
  
   def with_color
@@ -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
  
@@ -18,7 +18,7 @@ class Comment &lt; ActiveRecord::Base
   has_many :children, :class_name => 'Comment', :foreign_key => 'reply_of_id', :dependent => :destroy
   belongs_to :reply_of, :class_name => 'Comment', :foreign_key => 'reply_of_id'
  
-  scope :without_reply, :conditions => ['reply_of_id IS NULL']
+  scope :without_reply, -> { where 'reply_of_id IS NULL' }
  
   include TimeScopes
  
@@ -34,7 +34,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!
@@ -16,13 +16,13 @@ class CreateEnterprise &lt; Task
     settings_items field.to_sym
   end
  
-  # checks for virtual attributes 
+  # checks for virtual attributes
   validates_presence_of :name, :identifier
  
   #checks if the validation method is region to validates
   validates_presence_of :region_id, :if => lambda { |obj| obj.environment.organization_approval_method == :region }
  
-  validates_format_of :foundation_year, :with => /^\d*$/
+  validates_numericality_of :foundation_year, only_integer: true, if: -> o { o.foundation_year.present? }
  
   # checks for actual attributes
   validates_presence_of :requestor_id, :target_id
@@ -129,7 +129,7 @@ class CreateEnterprise &lt; Task
     finish
   end
  
-  # tells if this request was appoved 
+  # tells if this request was appoved
   def approved?
     self.status == Task::Status::FINISHED
   end
@@ -14,7 +14,7 @@ class Domain &lt; ActiveRecord::Base
  
   # <tt>name</tt> must be sequences of alphanumeric characters (a to z,
   # 0 to 9), plus '_' or '-', separated by dots. Letters must be lowercase.
-  validates_format_of :name, :with => /^([a-z0-9_-]+\.)+[a-z0-9_-]+$/, :message => N_('{fn} must be composed of sequences of lowercase letters (a to z), numbers (0 to 9), "_" and "-", separated by dots.').fix_i18n
+  validates_format_of :name, with: /\A([a-z0-9_-]+\.)+[a-z0-9_-]+\z/, message: N_('{fn} must be composed of sequences of lowercase letters (a to z), numbers (0 to 9), "_" and "-", separated by dots.').fix_i18n
  
   # checks validations that could not be expressed using Rails' predefined
   # validations. In particular:
@@ -37,7 +37,7 @@ class Domain &lt; ActiveRecord::Base
   # "www.", but it will be removed before searching. So searching for
   # 'www.example.net' is exactly the same as searching for just 'example.net'
   def self.find_by_name(name)
-    self.find(:first, :conditions => [ 'name = ?', self.extract_domain_name(name) ])
+    self.where('name = ?', self.extract_domain_name(name)).first
   end
  
   # turns the argument (expected to be a String) into a domain name that is
@@ -71,7 +71,7 @@ class Enterprise &lt; Organization
   end
  
   def highlighted_products_with_image(options = {})
-    Product.find(:all, {:conditions => {:highlighted => true}, :joins => :image}.merge(options))
+    Product.where(:highlighted => true).joins(:image)
   end
  
   def required_fields
@@ -17,7 +17,10 @@ class Environment &lt; ActiveRecord::Base
  
   has_many :users
  
-  self.partial_updates = false
+  # allow roles use
+  def self.dangerous_attribute_method? name
+    false
+  end
  
   has_many :tasks, :dependent => :destroy, :as => 'target'
   has_many :search_terms, :as => :context
@@ -108,7 +111,7 @@ class Environment &lt; ActiveRecord::Base
   def admins
     admin_role = Environment::Roles.admin(self)
     return [] if admin_role.blank?
-    Person.members_of(self).all(:conditions => ['role_assignments.role_id = ?', admin_role.id])
+    Person.members_of(self).where 'role_assignments.role_id = ?', admin_role.id
   end
  
   # returns the available features for a Environment, in the form of a
@@ -223,9 +226,11 @@ class Environment &lt; ActiveRecord::Base
   has_many :licenses
  
   has_many :categories
-  has_many :display_categories, :class_name => 'Category', :conditions => 'display_color is not null and parent_id is null', :order => 'display_color'
+  has_many :display_categories, -> {
+    order('display_color').where('display_color is not null and parent_id is null')
+  }, class_name: 'Category'
  
-  has_many :product_categories, :conditions => { :type => 'ProductCategory'}
+  has_many :product_categories, -> { where type: 'ProductCategory'}
   has_many :regions
   has_many :states
   has_many :cities
@@ -690,7 +695,7 @@ class Environment &lt; ActiveRecord::Base
  
   # the default Environment.
   def self.default
-    self.find(:first, :conditions => [ 'is_default = ?', true ] )
+    self.where('is_default = ?', true).first
   end
  
   # returns an array with the top level categories for this environment.
@@ -871,7 +876,7 @@ class Environment &lt; ActiveRecord::Base
   end
  
   def portal_folders
-    (settings[:portal_folders] || []).map{|fid| portal_community.articles.find(:first, :conditions => { :id => fid }) }.compact
+    (settings[:portal_folders] || []).map{|fid| portal_community.articles.where(id: fid).first }.compact
   end
  
   def portal_folders=(folders)
@@ -958,7 +963,7 @@ class Environment &lt; ActiveRecord::Base
   end
  
   def highlighted_products_with_image(options = {})
-    Product.find(:all, {:conditions => {:highlighted => true, :profile_id => self.enterprises.find(:all, :select => :id) }, :joins => :image}.merge(options))
+    self.products.where(highlighted: true).joins(:image).order('created_at ASC')
   end
  
   settings_items :home_cache_in_minutes, :type => :integer, :default => 5
@@ -12,14 +12,14 @@ class Event &lt; Article
   settings_items :address, :type => :string
  
   def link=(value)
-    self.setting[:link] = maybe_add_http(value)
+    self.setting[:link] = maybe_add_http(URI.escape value.to_s)
   end
  
   def link
     maybe_add_http(self.setting[:link])
   end
  
-  xss_terminate :only => [ :name, :body, :link, :address ], :with => 'white_list', :on => 'validation'
+  xss_terminate :only => [ :name, :body, :address ], :with => 'white_list', :on => 'validation'
  
   def initialize(*args)
     super(*args)
@@ -34,23 +34,21 @@ class Event &lt; Article
     end
   end
  
-  scope :by_day, lambda { |date|
-    { :conditions => [' start_date >= :start_date AND start_date <= :end_date AND end_date IS NULL OR (start_date <= :end_date  AND end_date >= :start_date)', {:start_date => date.beginning_of_day, :end_date => date.end_of_day}],
-      :order => 'start_date ASC'
-    }
+  scope :by_day, -> date {
+    where('start_date >= :start_date AND start_date <= :end_date AND end_date IS NULL OR (start_date <= :end_date  AND end_date >= :start_date)',
+          start_date: date.beginning_of_day, end_date: date.end_of_day).
+    order('start_date ASC')
   }
  
-  scope :next_events_from_month, lambda { |date|
+  scope :next_events_from_month, -> date {
     date_temp = date.strftime("%Y-%m-%d")
-    { :conditions => ["start_date >= ?","#{date_temp}"],
-      :order => 'start_date ASC'
-    }
+    order('start_date ASC')
+    .where("start_date >= ?","#{date_temp}")
   }
  
-  scope :by_month, lambda { |date|
-    { :conditions => ["EXTRACT(YEAR FROM start_date) = ? AND EXTRACT(MONTH FROM start_date) = ?",date.year,date.month],
-      :order => 'start_date ASC'
-    }
+  scope :by_month, -> date {
+    order('start_date ASC')
+    .where("EXTRACT(YEAR FROM start_date) = ? AND EXTRACT(MONTH FROM start_date) = ?", date.year, date.month)
   }
  
   include WhiteListFilter
@@ -71,12 +69,10 @@ class Event &lt; Article
     'event'
   end
  
-  scope :by_range, lambda { |range| {
-    :conditions => [
-      'start_date BETWEEN :start_day AND :end_day OR end_date BETWEEN :start_day AND :end_day',
-      { :start_day => range.first, :end_day => range.last }
-    ]
-  }}
+  scope :by_range, -> range {
+    where('start_date BETWEEN :start_day AND :end_day OR end_date BETWEEN :start_day AND :end_day',
+      {:start_day => range.first, :end_day => range.last})
+  }
  
   def self.date_range(year, month)
     if year.nil? || month.nil?
@@ -5,9 +5,9 @@ class ExternalFeed &lt; ActiveRecord::Base
   validates_presence_of :address, :if => lambda {|efeed| efeed.enabled}
   validates_uniqueness_of :blog_id
  
-  scope :enabled, :conditions => { :enabled => true }
-  scope :expired, lambda {
-    { :conditions => ['(fetched_at is NULL) OR (fetched_at < ?)', Time.now - FeedUpdater.update_interval] }
+  scope :enabled, -> { where enabled: true }
+  scope :expired, -> {
+    where '(fetched_at is NULL) OR (fetched_at < ?)', Time.now - FeedUpdater.update_interval
   }
  
   attr_accessible :address, :enabled, :only_once
@@ -11,7 +11,7 @@ class FeaturedProductsBlock &lt; Block
     if block.owner.kind_of?(Environment) && block.product_ids.blank?
       total = block.owner.products.count
       offset = rand([(total - block.groups_of * 3) + 1, 1].max)
-      block.product_ids = block.owner.highlighted_products_with_image(:offset => offset, :limit => block.groups_of * 3).map(&:id)
+      block.product_ids = block.owner.highlighted_products_with_image.offset(offset).limit(block.groups_of * 3).map(&:id)
     end
     block.groups_of = block.groups_of.to_i
   end
@@ -27,8 +27,8 @@ class FeedReaderBlock &lt; Block
   settings_items :update_errors, :type => :integer, :default => 0
   settings_items :error_message, :type => :string
  
-  scope :expired, lambda {
-    { :conditions => [ '(fetched_at is NULL) OR (fetched_at < ?)', Time.now - FeedUpdater.update_interval] }
+  scope :expired, -> {
+    where '(fetched_at is NULL) OR (fetched_at < ?)', Time.now - FeedUpdater.update_interval
   }
  
   before_create do |block|
@@ -56,10 +56,11 @@ class Folder &lt; Article
     profile.recent_documents(limit, ["articles.type != ? AND articles.highlighted = ? AND articles.parent_id = ?", 'Folder', highlight, id])
   end
  
-  has_many :images, :class_name => 'Article',
-                    :foreign_key => 'parent_id',
-                    :order => 'articles.type, articles.name',
-                    :conditions => ["articles.type = 'UploadedFile' and articles.content_type in (?) or articles.type in ('Folder','Gallery')", UploadedFile.content_types]
+  has_many :images, -> {
+    order('articles.type, articles.name').
+    where("articles.type = 'UploadedFile' and articles.content_type in (?) or articles.type in ('Folder','Gallery')", UploadedFile.content_types)
+  }, class_name: 'Article', foreign_key: 'parent_id'
+
  
   def accept_uploads?
     !self.has_posts? || self.gallery?
 class Image < ActiveRecord::Base
  
+  attr_accessible :uploaded_data, :label, :remove_image
+  attr_accessor :remove_image
+
   def self.max_size
     Image.attachment_options[:max_size]
   end
  
   sanitize_filename
  
-  has_attachment :content_type => :image, 
-                 :storage => :file_system, 
+  has_attachment :content_type => :image,
+                 :storage => :file_system,
                  :path_prefix => 'public/image_uploads',
                  :resize_to => '800x600>',
                  :thumbnails => { :big      => '150x150',
@@ -15,7 +18,8 @@ class Image &lt; ActiveRecord::Base
                                   :portrait => '64x64',
                                   :minor    => '50x50>',
                                   :icon     => '20x20!' },
-                 :max_size => 5.megabytes # remember to update validate message below
+                 :max_size => 5.megabytes, # remember to update validate message below
+                 processor: 'Rmagick'
  
   validates_attachment :size => N_("{fn} of uploaded file was larger than the maximum size of 5.0 MB").fix_i18n
  
@@ -23,10 +27,8 @@ class Image &lt; ActiveRecord::Base
  
   postgresql_attachment_fu
  
-  attr_accessible :uploaded_data, :label, :remove_image
-  attr_accessor :remove_image
-
   def current_data
     File.file?(full_filename) ? File.read(full_filename) : nil
   end
+
 end
@@ -13,7 +13,7 @@ class Input &lt; ActiveRecord::Base
  
   belongs_to :unit
  
-  scope :relevant_to_price, :conditions => { :relevant_to_price => true }
+  scope :relevant_to_price, -> { where relevant_to_price: true }
  
   include FloatHelper
  
@@ -45,7 +45,7 @@ class InviteFriend &lt; Invitation
   private
   def check_for_invitation_existence
     if friend
-      friend.tasks.pending.of("InviteFriend").find(:all, :conditions => {:requestor_id => person.id, :target_id => friend.id}).blank?
+      friend.tasks.pending.of("InviteFriend").where(requestor_id: person.id, target_id: friend.id).blank?
     end
   end
  
@@ -65,7 +65,7 @@ class InviteMember &lt; Invitation
   private
   def check_for_invitation_existence
     if friend
-      friend.tasks.pending.of("InviteMember").find(:all, :conditions => {:requestor_id => person.id}).select { |t| t.data[:community_id] == community_id }.blank?
+      friend.tasks.pending.of("InviteMember").where(requestor_id: person.id).select{ |t| t.data[:community_id] == community_id }.blank?
     end
   end
  
@@ -41,7 +41,7 @@ class LinkListBlock &lt; Block
     [N_('New window'), '_new'],
   ]
  
-  settings_items :links, Array, :default => []
+  settings_items :links, type: Array, :default => []
  
   before_save do |block|
     block.links = block.links.delete_if {|i| i[:name].blank? and i[:address].blank?}
@@ -84,7 +84,7 @@ class Organization &lt; Profile
   end
  
   def find_pending_validation(code)
-    validations.pending.find(:first, :conditions => {:code => code})
+    validations.pending.where(code: code).first
   end
  
   def processed_validations
@@ -92,7 +92,7 @@ class Organization &lt; Profile
   end
  
   def find_processed_validation(code)
-    validations.finished.find(:first, :conditions => {:code => code})
+    validations.finished.where(code: code).first
   end
  
   def is_validation_entity?
@@ -140,7 +140,7 @@ class Organization &lt; Profile
  
   settings_items :zip_code, :city, :state, :country
  
-  validates_format_of :foundation_year, :with => Noosfero::Constants::INTEGER_FORMAT
+  validates_numericality_of :foundation_year, only_integer: true, if: -> o { o.foundation_year.present? }
   validates_format_of :contact_email, :with => Noosfero::Constants::EMAIL_FORMAT, :if => (lambda { |org| !org.contact_email.blank? })
   validates_as_cnpj :cnpj
  
@@ -190,7 +190,7 @@ class Organization &lt; Profile
   end
  
   def already_request_membership?(person)
-    self.tasks.pending.find_by_requestor_id(person.id, :conditions => { :type => 'AddMember' })
+    self.tasks.pending.where(type: 'AddMember', requestor_id: person.id).first
   end
  
   def jid(options = {})
@@ -16,27 +16,26 @@ class Person &lt; Profile
   acts_as_trackable :after_add => Proc.new {|p,t| notify_activity(t)}
   acts_as_accessor
  
-  scope :members_of, lambda { |resources|
-    resources = [resources] if !resources.kind_of?(Array)
+  scope :members_of, -> resources {
+    resources = Array(resources)
     conditions = resources.map {|resource| "role_assignments.resource_type = '#{resource.class.base_class.name}' AND role_assignments.resource_id = #{resource.id || -1}"}.join(' OR ')
-    { :select => 'DISTINCT profiles.*', :joins => :role_assignments, :conditions => [conditions] }
+    select('DISTINCT profiles.*').joins(:role_assignments).where([conditions])
   }
  
-  scope :not_members_of, lambda { |resources|
-    resources = [resources] if !resources.kind_of?(Array)
+  scope :not_members_of, -> resources {
+    resources = Array(resources)
     conditions = resources.map {|resource| "role_assignments.resource_type = '#{resource.class.base_class.name}' AND role_assignments.resource_id = #{resource.id || -1}"}.join(' OR ')
-    { :select => 'DISTINCT profiles.*', :conditions => ['"profiles"."id" NOT IN (SELECT DISTINCT profiles.id FROM "profiles" INNER JOIN "role_assignments" ON "role_assignments"."accessor_id" = "profiles"."id" AND "role_assignments"."accessor_type" = (\'Profile\') WHERE "profiles"."type" IN (\'Person\') AND (%s))' % conditions] }
+    select('DISTINCT profiles.*').where('"profiles"."id" NOT IN (SELECT DISTINCT profiles.id FROM "profiles" INNER JOIN "role_assignments" ON "role_assignments"."accessor_id" = "profiles"."id" AND "role_assignments"."accessor_type" = (\'Profile\') WHERE "profiles"."type" IN (\'Person\') AND (%s))' % conditions)
   }
  
-  scope :by_role, lambda { |roles|
-    roles = [roles] unless roles.kind_of?(Array)
-    { :select => 'DISTINCT profiles.*', :joins => :role_assignments, :conditions => ['role_assignments.role_id IN (?)',
-roles] }
+  scope :by_role, -> roles {
+    roles = Array(roles)
+    select('DISTINCT profiles.*').joins(:role_assignments).where('role_assignments.role_id IN (?)', roles)
   }
  
-  scope :not_friends_of, lambda { |resources|
+  scope :not_friends_of, -> resources {
     resources = Array(resources)
-    { :select => 'DISTINCT profiles.*', :conditions => ['"profiles"."id" NOT IN (SELECT DISTINCT profiles.id FROM "profiles" INNER JOIN "friendships" ON "friendships"."person_id" = "profiles"."id" WHERE "friendships"."friend_id" IN (%s))' % resources.map(&:id)] }
+    select('DISTINCT profiles.*').where('"profiles"."id" NOT IN (SELECT DISTINCT profiles.id FROM "profiles" INNER JOIN "friendships" ON "friendships"."person_id" = "profiles"."id" WHERE "friendships"."friend_id" IN (%s))' % resources.map(&:id))
   }
  
   scope :visible_for_person, lambda { |person|
@@ -68,6 +67,9 @@ roles] }
   end
   alias_method_chain :has_permission?, :plugins
  
+  # for eager loading
+  has_many :memberships, through: :role_assignments, source: :resource, source_type: 'Profile'
+
   def memberships
     scopes = []
     plugins_scopes = plugins.dispatch_scopes(:person_memberships, self)
@@ -84,7 +86,9 @@ roles] }
   has_many :friendships, :dependent => :destroy
   has_many :friends, :class_name => 'Person', :through => :friendships
  
-  scope :online, lambda { { :include => :user, :conditions => ["users.chat_status != '' AND users.chat_status_at >= ?", DateTime.now - User.expires_chat_status_every.minutes] } }
+  scope :online, -> {
+    joins(:user).where("users.chat_status != '' AND users.chat_status_at >= ?", DateTime.now - User.expires_chat_status_every.minutes)
+  }
  
   has_many :requested_tasks, :class_name => 'Task', :foreign_key => :requestor_id, :dependent => :destroy
  
@@ -100,21 +104,31 @@ roles] }
   has_and_belongs_to_many :acepted_forums, :class_name => 'Forum', :join_table => 'terms_forum_people'
   has_and_belongs_to_many :articles_with_access, :class_name => 'Article', :join_table => 'article_privacy_exceptions'
  
-  has_many :suggested_profiles, :class_name => 'ProfileSuggestion', :foreign_key => :person_id, :order => 'score DESC', :dependent => :destroy
-  has_many :suggested_people, :through => :suggested_profiles, :source => :suggestion, :conditions => ['profile_suggestions.suggestion_type = ? AND profile_suggestions.enabled = ?', 'Person', true]
-  has_many :suggested_communities, :through => :suggested_profiles, :source => :suggestion, :conditions => ['profile_suggestions.suggestion_type = ? AND profile_suggestions.enabled = ?', 'Community', true]
+  has_many :suggested_profiles, class_name: 'ProfileSuggestion', foreign_key: :person_id, order: 'score DESC', dependent: :destroy
+  has_many :suggested_people, -> {
+    where 'profile_suggestions.suggestion_type = ? AND profile_suggestions.enabled = ?', 'Person', true
+  }, through: :suggested_profiles, source: :suggestion
+  has_many :suggested_communities, -> {
+    where 'profile_suggestions.suggestion_type = ? AND profile_suggestions.enabled = ?', 'Community', true
+  }, through: :suggested_profiles, source: :suggestion
  
-  scope :more_popular, :order => 'friends_count DESC'
+  scope :more_popular, -> { order 'friends_count DESC' }
  
-  scope :abusers, :joins => :abuse_complaints, :conditions => ['tasks.status = 3'], :select => 'DISTINCT profiles.*'
-  scope :non_abusers, :joins => "LEFT JOIN tasks ON profiles.id = tasks.requestor_id AND tasks.type='AbuseComplaint'", :conditions => ["tasks.status != 3 OR tasks.id is NULL"], :select => "DISTINCT profiles.*"
+  scope :abusers, -> {
+    joins(:abuse_complaints).where('tasks.status = 3').select('DISTINCT profiles.*')
+  }
+  scope :non_abusers, -> {
+    select("DISTINCT profiles.*").
+    joins("LEFT JOIN tasks ON profiles.id = tasks.requestor_id AND tasks.type='AbuseComplaint'").
+    where("tasks.status != 3 OR tasks.id is NULL")
+  }
  
-  scope :admins, :joins => [:role_assignments => :role], :conditions => ['roles.key = ?', 'environment_administrator' ]
-  scope :activated, :joins => :user, :conditions => ['users.activation_code IS NULL AND users.activated_at IS NOT NULL']
-  scope :deactivated, :joins => :user, :conditions => ['NOT (users.activation_code IS NULL AND users.activated_at IS NOT NULL)']
+  scope :admins, -> { joins(:role_assignments => :role).where('roles.key = ?', 'environment_administrator') }
+  scope :activated, -> { joins(:user).where('users.activation_code IS NULL AND users.activated_at IS NOT NULL') }
+  scope :deactivated, -> { joins(:user).where('NOT (users.activation_code IS NULL AND users.activated_at IS NOT NULL)') }
  
   after_destroy do |person|
-    Friendship.find(:all, :conditions => { :friend_id => person.id}).each { |friendship| friendship.destroy }
+    Friendship.where(friend_id: person.id).each{ |friendship| friendship.destroy }
   end
  
   belongs_to :user, :dependent => :delete
@@ -177,7 +191,7 @@ roles] }
   end
  
   def remove_friend(friend)
-    Friendship.find(:first, :conditions => {:friend_id => friend, :person_id => id}).destroy
+    Friendship.where(friend_id: friend, person_id: id).first.destroy
   end
  
   FIELDS = %w[
@@ -288,7 +302,7 @@ roles] }
   end
  
   validates_each :email, :on => :update do |record,attr,value|
-    if User.find(:first, :conditions => ['email = ? and id != ? and environment_id = ?', value, record.user.id, record.environment.id])
+    if User.where('email = ? and id != ? and environment_id = ?', value, record.user.id, record.environment.id).first
       record.errors.add(attr, _('{fn} is already used by other user').fix_i18n)
     end
   end
@@ -397,7 +411,7 @@ roles] }
   def ask_to_join?(community)
     return false if !community.visible?
     return false if memberships.include?(community)
-    return false if AddMember.find(:first, :conditions => {:requestor_id => self.id, :target_id => community.id})
+    return false if AddMember.where(requestor_id: self.id, target_id: community.id).first
     !refused_communities.include?(community)
   end
  
+# FIXME needed by test/units/application_helper.rb
+require_dependency 'application_helper'
+
 class PersonNotifier
  
   def initialize(person)
@@ -28,7 +31,7 @@ class PersonNotifier
  
   def notify
     if @person.notification_time && @person.notification_time > 0
-      notifications = @person.tracked_notifications.find(:all, :conditions => ["created_at > ?", notify_from])
+      notifications = @person.tracked_notifications.where("created_at > ?", notify_from)
       tasks = Task.to(@person).without_spam.pending.where("created_at > ?", notify_from).order_by('created_at', 'asc')
  
       Noosfero.with_locale @person.environment.default_language do
@@ -74,9 +77,8 @@ class PersonNotifier
  
   end
  
-  class Mailer < ActionMailer::Base
+  class Mailer < ApplicationMailer
  
-    helper ApplicationHelper
     helper ActionTrackerHelper
  
     def session
@@ -85,8 +87,8 @@ class PersonNotifier
  
     def content_summary(person, notifications, tasks)
       if person.environment
-        ActionMailer::Base.asset_host = person.environment.top_url
-        ActionMailer::Base.default_url_options[:host] = person.environment.default_hostname
+        ApplicationMailer.asset_host = person.environment.top_url
+        ApplicationMailer.default_url_options[:host] = person.environment.default_hostname
       end
  
       @current_theme = 'default'
@@ -94,7 +96,7 @@ class PersonNotifier
       @recipient = @profile.nickname || @profile.name
       @notifications = notifications
       @tasks = tasks
-      @environment = @profile.environment.name
+      @environment = @profile.environment
       @url = @profile.environment.top_url
       mail(
         content_type: "text/html",
@@ -52,8 +52,8 @@ class Product &lt; ActiveRecord::Base
  
   scope :more_recent, :order => "created_at DESC"
  
-  scope :from_category, lambda { |category|
-    {:joins => :product_category, :conditions => ['categories.path LIKE ?', "%#{category.slug}%"]} if category
+  scope :from_category, -> category {
+    joins(:product_category).where('categories.path LIKE ?', "%#{category.slug}%") if category
   }
  
   scope :visible_for_person, lambda { |person|
@@ -6,19 +6,19 @@ class ProductCategory &lt; Category
   attr_accessible :name, :parent, :environment
  
   scope :unique, :select => 'DISTINCT ON (path) categories.*'
-  scope :by_enterprise, lambda { |enterprise| {
-    :joins => :products,
-    :conditions => ['products.profile_id = ?', enterprise.id]
-  }}
-  scope :by_environment, lambda { |environment| {
-    :conditions => ['environment_id = ?', environment.id]
-  }}
-  scope :unique_by_level, lambda { |level| {
-    :select => "DISTINCT ON (filtered_category) split_part(path, '/', #{level}) AS filtered_category, categories.*"
-  }}
+  scope :by_enterprise, -> enterprise {
+    joins(:products).
+    where('products.profile_id = ?', enterprise.id)
+  }
+  scope :by_environment, -> environment {
+    where 'environment_id = ?', environment.id
+  }
+  scope :unique_by_level, -> level {
+    select "DISTINCT ON (filtered_category) split_part(path, '/', #{level}) AS filtered_category, categories.*"
+  }
  
   def all_products
-    Product.find(:all, :conditions => { :product_category_id => (all_children << self).map(&:id) })
+    Product.where(product_category_id: (all_children << self).map(&:id))
   end
  
   def self.menu_categories(top_category, env)
@@ -39,7 +39,7 @@ class ProductsBlock &lt; Block
     link_to(_('View all products'), owner.public_profile_url.merge(:controller => 'catalog', :action => 'index'))
   end
  
-  settings_items :product_ids, Array
+  settings_items :product_ids, type: Array
   def product_ids=(array)
     self.settings[:product_ids] = array
     if self.settings[:product_ids]
@@ -50,7 +50,7 @@ class Profile &lt; ActiveRecord::Base
       all_roles(env_id).select{ |r| r.key.match(/^profile_/) unless r.key.blank? || !r.profile_id.nil?}
     end
     def self.all_roles(env_id)
-      Role.all :conditions => { :environment_id => env_id }
+      Role.where(environment_id: env_id)
     end
     def self.method_missing(m, *args, &block)
       role = find_role(m, args[0])
@@ -87,20 +87,28 @@ class Profile &lt; ActiveRecord::Base
  
   include Noosfero::Plugin::HotSpot
  
-  scope :memberships_of, lambda { |person| { :select => 'DISTINCT profiles.*', :joins => :role_assignments, :conditions => ['role_assignments.accessor_type = ? AND role_assignments.accessor_id = ?', person.class.base_class.name, person.id ] } }
+  scope :memberships_of, -> person {
+    select('DISTINCT profiles.*').
+    joins(:role_assignments).
+    where('role_assignments.accessor_type = ? AND role_assignments.accessor_id = ?', person.class.base_class.name, person.id)
+  }
   #FIXME: these will work only if the subclass is already loaded
-  scope :enterprises, lambda { {:conditions => (Enterprise.send(:subclasses).map(&:name) << 'Enterprise').map { |klass| "profiles.type = '#{klass}'"}.join(" OR ")} }
-  scope :communities, lambda { {:conditions => (Community.send(:subclasses).map(&:name) << 'Community').map { |klass| "profiles.type = '#{klass}'"}.join(" OR ")} }
-  scope :templates, lambda { |template_id = nil|
-    conditions = {:conditions => {:is_template => true}}
-    conditions[:conditions].merge!({:id => template_id}) unless template_id.nil?
-    conditions
+  scope :enterprises, -> {
+    where((Enterprise.send(:subclasses).map(&:name) << 'Enterprise').map { |klass| "profiles.type = '#{klass}'"}.join(" OR "))
+  }
+  scope :communities, -> {
+    where((Community.send(:subclasses).map(&:name) << 'Community').map { |klass| "profiles.type = '#{klass}'"}.join(" OR "))
+  }
+  scope :templates, -> template_id = nil {
+    s = where is_template: true
+    s = s.where id: template_id if template_id
+    s
   }
  
-  scope :with_templates, lambda { |templates|
-    {:conditions => {:template_id => templates}}
+  scope :with_templates, -> templates {
+    where template_id: templates
   }
-  scope :no_templates, {:conditions => {:is_template => false}}
+  scope :no_templates, -> { where is_template: false }
  
   # Returns a scoped object to select profiles in a given location or in a radius
   # distance from the given location center.
@@ -177,10 +185,10 @@ class Profile &lt; ActiveRecord::Base
     Profile.column_names.map{|n| [Profile.table_name, n].join('.')}.join(',')
   end
  
-  scope :visible, :conditions => { :visible => true, :secret => false }
-  scope :disabled, :conditions => { :visible => false }
-  scope :public, :conditions => { :visible => true, :public_profile => true, :secret => false }
-  scope :enabled, :conditions => { :enabled => true }
+  scope :visible, -> { where visible: true, secret: false }
+  scope :disabled, -> { where visible: false }
+  scope :is_public, -> { where visible: true, public_profile: true, secret: false }
+  scope :enabled, -> { where enabled: true }
  
   # Subclasses must override this method
   scope :more_popular
@@ -229,7 +237,7 @@ class Profile &lt; ActiveRecord::Base
   validates_length_of :description, :maximum => 550, :allow_nil => true
  
   # Valid identifiers must match this format.
-  IDENTIFIER_FORMAT = /^#{Noosfero.identifier_format}$/
+  IDENTIFIER_FORMAT = /\A#{Noosfero.identifier_format}\Z/
  
   # These names cannot be used as identifiers for Profiles
   RESERVED_IDENTIFIERS = %w[
@@ -281,7 +289,7 @@ class Profile &lt; ActiveRecord::Base
     end
   end
  
-  has_many :profile_categorizations, :conditions => [ 'categories_profiles.virtual = ?', false ]
+  has_many :profile_categorizations, -> { where 'categories_profiles.virtual = ?', false }
   has_many :categories, :through => :profile_categorizations
  
   has_many :profile_categorizations_including_virtual, :class_name => 'ProfileCategorization'
@@ -508,14 +516,13 @@ class Profile &lt; ActiveRecord::Base
     self.articles.recent(limit, options, pagination)
   end
  
-  def last_articles(limit = 10, options = {})
-    options = { :limit => limit,
-                :conditions => ["advertise = ? AND published = ? AND
-                                 ((articles.type != ? and articles.type != ? and articles.type != ?) OR
-                                 articles.type is NULL)",
-                                 true, true, 'UploadedFile', 'RssFeed', 'Blog'],
-                :order => 'articles.published_at desc, articles.id desc' }.merge(options)
-    self.articles.find(:all, options)
+  def last_articles limit = 10
+    self.articles.limit(limit).where(
+      "advertise = ? AND published = ? AND
+      ((articles.type != ? and articles.type != ? and articles.type != ?) OR
+      articles.type is NULL)",
+      true, true, 'UploadedFile', 'RssFeed', 'Blog'
+    ).order('articles.published_at desc, articles.id desc')
   end
  
   class << self
@@ -713,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
@@ -25,8 +25,8 @@ class ProfileActivity &lt; ActiveRecord::Base
   protected
  
   def copy_timestamps
-    self.created_at = self.activity.created_at
-    self.updated_at = self.activity.updated_at
+    self.created_at = self.activity.created_at if self.activity.created_at
+    self.updated_at = self.activity.updated_at if self.activity.updated_at
   end
  
 end
 class ProfileCategorization < ActiveRecord::Base
-  set_table_name :categories_profiles
+  self.table_name = :categories_profiles
   belongs_to :profile
   belongs_to :category
  
@@ -13,7 +13,7 @@ class ProfileCategorization &lt; ActiveRecord::Base
   end
  
   def self.remove_region(profile)
-    region = profile.categories.find(:first, :conditions => { :type => [Region, State, City].map(&:name) })
+    region = profile.categories.where(type: [Region, State, City].map(&:name)).first
     if region
       ids = region.hierarchy.map(&:id)
       self.delete_all(:profile_id => profile.id, :category_id => ids)
@@ -16,7 +16,7 @@ class ProfileListBlock &lt; Block
  
   def profile_list
     result = nil
-    public_profiles = profiles.public.includes([:image,:domains,:preferred_domain,:environment])
+    public_profiles = profiles.is_public.includes([:image,:domains,:preferred_domain,:environment])
     if !prioritize_profiles_with_image
 result = public_profiles.all(:limit => get_limit, :order => 'profiles.updated_at DESC').sort_by{ rand }
     elsif profiles.visible.with_image.count >= get_limit
@@ -28,7 +28,7 @@ result = public_profiles.all(:limit =&gt; get_limit, :order =&gt; &#39;profiles.updated_at
   end
  
   def profile_count
-    profiles.public.length
+    profiles.is_public.length
   end
  
   # the title of the block. Probably will be overriden in subclasses.
@@ -26,9 +26,9 @@ class ProfileSuggestion &lt; ActiveRecord::Base
   end
  
   validates_uniqueness_of :suggestion_id, :scope => [ :person_id ]
-  scope :of_person, :conditions => { :suggestion_type => 'Person' }
-  scope :of_community, :conditions => { :suggestion_type => 'Community' }
-  scope :enabled, :conditions => { :enabled => true }
+  scope :of_person, -> { where suggestion_type: 'Person' }
+  scope :of_community, -> { where suggestion_type: 'Community' }
+  scope :enabled, -> { where enabled: true }
  
   # {:category_type => ['category-icon', 'category-label']}
   CATEGORIES = {
@@ -55,13 +55,13 @@ class ProfileSuggestion &lt; ActiveRecord::Base
       :threshold => 2, :weight => 1, :connection => 'Profile'
     },
     :people_with_common_tags => {
-      :threshold => 2, :weight => 1, :connection => 'ActsAsTaggableOn::Tag'
+      :threshold => 2, :weight => 1, :connection => 'Tag'
     },
     :communities_with_common_friends => {
       :threshold => 2, :weight => 1, :connection => 'Profile'
     },
     :communities_with_common_tags => {
-      :threshold => 2, :weight => 1, :connection => 'ActsAsTaggableOn::Tag'
+      :threshold => 2, :weight => 1, :connection => 'Tag'
     }
   }
  
@@ -127,17 +127,15 @@ class ProfileSuggestion &lt; ActiveRecord::Base
         rescue NoMethodError
           next
         end
-        connections = suggested_profile.send("#{rule}_connections")
-        if connections.present?
-          connections = connections[1..-2].split(',')
-        else
-          connections = []
-        end
-        suggestion.send("#{rule}=", value)
+
+        connections = suggested_profile.send("#{rule}_connections") || []
+        connections = connections[1..-2] if connections.present?
         connections.each do |connection_id|
           next if SuggestionConnection.where(:suggestion_id => suggestion.id, :connection_id => connection_id, :connection_type => options[:connection]).present?
-           SuggestionConnection.create!(:suggestion => suggestion, :connection_id => connection_id, :connection_type => options[:connection])
+          SuggestionConnection.create!(:suggestion => suggestion, :connection_id => connection_id, :connection_type => options[:connection])
         end
+
+        suggestion.send("#{rule}=", value)
         suggestion.score += value * options[:weight]
       end
       suggestion.save!
@@ -5,6 +5,7 @@ class ReportedImage &lt; ActiveRecord::Base
  
   has_attachment :content_type => :image,
     :storage     => :file_system,
-    :max_size => 5.megabytes
+    :max_size => 5.megabytes,
+    processor: 'Rmagick'
  
 end
@@ -40,7 +40,7 @@ class RssFeed &lt; Article
  
   # FIXME this should be validates_numericality_of, but Rails 2.0.2 does not
   # support validates_numericality_of with virtual attributes
-  validates_format_of :limit, :with => /^\d+$/, :if => :limit
+  validates_format_of :limit, :with => /\d+/, :if => :limit
  
   # determinates what to include in the feed. Possible values are +:all+
   # (include everything from the profile) and :parent_and_children (include
@@ -69,7 +69,7 @@ class RssFeed &lt; Article
   def fetch_articles
     if parent && parent.has_posts?
       language = self.language.blank? ? {} : { :language => self.language }
-      return parent.posts.find(:all, :conditions => { :published => true }.merge(language), :limit => self.limit, :order => 'id desc')
+      return parent.posts.where({published: true}.merge language).limit(self.limit).order('id desc')
     end
  
     articles =
@@ -18,9 +18,9 @@ class Scrap &lt; ActiveRecord::Base
   after_create :create_activity
   after_update :update_activity
  
-  scope :all_scraps, lambda {|profile| {:conditions => ["receiver_id = ? OR sender_id = ?", profile, profile], :limit => 30}}
+  scope :all_scraps, -> profile { limit(30).where("receiver_id = ? OR sender_id = ?", profile, profile) }
  
-  scope :not_replies, :conditions => {:scrap_id => nil}
+  scope :not_replies, -> { where scrap_id: nil }
  
   track_actions :leave_scrap, :after_create, :keep_params => ['sender.name', 'content', 'receiver.name', 'receiver.url'], :if => Proc.new{|s| s.sender != s.receiver && s.sender != s.top_root.receiver}, :custom_target => :action_tracker_target, :custom_user => :sender
  
 class SearchTermOccurrence < ActiveRecord::Base
+
   belongs_to :search_term
   validates_presence_of :search_term
   attr_accessible :search_term, :created_at, :total, :indexed
  
   EXPIRATION_TIME = 1.year
  
-  scope :valid, :conditions => ["search_term_occurrences.created_at > ?", DateTime.now - EXPIRATION_TIME]
+  scope :valid, -> { where "search_term_occurrences.created_at > ?", DateTime.now - EXPIRATION_TIME }
+
 end
@@ -13,7 +13,7 @@ class SlideshowBlock &lt; Block
   end
  
   def gallery
-    gallery_id ? Gallery.find(:first, :conditions => { :id => gallery_id }) : nil
+    if gallery_id then Gallery.where(id: gallery_id).first else nil end
   end
  
   def public_filename_for(image)
 class SuggestionConnection < ActiveRecord::Base
-  attr_accessible :suggestion, :connection_type, :connection_id
+  attr_accessible :suggestion, :suggestion_id, :connection_type, :connection_id
  
   belongs_to :suggestion, :class_name => 'ProfileSuggestion', :foreign_key => 'suggestion_id'
   belongs_to :connection, :polymorphic => true
@@ -0,0 +1,34 @@
+Tag = ActsAsTaggableOn::Tag
+class Tag
+
+  attr_accessible :name, :parent_id, :pending
+
+  has_many :children, class_name: 'Tag', foreign_key: 'parent_id', dependent: :destroy
+
+  @@original_find = self.method(:find)
+  # Rename the find method to find_with_pendings that includes all tags in the search regardless if its pending or not
+  def self.find_with_pendings(*args)
+    @@original_find.call(*args)
+  end
+
+  # Redefine the find method to exclude the pending tags from the search not allowing to tag something with an unapproved tag
+  def self.find(*args)
+    self.where(pending: false).find_with_pendings(*args)
+  end
+
+  # Return all the tags that were suggested but not yet approved
+  def self.find_pendings
+    self.where(pending: true)
+  end
+
+  # All the tags that can be a new parent for this tag, that is all but itself and its descendents to avoid loops
+  def parent_candidates
+    ActsAsTaggableOn::Tag.all - descendents - [self]
+  end
+
+  # All tags that have this tag as its one of its ancestors
+  def descendents
+    children.to_a.sum([], &:descendents) + children
+  end
+
+end
@@ -50,7 +50,7 @@ class Task &lt; ActiveRecord::Base
   before_validation(:on => :create) do |task|
     if task.code.nil?
       task.code = Task.generate_code(task.code_length)
-      while (Task.find_by_code(task.code))
+      while Task.from_code(task.code).first
         task.code = Task.generate_code(task.code_length)
       end
     end
@@ -259,27 +259,28 @@ class Task &lt; ActiveRecord::Base
     end
   end
  
-  scope :pending, :conditions => { :status =>  Task::Status::ACTIVE }
-  scope :hidden, :conditions => { :status =>  Task::Status::HIDDEN }
-  scope :finished, :conditions => { :status =>  Task::Status::FINISHED }
-  scope :canceled, :conditions => { :status =>  Task::Status::CANCELLED }
-  scope :closed, :conditions => { :status =>  [Task::Status::CANCELLED, Task::Status::FINISHED] }
-  scope :opened, :conditions => { :status =>  [Task::Status::ACTIVE, Task::Status::HIDDEN] }
-  scope :of, lambda { |type| conditions = type ? "type LIKE '#{type}'" : "1=1"; {:conditions =>  [conditions]} }
-  scope :order_by, lambda { |attribute, ord| {:order => "#{attribute} #{ord}"} }
-  scope :like, lambda { |field, value| where("LOWER(#{field}) LIKE ?", "%#{value.downcase}%") if value}
-  scope :pending_all, lambda { |profile, filter_type, filter_text|
+  scope :pending, -> { where status: Task::Status::ACTIVE }
+  scope :hidden, -> { where status: Task::Status::HIDDEN }
+  scope :finished, -> { where status: Task::Status::FINISHED }
+  scope :canceled, -> { where status: Task::Status::CANCELLED }
+  scope :closed, -> { where status: [Task::Status::CANCELLED, Task::Status::FINISHED] }
+  scope :opened, -> { where status: [Task::Status::ACTIVE, Task::Status::HIDDEN] }
+  scope :of, -> type { where "type LIKE ?", type if type }
+  scope :order_by, -> attribute, ord { order "#{attribute} #{ord}" }
+  scope :like, -> field, value { where "LOWER(#{field}) LIKE ?", "%#{value.downcase}%" if value }
+  scope :pending_all, -> profile, filter_type, filter_text {
     self.to(profile).without_spam.pending.of(filter_type).like('data', filter_text)
   }
  
   scope :to, lambda { |profile|
     environment_condition = nil
     if profile.person?
-      envs_ids = Environment.find(:all).select{ |env| profile.is_admin?(env) }.map { |env| "target_id = #{env.id}"}.join(' OR ')
+      envs_ids = Environment.all.select{ |env| profile.is_admin?(env) }.map{ |env| "target_id = #{env.id}"}.join(' OR ')
       environment_condition = envs_ids.blank? ? nil : "(target_type = 'Environment' AND (#{envs_ids}))"
     end
     profile_condition = "(target_type = 'Profile' AND target_id = #{profile.id})"
-    { :conditions => [environment_condition, profile_condition].compact.join(' OR ') }
+
+    where [environment_condition, profile_condition].compact.join(' OR ')
   }
  
   def self.pending_types_for(profile)
@@ -337,6 +338,12 @@ class Task &lt; ActiveRecord::Base
     end
   end
  
+  # finds a task by its (generated) code. Only returns a task with the
+  # specified code AND with status = Task::Status::ACTIVE.
+  #
+  # Can be used in subclasses to find only their instances.
+  scope :from_code, -> code { where code: code, status: Task::Status::ACTIVE }
+
   class << self
  
     # generates a random code string consisting of length characters (or 36 by
@@ -350,14 +357,6 @@ class Task &lt; ActiveRecord::Base
       code
     end
  
-    # finds a task by its (generated) code. Only returns a task with the
-    # specified code AND with status = Task::Status::ACTIVE.
-    #
-    # Can be used in subclasses to find only their instances.
-    def find_by_code(code)
-      self.find(:first, :conditions => { :code => code, :status => Task::Status::ACTIVE })
-    end
-
     def per_page
       15
     end
 class Thumbnail < ActiveRecord::Base
+
+  attr_accessible :uploaded_data
+  # mass assigned by attachment_fu
+  attr_accessible :content_type, :filename, :thumbnail_resize_options, :thumbnail, :parent_id
+
   has_attachment :storage => :file_system,
-    :content_type => :image, :max_size => 5.megabytes
+    :content_type => :image, :max_size => 5.megabytes, processor: 'Rmagick'
   validates_as_attachment
  
   sanitize_filename
  
   postgresql_attachment_fu
  
-  attr_accessible :uploaded_data
 end
@@ -9,7 +9,7 @@ class TinyMceArticle &lt; TextArticle
   def self.description
     _('Not accessible for visually impaired users.')
   end
-  
+
   xss_terminate :only => [  ]
  
   xss_terminate :only => [ :name, :abstract, :body ], :with => 'white_list', :on => 'validation'
@@ -63,7 +63,8 @@ class UploadedFile &lt; Article
   has_attachment :storage => :file_system,
     :thumbnails => { :icon => [24,24], :bigicon => [50,50], :thumb => '130x130>', :slideshow => '320x240>', :display => '640X480>' },
     :thumbnail_class => Thumbnail,
-    :max_size => self.max_size
+    :max_size => self.max_size,
+    processor: 'Rmagick'
  
   validates_attachment :size => N_("{fn} of uploaded file was larger than the maximum size of %{size}").sub('%{size}', self.max_size.to_humanreadable).fix_i18n
  
@@ -102,7 +102,8 @@ class User &lt; ActiveRecord::Base
     end
   end
  
-  has_one :person, :dependent => :destroy
+  # set autosave to false as we do manually when needed and Person syncs with us
+  has_one :person, dependent: :destroy, autosave: false
   belongs_to :environment
  
   has_many :sessions, dependent: :destroy
@@ -356,12 +357,12 @@ class User &lt; ActiveRecord::Base
   end
  
   def name
-    name = (self[:name] || login)
+    name = (@name || login)
     person.nil? ? name : (person.name || name)
   end
  
   def name= name
-    self[:name] = name
+    @name = name
   end
  
   def enable_email!
@@ -15,7 +15,7 @@
   <div>
     <%= link_to _('Add subcategory'), :action => 'new', :parent_id => category %>
     <%= link_to _('Edit'), :action => 'edit', :id => category %>
-    <%= link_to _('Remove'), { :action => 'remove', :id => category, }, :method => 'post', :confirm => (category.children.empty? ? (_('Are you sure you want to remove "%s"?') % category.name) : (_('Are you sure you want to remove "%s" and all its subcategories?') % category.name) ) %>
+    <%= link_to _('Remove'), { action: 'remove', id: category, }, method: 'post', data: {confirm: if category.children.empty? then _('Are you sure you want to remove "%s"?') % category.name else _('Are you sure you want to remove "%s" and all its subcategories?') % category.name end} %>
   </div>
 </div>
  
@@ -9,7 +9,7 @@
  
   <%= hidden_field_tag('success_back_to', @success_back_to) %>
  
-  <%= render :partial => partial_for_class(@article.class), :locals => { :f => f } %>
+  <%= render_partial_for_class @article.class, f: f %>
  
   <% if environment.is_portal_community?(profile) %>
     <div>
@@ -58,7 +58,7 @@
  
     <% unless @article.new_record? %>
       <%= button :delete, _('Delete'), {:controller => :cms, :action => :destroy, :id => @article},
-        :method => :post, :confirm => delete_article_message(@article) %>
+        :method => :post, data: {confirm: delete_article_message(@article)} %>
     <% end %>
   <% end %>
 <% end %>
@@ -55,7 +55,7 @@ function check_captcha(button, confirm_action) {
 <div class="post_comment_box <%= ((defined? show_form) && show_form) ? 'opened' : 'closed' %>">
  
   <%= link_to(_('Post a comment'), '#', :class => 'display-comment-form') if display_link && @comment.reply_of_id.blank? %>
-<%= remote_form_for(:comment, @comment, :url => {:profile => profile.identifier, :controller => 'comment', :action => (edition_mode ? 'update' : 'create'), :id => (edition_mode ?  @comment.id : @page.id)}, :html => { :class => 'comment_form' } ) do |f| %>
+<%= remote_form_for(@comment, :url => {:profile => profile.identifier, :controller => 'comment', :action => (edition_mode ? 'update' : 'create'), :id => (edition_mode ?  @comment.id : @page.id)}, :html => { :class => 'comment_form' } ) do |f| %>
  
   <%= required_fields_message %>
  
@@ -18,5 +18,5 @@
 <%= _("Greetings,") %>
  
 --
-<%= _('%s team.') % @environment %>
+<%= _('%s team.') % @environment.name %>
 <%= url_for @url %>
@@ -15,5 +15,5 @@
 <%= _("Greetings,") %>
  
 --
-<%= _('%s team.') % @environment %>
+<%= _('%s team.') % @environment.name %>
 <%= url_for @url %>
...	...	@@ -8,8 +8,7 @@ notifications:
8	8
9	9	language: ruby
10	10	rvm:
11		-# for 2.2 support we need to upgrade the pg gem
12		- - 2.1.6
	11	+ - 2.2.3
13	12
14	13	sudo: false
15	14	addons:
...	...	@@ -26,19 +25,16 @@ addons:
26	25	- libsqlite3-dev
27	26	- libxslt1-dev
28	27
29		-before_install:
30		- - gem env
31		-
32	28	# workaround for https://github.com/travis-ci/travis-ci/issues/4536
33	29	before_install:
34		- - export GEM_HOME=$PWD/vendor/bundle/ruby/2.1.0
	30	+ - export GEM_HOME=$PWD/vendor/bundle/ruby/2.2.0
35	31	- gem install bundler
36	32	cache: bundler
37	33
38	34	before_script:
39	35	- mkdir -p tmp/{pids,cache} log
40	36	- script/noosfero-plugins disableall
41		- - bundle exec rake makemo &>/dev/null
	37	+ #- bundle exec rake makemo &>/dev/null
42	38	# database
43	39	- cp config/database.yml.travis config/database.yml
44	40	- psql -c 'create database myapp_test;' -U postgres
...	...	@@ -49,7 +45,7 @@ env:
49	45	- TASK=test:units
50	46	- TASK=test:functionals
51	47	- TASK=test:integration
52		- - TASK=cucumber
	48	+ - TASK=cucumber LANG=en
53	49	- TASK=selenium
54	50	- TASK=test:noosfero_plugins BUNDLE_OPTS=install
55	51
...	...
1	1	source "https://rubygems.org"
2		-gem 'rails', '~> 3.2.22'
3		-gem 'minitest', '~> 3.2.0'
4		-gem 'fast_gettext', '~> 0.6.8'
5		-gem 'acts-as-taggable-on', '~> 3.4.2'
	2	+gem 'rails', '~> 4.2.4'
	3	+gem 'fast_gettext', '~> 0.9'
	4	+gem 'acts-as-taggable-on', '~> 3.5'
6	5	gem 'rails_autolink', '~> 1.1.5'
7		-gem 'pg', '~> 0.13.2'
8		-gem 'rmagick', '~> 2.13.1'
9		-gem 'RedCloth', '~> 4.2.9'
10		-gem 'will_paginate', '~> 3.0.3'
	6	+gem 'pg', '~> 0.17'
	7	+gem 'rmagick', '~> 2.13'
	8	+gem 'RedCloth', '~> 4.2'
11	9	gem 'ruby-feedparser', '~> 0.7'
12		-gem 'daemons', '~> 1.1.5'
	10	+gem 'daemons', '~> 1.1'
13	11	gem 'unicorn', '~> 4.8'
14		-gem 'nokogiri', '~> 1.5.5'
	12	+gem 'nokogiri', '~> 1.6.0'
	13	+gem 'will_paginate'
	14	+gem 'pothoven-attachment_fu', '~> 3.2.16'
	15	+gem 'delayed_job'
	16	+gem 'delayed_job_active_record'
15	17	gem 'rake', :require => false
16		-gem 'rest-client', '~> 1.6.7'
	18	+gem 'rest-client', '~> 1.6'
17	19	gem 'exception_notification', '~> 4.0.1'
18		-gem 'gettext', '~> 2.2.1', :require => false
19		-gem 'locale', '~> 2.0.5'
	20	+gem 'gettext', '~> 3.1', :require => false
	21	+gem 'locale', '~> 2.1'
20	22	gem 'whenever', :require => false
21		-gem 'eita-jrails', '~> 0.9.5', require: 'jrails'
	23	+gem 'eita-jrails', '~> 0.10.0', require: 'jrails'
	24	+gem 'diffy', '~> 3.0'
22	25
23	26	# API dependencies
24	27	gem 'grape', '~> 0.12'
...	...	@@ -30,30 +33,55 @@ gem 'rack-contrib'
30	33	# asset pipeline
31	34	gem 'uglifier', '>= 1.0.3'
32	35	gem 'sass-rails'
33		-gem 'sass', '~> 3.1.19'
	36	+
	37	+# gems to enable rails3 behaviour
	38	+gem 'protected_attributes'
	39	+gem 'rails-observers'
	40	+gem 'actionpack-page_caching'
	41	+gem 'actionpack-action_caching'
	42	+gem 'activerecord-session_store'
	43	+gem 'activerecord-deprecated_finders', require: 'active_record/deprecated_finders'
34	44
35	45	group :production do
36	46	gem 'dalli', '~> 2.7.0'
37	47	end
38	48
	49	+group :development do
	50	+end
	51	+
	52	+group :development, :test do
	53	+ gem 'spring'
	54	+end
	55	+
39	56	group :test do
40		- gem 'rspec', '~> 2.14.0'
41		- gem 'rspec-rails', '~> 2.14.1'
	57	+ gem 'rspec', '~> 3.3'
	58	+ gem 'rspec-rails', '~> 3.2'
42	59	gem 'mocha', '~> 1.1.0', :require => false
43	60	gem 'test-unit' if RUBY_VERSION >= '2.2.0'
	61	+ gem 'minitest'
	62	+ gem 'minitest-reporters'
44	63	end
45	64
46	65	group :cucumber do
47		- gem 'cucumber-rails', '~> 1.0.6', :require => false
48		- gem 'capybara', '~> 2.1.0'
49		- gem 'cucumber', '~> 1.0.6'
50		- gem 'database_cleaner', '~> 1.2.0'
51		- gem 'selenium-webdriver', '~> 2.47.0'
	66	+ gem 'capybara', '~> 2.2'
	67	+ gem 'launchy'
	68	+ gem 'cucumber'
	69	+ gem 'cucumber-rails', '~> 1.4.2', :require => false
	70	+ gem 'database_cleaner', '~> 1.3'
	71	+ gem 'selenium-webdriver'
52	72	end
53	73
54	74	# Requires custom dependencies
55	75	eval(File.read('config/Gemfile'), binding) rescue nil
56	76
	77	+vendor = Dir.glob('vendor/{,plugins/}*') - ['vendor/plugins']
	78	+vendor.each do \|dir\|
	79	+ plugin = File.basename dir
	80	+ version = if Dir.glob("#{dir}/*.gemspec").length > 0 then '> 0.0.0' else '0.0.0' end
	81	+
	82	+ gem plugin, version, path: dir
	83	+end
	84	+
57	85	# include gemfiles from enabled plugins
58	86	# plugins in baseplugins/ are not included on purpose. They should not have any
59	87	# dependencies.
...	...
...	...	@@ -3,7 +3,7 @@
3	3
4	4	VAGRANTFILE_API_VERSION = "2"
5	5	Vagrant.configure(VAGRANTFILE_API_VERSION) do \|config\|
6		- config.vm.box = ENV.fetch('VAGRANT_BOX', "debian-wheezy")
	6	+ config.vm.box = ENV.fetch('VAGRANT_BOX', "debian/jessie64")
7	7	config.vm.network :forwarded_port, host: 3000, guest: 3000
8	8	config.vm.provision :shell do \|shell\|
9	9	shell.inline = 'su vagrant -c /vagrant/script/vagrant'
...	...
...	...	@@ -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
...	...	@@ -54,7 +54,7 @@ class AdminPanelController < AdminController
54	54
55	55	if request.post?
56	56	env = environment
57		- folders = params[:folders].map{\|fid\| Folder.find(:first, :conditions => {:profile_id => env.portal_community, :id => fid})} if params[:folders]
	57	+ folders = env.portal_community.folders.where(id: params[:folders]).order(params[:folders].reverse.map{ \|f\| "id=#{f}" }) if params[:folders]
58	58	env.portal_folders = folders
59	59	if env.save
60	60	session[:notice] = _('Saved the portal folders')
...	...	@@ -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
...	...
1	1	class CategoriesController < AdminController
2	2
3	3	protect 'manage_environment_categories', :environment
4		-
	4	+
5	5	helper :categories
6	6
7	7	def index
8		- @categories = environment.categories.find(:all, :conditions => "parent_id is null AND type is null")
9		- @regions = environment.regions.find(:all, :conditions => {:parent_id => nil})
10		- @product_categories = environment.product_categories.find(:all, :conditions => {:parent_id => nil})
	8	+ @categories = environment.categories.where("parent_id is null AND type is null")
	9	+ @regions = environment.regions.where(:parent_id => nil)
	10	+ @product_categories = environment.product_categories.where(:parent_id => nil)
11	11	end
12	12
13	13	def get_children
...	...	@@ -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'
...	...
...	...	@@ -2,7 +2,7 @@ class EnvironmentRoleManagerController < AdminController
2	2	protect 'manage_environment_roles', :environment
3	3
4	4	def index
5		- @admins = Person.find(:all, :conditions => ['role_assignments.resource_type = ?', 'Environment'], :include => :role_assignments )
	5	+ @admins = Person.where('role_assignments.resource_type = ?', 'Environment').includes(:role_assignments)
6	6	end
7	7
8	8	def change_roles
...	...
1	1	class FeaturesController < AdminController
2	2	protect 'edit_environment_features', :environment
3		-
	3	+
4	4	def index
5	5	@features = Environment.available_features.sort_by{\|k,v\|v}
6	6	end
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
...	...	@@ -53,7 +53,7 @@ class FeaturesController < AdminController
53	53
54	54	def search_members
55	55	arg = params[:q].downcase
56		- result = environment.people.find(:all, :conditions => ['LOWER(name) LIKE ? OR identifier LIKE ?', "%#{arg}%", "%#{arg}%"])
	56	+ result = environment.people.where('LOWER(name) LIKE ? OR identifier LIKE ?', "%#{arg}%", "%#{arg}%")
57	57	render :text => prepare_to_token_input(result).to_json
58	58	end
59	59
...	...
...	...	@@ -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')
...	...