On the list of uncertain prices there is an option to show orders by "me" or "Everyone." Submitting this form appears to do nothing because the form action clears quantity and pricing information from the orders. When the page refreshes there are no results because the "uncertain" flag has been cleared along with other order info from *all* orders which have uncertain prices. In the template, the filter form includes the op=validate field which is the parameter that tells the script to perform the ModOrder operation. Removing that form field should correct the problem.
Created attachment 13530 [details] [review] Bug 9106 - Filtering list of uncertain price list by owner empties order information The form which filters the list of orders with uncertain prices includes a parameter which tells the script to modify the orders, resulting in order information being lost. This patch removes the unnecessary form field. To test you must have orders with uncertain prices, preferably from more than one user. Go to the uncertain prices page. Before applying this patch, submitting the "orders from" form would cause the page to refresh with no results. This is because the script cleared the orders of price, quantity, etc. information. After applying this patch the uncertain prices list should refresh and show the correct results, either your orders ("me") or all ("Everyone").
Created attachment 13598 [details] [review] Bug 9106 - Filtering list of uncertain price list by owner empties order information The form which filters the list of orders with uncertain prices includes a parameter which tells the script to modify the orders, resulting in order information being lost. This patch removes the unnecessary form field. To test you must have orders with uncertain prices, preferably from more than one user. Go to the uncertain prices page. Before applying this patch, submitting the "orders from" form would cause the page to refresh with no results. This is because the script cleared the orders of price, quantity, etc. information. After applying this patch the uncertain prices list should refresh and show the correct results, either your orders ("me") or all ("Everyone"). Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
patch tested, fixes the nasty problem QA comment: just remove one line in the template, passed QA SO & QA in one go
This patch has been pushed to master.
Pushed to 3.10.x, will be in 3.10.1
Pushed to 3.8.x, will be in 3.8.8