[Bug 21284] New: ILS-DI: Allow GetPatronInfo to tell if a loaned item is on hold by someone else.

classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 21284] New: ILS-DI: Allow GetPatronInfo to tell if a loaned item is on hold by someone else.

bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21284

            Bug ID: 21284
           Summary: ILS-DI: Allow GetPatronInfo to tell if a loaned item
                    is on hold by someone else.
 Change sponsored?: ---
           Product: Koha
           Version: unspecified
          Hardware: All
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P5 - low
         Component: Web services
          Assignee: [hidden email]
          Reporter: [hidden email]
        QA Contact: [hidden email]

This patch adds two entries in the loans section of GetPatronInfo's response:

 - itemonhold: number of holds on this specific item.
 - recordonhold: number of holds on the record.


It allows an ILS-DI client to know if a loaned item is already on hold by
someone else, and how much holds there are.

--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://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/
Reply | Threaded
Open this post in threaded view
|

[Bug 21284] ILS-DI: Allow GetPatronInfo to tell if a loaned item is on hold by someone else.

bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21284

--- Comment #1 from Matthias Meusburger <[hidden email]> ---
Created attachment 78268
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=78268&action=edit
Bug 21284: ILS-DI: Allow GetPatronInfo to tell if a loaned item is on hold by
someone else.

This patch adds two entries in the loans section of GetPatronInfo's response:

 - itemonhold: number of holds on this specific item.
 - recordonhold: number of holds on the record.

Test plan:

 - Call GetPatronInfo on a user with a loan that has no holds
 - Check that itemonhold and recordonhold entries are equal to zero

 - Add holds on biblio and/or item level
 - Check that itemonhold and recordonhold are incremented accordingly

Please note that a hold on item level counts as a hold on record level, but not
vice-versa.

--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://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/
Reply | Threaded
Open this post in threaded view
|

[Bug 21284] ILS-DI: Allow GetPatronInfo to tell if a loaned item is on hold by someone else.

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21284

--- Comment #2 from Matthias Meusburger <[hidden email]> ---
Created attachment 78269
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=78269&action=edit
Bug 21284: Unit tests

--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://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/
Reply | Threaded
Open this post in threaded view
|

[Bug 21284] ILS-DI: Allow GetPatronInfo to tell if a loaned item is on hold by someone else.

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21284

Matthias Meusburger <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |Needs Signoff

--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://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/
Reply | Threaded
Open this post in threaded view
|

[Bug 21284] ILS-DI: Allow GetPatronInfo to tell if a loaned item is on hold by someone else.

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21284

Katrin Fischer <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
           Assignee|[hidden email]-commun |matthias.meusburger@biblibr
                   |ity.org                     |e.com

--- Comment #3 from Katrin Fischer <[hidden email]> ---
Hi Matts, please don't forget to assign the bug to yourself when providing a
patch!

--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://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/
Reply | Threaded
Open this post in threaded view
|

[Bug 21284] ILS-DI: Allow GetPatronInfo to tell if a loaned item is on hold by someone else.

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21284

Arthur Suzuki <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://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/
Reply | Threaded
Open this post in threaded view
|

[Bug 21284] ILS-DI: Allow GetPatronInfo to tell if a loaned item is on hold by someone else.

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21284

Fridolin SOMERS <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |In Discussion
                 CC|                            |[hidden email]
                   |                            |m

--- Comment #4 from Fridolin SOMERS <[hidden email]> ---
 use C4::Reserves qw(AddReserve CanBookBeReserved CanItemBeReserved
IsAvailableForItemLevelRequest);
 use C4::Context;
 use C4::AuthoritiesMarc;
+use C4::Reserves;
 use XML::Simple;

I think this is a mistake, use C4::Reserves already exists

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://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/