Merge pull request #992 from rud/chore/cleanups

Rubocop cleanups - it's time

Merge pull request #992 from rud/chore/cleanups
Rubocop cleanups - it's time
Stephen Crosby
2 parents 9d51326c 311c015d
Showing 160 changed files with 1388 additions and 1505 deletions Show diff stats
.rubocop.yml
.rubocop_todo.yml
Gemfile
Rakefile
app/controllers/api/v1/notices_controller.rb
app/controllers/api/v1/problems_controller.rb
app/controllers/api/v1/stats_controller.rb
app/controllers/api/v3/notices_controller.rb
app/controllers/application_controller.rb
app/controllers/apps_controller.rb
app/controllers/comments_controller.rb
app/controllers/deploys_controller.rb
app/controllers/notices_controller.rb
app/controllers/problems_controller.rb
app/controllers/problems_searcher.rb
app/controllers/site_config_controller.rb
app/controllers/users/omniauth_callbacks_controller.rb
app/controllers/users_controller.rb
app/controllers/watchers_controller.rb
app/decorators/app_decorator.rb
@@ -29,11 +29,20 @@ Style/AccessModifierIndentation:
 Style/AlignParameters:
   EnforcedStyle: with_fixed_indentation
  
+Style/DotPosition:
+  # Support code copy/paste into console
+  EnforcedStyle: trailing
+
 Style/IndentHash:
   EnforcedStyle: consistent
  
+Style/AlignHash:
+  EnforcedHashRocketStyle: table
+  EnforcedColonStyle: table
+
 Style/SpaceAroundOperators:
   MultiSpaceAllowedForOperators:
     - '='
     - '=>'
     - '||='
+    - 'then'
@@ -53,29 +53,12 @@ Rails/Validation:
     - 'app/models/problem.rb'
     - 'app/models/user.rb'
  
-# Offense count: 8
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedHashRocketStyle, EnforcedColonStyle, EnforcedLastArgumentHashStyle, SupportedLastArgumentHashStyles.
-Style/AlignHash:
-  Exclude:
-    - 'app/models/app.rb'
-    - 'spec/controllers/apps_controller_spec.rb'
-    - 'spec/controllers/comments_controller_spec.rb'
-    - 'spec/controllers/problems_controller_spec.rb'
-    - 'spec/models/problem_spec.rb'
-
 # Offense count: 105
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles, ProceduralMethods, FunctionalMethods, IgnoredMethods.
 Style/BlockDelimiters:
   Enabled: false
  
-# Offense count: 49
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-Style/BracesAroundHashParameters:
-  Enabled: false
-
 # Offense count: 15
 # Configuration parameters: EnforcedStyle, SupportedStyles.
 Style/ClassAndModuleChildren:
@@ -96,31 +79,11 @@ Style/ClassAndModuleChildren:
     - 'app/models/notification_services/webhook_service.rb'
     - 'config/initializers/overrides.rb'
  
-# Offense count: 23
-Style/ConstantName:
-  Exclude:
-    - 'app/models/notification_service.rb'
-    - 'app/models/notification_services/campfire_service.rb'
-    - 'app/models/notification_services/flowdock_service.rb'
-    - 'app/models/notification_services/gtalk_service.rb'
-    - 'app/models/notification_services/hipchat_service.rb'
-    - 'app/models/notification_services/hoiio_service.rb'
-    - 'app/models/notification_services/hubot_service.rb'
-    - 'app/models/notification_services/pushover_service.rb'
-    - 'app/models/notification_services/slack_service.rb'
-    - 'app/models/notification_services/webhook_service.rb'
-
 # Offense count: 70
 # Configuration parameters: Exclude.
 Style/Documentation:
   Enabled: false
  
-# Offense count: 9
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-Style/DotPosition:
-  Enabled: false
-
 # Offense count: 5
 Style/EachWithObject:
   Exclude:
@@ -161,12 +124,6 @@ Style/FormatString:
 Style/GuardClause:
   Enabled: false
  
-# Offense count: 1168
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles, UseHashRocketsWithSymbolValues.
-Style/HashSyntax:
-  Enabled: false
-
 # Offense count: 6
 # Cop supports --auto-correct.
 # Configuration parameters: MaxLineLength.
@@ -178,15 +135,6 @@ Style/IfUnlessModifier:
     - 'app/models/notice.rb'
     - 'app/models/user.rb'
  
-# Offense count: 6
-# Cop supports --auto-correct.
-Style/Lambda:
-  Exclude:
-    - 'app/models/app.rb'
-    - 'app/models/problem.rb'
-    - 'config/load.rb'
-    - 'spec/models/notice_observer_spec.rb'
-
 # Offense count: 17
 # Cop supports --auto-correct.
 Style/LeadingCommentSpace:
@@ -200,17 +148,6 @@ Style/LeadingCommentSpace:
     - 'spec/models/notification_service/hoiio_service_spec.rb'
     - 'spec/models/notification_service/pushover_service_spec.rb'
  
-# Offense count: 7
-# Cop supports --auto-correct.
-Style/MultilineBlockLayout:
-  Exclude:
-    - 'lib/tasks/errbit/demo.rake'
-    - 'spec/fabricators/issue_tracker_fabricator.rb'
-    - 'spec/interactors/user_destroy_spec.rb'
-    - 'spec/models/app_spec.rb'
-    - 'spec/models/error_report_spec.rb'
-    - 'spec/views/apps/show.atom.builder_spec.rb'
-
 # Offense count: 10
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles.
@@ -228,71 +165,11 @@ Style/PercentLiteralDelimiters:
     - 'spec/models/notice_spec.rb'
  
 # Offense count: 1
-# Cop supports --auto-correct.
-Style/PerlBackrefs:
-  Exclude:
-    - 'spec/acceptance/reset_password_token.rb'
-
-# Offense count: 3
-# Cop supports --auto-correct.
-Style/Proc:
-  Exclude:
-    - 'app/models/notice.rb'
-    - 'app/models/problem.rb'
-
-# Offense count: 2
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-Style/RaiseArgs:
-  Enabled: false
-
-# Offense count: 3
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles, AllowInnerSlashes.
-Style/RegexpLiteral:
-  Exclude:
-    - 'app/models/app.rb'
-    - 'config/initializers/ssl_enforcer.rb'
-    - 'config/load.rb'
-
-# Offense count: 2
-# Cop supports --auto-correct.
-# Configuration parameters: AllowAsExpressionSeparator.
-Style/Semicolon:
-  Exclude:
-    - 'app/decorators/issue_tracker_type_decorator.rb'
-    - 'app/models/notice_fingerprinter.rb'
-
-# Offense count: 6
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-Style/SignalException:
-  Exclude:
-    - 'app/controllers/notices_controller.rb'
-    - 'app/interactors/problem_merge.rb'
-    - 'app/models/problem.rb'
-    - 'lib/airbrake_api/v3/notice_parser.rb'
-    - 'lib/hoptoad.rb'
-
-# Offense count: 4
 # Configuration parameters: Methods.
 Style/SingleLineBlockParams:
   Exclude:
-    - 'app/helpers/application_helper.rb'
-    - 'app/helpers/form_helper.rb'
-    - 'app/models/notice_fingerprinter.rb'
     - 'lib/hoptoad/v2.rb'
  
-# Offense count: 19
-# Cop supports --auto-correct.
-# Configuration parameters: AllowIfMethodIsEmpty.
-Style/SingleLineMethods:
-  Exclude:
-    - 'app/models/notification_service.rb'
-    - 'spec/decorators/issue_tracker_decorator_spec.rb'
-    - 'spec/decorators/issue_tracker_type_decorator_spec.rb'
-    - 'spec/errbit_plugin/mock_issue_tracker.rb'
-    - 'spec/views/problems/show.html.haml_spec.rb'
-
 # Offense count: 11
 # Cop supports --auto-correct.
 Style/SingleSpaceBeforeFirstArg:
@@ -304,90 +181,8 @@ Style/SingleSpaceBeforeFirstArg:
     - 'spec/fabricators/notice_fabricator.rb'
     - 'spec/fabricators/user_fabricator.rb'
  
-# Offense count: 2
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-Style/SpaceAroundEqualsInParameterDefault:
-  Enabled: false
-
-# Offense count: 27
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-Style/SpaceBeforeBlockBraces:
-  Enabled: false
-
-# Offense count: 3
-# Cop supports --auto-correct.
-Style/SpaceBeforeComment:
-  Exclude:
-    - 'spec/controllers/api/v1/problems_controller_spec.rb'
-
-# Offense count: 75
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles, EnforcedStyleForEmptyBraces, SpaceBeforeBlockParameters.
-Style/SpaceInsideBlockBraces:
-  Enabled: false
-
-# Offense count: 26
-# Cop supports --auto-correct.
-Style/SpaceInsideBrackets:
-  Exclude:
-    - 'app/helpers/apps_helper.rb'
-    - 'app/models/issue.rb'
-    - 'app/models/notice_fingerprinter.rb'
-    - 'app/models/notification_services/gtalk_service.rb'
-    - 'config/initializers/devise.rb'
-    - 'lib/tasks/errbit/database.rake'
-    - 'spec/lib/configurator_spec.rb'
-    - 'spec/models/issue_spec.rb'
-
-# Offense count: 189
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, EnforcedStyleForEmptyBraces, SupportedStyles.
-Style/SpaceInsideHashLiteralBraces:
-  Enabled: false
-
 # Offense count: 873
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles.
 Style/StringLiterals:
   Enabled: false
-
-# Offense count: 3
-# Cop supports --auto-correct.
-# Configuration parameters: IgnoredMethods.
-Style/SymbolProc:
-  Exclude:
-    - 'app/controllers/users/omniauth_callbacks_controller.rb'
-    - 'app/models/deploy.rb'
-    - 'spec/models/deploy_spec.rb'
-
-# Offense count: 13
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyleForMultiline, SupportedStyles.
-Style/TrailingComma:
-  Exclude:
-    - 'app/controllers/deploys_controller.rb'
-    - 'app/helpers/problems_helper.rb'
-    - 'app/models/notification_services/hipchat_service.rb'
-    - 'app/models/problem.rb'
-    - 'config/deploy.example.rb'
-    - 'config/initializers/action_mailer.rb'
-    - 'config/load.rb'
-    - 'lib/tasks/errbit/demo.rake'
-    - 'spec/decorators/issue_tracker_type_decorator_spec.rb'
-    - 'spec/initializers/action_mailer_spec.rb'
-    - 'spec/initializers/devise_spec.rb'
-
-# Offense count: 9
-# Cop supports --auto-correct.
-Style/TrailingWhitespace:
-  Exclude:
-    - 'app/models/notification_service.rb'
-    - 'app/models/notification_services/gtalk_service.rb'
-    - 'config/initializers/cve-2013-0156.rb'
-
-# Offense count: 2
-Style/UnlessElse:
-  Exclude:
-    - 'app/controllers/problems_searcher.rb'
@@ -29,8 +29,8 @@ gem &#39;htmlentities&#39;
 gem 'kaminari', '>= 0.14.1'
 gem 'mongoid', '5.0.0'
 gem 'mongoid_rails_migrations'
-gem 'rack-ssl', :require => 'rack/ssl' # force SSL
-gem 'rack-ssl-enforcer', :require => false
+gem 'rack-ssl', require: 'rack/ssl' # force SSL
+gem 'rack-ssl-enforcer', require: false
 gem 'rails_autolink'
 gem 'useragent'
  
@@ -44,7 +44,7 @@ gem &#39;campy&#39;
 # Hipchat
 gem 'hipchat'
 # Google Talk
-gem 'xmpp4r', :require => ["xmpp4r", "xmpp4r/muc"]
+gem 'xmpp4r', require: ["xmpp4r", "xmpp4r/muc"]
 # Hoiio (SMS)
 gem 'hoi'
 # Pushover (iOS Push notifications)
@@ -64,7 +64,7 @@ gem &#39;yajl-ruby&#39;, platform: &#39;ruby&#39;
 gem 'json', platform: 'jruby'
  
 group :development, :test do
-  gem 'airbrake', :require => false
+  gem 'airbrake', require: false
   gem 'pry-rails'
   gem 'pry-byebug', platforms: [:mri]
   gem 'quiet_assets'
@@ -103,7 +103,7 @@ group :heroku, :production do
   gem 'unicorn', require: false, platform: 'ruby'
 end
  
-gem 'therubyracer', :platform => :ruby # C Ruby (MRI) or Rubinius, but NOT Windows
+gem 'therubyracer', platform: :ruby # C Ruby (MRI) or Rubinius, but NOT Windows
 gem 'sass-rails'
 gem 'uglifier'
 # We can't upgrade because not compatible to jquery >= 1.9.
@@ -8,7 +8,7 @@ Rails.application.load_tasks
 begin
   require 'rspec/core/rake_task'
   RSpec::Core::RakeTask.new(:spec)
-  task :default => :spec
+  task default: :spec
 rescue LoadError
   warn "Notice: no rspec tasks available in this environment"
 end
@@ -8,16 +8,16 @@ class Api::V1::NoticesController &lt; ApplicationController
     if params.key?(:start_date) && params.key?(:end_date)
       start_date = Time.zone.parse(params[:start_date]).utc
       end_date = Time.zone.parse(params[:end_date]).utc
-      query = {:created_at => {"$lte" => end_date, "$gte" => start_date}}
+      query = { created_at: { "$lte" => end_date, "$gte" => start_date } }
     end
  
     results = benchmark("[api/v1/notices_controller] query time") do
-      Notice.where(query).with(:consistency => :strong).only(fields).to_a
+      Notice.where(query).with(consistency: :strong).only(fields).to_a
     end
  
     respond_to do |format|
-      format.any(:html, :json) { render :json => JSON.dump(results) } # render JSON if no extension specified on path
-      format.xml { render :xml => results }
+      format.any(:html, :json) { render json: JSON.dump(results) } # render JSON if no extension specified on path
+      format.xml { render xml: results }
     end
   end
 end
@@ -13,8 +13,8 @@ class Api::V1::ProblemsController &lt; ApplicationController
     end
  
     respond_to do |format|
-      format.any(:html, :json) { render :json => result } # render JSON if no extension specified on path
-      format.xml { render :xml => result }
+      format.any(:html, :json) { render json: result } # render JSON if no extension specified on path
+      format.xml { render xml: result }
     end
   end
  
@@ -24,16 +24,16 @@ class Api::V1::ProblemsController &lt; ApplicationController
     if params.key?(:start_date) && params.key?(:end_date)
       start_date = Time.parse(params[:start_date]).utc
       end_date = Time.parse(params[:end_date]).utc
-      query = {:first_notice_at => {"$lte" => end_date}, "$or" => [{:resolved_at => nil}, {:resolved_at => {"$gte" => start_date}}]}
+      query = { :first_notice_at => { "$lte" => end_date }, "$or" => [{ resolved_at: nil }, { resolved_at: { "$gte" => start_date } }] }
     end
  
     results = benchmark("[api/v1/problems_controller/index] query time") do
-      Problem.where(query).with(:consistency => :strong).only(FIELDS).to_a
+      Problem.where(query).with(consistency: :strong).only(FIELDS).to_a
     end
  
     respond_to do |format|
-      format.any(:html, :json) { render :json => JSON.dump(results) } # render JSON if no extension specified on path
-      format.xml { render :xml => results }
+      format.any(:html, :json) { render json: JSON.dump(results) } # render JSON if no extension specified on path
+      format.xml { render xml: results }
     end
   end
 end
@@ -11,21 +11,21 @@ class Api::V1::StatsController &lt; ApplicationController
     end
  
     stats = {
-      :name => @app.name,
-      :id => @app.id,
-      :last_error_time => @last_error_time,
-      :unresolved_errors => @app.unresolved_count
+      name:              @app.name,
+      id:                @app.id,
+      last_error_time:   @last_error_time,
+      unresolved_errors: @app.unresolved_count
     }
  
     respond_to do |format|
-      format.any(:html, :json) { render :json => JSON.dump(stats) } # render JSON if no extension specified on path
-      format.xml { render :xml => stats }
+      format.any(:html, :json) { render json: JSON.dump(stats) } # render JSON if no extension specified on path
+      format.xml { render xml: stats }
     end
   end
  
   protected def require_api_key_or_authenticate_user!
     if params[:api_key].present?
-      if (@app = App.where(:api_key => params[:api_key]).first)
+      if (@app = App.where(api_key: params[:api_key]).first)
         return true
       end
     end
@@ -15,7 +15,7 @@ class Api::V3::NoticesController &lt; ApplicationController
       if report.should_keep?
         report.generate_notice!
         render json: {
-          id: report.notice.id,
+          id:  report.notice.id,
           url: app_problem_url(
             report.app,
             report.error.problem_id)
@@ -5,7 +5,7 @@ class ApplicationController &lt; ActionController::Base
   before_action :authenticate_user!
   before_action :set_time_zone
  
-  rescue_from ActionController::RedirectBackError, :with => :redirect_to_root
+  rescue_from ActionController::RedirectBackError, with: :redirect_to_root
  
   class StrongParametersWithEagerAttributesStrategy < DecentExposure::StrongParametersStrategy
     def assign_attributes?
 class AppsController < ApplicationController
   include ProblemsSearcher
  
-  before_action :require_admin!, :except => [:index, :show]
-  before_action :parse_email_at_notices_or_set_default, :only => [:create, :update]
-  before_action :parse_notice_at_notices_or_set_default, :only => [:create, :update]
+  before_action :require_admin!, except: [:index, :show]
+  before_action :parse_email_at_notices_or_set_default, only: [:create, :update]
+  before_action :parse_notice_at_notices_or_set_default, only: [:create, :update]
   respond_to :html
  
   expose(:app_scope) { App }
@@ -39,7 +39,7 @@ class AppsController &lt; ApplicationController
   }
  
   expose(:users) {
-    User.all.sort_by {|u| u.name.downcase }
+    User.all.sort_by { |u| u.name.downcase }
   }
  
   def index; end
@@ -55,7 +55,7 @@ class AppsController &lt; ApplicationController
   def create
     initialize_subclassed_notification_service
     if app.save
-      redirect_to app_url(app), :flash => { :success => I18n.t('controllers.apps.flash.create.success') }
+      redirect_to app_url(app), flash: { success: I18n.t('controllers.apps.flash.create.success') }
     else
       flash[:error] = I18n.t('controllers.apps.flash.create.error')
       render :new
@@ -65,7 +65,7 @@ class AppsController &lt; ApplicationController
   def update
     initialize_subclassed_notification_service
     if app.save
-      redirect_to app_url(app), :flash => { :success => I18n.t('controllers.apps.flash.update.success') }
+      redirect_to app_url(app), flash: { success: I18n.t('controllers.apps.flash.update.success') }
     else
       flash[:error] = I18n.t('controllers.apps.flash.update.error')
       render :edit
@@ -78,7 +78,7 @@ class AppsController &lt; ApplicationController
  
   def destroy
     if app.destroy
-      redirect_to apps_url, :flash => { :success => I18n.t('controllers.apps.flash.destroy.success') }
+      redirect_to apps_url, flash: { success: I18n.t('controllers.apps.flash.destroy.success') }
     else
       flash[:error] = I18n.t('controllers.apps.flash.destroy.error')
       render :show
@@ -96,7 +96,7 @@ protected
     # set the app's notification service
     if params[:app][:notification_service_attributes] && (notification_type = params[:app][:notification_service_attributes][:type])
       available_notification_classes = [NotificationService] + NotificationService.subclasses
-      notification_class = available_notification_classes.detect{|c| c.name == notification_type}
+      notification_class = available_notification_classes.detect { |c| c.name == notification_type }
       if notification_class.present?
         app.notification_service = notification_class.new(params[:app][:notification_service_attributes])
       end
@@ -116,7 +116,7 @@ protected
       # Sanitize negative values, split on comma,
       # strip, parse as integer, remove all '0's.
       # If empty, set as default and show an error message.
-      email_at_notices = val.gsub(/-\d+/, "").split(",").map{|v| v.strip.to_i }.reject{|v| v == 0}
+      email_at_notices = val.gsub(/-\d+/, "").split(",").map { |v| v.strip.to_i }.reject { |v| v == 0 }
       if email_at_notices.any?
         params[:app][:email_at_notices] = email_at_notices
       else
@@ -131,7 +131,7 @@ protected
       # Sanitize negative values, split on comma,
       # strip, parse as integer, remove all '0's.
       # If empty, set as default and show an error message.
-      notify_at_notices = val.gsub(/-\d+/, "").split(",").map{|v| v.strip.to_i }
+      notify_at_notices = val.gsub(/-\d+/, "").split(",").map { |v| v.strip.to_i }
       if notify_at_notices.any?
         params[:app][:notification_service_attributes][:notify_at_notices] = notify_at_notices
       else
@@ -3,7 +3,7 @@ class CommentsController &lt; ApplicationController
   before_action :find_problem
  
   def create
-    @comment = Comment.new(comment_params.merge(:user_id => current_user.id))
+    @comment = Comment.new(comment_params.merge(user_id: current_user.id))
     if @comment.valid?
       @problem.comments << @comment
       @problem.save
 class DeploysController < ApplicationController
-  protect_from_forgery :except => :create
+  protect_from_forgery except: :create
  
-  skip_before_action :verify_authenticity_token, :only => :create
-  skip_before_action :authenticate_user!, :only => :create
+  skip_before_action :verify_authenticity_token, only: :create
+  skip_before_action :authenticate_user!, only: :create
  
   def create
     @app = App.find_by_api_key!(params[:api_key])
     @deploy = @app.deploys.create!(default_deploy || heroku_deploy)
-    render :xml => @deploy
+    render xml: @deploy
   end
  
   def index
@@ -21,11 +21,11 @@ private
   def default_deploy
     if params[:deploy]
       {
-        :username     => params[:deploy][:local_username],
-        :environment  => params[:deploy][:rails_env],
-        :repository   => params[:deploy][:scm_repository],
-        :revision     => params[:deploy][:scm_revision],
-        :message      => params[:deploy][:message]
+        username:    params[:deploy][:local_username],
+        environment: params[:deploy][:rails_env],
+        repository:  params[:deploy][:scm_repository],
+        revision:    params[:deploy][:scm_revision],
+        message:     params[:deploy][:message]
       }
     end
   end
@@ -33,10 +33,10 @@ private
   # handle Heroku's HTTP post deployhook format
   def heroku_deploy
     {
-      :username     => params[:user],
-      :environment  => params[:rack_env].try(:downcase) || params[:app],
-      :repository   => "git@heroku.com:#{params[:app]}.git",
-      :revision     => params[:head],
+      username:    params[:user],
+      environment: params[:rack_env].try(:downcase) || params[:app],
+      repository:  "git@heroku.com:#{params[:app]}.git",
+      revision:    params[:head]
     }
   end
 end
@@ -4,7 +4,7 @@ class NoticesController &lt; ApplicationController
   skip_before_action :authenticate_user!, only: :create
   skip_before_action :verify_authenticity_token, only: :create
  
-  rescue_from ParamsError, :with => :bad_params
+  rescue_from ParamsError, with: :bad_params
  
   def create
     # params[:data] if the notice came from a GET request, raw_post if it came via POST
@@ -13,18 +13,18 @@ class NoticesController &lt; ApplicationController
     if report.valid?
       if report.should_keep?
         report.generate_notice!
-        api_xml = report.notice.to_xml(:only => false, :methods => [:id]) do |xml|
-          xml.url locate_url(report.notice.id, :host => Errbit::Config.host)
+        api_xml = report.notice.to_xml(only: false, methods: [:id]) do |xml|
+          xml.url locate_url(report.notice.id, host: Errbit::Config.host)
         end
-        render :xml => api_xml
+        render xml: api_xml
       else
-        render :text => "Notice for old app version ignored"
+        render text: "Notice for old app version ignored"
       end
     else
-      render :text => "Your API key is unknown", :status => 422
+      render text: "Your API key is unknown", status: 422
     end
   rescue Nokogiri::XML::SyntaxError
-    render :text => 'The provided XML was not well-formed', :status => 422
+    render text: 'The provided XML was not well-formed', status: 422
   end
  
   # Redirects a notice to the problem page. Useful when using User Information at Airbrake gem.
@@ -39,12 +39,12 @@ private
     return @notice_params if @notice_params
     @notice_params = params[:data] || request.raw_post
     if @notice_params.blank?
-      raise ParamsError.new('Need a data params in GET or raw post data')
+      fail ParamsError, 'Need a data params in GET or raw post data'
     end
     @notice_params
   end
  
   def bad_params(exception)
-    render :text => exception.message, :status => :bad_request
+    render text: exception.message, status: :bad_request
   end
 end
@@ -7,12 +7,12 @@
 class ProblemsController < ApplicationController
   include ProblemsSearcher
  
