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

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

Merge pull request #55 from hgq/08_06_pt15169665_head

08 06 pt15169665 head

[Fixes #15466993] tweak verbiage of debt collect by using notmanual flag

[delivers #14668229] Broken advanced search with branch groups

Parentheses were missing from around branches list.

[Fixes #15475375] goto UpdateStats differently in UpdateFine()
depending on insert (fine_new) or update (fine_update)

[Fixes #15169665] use locale dateformat from syspref instead of hardcoded
US date.

[Fixes #15484033] stray code was preventing processing of refund
owed for payment on lost item upon actual checkin

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

[#15440423] Fix several XSS vectors

As listed in http://en.securitylab.ru/lab/PT-2011-05.

Merge pull request #51 from hgq/08_03_pt1531769_head

[Fixes #15351769] regex was failing to match November for hold resumedate

Merge pull request #50 from hgq/08_01_pt14997333

resubmit for [Fixes #14997333] really fix problem of found=NULL,priority=0:

[delivers #15409175] Editing receipt templates requires wrong perms

SCLS-CIRC-2200-1 - Receipt Templates - BugFix - Fix Permissions

[Fixes #15351769] regex was failing to match November for hold resumedate

[Fixes #14997333] really fix problem of found=NULL,priority=0:
GetOtherReserves() should be deprecated.  Anytime priorities or
found status change, all the Reserves.pm routines call _Normalize*,
so there's no need to decrement priorities anymore.  Also, when we
requeue the impossible case of Waiting at A but checkin at B, requeue
as bib-level hold.

Update code micro version to 4.7.3

Fix Koha::RoseDB to work with Plack-local contexts

Fix error in C4::Context::DESTROY()

Rely on default Plack staff resolver by default in app.psgi

Add benchmarking Plack middleware

Provide default Plack staff resolver function

[delivers #15285829] Provide a function to resolve Plack localizer configuration

Update Plack's method of canonicalizing the hostname

[delivers #15284861] Explicitly tear down Z39.50 connections

Merge pull request #49 from hgq/08_02_pt5928993

[Fixes #5928993] fix regex in CCL query to correctly capture

Merge pull request #46 from hgq/07_05_pt13091533_head

[Fixes #13091533] autoBarcode off: do blank barcode on Add+Duplicate

Merge pull request #44 from hgq/07_04_pt15121665_head

[Fixes #15121665] prevent double-submit of checkin form when pressing Ent

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

Allow for dynamic multi-instance servicing

By handing off a hash of hostname-to-KOHA_CONF mappings to the
Koha::Plack::Localize middleware via the host_map parameter,
a single running Plack instance can be used to service multiple
installations for distinct libraries. The absence of this
parameter causes the traditional $ENV{KOHA_CONF} to be used.

It may be useful in the future to also add a "host_mapper"
parameter which could be handed a coderef that would return
the appropriate config file based on any number of factors.

[Fixes #5928993] fix regex in CCL query to correctly capture
from and to daterange, actually was not end of string

Fix for inaccessible $template variable in receipt template editor

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

Merge pull request #47 from ctfliblime/PT15117555

PTFS PT 11434853 remove paging from Holds Queue Report

[delivers #15123975, #15117555] Authval code caching hooked to wrong key

Single-quoting the cache key was causing all authval codes to
reference the first lookup that gets called. Interpolating the
string fixes this.

[Fixes #13091533] autoBarcode off: do blank barcode on Add+Duplicate

[Fixes #15121665] prevent double-submit of checkin form when pressing Enter,
esp. in IE.  After observing IE's behavior on the screen, I noticed that IE
emulates actually pressing on the Submit button.  Previous code submitted
the form once in setOverride*() upon capturing the Entering key and again
upon submitting the form by pressing the Submit button.  I split up the
functionality of submitting the form so that pressing the Enter key does
not submit the form.  Also tested against other browsers and works OK.

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

Merge pull request #42 from hgq/07_03_pt13590281_head

[addresses #13590281] comment out broken syspref ReservesNeedReturns

Update code micro version to 4.7.2

[addresses #13590281] comment out broken syspref ReservesNeedReturns

Merge branch 'opac-speedup-many-items' into ptfs-master

Add flexibility to how staff vs. OPAC interface is determined

Rename C4::Plack::* to Koha::Plack::*

Eliminate duplicate item details collection in OPAC

If the OPAC is using XSLT for search results display it does
not need to gather item details when collating search result
data. This patch simply triggers that process based on those
conditions.

Remove abandoned data collection in C4::Items

I can find no evidence of this data being read from any templates
or library functions.