Galette

Alain Paris a écrit (#note-5):

Il suffit choisir un autre fichier avec une hauteur de 150 (chez moi 200x150 cela fonctionne ).

Après avoir jeté un œil au code, tout dépend du ratio de l'image, et normalement une largeur et une hauteur max sont imposées.
Les images en portrait sont redimensionnées en se référant à leur hauteur.
Les images en paysage, en se référant à leur largeur.

Il y a bien des erreurs dans le code. Les dimensions de l'image sont comparées aux limites imposées, mais sur de mauvaises valeurs et avec des unités différentes (pixels pour l'image, contre millimètres pour les valeurs utilisées dans le PDF).
Et le cas des images carrées (ou proche d'un ratio 1 en paysage) n'est pas vraiment traité ce qui entraîne leur agrandissement au delà de ce qui est prévu comme c'est le cas de Peire.

source:/galette/lib/Galette/IO/PdfMembersCards.php#L165

La hauteur indiquée n'est plus celle utilisée après dans le code :

  // Set logo size to max width 30 mm or max height 25 mm

getHeight() et getWidth() renvoient une valeur en pixels et sont comparées à une mauvaise valeur en millimètres :

  $ratio = $print_logo->getWidth() / $print_logo->getHeight();
  if ($ratio < 1) {
      if ($print_logo->getHeight() > 16) {
          $this->hlogo = 20; 
      } else {
          $this->hlogo = $print_logo->getHeight();
      }
      $this->wlogo = round($this->hlogo * $ratio);                 

Ensuite les images carrées sont redimensionnées comme des images paysage en fonction de leur largeur et dépassent alors la hauteur max imposée pour le PDF :

  } else { 
      if ($print_logo->getWidth() > 16) {
          $this->wlogo = 30;
      } else {
          $this->wlogo = $print_logo->getWidth();
      }
      $this->hlogo = round($this->wlogo / $ratio);
  }

Les modifs proposées corrigent ce que je pense être le comportement normal souhaité pour l'affichage du logo :
https://github.com/galette/galette/pull/215/files

Si les dimensions du fichier original sont inférieures à 113x64px (30x17mm), ce sont elles qui seront utilisées.

Je joins plusieurs captures des résultats obtenus :)

• si l'option "Afficher les identifiants" est cochée dans les paramètres, c'est cet identifiant qui est utilisé (comportement actuel par défaut).
• si le champ "Numéro adhérent" est utilisé ET renseigné, c'est ce numéro qui est utilisé.
• si ce même champ est utilisé ET renseigné ET que l'option "Afficher les identifiants" est cochée, c'est le numéro qui est privilégié.
• si ni l'option, ni le champ ne sont utilisés, rien n'est affiché.

Appliqué par commit a1bde60ef66441b2cca0c857ab8e20a7accaf01d.