To test: 1. Do an item search that returns results 2. Use the column filtering available at the top of the table for each column 3. Try itemnumber, get a 500 error each time 4. As long as there is data in the itemnumber column and you attempt to use another column you'll see a 500 error. Without anything in the itemnumber column: 5. Try filtering by home library, currently library, item type, or shevling location. 6. The filtering doesn't work unless you type the code, not the description.
Created attachment 168478 [details] [review] Bug 37249: disambiguate itemnumber with USING clause 1. Do an item search that returns results 2. Use the column filtering available at the top of the table for each column 3. Try itemnumber, get a 500 error each time 4. As long as there is data in the itemnumber column and you attempt to use another column you'll see a 500 error. 5. APPLY PATCH, restart_all 6. The itemnumber column filtering should now work as expected
Created attachment 168486 [details] [review] Bug 37249: disambiguate itemnumber with USING clause 1. Do an item search that returns results 2. Use the column filtering available at the top of the table for each column 3. Try itemnumber, get a 500 error each time 4. As long as there is data in the itemnumber column and you attempt to use another column you'll see a 500 error. 5. APPLY PATCH, restart_all 6. The itemnumber column filtering should now work as expected Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org>
Created attachment 168505 [details] [review] Bug 37249: disambiguate itemnumber with USING clause 1. Do an item search that returns results 2. Use the column filtering available at the top of the table for each column 3. Try itemnumber, get a 500 error each time 4. As long as there is data in the itemnumber column and you attempt to use another column you'll see a 500 error. 5. APPLY PATCH, restart_all 6. The itemnumber column filtering should now work as expected Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org> Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Ok, help me :) This works, but I would have thought that with the table as prefix, they already should have been disambiguated. Can you explain?
Pushed for 24.11! Well done everyone, thank you!
Backported to 24.05.x for upcoming 24.05.06