Created attachment 37415 [details] [review] Bug 13936: Last patch contained wrong change One of the 2 changes done in the last patch was not for this bug report. The tt change was intended for bug 13926. This patch revert it. ! BE CARE: This should NOT be pushed AFTER bug 13926!
Test plan?
Created attachment 37535 [details] [review] Bug 13936: Last patch contained wrong change One of the 2 changes done in the last patch was not for this bug report. The tt change was intended for bug 13926. This patch reverts it. BE CARE: This should NOT be pushed AFTER bug 13926! Test plan: Search for a patron using the 'check out' tab in the circulation module.
Created attachment 37561 [details] [review] Bug 13936: Last patch contained wrong change One of the 2 changes done in the last patch was not for this bug report. The tt change was intended for bug 13926. This patch reverts it. BE CARE: This should NOT be pushed AFTER bug 13926! Test plan: Search for a patron using the 'check out' tab in the circulation module.
Created attachment 37562 [details] [review] Bug 13936: Last patch contained wrong change One of the 2 changes done in the last patch was not for this bug report. The tt change was intended for bug 13926. This patch reverts it. BE CARE: This should NOT be pushed AFTER bug 13926! Test plan: Search for a patron using the 'check out' tab in the circulation module. Signed-off-by: Nicolas Legrand <nicolas.legrand@bulac.fr>
Created attachment 37632 [details] [review] [PASSED QA] Bug 13936: Last patch contained wrong change One of the 2 changes done in the last patch was not for this bug report. The tt change was intended for bug 13926. This patch reverts it. BE CARE: This should NOT be pushed AFTER bug 13926! Test plan: Search for a patron using the 'check out' tab in the circulation module. Signed-off-by: Nicolas Legrand <nicolas.legrand@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> - use the checkout tab to search for a patron not using the autocomplete, but hitting enter - without the patch, there won't be results - with the patch there are Passes all tests and QA script.
Patch pushed to master. Thanks JOnathan!
Pinging Chris
(In reply to Tomás Cohen Arazi from comment #8) > Pinging Chris This has been pushed to master before the 3.20 release, so it can be marked as resolved.