Bug 34252 - Ability to link to ILL request in external ILL system
Summary: Ability to link to ILL request in external ILL system
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: ILL (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Bugs List
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-07-11 13:39 UTC by Katrin Fischer
Modified: 2023-11-30 09:10 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2023-07-11 13:39:28 UTC
When implementing backends for ILL it would sometimes be helfpul if we could link to the request in the external system or portal. 

The backends could create/fill this link. If set, it should display in the request detail page in the ILL module.
Comment 1 Pedro Amorim 2023-07-11 13:51:51 UTC
Does the existent accessurl column in illrequests satisfy this requirement?
Comment 2 Katrin Fischer 2023-07-11 14:32:59 UTC
Hm, I am not sure. We don't use that yet and I thought it was supposed to be the URL to the document, like a delivery URL.
Comment 3 Pedro Amorim 2023-07-11 15:02:57 UTC
(In reply to Katrin Fischer from comment #2)
> Hm, I am not sure. We don't use that yet and I thought it was supposed to be
> the URL to the document, like a delivery URL.

What about orderid? We use that to store the ID of the order placed in the supplier in some of our backends.
Comment 4 Katrin Fischer 2023-07-16 11:51:27 UTC
The order ID can be used to construct the URL. I think was looking for some way to add this nicely to the GUI without resorting to jQuery. 

Might need some more thought.

Having something configurable might be nice.
Comment 5 Lisette Scheer 2023-11-29 19:14:26 UTC
Could we add a column to the table externalurl that's hidden by default but plugins could choose to fill per backend?
Comment 6 Pedro Amorim 2023-11-30 09:10:32 UTC
(In reply to Lisette Scheer from comment #5)
> Could we add a column to the table externalurl that's hidden by default but
> plugins could choose to fill per backend?

Yes, I think this would be ideal. Use illrequestattributes instead of adding a new column to the illrequests table.

To add to this suggestion, we could also consider a way to make the table columns themselves hookable, so that plugins are able to add columns dynamically. This, of course, is a lot more work but would also address future needs similar to this one (without having to push to core every time). This is just my opinion though, take it for what it is!