refactor(graphql): use shorthand syntax in mutations

This commit is contained in:
Paul Chavard 2022-12-07 13:21:55 +01:00
parent 4fe081e4be
commit 30a5c592e6
17 changed files with 27 additions and 52 deletions

View file

@ -14,9 +14,9 @@ module Mutations
field :errors, [Types::ValidationErrorType], null: true field :errors, [Types::ValidationErrorType], null: true
def resolve(dossier:, instructeur:, motivation: nil, justificatif: nil, disable_notification:) 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 end
def ready?(justificatif: nil, **args) def ready?(justificatif: nil, **args)

View file

@ -11,7 +11,7 @@ module Mutations
def resolve(dossier:, instructeur:) def resolve(dossier:, instructeur:)
dossier.archiver!(instructeur) dossier.archiver!(instructeur)
{ dossier: dossier } { dossier: }
end end
def authorized?(dossier:, instructeur:) def authorized?(dossier:, instructeur:)

View file

@ -11,9 +11,9 @@ module Mutations
field :errors, [Types::ValidationErrorType], null: true field :errors, [Types::ValidationErrorType], null: true
def resolve(dossier:, groupe_instructeur:) def resolve(dossier:, groupe_instructeur:)
dossier.update!(groupe_instructeur: groupe_instructeur) dossier.update!(groupe_instructeur:)
{ dossier: dossier } { dossier: }
end end
def authorized?(dossier:, groupe_instructeur:) def authorized?(dossier:, groupe_instructeur:)

View file

@ -14,9 +14,9 @@ module Mutations
field :errors, [Types::ValidationErrorType], null: true field :errors, [Types::ValidationErrorType], null: true
def resolve(dossier:, instructeur:, motivation:, justificatif: nil, disable_notification:) 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 end
def ready?(justificatif: nil, **args) def ready?(justificatif: nil, **args)

View file

@ -14,7 +14,7 @@ module Mutations
message = CommentaireService.create(instructeur, dossier, body: body, piece_jointe: attachment) message = CommentaireService.create(instructeur, dossier, body: body, piece_jointe: attachment)
if message.errors.empty? if message.errors.empty?
{ message: message } { message: }
else else
{ errors: message.errors.full_messages } { errors: message.errors.full_messages }
end end

View file

@ -23,7 +23,7 @@ module Mutations
if annotation.save if annotation.save
dossier.log_modifier_annotation!(annotation, instructeur) dossier.log_modifier_annotation!(annotation, instructeur)
{ annotation: annotation } { annotation: }
else else
{ errors: annotation.errors.full_messages } { errors: annotation.errors.full_messages }
end end
@ -44,9 +44,9 @@ module Mutations
Champ.joins(:type_de_champ).find_by(type_de_champ: { Champ.joins(:type_de_champ).find_by(type_de_champ: {
type_champ: annotation_type_champ, type_champ: annotation_type_champ,
stable_id: stable_id, stable_id:,
private: true private: true
}, private: true, row: row, dossier: dossier) }, private: true, row:, dossier:)
end end
def annotation_type_champ def annotation_type_champ

View file

@ -16,7 +16,7 @@ module Mutations
annotation.add_row(dossier.revision) annotation.add_row(dossier.revision)
{ annotation: annotation, errors: nil } { annotation:, errors: nil }
end end
def authorized?(dossier:, instructeur:, **args) def authorized?(dossier:, instructeur:, **args)

View file

@ -5,12 +5,7 @@ module Mutations
argument :value, Boolean, required: true argument :value, Boolean, required: true
def resolve(dossier:, annotation_id:, instructeur:, value:) def resolve(dossier:, annotation_id:, instructeur:, value:)
resolve_with_type( resolve_with_type(dossier:, annotation_id:, instructeur:, value:) do |type_champ, value|
dossier: dossier,
annotation_id: annotation_id,
instructeur: instructeur,
value: value
) do |type_champ, value|
if type_champ == TypeDeChamp.type_champs.fetch(:yes_no) if type_champ == TypeDeChamp.type_champs.fetch(:yes_no)
value ? 'true' : 'false' value ? 'true' : 'false'
else else

