If an video has a UPC but not an ISBN the cover art does not show on the results page. the UPC code is used to present the cover art on the details page but not the results page. Ed
Created attachment 54940 [details] [review] Bug 15388 - Show Syndetics covers by UPC in search results Syndetics code accepts UPC and OCLC numbers, however, we were only passing this if there was an ISBN. This patch alters the code to search if either an ISBN or UPC is present To test: Enable syndetics (will need an account) Search for items with UPCs or OCLC numbers and no ISBN (DVDs) Note there are no images in search results but there are in details Apply patch Note that items with images in details now also have images in results
Created attachment 55007 [details] [review] [SIGNED-OFF] Bug 15388 - Show Syndetics covers by UPC in search results Syndetics code accepts UPC and OCLC numbers, however, we were only passing this if there was an ISBN. This patch alters the code to search if either an ISBN or UPC is present To test: Enable syndetics (will need an account) Search for items with UPCs or OCLC numbers and no ISBN (DVDs) Note there are no images in search results but there are in details Apply patch Note that items with images in details now also have images in results Signed-off-by: Barbara.Johnson@bedfordtx.gov
Would like to see this QA'd. We're running into this problem.
Created attachment 56315 [details] [review] [PASSED QA] Bug 15388 - Show Syndetics covers by UPC in search results Syndetics code accepts UPC and OCLC numbers, however, we were only passing this if there was an ISBN. This patch alters the code to search if either an ISBN or UPC is present To test: Enable syndetics (will need an account) Search for items with UPCs or OCLC numbers and no ISBN (DVDs) Note there are no images in search results but there are in details Apply patch Note that items with images in details now also have images in results Signed-off-by: Barbara.Johnson@bedfordtx.gov Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Minor change, all works as expected, no breakage.
Pushed to Master - Should be in the November 16.11 Release. Thanks!
Can this be ported to 3.22?
I second Christopher's request to port this fix to 3.22.
*** Bug 12420 has been marked as a duplicate of this bug. ***
Pushed in 16.05. Will be in 16.05.05.