Bug 18837 - Add an unmediated Interlibrary Loans workflow
Summary: Add an unmediated Interlibrary Loans workflow
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: ILL (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Target Milestone: ---
Assignee: Andrew Isherwood
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 21063
Blocks: 20563
  Show dependency treegraph
 
Reported: 2017-06-21 14:44 UTC by Alex Sassmannshausen
Modified: 2018-11-19 14:36 UTC (History)
7 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Bot Control: ---
When did the bot last check this:
Who signed the patch off:
Text to go in the release notes:


Attachments
Add ILL Unmediated syspref stuff. (3.13 KB, patch)
2017-08-30 15:43 UTC, Alex Sassmannshausen
Details | Diff | Splinter Review
Introduce delegation to backends for unmediated workflows. (2.48 KB, patch)
2017-08-30 15:43 UTC, Alex Sassmannshausen
Details | Diff | Splinter Review
Bug 18837: Add ILL Unmediated syspref stuff (3.31 KB, patch)
2018-04-26 11:09 UTC, Andrew Isherwood
Details | Diff | Splinter Review
Bug 18837: Introduce delegation to backends for unmediated workflows (2.00 KB, patch)
2018-04-26 11:09 UTC, Andrew Isherwood
Details | Diff | Splinter Review
Bug 18837: Add ILL Unmediated syspref stuff (3.45 KB, patch)
2018-09-27 16:45 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 18837: Introduce delegation to backends for unmediated workflows (2.16 KB, patch)
2018-09-27 16:45 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 18837: Add ILL Unmediated syspref stuff (3.47 KB, patch)
2018-11-09 11:45 UTC, Andrew Isherwood
Details | Diff | Splinter Review
Bug 18837: Introduce delegation to backends for unmediated workflows (2.16 KB, patch)
2018-11-09 11:45 UTC, Andrew Isherwood
Details | Diff | Splinter Review
Bug 18837: Add ILL Unmediated syspref stuff (3.47 KB, patch)
2018-11-19 14:36 UTC, Andrew Isherwood
Details | Diff | Splinter Review
Bug 18837: Introduce delegation to backends for unmediated workflows (2.16 KB, patch)
2018-11-19 14:36 UTC, Andrew Isherwood
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Sassmannshausen 2017-06-21 14:44:04 UTC
Some libraries may wish to allow their patrons to place Interlibrary loan requests without those requests going through a formal moderation process.

The ILLModule cannot guarantee an unmediated workflow for every backend.

I propose the following approach:
- A system preference switching this functionality on & off.
- Use ILL module's generic capabilities functionality to allow execution of unmediated requests on a per backend basis, but falling back on the normal process for backends that don't support unmediated.

Alex
Comment 1 Alex Sassmannshausen 2017-08-30 15:43:29 UTC
Created attachment 66628 [details] [review]
Add ILL Unmediated syspref stuff.

* installer/data/mysql/atomicupdate/illunmediated_tables.sql: New
  file.
* installer/data/mysql/sysprefs.sql: Add `ILLModuleUnmediated`.
* koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref:
  Add `ILLModuleUnmediated` handling.
Comment 2 Alex Sassmannshausen 2017-08-30 15:43:32 UTC
Created attachment 66629 [details] [review]
Introduce delegation to backends for unmediated workflows.

* Koha/Illrequest.pm (backend_create): Add dispatch.
Comment 3 Andrew Isherwood 2018-04-26 11:09:10 UTC
Created attachment 74874 [details] [review]
Bug 18837: Add ILL Unmediated syspref stuff

This patch adds the syspref requirements for unmediated ILL requests. It
is a recreation of
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66628&action=diff
which had diverged sufficiently as to make it impossible to apply.

* installer/data/mysql/atomicupdate/illunmediated_tables.sql: New
  file.
* installer/data/mysql/sysprefs.sql: Add `ILLModuleUnmediated`.
* koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref:
    Add `ILLModuleUnmediated` handling.
Comment 4 Andrew Isherwood 2018-04-26 11:09:16 UTC
Created attachment 74875 [details] [review]
Bug 18837: Introduce delegation to backends for unmediated workflows

This patch adds support to Illrequest.pm to allow delegation of
unmediated requests to happen in backends that support them.

It is a recreation of
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66629&action=diff
which had diverged sufficiently as to make it impossible to apply
Comment 5 Katrin Fischer 2018-04-27 05:49:14 UTC
Hi Andrew, please always make sure to include a test plan.
Comment 6 Andrew Isherwood 2018-04-27 06:59:05 UTC
Hi Katrin

Thanks for the reminder. I'd not added test plans because these patches were simply direct replacements for Alex's, which had become impossible to apply. The work on this bug is incomplete so there's a good chance these patches will be obsoleted as I complete the work. I figured it was probably OK to not include test plans yet as no-one would be trying to test it as it's not yet in "Needs signoff", and won't be until the work is complete.

Hope that's OK.

Andrew
Comment 7 Katrin Fischer 2018-04-27 08:27:13 UTC
Ah sure, I missed ASSIGNED the status!
Comment 8 Andrew Isherwood 2018-05-09 13:49:30 UTC
To test:

- Apply patches
- Ensure ILL is enabled and you have at least one backend available that supports unmediated requests (currently only the BLDSS backend supports this)
- Enable "ILLModuleUnmediated" preference
- As an OPAC user, navigate to "Your summary" > "Your interlibrary loan requests"
- Select "Create a new request" > "BLDSS"
- Search for an item and create an interlibrary loan request
- Observe that in the "Your interlibrary loan requests" view, the request now has the status of "Requested" rather than the "New request" status that it would have in an mediated workflow.
Comment 9 Martin Renvoize 2018-09-27 16:45:10 UTC
Created attachment 79542 [details] [review]
Bug 18837: Add ILL Unmediated syspref stuff

This patch adds the syspref requirements for unmediated ILL requests. It
is a recreation of
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66628&action=diff
which had diverged sufficiently as to make it impossible to apply.

* installer/data/mysql/atomicupdate/illunmediated_tables.sql: New
  file.
* installer/data/mysql/sysprefs.sql: Add `ILLModuleUnmediated`.
* koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref:
    Add `ILLModuleUnmediated` handling.

Signed-off-by: Stephen Graham <s.graham4@herts.ac.uk>
Signed-off-by: David Peacock <d.m.peacock@herts.ac.uk>
Signed-off-by: Jayne Maisey <j.maisey@herts.ac.uk>
Comment 10 Martin Renvoize 2018-09-27 16:45:14 UTC
Created attachment 79543 [details] [review]
Bug 18837: Introduce delegation to backends for unmediated workflows

This patch adds support to Illrequest.pm to allow delegation of
unmediated requests to happen in backends that support them.

It is a recreation of
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66629&action=diff
which had diverged sufficiently as to make it impossible to apply

Signed-off-by: Stephen Graham <s.graham4@herts.ac.uk>
Signed-off-by: David Peacock <d.m.peacock@herts.ac.uk>
Signed-off-by: Jayne Maisey <j.maisey@herts.ac.uk>
Comment 11 Martin Renvoize 2018-09-27 16:46:30 UTC
Hertfordshire University have undertaken lots of testing upon this work and have been running it in production for some time now. 

They have asked me to sign it off on their behalf.. I have added their testing team to the patches signoff lines.

Thanks guys :)
Comment 12 Andrew Isherwood 2018-11-09 11:45:49 UTC
Created attachment 82158 [details] [review]
Bug 18837: Add ILL Unmediated syspref stuff

