Micro release: 4.8.19

Improper variable scope in patron-attr-types.pl

Bad practice to export class methods

Also clean up useless debugging noise

Caching cleanups for C4::Members::Attributes

Micro release: 4.8.18

Merge pull request #154 from hgq/13_01_pt14554159_4_08

13 01 pt14554159 4 08

Merge pull request #156 from hgq/13_02_pt16905729_4_08

13 02 pt16905729 4 08

[delivers #17078993] Clean up caching of MARC framework maps

The meat of this is the changes to C4/Biblio.pm, using a safe
caching function instead of storing it in a package global. The
changes to C4/Items.pm are preventative cleanup to keep this
variable from being set at some point in the future.

[#16905729] lost item's title wasn't displayin in Checkout:
deprecate C4::Members::GetMemberLostItems() and use instead
previously fixed C4::LostItems::GetLostItems()

[#14554159] fines: fix floating point problem w/out changing schema

Micro release: 4.8.17

PT16741047: Turning off the syspref OpacHighlightedWords does not have any affect

The code was not utilizing the OpacHighlightedWords syspref.  This has
been fixed.

[delivers #16895329] Implement safe caching of MARC structure

Micro release: 4.8.16

PT17093249: Call to deprecated routine ModReserveFillCheckout in SIP2

Reserves::ModReserveFillCheckout was removed from the API.  Removed the
SIP call to this subroutine and cleaned up a few other references to
ModReserveFill which has also been deprecated.

PT16355737: Advanced patron search breaks masthead email and phone patron search

Removed the "Search Fields" drop-down menu in the patron-search.inc since
it was previously rendered non-functional.  Modified searching label as
well as a result.

PT15531909: Editing a notice file produces an error

Error was caused by variable scoping diffences within the
Plack environment. Switched global my(s) to our(s).

PT15650379: Cannot Save Changes to Leader

The record length (leader positions 0-4) was not being passed by
the MARC editor form variable.  A conditional was then expecting
the leader to be 24 characters when it was actually only 19 characters.

PT15595769: Debt collect script ignoring threshold balance

There was a disconnect between the debt_collect.pl program and the
OwedNotificationValue syspref.  Essentially, debt_collect.pl had
a static value of $25.  If OwedNotificationValue was higher than this
value, then the possibility existed to have a patron be sent to debt
collect despite their balance being below OwedNotificationValue.

In addition, the script wasn't parsing ignored patron categories correctly.

Merge pull request #146 from hgq/12_05_pt16905729_4_08

12 05 pt16905729 4 08

Merge pull request #143 from hgq/12_04_pt16894109_4_08

12 04 pt16894109 4 08

Merge pull request #136 from hgq/12_02_pt16408283_4_08

12 02 pt16408283 4 08

Micro release: 4.8.15

[deliver #17087005] Rose::DB registry not cleaned up

[#16905729] make lost item's title display

[followup to #16894109] suppress internal cron user from patron fines
in accountlines description

Merge pull request #121 from hgq/11_09_pt14999551_4_08

11 09 pt14999551 4 08

Micro release: 4.8.14

[delivers #16913917] Global namespace pollution in C4::Letters

%C4::Letters::handles can end up bleeding information from one
session to the next when running in a persistent environment.
Caching of this data does not create a material performance
improvement, so this patch just removes it altogether. If it
turns out there are some use cases where caching is indicated,
a new mechansim using CHI handles stored in C4::Context is
required.

[#16893161] (fixup) PDF::Reuse writes to wrong STDOUT under Plack

[#16408283] make label exports run under Plack
- our-ify some variables
- add __END__ token to prevent closure, exit, and POD clash

Micro release: 4.8.13

[#16893161] (fixup) comparing integers with string operators

[#16893161] (fixup) another sub calling package globals

Micro release: 4.8.12

Merge branch '4_08' of github.com:pjones9/ptfs-master into 4_08

PT16310341: SIP2 differentiation between item-level and title-level holds

The SIP2 code was essentially forcing a priority hold regardless of whether
it was a title-level or item-level hold.  Altered the code to account for
an item level hold by testing for an itemnumber and non-found status in the
reserves table.  Needed to regain patron info if an item level hold was in
place for an item that was being checked in.

[delivers #16893161] reference to unavailable package global in label-create-pdf.pl

Merge pull request #133 from hgq/12_01_pt16894109_4_08

12 01 pt16894109 4 08

[#16894109] suppress privacy sensitive info from OPAC fines display

Micro release: 4.8.11

Update incorrect exit() calls in reports and labels

Unix' convention is that a 0 exit status indicates success and
non-zero failure, the inverse of Perl/C/etc.

Micro release: 4.8.10

[delivers #16861699] Inject Biblios iframe link URL via $KOHA_CONF

In addition to $ENV{BIBLIOS_LINK} this option can now be added
through a <biblios_link>URL</biblios_link> tag within the
<config> section of the koha-conf.xml file.

Micro release: 4.8.9

[Fixes #16684897] Holds queue displaying incorrect call numbers

This fix uses items.itemcallnumber instead of tmp_holdsqueue.itemcallnumber.
A schema change is needed in the future to use foreign keys instead of
static stale data.

Micro release: 4.8.8

swap itemnumber and reservenumber in FillReserve w/ unshift

Merge pull request #130 from hgq/11_13_rmDupesHoldsQueue

11 13 rm dupes holds queue

one more place for dupecheck in build_holds_queue.pl

Micro release: 4.8.7

More C4::Members::GetMember specs

[delivers #16677493] Error when viewing patron lists

Micro release: 4.8.6

Merge pull request #129 from hgq/11_12_pt1667219_4_08

11 12 pt1667219 4 08

Completely specify GetMember package

[#16672179] internal server error on confirm checkout hold override
- fixed wrong way to get superlibrarian permission
- fixed namespace problem in Circulation.pm calling ModReserve

Micro release: 4.8.5

[#16468403] (fixup) Account for discrepancy in itype definition

[#16029551] (fixup) Was dereferencing a non-reference

Micro release: 4.8.4

Merge pull request #124 from hgq/11_10_pt16468403_4_08

11 10 pt16468403 4 08

[#16468403] fix no userenv branch when running longoverdue cron
and we do a MarkLostItemsReturned upon a Lost+Charge on issued item,
and we have to get the branch for looking up which issuing rule to use

[#14999551] catch other cases of cascading dialog forms, esp. lost+transfer

Merge pull request #103 from ctfliblime/PT16371557-4_08

Pt16371557 4 08

[#16042719] (followup) Normalize null options and default values of sysprefs

[delivers #16371557] Cordon off administrative functions from web access

Directories such as misc/ C4/ Koha/ and so on should not be accessible
via URI. This patch uses middleware to exclude access to shell-only
areas.

DEPENDENCIES:
 * ADDS: Plack::Middleware::Status

[delivers #16336385] Deprecate Koha::Plack::ScrubStatus

DEPENDENCIES:
 * ADDS: Plack::Middleware::Header

Merge pull request #100 from ctfliblime/PT16029551-4_08

Pt16029551 4 08

[delivers #16029551] COinS calculator dying on unchecked return value

If, as in the case of a newly deleted title, an OPAC search result list
includes a title that no longer exists, the COinS creator would die
from accessing an unchecked return value. This patch explicitly croaks
in that case and places catch()es within the caller to emit a simple
warn instead of dying.

Micro release: 4.8.3

Merge pull request #94 from hgq/11_02_pt16152385_4_08

11 02 pt16152385 4 08

[fixes #16152385] our above subroutines in main under Plack... this needs to
be done in so many places

Merge pull request #91 from hgq/10_10_pt14149993_4_08

10 10 pt14149993 4 08

minor tweaks:
- suppress Perl warning of uninit var in numeric comparison of
  $lasttime when there isn't any (maybe we logged out properly?)
- Plack eval()'s expect clean exits, so installer.pl was backwards
  and never reached login page after Web installation step 3,
  cannot exit nonzero

[#15592081] differentiate verbiage between returned and renewed in fines

[fixes #14149993] make a transfer at checkin: lost, notlost, &c. Other fixes:
- fix undef Perl warning in mapping in fines.pl
- fix overdue from datedue calculation, was using object instead of ISO date
  from issues
- finally finished work on refund owed for backdate checkin resulting
  in downward adjustment of fine
- converted Yes|No option to Continue prompt: Remove from patron's Lost Items
- retested and fixed some lost prompts +notissued/issued, +hold/nohold, &c.

[fixes #14999551] fix general case of lost:missing (not lost+charge),
notissued, noholds.  Tested against other cases:
case 1) lostNcharge+nohold+notissued
     1.a. checkin at A - OK
     1.b. checkin at B - OK
        1.b.ii. had previous transfer - OK
case 2) lostMissing+nothold+notissued
     2.a. checkin at A - OK, click <OK> and form will now submit
     2.b. checkin at B
        2.b.i.  no prior transfer   - OK
        2.b.ii. have prior transfer - OK

[addresses #15528107 and #15895609] more holds and tranfer fixes
- handle case: item already waiting at pickup branch, checkin at pickup
  branch; relax previous assumption that item is permanent collection (not always true)
- allow ignore hold upon checkin at pickup library and not yet waiting
  (don't automatically set hold to waiting in AddReturn())

[Fixes #15841545] some fixes for intransit holds
- <Confirm and transfer> button was dead
- patron details: On Holds tab was displaying both intransit and waiting
- put intransit and add to branchtransfers when notlost, notissued
* TODO: fault tolerance for intransit but no entry in branchtransfers

[fixes #15483899] longoverdue script needs to honor syspref MarkLostItemsReturned

Merge pull request #90 from ctfliblime/PT15591737-4_08

Pt15591737 4 08

Merge pull request #88 from ctfliblime/PT16042719-4_08

Pt16042719 4 08

[delivers #15591737] Fix improper use of cache handles

C4::Koha::GetAuthorisedValue had been improperly refactored
to use the CHI cache object handle rather than the computed
cache contents. This of course resulted in all calls to
GetAuthorisedValue to return undef.

[delivers #16042719] Installer should load default sysprefs

Micro release: 4.8.2

PT15596575: SCLS sandbox failed ITG selfcheck (HIGH -- SC 15960)

Previous processing of the item due date was written as if the date was
a C4::Dates object.  It isn't, so I removed the call to its output method.

PT13633157: OPAC not obeying 942$n bib suppression setting (SC #14321)

If the framework allows the biblionumber (999$c) field to be displayed and
edited within the MARC editor, then this results in an additional 999$c value
being added to the MARC record.  This is because the code already pulled the
biblionumber from the biblio table.  This patch removes the addition of the
extra 999$c field, although this field shouldn't really be editable in the
first place.

[fixes #15236079] spine label typo Spilt to Split

Merge pull request #80 from hgq/10_07_pt14986797_4_08

10 07 pt14986797 4 08

[fixes #14986797] fixes for fastadd
- in circulation, allow creation of bib and item using FA framework which,
  after deliberation, is decided to be hardcoded rather than controlled by
  syspref.  If the FA framework doesn't exist yet, the librarian will get
  a friendly warning message but can proceed w/ fastadd and checkout using
  default framework
- get 952$c explicitly (the barcode) for displaying at top of framework in
  additem.pl, eliminating the bug that the field's text never displayed
  and merely left behind a hyphen since code previously used regex instead
  of hash keys.  This also incidentally fixes the bug that attempting to
  submit the form fails due to a JS error with trying to pass rmBarcode
  over a nondisplaying field

Micro release: 4.8.1

Update Makefile.PL with new dependencies

[delivers #15796177] Enable setting of timezone in $KOHA_CONF

Koha can't rely on environment variables if using Plack
with multiple libraries since those vars could vary from
instance to instance. This patch allows setting TZ based
on instance-specific configuration info.

Additionally, this displays the active local time on the
about.pl page.

Have middleware copy existing %ENV into localized %ENV

Neglected to include JSON module from C4::Context

Minor release: 4.8.0

Merge pull request #57 from ctfliblime/PT14668229

PTFS PT 11434853 remove paging from Holds Queue Report

Merge pull request #56 from hgq/08_07_pt15475375_head

[Fixes #15475375] goto UpdateStats differently in UpdateFine()

Merge branch 'ptfs-master' of git@github.com:pjones9/ptfs-master into ptfs-master