-  before_action :need_selected_problem, :only => [
+  before_action :need_selected_problem, only: [
     :resolve_several, :unresolve_several, :unmerge_several
   ]
  
   expose(:app_scope) {
-    params[:app_id] ? App.where(:_id => params[:app_id]) : App.all
+    params[:app_id] ? App.where(_id: params[:app_id]) : App.all
   }
  
   expose(:app) {
@@ -32,11 +32,11 @@ class ProblemsController &lt; ApplicationController
   }
  
   expose(:problems) {
-    pro = Problem
-      .for_apps(app_scope)
-      .in_env(params_environement)
-      .all_else_unresolved(all_errs)
-      .ordered_by(params_sort, params_order)
+    pro = Problem.
+      for_apps(app_scope).
+      in_env(params_environement).
+      all_else_unresolved(all_errs).
+      ordered_by(params_sort, params_order)
  
     if request.format == :html
       pro.page(params[:page]).per(current_user.per_page)
@@ -48,8 +48,8 @@ class ProblemsController &lt; ApplicationController
   def index; end
  
   def show
-    @notices = problem.object.notices.reverse_ordered
-      .page(params[:notice]).per(1)
+    @notices = problem.object.notices.reverse_ordered.
+      page(params[:notice]).per(1)
     @notice  = NoticeDecorator.new @notices.first
     @comment = Comment.new
   end
@@ -80,13 +80,13 @@ class ProblemsController &lt; ApplicationController
  
   def resolve_several
     selected_problems.each(&:resolve!)
-    flash[:success] = "Great news everyone! #{I18n.t(:n_errs_have, :count => selected_problems.count)} been resolved."
+    flash[:success] = "Great news everyone! #{I18n.t(:n_errs_have, count: selected_problems.count)} been resolved."
     redirect_to :back
   end
  
   def unresolve_several
     selected_problems.each(&:unresolve!)
-    flash[:success] = "#{I18n.t(:n_errs_have, :count => selected_problems.count)} been unresolved."
+    flash[:success] = "#{I18n.t(:n_errs_have, count: selected_problems.count)} been unresolved."
     redirect_to :back
   end
  
@@ -100,26 +100,26 @@ class ProblemsController &lt; ApplicationController
       flash[:notice] = I18n.t('controllers.problems.flash.need_two_errors_merge')
     else
       ProblemMerge.new(selected_problems).merge
-      flash[:notice] = I18n.t('controllers.problems.flash.merge_several.success', :nb => selected_problems.count)
+      flash[:notice] = I18n.t('controllers.problems.flash.merge_several.success', nb: selected_problems.count)
     end
     redirect_to :back
   end
  
   def unmerge_several
     all = selected_problems.map(&:unmerge!).flatten
-    flash[:success] = "#{I18n.t(:n_errs_have, :count => all.length)} been unmerged."
+    flash[:success] = "#{I18n.t(:n_errs_have, count: all.length)} been unmerged."
     redirect_to :back
   end
  
   def destroy_several
     nb_problem_destroy = ProblemDestroy.execute(selected_problems)
-    flash[:notice] = "#{I18n.t(:n_errs_have, :count => nb_problem_destroy)} been deleted."
+    flash[:notice] = "#{I18n.t(:n_errs_have, count: nb_problem_destroy)} been deleted."
     redirect_to :back
   end
  
   def destroy_all
     nb_problem_destroy = ProblemDestroy.execute(app.problems)
-    flash[:success] = "#{I18n.t(:n_errs_have, :count => nb_problem_destroy)} been deleted."
+    flash[:success] = "#{I18n.t(:n_errs_have, count: nb_problem_destroy)} been deleted."
     redirect_to :back
   rescue ActionController::RedirectBackError
     redirect_to app_path(app)
@@ -6,18 +6,18 @@ module ProblemsSearcher
  
   included do
     expose(:params_sort) {
-      unless %w{app message last_notice_at last_deploy_at count}.member?(params[:sort])
-        "last_notice_at"
-      else
+      if %w{app message last_notice_at last_deploy_at count}.member?(params[:sort])
         params[:sort]
+      else
+        "last_notice_at"
       end
     }
  
-    expose(:params_order){
-      unless %w{asc desc}.member?(params[:order])
-        'desc'
-      else
+    expose(:params_order) {
+      if %w{asc desc}.member?(params[:order])
         params[:order]
+      else
+        'desc'
       end
     }
  
@@ -13,10 +13,10 @@ class SiteConfigController &lt; ApplicationController
   end
  
   private def filtered_update_params
-    params
-      .require(:site_config)
-      .require(:notice_fingerprinter_attributes)
-      .permit(
+    params.
+      require(:site_config).
+      require(:notice_fingerprinter_attributes).
+      permit(
         :error_class,
         :message,
         :backtrace_lines,
@@ -2,7 +2,7 @@ class Users::OmniauthCallbacksController &lt; Devise::OmniauthCallbacksController
   def github
     github_login = env["omniauth.auth"].extra.raw_info.login
     github_token = env["omniauth.auth"].credentials.token
-    github_user  = User.where(:github_login => github_login).first
+    github_user  = User.where(github_login: github_login).first
     github_site_title = Errbit::Config.github_site_title
  
     if github_user.nil? && (github_org_id = Errbit::Config.github_org_id)
@@ -10,7 +10,7 @@ class Users::OmniauthCallbacksController &lt; Devise::OmniauthCallbacksController
       # If they are, automatically create an account
       client = Octokit::Client.new(access_token: github_token)
       client.api_endpoint = Errbit::Config.github_api_url
-      org_ids = client.organizations.map { |org| org.id }
+      org_ids = client.organizations.map(&:id)
       if org_ids.include?(github_org_id)
         github_user = User.create(name: env["omniauth.auth"].extra.raw_info.name, email: env["omniauth.auth"].extra.raw_info.email)
       end
@@ -31,8 +31,8 @@ class Users::OmniauthCallbacksController &lt; Devise::OmniauthCallbacksController
     elsif github_user
       # Store OAuth token
       update_user_with_github_attributes(github_user, github_login, github_token)
-      flash[:success] = I18n.t "devise.omniauth_callbacks.success", :kind => github_site_title
-      sign_in_and_redirect github_user, :event => :authentication
+      flash[:success] = I18n.t "devise.omniauth_callbacks.success", kind: github_site_title
+      sign_in_and_redirect github_user, event: :authentication
     else
       flash[:error] = "There are no authorized users with #{github_site_title} login '#{github_login}'. Please ask an administrator to register your user account."
       redirect_to new_user_session_path
@@ -41,8 +41,8 @@ class Users::OmniauthCallbacksController &lt; Devise::OmniauthCallbacksController
  
   private def update_user_with_github_attributes(user, login, token)
     user.update_attributes(
-      :github_login        => login,
-      :github_oauth_token  => token
+      github_login:       login,
+      github_oauth_token: token
     )
   end
 end
 class UsersController < ApplicationController
   respond_to :html
  
-  before_action :require_admin!, :except => [:edit, :update]
-  before_action :require_user_edit_priviledges, :only => [:edit, :update]
+  before_action :require_admin!, except: [:edit, :update]
+  before_action :require_user_edit_priviledges, only: [:edit, :update]
  
-  expose(:user, :attributes => :user_params)
+  expose(:user, attributes: :user_params)
   expose(:users) {
     User.all.page(params[:page]).per(current_user.per_page)
   }
@@ -24,7 +24,7 @@ class UsersController &lt; ApplicationController
  
   def update
     if user.update_attributes(user_params)
-      flash[:success] = I18n.t('controllers.users.flash.update.success', :name => user.name)
+      flash[:success] = I18n.t('controllers.users.flash.update.success', name: user.name)
       redirect_to user_path(user)
     else
       render :edit
@@ -41,13 +41,13 @@ class UsersController &lt; ApplicationController
       flash[:error] = I18n.t('controllers.users.flash.destroy.error')
     else
       UserDestroy.new(user).destroy
-      flash[:success] = I18n.t('controllers.users.flash.destroy.success', :name => user.name)
+      flash[:success] = I18n.t('controllers.users.flash.destroy.success', name: user.name)
     end
     redirect_to users_path
   end
  
   def unlink_github
-    user.update_attributes :github_login => nil, :github_oauth_token => nil
+    user.update_attributes github_login: nil, github_oauth_token: nil
     redirect_to user_path(user)
   end
  
@@ -65,7 +65,7 @@ protected
   def user_permit_params
     @user_permit_params ||= [:name, :username, :email, :github_login, :per_page, :time_zone]
     @user_permit_params << :admin if current_user.admin? && current_user.id != params[:id]
-    @user_permit_params |= [:password, :password_confirmation] if user_password_params.values.all?{|pa| !pa.blank? }
+    @user_permit_params |= [:password, :password_confirmation] if user_password_params.values.all? { |pa| !pa.blank? }
     @user_permit_params
   end
  
@@ -6,7 +6,7 @@ class WatchersController &lt; ApplicationController
   end
  
   def destroy
-    watcher = app.watchers.where(:user_id => params[:id]).first
+    watcher = app.watchers.where(user_id: params[:id]).first
     app.watchers.delete(watcher)
     flash[:success] = t('.success', app: app.name)
     redirect_to app_path(app)
 class AppDecorator < Draper::Decorator
   decorates_association :watchers
-  decorates_association :issue_tracker, :with => IssueTrackerDecorator
+  decorates_association :issue_tracker, with: IssueTrackerDecorator
   delegate_all
  
   def email_at_notices
@@ -44,9 +44,9 @@ class BacktraceLineDecorator &lt; Draper::Decorator
   end
  
   def decorated_path
-    path
-      .sub(Backtrace::IN_APP_PATH, '')
-      .sub(Backtrace::GEMS_PATH, "<strong>\\1</strong>")
+    path.
+      sub(Backtrace::IN_APP_PATH, '').
+      sub(Backtrace::GEMS_PATH, "<strong>\\1</strong>")
   end
  
 private
@@ -67,25 +67,25 @@ private
  
   def link_to_hosted_javascript(app, text)
     if app.asset_host?
-      h.link_to(text, "#{app.asset_host}/#{file_relative}", :target => '_blank')
+      h.link_to(text, "#{app.asset_host}/#{file_relative}", target: '_blank')
     end
   end
  
   def link_to_github(app, text = nil)
     return unless app.github_repo?
     href = "%s#L%s" % [app.github_url_to_file(decorated_path + file_name), number]
-    h.link_to(text || file_name, href, :target => '_blank')
+    h.link_to(text || file_name, href, target: '_blank')
   end
  
   def link_to_bitbucket(app, text = nil)
     return unless app.bitbucket_repo?
     href = "%s#%s-%s" % [app.bitbucket_url_to_file(decorated_path + file_name), file_name, number]
-    h.link_to(text || file_name, href, :target => '_blank')
+    h.link_to(text || file_name, href, target: '_blank')
   end
  
   def link_to_issue_tracker_file(app, text = nil)
     return unless app.issue_tracker && app.issue_tracker.respond_to?(:url_to_file)
     href = app.issue_tracker.url_to_file(file_relative, number)
-    h.link_to(text || file_name, href, :target => '_blank')
+    h.link_to(text || file_name, href, target: '_blank')
   end
 end
@@ -13,8 +13,8 @@ class IssueTrackerFieldDecorator &lt; Draper::Decorator
  
   def input(form, issue_tracker)
     form.send(input_field, key.to_s,
-      :placeholder => field_info[:placeholder],
-      :value => issue_tracker.options[key.to_s])
+      placeholder: field_info[:placeholder],
+      value:       issue_tracker.options[key.to_s])
   end
  
   private def input_field
@@ -7,7 +7,8 @@ class IssueTrackerTypeDecorator &lt; Draper::Decorator
     return unless object.icons
  
     object.icons.reduce({}) do |c, (k, v)|
-      c[k] = "data:#{v[0]};base64,#{Base64.encode64(v[1])}"; c
+      c[k] = "data:#{v[0]};base64,#{Base64.encode64(v[1])}"
+      c
     end
   end
  
@@ -13,7 +13,7 @@ module ApplicationHelper
           event.dtend       = notice.created_at.utc + 60.minutes
           event.organizer   = notice.server_environment && notice.server_environment["hostname"]
           event.location    = notice.project_root
-          event.url         = app_problem_url(:app_id => notice.problem.app.id, :id => notice.problem)
+          event.url         = app_problem_url(app_id: notice.problem.app.id, id: notice.problem)
         end
       end
     end.to_s
@@ -53,12 +53,12 @@ module ApplicationHelper
     end
   end
  
-  def create_percentage_table_from_tallies(tallies, options={})
+  def create_percentage_table_from_tallies(tallies, options = {})
     total   = (options[:total] || total_from_tallies(tallies))
     percent = 100.0 / total.to_f
-    rows    = tallies.map {|value, count| [(count.to_f * percent), value]} \
-                     .sort {|a, b| b[0] <=> a[0]}
-    render "problems/tally_table", :rows => rows
+    rows    = tallies.map { |value, count| [(count.to_f * percent), value] }. \
+                     sort { |a, b| b[0] <=> a[0] }
+    render "problems/tally_table", rows: rows
   end
  
   def head(collection)
@@ -78,7 +78,7 @@ module ApplicationHelper
 private
  
   def total_from_tallies(tallies)
-    tallies.values.inject(0) {|sum, n| sum + n}
+    tallies.values.sum
   end
  
   def head_size
@@ -2,11 +2,11 @@ module AppsHelper
   def link_to_copy_attributes_from_other_app
     if App.count > 1
       html =  link_to('copy settings from another app', '#',
-        :class => 'button copy_config')
+        class: 'button copy_config')
       html << select("duplicate", "app",
-        App.all.asc(:name).reject{|a| a == @app }.
-        collect{|p| [ p.name, p.id ] }, {:include_blank => "[choose app]"},
-        {:class => "choose_other_app", :style => "display: none;"})
+        App.all.asc(:name).reject { |a| a == @app }.
+        collect { |p| [p.name, p.id] }, { include_blank: "[choose app]" },
+        class: "choose_other_app", style: "display: none;")
       return html
     end
   end
@@ -2,10 +2,12 @@ module FormHelper
   def errors_for(document)
     return unless document.errors.any?
  
-    content_tag(:div, :class => 'error-messages') do
+    content_tag(:div, class: 'error-messages') do
       body = content_tag(:h2, 'Dang. The following errors are keeping this from being a success.')
       body + content_tag(:ul) do
-        document.errors.full_messages.inject('') {|errs, msg| errs + content_tag(:li, h(msg)) }.html_safe
+        document.errors.full_messages.inject('') do |errs, msg|
+          errs + content_tag(:li, h(msg))
+        end.html_safe
       end
     end
   end
@@ -19,10 +19,10 @@ module NavigationHelper
                   matches
                 when Array
                   s = {}
-                  matches.each {|c| s[c] = :all}
+                  matches.each { |c| s[c] = :all }
                   s
                 else
-                  {matches => :all}
+                  { matches => :all }
                 end
  
     active = nil
 # encoding: utf-8
 module NoticesHelper
   def notice_atom_summary(notice)
-    render "notices/atom_entry", :notice => notice
+    render "notices/atom_entry", notice: notice
   end
 end
@@ -6,21 +6,21 @@ module ProblemsHelper
   def truncated_problem_message(problem)
     unless (msg = problem.message).blank?
       # Truncate & insert invisible chars so that firefox can emulate 'word-wrap: break-word' CSS rule
-      truncate(msg, :length => 300, :escape => false).scan(/.{1,5}/).map { |s| h(s) }.join("&#8203;").html_safe
+      truncate(msg, length: 300, escape: false).scan(/.{1,5}/).map { |s| h(s) }.join("&#8203;").html_safe
     end
   end
  
   def gravatar_tag(email, options = {})
     return nil unless email.present?
  
-    image_tag gravatar_url(email, options), :alt => email, :class => 'gravatar'
+    image_tag gravatar_url(email, options), alt: email, class: 'gravatar'
   end
  
   def gravatar_url(email, options = {})
     return nil unless email.present?
  
     default_options = {
-      :d => Errbit::Config.gravatar_default,
+      d: Errbit::Config.gravatar_default
     }
     options.reverse_merge! default_options
     params = options.extract!(:s, :d).delete_if { |_k, v| v.blank? }
 # encoding: utf-8
 module SortHelper
-  def link_for_sort(name, field=nil)
+  def link_for_sort(name, field = nil)
     field ||= name.underscore
     current = (params_sort == field)
     order   = (current && (params_order == "asc")) ? "desc" : "asc"
     url     = request.path + "?sort=#{field}&order=#{order}"
     options = {}
-    options.merge!(:class => "current #{order}") if current
+    options.merge!(class: "current #{order}") if current
     link_to(name, url, options)
   end
 end
@@ -20,7 +20,7 @@ class ProblemDestroy
   #   the number of problem destroy
   #
   def self.execute(problems)
-    Array(problems).each{ |problem|
+    Array(problems).each { |problem|
       ProblemDestroy.new(problem).execute
     }.count
   end
@@ -36,11 +36,11 @@ private
   end
  
   def delete_errs
-    Notice.delete_all(:err_id => { '$in' => errs_id })
-    Err.delete_all(:_id => { '$in' => errs_id })
+    Notice.delete_all(err_id: { '$in' => errs_id })
+    Err.delete_all(_id: { '$in' => errs_id })
   end
  
   def delete_comments
-    Comment.delete_all(:_id => { '$in' => comments_id })
+    Comment.delete_all(_id: { '$in' => comments_id })
   end
 end
@@ -5,7 +5,7 @@ class ProblemMerge
     problems = problems.flatten.uniq
     @merged_problem = problems[0]
     @child_problems = problems[1..-1]
-    raise ArgumentError.new("need almost 2 uniq different problems") if @child_problems.empty?
+    fail ArgumentError, "need almost 2 uniq different problems" if @child_problems.empty?
   end
   attr_reader :merged_problem, :child_problems
  
@@ -26,6 +26,6 @@ private
   end
  
   def repair_database
-    Mongoid.default_client.command :repairDatabase => 1
+    Mongoid.default_client.command repairDatabase: 1
   end
 end
@@ -5,12 +5,12 @@ require Rails.root.join(&#39;config/routes.rb&#39;)
 class Mailer < ActionMailer::Base
   helper ApplicationHelper
  
-  default :from => Errbit::Config.email_from,
-          'X-Errbit-Host' => Errbit::Config.host,
-          'X-Mailer' => 'Errbit',
+  default :from                      => Errbit::Config.email_from,
+          'X-Errbit-Host'            => Errbit::Config.host,
+          'X-Mailer'                 => 'Errbit',
           'X-Auto-Response-Suppress' => 'OOF, AutoReply',
-          'Precedence' => 'bulk',
-          'Auto-Submitted' => 'auto-generated'
+          'Precedence'               => 'bulk',
+          'Auto-Submitted'           => 'auto-generated'
  
   def err_notification(error_report)
     @notice   = NoticeDecorator.new error_report.notice
@@ -19,25 +19,25 @@ class Mailer &lt; ActionMailer::Base
     count = error_report.problem.notices_count
     count = count > 1 ? "(#{count}) " : ""
  
-    errbit_headers 'App' => @app.name,
+    errbit_headers 'App'         => @app.name,
                    'Environment' => @notice.environment_name,
-                   'Error-Id' => @notice.err_id
+                   'Error-Id'    => @notice.err_id
  
-    mail :to      => @app.notification_recipients,
-         :subject => "#{count}[#{@app.name}][#{@notice.environment_name}] #{@notice.message.truncate(50)}"
+    mail to:      @app.notification_recipients,
+         subject: "#{count}[#{@app.name}][#{@notice.environment_name}] #{@notice.message.truncate(50)}"
   end
  
   def deploy_notification(deploy)
     @deploy = deploy
     @app = AppDecorator.new deploy.app
  
-    errbit_headers 'App' => @app.name,
-                   'Environment' => @deploy.environment,
+    errbit_headers 'App'             => @app.name,
+                   'Environment'     => @deploy.environment,
                    'Deploy-Revision' => @deploy.revision,
-                   'Deploy-User' => @deploy.username
+                   'Deploy-User'     => @deploy.username
  
-    mail :to       => @app.notification_recipients,
-         :subject  => "[#{@app.name}] Deployed to #{@deploy.environment} by #{@deploy.username}"
+    mail to:      @app.notification_recipients,
+         subject: "[#{@app.name}] Deployed to #{@deploy.environment} by #{@deploy.username}"
   end
  
   def comment_notification(comment)
@@ -49,13 +49,13 @@ class Mailer &lt; ActionMailer::Base
  
     recipients = @comment.notification_recipients
  
-    errbit_headers 'App' => @app.name,
-                   'Environment' => @notice.environment_name,
-                   'Problem-Id' => @problem.id,
+    errbit_headers 'App'            => @app.name,
+                   'Environment'    => @notice.environment_name,
+                   'Problem-Id'     => @problem.id,
                    'Comment-Author' => @user.name
  
-    mail :to      => recipients,
-         :subject => "#{@user.name} commented on [#{@app.name}][#{@notice.environment_name}] #{@notice.message.truncate(50)}"
+    mail to:      recipients,
+         subject: "#{@user.name} commented on [#{@app.name}][#{@notice.environment_name}] #{@notice.message.truncate(50)}"
   end
  
   private def errbit_headers(header)
@@ -3,56 +3,59 @@ class App
   include Mongoid::Document
   include Mongoid::Timestamps
  
-  field :name, :type => String
+  field :name, type: String
   field :api_key
   field :github_repo
   field :bitbucket_repo
   field :asset_host
   field :repository_branch
   field :current_app_version
-  field :resolve_errs_on_deploy, :type => Boolean, :default => false
-  field :notify_all_users, :type => Boolean, :default => false
-  field :notify_on_errs, :type => Boolean, :default => true
-  field :notify_on_deploys, :type => Boolean, :default => false
-  field :email_at_notices, :type => Array, :default => Errbit::Config.email_at_notices
+  field :resolve_errs_on_deploy, type: Boolean, default: false
+  field :notify_all_users, type: Boolean, default: false
+  field :notify_on_errs, type: Boolean, default: true
+  field :notify_on_deploys, type: Boolean, default: false
+  field :email_at_notices, type: Array, default: Errbit::Config.email_at_notices
  
   # Some legacy apps may have string as key instead of BSON::ObjectID
   # identity :type => String
   field :_id,
-    type: String,
+    type:          String,
     pre_processed: true,
-    default: ->{ BSON::ObjectId.new.to_s }
+    default:       -> { BSON::ObjectId.new.to_s }
  
   embeds_many :watchers
   embeds_many :deploys
-  embeds_one :issue_tracker, :class_name => 'IssueTracker'
+  embeds_one :issue_tracker, class_name: 'IssueTracker'
   embeds_one :notification_service
   embeds_one :notice_fingerprinter, autobuild: true
  
-  has_many :problems, :inverse_of => :app, :dependent => :destroy
+  has_many :problems, inverse_of: :app, dependent: :destroy
  
-  before_validation :generate_api_key, :on => :create
+  before_validation :generate_api_key, on: :create
   before_save :normalize_github_repo
   after_update :store_cached_attributes_on_problems
  
   validates_presence_of :name, :api_key
-  validates_uniqueness_of :name, :allow_blank => true
-  validates_uniqueness_of :api_key, :allow_blank => true
+  validates_uniqueness_of :name, allow_blank: true
+  validates_uniqueness_of :api_key, allow_blank: true
   validates_associated :watchers
   validates_associated :notice_fingerprinter
   validate :check_issue_tracker
  
-  accepts_nested_attributes_for :watchers, :allow_destroy => true,
-    :reject_if => proc { |attrs| attrs[:user_id].blank? && attrs[:email].blank? }
-  accepts_nested_attributes_for :issue_tracker, :allow_destroy => true,
-    :reject_if => proc { |attrs| !ErrbitPlugin::Registry.issue_trackers.keys.map(&:to_s).include?(attrs[:type_tracker].to_s) }
-  accepts_nested_attributes_for :notification_service, :allow_destroy => true,
-    :reject_if => proc { |attrs| !NotificationService.subclasses.map(&:to_s).include?(attrs[:type].to_s) }
+  accepts_nested_attributes_for :watchers,
+    allow_destroy: true,
+    reject_if:     proc { |attrs| attrs[:user_id].blank? && attrs[:email].blank? }
+  accepts_nested_attributes_for :issue_tracker,
+    allow_destroy: true,
+    reject_if:     proc { |attrs| !ErrbitPlugin::Registry.issue_trackers.keys.map(&:to_s).include?(attrs[:type_tracker].to_s) }
+  accepts_nested_attributes_for :notification_service,
+    allow_destroy: true,
+    reject_if:     proc { |attrs| !NotificationService.subclasses.map(&:to_s).include?(attrs[:type].to_s) }
   accepts_nested_attributes_for :notice_fingerprinter
  
-  scope :watched_by, ->(user) do
+  scope :watched_by, lambda { |user|
     where watchers: { "$elemMatch" => { "user_id" => user.id } }
-  end
+  }
  
   def watched_by?(user)
     watchers.pluck("user_id").include? user.id
@@ -71,7 +74,7 @@ class App
     problem = problems.create!(
       error_class: attrs[:error_class],
       environment: attrs[:environment],
-      app_name: name
+      app_name:    name
     )
     problem.errs.create!(attrs.slice(:fingerprint, :problem_id))
   end
@@ -82,7 +85,7 @@ class App
   end
  
   def self.find_by_api_key!(key)
-    find_by(:api_key => key)
+    find_by(api_key: key)
   end
  
   def last_deploy_at
@@ -151,7 +154,7 @@ class App
  
   # Copy app attributes from another app.
   def copy_attributes_from(app_id)
-    if (copy_app = App.where(:_id => app_id).first)
+    if (copy_app = App.where(_id: app_id).first)
       # Copy fields
       (copy_app.fields.keys - %w(_id name created_at updated_at)).each do |k|
         send("#{k}=", copy_app.send(k))
@@ -191,7 +194,7 @@ class App
 protected
  
   def store_cached_attributes_on_problems
-    Problem.where(:app_id => id).update_all(
+    Problem.where(app_id: id).update_all(
       app_name: name
     )
   end
@@ -214,7 +217,7 @@ protected
     github_host = URI.parse(Errbit::Config.github_url).host
     github_host = Regexp.escape(github_host)
     github_repo.strip!
-    github_repo.sub!(/(git@|https?:\/\/)#{github_host}(\/|:)/, '')
+    github_repo.sub!(%r{(git@|https?://)#{github_host}(/|:)}, '')
     github_repo.sub!(/\.git$/, '')
   end
 end
@@ -8,14 +8,14 @@ class Backtrace
   field :fingerprint
   field :lines
  
-  index :fingerprint => 1
+  index fingerprint: 1
  
   def self.find_or_create(lines)
     fingerprint = generate_fingerprint(lines)
  
     where(fingerprint: fingerprint).find_one_and_update(
       { '$setOnInsert' => { fingerprint: fingerprint, lines: lines } },
-      { return_document: :after, upsert: true })
+      return_document: :after, upsert: true)
   end
  
   def self.generate_fingerprint(lines)
@@ -5,14 +5,14 @@ class Comment
   after_create :increase_counter_cache
   before_destroy :decrease_counter_cache
  
-  after_create :deliver_email, :if => :emailable?
+  after_create :deliver_email, if: :emailable?
  
-  field :body, :type => String
-  index(:user_id => 1)
+  field :body, type: String
+  index(user_id: 1)
  
-  belongs_to :err, :class_name => "Problem"
+  belongs_to :err, class_name: "Problem"
   belongs_to :user
-  delegate   :app, :to => :err
+  delegate   :app, to: :err
  
   validates_presence_of :body
  
@@ -8,18 +8,18 @@ class Deploy
   field :revision
   field :message
  
-  index(:created_at => -1)
+  index(created_at: -1)
  
-  embedded_in :app, :inverse_of => :deploys
+  embedded_in :app, inverse_of: :deploys
  
-  after_create :resolve_app_errs, :if => :should_resolve_app_errs?
+  after_create :resolve_app_errs, if: :should_resolve_app_errs?
   after_create :store_cached_attributes_on_problems
   after_create :deliver_email
  
   validates_presence_of :username, :environment
  
   def resolve_app_errs
-    app.problems.unresolved.in_env(environment).each {|problem| problem.resolve!}
+    app.problems.unresolved.in_env(environment).each(&:resolve!)
   end
  
   def short_revision
@@ -33,7 +33,7 @@ protected
   end
  
   def store_cached_attributes_on_problems
-    Problem.where(:app_id => app.id).update_all(
+    Problem.where(app_id: app.id).update_all(
       last_deploy_at: created_at
     )
   end
@@ -12,9 +12,9 @@ class Err
   index fingerprint: 1
  
   belongs_to :problem
-  has_many :notices, :inverse_of => :err, :dependent => :destroy
+  has_many :notices, inverse_of: :err, dependent: :destroy
  
   validates_presence_of :problem_id, :fingerprint
  
-  delegate :app, :resolved?, :to => :problem
+  delegate :app, :resolved?, to: :problem
 end
@@ -61,15 +61,15 @@ class ErrorReport
  
   def make_notice
     @notice = Notice.new(
-      app: app,
-      message: message,
-      error_class: error_class,
-      backtrace: backtrace,
-      request: request,
+      app:                app,
+      message:            message,
+      error_class:        error_class,
+      backtrace:          backtrace,
+      request:            request,
       server_environment: server_environment,
-      notifier: notifier,
-      user_attributes: user_attributes,
-      framework: framework
+      notifier:           notifier,
+      user_attributes:    user_attributes,
+      framework:          framework
     )
   end
  
@@ -14,7 +14,7 @@ class Issue
     if tracker.respond_to?(:render_body_args)
       tracker.render_body_args
     else
-      [ 'issue_trackers/issue', formats: [:md] ]
+      ['issue_trackers/issue', formats: [:md]]
     end
   end
  
@@ -2,10 +2,10 @@ class IssueTracker
   include Mongoid::Document
   include Mongoid::Timestamps
  
-  embedded_in :app, :inverse_of => :issue_tracker
+  embedded_in :app, inverse_of: :issue_tracker
  
-  field :type_tracker, :type => String
-  field :options, :type => Hash, :default => {}
+  field :type_tracker, type: String
+  field :options, type: Hash, default: {}
  
   validate :validate_tracker
  
@@ -15,7 +15,7 @@ class IssueTracker
         klass = ErrbitPlugin::Registry.issue_trackers[type_tracker] || ErrbitPlugin::NoneIssueTracker
         # TODO: we need to find out a better way to pass those config to the issue tracker
         klass.new(options.merge(
-          github_repo: app.try(:github_repo),
+          github_repo:    app.try(:github_repo),
           bitbucket_repo: app.try(:bitbucket_repo)
         ))
       end
@@ -32,7 +32,7 @@ class IssueTracker
     end
   end
  
-  delegate :configured?, :to => :tracker
-  delegate :create_issue, :to => :tracker
-  delegate :url, :to => :tracker
+  delegate :configured?, to: :tracker
+  delegate :create_issue, to: :tracker
+  delegate :url, to: :tracker
 end
@@ -5,30 +5,30 @@ class Notice
   include Mongoid::Timestamps
  
   field :message
-  field :server_environment, :type => Hash
-  field :request, :type => Hash
-  field :notifier, :type => Hash
-  field :user_attributes, :type => Hash
+  field :server_environment, type: Hash
+  field :request, type: Hash
+  field :notifier, type: Hash
+  field :user_attributes, type: Hash
   field :framework
   field :error_class
-  delegate :lines, :to => :backtrace, :prefix => true
-  delegate :problem, :to => :err
+  delegate :lines, to: :backtrace, prefix: true
+  delegate :problem, to: :err
  
   belongs_to :app
   belongs_to :err
-  belongs_to :backtrace, :index => true
+  belongs_to :backtrace, index: true
  
-  index(:created_at => 1)
-  index(:err_id => 1, :created_at => 1, :_id => 1)
+  index(created_at: 1)
+  index(err_id: 1, created_at: 1, _id: 1)
  
   before_save :sanitize
   before_destroy :problem_recache
  
   validates_presence_of :backtrace_id, :server_environment, :notifier
  
-  scope :ordered, ->{ order_by(:created_at.asc) }
-  scope :reverse_ordered, ->{ order_by(:created_at.desc) }
-  scope :for_errs, Proc.new { |errs|
+  scope :ordered, -> { order_by(:created_at.asc) }
+  scope :reverse_ordered, -> { order_by(:created_at.desc) }
+  scope :for_errs, lambda { |errs|
     where(:err_id.in => errs.all.map(&:id))
   }
  
@@ -14,7 +14,7 @@ class NoticeFingerprinter
   embedded_in :site_config
  
   def generate(api_key, notice, backtrace)
-    material = [ api_key ]
+    material = [api_key]
     material << notice.error_class if error_class
     material << notice.filtered_message if message
     material << notice.component if component
@@ -27,6 +27,6 @@ class NoticeFingerprinter
       material << backtrace.lines.slice(0, backtrace_lines)
     end
  
-    Digest::MD5.hexdigest(material.reduce('') { |c, m| c << m.to_s; c })
+    Digest::MD5.hexdigest(material.map(&:to_s).join)
   end
 end
@@ -5,26 +5,26 @@ class NotificationService
   default_url_options[:host] = ActionMailer::Base.default_url_options[:host]
   default_url_options[:port] = ActionMailer::Base.default_url_options[:port]
  
-  field :room_id, :type => String
-  field :user_id, :type => String
-  field :service_url, :type => String
-  field :service, :type => String
-  field :api_token, :type => String
-  field :subdomain, :type => String
-  field :sender_name, :type => String
-  field :notify_at_notices, :type => Array, :default => Errbit::Config.notify_at_notices
-  embedded_in :app, :inverse_of => :notification_service
+  field :room_id, type: String
+  field :user_id, type: String
+  field :service_url, type: String
+  field :service, type: String
+  field :api_token, type: String
+  field :subdomain, type: String
+  field :sender_name, type: String
+  field :notify_at_notices, type: Array, default: Errbit::Config.notify_at_notices
+  embedded_in :app, inverse_of: :notification_service
  
   validate :check_params
  
   if Errbit::Config.per_app_notify_at_notices
-    Fields = [[:notify_at_notices, 
-               { :placeholder => 'comma separated numbers or simply 0 for every notice',
-                 :label => 'notify on errors (0 for all errors)'
+    FIELDS = [[:notify_at_notices,
+               { placeholder: 'comma separated numbers or simply 0 for every notice',
+                 label:       'notify on errors (0 for all errors)'
                }
               ]]
   else
-    Fields = []
+    FIELDS = []
   end
  
   def notify_at_notices
@@ -32,22 +32,36 @@ class NotificationService
   end
  
   # Subclasses are responsible for overwriting this method.
-  def check_params; true; end
+  def check_params
+    true
+  end
  
   def notification_description(problem)
     "[#{problem.environment}][#{problem.where}] #{problem.message.to_s.truncate(100)}"
   end
  
   # Allows us to set the issue tracker class from a single form.
-  def type; _type; end
-  def type=(t); self._type = t; end
+  def type
+    _type
+  end
+
+  def type=(t)
+    self._type = t
+  end
  
-  def url; nil; end
+  def url
+    nil
+  end
  
   # Retrieve tracker label from either class or instance.
-  Label = ''
-  def self.label; self::Label; end
-  def label; self.class.label; end
+  LABEL = ''
+  def self.label
+    self::LABEL
+  end
+
+  def label
+    self.class.label
+  end
  
   def configured?
     api_token.present?
 if defined? Campy
   class NotificationServices::CampfireService < NotificationService
-    Label = "campfire"
-    Fields += [
+    LABEL = "campfire"
+    FIELDS += [
       [:subdomain, {
-        :label       => "Subdomain",
-        :placeholder => "subdomain from http://{{subdomain}}.campfirenow.com"
+        label:       "Subdomain",
+        placeholder: "subdomain from http://{{subdomain}}.campfirenow.com"
       }],
       [:api_token, {
-        :label       => "API Token",
-        :placeholder => "123456789abcdef123456789abcdef"
+        label:       "API Token",
+        placeholder: "123456789abcdef123456789abcdef"
       }],
       [:room_id, {
-        :label       => "Room ID",
-        :placeholder => "123456"
+        label:       "Room ID",
+        placeholder: "123456"
       }]
     ]
  
     def check_params
-      if Fields.detect {|f| self[f[0]].blank? }
+      if FIELDS.detect { |f| self[f[0]].blank? }
         errors.add :base, 'You must specify your Campfire Subdomain, API token and Room ID'
       end
     end
@@ -28,7 +28,7 @@ if defined? Campy
  
     def create_notification(problem)
       # build the campfire client
-      campy = Campy::Room.new(:account => subdomain, :token => api_token, :room_id => room_id)
+      campy = Campy::Room.new(account: subdomain, token: api_token, room_id: room_id)
       # post the issue to the campfire room
       campy.speak "[errbit] #{problem.app.name} #{notification_description problem} - #{Errbit::Config.protocol}://#{Errbit::Config.host}/apps/#{problem.app.id}/problems/#{problem.id}"
     end
 if defined? Flowdock
   class NotificationServices::FlowdockService < NotificationService
-    Label = 'flowdock'
-    Fields += [
+    LABEL = 'flowdock'
+    FIELDS += [
       [
         :api_token, {
-          :label       => 'Flow API Token',
-          :placeholder => '123456789abcdef123456789abcdefgh'
+          label:       'Flow API Token',
+          placeholder: '123456789abcdef123456789abcdefgh'
         }
       ]
     ]
  
     def check_params
-      if Fields.any? { |f, _| self[f].blank? }
+      if FIELDS.any? { |f, _| self[f].blank? }
         errors.add :base, 'You must specify your Flowdock(Flow) API token'
       end
     end
@@ -21,10 +21,22 @@ if defined? Flowdock
     end
  
     def create_notification(problem)
-      flow = Flowdock::Flow.new(:api_token => api_token, :source => "Errbit", :from => {:name => "Errbit", :address => ENV['ERRBIT_EMAIL_FROM'] || 'support@flowdock.com'})
+      flow = Flowdock::Flow.new(
+        api_token: api_token,
+        source:    "Errbit",
+        from:      {
+          name:    "Errbit",
+          address: ENV['ERRBIT_EMAIL_FROM'] || 'support@flowdock.com'
+        }
+      )
       subject = "[#{problem.environment}] #{problem.message.to_s.truncate(100)}"
       url = app_problem_url problem.app, problem
-      flow.push_to_team_inbox(:subject => subject, :content => content(problem, url), :project => project_name(problem), :link => url)
+      flow.push_to_team_inbox(
+        subject: subject,
+        content: content(problem, url),
+        project: project_name(problem),
+        link:    url
+      )
     end
  
   private
 class NotificationServices::GtalkService < NotificationService
-  Label = "gtalk"
-  Fields += [
+  LABEL = "gtalk"
+  FIELDS += [
     [:subdomain, {
-      :placeholder => "username@example.com",
-      :label       => "Username"
+      placeholder: "username@example.com",
+      label:       "Username"
     }],
     [:api_token, {
-      :placeholder => "password",
-      :label       => "Password"
+      placeholder: "password",
+      label:       "Password"
     }],
     [:user_id, {
-      :placeholder => "touser@example.com, anotheruser@example.com",
-      :label => "Send To User(s)"
+      placeholder: "touser@example.com, anotheruser@example.com",
+      label:       "Send To User(s)"
     }, :room_id],
     [:room_id, {
-      :placeholder => "toroom@conference.example.com",
-      :label       =>  "Send To Room (one only)"
+      placeholder: "toroom@conference.example.com",
+      label:       "Send To Room (one only)"
     }, :user_id],
-    [ :service, {
-      :placeholder => "talk.google.com",
-      :label => "Jabber Service"
+    [:service, {
+      placeholder: "talk.google.com",
+      label:       "Jabber Service"
     }],
-    [ :service_url, {
-      :placeholder => "http://www.google.com/talk/",
-      :label => "Link To Jabber Service"
+    [:service_url, {
+      placeholder: "http://www.google.com/talk/",
+      label:       "Link To Jabber Service"
     }]
   ]
  
   def check_params
-    if Fields.detect { |f| self[f[0]].blank? && self[f[2]].blank? } 
-      errors.add :base, 
-        """You must specify your Username, Password, service, service_url 
+    if FIELDS.detect { |f| self[f[0]].blank? && self[f[2]].blank? }
+      errors.add :base,
+        """You must specify your Username, Password, service, service_url
            and either rooms or users to send to or both"""
     end
   end
@@ -58,7 +58,7 @@ class NotificationServices::GtalkService &lt; NotificationService
   end
  
 private
- 
+
   def send_to_users(client, message)
     user_id.tr(' ', ",").tr(';', ",").split(",").map(&:strip).reject(&:empty?).each do |user|
       client.send(Jabber::Message.new(user, message))
@@ -66,9 +66,9 @@ private
   end
  
   def send_to_muc(client, message)
-    #TODO: set this so that it can send to multiple rooms like users, nb multiple room joins in one send fail randomly so leave as one room for the moment 
+    #TODO: set this so that it can send to multiple rooms like users, nb multiple room joins in one send fail randomly so leave as one room for the moment
     muc = Jabber::MUC::SimpleMUCClient.new(client)
-    muc.join(room_id + "/errbit") 
+    muc.join(room_id + "/errbit")
     muc.send(Jabber::Message.new(room_id, message))
   end
 end
 if defined? HipChat
   class NotificationServices::HipchatService < NotificationService
-    Label = 'hipchat'
-    Fields += [
+    LABEL = 'hipchat'
+    FIELDS += [
       [:service, {
-        :placeholder => "'v1' (admin API token) or 'v2' (account API token)",
-        :label       => "HipChat API version"
+        placeholder: "'v1' (admin API token) or 'v2' (account API token)",
+        label:       "HipChat API version"
       }],
       [:service_url, {
-        :placeholder => "Optional, leave empty for HipChat.com",
-        :label       => "Custom HipChat Server URL"
+        placeholder: "Optional, leave empty for HipChat.com",
+        label:       "Custom HipChat Server URL"
       }],
       [:api_token, {
-        :placeholder => "API token",
-        :label       => "API token"
+        placeholder: "API token",
+        label:       "API token"
       }],
       [:room_id, {
-        :placeholder => "Room name",
-        :label       => "Room name"
-      }],
+        placeholder: "Room name",
+        label:       "Room name"
+      }]
     ]
-    Mandatory_fields = [:service, :api_token, :room_id]
-    API_versions = %w(v1 v2)
+    MANDATORY_FIELDS = [:service, :api_token, :room_id]
+    API_VERSIONS = %w(v1 v2)
  
     def check_params
-      Fields.each do |field, hash|
-        if Mandatory_fields.include?(field) && self[field].blank?
+      FIELDS.each do |field, hash|
+        if MANDATORY_FIELDS.include?(field) && self[field].blank?
           errors.add field, "You must specify #{hash[:label]}"
         end
       end
-      unless API_versions.include?(self[:service])
-        errors.add :service, "API version must be #{API_versions.join(' or ')}"
+      unless API_VERSIONS.include?(self[:service])
+        errors.add :service, "API version must be #{API_VERSIONS.join(' or ')}"
       end
     end
  
@@ -45,11 +45,11 @@ if defined? HipChat
         &nbsp;&nbsp;Times occurred: #{problem.notices_count}
       MSG
  
-      options = { :api_version => self[:service] }
+      options = { api_version: self[:service] }
       options[:server_url] = self[:service_url] if service_url.present?
  
       client = HipChat::Client.new(api_token, options)
-      client[room_id].send('Errbit', message, :color => 'red', :notify => true)
+      client[room_id].send('Errbit', message, color: 'red', notify: true)
     end
   end
 end
 class NotificationServices::HoiioService < NotificationService
-  Label = "hoiio"
-  Fields += [
+  LABEL = "hoiio"
+  FIELDS += [
     [:api_token, {
-      :placeholder => "App ID",
-      :label => "App ID"
+      placeholder: "App ID",
+      label:       "App ID"
     }],
     [:subdomain, {
-      :placeholder => "Access Token",
-      :label => "Access Token"
+      placeholder: "Access Token",
+      label:       "Access Token"
     }],
     [:room_id, {
-      :placeholder => "+6511111111, +6511111111",
-      :label       => "Recipient's phone numbers seperated by comma. Phone numbers should start with a \"+\" and country code."
+      placeholder: "+6511111111, +6511111111",
+      label:       "Recipient's phone numbers seperated by comma. Phone numbers should start with a \"+\" and country code."
     }]
   ]
  
   def check_params
-    if Fields.detect {|f| self[f[0]].blank? }
+    if FIELDS.detect { |f| self[f[0]].blank? }
       errors.add :base, 'You must specify your App ID, Access Token and Recipient\'s phone numbers'
     end
   end
@@ -35,7 +35,7 @@ class NotificationServices::HoiioService &lt; NotificationService
  
     # send sms
     room_id.split(',').each do |number|
-      sms.send :dest => number, :msg => "#{Errbit::Config.protocol}://#{Errbit::Config.host}/apps/#{problem.app.id} #{notification_description problem}"
+      sms.send dest: number, msg: "#{Errbit::Config.protocol}://#{Errbit::Config.host}/apps/#{problem.app.id} #{notification_description problem}"
     end
   end
 end
 class NotificationServices::HubotService < NotificationService
-  Label = "hubot"
-  Fields += [
+  LABEL = "hubot"
+  FIELDS += [
     [:api_token, {
-      :placeholder => 'http://hubot.example.org:8080/hubot/say',
-      :label => 'Hubot URL'
+      placeholder: 'http://hubot.example.org:8080/hubot/say',
+      label:       'Hubot URL'
     }],
     [:room_id, {
-      :placeholder => '#dev',
-      :label => 'Room where Hubot should notify'
+      placeholder: '#dev',
+      label:       'Room where Hubot should notify'
     }]
   ]
  
   def check_params
-    if Fields.detect {|f| self[f[0]].blank? }
+    if FIELDS.detect { |f| self[f[0]].blank? }
       errors.add :base, 'You must specify the URL of your hubot'
     end
   end
@@ -26,6 +26,6 @@ class NotificationServices::HubotService &lt; NotificationService
   end
  
   def create_notification(problem)
-    HTTParty.post(url, :body => {:message => message_for_hubot(problem), :room => room_id})
+    HTTParty.post(url, body: { message: message_for_hubot(problem), room: room_id })
   end
 end
 class NotificationServices::PushoverService < NotificationService
-  Label = "pushover"
-  Fields += [
+  LABEL = "pushover"
+  FIELDS += [
     [:api_token, {
-      :placeholder => "User Key",
-      :label => "User Key"
+      placeholder: "User Key",
+      label:       "User Key"
     }],
     [:subdomain, {
-      :placeholder => "Application API Token",
-      :label => "Application API Token"
+      placeholder: "Application API Token",
+      label:       "Application API Token"
     }]
   ]
  
   def check_params
-    if Fields.detect {|f| self[f[0]].blank? }
+    if FIELDS.detect { |f| self[f[0]].blank? }
       errors.add :base, 'You must specify your User Key and Application API Token.'
     end
   end
@@ -26,6 +26,6 @@ class NotificationServices::PushoverService &lt; NotificationService
     notification = Rushover::Client.new(subdomain)
  
     # send push notification to pushover
-    notification.notify(api_token, "#{notification_description problem}", :priority => 1, :title => "Errbit Notification", :url => "#{Errbit::Config.protocol}://#{Errbit::Config.host}/apps/#{problem.app.id}", :url_title => "Link to error")
+    notification.notify(api_token, "#{notification_description problem}", priority: 1, title: "Errbit Notification", url: "#{Errbit::Config.protocol}://#{Errbit::Config.host}/apps/#{problem.app.id}", url_title: "Link to error")
   end
 end
 class NotificationServices::SlackService < NotificationService
-  Label = "slack"
-  Fields += [
+  LABEL = "slack"
+  FIELDS += [
     [:service_url, {
-      :placeholder => 'Slack Hook URL (https://hooks.slack.com/services/XXXXXXXXX/XXXXXXXXX/XXXXXXXXX)',
-      :label => 'Hook URL'
+      placeholder: 'Slack Hook URL (https://hooks.slack.com/services/XXXXXXXXX/XXXXXXXXX/XXXXXXXXX)',
+      label:       'Hook URL'
     }]
   ]
  
   def check_params
-    if Fields.detect {|f| self[f[0]].blank? }
+    if FIELDS.detect { |f| self[f[0]].blank? }
       errors.add :base, "You must specify your Slack Hook url."
     end
   end
@@ -19,41 +19,41 @@ class NotificationServices::SlackService &lt; NotificationService
  
   def post_payload(problem)
     {
-      :attachments => [
+      attachments: [
         {
-          :fallback => message_for_slack(problem),
-          :pretext => "<#{problem_url(problem)}|Errbit - #{problem.app.name}: #{problem.error_class}>",
-          :color => "#D00000",
-          :fields => [
+          fallback: message_for_slack(problem),
+          pretext:  "<#{problem_url(problem)}|Errbit - #{problem.app.name}: #{problem.error_class}>",
+          color:    "#D00000",
+          fields:   [
             {
-              :title => "Environment",
-              :value => problem.environment,
-              :short => false
+              title: "Environment",
+              value: problem.environment,
+              short: false
             },
             {
-              :title => "Location",
-              :value => problem.where,
-              :short => false
+              title: "Location",
+              value: problem.where,
+              short: false
             },
             {
-              :title => "Message",
-              :value => problem.message.to_s,
-              :short => false
+              title: "Message",
+              value: problem.message.to_s,
+              short: false
             },
             {
-              :title => "First Noticed",
-              :value => problem.first_notice_at,
-              :short => false
+              title: "First Noticed",
+              value: problem.first_notice_at,
+              short: false
             },
             {
-              :title => "Last Noticed",
-              :value => problem.last_notice_at,
-              :short => false
+              title: "Last Noticed",
+              value: problem.last_notice_at,
+              short: false
             },
             {
-              :title => "Times Occurred",
-              :value => problem.notices_count,
-              :short => false
+              title: "Times Occurred",
+              value: problem.notices_count,
+              short: false
             }
           ]
         }
@@ -62,7 +62,13 @@ class NotificationServices::SlackService &lt; NotificationService
   end
  
   def create_notification(problem)
-    HTTParty.post(service_url, :body => post_payload(problem), :headers => { 'Content-Type' => 'application/json' })
+    HTTParty.post(
+      service_url,
+      body:    post_payload(problem),
+      headers: {
+        'Content-Type' => 'application/json'
+      }
+    )
   end
  
   def configured?
 class NotificationServices::WebhookService < NotificationService
-  Label = "webhook"
-  Fields = [
+  LABEL = "webhook"
+  FIELDS = [
     [:api_token, {
-      :placeholder => 'URL to receive a POST request when an error occurs',
-      :label => 'URL'
+      placeholder: 'URL to receive a POST request when an error occurs',
+      label:       'URL'
     }]
   ]
  
   def check_params
-    if Fields.detect {|f| self[f[0]].blank? }
+    if FIELDS.detect { |f| self[f[0]].blank? }
       errors.add :base, 'You must specify the URL'
     end
   end
  
   def message_for_webhook(problem)
-    {:problem => {:url => problem_url(problem)}.merge(problem.as_json).to_json}
+    {
+      problem: {
+        url: problem_url(problem)
+      }.merge(problem.as_json).to_json
+    }
   end
  
   def create_notification(problem)
-    HTTParty.post(api_token, :body => message_for_webhook(problem))
+    HTTParty.post(api_token, body: message_for_webhook(problem))
   end
 end
@@ -7,61 +7,61 @@ class Problem
   include Mongoid::Timestamps
  
   CACHED_NOTICE_ATTRIBUTES = {
-    messages: :message,
-    hosts: :host,
+    messages:    :message,
+    hosts:       :host,
     user_agents: :user_agent_string
   }.freeze
  
-  field :last_notice_at, :type => ActiveSupport::TimeWithZone, :default => Proc.new { Time.zone.now }
-  field :first_notice_at, :type => ActiveSupport::TimeWithZone, :default => Proc.new { Time.zone.now }
-  field :last_deploy_at, :type => Time
-  field :resolved, :type => Boolean, :default => false
-  field :resolved_at, :type => Time
-  field :issue_link, :type => String
-  field :issue_type, :type => String
+  field :last_notice_at, type: ActiveSupport::TimeWithZone, default: proc { Time.zone.now }
+  field :first_notice_at, type: ActiveSupport::TimeWithZone, default: proc { Time.zone.now }
+  field :last_deploy_at, type: Time
+  field :resolved, type: Boolean, default: false
+  field :resolved_at, type: Time
+  field :issue_link, type: String
+  field :issue_type, type: String
  
   # Cached fields
-  field :app_name, :type => String
-  field :notices_count, :type => Integer, :default => 0
+  field :app_name, type: String
+  field :notices_count, type: Integer, default: 0
   field :message
   field :environment
   field :error_class
   field :where
-  field :user_agents, :type => Hash, :default => {}
-  field :messages,    :type => Hash, :default => {}
-  field :hosts,       :type => Hash, :default => {}
-  field :comments_count, :type => Integer, :default => 0
-
-  index :app_id => 1
-  index :app_name => 1
-  index :message => 1
-  index :last_notice_at => 1
-  index :first_notice_at => 1
-  index :last_deploy_at => 1
-  index :resolved_at => 1
-  index :notices_count => 1
+  field :user_agents, type: Hash, default: {}
+  field :messages,    type: Hash, default: {}
+  field :hosts,       type: Hash, default: {}
+  field :comments_count, type: Integer, default: 0
+
+  index app_id: 1
+  index app_name: 1
+  index message: 1
+  index last_notice_at: 1
+  index first_notice_at: 1
+  index last_deploy_at: 1
+  index resolved_at: 1
+  index notices_count: 1
  
   index({
     error_class: "text",
-    where: "text",
-    message: "text",
-    app_name: "text",
+    where:       "text",
+    message:     "text",
+    app_name:    "text",
     environment: "text"
   }, default_language: "english")
  
   belongs_to :app
-  has_many :errs, :inverse_of => :problem, :dependent => :destroy
-  has_many :comments, :inverse_of => :err, :dependent => :destroy
+  has_many :errs, inverse_of: :problem, dependent: :destroy
+  has_many :comments, inverse_of: :err, dependent: :destroy
  
   validates_presence_of :environment
  
   before_create :cache_app_attributes
   before_save :truncate_message
  
-  scope :resolved, ->{ where(:resolved => true) }
-  scope :unresolved, ->{ where(:resolved => false) }
-  scope :ordered, ->{ order_by(:last_notice_at.desc) }
-  scope :for_apps, lambda {|apps| where(:app_id.in => apps.all.map(&:id))}
+  scope :resolved, -> { where(resolved: true) }
+  scope :unresolved, -> { where(resolved: false) }
+  scope :ordered, -> { order_by(:last_notice_at.desc) }
+  scope :for_apps, ->(apps) { where(:app_id.in => apps.all.map(&:id)) }
  
   validates_presence_of :last_notice_at, :first_notice_at
  
@@ -69,12 +69,12 @@ class Problem
     if fetch_all
       all
     else
-      where(:resolved => false)
+      where(resolved: false)
     end
   end
  
   def self.in_env(env)
-    env.present? ? where(:environment => env) : scoped
+    env.present? ? where(environment: env) : scoped
   end
  
   def self.cache_notice(id, notice)
@@ -85,22 +85,22 @@ class Problem
  
     Problem.where('_id' => id).find_one_and_update({
       '$set' => {
-        'environment' => notice.environment_name,
-        'error_class' => notice.error_class,
-        'last_notice_at' => notice.created_at.utc,
-        'message' => notice.message,
-        'resolved' => false,
-        'resolved_at' => nil,
-        'where' => notice.where,
-        "messages.#{message_digest}.value" => notice.message,
-        "hosts.#{host_digest}.value" => notice.host,
-        "user_agents.#{user_agent_digest}.value" => notice.user_agent_string,
+        'environment'                            => notice.environment_name,
+        'error_class'                            => notice.error_class,
+        'last_notice_at'                         => notice.created_at.utc,
+        'message'                                => notice.message,
+        'resolved'                               => false,
+        'resolved_at'                            => nil,
+        'where'                                  => notice.where,
+        "messages.#{message_digest}.value"       => notice.message,
+        "hosts.#{host_digest}.value"             => notice.host,
+        "user_agents.#{user_agent_digest}.value" => notice.user_agent_string
       },
       '$inc' => {
-        'notices_count' => 1,
-        "messages.#{message_digest}.count" => 1,
-        "hosts.#{host_digest}.count" => 1,
-        "user_agents.#{user_agent_digest}.count" => 1,
+        'notices_count'                          => 1,
+        "messages.#{message_digest}.count"       => 1,
+        "hosts.#{host_digest}.count"             => 1,
+        "user_agents.#{user_agent_digest}.count" => 1
       }
     }, return_document: :after)
   end
@@ -110,12 +110,12 @@ class Problem
  
     atomically do |doc|
       doc.set(
-        'environment' => last_notice.environment_name,
-        'error_class' => last_notice.error_class,
+        'environment'    => last_notice.environment_name,
+        'error_class'    => last_notice.error_class,
         'last_notice_at' => last_notice.created_at,
-        'message' => last_notice.message,
-        'where' => last_notice.where,
-        'notices_count' => notices_count.to_i > 1 ? notices_count - 1 : 0
+        'message'        => last_notice.message,
+        'where'          => last_notice.where,
+        'notices_count'  => notices_count.to_i > 1 ? notices_count - 1 : 0
       )
  
       CACHED_NOTICE_ATTRIBUTES.each do |k, v|
@@ -139,7 +139,7 @@ class Problem
       # find only notices related to this problem
       Notice.collection.find.aggregate([
         { "$match" => { err_id: { "$in" => err_ids } } },
-        { "$group" => { _id: "$#{v}", count: {"$sum" => 1} } }
+        { "$group" => { _id: "$#{v}", count: { "$sum" => 1 } } }
       ]).each do |agg|
         send(k)[Digest::MD5.hexdigest(agg[:_id] || 'N/A')] = {
           'value' => agg[:_id] || 'N/A',
@@ -164,8 +164,8 @@ class Problem
     Rails.application.routes.url_helpers.app_problem_url(
       app,
       self,
-      :host => Errbit::Config.host,
-      :port => Errbit::Config.port
+      host: Errbit::Config.host,
+      port: Errbit::Config.port
     )
   end
  
@@ -174,11 +174,11 @@ class Problem
   end
  
   def resolve!
-    self.update_attributes!(:resolved => true, :resolved_at => Time.zone.now)
+    self.update_attributes!(resolved: true, resolved_at: Time.zone.now)
   end
  
   def unresolve!
-    self.update_attributes!(:resolved => false, :resolved_at => nil)
+    self.update_attributes!(resolved: false, resolved_at: nil)
   end
  
   def unresolved?
@@ -194,7 +194,7 @@ class Problem
   end
  
   def unmerge!
-    attrs = {:error_class => error_class, :environment => environment}
+    attrs = { error_class: error_class, environment: environment }
     problem_errs = errs.to_a
  
     # associate and return all the problems
@@ -214,12 +214,12 @@ class Problem
  
   def self.ordered_by(sort, order)
     case sort
-    when "app" then            order_by(["app_name", order])
-    when "message" then        order_by(["message", order])
+    when "app"            then order_by(["app_name", order])
+    when "message"        then order_by(["message", order])
     when "last_notice_at" then order_by(["last_notice_at", order])
     when "last_deploy_at" then order_by(["last_deploy_at", order])
-    when "count" then          order_by(["notices_count", order])
-    else raise("\"#{sort}\" is not a recognized sort")
+    when "count"          then order_by(["notices_count", order])
+    else fail("\"#{sort}\" is not a recognized sort")
     end
   end
  
@@ -241,7 +241,7 @@ class Problem
   end
  
   def self.search(value)
-    Problem.where({'$text' => {'$search' => value}})
+    Problem.where('$text' => { '$search' => value })
   end
  
 private
@@ -9,37 +9,37 @@ class User
   field :github_login
   field :github_oauth_token
   field :name
-  field :admin, :type => Boolean, :default => false
-  field :per_page, :type => Fixnum, :default => PER_PAGE
-  field :time_zone, :default => "UTC"
+  field :admin, type: Boolean, default: false
+  field :per_page, type: Fixnum, default: PER_PAGE
+  field :time_zone, default: "UTC"
  
   ## Devise field
   ### Database Authenticatable
-  field :encrypted_password, :type => String
+  field :encrypted_password, type: String
  
   ### Recoverable
-  field :reset_password_token, :type => String
-  field :reset_password_sent_at, :type => Time
+  field :reset_password_token, type: String
+  field :reset_password_sent_at, type: Time
  
   ### Rememberable
-  field :remember_created_at, :type => Time
+  field :remember_created_at, type: Time
  
   ### Trackable
-  field :sign_in_count,      :type => Integer
-  field :current_sign_in_at, :type => Time
-  field :last_sign_in_at,    :type => Time
-  field :current_sign_in_ip, :type => String
-  field :last_sign_in_ip,    :type => String
+  field :sign_in_count,      type: Integer
+  field :current_sign_in_at, type: Time
+  field :last_sign_in_at,    type: Time
+  field :current_sign_in_ip, type: String
+  field :last_sign_in_ip,    type: String
  
   ### Token_authenticatable
-  field :authentication_token, :type => String
+  field :authentication_token, type: String
  
-  index :authentication_token => 1
+  index authentication_token: 1
  
   before_save :ensure_authentication_token
  
   validates_presence_of :name
-  validates_uniqueness_of :github_login, :allow_nil => true
+  validates_uniqueness_of :github_login, allow_nil: true
  
   if Errbit::Config.user_has_username
     field :username
@@ -4,7 +4,7 @@ class Watcher
  
   field :email
  
-  embedded_in :app, :inverse_of => :watchers
+  embedded_in :app, inverse_of: :watchers
   belongs_to :user
  
   validate :ensure_user_or_email
@@ -14,7 +14,7 @@
       %div.notification_params.none{:class => (w.object && !(w.object.class < NotificationService)) ? 'chosen' : nil}
       - NotificationService.subclasses.each do |notification_service|
         %div.notification_params{:class => (w.object.is_a?(notification_service) ? 'chosen ' : '') << notification_service.label}
-          - notification_service::Fields.each do |field, field_info|
+          - notification_service::FIELDS.each do |field, field_info|
             = w.label field, field_info[:label] || field.to_s.titleize
             - field_type = field == :password ? :password_field : :text_field
             - value = field == :notify_at_notices ? w.object.notify_at_notices.join(", ") : w.object.send(field)
@@ -39,7 +39,7 @@ module Errbit
     config.generators do |g|
       g.orm             :mongoid
       g.template_engine :haml
-      g.test_framework  :rspec, :fixture => false
+      g.test_framework  :rspec, fixture: false
       g.fixture_replacement :fabrication
     end
  
@@ -46,7 +46,7 @@ namespace :errbit do
  
       {
         'config/newrelic.example.yml' => 'config/newrelic.yml',
-        'config/unicorn.default.rb' => 'config/unicorn.rb',
+        'config/unicorn.default.rb'   => 'config/unicorn.rb'
       }.each do |src, target|
         unless test("[ -f #{shared_path}/#{target} ]")
           upload! src, "#{shared_path}/#{target}"
@@ -2,12 +2,12 @@
 if Errbit::Config.email_delivery_method == :smtp
   ActionMailer::Base.delivery_method = :smtp
   ActionMailer::Base.smtp_settings = {
-    :address        => Errbit::Config.smtp_address,
-    :port           => Errbit::Config.smtp_port,
-    :authentication => Errbit::Config.smtp_authentication,
-    :user_name      => Errbit::Config.smtp_user_name,
-    :password       => Errbit::Config.smtp_password,
-    :domain         => Errbit::Config.smtp_domain,
+    address:        Errbit::Config.smtp_address,
+    port:           Errbit::Config.smtp_port,
+    authentication: Errbit::Config.smtp_authentication,
+    user_name:      Errbit::Config.smtp_user_name,
+    password:       Errbit::Config.smtp_password,
+    domain:         Errbit::Config.smtp_domain
   }
 end
  
@@ -23,8 +23,8 @@ end
 # Set config specific values
 (ActionMailer::Base.default_url_options ||= {}).tap do |default|
   options_from_config = {
-    host: Errbit::Config.host,
-    port: Errbit::Config.port,
+    host:     Errbit::Config.host,
+    port:     Errbit::Config.port,
     protocol: Errbit::Config.protocol
   }.select { |_k, v| v }
  
 ActionDispatch::ParamsParser::DEFAULT_PARSERS.delete(Mime::YAML)
-ActiveSupport::XmlMini::PARSING.delete("symbol") 
-ActiveSupport::XmlMini::PARSING.delete("yaml") 
+ActiveSupport::XmlMini::PARSING.delete("symbol")
+ActiveSupport::XmlMini::PARSING.delete("yaml")
@@ -29,7 +29,7 @@ Devise.setup do |config|
   # session. If you need permissions, you should implement that in a before filter.
   # You can also supply a hash where the value is a boolean determining whether
   # or not authentication should be aborted when the value is not present.
-  config.authentication_keys = [ Errbit::Config.user_has_username ? :username : :email ]
+  config.authentication_keys = [Errbit::Config.user_has_username ? :username : :email]
  
   # Configure parameters from the request object used for authentication. Each entry
   # given should be a request method and it will automatically be passed to the
@@ -41,12 +41,12 @@ Devise.setup do |config|
   # Configure which authentication keys should be case-insensitive.
   # These keys will be downcased upon creating or modifying a user and when used
   # to authenticate or find a user. Default is :email.
-  config.case_insensitive_keys = [ Errbit::Config.user_has_username ? :username : :email ]
+  config.case_insensitive_keys = [Errbit::Config.user_has_username ? :username : :email]
  
   # Configure which authentication keys should have whitespace stripped.
   # These keys will have whitespace before and after removed upon creating or
   # modifying a user and when used to authenticate or find a user. Default is :email.
-  config.strip_whitespace_keys = [ Errbit::Config.user_has_username ? :username : :email ]
+  config.strip_whitespace_keys = [Errbit::Config.user_has_username ? :username : :email]
  
   # Tell if authentication through request.params is enabled. True by default.
   # It can be set to an array that will enable params authentication only for the
@@ -237,12 +237,12 @@ Devise.setup do |config|
  
   if Errbit::Config.github_authentication || Rails.env.test?
     github_options = {
-      :scope => Errbit::Config.github_access_scope.join(','),
-      :skip_info => true,
-      :client_options => {
-        :site => Errbit::Config.github_api_url,
-        :authorize_url => "#{Errbit::Config.github_url}/login/oauth/authorize",
-        :token_url => "#{Errbit::Config.github_url}/login/oauth/access_token"
+      scope:          Errbit::Config.github_access_scope.join(','),
+      skip_info:      true,
+      client_options: {
+        site:          Errbit::Config.github_api_url,
+        authorize_url: "#{Errbit::Config.github_url}/login/oauth/authorize",
+        token_url:     "#{Errbit::Config.github_url}/login/oauth/access_token"
       }
     }
  
 # Enforce SSL connections, if configured
 if Errbit::Config.enforce_ssl
   require 'rack/ssl-enforcer'
-  ActionMailer::Base.default_url_options.merge!(:protocol => 'https://')
+  ActionMailer::Base.default_url_options.merge!(protocol: 'https://')
   Rails.application.configure do
-    config.middleware.use Rack::SslEnforcer, :except => /^\/deploys/
+    config.middleware.use Rack::SslEnforcer, except: %r{^/deploys}
   end
 end
@@ -7,7 +7,7 @@ require_relative &#39;../lib/configurator&#39;
 #
 # We use the first non-nil environment variable in the list. If the last array
 # element is a proc, it runs at the end, overriding the config value
-Errbit::Config = Configurator.run({
+Errbit::Config = Configurator.run(
   host:                      ['ERRBIT_HOST'],
   protocol:                  ['ERRBIT_PROTOCOL'],
   port:                      ['ERRBIT_PORT'],
@@ -29,8 +29,8 @@ Errbit::Config = Configurator.run({
   mongo_url:                 %w(MONGOLAB_URI MONGOHQ_URL MONGODB_URL MONGO_URL),
  
   # github
-  github_url:                ['GITHUB_URL', ->(values) {
-    values[:github_url].gsub(/\/*\z/, '')
+  github_url:                ['GITHUB_URL', lambda { |values|
+    values[:github_url].gsub(%r{/*\z}, '')
   }],
   github_authentication:     ['GITHUB_AUTHENTICATION'],
   github_client_id:          ['GITHUB_CLIENT_ID'],
@@ -40,7 +40,7 @@ Errbit::Config = Configurator.run({
   github_api_url:            ['GITHUB_API_URL'],
   github_site_title:         ['GITHUB_SITE_TITLE'],
  
-  email_delivery_method:     ['EMAIL_DELIVERY_METHOD', -> (values) {
+  email_delivery_method:     ['EMAIL_DELIVERY_METHOD', lambda { |values|
     values[:email_delivery_method] && values[:email_delivery_method].to_sym
   }],
  
@@ -50,7 +50,7 @@ Errbit::Config = Configurator.run({
   smtp_authentication:       ['SMTP_AUTHENTICATION'],
   smtp_user_name:            %w(SMTP_USERNAME SENDGRID_USERNAME),
   smtp_password:             %w(SMTP_PASSWORD SENDGRID_PASSWORD),
-  smtp_domain:               ['SMTP_DOMAIN', 'SENDGRID_DOMAIN', ->(values) {
+  smtp_domain:               ['SMTP_DOMAIN', 'SENDGRID_DOMAIN', lambda { |values|
     values[:smtp_domain] ||
     (values[:email_from] && values[:email_from].split('@').last) || nil
   }],
@@ -59,5 +59,5 @@ Errbit::Config = Configurator.run({
   sendmail_location:         ['SENDMAIL_LOCATION'],
   sendmail_arguments:        ['SENDMAIL_ARGUMENTS'],
  
-  devise_modules:            ['DEVISE_MODULES'],
-})
+  devise_modules:            ['DEVISE_MODULES']
+)
@@ -10,7 +10,7 @@ Mongoid.configure do |config|
           Errbit::Config.mongo_url
         end
  
-  config.load_configuration({
+  config.load_configuration(
     clients: {
       default: {
         uri: uri
@@ -19,5 +19,5 @@ Mongoid.configure do |config|
     options: {
       use_activesupport_time_zone: true
     }
-  })
+  )
 end
 Rails.application.routes.draw do
-  devise_for :users, :controllers => { :omniauth_callbacks => "users/omniauth_callbacks" }
+  devise_for :users, controllers: { omniauth_callbacks: "users/omniauth_callbacks" }
  
   # Hoptoad Notifier Routes
   match '/notifier_api/v2/notices' => 'notices#create', via: [:get, :post]
   get '/locate/:id' => 'notices#locate', :as => :locate
   post '/deploys.txt' => 'deploys#create'
  
-  resources :notices,   :only => [:show]
-  resources :deploys,   :only => [:show]
+  resources :notices,   only: [:show]
+  resources :deploys,   only: [:show]
   resources :users do
     member do
       delete :unlink_github
     end
   end
  
-  resources :site_config, :only => [:index] do
+  resources :site_config, only: [:index] do
     collection do
       put :update
     end
   end
  
-  resources :problems, :only => [:index] do
+  resources :problems, only: [:index] do
     collection do
       post :destroy_several
       post :resolve_several
@@ -34,7 +34,7 @@ Rails.application.routes.draw do
   resources :apps do
     resources :problems do
       resources :notices
-      resources :comments, :only => [:create, :destroy]
+      resources :comments, only: [:create, :destroy]
  
       collection do
         post :destroy_all
@@ -47,8 +47,8 @@ Rails.application.routes.draw do
         delete :unlink_issue
       end
     end
-    resources :deploys, :only => [:index]
-    resources :watchers, :only => [:destroy, :update]
+    resources :deploys, only: [:index]
+    resources :watchers, only: [:destroy, :update]
     member do
       post :regenerate_api_key
     end
@@ -56,9 +56,9 @@ Rails.application.routes.draw do
  
   namespace :api do
     namespace :v1 do
-      resources :problems, :only => [:index, :show], :defaults => { :format => 'json' }
-      resources :notices,  :only => [:index], :defaults => { :format => 'json' }
-      resources :stats, :only => [], :defaults => { :format => 'json' } do
+      resources :problems, only: [:index, :show], defaults: { format: 'json' }
+      resources :notices,  only: [:index], defaults: { format: 'json' }
+      resources :stats, only: [], defaults: { format: 'json' } do
         collection do
           get :app
         end
@@ -69,5 +69,5 @@ Rails.application.routes.draw do
   match '/api/v3/projects/:project_id/create-notice' => 'api/v3/notices#create', via: [:post]
   match '/api/v3/projects/:project_id/notices' => 'api/v3/notices#create', via: [:post]
  
-  root :to => 'apps#index'
+  root to: 'apps#index'
 end
 class ExtractIssueTracker < Mongoid::Migration
   TRACKER_MAPPING = {
-    'ErrbitTracPlugin::IssueTracker' => 'trac',
+    'ErrbitTracPlugin::IssueTracker'        => 'trac',
     'IssueTrackers::BitbucketIssuesTracker' => 'bitbucket',
-    'IssueTrackers::FogbugzTracker' => 'fogbugz',
-    'IssueTrackers::GithubIssuesTracker' => 'github',
-    'IssueTrackers::GitlabTracker' => 'gitlab',
-    'IssueTrackers::JiraTracker' => 'jira',
-    'IssueTrackers::LighthouseTracker' => 'lighthouse',
-    'IssueTrackers::PivotalLabsTracker' => 'pivotal',
-    'IssueTrackers::RedmineTracker' => 'redmine',
-    'IssueTrackers::UnfuddleTracker' => 'unfuddle'
+    'IssueTrackers::FogbugzTracker'         => 'fogbugz',
+    'IssueTrackers::GithubIssuesTracker'    => 'github',
+    'IssueTrackers::GitlabTracker'          => 'gitlab',
+    'IssueTrackers::JiraTracker'            => 'jira',
+    'IssueTrackers::LighthouseTracker'      => 'lighthouse',
+    'IssueTrackers::PivotalLabsTracker'     => 'pivotal',
+    'IssueTrackers::RedmineTracker'         => 'redmine',
+    'IssueTrackers::UnfuddleTracker'        => 'unfuddle'
   }
  
   def self.up
@@ -29,14 +29,12 @@ class ExtractIssueTracker &lt; Mongoid::Migration
  
       tracker = {
         'type_tracker' => TRACKER_MAPPING[type],
-        'options' => options,
-        'updated_at' => updated_at,
-        'created_at' => created_at
+        'options'      => options,
+        'updated_at'   => updated_at,
+        'created_at'   => created_at
       }
  
-      App.where({ _id: app.id }).update({
-        "$set" => { :issue_tracker => tracker }
-      })
+      App.where(_id: app.id).update("$set" => { issue_tracker: tracker })
     end
   end
  
@@ -15,7 +15,7 @@ puts &quot;-- password: #{admin_pass}&quot;
 puts ""
 puts "Be sure to note down these credentials now!"
  
-user = User.find_or_initialize_by(:email => admin_email)
+user = User.find_or_initialize_by(email: admin_email)
  
 user.name = 'Errbit Admin'
 user.password = admin_pass
@@ -11,14 +11,14 @@ module AirbrakeApi
  
       def report
         attributes = {
-          error_class: error['type'],
-          message: error['message'],
-          backtrace: backtrace,
-          request: request,
+          error_class:        error['type'],
+          message:            error['message'],
+          backtrace:          backtrace,
+          request:            request,
           server_environment: server_environment,
-          api_key: params['key'].present? ? params['key'] : params['project_id'],
-          notifier: params['notifier'],
-          user_attributes: user_attributes
+          api_key:            params['key'].present? ? params['key'] : params['project_id'],
+          notifier:           params['notifier'],
+          user_attributes:    user_attributes
         }
  
         ErrorReport.new(attributes)
@@ -27,7 +27,7 @@ module AirbrakeApi
     private
  
       def error
-        raise AirbrakeApi::ParamsError unless params.key?('errors') && params['errors'].any?
+        fail AirbrakeApi::ParamsError unless params.key?('errors') && params['errors'].any?
         @error ||= params['errors'].first
       end
  
@@ -35,7 +35,7 @@ module AirbrakeApi
         (error['backtrace'] || []).map do |backtrace_line|
           {
             method: backtrace_line['function'],
-            file: backtrace_line['file'],
+            file:   backtrace_line['file'],
             number: backtrace_line['line'],
             column: backtrace_line['column']
           }
@@ -45,9 +45,9 @@ module AirbrakeApi
       def server_environment
         {
           'environment-name' => context['environment'],
-          'hostname' => hostname,
-          'project-root' => context['rootDirectory'],
-          'app-version' => context['version']
+          'hostname'         => hostname,
+          'project-root'     => context['rootDirectory'],
+          'app-version'      => context['version']
         }
       end
  
@@ -57,12 +57,12 @@ module AirbrakeApi
         )
  
         {
-          'cgi-data' => environment,
-          'session' => params['session'],
-          'params' => params['params'],
-          'url' => url,
+          'cgi-data'  => environment,
+          'session'   => params['session'],
+          'params'    => params['params'],
+          'url'       => url,
           'component' => context['component'],
-          'action' => context['action']
+          'action'    => context['action']
         }
       end
  
@@ -9,7 +9,7 @@ module Hoptoad
  
   class << self
     def parse_xml!(xml)
-      parsed = ActiveSupport::XmlMini.backend.parse(xml)['notice'] || raise(ApiVersionError)
+      parsed = ActiveSupport::XmlMini.backend.parse(xml)['notice'] || fail(ApiVersionError)
       processor = get_version_processor(parsed['version'])
       processor.process_notice(parsed)
     end
@@ -17,7 +17,7 @@ module Hoptoad
     private def get_version_processor(version)
       case version
       when /2\.[01234]/ then Hoptoad::V2
-      else; raise ApiVersionError
+      else; fail ApiVersionError
       end
     end
   end
@@ -12,23 +12,23 @@ module Hoptoad
       case node
       when Hash
         if node.key?('var') && node.key?('key')
-          {normalize_key(node['key']) => rekey(node['var'])}
+          { normalize_key(node['key']) => rekey(node['var']) }
         elsif node.key?('var')
           rekey(node['var'])
         elsif node.key?('__content__') && node.key?('key')
-          {normalize_key(node['key']) => rekey(node['__content__'])}
+          { normalize_key(node['key']) => rekey(node['__content__']) }
         elsif node.key?('__content__')
           rekey(node['__content__'])
         elsif node.key?('key')
-          {normalize_key(node['key']) => nil}
+          { normalize_key(node['key']) => nil }
         else
-          node.inject({}) {|rekeyed, (key, val)| rekeyed.merge(normalize_key(key) => rekey(val))}
+          node.inject({}) { |rekeyed, (key, val)| rekeyed.merge(normalize_key(key) => rekey(val)) }
         end
       when Array
         if node.first.key?('key')
-          node.inject({}) {|rekeyed, keypair| rekeyed.merge(rekey(keypair))}
+          node.inject({}) { |rekeyed, keypair| rekeyed.merge(rekey(keypair)) }
         else
-          node.map {|n| rekey(n)}
+          node.map { |n| rekey(n) }
         end
       else
         node
@@ -53,18 +53,18 @@ module Hoptoad
  
     def self.for_errbit_api(notice)
       {
-        :error_class        => notice['error']['class'] || notice['error']['key'],
-        :message            => notice['error']['message'],
-        :backtrace          => notice['error']['backtrace']['line'],
+        error_class:        notice['error']['class'] || notice['error']['key'],
+        message:            notice['error']['message'],
+        backtrace:          notice['error']['backtrace']['line'],
  
-        :request            => notice['request'],
-        :server_environment => notice['server-environment'],
+        request:            notice['request'],
+        server_environment: notice['server-environment'],
  
-        :api_key            => notice['api-key'],
-        :notifier           => notice['notifier'],
+        api_key:            notice['api-key'],
+        notifier:           notice['notifier'],
         # 'current-user' from airbrake, 'user-attributes' from airbrake_user_attributes gem
-        :user_attributes    => notice['current-user'] || notice['user-attributes'] || {},
-        :framework          => notice['framework']
+        user_attributes:    notice['current-user'] || notice['user-attributes'] || {},
+        framework:          notice['framework']
       }
     end
   end
@@ -7,7 +7,7 @@ HoptoadNotifier.module_eval do
     private def send_notice(notice)
       # Log the error internally if we are not in a development environment.
       if configuration.public?
-        app = App.find_or_initialize_by(:name => "Self.Errbit")
+        app = App.find_or_initialize_by(name: "Self.Errbit")
         app.github_repo = "errbit/errbit"
         app.save!
         notice.send("api_key=", app.api_key)
 namespace :errbit do
   desc "Updates cached attributes on Problem"
-  task :problem_recache => :environment do
+  task problem_recache: :environment do
     ProblemRecacher.run
   end
  
   desc "Delete resolved errors from the database. (Useful for limited heroku databases)"
-  task :clear_resolved => :environment do
+  task clear_resolved: :environment do
     require 'resolved_problem_clearer'
     puts "=== Cleared #{ResolvedProblemClearer.new.execute} resolved errors from the database."
   end
  
   desc "Regenerate fingerprints"
-  task :notice_refingerprint => :environment do
+  task notice_refingerprint: :environment do
     NoticeRefingerprinter.run
     ProblemRecacher.run
   end
  
   desc "Remove notices in batch"
-  task :notices_delete, [ :problem_id ] => [ :environment ] do
+  task :notices_delete, [:problem_id] => [:environment] do
     BATCH_SIZE = 1000
     if args[:problem_id]
       item_count = Problem.find(args[:problem_id]).notices.count
 namespace :errbit do
   desc "Add a demo app & errors to your database (for testing)"
-  task :demo => :environment do
+  task demo: :environment do
     require 'fabrication'
  
-    app = Fabricate(:app, :name => "Demo App #{Time.zone.now.strftime('%N')}")
+    app = Fabricate(:app, name: "Demo App #{Time.zone.now.strftime('%N')}")
  
     # Report a number of errors for the application
     app.problems.delete_all
  
     errors = [{
-      :error_class => "ArgumentError",
-      :message => "wrong number of arguments (3 for 0)"
+      error_class: "ArgumentError",
+      message:     "wrong number of arguments (3 for 0)"
     }, {
-      :error_class => "RuntimeError",
-      :message => "Could not find Red October"
+      error_class: "RuntimeError",
+      message:     "Could not find Red October"
     }, {
-      :error_class => "TypeError",
-      :message => "can't convert Symbol into Integer"
+      error_class: "TypeError",
+      message:     "can't convert Symbol into Integer"
     }, {
-      :error_class => "ActiveRecord::RecordNotFound",
-      :message => "could not find a record with the id 5"
+      error_class: "ActiveRecord::RecordNotFound",
+      message:     "could not find a record with the id 5"
     }, {
-      :error_class => "NameError",
-      :message => "uninitialized constant Tag"
+      error_class: "NameError",
+      message:     "uninitialized constant Tag"
     }, {
-      :error_class => "SyntaxError",
-      :message => "unexpected tSTRING_BEG, expecting keyword_do or '{' or '('"
+      error_class: "SyntaxError",
+      message:     "unexpected tSTRING_BEG, expecting keyword_do or '{' or '('"
     }]
  
     RANDOM_METHODS = ActiveSupport.methods.shuffle[1..8]
  
     def random_backtrace
       backtrace = []
-      99.times {|t| backtrace << {
-        'number'  => t.hash % 1000,
-        'file'    => "/path/to/file.rb",
-        'method'  => RANDOM_METHODS.sample.to_s
-      }}
+      99.times {|t|
+        backtrace << {
+          'number' => t.hash % 1000,
+          'file'   => "/path/to/file.rb",
+          'method' => RANDOM_METHODS.sample.to_s
+        }}
       backtrace
     end
  
     errors.each do |error_template|
       rand(34).times do
-        ErrorReport.new(error_template.reverse_merge({
-          :api_key => app.api_key,
-          :error_class => "StandardError",
-          :message => "Oops. Something went wrong!",
-          :backtrace => random_backtrace,
-          :request => {
-            'component' => 'main',
-            'action' => 'error',
-            'url' => "http://example.com/post/#{[111, 222, 333].sample}",
-          },
-          :server_environment => {'environment-name' => Rails.env.to_s},
-          :notifier => {:name => "seeds.rb"},
-          :app_user => {
-            :id => "1234",
-            :username => "jsmith",
-            :name => "John Smith",
-            :url => "http://www.example.com/users/jsmith"
-          }
-        })).generate_notice!
+        ErrorReport.new(
+          error_template.reverse_merge(
+            api_key:            app.api_key,
+            error_class:        "StandardError",
+            message:            "Oops. Something went wrong!",
+            backtrace:          random_backtrace,
+            request:            {
+              'component' => 'main',
+              'action'    => 'error',
+              'url'       => "http://example.com/post/#{[111, 222, 333].sample}"
+            },
+            server_environment: { 'environment-name' => Rails.env.to_s },
+            notifier:           { name: "seeds.rb" },
+            app_user:           {
+              id:       "1234",
+              username: "jsmith",
+              name:     "John Smith",
+              url:      "http://www.example.com/users/jsmith"
+            }
+          )
+        ).generate_notice!
       end
     end
  
-    Fabricate(:notice, :err => Fabricate(:err, :problem => Fabricate(:problem, :app => app)))
+    Fabricate(:notice, err: Fabricate(:err, problem: Fabricate(:problem, app: app)))
     puts "=== Created demo app: '#{app.name}', with example errors."
   end
 end
 desc "This task is called by the Heroku cron add-on"
-task :cron => :environment do
+task cron: :environment do
   Rake::Task["errbit:db:clear_resolved"].invoke
 end
@@ -7,9 +7,9 @@ OmniAuth.config.test_mode = true
  
 def mock_auth(user = "test_user", token = "abcdef")
   OmniAuth.config.mock_auth[:github] = Hashie::Mash.new(
-    'provider' => 'github',
-    'uid' => '1763',
-    'extra' => {
+    'provider'    => 'github',
+    'uid'         => '1763',
+    'extra'       => {
       'raw_info' => {
         'login' => user
       }
@@ -22,7 +22,7 @@ end
  
 def log_in(user)
   visit '/'
-  fill_in :user_email, :with => user.email
-  fill_in :user_password, :with => 'password'
+  fill_in :user_email, with: user.email
+  fill_in :user_password, with: 'password'
   click_on I18n.t('devise.sessions.new.sign_in')
 end
@@ -4,7 +4,7 @@ feature &quot;Regeneration api_Key&quot; do
   let(:app) { Fabricate(:app) }
   let(:admin) { Fabricate(:admin) }
   let(:user) {
-    Fabricate(:user_watcher, :app => app).user
+    Fabricate(:user_watcher, app: app).user
   }
  
   before do
@@ -37,7 +37,7 @@ end
 feature "Create an application" do
   let(:admin) { Fabricate(:admin) }
   let(:user) {
-    Fabricate(:user_watcher, :app => app).user
+    Fabricate(:user_watcher, app: app).user
   }
  
   before do
@@ -48,37 +48,37 @@ feature &quot;Create an application&quot; do
     visit '/'
     log_in admin
     click_on I18n.t('apps.index.new_app')
-    fill_in 'app_name', :with => 'My new app'
+    fill_in 'app_name', with: 'My new app'
     click_on I18n.t('apps.new.add_app')
     page.has_content?(I18n.t('controllers.apps.flash.create.success'))
-    expect(App.where(:name => 'My new app').count).to eq 1
-    expect(App.where(:name => 'My new app 2').count).to eq 0
+    expect(App.where(name: 'My new app').count).to eq 1
+    expect(App.where(name: 'My new app 2').count).to eq 0
  
     click_on I18n.t('shared.navigation.apps')
     click_on 'My new app'
     click_link I18n.t('apps.show.edit')
-    fill_in 'app_name', :with => 'My new app 2'
+    fill_in 'app_name', with: 'My new app 2'
     click_on I18n.t('apps.edit.update')
     page.has_content?(I18n.t('controllers.apps.flash.update.success'))
-    expect(App.where(:name => 'My new app').count).to eq 0
-    expect(App.where(:name => 'My new app 2').count).to eq 1
+    expect(App.where(name: 'My new app').count).to eq 0
+    expect(App.where(name: 'My new app 2').count).to eq 1
   end
  
-  scenario "create an apps with issue tracker and edit it", :js => true do
+  scenario "create an apps with issue tracker and edit it", js: true do
     visit '/'
     log_in admin
     click_on I18n.t('apps.index.new_app')
-    fill_in 'app_name', :with => 'My new app'
+    fill_in 'app_name', with: 'My new app'
     find('.label_radio.github').click
  
     fill_in 'app_github_repo', with: 'foo/bar'
     within ".github.tracker_params" do
-      fill_in 'app_issue_tracker_attributes_options_username', :with => 'token'
-      fill_in 'app_issue_tracker_attributes_options_password', :with => 'pass'
+      fill_in 'app_issue_tracker_attributes_options_username', with: 'token'
+      fill_in 'app_issue_tracker_attributes_options_password', with: 'pass'
     end
     click_on I18n.t('apps.new.add_app')
     expect(page.has_content?(I18n.t('controllers.apps.flash.create.success'))).to eql true
-    app = App.where(:name => 'My new app').first
+    app = App.where(name: 'My new app').first
     expect(app.issue_tracker.type_tracker).to eql 'github'
     expect(app.issue_tracker.options['username']).to eql 'token'
     expect(app.issue_tracker.options['password']).to eql 'pass'
@@ -89,7 +89,7 @@ feature &quot;Create an application&quot; do
     find('.issue_tracker .label_radio.none').click
     click_on I18n.t('apps.edit.update')
     expect(page.has_content?(I18n.t('controllers.apps.flash.update.success'))).to eql true
-    app = App.where(:name => 'My new app').first
+    app = App.where(name: 'My new app').first
     expect(app.issue_tracker.tracker).to be_a ErrbitPlugin::NoneIssueTracker
   end
 end
@@ -20,7 +20,7 @@ feature &#39;password reset token&#39; do
     expect(mail.body.encoded).to match(/change your password/)
     expect(mail.body.encoded).to match(regex)
     if mail.body.encoded =~ regex
-      visit "/users/password/edit?reset_password_token=#{$1}"
+      visit "/users/password/edit?reset_password_token=#{Regexp.last_match(1)}"
       expect(page).to have_content 'Change your password'
       fill_in 'New password', with: 'test12345'
       fill_in 'Type your new password again', with: 'test12345'
@@ -3,7 +3,7 @@ require &#39;acceptance/acceptance_helper&#39;
 feature 'Sign in with GitHub' do
   background do
     allow(Errbit::Config).to receive(:github_authentication).and_return(true)
-    Fabricate(:user, :github_login => 'nashby')
+    Fabricate(:user, github_login: 'nashby')
   end
  
   scenario 'log in via GitHub with recognized user' do
@@ -11,7 +11,7 @@ feature &#39;Sign in with GitHub&#39; do
  
     visit '/'
     click_link 'Sign in with GitHub'
-    expect(page).to have_content I18n.t("devise.omniauth_callbacks.success", :kind => 'GitHub')
+    expect(page).to have_content I18n.t("devise.omniauth_callbacks.success", kind: 'GitHub')
   end
  
   scenario 'reject unrecognized user if authenticating via GitHub' do
@@ -6,30 +6,30 @@ describe Api::V1::NoticesController, type: &#39;controller&#39; do
  
     describe "GET /api/v1/notices" do
       before do
-        Fabricate(:notice, :created_at => Time.zone.parse('2012-08-01'))
-        Fabricate(:notice, :created_at => Time.zone.parse('2012-08-01'))
-        Fabricate(:notice, :created_at => Time.zone.parse('2012-08-21'))
-        Fabricate(:notice, :created_at => Time.zone.parse('2012-08-30'))
+        Fabricate(:notice, created_at: Time.zone.parse('2012-08-01'))
+        Fabricate(:notice, created_at: Time.zone.parse('2012-08-01'))
+        Fabricate(:notice, created_at: Time.zone.parse('2012-08-21'))
+        Fabricate(:notice, created_at: Time.zone.parse('2012-08-30'))
       end
  
       it "should return JSON if JSON is requested" do
-        get :index, :auth_token => @user.authentication_token, :format => "json"
+        get :index, auth_token: @user.authentication_token, format: "json"
         expect { JSON.load(response.body) }.not_to raise_error #JSON::ParserError)
       end
  
       it "should return XML if XML is requested" do
-        get :index, :auth_token => @user.authentication_token, :format => "xml"
+        get :index, auth_token: @user.authentication_token, format: "xml"
         expect(Nokogiri::XML(response.body).errors).to be_empty
       end
  
       it "should return JSON by default" do
-        get :index, :auth_token => @user.authentication_token
+        get :index, auth_token: @user.authentication_token
         expect { JSON.load(response.body) }.not_to raise_error #JSON::ParserError)
       end
  
       describe "given a date range" do
         it "should return only the notices created during the date range" do
-          get :index, {:auth_token => @user.authentication_token, :start_date => "2012-08-01", :end_date => "2012-08-27"}
+          get :index, auth_token: @user.authentication_token, start_date: "2012-08-01", end_date: "2012-08-27"
           expect(response).to be_success
           notices = JSON.load response.body
           expect(notices.length).to eq 3
@@ -37,7 +37,7 @@ describe Api::V1::NoticesController, type: &#39;controller&#39; do
       end
  
       it "should return all notices" do
-        get :index, {:auth_token => @user.authentication_token}
+        get :index, auth_token: @user.authentication_token
         expect(response).to be_success
         notices = JSON.load response.body
         expect(notices.length).to eq 4
@@ -7,34 +7,34 @@ describe Api::V1::ProblemsController, type: &#39;controller&#39; do
     describe "GET /api/v1/problems/:id" do
       before do
         notice = Fabricate(:notice)
-        err = Fabricate(:err, :notices => [notice])
-        @problem = Fabricate(:problem, :errs => [err])
+        err = Fabricate(:err, notices: [notice])
+        @problem = Fabricate(:problem, errs: [err])
       end
  
       it "should return JSON if JSON is requested" do
-        get :show, :auth_token => @user.authentication_token, :format => "json", :id => Problem.first.id
+        get :show, auth_token: @user.authentication_token, format: "json", id: Problem.first.id
         expect { JSON.load(response.body) }.not_to raise_error #JSON::ParserError
       end
  
       it "should return XML if XML is requested" do
-        get :index, :auth_token => @user.authentication_token, :format => "xml", :id => @problem.id
+        get :index, auth_token: @user.authentication_token, format: "xml", id: @problem.id
         expect(Nokogiri::XML(response.body).errors).to be_empty
       end
  
       it "should return JSON by default" do
-        get :show, :auth_token => @user.authentication_token, :id => @problem.id
-        expect { JSON.load(response.body) }.not_to raise_error#JSON::ParserError)
+        get :show, auth_token: @user.authentication_token, id: @problem.id
+        expect { JSON.load(response.body) }.not_to raise_error #JSON::ParserError)
       end
  
       it "should return the correct problem" do
-        get :show, :auth_token => @user.authentication_token, :format => "json", :id => @problem.id
+        get :show, auth_token: @user.authentication_token, format: "json", id: @problem.id
  
         returned_problem = JSON.parse(response.body)
         expect(returned_problem["_id"]).to eq(@problem.id.to_s)
       end
  
       it "should return only the correct fields" do
-        get :show, :auth_token => @user.authentication_token, :format => "json", :id => @problem.id
+        get :show, auth_token: @user.authentication_token, format: "json", id: @problem.id
         returned_problem = JSON.parse(response.body)
  
         expect(returned_problem.keys).to match_array(%w(
@@ -52,37 +52,37 @@ describe Api::V1::ProblemsController, type: &#39;controller&#39; do
       end
  
       it "returns a 404 if the problem cannot be found" do
-        get :show, :auth_token => @user.authentication_token, :format => "json", :id => 'IdontExist'
+        get :show, auth_token: @user.authentication_token, format: "json", id: 'IdontExist'
         expect(response.status).to eq(404)
       end
     end
  
     describe "GET /api/v1/problems" do
       before do
-        Fabricate(:problem, :first_notice_at => Date.new(2012, 8, 01), :resolved_at => Date.new(2012, 8, 02))
-        Fabricate(:problem, :first_notice_at => Date.new(2012, 8, 01), :resolved_at => Date.new(2012, 8, 21))
-        Fabricate(:problem, :first_notice_at => Date.new(2012, 8, 21))
-        Fabricate(:problem, :first_notice_at => Date.new(2012, 8, 30))
+        Fabricate(:problem, first_notice_at: Date.new(2012, 8, 01), resolved_at: Date.new(2012, 8, 02))
+        Fabricate(:problem, first_notice_at: Date.new(2012, 8, 01), resolved_at: Date.new(2012, 8, 21))
+        Fabricate(:problem, first_notice_at: Date.new(2012, 8, 21))
+        Fabricate(:problem, first_notice_at: Date.new(2012, 8, 30))
       end
  
       it "should return JSON if JSON is requested" do
-        get :index, :auth_token => @user.authentication_token, :format => "json"
-        expect { JSON.load(response.body) }.not_to raise_error#JSON::ParserError)
+        get :index, auth_token: @user.authentication_token, format: "json"
+        expect { JSON.load(response.body) }.not_to raise_error #JSON::ParserError)
       end
  
       it "should return XML if XML is requested" do
-        get :index, :auth_token => @user.authentication_token, :format => "xml"
+        get :index, auth_token: @user.authentication_token, format: "xml"
         expect(Nokogiri::XML(response.body).errors).to be_empty
       end
  
       it "should return JSON by default" do
-        get :index, :auth_token => @user.authentication_token
-        expect { JSON.load(response.body) }.not_to raise_error#JSON::ParserError)
+        get :index, auth_token: @user.authentication_token
+        expect { JSON.load(response.body) }.not_to raise_error #JSON::ParserError)
       end
  
       describe "given a date range" do
         it "should return only the problems open during the date range" do
-          get :index, {:auth_token => @user.authentication_token, :start_date => "2012-08-20", :end_date => "2012-08-27"}
+          get :index, auth_token: @user.authentication_token, start_date: "2012-08-20", end_date: "2012-08-27"
           expect(response).to be_success
           problems = JSON.load response.body
           expect(problems.length).to eq 2
@@ -90,7 +90,7 @@ describe Api::V1::ProblemsController, type: &#39;controller&#39; do
       end
  
       it "should return all problems" do
-        get :index, {:auth_token => @user.authentication_token}
+        get :index, auth_token: @user.authentication_token
         expect(response).to be_success
         problems = JSON.load response.body
         expect(problems.length).to eq 4
@@ -15,22 +15,22 @@ describe Api::V3::NoticesController, type: :controller do
   it 'returns created notice id in json format' do
     post :create, legit_body, legit_params
     notice = Notice.last
-    expect(JSON.parse(response.body)).to eq({
-      'id' => notice.id.to_s,
+    expect(JSON.parse(response.body)).to eq(
+      'id'  => notice.id.to_s,
       'url' => app_problem_url(app, notice.problem)
-    })
+    )
   end
  
   it 'responds with 400 when request attributes are not valid' do
-    allow_any_instance_of(AirbrakeApi::V3::NoticeParser)
-      .to receive(:report).and_raise(AirbrakeApi::ParamsError)
+    allow_any_instance_of(AirbrakeApi::V3::NoticeParser).
+      to receive(:report).and_raise(AirbrakeApi::ParamsError)
     post :create, project_id: 'ID'
     expect(response.status).to eq(400)
     expect(response.body).to eq('Invalid request')
   end
  
   it 'responds with 422 when project_id is invalid' do
-    post :create, legit_body, { project_id: 'hm?', key: 'wha?' }
+    post :create, legit_body, project_id: 'hm?', key: 'wha?'
  
     expect(response.status).to eq(422)
     expect(response.body).to eq('Your API key is unknown')
@@ -38,7 +38,7 @@ describe Api::V3::NoticesController, type: :controller do
  
   it 'ignores notices for older api' do
     app = Fabricate(:app, current_app_version: '2.0')
-    post :create, legit_body, { project_id: app.api_key, key: app.api_key }
+    post :create, legit_body, project_id: app.api_key, key: app.api_key
     expect(response.body).to eq('Notice for old app version ignored')
     expect(Notice.count).to eq(0)
   end
 describe AppsController, type: 'controller' do
   it_requires_authentication
-  it_requires_admin_privileges :for => {:new => :get, :edit => :get, :create => :post, :update => :put, :destroy => :delete}
+  it_requires_admin_privileges for: { new: :get, edit: :get, create: :post, update: :put, destroy: :delete }
  
   let(:admin) { Fabricate(:admin) }
   let(:user) { Fabricate(:user) }
-  let(:watcher) { Fabricate(:user_watcher, :app => app, :user => user) }
+  let(:watcher) { Fabricate(:user_watcher, app: app, user: user) }
   let(:unwatched_app) { Fabricate(:app) }
   let(:app) { unwatched_app }
   let(:watched_app1) do
     a = Fabricate(:app)
-    Fabricate(:user_watcher, :user => user, :app => a)
+    Fabricate(:user_watcher, user: user, app: a)
     a
   end
   let(:watched_app2) do
     a = Fabricate(:app)
-    Fabricate(:user_watcher, :user => user, :app => a)
+    Fabricate(:user_watcher, user: user, app: a)
     a
   end
   let(:err) do
-    Fabricate(:err, :problem => problem)
+    Fabricate(:err, problem: problem)
   end
   let(:notice) do
-    Fabricate(:notice, :err => err)
+    Fabricate(:notice, err: err)
   end
   let(:problem) do
-    Fabricate(:problem, :app => app)
+    Fabricate(:problem, app: app)
   end
-  let(:problem_resolved) { Fabricate(:problem_resolved, :app => app) }
+  let(:problem_resolved) { Fabricate(:problem_resolved, app: app) }
  
   describe "GET /apps" do
     context 'when logged in as an admin' do
@@ -55,43 +55,43 @@ describe AppsController, type: &#39;controller&#39; do
       end
  
       it 'finds the app' do
-        get :show, :id => app.id
+        get :show, id: app.id
         expect(controller.app).to eq app
       end
  
       it "should not raise errors for app with err without notices" do
         err
-        expect{ get :show, :id => app.id }.to_not raise_error
+        expect { get :show, id: app.id }.to_not raise_error
       end
  
       it "should list atom feed successfully" do
-        get :show, :id => app.id, :format => "atom"
+        get :show, id: app.id, format: "atom"
         expect(response).to be_success
       end
  
       it "should list available watchers by name" do
-        Fabricate(:user, :name => "Carol")
-        Fabricate(:user, :name => "Alice")
-        Fabricate(:user, :name => "Betty")
+        Fabricate(:user, name: "Carol")
+        Fabricate(:user, name: "Alice")
+        Fabricate(:user, name: "Betty")
  
-        get :show, :id => app.id
+        get :show, id: app.id
  
         expect(controller.users.to_a).to eq(User.all.to_a.sort_by(&:name))
       end
  
       context "pagination" do
         before(:each) do
-          35.times { Fabricate(:err, :problem => Fabricate(:problem, :app => app)) }
+          35.times { Fabricate(:err, problem: Fabricate(:problem, app: app)) }
         end
  
         it "should have default per_page value for user" do
-          get :show, :id => app.id
+          get :show, id: app.id
           expect(controller.problems.to_a.size).to eq User::PER_PAGE
         end
  
         it "should be able to override default per_page value" do
           admin.update_attribute :per_page, 10
-          get :show, :id => app.id
+          get :show, id: app.id
           expect(controller.problems.to_a.size).to eq 10
         end
       end
@@ -103,14 +103,14 @@ describe AppsController, type: &#39;controller&#39; do
  
         context 'and no params' do
           it 'shows only unresolved problems' do
-            get :show, :id => app.id
+            get :show, id: app.id
             expect(controller.problems.size).to eq 1
           end
         end
  
         context 'and all_problems=true params' do
           it 'shows all errors' do
-            get :show, :id => app.id, :all_errs => true
+            get :show, id: app.id, all_errs: true
             expect(controller.problems.size).to eq 2
           end
         end
@@ -120,41 +120,41 @@ describe AppsController, type: &#39;controller&#39; do
         before(:each) do
           environments = %w(production test development staging)
           20.times do |i|
-            Fabricate(:problem, :app => app, :environment => environments[i % environments.length])
+            Fabricate(:problem, app: app, environment: environments[i % environments.length])
           end
         end
  
         context 'no params' do
           it 'shows errs for all environments' do
-            get :show, :id => app.id
+            get :show, id: app.id
             expect(controller.problems.size).to eq 20
           end
         end
  
         context 'environment production' do
           it 'shows errs for just production' do
-            get :show, :id => app.id, :environment => 'production'
+            get :show, id: app.id, environment: 'production'
             expect(controller.problems.size).to eq 5
           end
         end
  
         context 'environment staging' do
           it 'shows errs for just staging' do
-            get :show, :id => app.id, :environment => 'staging'
+            get :show, id: app.id, environment: 'staging'
             expect(controller.problems.size).to eq 5
           end
         end
  
         context 'environment development' do
           it 'shows errs for just development' do
-            get :show, :id => app.id, :environment => 'development'
+            get :show, id: app.id, environment: 'development'
             expect(controller.problems.size).to eq 5
           end
         end
  
         context 'environment test' do
           it 'shows errs for just test' do
-            get :show, :id => app.id, :environment => 'test'
+            get :show, id: app.id, environment: 'test'
             expect(controller.problems.size).to eq 5
           end
         end
@@ -166,7 +166,7 @@ describe AppsController, type: &#39;controller&#39; do
         sign_in Fabricate(:user)
         app = Fabricate(:app)
  
-        get :show, :id => app.id
+        get :show, id: app.id
         expect(controller.app).to eq app
       end
     end
@@ -186,9 +186,9 @@ describe AppsController, type: &#39;controller&#39; do
       end
  
       it "should copy attributes from an existing app" do
-        @app = Fabricate(:app, :name => "do not copy",
-                             :github_repo => "test/example")
-        get :new, :copy_attributes_from => @app.id
+        @app = Fabricate(:app, name:        "do not copy",
+                               github_repo: "test/example")
+        get :new, copy_attributes_from: @app.id
         expect(controller.app).to be_a(App)
         expect(controller.app).to be_new_record
         expect(controller.app.name).to be_blank
@@ -199,7 +199,7 @@ describe AppsController, type: &#39;controller&#39; do
     describe "GET /apps/:id/edit" do
       it 'finds the correct app' do
         app = Fabricate(:app)
-        get :edit, :id => app.id
+        get :edit, id: app.id
         expect(controller.app).to eq app
       end
     end
@@ -216,12 +216,12 @@ describe AppsController, type: &#39;controller&#39; do
         end
  
         it "should redirect to the app page" do
-          post :create, :app => {}
+          post :create, app: {}
           expect(response).to redirect_to(app_path(@app))
         end
  
         it "should display a message" do
-          post :create, :app => {}
+          post :create, app: {}
           expect(request.flash[:success]).to match(/success/)
         end
       end
@@ -234,12 +234,12 @@ describe AppsController, type: &#39;controller&#39; do
  
       context "when the update is successful" do
         it "should redirect to the app page" do
-          put :update, :id => @app.id, :app => {}
+          put :update, id: @app.id, app: {}
           expect(response).to redirect_to(app_path(@app))
         end
  
         it "should display a message" do
-          put :update, :id => @app.id, :app => {}
+          put :update, id: @app.id, app: {}
           expect(request.flash[:success]).to match(/success/)
         end
       end
@@ -247,14 +247,14 @@ describe AppsController, type: &#39;controller&#39; do
       context "changing name" do
         it "should redirect to app page" do
           id = @app.id
-          put :update, :id => id, :app => {:name => "new name"}
+          put :update, id: id, app: { name: "new name" }
           expect(response).to redirect_to(app_path(id))
         end
       end
  
       context "when the update is unsuccessful" do
         it "should render the edit page" do
-          put :update, :id => @app.id, :app => { :name => '' }
+          put :update, id: @app.id, app: { name: '' }
           expect(response).to render_template(:edit)
         end
       end
@@ -265,30 +265,30 @@ describe AppsController, type: &#39;controller&#39; do
         end
  
         it "should parse legal csv values" do
-          put :update, :id => @app.id, :app => { :email_at_notices => '1,   4,      7,8,  10' }
+          put :update, id: @app.id, app: { email_at_notices: '1,   4,      7,8,  10' }
           @app.reload
           expect(@app.email_at_notices).to eq [1, 4, 7, 8, 10]
         end
         context "failed parsing of CSV" do
           it "should set the default value" do
-            @app = Fabricate(:app, :email_at_notices => [1, 2, 3, 4])
-            put :update, :id => @app.id, :app => { :email_at_notices => 'asdf, -1,0,foobar,gd00,0,abc' }
+            @app = Fabricate(:app, email_at_notices: [1, 2, 3, 4])
+            put :update, id: @app.id, app: { email_at_notices: 'asdf, -1,0,foobar,gd00,0,abc' }
             @app.reload
             expect(@app.email_at_notices).to eq Errbit::Config.email_at_notices
           end
  
           it "should display a message" do
-            put :update, :id => @app.id, :app => { :email_at_notices => 'qwertyuiop' }
+            put :update, id: @app.id, app: { email_at_notices: 'qwertyuiop' }
             expect(request.flash[:error]).to match(/Couldn't parse/)
           end
         end
       end
  
-      context "setting up issue tracker", :cur => true do
+      context "setting up issue tracker", cur: true do
         context "unknown tracker type" do
           before(:each) do
-            put :update, :id => @app.id, :app => { :issue_tracker_attributes => {
-              :type_tracker => 'unknown', :options => {:project_id => '1234', :api_token => '123123', :account => 'myapp'}
+            put :update, id: @app.id, app: { issue_tracker_attributes: {
+              type_tracker: 'unknown', options: { project_id: '1234', api_token: '123123', account: 'myapp' }
             } }
             @app.reload
           end
@@ -306,24 +306,24 @@ describe AppsController, type: &#39;controller&#39; do
       end
  
       it "should find the app" do
-        delete :destroy, :id => @app.id
+        delete :destroy, id: @app.id
         expect(controller.app).to eq @app
       end
  
       it "should destroy the app" do
-        delete :destroy, :id => @app.id
+        delete :destroy, id: @app.id
         expect {
           @app.reload
         }.to raise_error(Mongoid::Errors::DocumentNotFound)
       end
  
       it "should display a message" do
-        delete :destroy, :id => @app.id
+        delete :destroy, id: @app.id
         expect(request.flash[:success]).to match(/success/)
       end
  
       it "should redirect to the apps page" do
-        delete :destroy, :id => @app.id
+        delete :destroy, id: @app.id
         expect(response).to redirect_to(apps_path)
       end
     end
@@ -336,7 +336,7 @@ describe AppsController, type: &#39;controller&#39; do
       end
  
       it 'redirect to root with flash error' do
-        post :regenerate_api_key, :id => 'foo'
+        post :regenerate_api_key, id: 'foo'
         expect(request).to redirect_to root_path
       end
     end
@@ -348,7 +348,7 @@ describe AppsController, type: &#39;controller&#39; do
  
       it 'redirect_to app view' do
         expect do
-          post :regenerate_api_key, :id => app.id
+          post :regenerate_api_key, id: app.id
           expect(request).to redirect_to edit_app_path(app)
         end.to change { app.reload.api_key }
       end
 describe CommentsController, type: 'controller' do
   let(:app) { Fabricate(:app) }
-  let(:err) { Fabricate(:err, :problem => Fabricate(:problem, :app => app, :environment => "production")) }
+  let(:err) { Fabricate(:err, problem: Fabricate(:problem, app: app, environment: "production")) }
  
   describe "POST /apps/:app_id/errs/:id/comments/create" do
     render_views
@@ -14,8 +14,8 @@ describe CommentsController, type: &#39;controller&#39; do
       let(:user) { Fabricate(:user) }
  
       before(:each) do
-        post :create, :app_id => problem.app.id, :problem_id => problem.id,
-             :comment => { :body => "One test comment", :user_id => user.id }
+        post :create, app_id: problem.app.id, problem_id: problem.id,
+             comment: { body: "One test comment", user_id: user.id }
         problem.reload
       end
  
@@ -41,12 +41,12 @@ describe CommentsController, type: &#39;controller&#39; do
       let(:comment) { problem.reload.comments.first }
  
       before(:each) do
-        delete :destroy, :app_id => problem.app.id, :problem_id => problem.id, :id => comment.id.to_s
+        delete :destroy, app_id: problem.app.id, problem_id: problem.id, id: comment.id.to_s
         problem.reload
       end
  
       it "should delete the comment" do
-        expect(problem.comments.detect{|c| c.id.to_s == comment.id }).to be nil
+        expect(problem.comments.detect { |c| c.id.to_s == comment.id }).to be nil
       end
  
       it "should redirect to problem page" do
@@ -10,30 +10,28 @@ describe DeploysController, type: &#39;controller&#39; do
         'scm_revision'   => '19d77837eef37902cf5df7e4445c85f392a8d0d5',
         'message'        => 'johns first deploy'
       }
-      @app = Fabricate(:app_with_watcher, :notify_on_deploys => true, :api_key => 'APIKEY')
+      @app = Fabricate(:app_with_watcher, notify_on_deploys: true, api_key: 'APIKEY')
     end
  
     it 'finds the app via the api key' do
       expect(App).to receive(:find_by_api_key!).with('APIKEY').and_return(@app)
-      post :create, :deploy => @params, :api_key => 'APIKEY'
+      post :create, deploy: @params, api_key: 'APIKEY'
     end
  
     it 'creates a deploy' do
       expect(App).to receive(:find_by_api_key!).and_return(@app)
       expect(@app.deploys).to receive(:create!).
-        with({
-          :username     => 'john.doe',
-          :environment  => 'production',
-          :repository   => 'git@github.com/errbit/errbit.git',
-          :revision     => '19d77837eef37902cf5df7e4445c85f392a8d0d5',
-          :message      => 'johns first deploy'
-
-        }).and_return(Fabricate(:deploy))
-      post :create, :deploy => @params, :api_key => 'APIKEY'
+        with(
+          username:    'john.doe',
+          environment: 'production',
+          repository:  'git@github.com/errbit/errbit.git',
+          revision:    '19d77837eef37902cf5df7e4445c85f392a8d0d5',
+          message:     'johns first deploy').and_return(Fabricate(:deploy))
+      post :create, deploy: @params, api_key: 'APIKEY'
     end
  
     it 'sends an email notification when configured to do so' do
-      post :create, :deploy => @params, :api_key => 'APIKEY'
+      post :create, deploy: @params, api_key: 'APIKEY'
       email = ActionMailer::Base.deliveries.last
       expect(email.to).to include(@app.watchers.first.email)
       expect(email.subject).to eq "[#{@app.name}] Deployed to production by john.doe"
@@ -44,7 +42,7 @@ describe DeploysController, type: &#39;controller&#39; do
     before(:each) do
       @deploy = Fabricate :deploy
       sign_in Fabricate(:admin)
-      get :index, :app_id => @deploy.app.id
+      get :index, app_id: @deploy.app.id
     end
  
     it "should render successfully" do
@@ -10,12 +10,12 @@ describe Devise::SessionsController, type: &#39;controller&#39; do
     let(:user) { Fabricate(:user) }
  
     it 'redirects to app index page if there are no apps for the user' do
-      post :create, { :user => { 'email' => user.email, 'password' => user.password } }
+      post :create, user: { 'email' => user.email, 'password' => user.password }
       expect(response).to redirect_to(root_path)
     end
  
     it 'displays a friendly error when credentials are invalid' do
-      post :create, { :user => { 'email' => 'whatever', 'password' => 'somethinginvalid' } }
+      post :create, user: { 'email' => 'whatever', 'password' => 'somethinginvalid' }
       expect(request.flash["alert"]).to eq(I18n.t 'devise.failure.user.email_invalid')
     end
   end
 describe NoticesController, type: 'controller' do
-  it_requires_authentication :for => { :locate => :get }
+  it_requires_authentication for: { locate: :get }
  
   let(:notice) { Fabricate(:notice) }
   let(:xml) { Rails.root.join('spec', 'fixtures', 'hoptoad_test_notice.xml').read }
   let(:app) { Fabricate(:app) }
-  let(:error_report) { double(:valid? => true, :generate_notice! => true, :notice => notice, :should_keep? => true) }
+  let(:error_report) { double(valid?: true, generate_notice!: true, notice: notice, should_keep?: true) }
  
   context 'notices API' do
     context "with bogus xml" do
       it "returns an error" do
         expect(request).to receive(:raw_post).and_return('<r><b>notxml</r>')
-        post :create, :format => :xml
+        post :create, format: :xml
         expect(response.status).to eq(422)
         expect(response.body).to eq('The provided XML was not well-formed')
       end
@@ -24,7 +24,7 @@ describe NoticesController, type: &#39;controller&#39; do
       context "with xml pass in raw_port" do
         before do
           expect(request).to receive(:raw_post).and_return(xml)
-          post :create, :format => :xml
+          post :create, format: :xml
         end
  
         it "generates a notice from raw xml [POST]" do
@@ -37,7 +37,7 @@ describe NoticesController, type: &#39;controller&#39; do
       end
  
       it "generates a notice from xml in a data param [POST]" do
-        post :create, :data => xml, :format => :xml
+        post :create, data: xml, format: :xml
         expect(response).to be_success
         # Same RegExp from Airbrake::Sender#send_to_airbrake (https://github.com/airbrake/airbrake/blob/master/lib/airbrake/sender.rb#L53)
         # Inspired by https://github.com/airbrake/airbrake/blob/master/test/sender_test.rb
@@ -46,15 +46,15 @@ describe NoticesController, type: &#39;controller&#39; do
       end
  
       it "generates a notice from xml [GET]" do
-        get :create, :data => xml, :format => :xml
+        get :create, data: xml, format: :xml
         expect(response).to be_success
         expect(response.body).to match(%r{<id[^>]*>#{notice.id}</id>})
         expect(response.body).to match(%r{<url[^>]*>(.+)#{locate_path(notice.id)}</url>})
       end
       context "with an invalid API_KEY" do
-        let(:error_report) { double(:valid? => false) }
+        let(:error_report) { double(valid?: false) }
         it 'return 422' do
-          post :create, :format => :xml, :data => xml
+          post :create, format: :xml, data: xml
           expect(response.status).to eq 422
         end
       end
@@ -62,7 +62,7 @@ describe NoticesController, type: &#39;controller&#39; do
  
     context "without params needed" do
       it 'return 400' do
-        post :create, :format => :xml
+        post :create, format: :xml
         expect(response.status).to eq 400
         expect(response.body).to eq 'Need a data params in GET or raw post data'
       end
@@ -77,9 +77,9 @@ describe NoticesController, type: &#39;controller&#39; do
       end
  
       it "should locate notice and redirect to problem" do
-        problem = Fabricate(:problem, :app => app, :environment => "production")
-        notice = Fabricate(:notice, :err => Fabricate(:err, :problem => problem))
-        get :locate, :id => notice.id
+        problem = Fabricate(:problem, app: app, environment: "production")
+        notice = Fabricate(:notice, err: Fabricate(:err, problem: problem))
+        get :locate, id: notice.id
         expect(response).to redirect_to(app_problem_path(problem.app, problem))
       end
     end
 describe ProblemsController, type: 'controller' do
-  it_requires_authentication :for => {
-    :index => :get, :show => :get, :resolve => :put, :search => :get
+  it_requires_authentication for:    {
+    index: :get, show: :get, resolve: :put, search: :get
   },
-  :params => {:app_id => 'dummyid', :id => 'dummyid'}
+                             params: { app_id: 'dummyid', id: 'dummyid' }
  
   let(:app) { Fabricate(:app) }
-  let(:err) { Fabricate(:err, :problem => problem) }
+  let(:err) { Fabricate(:err, problem: problem) }
   let(:user) { Fabricate(:user) }
-  let(:problem) { Fabricate(:problem, :app => app, :environment => "production") }
+  let(:problem) { Fabricate(:problem, app: app, environment: "production") }
  
   describe "GET /problems" do
     before(:each) do
       sign_in user
-      @problem = Fabricate(:notice, :err => Fabricate(:err, :problem => Fabricate(:problem, :app => app, :environment => "production"))).problem
+      @problem = Fabricate(:notice, err: Fabricate(:err, problem: Fabricate(:problem, app: app, environment: "production"))).problem
     end
  
     context "pagination" do
@@ -36,7 +36,7 @@ describe ProblemsController, type: &#39;controller&#39; do
       before(:each) do
         environments = %w(production test development staging)
         20.times do |i|
-          Fabricate(:problem, :environment => environments[i % environments.length])
+          Fabricate(:problem, environment: environments[i % environments.length])
         end
       end
  
@@ -49,28 +49,28 @@ describe ProblemsController, type: &#39;controller&#39; do
  
       context 'environment production' do
         it 'shows problems for just production' do
-          get :index, :environment => 'production'
+          get :index, environment: 'production'
           expect(controller.problems.size).to eq 6
         end
       end
  
       context 'environment staging' do
         it 'shows problems for just staging' do
-          get :index, :environment => 'staging'
+          get :index, environment: 'staging'
           expect(controller.problems.size).to eq 5
         end
       end
  
       context 'environment development' do
         it 'shows problems for just development' do
-          get :index, :environment => 'development'
+          get :index, environment: 'development'
           expect(controller.problems.size).to eq 5
         end
       end
  
       context 'environment test' do
         it 'shows problems for just test' do
-          get :index, :environment => 'test'
+          get :index, environment: 'test'
           expect(controller.problems.size).to eq 5
         end
       end
@@ -82,11 +82,11 @@ describe ProblemsController, type: &#39;controller&#39; do
       sign_in Fabricate(:user)
       problems = Kaminari.paginate_array((1..30).to_a)
       3.times { problems << Fabricate(:err).problem }
-      3.times { problems << Fabricate(:err, :problem => Fabricate(:problem, :resolved => true)).problem }
+      3.times { problems << Fabricate(:err, problem: Fabricate(:problem, resolved: true)).problem }
       expect(Problem).to receive(:ordered_by).and_return(
-        double('proxy', :page => double('other_proxy', :per => problems))
+        double('proxy', page: double('other_proxy', per: problems))
       )
-      get :index, :all_errs => true
+      get :index, all_errs: true
       expect(controller.problems).to eq problems
     end
   end
@@ -95,8 +95,8 @@ describe ProblemsController, type: &#39;controller&#39; do
     before do
       sign_in user
       @app      = Fabricate(:app)
-      @problem1 = Fabricate(:problem, :app => @app, message: "Most important")
-      @problem2 = Fabricate(:problem, :app => @app, message: "Very very important")
+      @problem1 = Fabricate(:problem, app: @app, message: "Most important")
+      @problem2 = Fabricate(:problem, app: @app, message: "Very very important")
     end
  
     it "renders successfully" do
@@ -110,7 +110,7 @@ describe ProblemsController, type: &#39;controller&#39; do
     end
  
     it "searches problems for given string" do
-      get :search, :search => "\"Most important\""
+      get :search, search: "\"Most important\""
       expect(controller.problems).to include(@problem1)
       expect(controller.problems).to_not include(@problem2)
     end
@@ -122,35 +122,35 @@ describe ProblemsController, type: &#39;controller&#39; do
     end
  
     it "finds the app" do
-      get :show, :app_id => app.id, :id => err.problem.id
+      get :show, app_id: app.id, id: err.problem.id
       expect(controller.app).to eq app
     end
  
     it "finds the problem" do
-      get :show, :app_id => app.id, :id => err.problem.id
+      get :show, app_id: app.id, id: err.problem.id
       expect(controller.problem).to eq err.problem
     end
  
     it "successfully render page" do
-      get :show, :app_id => app.id, :id => err.problem.id
+      get :show, app_id: app.id, id: err.problem.id
       expect(response).to be_success
     end
  
     context 'pagination' do
       let!(:notices) do
         3.times.reduce([]) do |coll, i|
-          coll << Fabricate(:notice, :err => err, :created_at => (i.seconds.from_now))
+          coll << Fabricate(:notice, err: err, created_at: (i.seconds.from_now))
         end
       end
  
       it "paginates the notices 1 at a time, starting with the most recent" do
-        get :show, :app_id => app.id, :id => err.problem.id
+        get :show, app_id: app.id, id: err.problem.id
         expect(assigns(:notices).entries.count).to eq 1
         expect(assigns(:notices)).to include(notices.last)
       end
  
       it "paginates the notices 1 at a time, based on then notice param" do
-        get :show, :app_id => app.id, :id => err.problem.id, :notice => 3
+        get :show, app_id: app.id, id: err.problem.id, notice: 3
         expect(assigns(:notices).entries.count).to eq 1
         expect(assigns(:notices)).to include(notices.first)
       end
@@ -165,29 +165,29 @@ describe ProblemsController, type: &#39;controller&#39; do
     end
  
     it 'finds the app and the problem' do
-      put :resolve, :app_id => @err.app.id, :id => @err.problem.id
+      put :resolve, app_id: @err.app.id, id: @err.problem.id
       expect(controller.app).to eq @err.app
       expect(controller.problem).to eq @err.problem
     end
  
     it "should resolve the issue" do
-      put :resolve, :app_id => @err.app.id, :id => @err.problem.id
+      put :resolve, app_id: @err.app.id, id: @err.problem.id
       expect(@err.problem.reload.resolved).to be(true)
     end
  
     it "should display a message" do
-      put :resolve, :app_id => @err.app.id, :id => @err.problem.id
+      put :resolve, app_id: @err.app.id, id: @err.problem.id
       expect(request.flash[:success]).to match(/Great news/)
     end
  
     it "should redirect to the app page" do
-      put :resolve, :app_id => @err.app.id, :id => @err.problem.id
+      put :resolve, app_id: @err.app.id, id: @err.problem.id
       expect(response).to redirect_to(app_path(@err.app))
     end
  
     it "should redirect back to problems page" do
       request.env["HTTP_REFERER"] = problems_path
-      put :resolve, :app_id => @err.app.id, :id => @err.problem.id
+      put :resolve, app_id: @err.app.id, id: @err.problem.id
       expect(response).to redirect_to(problems_path)
     end
   end
@@ -245,7 +245,7 @@ describe ProblemsController, type: &#39;controller&#39; do
  
         it "should render whatever the issue tracker says" do
           allow_any_instance_of(Issue).to receive(:render_body_args).and_return(
-            [{ :inline => 'one <%= problem.id %> two' }])
+            [{ inline: 'one <%= problem.id %> two' }])
           post :create_issue, app_id: problem.app.id, id: problem.id, format: 'html'
           line = issue_tracker.tracker.output.shift
           expect(line[1]).to eq("one #{problem.id} two")
@@ -268,10 +268,10 @@ describe ProblemsController, type: &#39;controller&#39; do
     end
  
     context "problem with issue" do
-      let(:err) { Fabricate(:err, :problem => Fabricate(:problem, :issue_link => "http://some.host")) }
+      let(:err) { Fabricate(:err, problem: Fabricate(:problem, issue_link: "http://some.host")) }
  
       before(:each) do
-        delete :unlink_issue, :app_id => err.app.id, :id => err.problem.id
+        delete :unlink_issue, app_id: err.app.id, id: err.problem.id
         err.problem.reload
       end
  
@@ -288,7 +288,7 @@ describe ProblemsController, type: &#39;controller&#39; do
       let(:err) { Fabricate :err }
  
       before(:each) do
-        delete :unlink_issue, :app_id => err.app.id, :id => err.problem.id
+        delete :unlink_issue, app_id: err.app.id, id: err.problem.id
         err.problem.reload
       end
  
@@ -301,33 +301,33 @@ describe ProblemsController, type: &#39;controller&#39; do
   describe "Bulk Actions" do
     before(:each) do
       sign_in user
-      @problem1 = Fabricate(:err, :problem => Fabricate(:problem, :resolved => true)).problem
-      @problem2 = Fabricate(:err, :problem => Fabricate(:problem, :resolved => false)).problem
+      @problem1 = Fabricate(:err, problem: Fabricate(:problem, resolved: true)).problem
+      @problem2 = Fabricate(:err, problem: Fabricate(:problem, resolved: false)).problem
     end
  
     context "POST /problems/merge_several" do
       it "should require at least two problems" do
-        post :merge_several, :problems => [@problem1.id.to_s]
+        post :merge_several, problems: [@problem1.id.to_s]
         expect(request.flash[:notice]).to eql I18n.t('controllers.problems.flash.need_two_errors_merge')
       end
  
       it "should merge the problems" do
-        expect(ProblemMerge).to receive(:new).and_return(double(:merge => true))
-        post :merge_several, :problems => [@problem1.id.to_s, @problem2.id.to_s]
+        expect(ProblemMerge).to receive(:new).and_return(double(merge: true))
+        post :merge_several, problems: [@problem1.id.to_s, @problem2.id.to_s]
       end
     end
  
     context "POST /problems/unmerge_several" do
       it "should require at least one problem" do
-        post :unmerge_several, :problems => []
+        post :unmerge_several, problems: []
         expect(request.flash[:notice]).to eql I18n.t('controllers.problems.flash.no_select_problem')
       end
  
       it "should unmerge a merged problem" do
         merged_problem = Problem.merge!(@problem1, @problem2)
         expect(merged_problem.errs.length).to eq 2
-        expect{
-          post :unmerge_several, :problems => [merged_problem.id.to_s]
+        expect {
+          post :unmerge_several, problems: [merged_problem.id.to_s]
           expect(merged_problem.reload.errs.length).to eq 1
         }.to change(Problem, :count).by(1)
       end
@@ -335,22 +335,22 @@ describe ProblemsController, type: &#39;controller&#39; do
  
     context "POST /problems/resolve_several" do
       it "should require at least one problem" do
-        post :resolve_several, :problems => []
+        post :resolve_several, problems: []
         expect(request.flash[:notice]).to eql I18n.t('controllers.problems.flash.no_select_problem')
       end
  
       it "should resolve the issue" do
-        post :resolve_several, :problems => [@problem2.id.to_s]
+        post :resolve_several, problems: [@problem2.id.to_s]
         expect(@problem2.reload.resolved?).to eq true
       end
  
       it "should display a message about 1 err" do
-        post :resolve_several, :problems => [@problem2.id.to_s]
+        post :resolve_several, problems: [@problem2.id.to_s]
         expect(flash[:success]).to match(/1 error has been resolved/)
       end
  
       it "should display a message about 2 errs" do
-        post :resolve_several, :problems => [@problem1.id.to_s, @problem2.id.to_s]
+        post :resolve_several, problems: [@problem1.id.to_s, @problem2.id.to_s]
         expect(flash[:success]).to match(/2 errors have been resolved/)
         expect(controller.selected_problems).to eq [@problem1, @problem2]
       end
@@ -358,20 +358,20 @@ describe ProblemsController, type: &#39;controller&#39; do
  
     context "POST /problems/unresolve_several" do
       it "should require at least one problem" do
-        post :unresolve_several, :problems => []
+        post :unresolve_several, problems: []
         expect(request.flash[:notice]).to eql I18n.t('controllers.problems.flash.no_select_problem')
       end
  
       it "should unresolve the issue" do
-        post :unresolve_several, :problems => [@problem1.id.to_s]
+        post :unresolve_several, problems: [@problem1.id.to_s]
         expect(@problem1.reload.resolved?).to eq false
       end
     end
  
     context "POST /problems/destroy_several" do
       it "should delete the problems" do
-        expect{
-          post :destroy_several, :problems => [@problem1.id.to_s]
+        expect {
+          post :destroy_several, problems: [@problem1.id.to_s]
         }.to change(Problem, :count).by(-1)
       end
     end
@@ -380,25 +380,25 @@ describe ProblemsController, type: &#39;controller&#39; do
       before do
         sign_in user
         @app      = Fabricate(:app)
-        @problem1 = Fabricate(:problem, :app => @app)
-        @problem2 = Fabricate(:problem, :app => @app)
+        @problem1 = Fabricate(:problem, app: @app)
+        @problem2 = Fabricate(:problem, app: @app)
       end
  
       it "destroys all problems" do
         expect {
-          post :destroy_all, :app_id => @app.id
+          post :destroy_all, app_id: @app.id
         }.to change(Problem, :count).by(-2)
         expect(controller.app).to eq @app
       end
  
       it "should display a message" do
-        put :destroy_all, :app_id => @app.id
+        put :destroy_all, app_id: @app.id
         expect(request.flash[:success]).to match(/been deleted/)
       end
  
       it "should redirect back to the app page" do
         request.env["HTTP_REFERER"] = edit_app_path(@app)
-        put :destroy_all, :app_id => @app.id
+        put :destroy_all, app_id: @app.id
         expect(response).to redirect_to(edit_app_path(@app))
       end
     end
 describe SiteConfigController, type: 'controller' do
   it_requires_admin_privileges for: {
-    index: :get,
+    index:  :get,
     update: :put
   }
  
@@ -18,7 +18,7 @@ describe SiteConfigController, type: &#39;controller&#39; do
     it 'updates' do
       put :update, site_config: {
         notice_fingerprinter_attributes: {
-          backtrace_lines: 3,
+          backtrace_lines:  3,
           environment_name: false
         }
       }
@@ -4,9 +4,9 @@ describe Users::OmniauthCallbacksController, type: &#39;controller&#39; do
     request.env["devise.mapping"] = Devise.mappings[:user]
     env = {
       "omniauth.auth" => Hashie::Mash.new(
-        :provider => 'github',
-        :extra => { :raw_info => { :login => login }},
-        :credentials => { :token => token }
+        provider:    'github',
+        extra:       { raw_info: { login: login } },
+        credentials: { token: token }
       )
     }
     allow(@controller).to receive(:env).and_return(env)
@@ -18,7 +18,7 @@ describe Users::OmniauthCallbacksController, type: &#39;controller&#39; do
     end
  
     it "should show an error if another user already has that github login" do
-      Fabricate(:user, :github_login => "existing_user")
+      Fabricate(:user, github_login: "existing_user")
       stub_env_for_github_omniauth("existing_user")
       get :github
  
 describe UsersController, type: 'controller' do
   it_requires_authentication
-  it_requires_admin_privileges :for => {
-    :index    => :get,
-    :show     => :get,
-    :new      => :get,
-    :create   => :post,
-    :destroy  => :delete
+  it_requires_admin_privileges for: {
+    index:   :get,
+    show:    :get,
+    new:     :get,
+    create:  :post,
+    destroy: :delete
   }
  
   let(:admin) { Fabricate(:admin) }
@@ -23,14 +23,14 @@ describe UsersController, type: &#39;controller&#39; do
  
     context "GET /users/:other_id/edit" do
       it "redirects to the home page" do
-        get :edit, :id => other_user.id
+        get :edit, id: other_user.id
         expect(response).to redirect_to(root_path)
       end
     end
  
     context "GET /users/:my_id/edit" do
       it 'finds the user' do
-        get :edit, :id => user.id
+        get :edit, id: user.id
         expect(controller.user).to eq(user)
         expect(response).to render_template 'edit'
       end
@@ -38,7 +38,7 @@ describe UsersController, type: &#39;controller&#39; do
  
     context "PUT /users/:other_id" do
       it "redirects to the home page" do
-        put :update, :id => other_user.id
+        put :update, id: other_user.id
         expect(response).to redirect_to(root_path)
       end
     end
@@ -46,47 +46,47 @@ describe UsersController, type: &#39;controller&#39; do
     context "PUT /users/:my_id/id" do
       context "when the update is successful" do
         it "sets a message to display" do
-          put :update, :id => user.to_param, :user => {:name => 'Kermit'}
+          put :update, id: user.to_param, user: { name: 'Kermit' }
           expect(request.flash[:success]).to include('updated')
         end
  
         it "redirects to the user's page" do
-          put :update, :id => user.to_param, :user => {:name => 'Kermit'}
+          put :update, id: user.to_param, user: { name: 'Kermit' }
           expect(response).to redirect_to(user_path(user))
         end
  
         it "should not be able to become an admin" do
           expect {
-            put :update, :id => user.to_param, :user => {:admin => true}
+            put :update, id: user.to_param, user: { admin: true }
           }.to_not change {
             user.reload.admin
           }.from(false)
         end
  
         it "should be able to set per_page option" do
-          put :update, :id => user.to_param, :user => {:per_page => 555}
+          put :update, id: user.to_param, user: { per_page: 555 }
           expect(user.reload.per_page).to eq 555
         end
  
         it "should be able to set time_zone option" do
-          put :update, :id => user.to_param, :user => {:time_zone => "Warsaw"}
+          put :update, id: user.to_param, user: { time_zone: "Warsaw" }
           expect(user.reload.time_zone).to eq "Warsaw"
         end
  
         it "should be able to not set github_login option" do
-          put :update, :id => user.to_param, :user => {:github_login => " "}
+          put :update, id: user.to_param, user: { github_login: " " }
           expect(user.reload.github_login).to eq nil
         end
  
         it "should be able to set github_login option" do
-          put :update, :id => user.to_param, :user => {:github_login => "awesome_name"}
+          put :update, id: user.to_param, user: { github_login: "awesome_name" }
           expect(user.reload.github_login).to eq "awesome_name"
         end
       end
  
       context "when the update is unsuccessful" do
         it "renders the edit page" do
-          put :update, :id => user.to_param, :user => {:name => nil}
+          put :update, id: user.to_param, user: { name: nil }
           expect(response).to render_template(:edit)
         end
       end
@@ -109,7 +109,7 @@ describe UsersController, type: &#39;controller&#39; do
  
     context "GET /users/:id" do
       it 'finds the user' do
-        get :show, :id => user.id
+        get :show, id: user.id
         expect(controller.user).to eq user
       end
     end
@@ -124,14 +124,14 @@ describe UsersController, type: &#39;controller&#39; do
  
     context "GET /users/:id/edit" do
       it 'finds the user' do
-        get :edit, :id => user.id
+        get :edit, id: user.id
         expect(controller.user).to eq user
       end
     end
  
     context "POST /users" do
       context "when the create is successful" do
-        let(:attrs) { {:user => Fabricate.to_params(:user)} }
+        let(:attrs) { { user: Fabricate.to_params(:user) } }
  
         it "sets a message to display" do
           post :create, attrs
@@ -166,7 +166,7 @@ describe UsersController, type: &#39;controller&#39; do
         end
  
         it "renders the new page" do
-          post :create, :user => { :username => 'foo' }
+          post :create, user: { username: 'foo' }
           expect(response).to render_template(:new)
         end
       end
@@ -175,20 +175,20 @@ describe UsersController, type: &#39;controller&#39; do
     context "PUT /users/:id" do
       context "when the update is successful" do
         before {
-          put :update, :id => user.to_param, :user => user_params
+          put :update, id: user.to_param, user: user_params
         }
  
         context "with normal params" do
-          let(:user_params) { {:name => 'Kermit'} }
+          let(:user_params) { { name: 'Kermit' } }
           it "sets a message to display" do
-            expect(request.flash[:success]).to eq I18n.t('controllers.users.flash.update.success', :name => user.reload.name)
+            expect(request.flash[:success]).to eq I18n.t('controllers.users.flash.update.success', name: user.reload.name)
             expect(response).to redirect_to(user_path(user))
           end
         end
       end
       context "when the update is unsuccessful" do
         it "renders the edit page" do
-          put :update, :id => user.to_param, :user => {:name => nil}
+          put :update, id: user.to_param, user: { name: nil }
           expect(response).to render_template(:edit)
         end
       end
@@ -196,15 +196,15 @@ describe UsersController, type: &#39;controller&#39; do
  
     context "DELETE /users/:id" do
       context "with a destroy success" do
-        let(:user_destroy) { double(:destroy => true) }
+        let(:user_destroy) { double(destroy: true) }
  
         before {
           expect(UserDestroy).to receive(:new).with(user).and_return(user_destroy)
-          delete :destroy, :id => user.id
+          delete :destroy, id: user.id
         }
  
         it 'should destroy user' do
-          expect(request.flash[:success]).to eq I18n.t('controllers.users.flash.destroy.success', :name => user.name)
+          expect(request.flash[:success]).to eq I18n.t('controllers.users.flash.destroy.success', name: user.name)
           expect(response).to redirect_to(users_path)
         end
       end
@@ -212,7 +212,7 @@ describe UsersController, type: &#39;controller&#39; do
       context "with trying destroy himself" do
         before {
           expect(UserDestroy).to_not receive(:new)
-          delete :destroy, :id => admin.id
+          delete :destroy, id: admin.id
         }
  
         it 'should not destroy user' do
@@ -230,14 +230,14 @@ describe UsersController, type: &#39;controller&#39; do
             ActionController::Parameters.new(user_param)
           )
         }
-        let(:user_param) { {'user' => { :name => 'foo', :admin => true }} }
+        let(:user_param) { { 'user' => { name: 'foo', admin: true } } }
         it 'not have admin field' do
-          expect(controller.send(:user_params)).to eq({'name' => 'foo'})
+          expect(controller.send(:user_params)).to eq('name' => 'foo')
         end
         context "with password and password_confirmation empty?" do
-          let(:user_param) { {'user' => { :name => 'foo', 'password' => '', 'password_confirmation' => '' }} }
+          let(:user_param) { { 'user' => { :name => 'foo', 'password' => '', 'password_confirmation' => '' } } }
           it 'not have password and password_confirmation field' do
-            expect(controller.send(:user_params)).to eq({'name' => 'foo'})
+            expect(controller.send(:user_params)).to eq('name' => 'foo')
           end
         end
       end
@@ -15,12 +15,12 @@ describe WatchersController, type: &#39;controller&#39; do
       let(:watcher) { app.watchers.first }
  
       before(:each) do
-        delete :destroy, :app_id => app.id, :id => watcher.user.id.to_s
+        delete :destroy, app_id: app.id, id: watcher.user.id.to_s
         problem.reload
       end
  
       it "should delete the watcher" do
-        expect(app.watchers.detect{|w| w.id.to_s == watcher.id }).to be nil
+        expect(app.watchers.detect { |w| w.id.to_s == watcher.id }).to be nil
       end
  
       it "should redirect to app page" do
@@ -34,7 +34,7 @@ describe WatchersController, type: &#39;controller&#39; do
  
     context "successful watcher update" do
       before(:each) do
-        put :update, :app_id => app.id, :id => user.id.to_s
+        put :update, app_id: app.id, id: user.id.to_s
         problem.reload
       end
  
 describe AppDecorator do
   describe "#email_at_notices" do
     it 'return the list separate by comma' do
-      expect(AppDecorator.new(double(:email_at_notices => [2, 3])).email_at_notices).to eql '2, 3'
+      expect(AppDecorator.new(double(email_at_notices: [2, 3])).email_at_notices).to eql '2, 3'
     end
   end
  
   describe "#notify_user_display" do
     it 'return display:none if notify' do
-      expect(AppDecorator.new(double(:notify_all_users => true)).notify_user_display).to eql 'display: none;'
+      expect(AppDecorator.new(double(notify_all_users: true)).notify_user_display).to eql 'display: none;'
     end
  
     it 'return blank if no notify' do
-      expect(AppDecorator.new(double(:notify_all_users => false)).notify_user_display).to eql ''
+      expect(AppDecorator.new(double(notify_all_users: false)).notify_user_display).to eql ''
     end
   end
  
   describe "#notify_err_display" do
     it 'return display:none if no notify' do
-      expect(AppDecorator.new(double(:notify_on_errs => false)).notify_err_display).to eql 'display: none;'
+      expect(AppDecorator.new(double(notify_on_errs: false)).notify_err_display).to eql 'display: none;'
     end
  
     it 'return blank if no notify' do
-      expect(AppDecorator.new(double(:notify_on_errs => true)).notify_err_display).to eql ''
+      expect(AppDecorator.new(double(notify_on_errs: true)).notify_err_display).to eql ''
     end
   end
 end
@@ -3,22 +3,22 @@ describe BacktraceDecorator, type: :decorator do
     described_class.new(Backtrace.new(
       lines: [
         { number: 131,
-          file: '[PROJECT_ROOT]app/controllers/accounts_controller.rb',
+          file:   '[PROJECT_ROOT]app/controllers/accounts_controller.rb',
           method: :update_preferences },
         { number: 61,
-          file: '[PROJECT_ROOT]app/controllers/application_controller.rb',
+          file:   '[PROJECT_ROOT]app/controllers/application_controller.rb',
           method: :call },
         { number: 182,
-          file: '[GEM_ROOT]activesupport-2.3.18/lib/active_support/callbacks.rb',
+          file:   '[GEM_ROOT]activesupport-2.3.18/lib/active_support/callbacks.rb',
           method: :call },
         { number: 384,
-          file: '[PROJECT_ROOT]app/models/account.rb',
+          file:   '[PROJECT_ROOT]app/models/account.rb',
           method: :update_server_tag_scope },
         { number: 182,
-          file: '[GEM_ROOT]activesupport-2.3.18/lib/active_support/callbacks.rb',
+          file:   '[GEM_ROOT]activesupport-2.3.18/lib/active_support/callbacks.rb',
           method: :evaluate_method },
         { number: 23,
-          file: '/home/rails/library/current/vendor/bundle/ruby/2.1.0/bin/rainbows',
+          file:   '/home/rails/library/current/vendor/bundle/ruby/2.1.0/bin/rainbows',
           method: '<main>' }
       ]
     ))
@@ -2,13 +2,13 @@ describe BacktraceLineDecorator, type: :decorator do
   let(:backtrace_line) do
     described_class.new(
       number: 884,
-      file: '/path/to/file/ea315ea4.rb',
+      file:   '/path/to/file/ea315ea4.rb',
       method: :instance_eval)
   end
   let(:backtrace_line_in_app) do
     described_class.new(
       number: 884,
-      file: '[PROJECT_ROOT]/path/to/file/ea315ea4.rb',
+      file:   '[PROJECT_ROOT]/path/to/file/ea315ea4.rb',
       method: :instance_eval)
   end
   let(:app) { Fabricate(:app, github_repo: 'foo/bar') }
 describe IssueTrackerDecorator do
   let(:fake_tracker) do
     klass = Class.new(ErrbitPlugin::IssueTracker) {
-      def self.label; 'fake'; end
-      def self.note; 'a note'; end
+      def self.label
+        'fake'
+      end
+
+      def self.note
+        'a note'
+      end
  
       def self.fields
         {
-          :foo => {:label => 'foo'},
-          :bar => {:label => 'bar'}
+          foo: { label: 'foo' },
+          bar: { label: 'bar' }
         }
       end
  
-      def configured?; true; end
+      def configured?
+        true
+      end
     }
     klass.new 'nothing special'
   end
 describe IssueTrackerFieldDecorator do
   describe "#label" do
     it 'return the label of field_info by default' do
-      expect(IssueTrackerFieldDecorator.new(:foo, {:label => 'hello'}).label).to eq 'hello'
+      expect(IssueTrackerFieldDecorator.new(:foo, label: 'hello').label).to eq 'hello'
     end
  
     it 'return the key of field if no label define' do
 describe IssueTrackerDecorator do
   let(:fake_tracker_class) do
     klass = Class.new(ErrbitPlugin::IssueTracker) do
-      def self.label; 'fake'; end
-      def self.note; 'a note'; end
+      def self.label
+        'fake'
+      end
+
+      def self.note
+        'a note'
+      end
  
       def self.fields
         {
-          :foo => {:label => 'foo'},
-          :bar => {:label => 'bar'}
+          foo: { label: 'foo' },
+          bar: { label: 'bar' }
         }
       end
  
       def self.icons
         {
           one: ['text/plain', 'all your base are belong to us'],
-          two: ['application/xml', '<root></root>'],
+          two: ['application/xml', '<root></root>']
         }
       end
     end
  
-    allow(ErrbitPlugin::Registry).to receive(:issue_trackers).and_return({
-      fake: klass
-    })
+    allow(ErrbitPlugin::Registry).to receive(:issue_trackers).and_return(fake: klass)
  
     klass
   end
@@ -36,11 +39,11 @@ describe IssueTrackerDecorator do
   end
  
   describe "#fields" do
-    it 'return all Fields define decorate' do
+    it 'return all FIELDS define decorate' do
       decorator.fields do |itf|
         expect(itf).to be_a(IssueTrackerFieldDecorator)
         expect([:foo, :bar]).to be_include(itf.object)
-        expect([{:label => 'foo'}, {:label => 'bar'}]).to be_include(itf.field_info)
+        expect([{ label: 'foo' }, { label: 'bar' }]).to be_include(itf.field_info)
       end
     end
   end
@@ -2,13 +2,13 @@ describe WatcherDecorator do
   describe "#email_choosen" do
     context "with email define" do
       it 'return blank' do
-        expect(WatcherDecorator.new(double(:email => 'foo')).email_choosen).to eql ''
+        expect(WatcherDecorator.new(double(email: 'foo')).email_choosen).to eql ''
       end
     end
  
     context "without email define" do
       it 'return choosen' do
-        expect(WatcherDecorator.new(double(:email => '')).email_choosen).to eql 'chosen'
+        expect(WatcherDecorator.new(double(email: '')).email_choosen).to eql 'chosen'
       end
     end
   end
...	...	@@ -29,11 +29,20 @@ Style/AccessModifierIndentation:
29	29	Style/AlignParameters:
30	30	EnforcedStyle: with_fixed_indentation
31	31
	32	+Style/DotPosition:
	33	+ # Support code copy/paste into console
	34	+ EnforcedStyle: trailing
	35	+
32	36	Style/IndentHash:
33	37	EnforcedStyle: consistent
34	38
	39	+Style/AlignHash:
	40	+ EnforcedHashRocketStyle: table
	41	+ EnforcedColonStyle: table
	42	+
35	43	Style/SpaceAroundOperators:
36	44	MultiSpaceAllowedForOperators:
37	45	- '='
38	46	- '=>'
39	47	- '\|\|='
	48	+ - 'then'
...	...
...	...	@@ -53,29 +53,12 @@ Rails/Validation:
53	53	- 'app/models/problem.rb'
54	54	- 'app/models/user.rb'
55	55
56		-# Offense count: 8
57		-# Cop supports --auto-correct.
58		-# Configuration parameters: EnforcedHashRocketStyle, EnforcedColonStyle, EnforcedLastArgumentHashStyle, SupportedLastArgumentHashStyles.
59		-Style/AlignHash:
60		- Exclude:
61		- - 'app/models/app.rb'
62		- - 'spec/controllers/apps_controller_spec.rb'
63		- - 'spec/controllers/comments_controller_spec.rb'
64		- - 'spec/controllers/problems_controller_spec.rb'
65		- - 'spec/models/problem_spec.rb'
66		-
67	56	# Offense count: 105
68	57	# Cop supports --auto-correct.
69	58	# Configuration parameters: EnforcedStyle, SupportedStyles, ProceduralMethods, FunctionalMethods, IgnoredMethods.
70	59	Style/BlockDelimiters:
71	60	Enabled: false
72	61
73		-# Offense count: 49
74		-# Cop supports --auto-correct.
75		-# Configuration parameters: EnforcedStyle, SupportedStyles.
76		-Style/BracesAroundHashParameters:
77		- Enabled: false
78		-
79	62	# Offense count: 15
80	63	# Configuration parameters: EnforcedStyle, SupportedStyles.
81	64	Style/ClassAndModuleChildren:
...	...	@@ -96,31 +79,11 @@ Style/ClassAndModuleChildren:
96	79	- 'app/models/notification_services/webhook_service.rb'
97	80	- 'config/initializers/overrides.rb'
98	81
99		-# Offense count: 23
100		-Style/ConstantName:
101		- Exclude:
102		- - 'app/models/notification_service.rb'
103		- - 'app/models/notification_services/campfire_service.rb'
104		- - 'app/models/notification_services/flowdock_service.rb'
105		- - 'app/models/notification_services/gtalk_service.rb'
106		- - 'app/models/notification_services/hipchat_service.rb'
107		- - 'app/models/notification_services/hoiio_service.rb'
108		- - 'app/models/notification_services/hubot_service.rb'
109		- - 'app/models/notification_services/pushover_service.rb'
110		- - 'app/models/notification_services/slack_service.rb'
111		- - 'app/models/notification_services/webhook_service.rb'
112		-
113	82	# Offense count: 70
114	83	# Configuration parameters: Exclude.
115	84	Style/Documentation:
116	85	Enabled: false
117	86
118		-# Offense count: 9
119		-# Cop supports --auto-correct.
120		-# Configuration parameters: EnforcedStyle, SupportedStyles.
121		-Style/DotPosition:
122		- Enabled: false
123		-
124	87	# Offense count: 5
125	88	Style/EachWithObject:
126	89	Exclude:
...	...	@@ -161,12 +124,6 @@ Style/FormatString:
161	124	Style/GuardClause:
162	125	Enabled: false
163	126
164		-# Offense count: 1168
165		-# Cop supports --auto-correct.
166		-# Configuration parameters: EnforcedStyle, SupportedStyles, UseHashRocketsWithSymbolValues.
167		-Style/HashSyntax:
168		- Enabled: false
169		-
170	127	# Offense count: 6
171	128	# Cop supports --auto-correct.
172	129	# Configuration parameters: MaxLineLength.
...	...	@@ -178,15 +135,6 @@ Style/IfUnlessModifier:
178	135	- 'app/models/notice.rb'
179	136	- 'app/models/user.rb'
180	137
181		-# Offense count: 6
182		-# Cop supports --auto-correct.
183		-Style/Lambda:
184		- Exclude:
185		- - 'app/models/app.rb'
186		- - 'app/models/problem.rb'
187		- - 'config/load.rb'
188		- - 'spec/models/notice_observer_spec.rb'
189		-
190	138	# Offense count: 17
191	139	# Cop supports --auto-correct.
192	140	Style/LeadingCommentSpace:
...	...	@@ -200,17 +148,6 @@ Style/LeadingCommentSpace:
200	148	- 'spec/models/notification_service/hoiio_service_spec.rb'
201	149	- 'spec/models/notification_service/pushover_service_spec.rb'
202	150
203		-# Offense count: 7
204		-# Cop supports --auto-correct.
205		-Style/MultilineBlockLayout:
206		- Exclude:
207		- - 'lib/tasks/errbit/demo.rake'
208		- - 'spec/fabricators/issue_tracker_fabricator.rb'
209		- - 'spec/interactors/user_destroy_spec.rb'
210		- - 'spec/models/app_spec.rb'
211		- - 'spec/models/error_report_spec.rb'
212		- - 'spec/views/apps/show.atom.builder_spec.rb'
213		-
214	151	# Offense count: 10
215	152	# Cop supports --auto-correct.
216	153	# Configuration parameters: EnforcedStyle, SupportedStyles.
...	...	@@ -228,71 +165,11 @@ Style/PercentLiteralDelimiters:
228	165	- 'spec/models/notice_spec.rb'
229	166
230	167	# Offense count: 1
231		-# Cop supports --auto-correct.
232		-Style/PerlBackrefs:
233		- Exclude:
234		- - 'spec/acceptance/reset_password_token.rb'
235		-
236		-# Offense count: 3
237		-# Cop supports --auto-correct.
238		-Style/Proc:
239		- Exclude:
240		- - 'app/models/notice.rb'
241		- - 'app/models/problem.rb'
242		-
243		-# Offense count: 2
244		-# Configuration parameters: EnforcedStyle, SupportedStyles.
245		-Style/RaiseArgs:
246		- Enabled: false
247		-
248		-# Offense count: 3
249		-# Cop supports --auto-correct.
250		-# Configuration parameters: EnforcedStyle, SupportedStyles, AllowInnerSlashes.
251		-Style/RegexpLiteral:
252		- Exclude:
253		- - 'app/models/app.rb'
254		- - 'config/initializers/ssl_enforcer.rb'
255		- - 'config/load.rb'
256		-
257		-# Offense count: 2
258		-# Cop supports --auto-correct.
259		-# Configuration parameters: AllowAsExpressionSeparator.
260		-Style/Semicolon:
261		- Exclude:
262		- - 'app/decorators/issue_tracker_type_decorator.rb'
263		- - 'app/models/notice_fingerprinter.rb'
264		-
265		-# Offense count: 6
266		-# Cop supports --auto-correct.
267		-# Configuration parameters: EnforcedStyle, SupportedStyles.
268		-Style/SignalException:
269		- Exclude:
270		- - 'app/controllers/notices_controller.rb'
271		- - 'app/interactors/problem_merge.rb'
272		- - 'app/models/problem.rb'
273		- - 'lib/airbrake_api/v3/notice_parser.rb'
274		- - 'lib/hoptoad.rb'
275		-
276		-# Offense count: 4
277	168	# Configuration parameters: Methods.
278	169	Style/SingleLineBlockParams:
279	170	Exclude:
280		- - 'app/helpers/application_helper.rb'
281		- - 'app/helpers/form_helper.rb'
282		- - 'app/models/notice_fingerprinter.rb'
283	171	- 'lib/hoptoad/v2.rb'
284	172
285		-# Offense count: 19
286		-# Cop supports --auto-correct.
287		-# Configuration parameters: AllowIfMethodIsEmpty.
288		-Style/SingleLineMethods:
289		- Exclude:
290		- - 'app/models/notification_service.rb'
291		- - 'spec/decorators/issue_tracker_decorator_spec.rb'
292		- - 'spec/decorators/issue_tracker_type_decorator_spec.rb'
293		- - 'spec/errbit_plugin/mock_issue_tracker.rb'
294		- - 'spec/views/problems/show.html.haml_spec.rb'
295		-
296	173	# Offense count: 11
297	174	# Cop supports --auto-correct.
298	175	Style/SingleSpaceBeforeFirstArg:
...	...	@@ -304,90 +181,8 @@ Style/SingleSpaceBeforeFirstArg:
304	181	- 'spec/fabricators/notice_fabricator.rb'
305	182	- 'spec/fabricators/user_fabricator.rb'
306	183
307		-# Offense count: 2
308		-# Cop supports --auto-correct.
309		-# Configuration parameters: EnforcedStyle, SupportedStyles.
310		-Style/SpaceAroundEqualsInParameterDefault:
311		- Enabled: false
312		-
313		-# Offense count: 27
314		-# Cop supports --auto-correct.
315		-# Configuration parameters: EnforcedStyle, SupportedStyles.
316		-Style/SpaceBeforeBlockBraces:
317		- Enabled: false
318		-
319		-# Offense count: 3
320		-# Cop supports --auto-correct.
321		-Style/SpaceBeforeComment:
322		- Exclude:
323		- - 'spec/controllers/api/v1/problems_controller_spec.rb'
324		-
325		-# Offense count: 75
326		-# Cop supports --auto-correct.
327		-# Configuration parameters: EnforcedStyle, SupportedStyles, EnforcedStyleForEmptyBraces, SpaceBeforeBlockParameters.
328		-Style/SpaceInsideBlockBraces:
329		- Enabled: false
330		-
331		-# Offense count: 26
332		-# Cop supports --auto-correct.
333		-Style/SpaceInsideBrackets:
334		- Exclude:
335		- - 'app/helpers/apps_helper.rb'
336		- - 'app/models/issue.rb'
337		- - 'app/models/notice_fingerprinter.rb'
338		- - 'app/models/notification_services/gtalk_service.rb'
339		- - 'config/initializers/devise.rb'
340		- - 'lib/tasks/errbit/database.rake'
341		- - 'spec/lib/configurator_spec.rb'
342		- - 'spec/models/issue_spec.rb'
343		-
344		-# Offense count: 189
345		-# Cop supports --auto-correct.
346		-# Configuration parameters: EnforcedStyle, EnforcedStyleForEmptyBraces, SupportedStyles.
347		-Style/SpaceInsideHashLiteralBraces:
348		- Enabled: false
349		-
350	184	# Offense count: 873
351	185	# Cop supports --auto-correct.
352	186	# Configuration parameters: EnforcedStyle, SupportedStyles.
353	187	Style/StringLiterals:
354	188	Enabled: false
355		-
356		-# Offense count: 3
357		-# Cop supports --auto-correct.
358		-# Configuration parameters: IgnoredMethods.
359		-Style/SymbolProc:
360		- Exclude:
361		- - 'app/controllers/users/omniauth_callbacks_controller.rb'
362		- - 'app/models/deploy.rb'
363		- - 'spec/models/deploy_spec.rb'
364		-
365		-# Offense count: 13
366		-# Cop supports --auto-correct.
367		-# Configuration parameters: EnforcedStyleForMultiline, SupportedStyles.
368		-Style/TrailingComma:
369		- Exclude:
370		- - 'app/controllers/deploys_controller.rb'
371		- - 'app/helpers/problems_helper.rb'
372		- - 'app/models/notification_services/hipchat_service.rb'
373		- - 'app/models/problem.rb'
374		- - 'config/deploy.example.rb'
375		- - 'config/initializers/action_mailer.rb'
376		- - 'config/load.rb'
377		- - 'lib/tasks/errbit/demo.rake'
378		- - 'spec/decorators/issue_tracker_type_decorator_spec.rb'
379		- - 'spec/initializers/action_mailer_spec.rb'
380		- - 'spec/initializers/devise_spec.rb'
381		-
382		-# Offense count: 9
383		-# Cop supports --auto-correct.
384		-Style/TrailingWhitespace:
385		- Exclude:
386		- - 'app/models/notification_service.rb'
387		- - 'app/models/notification_services/gtalk_service.rb'
388		- - 'config/initializers/cve-2013-0156.rb'
389		-
390		-# Offense count: 2
391		-Style/UnlessElse:
392		- Exclude:
393		- - 'app/controllers/problems_searcher.rb'
...	...
...	...	@@ -29,8 +29,8 @@ gem 'htmlentities'
29	29	gem 'kaminari', '>= 0.14.1'
30	30	gem 'mongoid', '5.0.0'
31	31	gem 'mongoid_rails_migrations'
32		-gem 'rack-ssl', :require => 'rack/ssl' # force SSL
33		-gem 'rack-ssl-enforcer', :require => false
	32	+gem 'rack-ssl', require: 'rack/ssl' # force SSL
	33	+gem 'rack-ssl-enforcer', require: false
34	34	gem 'rails_autolink'
35	35	gem 'useragent'
36	36
...	...	@@ -44,7 +44,7 @@ gem 'campy'
44	44	# Hipchat
45	45	gem 'hipchat'
46	46	# Google Talk
47		-gem 'xmpp4r', :require => ["xmpp4r", "xmpp4r/muc"]
	47	+gem 'xmpp4r', require: ["xmpp4r", "xmpp4r/muc"]
48	48	# Hoiio (SMS)
49	49	gem 'hoi'
50	50	# Pushover (iOS Push notifications)
...	...	@@ -64,7 +64,7 @@ gem 'yajl-ruby', platform: 'ruby'
64	64	gem 'json', platform: 'jruby'
65	65
66	66	group :development, :test do
67		- gem 'airbrake', :require => false
	67	+ gem 'airbrake', require: false
68	68	gem 'pry-rails'
69	69	gem 'pry-byebug', platforms: [:mri]
70	70	gem 'quiet_assets'
...	...	@@ -103,7 +103,7 @@ group :heroku, :production do
103	103	gem 'unicorn', require: false, platform: 'ruby'
104	104	end
105	105
106		-gem 'therubyracer', :platform => :ruby # C Ruby (MRI) or Rubinius, but NOT Windows
	106	+gem 'therubyracer', platform: :ruby # C Ruby (MRI) or Rubinius, but NOT Windows
107	107	gem 'sass-rails'
108	108	gem 'uglifier'
109	109	# We can't upgrade because not compatible to jquery >= 1.9.
...	...
...	...	@@ -8,7 +8,7 @@ Rails.application.load_tasks
8	8	begin
9	9	require 'rspec/core/rake_task'
10	10	RSpec::Core::RakeTask.new(:spec)
11		- task :default => :spec
	11	+ task default: :spec
12	12	rescue LoadError
13	13	warn "Notice: no rspec tasks available in this environment"
14	14	end
...	...
...	...	@@ -8,16 +8,16 @@ class Api::V1::NoticesController < ApplicationController
8	8	if params.key?(:start_date) && params.key?(:end_date)
9	9	start_date = Time.zone.parse(params[:start_date]).utc
10	10	end_date = Time.zone.parse(params[:end_date]).utc
11		- query = {:created_at => {"$lte" => end_date, "$gte" => start_date}}
	11	+ query = { created_at: { "$lte" => end_date, "$gte" => start_date } }
12	12	end
13	13
14	14	results = benchmark("[api/v1/notices_controller] query time") do
15		- Notice.where(query).with(:consistency => :strong).only(fields).to_a
	15	+ Notice.where(query).with(consistency: :strong).only(fields).to_a
16	16	end
17	17
18	18	respond_to do \|format\|
19		- format.any(:html, :json) { render :json => JSON.dump(results) } # render JSON if no extension specified on path
20		- format.xml { render :xml => results }
	19	+ format.any(:html, :json) { render json: JSON.dump(results) } # render JSON if no extension specified on path
	20	+ format.xml { render xml: results }
21	21	end
22	22	end
23	23	end
...	...
...	...	@@ -13,8 +13,8 @@ class Api::V1::ProblemsController < ApplicationController
13	13	end
14	14
15	15	respond_to do \|format\|
16		- format.any(:html, :json) { render :json => result } # render JSON if no extension specified on path
17		- format.xml { render :xml => result }
	16	+ format.any(:html, :json) { render json: result } # render JSON if no extension specified on path
	17	+ format.xml { render xml: result }
18	18	end
19	19	end
20	20
...	...	@@ -24,16 +24,16 @@ class Api::V1::ProblemsController < ApplicationController
24	24	if params.key?(:start_date) && params.key?(:end_date)
25	25	start_date = Time.parse(params[:start_date]).utc
26	26	end_date = Time.parse(params[:end_date]).utc
27		- query = {:first_notice_at => {"$lte" => end_date}, "$or" => [{:resolved_at => nil}, {:resolved_at => {"$gte" => start_date}}]}
	27	+ query = { :first_notice_at => { "$lte" => end_date }, "$or" => [{ resolved_at: nil }, { resolved_at: { "$gte" => start_date } }] }
28	28	end
29	29
30	30	results = benchmark("[api/v1/problems_controller/index] query time") do
31		- Problem.where(query).with(:consistency => :strong).only(FIELDS).to_a
	31	+ Problem.where(query).with(consistency: :strong).only(FIELDS).to_a
32	32	end
33	33
34	34	respond_to do \|format\|
35		- format.any(:html, :json) { render :json => JSON.dump(results) } # render JSON if no extension specified on path
36		- format.xml { render :xml => results }
	35	+ format.any(:html, :json) { render json: JSON.dump(results) } # render JSON if no extension specified on path
	36	+ format.xml { render xml: results }
37	37	end
38	38	end
39	39	end
...	...
...	...	@@ -11,21 +11,21 @@ class Api::V1::StatsController < ApplicationController
11	11	end
12	12
13	13	stats = {
14		- :name => @app.name,
15		- :id => @app.id,
16		- :last_error_time => @last_error_time,
17		- :unresolved_errors => @app.unresolved_count
	14	+ name: @app.name,
	15	+ id: @app.id,
	16	+ last_error_time: @last_error_time,
	17	+ unresolved_errors: @app.unresolved_count
18	18	}
19	19
20	20	respond_to do \|format\|
21		- format.any(:html, :json) { render :json => JSON.dump(stats) } # render JSON if no extension specified on path
22		- format.xml { render :xml => stats }
	21	+ format.any(:html, :json) { render json: JSON.dump(stats) } # render JSON if no extension specified on path
	22	+ format.xml { render xml: stats }
23	23	end
24	24	end
25	25
26	26	protected def require_api_key_or_authenticate_user!
27	27	if params[:api_key].present?
28		- if (@app = App.where(:api_key => params[:api_key]).first)
	28	+ if (@app = App.where(api_key: params[:api_key]).first)
29	29	return true
30	30	end
31	31	end
...	...
...	...	@@ -15,7 +15,7 @@ class Api::V3::NoticesController < ApplicationController
15	15	if report.should_keep?
16	16	report.generate_notice!
17	17	render json: {
18		- id: report.notice.id,
	18	+ id: report.notice.id,
19	19	url: app_problem_url(
20	20	report.app,
21	21	report.error.problem_id)
...	...
...	...	@@ -5,7 +5,7 @@ class ApplicationController < ActionController::Base
5	5	before_action :authenticate_user!
6	6	before_action :set_time_zone
7	7
8		- rescue_from ActionController::RedirectBackError, :with => :redirect_to_root
	8	+ rescue_from ActionController::RedirectBackError, with: :redirect_to_root
9	9
10	10	class StrongParametersWithEagerAttributesStrategy < DecentExposure::StrongParametersStrategy
11	11	def assign_attributes?
...	...
1	1	class AppsController < ApplicationController
2	2	include ProblemsSearcher
3	3
4		- before_action :require_admin!, :except => [:index, :show]
5		- before_action :parse_email_at_notices_or_set_default, :only => [:create, :update]
6		- before_action :parse_notice_at_notices_or_set_default, :only => [:create, :update]
	4	+ before_action :require_admin!, except: [:index, :show]
	5	+ before_action :parse_email_at_notices_or_set_default, only: [:create, :update]
	6	+ before_action :parse_notice_at_notices_or_set_default, only: [:create, :update]
7	7	respond_to :html
8	8
9	9	expose(:app_scope) { App }
...	...	@@ -39,7 +39,7 @@ class AppsController < ApplicationController
39	39	}
40	40
41	41	expose(:users) {
42		- User.all.sort_by {\|u\| u.name.downcase }
	42	+ User.all.sort_by { \|u\| u.name.downcase }
43	43	}
44	44
45	45	def index; end
...	...	@@ -55,7 +55,7 @@ class AppsController < ApplicationController
55	55	def create
56	56	initialize_subclassed_notification_service
57	57	if app.save
58		- redirect_to app_url(app), :flash => { :success => I18n.t('controllers.apps.flash.create.success') }
	58	+ redirect_to app_url(app), flash: { success: I18n.t('controllers.apps.flash.create.success') }
59	59	else
60	60	flash[:error] = I18n.t('controllers.apps.flash.create.error')
61	61	render :new
...	...	@@ -65,7 +65,7 @@ class AppsController < ApplicationController
65	65	def update
66	66	initialize_subclassed_notification_service
67	67	if app.save
68		- redirect_to app_url(app), :flash => { :success => I18n.t('controllers.apps.flash.update.success') }
	68	+ redirect_to app_url(app), flash: { success: I18n.t('controllers.apps.flash.update.success') }
69	69	else
70	70	flash[:error] = I18n.t('controllers.apps.flash.update.error')
71	71	render :edit
...	...	@@ -78,7 +78,7 @@ class AppsController < ApplicationController
78	78
79	79	def destroy
80	80	if app.destroy
81		- redirect_to apps_url, :flash => { :success => I18n.t('controllers.apps.flash.destroy.success') }
	81	+ redirect_to apps_url, flash: { success: I18n.t('controllers.apps.flash.destroy.success') }
82	82	else
83	83	flash[:error] = I18n.t('controllers.apps.flash.destroy.error')
84	84	render :show
...	...	@@ -96,7 +96,7 @@ protected
96	96	# set the app's notification service
97	97	if params[:app][:notification_service_attributes] && (notification_type = params[:app][:notification_service_attributes][:type])
98	98	available_notification_classes = [NotificationService] + NotificationService.subclasses
99		- notification_class = available_notification_classes.detect{\|c\| c.name == notification_type}
	99	+ notification_class = available_notification_classes.detect { \|c\| c.name == notification_type }
100	100	if notification_class.present?
101	101	app.notification_service = notification_class.new(params[:app][:notification_service_attributes])
102	102	end
...	...	@@ -116,7 +116,7 @@ protected
116	116	# Sanitize negative values, split on comma,
117	117	# strip, parse as integer, remove all '0's.
118	118	# If empty, set as default and show an error message.
119		- email_at_notices = val.gsub(/-\d+/, "").split(",").map{\|v\| v.strip.to_i }.reject{\|v\| v == 0}
	119	+ email_at_notices = val.gsub(/-\d+/, "").split(",").map { \|v\| v.strip.to_i }.reject { \|v\| v == 0 }
120	120	if email_at_notices.any?
121	121	params[:app][:email_at_notices] = email_at_notices
122	122	else
...	...	@@ -131,7 +131,7 @@ protected
131	131	# Sanitize negative values, split on comma,
132	132	# strip, parse as integer, remove all '0's.
133	133	# If empty, set as default and show an error message.
134		- notify_at_notices = val.gsub(/-\d+/, "").split(",").map{\|v\| v.strip.to_i }
	134	+ notify_at_notices = val.gsub(/-\d+/, "").split(",").map { \|v\| v.strip.to_i }
135	135	if notify_at_notices.any?
136	136	params[:app][:notification_service_attributes][:notify_at_notices] = notify_at_notices
137	137	else
...	...