From 30a5c592e612db7e2d978393f5c393c81f6ee71b Mon Sep 17 00:00:00 2001
From: Paul Chavard <pro@paul.chavard.net>
Date: Wed, 7 Dec 2022 13:21:55 +0100
Subject: [PATCH] refactor(graphql): use shorthand syntax in mutations

---
 app/graphql/mutations/dossier_accepter.rb                  | 4 ++--
 app/graphql/mutations/dossier_archiver.rb                  | 2 +-
 .../mutations/dossier_changer_groupe_instructeur.rb        | 4 ++--
 app/graphql/mutations/dossier_classer_sans_suite.rb        | 4 ++--
 app/graphql/mutations/dossier_envoyer_message.rb           | 2 +-
 app/graphql/mutations/dossier_modifier_annotation.rb       | 6 +++---
 .../mutations/dossier_modifier_annotation_ajouter_ligne.rb | 2 +-
 .../mutations/dossier_modifier_annotation_checkbox.rb      | 7 +------
 app/graphql/mutations/dossier_modifier_annotation_date.rb  | 7 +------
 .../mutations/dossier_modifier_annotation_datetime.rb      | 7 +------
 .../dossier_modifier_annotation_integer_number.rb          | 7 +------
 app/graphql/mutations/dossier_modifier_annotation_text.rb  | 7 +------
 app/graphql/mutations/dossier_passer_en_instruction.rb     | 4 ++--
 app/graphql/mutations/dossier_refuser.rb                   | 4 ++--
 app/graphql/mutations/dossier_repasser_en_construction.rb  | 4 ++--
 app/graphql/mutations/dossier_repasser_en_instruction.rb   | 4 ++--
 app/graphql/mutations/groupe_instructeur_modifier.rb       | 4 ++--
 17 files changed, 27 insertions(+), 52 deletions(-)

diff --git a/app/graphql/mutations/dossier_accepter.rb b/app/graphql/mutations/dossier_accepter.rb
index 83f7c05d9..ef2525f92 100644
--- a/app/graphql/mutations/dossier_accepter.rb
+++ b/app/graphql/mutations/dossier_accepter.rb
@@ -14,9 +14,9 @@ module Mutations
     field :errors, [Types::ValidationErrorType], null: true
 
     def resolve(dossier:, instructeur:, motivation: nil, justificatif: nil, disable_notification:)
-      dossier.accepter!(instructeur: instructeur, motivation: motivation, justificatif: justificatif, disable_notification: disable_notification)
+      dossier.accepter!(instructeur:, motivation:, justificatif:, disable_notification:)
 
-      { dossier: dossier }
+      { dossier: }
     end
 
     def ready?(justificatif: nil, **args)
diff --git a/app/graphql/mutations/dossier_archiver.rb b/app/graphql/mutations/dossier_archiver.rb
index 26d4019a7..5a0dc7586 100644
--- a/app/graphql/mutations/dossier_archiver.rb
+++ b/app/graphql/mutations/dossier_archiver.rb
@@ -11,7 +11,7 @@ module Mutations
     def resolve(dossier:, instructeur:)
       dossier.archiver!(instructeur)
 
-      { dossier: dossier }
+      { dossier: }
     end
 
     def authorized?(dossier:, instructeur:)
diff --git a/app/graphql/mutations/dossier_changer_groupe_instructeur.rb b/app/graphql/mutations/dossier_changer_groupe_instructeur.rb
index 11a21db8f..7caf6769b 100644
--- a/app/graphql/mutations/dossier_changer_groupe_instructeur.rb
+++ b/app/graphql/mutations/dossier_changer_groupe_instructeur.rb
@@ -11,9 +11,9 @@ module Mutations
     field :errors, [Types::ValidationErrorType], null: true
 
     def resolve(dossier:, groupe_instructeur:)
-      dossier.update!(groupe_instructeur: groupe_instructeur)
+      dossier.update!(groupe_instructeur:)
 
