Skip to content

Comments

Fixed: Disable 'New Shipment For Ship Group' buttons for purchase orders that are not approved (OFBIZ-13337)#935

Merged
JacquesLeRoux merged 2 commits intoapache:trunkfrom
golja:disable_new_shipment_button_for_not_approved_orders
Jan 12, 2026
Merged

Fixed: Disable 'New Shipment For Ship Group' buttons for purchase orders that are not approved (OFBIZ-13337)#935
JacquesLeRoux merged 2 commits intoapache:trunkfrom
golja:disable_new_shipment_button_for_not_approved_orders

Conversation

@golja
Copy link
Contributor

@golja golja commented Jan 9, 2026

Explanation: When a PO is created but not yet approved, the Order Details screen displays the “New Shipment For Ship Group” buttons, which allows shipments to be created prior to order approval. This PR fixes this behavior by hiding the buttons until the order has been approved.

This is one of the issues reported in Step-by-step workflow: purchase orders with multiple shipments and invoices

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 9, 2026

@JacquesLeRoux
Copy link
Contributor

Hi Anahita,

It seems to me that it does more than expected. Also with the error: "ERROR: Sales Order Lines lookup failed." Is that intended?
Image 019

@golja
Copy link
Contributor Author

golja commented Jan 12, 2026

Hi Jacques,
I can’t reproduce the error you reported.
My changes should affect only purchase orders, not sales orders.
Could you please provide the steps to reproduce the issue?

@JacquesLeRoux
Copy link
Contributor

Hi Anihita,

Oops, my bad. Too lazy to create a non approved PO :/
It's OK with me, pushing...

@JacquesLeRoux JacquesLeRoux merged commit 31075bd into apache:trunk Jan 12, 2026
7 checks passed
@JacquesLeRoux
Copy link
Contributor

I let you handle the Jira...

@golja
Copy link
Contributor Author

golja commented Jan 12, 2026

Done! Thanks for the review.
Anahita

asf-gitbox-commits pushed a commit that referenced this pull request Jan 12, 2026
…ers that are not approved (OFBIZ-13337) (#935)

Explanation: When a PO is created but not yet approved, the Order
Details screen displays the “New Shipment For Ship Group” buttons, which
allows shipments to be created prior to order approval. This PR fixes
this behavior by hiding the buttons until the order has been approved.

This is one of the issues reported in [Step-by-step workflow: purchase
orders with multiple shipments and
invoices](https://cwiki.apache.org/confluence/display/OFBENDUSER/Step-by-step+workflow%3A+purchase+orders+with+multiple+shipments+and+invoices)
@golja golja deleted the disable_new_shipment_button_for_not_approved_orders branch January 15, 2026 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants