https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35581
Pedro Amorim <pedro.amo...@ptfs-europe.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #163812|0 |1
is obsolete| |
--- Comment #91 from Pedro Amorim <pedro.amo...@ptfs-europe.com> ---
Created attachment 164078
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164078&action=edit
Bug 35581: Koha::Illrequest -> Koha::ILL::Request
Koha/Illrequests.pm -> Koha/ILL/Requests.pm
Merged:
t/db_dependent/Koha/Illrequests.t
t/db_dependent/Illrequests.t
Into:
t/db_dependent/Koha/ILL/Requests.t
ILL classes file structure is, for the most part, around 7 years old and
doesn't follow a strict logic. It's so confusing that some test files exist
redundantly.
This housekeeping should help future work in regards to ISO18626 to add Koha as
a supplying agency instead of just requesting agency, as is now.
It should also help future housekeeping of moving backend related logic out of
the Illrequest.pm into Illbackend.pm (now ILL/Request.pm and ILL/Backend.pm as
of this patchset).
It should also help in structuring the addition of a master generic form (see
bug 35570)
This patchset will require existing backends to be updated to match the new
class names and structure, if they invoke them.
Test plan, k-t-d, run tests:
prove t/db_dependent/api/v1/ill_*
prove t/db_dependent/Koha/ILL/*
Test plan, k-t-d, manual:
1) Install FreeForm, enable ILL module, run:
bash <(curl -s
https://raw.githubusercontent.com/ammopt/koha-ill-dev/master/start-ill-dev.sh)
2) You'll have to switch the FreeForm repo to the one compatible with this
work, like:
cd /kohadevbox/koha/Koha/Illbackends/FreeForm
git checkout reorganize_ILL
3) Do some generic ILL testing:
3.1) Create a request
3.2) Add a comment to a request
3.3) Edit a request
3.4) Edit a request's item metadata
3.5) Confirm a request
3.6) List requests
3.7) Filter requests list using left side filters
4) Install a metadata enrichment plugin:
https://github.com/PTFS-Europe/koha-plugin-api-pubmed
4.1) Create an ILL batch and insert a pubmedid like 123
4.2) Add the request and finish batch
5) Verify all of the above works as expected
Signed-off-by: David Nind <da...@davidnind.com>
Signed-off-by: Tomas Cohen Arazi <tomasco...@theke.io>
Signed-off-by: Pedro Amorim <pedro.amo...@ptfs-europe.com>
--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