-      { dossier: dossier }
+      { dossier: }
     end
 
     def authorized?(dossier:, groupe_instructeur:)
diff --git a/app/graphql/mutations/dossier_classer_sans_suite.rb b/app/graphql/mutations/dossier_classer_sans_suite.rb
index 4d45935cd..5a6285dc2 100644
--- a/app/graphql/mutations/dossier_classer_sans_suite.rb
+++ b/app/graphql/mutations/dossier_classer_sans_suite.rb
@@ -14,9 +14,9 @@ module Mutations
     field :errors, [Types::ValidationErrorType], null: true
 
     def resolve(dossier:, instructeur:, motivation:, justificatif: nil, disable_notification:)
-      dossier.classer_sans_suite!(instructeur: instructeur, motivation: motivation, justificatif: justificatif, disable_notification: disable_notification)
+      dossier.classer_sans_suite!(instructeur:, motivation:, justificatif:, disable_notification:)
 
-      { dossier: dossier }
+      { dossier: }
     end
 
     def ready?(justificatif: nil, **args)
diff --git a/app/graphql/mutations/dossier_envoyer_message.rb b/app/graphql/mutations/dossier_envoyer_message.rb
index 0016879f1..4afeb16ba 100644
--- a/app/graphql/mutations/dossier_envoyer_message.rb
+++ b/app/graphql/mutations/dossier_envoyer_message.rb
@@ -14,7 +14,7 @@ module Mutations
       message = CommentaireService.create(instructeur, dossier, body: body, piece_jointe: attachment)
 
       if message.errors.empty?
-        { message: message }
+        { message: }
       else
         { errors: message.errors.full_messages }
       end
diff --git a/app/graphql/mutations/dossier_modifier_annotation.rb b/app/graphql/mutations/dossier_modifier_annotation.rb
index 572532d80..de1713f77 100644
--- a/app/graphql/mutations/dossier_modifier_annotation.rb
+++ b/app/graphql/mutations/dossier_modifier_annotation.rb
@@ -23,7 +23,7 @@ module Mutations
       if annotation.save
         dossier.log_modifier_annotation!(annotation, instructeur)
 
-        { annotation: annotation }
+        { annotation: }
       else
         { errors: annotation.errors.full_messages }
       end
@@ -44,9 +44,9 @@ module Mutations
 
       Champ.joins(:type_de_champ).find_by(type_de_champ: {
         type_champ: annotation_type_champ,
-        stable_id: stable_id,
+        stable_id:,
         private: true
-      }, private: true, row: row, dossier: dossier)
+      }, private: true, row:, dossier:)
     end
 
     def annotation_type_champ
diff --git a/app/graphql/mutations/dossier_modifier_annotation_ajouter_ligne.rb b/app/graphql/mutations/dossier_modifier_annotation_ajouter_ligne.rb
index ad59cdecc..11bfcbcd0 100644
--- a/app/graphql/mutations/dossier_modifier_annotation_ajouter_ligne.rb
+++ b/app/graphql/mutations/dossier_modifier_annotation_ajouter_ligne.rb
@@ -16,7 +16,7 @@ module Mutations
 
       annotation.add_row(dossier.revision)
 
-      { annotation: annotation, errors: nil }
+      { annotation:, errors: nil }
     end
 
     def authorized?(dossier:, instructeur:, **args)
diff --git a/app/graphql/mutations/dossier_modifier_annotation_checkbox.rb b/app/graphql/mutations/dossier_modifier_annotation_checkbox.rb
index 36aeda7fd..a94cb6110 100644
--- a/app/graphql/mutations/dossier_modifier_annotation_checkbox.rb
+++ b/app/graphql/mutations/dossier_modifier_annotation_checkbox.rb
@@ -5,12 +5,7 @@ module Mutations
     argument :value, Boolean, required: true
 
     def resolve(dossier:, annotation_id:, instructeur:, value:)
