audrius/stock-rma
1
0
Fork 0
mirror of https://github.com/ForgeFlow/stock-rma.git synced 2025-01-21 12:57:49 +02:00
Code Issues Packages Projects Releases Wiki Activity
329 Commits 119 Branches 0 Tags
13.0
Commit Graph

329 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Joan Sisquella
fad3ab7a79 [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 17:39:08 +02:00
Jordi Ballester Alomar
1e5c97c626 Merge pull request #405 from ForgeFlow/13.0-bp-404 
[13.0][IMP] rma_account: add index in account.move.line to field 'rma_line_id'
 2023-02-28 09:03:44 +01:00
Jordi Ballester Alomar
37e70e1767 [IMP] rma_account: add index in account.move.line to field 'rma_line_id' 2023-02-28 08:57:14 +01:00
Lois Rilo
c0ef1a5576 Merge pull request #395 from ForgeFlow/13.0-bp-393 
[13.0][IMP] rma: limit state statusbar
 2023-02-22 15:25:36 +01:00
DavidJForgeFlow
afb0533a7d [IMP] rma: limit state statusbar 2023-02-21 15:14:32 +01:00
Lois Rilo
edac875c65 Merge pull request #391 from ForgeFlow/13.0-bp-390 
[13.0][FIX] rma: in multi step routes, only reserve first step
 2023-02-21 15:03:57 +01:00
Lois Rilo
d62bcca3ec [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 14:59:36 +01:00
Lois Rilo
243fb06b5b Merge pull request #368 from ForgeFlow/13.0-bp-367 
[13.0][FIX] rma: add stock move in supplier group
 2023-02-21 14:51:08 +01:00
Lois Rilo
03f2732c64 Merge pull request #383 from ForgeFlow/13.0-bp-378 
[13.0][IMP] rma_sale: button cancel
 2023-02-21 14:50:36 +01:00
Lois Rilo
e3a7ce28d1 Merge pull request #384 from ForgeFlow/13.0-bp-379 
[13.0][IMP] rma_repair: button cancel
 2023-02-21 14:49:07 +01:00
Lois Rilo
bb78c60333 Merge pull request #382 from ForgeFlow/13.0-bp-377 
[13.0][IMP] rma_purchase: button cancel
 2023-02-21 14:48:30 +01:00
Lois Rilo
3d13c142ed Merge pull request #366 from ForgeFlow/13.0-bp-364 
[13.0][FIX] rma_sale: description not passed to SO
 2023-02-21 14:47:53 +01:00
Lois Rilo
bb4167fec4 Merge pull request #381 from ForgeFlow/13.0-bp-376 
[13.0][IMP] rma: cancel rma_lines
 2023-02-21 14:47:29 +01:00
DavidJForgeFlow
6979edfb17 [IMP] rma_repair: button cancel 2023-02-20 20:23:31 +01:00
DavidJForgeFlow
dfe546ac32 [IMP] rma_sale: button cancel 2023-02-20 20:22:35 +01:00
DavidJForgeFlow
b2951eaeff [IMP] rma_purchase: button cancel 2023-02-20 20:21:45 +01:00
DavidJForgeFlow
7c414b9ff5 [IMP] rma: cancel rma_lines 2023-02-20 20:20:35 +01:00
Lois Rilo
7ce2e72ec5 Merge pull request #374 from ForgeFlow/13.0-bp-373 
[13.0][FIX] rma_sale: duplicate SO also link rma
 2023-02-13 09:51:47 +01:00
DavidJForgeFlow
51f3ab21b8 [14.0][FIX] rma_sale: duplicate SO also link rma 2023-02-13 09:34:14 +01:00
Lois Rilo
5d8822a7eb Merge pull request #369 from ForgeFlow/13.0-update-flake8 
13.0 Update Copier
 2023-02-07 10:17:41 +01:00
DavidJForgeFlow
b55c36a063 [13.0] Update Copier 2023-02-07 10:06:16 +01:00
DavidJForgeFlow
f7641a4ab5 [14.0][FIX] rma: add stock move in supplier group 2023-02-01 13:05:30 +01:00
DavidJForgeFlow
ca017414ab [14.0][FIX] rma_sale: description not passed to SO 2023-01-27 12:01:31 +01:00
Lois Rilo
52a33a7648 Merge pull request #336 from ForgeFlow/13.0-fp-288 
[13.0][FIX] fix empty parter on rma picking
 2022-12-27 09:23:09 +01:00
Lois Rilo
30a8bfaeaa Merge pull request #272 from ForgeFlow/13.0-bp-269 
[13.0][FIX]rma: remove test_rma dependency to Account
 2022-12-27 09:13:16 +01:00
Dariusz Kubiak
44bffbe49a [12.0][FIX] fix empty parter on rma picking 2022-12-01 10:50:15 +01:00
Jordi Ballester Alomar
dcc307f730 Merge pull request #282 from ForgeFlow/13.0-bp-266 
[13.0][IMP] Improve multi-company record rules
 2022-07-26 18:30:54 +02:00
Jordi Ballester Alomar
17950147dc Merge pull request #290 from ForgeFlow/13.0-fp-264 
[13.0][IMP] rma: prevent the creation of zero qty moves
 2022-07-26 18:29:54 +02:00
Jordi Ballester Alomar
ef796829b1 Merge pull request #293 from ForgeFlow/13.0-bp-273 
[13.0] Hide smart button when empty
 2022-07-26 18:29:08 +02:00
Jordi Ballester Alomar
4f8883305f Merge pull request #294 from ForgeFlow/13.0-bp-283 
[13.0][IMP] Make rma order view cleaner for user
 2022-07-26 18:28:53 +02:00
Florian da Costa
aa7253dc30 [IMP] Make rma order view cleaner for user 
Hide button and fields depending on the policy chosen on the rma line
 2022-07-18 10:25:52 +02:00
Florian da Costa
20a758082b Hide smart button when empty 2022-07-18 09:45:45 +02:00
Stefan Rijnhart
cdafac7a61 [IMP] rma: prevent the creation of zero qty moves 2022-07-11 09:11:05 +02:00
Cas Vissers
866e623f6a [IMP] Improve multi-company record rules 2022-06-28 09:39:13 +02:00
DavidJForgeFlow
a875f0f67a [FIX]rma: remove test_rma dependency to Account 2022-06-28 09:30:28 +02:00
Lois Rilo
2760f14ed0 Merge pull request #262 from ForgeFlow/13.0-fp-253 
[13.0][IMP] rma: add RMA reference to delivery slip report + add rma lines to group selecting serial numbers
 2022-05-27 11:40:07 +02:00
Lois Rilo
58e43ae0e3 [IMP] rma: add rma lines to group selecting serial numbers 2022-05-27 10:19:24 +02:00
Lois Rilo
51fba11626 [IMP] rma: add RMA reference to delivery slip report 2022-05-27 09:43:03 +02:00
Lois Rilo
55dde2cb27 Merge pull request #258 from ForgeFlow/13.0-fix-fp-248 
[13.0][FIX] rma:  missing migration scripts and version bump
 2022-05-25 18:04:25 +02:00
Lois Rilo
a7b99765dd [FIX] rma: missing migration scripts and version bump 
Those are needed after making some fields company dependent
 2022-05-25 17:57:40 +02:00
Jordi Ballester Alomar
3fa15935c7 Merge pull request #254 from ForgeFlow/13.0_fix_product_templates_multicompany 
[13.0][FIX/IMP] Make RMA Operation settings company dependent
 2022-05-19 19:57:18 +02:00
DavidFIB
cefbecca9e [FIX]rma: test05 fixed 2022-05-18 10:20:36 +02:00
Andrea
259d95da06 [13.0][FIX/IMP] Make RMA Operation settings company dependent 2022-05-17 17:36:02 +02:00
Lois Rilo
3fd1982ebb Merge pull request #239 from ForgeFlow/13.0-fix-separate_stock_moves_handle_serial_lot 
[13.0][FIX] rma: separate stock.move by rma_line_id to fix picking association
 2022-03-08 09:08:48 +01:00
Christopher Ormaza
07645ab72b [IMP] write method replaced for backported implementation 2022-03-07 08:23:17 -05:00
Christopher Ormaza
4a4d9d8ec8 [14.0][FIX] rma: separate stock.move by rma_line_id to fix picking association 2022-03-07 07:45:00 -05:00
Lois Rilo
7b7a8b4f2b [FIX] rma: do not force reservation on supplier RMA deliveries 2022-03-07 07:44:40 -05:00
Lois Rilo
ae61efe191 Merge pull request #228 from ForgeFlow/13.0_imp_rma 
[13.0][IMP] rma: Fix Route View
 2022-02-23 14:24:39 +01:00
Jasmin Solanki
31d3c9fa21 [IMP] rma: Fix Route View 2022-02-23 17:29:00 +05:30
Christopher Ormaza
2d3cbd05ae [13.0][ADD] Mass action for request approval RMA order line 2022-02-11 11:19:14 -05:00