[Koha-bugs] [Bug 11299] Add a button to automatically link authority records in cataloguing (AJAX)

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Fri May 11 15:14:05 CEST 2018


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

--- Comment #72 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
(In reply to Blou from comment #70)
> If the value of the feature can't pass your smell test, I can't recommend
> that the user throw good money after bad into the re-engineering of the
> feature, indefinitely, to follow the latest architectural whims.  They coded
> this in 2010ish, so you can bet it's always playing catch-up, code-wise.

Coded in 2010 and submitted to the community in November 2013.

I do not need to read the test plan or the content of the patch when I see
that:
 C4/Biblio.pm                                       | 138 +++++++++++++-
 C4/Linker.pm                                       |   5 +-
 C4/Linker/Default.pm                               |   2 +-
 t/Biblio.t                                         |   2 +

This definitely is code smell, tests for changes in C4 and Koha are required
for years now.

Several questions have been asked by Marcel 4 months ago, it would be good to
get  an answer first. That should not take much time to just... answer, right?

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


More information about the Koha-bugs mailing list