Commit Graph

71 Commits

Author SHA1 Message Date
AaronHForgeFlow
2c9a213677 [FIX] rma: vendor rma lot mnmg 2024-07-12 10:47:57 +02:00
BernatPForgeFlow
6fab740b56 [FIX] rma: Do not set 'Destination Package' if RMA package comes from 'Customers' location 2024-05-08 16:36:41 +02:00
Christopher Ormaza
ab02ca6b36 [FIX] quants can be more than one on _gather function result 2024-05-08 16:32:41 +02:00
BernatPForgeFlow
3dfec266fd [IMP] rma: Auto-calculate package for serial products
When returning or delivering a serial product from a RMA, we will calculate by default its package. Later, it can be modyfied by the user.
2024-05-08 16:32:30 +02:00
AaronHForgeFlow
96b6024bc6 [FIX] rma: lot reservation error:
qty_done -> quantity
reserved_qty field does no exist anymore
2024-05-06 18:32:54 +02:00
Carlos Vallés Fuster
cb5a8a1ddd [MIG] rma: Migration to v17 2024-05-06 17:08:34 +02:00
AaronHForgeFlow
e23335cee5 [IMP] rma: black, isort, prettier 2024-05-06 17:08:10 +02:00
Florian da Costa
9956118af7 [FIX] Delete rma_picking_wizard_item on cascade to avoid failure 2024-05-06 17:07:50 +02:00
ChrisOForgeFlow
608f2910dc [14.0][IMP] added default operation on rma group, easy setup before rma lines created (#452)
* [14.0][IMP] added default operation on rma group, easy setup before rma lines created

* [IMP] added fields for default route created by wizard on rma group

* fix: get right price after create rma order line
2024-05-06 17:07:50 +02:00
Joan Sisquella
ecb63ea8c4 [FIX] rma: procurement origin
In the current implementation of Odoo's _assign_picking() method in stock.move, there's a conditional check that looks at whether all the moves associated with a picking have the same partner_id and origin. If any move doesn't align with these conditions, the origin of the picking is set to False.

        if any(picking.partner_id.id != m.partner_id.id or
                picking.origin != m.origin for m in moves):
            # If a picking is found, we'll append `move` to its move list and thus its
            # `partner_id` and `ref` field will refer to multiple records. In this
            # case, we chose to  wipe them.
            picking.write({
                'partner_id': False,
                'origin': False,
            })
In the context of RMA when we have multiple moves associated with a picking, each coming from a different RMA order line, we encounter a problem. Each move has its origin set as the name of the RMA orde line (line.name), so as soon as a second move from a different line is appended to the picking, the origin of the picking is wiped, because it doesn't match the origin of the first move.

In order to prevent the partner_id of the picking from being set to False when there are multiple associated moves, I propose that we change the origin of the procurement from the name of the RMA line to the name of the procurement group (group.name). This way, all moves associated with a picking will share the same origin, preserving the origin of the picking and ensuring it doesn't get inadvertently set to False.
2024-05-06 17:07:50 +02:00
DavidJForgeFlow
0667f3a146 [FIX] rma: dropship from vendor as outgoing 2024-05-06 17:07:50 +02:00
DavidJForgeFlow
60d770b208 [FIX] rma: some fixups 2024-05-06 17:07:50 +02:00
DavidJForgeFlow
e601c198ba [FIX] rma: make picking product_uom_qty 2024-05-06 17:07:50 +02:00
DavidJForgeFlow
8497049155 [FIX] rma: product_uom_qty not in move_line_ids 2024-05-06 17:07:50 +02:00
Lois Rilo
1dd8eb10c0 [FIX] rma: in multi step routes, only reserve first step
We shoul not force reservation on next steps on a multi step
route, oherwise a inconsistency is generated and the transfers
cannot be processed or cancel so the user gets stuck ("it is
not possible to unreserve more products that you have in stock"
error).
2024-05-06 17:07:50 +02:00
DavidJForgeFlow
f826845def [14.0][FIX] rma: add stock move in supplier group 2024-05-06 17:07:50 +02:00
Lois Rilo
739f67beca [IMP] rma: add description to rma order and copy it over to new lines 2024-05-06 17:07:50 +02:00
Lois Rilo
5f0f1c1681 [IMP] rma: rma group supplier form view as a inherited view.
The goal is to simplify view defintion and do not duplicate
things, requiring to do view changes in two places.

The same change was done from rma lines some time ago.
2024-05-06 17:07:50 +02:00
Lois Rilo
62d501d020 [FIX] rma: Ensure that configuration on the operation is applied
Without this, some policies are not being copied from the
operation selected when creating new rma line from a rma group.

In v16 this patch and the usage of such onchange can be removed
in favor of (pre)computed stored editable fields for all policies
and configuration in the RMA operation.
2024-05-06 17:07:50 +02:00
Florian da Costa
258021bdd1 [16.0][MIG] Migrate rma module to v16 2024-05-06 17:07:50 +02:00
DavidJForgeFlow
9db7ad8552 [15.0][FIX] rma_sale: fixup of procurement 2024-05-06 17:07:50 +02:00
Stefan Rijnhart
f411c0e3d0 [IMP] rma: prevent the creation of zero qty moves 2024-05-06 17:07:50 +02:00
Florian da Costa
f01f571888 [IMP] Make rma order view cleaner for user
Hide button and fields depending on the policy chosen on the rma line
2024-05-06 17:07:50 +02:00
Stefan Rijnhart
501418d132 [FIX] rma: improve check on rule selection during procurement
Fixes #274

Thanks to @florian-dacosta for suggesting this approach.
2024-05-06 17:07:50 +02:00
Stefan Rijnhart
faa1959d5e [FIX] rma: prevent against warehouse mismatch or missing rules
When creating pickings, ensure that the applied stock rule was taken from
the operation's routes. Otherwise, the default procurement rules for a
warehouse may kick in, creating incoming customer goods not from the customer
location but from the resupply warehouse.
2024-05-06 17:07:50 +02:00
Lois Rilo
747ac243c9 [IMP] rma: add rma lines to group selecting serial numbers 2024-05-06 17:07:50 +02:00
Florian da Costa
bcfa5a32c2 [FIX] Remove useless/bad wiz_id field in rma_make_picking.wizard item preparation 2024-05-06 17:07:50 +02:00
AaronHForgeFlow
f90e6d3096 [IMP] COPIER UPDATE: black, isort, prettier 2024-05-06 17:07:50 +02:00
Lois Rilo
764139d560 [FIX] rma: do not force reservation on supplier RMA deliveries
* remove code call to unexisting `force_asign`.
* only one model per model file and rename as appropriate.
2024-05-06 17:07:50 +02:00
Christopher Ormaza
c83918400d [FIX] rma: add lot to pickings created from wizard on RMA lines 2024-05-06 17:07:50 +02:00
Christopher Ormaza
dfd91f9ba4 [IMP] rma, rma_sale: added select all, messages for UX 2024-05-06 17:07:50 +02:00
Christopher Ormaza
9845c0f99d [IMP] rma: Added serial/lot selection on from stock move wizard on rma groups
- added restriction to approve rma with product tracking on serial, should be only one to receive
2024-05-06 17:07:50 +02:00
Jasmin Solanki
5771e63a55 [MIG] rma: Migration to 15.0 2024-05-06 17:07:50 +02:00
Lois Rilo
27960192f5 [14.0][MIG] rma*: ir.actions.act_window has different access
right in v14. Actions that read those records need to use
`sudo` to allow non-admin users to be able to use these actions.
2024-05-06 17:07:50 +02:00
Lois Rilo
c042bf3d2e [IMP] rma: Do not highlight all action buttons and simplify
rma.line views, supplier view is an extension of customer one.

This makes easier to edit common elements in both views and ease
maintenance.
2024-05-06 17:07:50 +02:00
Mateu Griful
9743e8d613 [FIX] rma: Create rma supplier action 2024-05-06 17:07:50 +02:00
Mateu Griful
b9f7d1ade7 [MIG] rma: Migration to 14.0 - fix 2024-05-06 17:07:50 +02:00
Mateu Griful
f6b27598fa [MIG] rma: Migration to 14.0 - fix 2024-05-06 17:07:50 +02:00
Mateu Griful
e069c204d0 [MIG] rma: Migration to 14.0 - fix 2024-05-06 17:07:50 +02:00
Mateu Griful
33fa9b0702 [MIG] rma: Migration to 14.0 2024-05-06 17:07:50 +02:00
Mateu Griful
5819ced3b4 [IMP] rma: black, isort, prettier 2024-05-06 17:07:50 +02:00
HviorForgeFlow
4ebef78ce4 [IMP] Update pre-commit lints according with OCA ones 2024-05-06 17:07:50 +02:00
hveficent
850c5ddc82 RMA as an APP 2024-05-06 17:07:50 +02:00
ahenriquez
db6a348e40 [MIG] rma: Migration to 13.0 2024-05-06 17:07:50 +02:00
ahenriquez
c4c1a4ea1a [IMP] : black, isort 2024-05-06 17:07:50 +02:00
mreficent
99e6cd069c [FIX] default_gets: avoid using shadowname 'fields' 2024-05-06 17:07:50 +02:00
Aaron Henriquez
ce95f8247c [FIX]wizard to create pickings, not to show rma groups if rma groups are not activated 2024-05-06 17:07:50 +02:00
Akim Juillerat
2053503429 [FIX] Remove picking_id from default_get as model does not define it 2024-05-06 17:07:50 +02:00
Akim Juillerat
86c1a44d48 rma: Fix stock_location res.groups 2024-05-06 17:07:50 +02:00
Adrià Gil Sorribes
424c278041 [12.0][MIG] Migrate rma module to v12.0 2024-05-06 17:07:50 +02:00