Thanks for testing Mike. I have it installed on a local 2.2.2 and I cannot duplicate the error you are seeing. Very odd. The change itself should just add a simple extra parameter in the query. It sounds like it might not be needed though anyways.
Although I do come across folks who like to purge their data manually in the database, and a simple change like this would protect the hold shelf list from items that were deleted outside of the normal methods. I understand that it's not something the system really should anticipate or protect against (manual data manipulation) but to some degree there would be a benefit to a change like this.
Thanks for testing Mike. I have it installed on a local 2.2.2 and I cannot duplicate the error you are seeing. Very odd. The change itself should just add a simple extra parameter in the query. It sounds like it might not be needed though anyways.
Although I do come across folks who like to purge their data manually in the database, and a simple change like this would protect the hold shelf list from items that were deleted outside of the normal methods. I understand that it's not something the system really should anticipate or protect against (manual data manipulation) but to some degree there would be a benefit to a change like this.
Steve