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

[FW][FIX] website_sale: removing invalid product from the cart #40241

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Nov 14, 2019

Reproduce the issue:

  • Install eCommerce app
  • Publish a product to your website
  • Add the product in your cart
  • Unpublish/make unsellable/archive the product

The product is still in the cart and the checkout process can be done.

Cause: There is no check for invalid products before displaying the cart

This commit remove the invalid products while loading the cart.

OPW-2092541

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

Forward-Port-Of: #40168
Forward-Port-Of: #39931

Reproduce the issue:
- Install eCommerce app
- Publish a product to your website
- Add the product in your cart
- Unpublish/make unsellable/archive the product

The product is still in the cart and the checkout process can be done.

Cause: There is no check for invalid products before displaying the cart

This commit remove the invalid products while loading the cart.

OPW-2092541

X-original-commit: ab9290e
@fw-bot
Copy link
Contributor Author

fw-bot commented Nov 14, 2019

Ping @jasonvanmalder, @nim-odoo
This PR targets master and is the last of the forward-port chain.

To merge the full chain, say

@fw-bot r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@robodoo robodoo added the forwardport This PR was created by @fw-bot label Nov 14, 2019
@nim-odoo
Copy link
Contributor

@fw-bot r+

@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Nov 14, 2019
@jvnm-dev
Copy link

@nim-odoo Can you robodoo retry, please? Thanks!

@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Nov 26, 2019
robodoo pushed a commit that referenced this pull request Nov 26, 2019
Reproduce the issue:
- Install eCommerce app
- Publish a product to your website
- Add the product in your cart
- Unpublish/make unsellable/archive the product

The product is still in the cart and the checkout process can be done.

Cause: There is no check for invalid products before displaying the cart

This commit remove the invalid products while loading the cart.

OPW-2092541

closes #40241

X-original-commit: ab9290e
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
@robodoo robodoo closed this Nov 26, 2019
@robodoo robodoo temporarily deployed to merge November 26, 2019 09:08 Inactive
@fw-bot fw-bot deleted the master-12.0-opw-2092541-fix-ecommerce_cart_archived_unpublished_unsellable_product-jvm-4tNe-fw branch December 10, 2019 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses forwardport This PR was created by @fw-bot OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants