[Koha-bugs] [Bug 32980] Checkouts for guarantees are not shown if patron also has a guarantor

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32980

--- Comment #18 from Julian Maurice <julian.maurice at biblibre.com> ---
(In reply to Nick Clemens from comment #17)
> I think we gain consistency - if we always use builder, then it makes it
> easy to see how to write tests. It is also safer for naming, to avoid
> collision with possible test data in the system generated by humans (CCCCCCC
> is something we could add for testing, and suddenly your tests are failing)
> If there are fields you expect to be empty when building, we have sub
> _gen_default_values and should add more cases as needed
'CCCC...' is a pattern for String::Random, so the risk of collision is minimal
(and the same as with TestBuilder I think ?).
I agree with the consistency point, but I really don't like that TestBuilder is
populating columns I don't want to populate. I'll see if I can make this
behaviour configurable (in another bug report)

-- 
You are receiving this mail because:
You are watching all bug changes.