Micro update: 4.10.5

Merge pull request #345 from dlbptfs/PT27547969_04_10

Pt27547969 04 10

[#27547969] Clear negative notforloan status on checkin

This patch incorporates a new syspref (ClearNotForLoan) that, when enabled,
will set the item.notforloan status to zero when an item is checked in and
previously had a negative value.

Merge pull request #343 from ctfliblime/PT26875895-patroncats_4.10

[#26875895] Fixes related to ConstrainPatronsDeeply

[#26875895] Update member viewing scripts to still obey patron groups

[#26875895] Create helper function BranchesAreSiblings

[#26875895] Push check for group constraint into its own function

[#26875895] Add "ConstrainPatronsDeeply" syspref

Conflicts:

installer/data/syspref_defaults.json

Micro update: 4.10.4

[#27040815] Don't overwrite location with null permanent location

Micro update: 4.10.3

Write all params to error log in CatchErrors middleware

[#24229095] Cannot add matchchecks to matching rules

The functional piece of this is changing the iterated variable
of the match-check loop from @mp_nums to @mc_nums, which looks
like it was introduced as a pasteo in the original development.

Also updated the script magic.

Merge pull request #340 from dlbptfs/PT25834941_04_10

Pt25834941 04 10

[#25834941] Add Shelving Location to SIP2 Script

This patch uses the items.location value for the SIP2 sort bin field (CL).
This SIP2 field is only used in the response message (10) to a checkin.

[#25834983] Enhance SIP2 for ITG communication to pass 'Lost Item' status

Utilized the SIP2 CH (item properties) field to pass the descriptive
infomation represented by the items.itemlost value.

Micro update: 4.10.2

Quiesce warnings in C4::Reserves::_GetNextReserve

[#24502945] Running Suppressed item through checkin removes suppress status

Changed the assignment of default values to an item's withdrawn, suppressed,
or not for loan status.

[#26190279] Give Checkout history table unique identifier

Added a table identifier (and reworded one of the headers slightly)
to allow use of jquery statements to affect the table display.

Added table identifier of checkouthistt -- now this jquery will hide
(for example) the second column (patron name)

$("#checkouthistt").find("tbody td:nth-child(2),thead th:nth-child(2)").hide();

Also changed one column header from Library to Owning Library.  These
jquery statements will allow relabeling the library headers:

$("#checkouthistt th:contains('Issuing Library')").replaceWith("\<th\>Checked out at\<\/th\>");
$("#checkouthistt th:contains('Owning Library')").replaceWith("\<th\>Belongs To\<\/th\>");

[#25974505] Disable patron group checking in reserves handling

In situations where a bib had reserves on it by patrons
belonging to branches not in the current agent's branch group,
CheckReserves would trigger an exception by eventually calling
GetCircControlBranch with an undefined borrower category.

This is not a nice solution, but it will do until we can
rethink the permissions architecture for how patron hiding
can work.

[#21102187] Refine display of item-specific reserves

[#16738683] "Ignore, Keep here" button initiates a transfer

[#20608005/#25662245] Item-level holds not getting fulfilled

Yet another instance of what amounts to a "type" error
emerging from the arbitrary and inconsistent naming
of data members. In this case it was that a data member
named "borrowercategory" was being passed to GetIssuingRule(),
but the data passed into the calling function had that value
stored as its column name, "categorycode".

Haskell is looking better all the time.

[#19174343] Dirty cache handling mutates branch info

C4::Branch::GetBranchInfo() returns a format just slightly
different from GetBranches(), renaming "category" attr to
"categories". The problem was that it was doing this to
the cached master copy, so all future calls to GetBranches
would return the mutated copy. This patch just clones the
branch info.

[#24589597] Passed holdsqueue items retain former callnumber

[#25237193] Delete member exits non-zero

[#25060133] Update opac-downloadshelf to use Perl's "given" feature

[#25161833] (more) Clean up IP address parsing

There are some proxies which insert a non-IP value into
the HTTP_X_FORWARDED_FOR header (e.g. "unknown") to
protect privacy or who knows why. This would bunge up
Koha's IP detection. This patch looks for something that
looks like a good address, then if all else fails it
resorts to reporting that the client is not in any branch.

Fix up miscellaneous errors resulting from bad bib numbers

[#25161833] Clean up IP address parsing

If there's more than one proxy standing between the client
and Koha's web server, the HTTP_X_FORWARDED_FOR header may
contain more than one IP address. C4::Branch::GetBranchByIp
was not prepared for this condition, and it would cause an
exception.

Add error catching middleware

Don't close STDOUT

[#25088735] OPAC users can renew arbitrary items

[#12917055] Customize search relevancy ranking

Adds possibility to adjust relevancy ranking search by allowing
user to prioritize segments of the CCL query.  For example, a
user could specifya CCL snippet like:

    (rtype=a not format,regExpr-1=[a-z]) and language=eng

in the AdjustRelevancyRanking syspref.  This will generate a
search result which will prioritize book materials in English.
If blank entry is specified the syspref, Zebra will use the
regular relevancy ranking defined in Koha.

Micro update: 4.10.1

[#24590587] Unable to pass some items in holds queue

The main functional part of this piece is the area
around line 423 of C4::Reserves, where being called
from ModReservePass had the borrowers.categorycode
stored in a different value than when called from
GetReservesForQueue. What a mess.

[#24643365] Revert "Simplify OPAC login linkage"

This reverts commit 0e9cce03aea10d2975e6106aa15a56d3e407ccae.

[#24638745] Database handle not setting timezone

Set KOHA_CONF env var based on load-time config

Improperly formatted branch IP ranges can result in exceptions

[#24588637] Fully specify call to CheckReserves

Depending on module load ordering, C4::Circulation may not
know the full qualification for C4::Reserves functions.

[#14554159] reverting invasive schema changes

Minor update: 4.10.0

Micro update: 4.9.8

Merge branch 'ptfs-master', remote-tracking branch 'origin' into ptfs-master

Merge pull request #303 from dlbptfs/PT23682115_rs1

[#23682115] patrons are now unable to place multiple item level holds (r...

[#23682115] patrons are now unable to place multiple item level holds (restart)

Warning for title already held needed to be turned off in the event that
the item was of a type that allowed for multi-item title holds.

Merge pull request #299 from wagnerj/PT16567199

[#16567199] Remove Accession Date link from Item tab

Merge branch 'PT22533621-forbidden_holds' into ptfs-master

[#21179665] Fully specify function calls in members/bulkedit.pl

[#23968415] Found lost items sometimes remain on patron record

Found lost items when checked in, if requiring transfer (but
with no pending hold), would remain on the patron's record.
Cause: itemnumber not being passed in by HTML form.

[#22533621] Holds getting fulfilled from wrong branches

Item-level itemtypes were not being respected in the
selection of reserve pick candidates.

Micro update: 4.9.7

Merge branch 'PT23632905-blocked_multi_opac' into ptfs-master

Merge branch 'PT20543497-stays_lost' into ptfs-master

[#23030301, #23030217] (fixup) Cannot reserve reserved on-shelf items

If there was a copy of a title still on the shelf (having not
yet been picked for a pending reserve), a patron in the branch
could place a reserve on the title. This work turns that around
such that the patron is blocked with a message that the item is
available on the shelf.

[#23632905] Multiple holds unnecessarily barred

Patrons reserving items through the OPAC were having
all of their reserve requests blocked if just one of
them was already reserved by the patron. This fix
allows patrons to continue with the process for the
remaining requests.

Merge pull request #298 from wagnerj/PT14695393

[#14695393] Removes broken link to checkouts

Merge pull request #297 from wagnerj/PT10324403

[#10324403] Adds missing fields to patron update email

Merge pull request #294 from dlbptfs/PT23682115

[#23682115] patrons are now unable to place multiple item level holds

[#20543497] Lost items not being detached from accounts

[#14695393] Removes broken link to checkouts

When the list of checked out items was removed from the Checkouts
tab, the link saying "Patron has ITEMS OVERDUE. See highlighted items
below" was not.  The link is now invalid; this commit removes the "See
highlighted items below" text and link.

[#10324403] Adds missing fields to patron update email

When a patron submitted an address/personal details change
request from the OPAC, certain fields were not being included
(especially address and email).  This commit adds those fields
into the email message to staff.

[#16567199] Remove Accession Date link from Item tab

In the items tab of a record display, the Accession Date was linking
to the acquisitions order record.  If the site had not used the
acquisitions module, the link was invalid and produced a
DBD::mysql::st execute failed error message.  This patch removes the
link from the items tab.

[#23774347] Make all guided reports use replica_dbh

[#23682115] patrons are now unable to place multiple item level holds

Warning for title already held needed to be turned off in the event that
the item was of a type that allowed for multi-item title holds.

Merge pull request #293 from wagnerj/PT20674153

[#20674153] Get reading history item type from item

[#19664789] Print Hold notice generated even if msg pref off

If a patron record had no email address, and the messaging
preference for Hold Filled was not checked, a print hold notice
was being generated anyway.  This commit adds a check for the
messaging preference before generating a print hold notice.

Merge pull request #291 from wagnerj/PT20749431

[#20749431] fix 008 default for Koha authority records

[#18084285] Removes New Patron button from Circ Checkout

The button for adding a new patron under Circ, Checkout was using
patron category_type instead of categorycode to create a new patron
which meant that new patron record defaulted to the first available
Child patron category.  Removing the button from that screen -- it's
not needed and trying to fix the underlying code is difficult.

Merge pull request #289 from wagnerj/PT12205373

[#12205373} Fix for Chrome browser receipt printing

[#22911025] (fixup) Normalize formatting for transfer screen

[#23030301, #23030217] Unable to place on-shelf holds on some items

Patrons using the OPAC from within a branch could not place holds
on some damaged or in-transit/waiting items, even though they
should be able to. This refines the criteria demanded for a true
return from BranchHasACopy().

[#20674153] Get reading history item type from item

On the staff patron reading history tab, the item type was being
pulled from the MARC record 942c entry rather than the item record's
item type.  This was frequently inaccurate or non-existent, and was
confusing.  The fix changes the source for the display field to
the item record's item type.

[#20749431] fix 008 default for Koha authority records

The tag 008 of the stubby authority record was very limited and needed to be
manually linked to a biblio record after the initial creation. This meant if
a another biblio was created that could have used a previous created stubby,
(e.g., same author) it wouldn't -- it created a new stubby, This creates duplicate
authorities. In addition, running link_bibs_to_authorities.pl unlinked all
stubby records because the 008 was not set to link to anything.  This fix sets
proper default values for the 008 so that multiple records can be found and
linked to the same authority record.

[#12205373} Fix for Chrome browser receipt printing

Chrome and other Webkit-based browsers would not print circ
slips; the page would open and close but never load content.
This adds a wrapper function to prevent that.  See
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6126
for more background.

Micro update: 4.9.6

Merge pull request #288 from wagnerj/PT14843887-rev

[#14843887] staff advanced search stripping letter x from ISBN search

[#14843887] staff advanced search stripping letter x from ISBN search

Modifies earlier code in staff advsearch.tmpl

    q = q.replace(/[^0-9]/g,'');

that was done in PT 6321425 search ISBN dashes. The above replacement was
unduly restrictive and dropped all non-numeric characters; this made it
impossible to search for an ISBN containing an X and narrow in on that
specific record.  Fix leaves alpha and numeric characters alone but strips
hyphens, whitespace, and other characters.

Merge pull request #277 from dlbptfs/PT20545679

PT20545679: Can't remove check in note during check in process

Merge pull request #281 from hgq/21_03_installer_ISUPPRESS

LLK #15659465 adds I_SUPPRESS auth val and 952 subfields

Merge pull request #287 from ctfliblime/PT21907399-receipts_error

[#21907399] Fatal DBI error when printing checkin receipts

Merge pull request #286 from dlbptfs/PT17988921_rs

[#17988921] Adjust / improve a message in OPAC (restart)

[#21907399] Fatal DBI error when printing checkin receipts

A DB query was inlining query params and doing so in a way
that assumed all numeric barcodes. This patch replaces that
with the use of proper placeholder params.

[#17988921] Adjust / improve a message in OPAC (restart)

Template variable hold_already_exists needed to be set outside syspref logic.

Merge pull request #285 from hgq/21_05_pt16634349_callnumsplit

LLK #16634349 redefine callnum_split as varchar instead of boolean

Merge pull request #282 from hgq/21_04_pt15656343_installer_straySemicolon

LLK #15656343 convert semicolon to comma in marc21_framework_DEFAULT.sql

Merge pull request #279 from hgq/21_02_pt22183151_clearCheckinBackdate

LLK #22183151 checkin clear backdate upon 'Clear Date' also submits form

Merge pull request #278 from hgq/21_01_pt23351075_patronEditHolds

LLK #23351075 JS auto sort DESC on matched search in Edit Holds

LLK #16634349 redefine callnum_split as varchar instead of boolean

LLK #15656343 convert semicolon to comma in marc21_framework_DEFAULT.sql

LLK #15659465 adds I_SUPPRESS auth val and 952 subfields

LLK #22183151 checkin clear backdate upon 'Clear Date' also submits form
to clear CGI session, not just cookies

LLK #23351075 JS auto sort DESC on matched search in Edit Holds

[#16336565] Replace Net::CIDR::Compare with Net::IP

Net::CIDR::Compare has buggy output with complaints
about an invalid DESTROY method. Replacing it with
Net::IP to eliminate concern that this could cause
issues.

[#22911025] Update formatting for transfer screen

PT20545679: Can't remove check in note during check in process

When multiple items were being checked in at the same time, if one item in
the group had a check-in note, then this would trigger the pop-up note
potentially for other items as well.  This patch limits the pop-up note
to only the most recent item checked in.

Merge pull request #276 from ctfliblime/rose_dbh

Use Rose::DB for DBI connection management

Switch from DBIx::Connector to Rose::DB for DBI management

Two key pieces to this commit:

1) Wraps all HTTP request dispatches within a database
   transaction. If the Plack app dies for any reason, all
   database writes are rolled back. This is to help ward
   off inconsistent state.

2) Removes the recently added DBIx::Connector support and
   instead uses Rose::DB for all DBI management. The use of
   Dx::C combined with the above transaction locking was
   creating problems in areas that used both connections.
   This also consolidates all database access to go
   through Rose::DB.