I tested this on an early June version of master, following Shae's steps and received the same 500 internal server error.
I then tested it with the patch applied to master and when I try to renew an item that has a booking I get a Permission Denied error, which is better than the 500 server error, and is the behavior described in the patch. So I think this is working like it should and takes care of the 500 server error.
I have tested this code and consent to signing off on it with my
name, Josh Stompro and email address, <email address hidden>.
I tested this on an early June version of master, following Shae's steps and received the same 500 internal server error.
I then tested it with the patch applied to master and when I try to renew an item that has a booking I get a Permission Denied error, which is better than the 500 server error, and is the behavior described in the patch. So I think this is working like it should and takes care of the 500 server error.
I have tested this code and consent to signing off on it with my
name, Josh Stompro and email address, <email address hidden>.