-      resolve_with_type(
-        dossier: dossier,
-        annotation_id: annotation_id,
-        instructeur: instructeur,
-        value: value
-      ) do |type_champ, value|
+      resolve_with_type(dossier:, annotation_id:, instructeur:, value:) do |type_champ, value|
         if type_champ == TypeDeChamp.type_champs.fetch(:yes_no)
           value ? 'true' : 'false'
         else
diff --git a/app/graphql/mutations/dossier_modifier_annotation_date.rb b/app/graphql/mutations/dossier_modifier_annotation_date.rb
index a3e50ad2b..d1616a22a 100644
--- a/app/graphql/mutations/dossier_modifier_annotation_date.rb
+++ b/app/graphql/mutations/dossier_modifier_annotation_date.rb
@@ -5,12 +5,7 @@ module Mutations
     argument :value, GraphQL::Types::ISO8601Date, required: true
 
     def resolve(dossier:, annotation_id:, instructeur:, value:)
-      resolve_with_type(
-        dossier: dossier,
-        annotation_id: annotation_id,
-        instructeur: instructeur,
-        value: value
-      )
+      resolve_with_type(dossier:, annotation_id:, instructeur:, value:)
     end
 
     private
diff --git a/app/graphql/mutations/dossier_modifier_annotation_datetime.rb b/app/graphql/mutations/dossier_modifier_annotation_datetime.rb
index a524c3641..ee4b2ed67 100644
--- a/app/graphql/mutations/dossier_modifier_annotation_datetime.rb
+++ b/app/graphql/mutations/dossier_modifier_annotation_datetime.rb
@@ -5,12 +5,7 @@ module Mutations
     argument :value, GraphQL::Types::ISO8601DateTime, required: true
 
     def resolve(dossier:, annotation_id:, instructeur:, value:)
-      resolve_with_type(
-        dossier: dossier,
-        annotation_id: annotation_id,
-        instructeur: instructeur,
-        value: value
-      )
+      resolve_with_type(dossier:, annotation_id:, instructeur:, value:)
     end
 
     private
diff --git a/app/graphql/mutations/dossier_modifier_annotation_integer_number.rb b/app/graphql/mutations/dossier_modifier_annotation_integer_number.rb
index 2906de550..14b68c7c1 100644
--- a/app/graphql/mutations/dossier_modifier_annotation_integer_number.rb
+++ b/app/graphql/mutations/dossier_modifier_annotation_integer_number.rb
@@ -5,12 +5,7 @@ module Mutations
     argument :value, Int, required: true
 
     def resolve(dossier:, annotation_id:, instructeur:, value:)
-      resolve_with_type(
-        dossier: dossier,
-        annotation_id: annotation_id,
-        instructeur: instructeur,
-        value: value
-      )
+      resolve_with_type(dossier:, annotation_id:, instructeur:, value:)
     end
 
     private
diff --git a/app/graphql/mutations/dossier_modifier_annotation_text.rb b/app/graphql/mutations/dossier_modifier_annotation_text.rb
index c70c055da..b13f3a836 100644
--- a/app/graphql/mutations/dossier_modifier_annotation_text.rb
+++ b/app/graphql/mutations/dossier_modifier_annotation_text.rb
@@ -5,12 +5,7 @@ module Mutations
     argument :value, String, required: true
 
     def resolve(dossier:, annotation_id:, instructeur:, value:)
-      resolve_with_type(
-        dossier: dossier,
-        annotation_id: annotation_id,
-        instructeur: instructeur,
-        value: value
-      )
+      resolve_with_type(dossier:, annotation_id:, instructeur:, value:)
     end
 
     private
diff --git a/app/graphql/mutations/dossier_passer_en_instruction.rb b/app/graphql/mutations/dossier_passer_en_instruction.rb
index 217b0a63b..1ece00ede 100644
--- a/app/graphql/mutations/dossier_passer_en_instruction.rb
+++ b/app/graphql/mutations/dossier_passer_en_instruction.rb
@@ -12,9 +12,9 @@ module Mutations
     field :errors, [Types::ValidationErrorType], null: true
 
     def resolve(dossier:, instructeur:, disable_notification:)
