Since bug 28653, the checkouts table is automatically refreshed after checkin/renew. This prevents seeing error messages that are normally put inside this table. Steps to reproduce: 1. Apply the patch on bug 27249 2. Follow the test plan in bug 27249 comment 3 3. See that the table is automatically refreshed. You should see a message replace the checkbox for a few milliseconds, and then the message will be replaced by a checkbox.
Thx for filing, Julian. I noticed the other day, it also prevents the success messages for reneals and checkins from being visible to the user.
Created attachment 153969 [details] [review] Bug 34302: DO NOT PUSH - to test failure Really, do not push! :D
Created attachment 153970 [details] [review] Bug 34302: (bug 28653 follow-up) Do not refresh the table if an error happened If checkin or renew failed, we should not refresh the table or it will hide the error message. Test plan: Apply the DO NOT PUSH patch Do a renew => No error in the table Apply this patch Do a renew => You see the error Revert the DO NOT PUSH patch Do a renew => The table is refreshed
Created attachment 155251 [details] [review] Bug 34302: (bug 28653 follow-up) Do not refresh the table if an error happened If checkin or renew failed, we should not refresh the table or it will hide the error message. Test plan: Apply the DO NOT PUSH patch Do a renew => No error in the table Apply this patch Do a renew => You see the error Revert the DO NOT PUSH patch Do a renew => The table is refreshed Signed-off-by: David Cook <dcook@prosentient.com.au>
Weird, I don't get the error. I also tried with "Always show checkouts automatically" ticked and I don't see the difference when doing renewals with just the "the DO NOT PUSH patch" and both patches. Services have been restarted after patch applications. Should the error be in the checkout list? I only get the yellow message about confirming renewal on the top of the page.
Created attachment 155910 [details] [review] Bug 34302: (bug 28653 follow-up) Do not refresh the table if an error happened If checkin or renew failed, we should not refresh the table or it will hide the error message. Test plan: Apply the DO NOT PUSH patch Do a renew => No error in the table Apply this patch Do a renew => You see the error Revert the DO NOT PUSH patch Do a renew => The table is refreshed Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
I believe this might only be a partial fix. We have still lost the success messages.
(In reply to Katrin Fischer from comment #7) > I believe this might only be a partial fix. We have still lost the success > messages. This is much more complicated...
Pushed to master for 23.11. Nice work everyone, thanks!
Pushed to 23.05.x for 23.05.04
Nice work everyone! Pushed to oldstable for 22.11.x