Bug 21702: (follow-up) Simplify checks and use standard fallback behaviour
authorTomas Cohen Arazi <tomascohen@theke.io>
Sat, 27 Oct 2018 12:22:00 +0000 (09:22 -0300)
committerNick Clemens <nick@bywatersolutions.com>
Wed, 31 Oct 2018 13:15:01 +0000 (13:15 +0000)
commita90502043f12055b7e6b61f9c23d924d2e02b999
tree428e8d360d9ff22ebf0150fc03da49dc7e6c075f
parent466e29263910c85ce6417f322dc505558eeaa8a3
Bug 21702: (follow-up) Simplify checks and use standard fallback behaviour

This patch acknowledges the fact that some checks here are done in a way
that is not standard these days. This checks are originally done
multiple times in different ways. output_and_exit_if_error should be
used.

This implies a minor behaviour change: if the passed (in the URL)
borrowernumber doesn't exist, it sends the user to a 'Patron doesn't
exist page' instead of the circulation page for the borrowernumber.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

Signed-off-by: Andrew Isherwood <andrew.isherwood@ptfs-europe.com>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
members/mancredit.pl