From bugzilla-daemon@bugs.koha-community.org Wed Nov  1 11:53:25 2017
From: bugzilla-daemon@bugs.koha-community.org
To: koha-bugs@lists.koha-community.org
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to	Circulation
 and OPAC
Date: Wed, 01 Nov 2017 10:53:21 +0000
Message-ID: <bug-7317-70-GElhIpwDPK@https.bugs.koha-community.org/bugzilla3/>
In-Reply-To: <bug-7317-70@https.bugs.koha-community.org/bugzilla3/>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============8206782820083274462=="

--===============8206782820083274462==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

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

Magnus Enger <magnus@libriotech.no> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #68874|0                           |1
        is obsolete|                            |

--- Comment #169 from Magnus Enger <magnus@libriotech.no> ---
Created attachment 68878
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=3D68878&action=3D=
edit
Bug 7317: Handle backend absense more gracefuly

5/ This patch is WIP, it makes Koha::Illrequest->load_backend raise an
exception
if the passed backend is invalid. This way we will catch more errors
introduced.

The patch also disables the 'New Ill request' when no backends are available.
Gets
rid of a related warning.

TODO: Should add a noticeable warning on the UI, or a tooltip on the button.
TODO: add a couple tests for the exception

4/ This patch fixes the path for the checkboxes jquery plugin, and removes the
include
for tablesorter, as this implementation uses Datatables. This is obviously co=
de
for older
Koha, ported to master.

3/ I know it doesn't look right, but am not sure how to display. This is the
usual empty
datatable display we have everywhere in Koha.

2/ We could fix it with a proper IDE, I use Komodo and refactoring code is
pretty straigh-forward.
Worth a separate bug report.

1/ That TODO is misleading. It refers to their own migration from a different
design. I checked this
code and cannot find occurences of ->status with that annotation. I think this
relates to backends code.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

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

--===============8206782820083274462==--


