Merge pull request #119 from sgmap/cleanup

Cleanup

app/controllers/users/carte_controller.rb

@@ -15,8 +15,8 @@ class Users::CarteController < UsersController
   def save
     dossier = current_user_dossier
 
-    dossier.quartier_prioritaires.map(&:destroy)
+    dossier.quartier_prioritaires.each(&:destroy)
-    dossier.cadastres.map(&:destroy)
+    dossier.cadastres.each(&:destroy)
 
     unless params[:json_latlngs].blank?
       ModuleApiCartoService.save_qp! dossier, params[:json_latlngs]

app/controllers/users/sessions_controller.rb

@@ -54,10 +54,11 @@ class Users::SessionsController < Sessions::SessionsController
 
       sign_out :user
 
-      if connected_with_france_connect == 'entreprise'
+      case connected_with_france_connect
+      when 'entreprise'
         redirect_to FRANCE_CONNECT.entreprise_logout_endpoint
         return
-      elsif connected_with_france_connect == 'particulier'
+      when 'particulier'
         redirect_to FRANCE_CONNECT.particulier_logout_endpoint
         return
       end

app/decorators/user_decorator.rb

@@ -2,8 +2,12 @@ class UserDecorator < Draper::Decorator
   delegate_all
 
   def gender_fr
-    return 'M.' if gender == 'male'
+    case gender
-    return 'Mme' if gender == 'female'
+    when 'male'
+      'M.'
+    when 'female'
+      'Mme'
+    end
   end
 
   def birthdate_fr

app/models/dossier.rb

@@ -111,47 +111,49 @@ class Dossier < ActiveRecord::Base
       fail 'role is not valid'
     end
 
-    if role == 'user'
+    case role
+    when 'user'
       case action
-        when 'initiate'
+      when 'initiate'
-          if draft?
+        if draft?
-            initiated!
+          initiated!
-          end
+        end
-        when 'update'
+      when 'update'
-          if replied?
+        if replied?
-            updated!
+          updated!
-          end
+        end
-        when 'comment'
+      when 'comment'
-          if replied?
+        if replied?
-            updated!
+          updated!
-          end
+        end
       end
-    elsif role == 'gestionnaire'
+    when 'gestionnaire'
       case action
-        when 'comment'
+      when 'comment'
-          if updated?
+        if updated?
-            replied!
+          replied!
-          elsif initiated?
+        elsif initiated?
-            replied!
+          replied!
-          end
+        end
-        when 'follow'
+      when 'follow'
-          if initiated?
+        if initiated?
-            updated!
+          updated!
-          end
+        end
-        when 'close'
+      when 'close'
-          if received?
+        if received?
-            closed!
+          closed!
-          end
+        end
-        when 'refuse'
+      when 'refuse'
-          if received?
+        if received?
-            refused!
+          refused!
-          end
+        end
-        when 'without_continuation'
+      when 'without_continuation'
-          if received?
+        if received?
-            without_continuation!
+          without_continuation!
-          end
+        end
       end
     end
+
     state
   end
 

app/services/accompagnateur_service.rb

@@ -9,9 +9,10 @@ class AccompagnateurService
   end
 
   def change_assignement!
-    if @to == ASSIGN
+    case @to
+    when ASSIGN
       AssignTo.create(gestionnaire: @accompagnateur, procedure: @procedure)
-    elsif @to == NOT_ASSIGN
+    when NOT_ASSIGN
       AssignTo.where(gestionnaire: @accompagnateur, procedure: @procedure).delete_all
     end
   end

app/views/backoffice/dossiers/_state_description.html.haml

@@ -5,17 +5,18 @@
         .col-md-1.col-lg-1.col-sm-1.col-xs-1.col-sm-1.col-xs-1
           .fa.fa-info-circle.text-info{ style: 'font-size: 2em; margin-top: 20%;' }
         .col-xs-11
-          - if dossiers_list_facade.liste == 'nouveaux'
+          - case dossiers_list_facade.liste
+          - when 'nouveaux'
             Tous les dossiers présents dans cette liste sont ceux qui
             %b
               n'ont jamais été ouvert par votre service.
             Il attende une première lecture et intervention de votre part.
-          - elsif dossiers_list_facade.liste == 'a_traiter'
+          - when 'a_traiter'
             Tous les dossiers présents dans cette liste sont ceux qui sont
             %b
               en cours de construction avec l'usager.
             Ils ne sont pas figés et ne sont donc pas complets.
-          - elsif dossiers_list_facade.liste == 'fige'
+          - when 'fige'
             Tous les dossiers présents dans cette liste ont été déclarés
             %b
               complets
@@ -23,14 +24,14 @@
             %b
               plus modifiables par l'usager.
             Ils attendent donc leurs dépots officiels qui doit être effectué par l'usager.
-          - elsif dossiers_list_facade.liste == 'deposes'
+          - when 'deposes'
             Tous les dossiers présents dans cette liste ont été
             %b
               officiellement déposé par l'usager pour examen.
             Il faut donc que vous confirmiez par "accusé de réception" la bonne réception de toutes les informations et documents demandés avant examen final.
-          - elsif dossiers_list_facade.liste == 'a_instruire'
+          - when 'a_instruire'
             Tous les dossiers présents dans cette liste sont à instruire. Ceux sont tous les dossiers
             %b
               qui ont reçu bonne réception.
-          - elsif dossiers_list_facade.liste == 'termine'
+          - when 'termine'
             Tous les dossiers présents dans cette liste sont considérés comme cloturé car ils ont tous reçu un verdict final qui peut être "Validé", "Refusé" ou "Sans suite".

