circulation: support non-circulating libraries

[BadrAly]

The transaction locations of a library is selected in the list of pickup
locations of the library. This causes issues with libraries that have
no pickup location setted.

With this commit, for libraries with no pickup locations the system
considers the first library location as transaction location for
circulation transactions. These libraries can now checkout items
directly from other libraries and once checked-in back they go
in_transit.
It is still mandatory to setup library hours and notification
settings before checkout items.

• Upgrades to invenio-circulation v1.0.0a34.
• Supports checkout of items of libraries with no opening hours externally.
• Supports fixed checkout of items of libraries with no opening hours externally.
• Fixes problem when in transit flash message was always displaying the pickup library.
• Fixes situation A of the issue 2419.
• Supports checkout of items of libraries with no pickup locations.
• Closes Checkout possible only after cancelling the request of the patron #2367.
• Closes Server error if the checkout is made from a library that does not have any pick-up location #2318.
• Adds a non-circulating library to units testing.

Co-Authored-by: Aly Badr aly.badr@rero.ch

Why are you opening this PR?

https://tree.taiga.io/project/rero21-reroils/task/2280
#2367

How to test:

• make sure there is a library in your organisation with no pickup_location defined.
• have a patron to place a request on any item of this library to pick it up at anther library.
• the librarian of this library should be able to send the item to the requested library without problem
• the librarian of the requested library should be able to receive the item and checkout to the requested patron
• the item can be returned with no problem at the owning library
• USE-CASE 2: checkout directly the item at the requested library without requesting it.

Code review check list

• Commit message template compliance.
• Commit message without typos.
• File names.
• Functions names.
• Functions docstrings.
• Unnecessary commited files?
• Cypress tests successful?

[pronguen]

• Item of a library without pick-up location stays in transit
  • This does not happen with item of a library with pick-up location
  • To reproduce
    • Log in in a library without pick-up location (ex: pid:24)
    • Check out an item of this library (ex: pid:3060)
    • Check in this item
    • See that it is in transit but should be on shelf
• Inconsistency between the transit destination in flash message and in interface
  • To reproduce
    • Log in in a library with pick-up location
    • Check out an item from a library without pick-up location
    • Check in the item
    • See inconsistency
      
  • Once in transit, we cannot checkin the item to change the status. Nothing happens.
• Unable to remove the pick-up attribute of location 11 ("Le formulaire contient des erreurs."). This problem does not happen on ils.test.rero.ch.
  https://user-images.githubusercontent.com/8154915/138037567-fa64c80e-6cd7-4ad3-a629-847d2f364c2a.mp4
• Fix date checkout does not work with item from a library without opening hours. However, these items can be checked-out without using the fixed date feature.
  • To reproduce
    • Find a library without opening hours
    • Take an item from this library
    • Log in from a library with opening hours
    • Do a standard checkout > it works fine
    • Do a fixed date checkout > error

[iGormilhit]

Commit message proposition:

circulation: support non-circulating libraries

The transaction locations of a library is selected in the list of pickup
locations of the library.  This causes issues with libraries that have
no pickup location setted.

With this commit, for libraries with no pickup locations the system
considers the first library location as transaction location for
circulation transactions.

* Supports checkout of items of libraries with no opening hours.
* Allows requests on items of non-circulating libraries at external
  locations.
* Closes #2367, #2419.
* Adds a non-circulating library to units testing.