[Koha-bugs] [Bug 22407] OMNIBUS: Use DBIC relations to fetch related object rather than searching for the object

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

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

--- Comment #17 from David Cook <dcook at prosentient.com.au> ---
(In reply to Julian Maurice from comment #16)
> In my opinion "maybe we will want to add something here" is not a good
> reason to refactor code. If that "maybe" ever comes true, then you can
> refactor the code while making the change. No need to do it now.

Given the number of legacy projects I've maintained over the years, we'll just
have to agree to disagree on that one ;).

> Note that I also tend to prefer your version, but I'm happy with both.
> If it ain't broke don't fix it ;)

True enough. There are better hills to die on heh.

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