app/views/users/description/champs/_render_list_champs.html.haml

@@ -11,40 +11,41 @@
             - if champ.mandatory?
               *
 
-          - if champ.type_champ == 'textarea'
+          - case champ.type_champ
+          - when 'textarea'
             = render partial: 'users/description/champs/textarea', locals: { champ: champ }
 
-          - elsif champ.type_champ == 'checkbox'
+          - when 'checkbox'
             = render partial: 'users/description/champs/checkbox', locals: { champ: champ }
 
-          - elsif champ.type_champ == 'civilite'
+          - when 'civilite'
             = render partial: 'users/description/champs/civilite', locals: { champ: champ }
 
-          - elsif champ.type_champ == 'datetime'
+          - when 'datetime'
             = render partial: 'users/description/champs/datetime', locals: { champ: champ }
 
-          - elsif champ.type_champ == 'yes_no'
+          - when 'yes_no'
             = render partial: 'users/description/champs/yes_no', locals: { champ: champ }
 
-          - elsif %w(drop_down_list multiple_drop_down_list).include?(champ.type_champ)
+          - when 'drop_down_list', 'multiple_drop_down_list'
             = render partial: 'users/description/champs/drop_down_list', locals: { champ: champ }
 
-          - elsif champ.type_champ == 'pays'
+          - when 'pays'
             = render partial: 'users/description/champs/pays', locals: { champ: champ }
 
-          - elsif champ.type_champ == 'regions'
+          - when 'regions'
             = render partial: 'users/description/champs/regions', locals: { champ: champ }
 
-          - elsif champ.type_champ == 'engagement'
+          - when 'engagement'
             = render partial: 'users/description/champs/engagement', locals: { champ: champ }
 
-          - elsif champ.type_champ == 'departements'
+          - when 'departements'
             = render partial: 'users/description/champs/departements', locals: { champ: champ }
 
-          - elsif champ.type_champ == 'dossier_link'
+          - when 'dossier_link'
             = render partial: 'users/description/champs/dossier_link', locals: { champ: champ }
 
-          - elsif champ.type_champ == 'explication'
+          - when 'explication'
 
           - else
             %input.form-control{ name: "champs['#{champ.id}']",

app/views/users/dossiers/_state_description.html.haml

@@ -5,33 +5,34 @@
         .col-md-1.col-lg-1.col-sm-1.col-xs-1
           .fa.fa-info-circle.text-info{ style: 'font-size: 2em; margin-top: 20%;' }
         .col-xs-11
-          - if dossiers_list_facade.liste == 'brouillon'
+          - case dossiers_list_facade.liste
+          - when 'brouillon'
             Les dossiers présents dans cette liste
             %b
               n'ont pas encore été soumis aux services instructeurs.
             Ils ne sont visibles que par vous.
-          - elsif dossiers_list_facade.liste == 'nouveaux'
+          - when 'nouveaux'
             Les dossiers présents dans cette liste
             %b
               n'ont pas encore été ouverts
             par les services instructeurs. Une notification vous sera envoyée quand votre demande aura été étudiée.
-          - elsif dossiers_list_facade.liste == 'a_traiter'
+          - when 'a_traiter'
             Les dossiers présents dans cette liste sont
             %b
               visibles par les services instructeurs.
             %br
             Ces dossiers ne sont pas encore figés et peuvent être modifiés à souhait.
-          - elsif dossiers_list_facade.liste == 'en_attente'
+          - when 'en_attente'
             Les dossiers présents dans cette liste sont
             %b
               en cours de relecture par les services instructeurs.
             Il reviendra vers vous si des informations ou documents sont manquants pour le futur examen de votre dossier.
-          - elsif dossiers_list_facade.liste == 'valides'
+          - when 'valides'
             Les dossiers présents dans cette liste ont été
             %b
               relus et considérés comme complets
             pour examen par les services instructeurs. Ceux-ci ne peuvent maintenant plus être modifiés. Il faut que vous procédiez à leur dépôt afin qu'une décision finale soit rendue.
-          - elsif dossiers_list_facade.liste == 'en_instruction'
+          - when 'en_instruction'
             Les dossiers présents dans cette liste sont
             %b
               en cours de réception
@@ -39,12 +40,12 @@
             %b
               en cours d'examen
             par les services instructeurs. Une notification vous sera envoyée une fois qu'une décision aura été rendue.
-          - elsif dossiers_list_facade.liste == 'termine'
+          - when 'termine'
             Les dossiers présents dans cette liste sont ceux qui ont été instruits et pour lesquels
             %b
               une décision finale a été rendue.
             Ils peuvent posséder trois états différents : Accepté, Refusé ou Sans Suite.
-          - elsif dossiers_list_facade.liste == 'invite'
+          - when 'invite'
             Les dossiers présents dans cette liste sont ceux
             %b
               auxquels vous avez été invités

config/deploy.rb

@@ -24,7 +24,8 @@ set :port, 2200
 
 set :deploy_to, '/var/www/tps_dev'
 
-if ENV["to"] == "staging"
+case ENV["to"]
+when "staging"
   if ENV['branch'].nil?
     set :branch, 'staging'
   else
@@ -33,7 +34,7 @@ if ENV["to"] == "staging"
   set :deploy_to, '/var/www/tps_dev'
   set :user, 'tps_dev' # Username in the server to SSH to.
   appname = 'tps_dev'
-elsif ENV["to"] == "production"
+when "production"
   if ENV['branch'].nil?
     set :branch, 'master'
   else