Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] stock: remove move 'without package' properly #36558

Closed
wants to merge 1 commit into from

Conversation

@Whenrow
Copy link
Contributor

commented Sep 9, 2019

on a picking, move_ids_without_package is a subset of move_lines.
The inverse function on move_ids_without_package only work well when we
add a move.

This commit improves the inverse function to take deletion into account

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

on a picking, move_ids_without_package is a subset of move_lines.
The inverse function on move_ids_without_package only work well when we
add a move.

This commit improves the inverse function to take deletion into account
@sle-odoo sle-odoo force-pushed the odoo-dev:saas-12.3-delete-move-whe branch from 6dec2e9 to 2ed1ff5 Sep 20, 2019
@sle-odoo

This comment has been minimized.

Copy link
Contributor

commented Sep 20, 2019

robodoo r+

robodoo pushed a commit that referenced this pull request Sep 20, 2019
on a picking, move_ids_without_package is a subset of move_lines.
The inverse function on move_ids_without_package only work well when we
add a move.

This commit improves the inverse function to take deletion into account

closes #36558

Signed-off-by: Simon Lejeune (sle) <sle@openerp.com>
@robodoo

This comment has been minimized.

Copy link
Contributor

commented Sep 20, 2019

Merged at 3431d7b, thanks!

@robodoo robodoo closed this Sep 20, 2019
@Whenrow Whenrow deleted the odoo-dev:saas-12.3-delete-move-whe branch Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.