-      dossier.passer_en_instruction!(instructeur: instructeur, disable_notification: disable_notification)
+      dossier.passer_en_instruction!(instructeur:, disable_notification:)
 
-      { dossier: dossier }
+      { dossier: }
     end
 
     def authorized?(dossier:, instructeur:, **args)
diff --git a/app/graphql/mutations/dossier_refuser.rb b/app/graphql/mutations/dossier_refuser.rb
index 0dfe7e568..1bbd68604 100644
--- a/app/graphql/mutations/dossier_refuser.rb
+++ b/app/graphql/mutations/dossier_refuser.rb
@@ -14,9 +14,9 @@ module Mutations
     field :errors, [Types::ValidationErrorType], null: true
 
     def resolve(dossier:, instructeur:, motivation:, justificatif: nil, disable_notification:)
-      dossier.refuser!(instructeur: instructeur, motivation: motivation, justificatif: justificatif, disable_notification: disable_notification)
+      dossier.refuser!(instructeur:, motivation:, justificatif:, disable_notification:)
 
-      { dossier: dossier }
+      { dossier: }
     end
 
     def ready?(justificatif: nil, **args)
diff --git a/app/graphql/mutations/dossier_repasser_en_construction.rb b/app/graphql/mutations/dossier_repasser_en_construction.rb
index 8bb99b260..1f30546d9 100644
--- a/app/graphql/mutations/dossier_repasser_en_construction.rb
+++ b/app/graphql/mutations/dossier_repasser_en_construction.rb
@@ -12,9 +12,9 @@ module Mutations
     field :errors, [Types::ValidationErrorType], null: true
 
     def resolve(dossier:, instructeur:, disable_notification:)
-      dossier.repasser_en_construction!(instructeur: instructeur, disable_notification: disable_notification)
+      dossier.repasser_en_construction!(instructeur:, disable_notification:)
 
-      { dossier: dossier }
+      { dossier: }
     end
 
     def authorized?(dossier:, instructeur:, **args)
diff --git a/app/graphql/mutations/dossier_repasser_en_instruction.rb b/app/graphql/mutations/dossier_repasser_en_instruction.rb
index d71dde304..1e3c341a0 100644
--- a/app/graphql/mutations/dossier_repasser_en_instruction.rb
+++ b/app/graphql/mutations/dossier_repasser_en_instruction.rb
@@ -12,9 +12,9 @@ module Mutations
     field :errors, [Types::ValidationErrorType], null: true
 
     def resolve(dossier:, instructeur:, disable_notification:)
-      dossier.repasser_en_instruction!(instructeur: instructeur, disable_notification: disable_notification)
+      dossier.repasser_en_instruction!(instructeur:, disable_notification:)
 
-      { dossier: dossier }
+      { dossier: }
     end
 
     def authorized?(dossier:, instructeur:, **args)
diff --git a/app/graphql/mutations/groupe_instructeur_modifier.rb b/app/graphql/mutations/groupe_instructeur_modifier.rb
index cd89174aa..3b994ef4f 100644
--- a/app/graphql/mutations/groupe_instructeur_modifier.rb
+++ b/app/graphql/mutations/groupe_instructeur_modifier.rb
@@ -10,8 +10,8 @@ module Mutations
     field :errors, [Types::ValidationErrorType], null: true
 
     def resolve(groupe_instructeur:, label: nil, closed: nil)
-      if groupe_instructeur.update({ label: label, closed: closed }.compact)
-        { groupe_instructeur: groupe_instructeur }
+      if groupe_instructeur.update({ label:, closed: }.compact)
+        { groupe_instructeur: }
       else
         { errors: groupe_instructeur.errors.full_messages }
       end