For items in the cart (opac-basket.pl), only location and call no. are displayed, change this page to display item availability status as well.
Created attachment 9874 [details] [review] Attached Patch. Attached Patch.
Created attachment 9878 [details] Screenshot showing inaccurate availability information The availability information shown now in the cart is inaccurate. As this screenshot shows, for a title with 4 items, 1 checked out, the cart says "Available" once. For this information to be accurate there needs to be a way to connect the availability/status information directly to each copy listed.
Created attachment 9907 [details] [review] Sending Improved Patch.
Comment on attachment 9874 [details] [review] Attached Patch. Don't forget to mark old patches obsolete when you submit revised versions.
Created attachment 9914 [details] Screenshot showing inaccurate availability information The revised patch is an improvement but there is still no connection between the detailed status information and the list of items. Comparing the cart view on the left with the detail view of the right we can see that the cart isn't showing accurate information. The cart view is going to have to tie the status information directly to the location/call number information somehow, probably by putting it in the same table cell, or perhaps by moving all the the item information into a nested table.
I also notice this patch doesn't add any information to the cart view one reaches by clicking the "more details" link.
Created attachment 9918 [details] [review] Latest Patch
Created attachment 9952 [details] Screenshot showing inaccurate availability information This patch is improving, but still isn't giving entirely accurate information. As this screenshot shows, lost, damaged, and withdrawn show correctly. However, checked-out and in-transit do not. Hold status is unknown on the Cart listing, but ambiguous on the detail page so I'm not sure the burden is on this patch to resolve it.
Created attachment 9978 [details] [review] Attaches Patch. Sending Improved patch for accurate availability status in cart. From this patch, Showing checked out and In transmit status in cart. But Hold is out of scope. So this patch doesn't showing Hold status.
Created attachment 10112 [details] [review] [SIGNED-OFF] Bug 7570 - Add Item availability status to cart. Added Item availability status to cart(Location table cell). Changes: -opac/opac-basket.pl: Added code to show In transmit status and modified @items to call "GetItemsInfo" instead of "GetItemsLocationInfo" to get accurate avalibility status. -koha-tmpl/opac-tmpl/prog/en/modules/opac-basket.tt: Added code for showing availibility staus in opac-basket page and in "More details" link. Test Cases: -Add records to your cart with different cases like 1 record with different Item status(Lost,Avaliable,Withdrawn,etc). -Click on your cart and see the results in a new window. -Click on "More details" link and check the results. Signed-off-by: Michael Davis <m.r.davis@cranfield.ac.uk>
Created attachment 10113 [details] [review] bug 7570: add a space to the table header Add a space to separate Location from (Status)
Created attachment 10115 [details] [review] [SIGNED-OFF] bug 7570: add a space to the table header Add a space to separate Location from (Status) Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Typo fix, all ok.
QA comment: * patch adding dependencies on C4::Circulation and C4::Branch, that is quite heavy, but the page is rarely used, so it's OK * The patch is only a few line, no other objection
Patch pushed, I've updated the follow-up to add another space in another similar place where we had no space in location(s)(status)