Refonte Trésorie > Facture - Ajout / Edition#2185
Conversation
vgreb
force-pushed
the
refacto/accounting-invoice-edit
branch
4 times, most recently
from
d755723 to
d8a8313
Compare
|
@Mopolo @stakovicz @agallou En partant du principe qu'une facture est générée à partir d'un devis, je n'autorise pas l'ajout ou la suppression d'une ligne de détails. Est-ce que ça raccord avec les process de la trésorerie ? |
vgreb
force-pushed
the
refacto/accounting-invoice-edit
branch
from
d8a8313 to
bc7e3af
Compare
Je pense que oui. Ça pourra faire l'objet d'une évolution plus tard sinon. |
| unset($pdf, $parser, $pages, $pageContent); | ||
| gc_collect_cycles(); |
There was a problem hiding this comment.
Hm j'avais eu quelques soucis de mémoire avec Behat justement, est-ce que ça aide pour ça ?
There was a problem hiding this comment.
Oui, j'ai été confronté aux problèmes de mémoire, c'est pour ça que j'ai rajouté ces lignes.
| $keepIds = array_filter(array_map(fn($d) => $d->getId(), $invoice->getDetails())); | ||
| $existingIds = $this->invoicingDetailRepository->getRowsIdsPerInvoicingId($invoiceId); | ||
| $toDelete = array_diff($existingIds, $keepIds); | ||
| if ($toDelete !== []) { | ||
| $this->invoicingDetailRepository->removeRowsPerIds($toDelete); | ||
| } | ||
| foreach ($invoice->getDetails() as $detail) { | ||
| $this->invoicingDetailRepository->save($detail); | ||
| } |
There was a problem hiding this comment.
je n'autorise pas l'ajout ou la suppression d'une ligne de détails.
On dirait que ce code n'a pas de raison d'exister du coup nan ?
There was a problem hiding this comment.
Oui le code est superflu. Si on va jusqu'au bout de la réflexion est-ce que les champs des lignes de détails ne devraient pas être en lecture seule ?
3 participants
resolves #2007