Commit Graph

41 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
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
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
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
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
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
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
Jasmin Solanki
5771e63a55 [MIG] rma: Migration to 15.0 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
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
Adrià Gil Sorribes
424c278041 [12.0][MIG] Migrate rma module to v12.0 2024-05-06 17:07:50 +02:00
aheficent
1c0f8927ea [MIG]rma v110.0.2.0.0 2024-05-06 17:07:50 +02:00
Bhavesh Odedra
e13df27c2b [FIX] TypeError: unhashable type: 'list' 2024-05-06 17:07:50 +02:00
Jordi Ballester
c91d48fd4d multiple fixes 2024-05-06 17:07:50 +02:00
Maxime Chambreuil
3435165198 [FIX] Errors from tests 2024-05-06 17:07:50 +02:00
Maxime Chambreuil
1a4d5d0fd4 [MIG] Migrate configuration and cleanup 2024-05-06 17:07:50 +02:00
Maxime Chambreuil
e5a5928651 [FIX] Permissions and remove (en)coding 2024-05-06 17:07:50 +02:00
Bhavesh Odedra
2967c167e7 [11.0] MIG: RMA module 2024-05-06 17:07:50 +02:00
Nikul Chaudhary
0ff91a2ea0 [IMP] Improved Unit Test Case and Fixed Travis 2024-05-06 17:07:50 +02:00
aheficent
18d6591106 [MIG]rma v10 2024-05-06 17:07:50 +02:00
lreficent
be384513e7 [9.0][REW] rma: workflow centralized on rma.order.line and the use of rma.order is optional. 2024-05-06 17:07:50 +02:00
lreficent
c894113a14 [9.0][FIX] rma:
* fix assignment of moves.
* default qty in rma lines.
* remove account dependency.
* test and flake8 fixes.
2024-05-06 17:07:50 +02:00
Jordi Ballester
58da75554a init branch 2024-05-06 17:07:50 +02:00