View file

@ -5,12 +5,7 @@ module Mutations
argument :value, GraphQL::Types::ISO8601Date, required: true argument :value, GraphQL::Types::ISO8601Date, required: true
def resolve(dossier:, annotation_id:, instructeur:, value:) def resolve(dossier:, annotation_id:, instructeur:, value:)
resolve_with_type( resolve_with_type(dossier:, annotation_id:, instructeur:, value:)
dossier: dossier,
annotation_id: annotation_id,
instructeur: instructeur,
value: value
)
end end
private private

View file

@ -5,12 +5,7 @@ module Mutations
argument :value, GraphQL::Types::ISO8601DateTime, required: true argument :value, GraphQL::Types::ISO8601DateTime, required: true
def resolve(dossier:, annotation_id:, instructeur:, value:) def resolve(dossier:, annotation_id:, instructeur:, value:)
resolve_with_type( resolve_with_type(dossier:, annotation_id:, instructeur:, value:)
dossier: dossier,
annotation_id: annotation_id,
instructeur: instructeur,
value: value
)
end end
private private

View file

@ -5,12 +5,7 @@ module Mutations
argument :value, Int, required: true argument :value, Int, required: true
def resolve(dossier:, annotation_id:, instructeur:, value:) def resolve(dossier:, annotation_id:, instructeur:, value:)
resolve_with_type( resolve_with_type(dossier:, annotation_id:, instructeur:, value:)
dossier: dossier,
annotation_id: annotation_id,
instructeur: instructeur,
value: value
)
end end
private private

View file

@ -5,12 +5,7 @@ module Mutations
argument :value, String, required: true argument :value, String, required: true
def resolve(dossier:, annotation_id:, instructeur:, value:) def resolve(dossier:, annotation_id:, instructeur:, value:)
resolve_with_type( resolve_with_type(dossier:, annotation_id:, instructeur:, value:)
dossier: dossier,
annotation_id: annotation_id,
instructeur: instructeur,
value: value
)
end end
private private

View file

@ -12,9 +12,9 @@ module Mutations
field :errors, [Types::ValidationErrorType], null: true field :errors, [Types::ValidationErrorType], null: true
def resolve(dossier:, instructeur:, disable_notification:) 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 end
def authorized?(dossier:, instructeur:, **args) def authorized?(dossier:, instructeur:, **args)

View file

@ -14,9 +14,9 @@ module Mutations
field :errors, [Types::ValidationErrorType], null: true field :errors, [Types::ValidationErrorType], null: true
def resolve(dossier:, instructeur:, motivation:, justificatif: nil, disable_notification:) 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 end
def ready?(justificatif: nil, **args) def ready?(justificatif: nil, **args)

View file

@ -12,9 +12,9 @@ module Mutations
field :errors, [Types::ValidationErrorType], null: true field :errors, [Types::ValidationErrorType], null: true
def resolve(dossier:, instructeur:, disable_notification:) 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 end
def authorized?(dossier:, instructeur:, **args) def authorized?(dossier:, instructeur:, **args)

View file

@ -12,9 +12,9 @@ module Mutations
field :errors, [Types::ValidationErrorType], null: true field :errors, [Types::ValidationErrorType], null: true
def resolve(dossier:, instructeur:, disable_notification:) 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 end
def authorized?(dossier:, instructeur:, **args) def authorized?(dossier:, instructeur:, **args)

View file

@ -10,8 +10,8 @@ module Mutations
field :errors, [Types::ValidationErrorType], null: true field :errors, [Types::ValidationErrorType], null: true
def resolve(groupe_instructeur:, label: nil, closed: nil) def resolve(groupe_instructeur:, label: nil, closed: nil)
if groupe_instructeur.update({ label: label, closed: closed }.compact) if groupe_instructeur.update({ label:, closed: }.compact)
{ groupe_instructeur: groupe_instructeur } { groupe_instructeur: }
else else
{ errors: groupe_instructeur.errors.full_messages } { errors: groupe_instructeur.errors.full_messages }
end end