https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15599
Tomás Cohen Arazi <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Patch doesn't apply CC| |[email protected] --- Comment #10 from Tomás Cohen Arazi <[email protected]> --- I don't agree with Jonathan on this patch. I think tests (like Reserves.t) should create well-formed scenarios. If they are raising warnings, then they need to be fixed. And that's not the situation here. because we are explicitly creating this 'bad' scenario, to test the behaviour in that specific scenario. So, worth testing for the warning this time. I might be missing something anyway, so I'm open to discuss this. It doesn't apply BTW, because of the latest rebase of bug 14598. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
