Use case statements instead of if statements where appropriated

2017-04-05 10:22:37 +02:00 · 2017-04-05 10:22:37 +02:00 · b00eb914da
commit b00eb914da
parent 9071656b55
8 changed files with 82 additions and 70 deletions
--- a/app/controllers/users/sessions_controller.rb
+++ b/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
--- a/app/decorators/user_decorator.rb
+++ b/app/decorators/user_decorator.rb
@ -2,8 +2,12 @@ class UserDecorator < Draper::Decorator
  delegate_all

  def gender_fr
-    return 'M.' if gender == 'male'
-    return 'Mme' if gender == 'female'
+    case gender
+    when 'male'
+      'M.'
+    when 'female'
+      'Mme'
+    end
  end

  def birthdate_fr
--- a/app/models/dossier.rb
+++ b/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'
-          if draft?
-            initiated!
-          end
-        when 'update'
-          if replied?
-            updated!
-          end
-        when 'comment'
-          if replied?
-            updated!
-          end
+      when 'initiate'
+        if draft?
+          initiated!
+        end
+      when 'update'
+        if replied?
+          updated!
+        end
+      when 'comment'
+        if replied?
+          updated!
+        end
      end
-    elsif role == 'gestionnaire'
+    when 'gestionnaire'
      case action
-        when 'comment'
-          if updated?
-            replied!
-          elsif initiated?
-            replied!
-          end
-        when 'follow'
-          if initiated?
-            updated!
-          end
-        when 'close'
-          if received?
-            closed!
-          end
-        when 'refuse'
-          if received?
-            refused!
-          end
-        when 'without_continuation'
-          if received?
-            without_continuation!
-          end
+      when 'comment'
+        if updated?
+          replied!
+        elsif initiated?
+          replied!
+        end
+      when 'follow'
+        if initiated?
+          updated!
+        end
+      when 'close'
+        if received?
+          closed!
+        end
+      when 'refuse'
+        if received?
+          refused!
+        end
+      when 'without_continuation'
+        if received?
+          without_continuation!
+        end
      end
    end
+
    state
  end

--- a/app/services/accompagnateur_service.rb
+++ b/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
--- a/app/views/backoffice/dossiers/_state_description.html.haml
+++ b/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".
--- a/app/views/users/description/champs/_render_list_champs.html.haml
+++ b/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}']",
--- a/app/views/users/dossiers/_state_description.html.haml
+++ b/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
--- a/config/deploy.rb
+++ b/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