Commit Graph

39 Commits

Author SHA1 Message Date
Christopher Ormaza
72adc1ba5b [16.0][FIX] rma, add support to handle package used on customer moves 2024-10-23 08:31:40 +02:00
BernatPForgeFlow
e43156e027 [FIX] rma: Do not set 'Destination Package' if RMA package comes from 'Customers' location 2024-01-09 13:22:26 +01:00
Christopher Ormaza
73b104175a [FIX] quants can be more than one on _gather function result 2024-01-09 12:55:35 +01:00
BernatPForgeFlow
38719ebf21 [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-01-09 12:54:17 +01:00
Florian da Costa
03ecdff7c4 [FIX] Delete rma_picking_wizard_item on cascade to avoid failure 2023-11-28 10:58:57 +00:00
Jordi Ballester Alomar
a42f14292d Merge pull request #413 from ForgeFlow/16.0-fix-rma_sml_product_uom_qty
[16.0][FIX] rma: multiple fixes
2023-07-07 19:00:19 +02:00
Joan Sisquella
2feb96db80 [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.
2023-05-30 15:49:37 +02:00
DavidJForgeFlow
3ba3dc1f85 [FIX] rma: dropship from vendor as outgoing 2023-05-02 10:13:08 +02:00
DavidJForgeFlow
decf9667bb [FIX] rma: make picking product_uom_qty 2023-03-09 18:19:17 +01:00
DavidJForgeFlow
621e5d4ec1 [FIX] rma: product_uom_qty not in move_line_ids 2023-03-03 17:00:11 +01:00
Lois Rilo
37088edc47 [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).
2023-02-21 15:06:38 +01:00
Florian da Costa
2559cb4660 [16.0][MIG] Migrate rma module to v16 2022-11-28 16:24:49 +01:00
DavidJForgeFlow
284891c09b [15.0][FIX] rma_sale: fixup of procurement 2022-11-28 16:11:02 +01:00
Stefan Rijnhart
60a9d6decb [IMP] rma: prevent the creation of zero qty moves 2022-11-28 16:11:02 +01:00
Stefan Rijnhart
a4e20f6abb [FIX] rma: improve check on rule selection during procurement
Fixes #274

Thanks to @florian-dacosta for suggesting this approach.
2022-11-28 16:11:02 +01:00
Stefan Rijnhart
cddf72d6c0 [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.
2022-11-28 16:11:02 +01:00
Florian da Costa
6bb73225f2 [FIX] Remove useless/bad wiz_id field in rma_make_picking.wizard item preparation 2022-11-28 16:11:02 +01:00
Lois Rilo
c2d491ede6 [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.
2022-11-28 16:11:02 +01:00
Christopher Ormaza
3b3675501f [FIX] rma: add lot to pickings created from wizard on RMA lines 2022-11-28 16:11:02 +01:00
Jasmin Solanki
7bc8ca36b1 [MIG] rma: Migration to 15.0 2022-11-28 16:11:02 +01:00
Mateu Griful
c8d492814b [MIG] rma: Migration to 14.0 - fix 2022-11-28 16:11:02 +01:00
Mateu Griful
b7c138f31b [MIG] rma: Migration to 14.0 - fix 2022-11-28 16:11:02 +01:00
hveficent
c141fd0660 RMA as an APP 2022-11-28 16:11:02 +01:00
ahenriquez
8b202f9b97 [MIG] rma: Migration to 13.0 2022-11-28 16:11:02 +01:00
ahenriquez
e0c8c99df4 [IMP] : black, isort 2022-11-28 16:11:02 +01:00
mreficent
1ba0793010 [FIX] default_gets: avoid using shadowname 'fields' 2022-11-28 16:11:02 +01:00
Adrià Gil Sorribes
6716a34b7b [12.0][MIG] Migrate rma module to v12.0 2022-11-28 16:11:02 +01:00
aheficent
29e30d5ec9 [MIG]rma v110.0.2.0.0 2022-11-28 16:11:02 +01:00
Bhavesh Odedra
88f6ce72cd [FIX] TypeError: unhashable type: 'list' 2022-11-28 16:11:02 +01:00
Jordi Ballester
bcb9122649 multiple fixes 2022-11-28 16:11:02 +01:00
Maxime Chambreuil
3c74ad1414 [FIX] Errors from tests 2022-11-28 16:11:02 +01:00
Maxime Chambreuil
7171fabb30 [MIG] Migrate configuration and cleanup 2022-11-28 16:11:02 +01:00
Maxime Chambreuil
6e787bc3e0 [FIX] Permissions and remove (en)coding 2022-11-28 16:11:02 +01:00
Bhavesh Odedra
178e2e6bae [11.0] MIG: RMA module 2022-11-28 16:11:02 +01:00
Nikul Chaudhary
2f1fba7594 [IMP] Improved Unit Test Case and Fixed Travis 2022-11-28 16:11:02 +01:00
aheficent
ca540457d6 [MIG]rma v10 2022-11-28 16:11:02 +01:00
lreficent
df1d5a483f [9.0][REW] rma: workflow centralized on rma.order.line and the use of rma.order is optional. 2022-11-28 16:11:02 +01:00
lreficent
0d4c438e26 [9.0][FIX] rma:
* fix assignment of moves.
* default qty in rma lines.
* remove account dependency.
* test and flake8 fixes.
2022-11-28 16:11:02 +01:00
Jordi Ballester
51d233bca8 init branch 2022-11-28 16:11:02 +01:00