FIX : Do not return ERROR when LogoThumb is empty #1280
Conversation
2024 rc -> 2024
…er_invoice_multicurrency fix multicurrency on supplier invoices
…tion-without-stock-movement FIX can validate shipment without stock movement
…-invoice-lines-computed-remain-to-pay FiX computed field remain to pay in export of invoice lines
…4_fix_supplier_invoice_multicurrency Revert "fix multicurrency on supplier invoices"
2024 rc -> 2024
…t-clone-task-start-date FIX calculate start date of cloned task from cloned project
…end-to-bcc FIX send mail to BCC when email formated as Fullname <email>
2024 rc -> 2024
…ion_Tiers Fix Tiers Fusion
2024_rc -> 2024
This reverts commit bab49d6.
This reverts commit 038f8d5.
…kports_for_ticket_list 2024 revert backports for ticket list
…4_revert_backports_for_ticket_list Revert "2024 revert backports for ticket list"
2024_rc -> 2024
2024_rc -> 2024
…hing is made for mode_reglement
…hing is made for mode_reglement
| @@ -142,7 +142,11 @@ public function get_substitutionarray_user($user, $outputlangs) | |||
| // phpcs:enable | |||
| global $conf, $extrafields; | |||
|
|
|||
| $logotouse = $conf->user->dir_output.'/'.get_exdir($user->id, 2, 0, 1, $user, 'user').'/'.$user->photo; | |||
There was a problem hiding this comment.
cette valeur ne peut pas être vide, ajouter ce test ne changera rien
| @@ -245,7 +249,11 @@ public function get_substitutionarray_mysoc($mysoc, $outputlangs) | |||
| $mysoc->state = getState($mysoc->state_code, 0); | |||
| } | |||
|
|
|||
| $logotouse = $conf->mycompany->dir_output.'/logos/thumbs/'.$mysoc->logo_small; | |||
There was a problem hiding this comment.
cette valeur ne peut pas être vide, ajouter ce test ne changera rien
C'est-à-dire ? "vide" dans le sens "égal à null, 0 ou chaîne vide" ou "vide" dans le sens "pointe vers un fichier inexistant" ? Quand je lis le code, la valeur peut être incohérente, mais pas vide.
Non. ErrorFileNotFound indique que le fichier n'a pas été trouvé, pas que le chemin est vide. Les modifications que tu propose ne vont rien changer au comportement car la valeur contrôlée n'est jamais vide. Tu peux tester le résultat de |
|
Après réflexion, je pense qu'il ne sert à rien de corriger $logotouse car le cas est trivial à corriger en modifiant le ODT. Le code dit bien 'si le fichier du logo n'existe pas, afficher une erreur'. 'mycompany_logo est identique pour tous les ODT sur une même entité. |
Actuellement $logotouse n'est pas contrôlé sur le fait d'être vide.
S'il est vide, une erreur est inscrite : ErrorFileNotFound
Le problème, c'est que lorsque l'on utilise des balises [!-- IF {mycompagny_logo} --] l'erreur est comprise comme un true.
Cette PR corrige le fichier commondocgenerator.class.php afin que $logotouse produise NULL si le fichier n'existe pas.
De ce fait, les balises peuvent fonctionner correctement.
EDIT :
C'est ajouté aussi l'ajout de la balise _amount_in_currency afin d'avoir un appel propre quelque soit la langue de "Montant exprimé en %s" où %s est substituté par la device en cours ($conf->currency).