Skip to content

Conversation

@vgreb
Copy link
Collaborator

@vgreb vgreb commented Jan 1, 2026

fix #2074

@vgreb vgreb force-pushed the refacto/accounting-journal-export branch 2 times, most recently from 4f0ac3b to 34bbce0 Compare January 5, 2026 12:43
@vgreb vgreb self-assigned this Jan 5, 2026
@vgreb vgreb force-pushed the refacto/accounting-journal-export branch from 34bbce0 to d18ddfa Compare January 5, 2026 13:13
@vgreb vgreb requested review from Mopolo and stakovicz and removed request for Mopolo January 5, 2026 15:43
@Mopolo
Copy link
Contributor

Mopolo commented Jan 5, 2026

ℹ️ On s'en sert beaucoup à la tréso donc je vais faire un test avec des données de prod.

@vgreb vgreb force-pushed the refacto/accounting-journal-export branch 2 times, most recently from 789ba90 to 4d4d104 Compare January 10, 2026 22:05
@vgreb vgreb force-pushed the refacto/accounting-journal-export branch 3 times, most recently from de24339 to 62406e2 Compare January 12, 2026 11:10
@vgreb vgreb requested a review from Mopolo January 12, 2026 11:11
@@ -0,0 +1,3 @@
Date;Compte;Événement;Catégorie;Description;Débit;Crédit;Règlement;Commentaire;Justificatif;"Nom justificatif";"Montant HT";TVA;"Montant HT non soumis à TVA";"Montant HT soumis à TVA 5,5";"TVA 5,5";"Montant HT soumis à TVA 10";"TVA 10";"Montant HT soumis à TVA 20";"TVA 20";"Zone de TVA"
2026-01-01;"Compte courant";AG;"A déterminer";"Description dépense 001";-12001.00;;;;Non;;0.00;0.000;;;;;;;;"Non définie"
2026-01-02;"Compte courant";AG;"A déterminer";"Description recette 001";;12002.00;;;Non;;0.00;0.000;;;;;;;;"Non définie"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai fait une comparaison d'un export de prod et avec ta branche sur les mêmes données et y'a quelques différences qui pourraient avoir de l'impact :

  • colonnes Débit et Crédit qui changent de format de décimale : -0,26 => -0.26
  • colonnes Débit et Crédit qui avaient un espace insécable pour les milliers : -4 872,00 => -4872.00 (et pas juste un espace simple)
  • certains montants float perdent en précision : 1.50 => 1.5
  • certains montants in perdent en précision : 125.00 => 125

Je ne sais pas si le cabinet comptable a besoin de ces formats exacts mais dans le doute faudrait les garder je pense.

Ce serait bien d'ajouter une ligne dans ce fichier de test avec toutes les colonnes renseignées, comme ça on se sécurise l'export au max.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

certains montants float perdent en précision : 1.50 => 1.5
certains montants in perdent en précision : 125.00 => 125

Ces montants correspondent à quelles colonnes ?

$value = $this->minkContext->getSession()->getPage()->getContent();

if ($expected !== $value) {
if (trim($expected) !== trim($value)) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

S'il y a besoin de ça, est-ce que ce serait pas un signe que les fichiers ne sont pas tout à fait exacts ?

Y'a un saut de ligne en plus quelque part peut-être ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fputcsv rajoute un saut de ligne à chaque ligne, ce qui n'est pas le cas de SplFileObject::fputcsv

@vgreb vgreb force-pushed the refacto/accounting-journal-export branch from 62406e2 to f06311d Compare January 12, 2026 18:39
@vgreb vgreb force-pushed the refacto/accounting-journal-export branch from f06311d to 3da0b77 Compare January 12, 2026 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refonte - Trésorerie > Journal > Export

3 participants