Steps to reproduce: 1) Open the Acquisition module 2) At the top of the page click "Orders search" 3) Fill-in a book title that has been already received and submit the form 4) Title is found displaying (among others) these columns: - Quantity received - Pending order The Pending order is NOT pending, in fact. The header should be renamed to "Quantity ordered".
Ccing Sonia, would you agree with the column's header renaming or do you think it's better to update the value (quantity ordered - quantity received)?
Hum... in my opinion, as when we partially received an order, it creates a new order, the quantity ordered isn't so interesting. It's just important to fix the number of pending orders. Just my 2 cents, I'm not sure what would be better for the greatest number of librarians. Sonia
I'd agree with Sonia. The total quantity ordered is important at the very beginning of acquisition process, but whe the order is received partially, it shoudl display the result of "ordered MINUS received", which makes more sense, I think. Or, another proposal... What if we display in the column "Pending order" a value like "2 of 5"?
Created attachment 97262 [details] [review] Bug 17611: Change 'Pending order' to 'Quantity ordered' Picking Quantity ordered as it seems more in line with other pages in Koha where we show information. We always show the quantity, so it makes sense to also have this label on the 'order search' and 'copy existing order'. Order search 1) Make sure you have some orders in your acq module 2) Go to the top and do an empty orders search 3) Review the column headings of the search results 4) Find "Pending order" Order from existing order (copy) 1) Create or choose an existing open basket 2) 'Add to basket' - 'From existing orders' 3) Search to find some of your orders 4) Look at the column headings, find 'Pending order' 5) Pick some orders and fill out order information, save 6) Summary: Find 'Pending order' in the table again x) Apply the patch and make sure all 'Pending' orders above have been changed to 'Quantity ordered'.
Don't be mad at me, but after reviewing other pages in the acquisition module and testing the partial receive process, I've decided to use the 'Quantity ordered' instead of changing the content of the column. It feels more in line with the other pages in the acquisition module for now.
Created attachment 97390 [details] [review] Bug 17611: Change 'Pending order' to 'Quantity ordered' Picking Quantity ordered as it seems more in line with other pages in Koha where we show information. We always show the quantity, so it makes sense to also have this label on the 'order search' and 'copy existing order'. Order search 1) Make sure you have some orders in your acq module 2) Go to the top and do an empty orders search 3) Review the column headings of the search results 4) Find "Pending order" Order from existing order (copy) 1) Create or choose an existing open basket 2) 'Add to basket' - 'From existing orders' 3) Search to find some of your orders 4) Look at the column headings, find 'Pending order' 5) Pick some orders and fill out order information, save 6) Summary: Find 'Pending order' in the table again x) Apply the patch and make sure all 'Pending' orders above have been changed to 'Quantity ordered'. Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 98583 [details] [review] Bug 17611: Change 'Pending order' to 'Quantity ordered' Picking Quantity ordered as it seems more in line with other pages in Koha where we show information. We always show the quantity, so it makes sense to also have this label on the 'order search' and 'copy existing order'. Order search 1) Make sure you have some orders in your acq module 2) Go to the top and do an empty orders search 3) Review the column headings of the search results 4) Find "Pending order" Order from existing order (copy) 1) Create or choose an existing open basket 2) 'Add to basket' - 'From existing orders' 3) Search to find some of your orders 4) Look at the column headings, find 'Pending order' 5) Pick some orders and fill out order information, save 6) Summary: Find 'Pending order' in the table again x) Apply the patch and make sure all 'Pending' orders above have been changed to 'Quantity ordered'. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Bouzid Fergani <bouzid.fergani@inlibro.com>
Created attachment 99266 [details] [review] Bug 17611: Change 'Pending order' to 'Quantity ordered' Picking Quantity ordered as it seems more in line with other pages in Koha where we show information. We always show the quantity, so it makes sense to also have this label on the 'order search' and 'copy existing order'. Order search 1) Make sure you have some orders in your acq module 2) Go to the top and do an empty orders search 3) Review the column headings of the search results 4) Find "Pending order" Order from existing order (copy) 1) Create or choose an existing open basket 2) 'Add to basket' - 'From existing orders' 3) Search to find some of your orders 4) Look at the column headings, find 'Pending order' 5) Pick some orders and fill out order information, save 6) Summary: Find 'Pending order' in the table again x) Apply the patch and make sure all 'Pending' orders above have been changed to 'Quantity ordered'. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Bouzid Fergani <bouzid.fergani@inlibro.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
String fix.. Passing QA
Nice work everyone! Pushed to master for 20.05
enhancement not backported to 19.11.x