Skip to content

[15.0][FIX] purchase_requisition_grouped_by_procurement: Correct grouping of purchase requisitions#2475

Merged
OCA-git-bot merged 1 commit intoOCA:15.0from
Tecnativa:15.0-fix-purchase_requisition_grouped_by_procurement
Dec 4, 2024
Merged

[15.0][FIX] purchase_requisition_grouped_by_procurement: Correct grouping of purchase requisitions#2475
OCA-git-bot merged 1 commit intoOCA:15.0from
Tecnativa:15.0-fix-purchase_requisition_grouped_by_procurement

Conversation

@carlos-lopez-tecnativa
Copy link
Copy Markdown
Contributor

Before this commit, if products had different customer_lead values, separate purchase.requisition records were incorrectly created instead of being grouped into a single record as intended.

Steps to reproduce:

  • Install the Sales module.
  • Configure two products with the following settings:
    • Different customer_lead values.
    • Procurement set to Propose a call for tenders.
    • Route set to Make to Order (MTO) and Buy.
  • Create a sales order with the two products and confirm it.
  • Go to Purchase > Purchase Agreements and search by the sales order name.

Issue: Two purchase requisition records are created instead of the expected single record.
Solution: This fix ensures that purchase requisitions are grouped correctly, even when products have different customer_lead values.

fwd-port of #2473

TT51858
@Tecnativa @pedrobaeza @victoralmau could you please review this

…f purchase requisitions

Before this commit, if products had different customer_lead values, separate purchase.requisition records were incorrectly created instead of being grouped into a single record as intended.

Steps to reproduce:

- Install the Sales module.
- Configure two products with the following settings:
  - Different customer_lead values.
  - Procurement set to Propose a call for tenders.
  - Route set to Make to Order (MTO) and Buy.
- Create a sales order with the two products and confirm it.
- Go to Purchase > Purchase Agreements and search by the sales order name.

Issue: Two purchase requisition records are created instead of the expected single record.
Solution: This fix ensures that purchase requisitions are grouped correctly, even when products have different customer_lead values.
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @victoralmau,
some modules you are maintaining are being modified, check this out!

@pedrobaeza pedrobaeza added this to the 15.0 milestone Nov 29, 2024
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same solution

@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot merge patch

1 similar comment
@HviorForgeFlow
Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 15.0-ocabot-merge-pr-2475-by-HviorForgeFlow-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Dec 4, 2024
Signed-off-by HviorForgeFlow
@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-2475-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 1cf6ab9. Thanks a lot for contributing to OCA. ❤️

@OCA-git-bot OCA-git-bot merged commit 4efa6af into OCA:15.0 Dec 4, 2024
@pedrobaeza pedrobaeza deleted the 15.0-fix-purchase_requisition_grouped_by_procurement branch December 4, 2024 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants