9449 ETQ instructeur ou admin, je peux apposer sur une attestation un tampon dédié à un groupe instructeur

[krichtof]

close #9449

Voici ce qu'apporte cette PR.

• en tant qu'admin ou instructeur, je peux apposer sur une attestation un tampon dédié à un groupe instructeur. Ce tampon remplacera le tampon de la procédure

• bug corrigé : lorsque le tampon (pour l'attestation en général ou pour un groupe instructeur en particulier) est un fichier png non interlacé, l'image n'est plus altérée.

• bug corrigé : il n'y a plus d'erreur lorsqu'on tente de supprimer le tampon et le remplacer par un autre.

Sur la page du groupe instructeur

Sur la page de l'attestation

[LeSim]

super, mega, top

[LeSim]

Proposition, je sais pas si c'est mieux.

Il y a beaucoup de ternaire du à la présence ou non du dossier dans les méthodes render_attributes_for et signature_to_render.

je propose de faire le test de présence de dossier une fois pour toute dans le render_attributes_for

def render_attributes_for(params = {})
    attributes = 
    {
      created_at: Time.zone.now,
      footer: params.fetch(:footer, footer),
      logo: params.fetch(:logo, logo.attached? ? logo : nil),
    }

  dossier = params[:dossier]

  if dossier.present?
    attributes.merge({
      title: replace_tags(title, dossier),
      body: replace_tags(body, dossier),
      signature: signature_to_render(dossier.groupe_instructeur)
    })
  else
    attributes.merge(
  {
      title: params.fetch(:title, title),
      body: params.fetch(:body, body),
      signature: signature_to_render(params[:groupe_instructeur])
    }
)
  end
end

Suggested change

	def signature_to_render(params)
	dossier = params.fetch(:dossier, false)
	groupe_instructeur = dossier ? dossier.groupe_instructeur : params[:groupe_instructeur]
	if groupe_instructeur && groupe_instructeur.signature.attached?
	groupe_instructeur.signature
	else
	signature
	end
	end
	def signature_to_render(groupe_instructeur)
	if groupe_instructeur&.signature&.attached?
	groupe_instructeur.signature
	else
	signature
	end
	end

[krichtof]

Je trouve ta proposition beaucoup plus lisible ! Merci 😊
Je viens de la commiter

[sentry-io]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ **NoMethodError: undefined method []' for nil:NilClass (NoMethodError)** Administrateurs::GroupeInstructeursController#a...` View Issue

_{Did you find this useful? React with a 👍 or 👎}