ForgeFlow / stock-rma

Stock RMA
GNU Affero General Public License v3.0
38 stars 41 forks source link

[16][FIX] Delete rma_picking_wizard_item on cascade to avoid failure #489

Closed florian-dacosta closed 10 months ago

florian-dacosta commented 10 months ago

Hello,

This small fixes an error on transient model cleaning : ForeignKeyViolation

ERREUR: UPDATE ou DELETE sur la table « rma_make_picking_wizard » viole la contrainte de clé étrangère « rma_make_picking_wizard_item_wiz_id_fkey » de la table « rma_make_picking_wizard_item » DETAIL: La clé (id)=(8594) est toujours référencée à partir de la table « rma_make_picking_wizard_item ».

@AaronHForgeFlow

codecov-commenter commented 10 months ago

Codecov Report

Merging #489 (f48bcd9) into 16.0 (334b86f) will not change coverage. The diff coverage is 100.00%.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/ForgeFlow/stock-rma/pull/489/graphs/tree.svg?width=650&height=150&src=pr&token=hagShlnibr&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eficent)](https://app.codecov.io/gh/ForgeFlow/stock-rma/pull/489?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eficent) ```diff @@ Coverage Diff @@ ## 16.0 #489 +/- ## ======================================= Coverage 80.60% 80.60% ======================================= Files 128 128 Lines 4889 4889 Branches 794 794 ======================================= Hits 3941 3941 Misses 728 728 Partials 220 220 ``` | [Files](https://app.codecov.io/gh/ForgeFlow/stock-rma/pull/489?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eficent) | Coverage Δ | | |---|---|---| | [rma/wizards/rma\_make\_picking.py](https://app.codecov.io/gh/ForgeFlow/stock-rma/pull/489?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eficent#diff-cm1hL3dpemFyZHMvcm1hX21ha2VfcGlja2luZy5weQ==) | `78.57% <100.00%> (ø)` | | ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/ForgeFlow/stock-rma/pull/489?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eficent). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eficent) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/ForgeFlow/stock-rma/pull/489?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eficent). Last update [334b86f...f48bcd9](https://app.codecov.io/gh/ForgeFlow/stock-rma/pull/489?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eficent). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eficent).
AaronHForgeFlow commented 10 months ago

Thank you @florian-dacosta