This patch adds the syspref requirements for unmediated ILL requests. It
is a recreation of
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66628&action=diff
which had diverged sufficiently as to make it impossible to apply.

* installer/data/mysql/atomicupdate/illunmediated_tables.sql: New
  file.
* installer/data/mysql/sysprefs.sql: Add `ILLModuleUnmediated`.
* koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref:
    Add `ILLModuleUnmediated` handling.

Signed-off-by: Stephen Graham <s.graham4@herts.ac.uk>
Signed-off-by: David Peacock <d.m.peacock@herts.ac.uk>
Signed-off-by: Jayne Maisey <j.maisey@herts.ac.uk>
Comment 13 Andrew Isherwood 2018-11-09 11:45:53 UTC
Created attachment 82159 [details] [review]
Bug 18837: Introduce delegation to backends for unmediated workflows

This patch adds support to Illrequest.pm to allow delegation of
unmediated requests to happen in backends that support them.

It is a recreation of
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66629&action=diff
which had diverged sufficiently as to make it impossible to apply

Signed-off-by: Stephen Graham <s.graham4@herts.ac.uk>
Signed-off-by: David Peacock <d.m.peacock@herts.ac.uk>
Signed-off-by: Jayne Maisey <j.maisey@herts.ac.uk>
Comment 14 Andrew Isherwood 2018-11-09 11:46:13 UTC
Rebased on top of current dependency tree
Comment 15 Andrew Isherwood 2018-11-19 14:36:28 UTC
Created attachment 82508 [details] [review]
Bug 18837: Add ILL Unmediated syspref stuff

This patch adds the syspref requirements for unmediated ILL requests. It
is a recreation of
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66628&action=diff
which had diverged sufficiently as to make it impossible to apply.

* installer/data/mysql/atomicupdate/illunmediated_tables.sql: New
  file.
* installer/data/mysql/sysprefs.sql: Add `ILLModuleUnmediated`.
* koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref:
    Add `ILLModuleUnmediated` handling.

Signed-off-by: Stephen Graham <s.graham4@herts.ac.uk>
Signed-off-by: David Peacock <d.m.peacock@herts.ac.uk>
Signed-off-by: Jayne Maisey <j.maisey@herts.ac.uk>
Comment 16 Andrew Isherwood 2018-11-19 14:36:33 UTC
Created attachment 82509 [details] [review]
Bug 18837: Introduce delegation to backends for unmediated workflows

This patch adds support to Illrequest.pm to allow delegation of
unmediated requests to happen in backends that support them.

It is a recreation of
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66629&action=diff
which had diverged sufficiently as to make it impossible to apply

Signed-off-by: Stephen Graham <s.graham4@herts.ac.uk>
Signed-off-by: David Peacock <d.m.peacock@herts.ac.uk>
Signed-off-by: Jayne Maisey <j.maisey@herts.ac.uk>