Transport_type print missing

This type is quite important for all the print notices

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT3346 : overdue_notices

-csv was not working as advertised.
This feature is now working as it is forecast

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT3438 : faulty subscription-add branch selection

branch selection could lead to two branches selected
this patch removes the faulty line

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT3499 : cataloguing authid not reported

authid would not be correctly updated in bibliorecord.
Biblio record would not receive the correct authid in subfield 9
but some erroneous data

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT3448 : searchauthorities problems

Problems with searching authorities :
The same index was always used

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT3440: Display only borrower's messages in borrower's profile

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4428: fixes minor XHTML errors in shelves.tmpl

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 4409, I18N/L10N tab not highlighted

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Markup corrections to fix for Bug 4248

- Missing "=" in attribute
- TMPL_IF nested in HTML tag, which breaks translation script

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

bug 4256: fix patron replication when using LDAP with auth_by_bind

Patch compiled from diffs created by Dobrica Pavlinušić <dpavlin@rot13.org> to:

* enable patron replication when using LDAP with auth_by_bind
* not scribble over extended patron attributes
* fix failure logging in to OPAC if patron has no extended attributes

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

bug 1532: ensure that empty hold request expiration is represented as NULL

Fixes bug reported by Colin Campbell where reserves.expiratedate
was defaulting to '0000-00-00' instead of NULL.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

fix typo in DBrev - should be 3.01.00.129

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4334: Fixes double quotes error on ODUE line of sample_notices in all languages but French. The error was preventing sample_notices, and any foreign key dependent tables, from loading. This blocked enhancement message preference editing.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 3785 - Fixing a bug that was stopping holds if no maximum number was set

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bugfix 4123 - CanBookBeRenewed fixed.

In Circulation::CanBookBeRenewed categorycode and itemtype were not being checked for * wildcard
if a specific code was not given.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Adds missing hint for upload-type syspref

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix typo admin/in brnaches template.

The branch notes were not displayed because the variable was called
branchnotesl rather than branchnotes

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Enhancment [3.4] Bug 4288: Hide circular 'Log in to Your Account' link in opac-auth.pl

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fixes bug 4201: Holds priority listings assigning large numbers

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(bug #3798) fix changing request branch

a variable was wrong in template.

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

bug 4333 followup - fix an uninitialized value warning

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4333 Fix erroneous message re Active Currency

Script was incorrectly asking user to define an active currency
Now test for active currency in the standard way
Use the currency routines in Budgets where applicable
Move the code paths in the if else to subroutines to clarify flow
Enable warnings

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

BUG 3715 added state to label

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(bug #4295) fix overduerules edition

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

bug 3981 follow up change GST to 'Tax Rate' in EN-US

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

bug 4170 updated overdue rules

I added a note that this tool should not be used for
advanced notices. The fact that the Delay is required
is actually true based on the tests we did - which
is another bug.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Added "ordered" column in acqui-home budget display

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Improve detail page MARC21 XSL file for 505 and 520 tags

Some tags weren't displayed propertly:

505  If there is no indicator 1, content label is not displayed and
     then repeated 505 tags are also incorrecly displayed. This patch
     add a default label.
502  Wasn't displayed at all. Added in this patch.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT2588: minor changes in column order & display

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(bug #4287) add enrolment period update in 22 to 30 script

this fix the upgrade script, to upgrade the enrolment period, that is no more in years, but in month unity.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT2345 : Changes on contracts

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Hide basket actions if no basket

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

New contract button

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fixes bug 4326: Items marked as returned yesterday

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 3619 - C4::Letters::_send_message_by_email does not obey AutoEmailPrimaryAddress

This enhancement would cause C4::Letters::_send_message_by_email to obey the
syspref.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Improve OAI Server MARC21 and UNIMARC XSL

Dublin Core schema name space was repeated for each field
as attribute rather than at <oai_dc:dc> level. This reduces
significantly the size of OAI server responses.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix to ORDER BY SQL in lists feature

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Perl documentation fixup in C4/Items.pm.

The documentation of the DelItem function did not menton the $dbh.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fixing members order when string searched

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4285: Intranet bib display links to opac-search.pl, instead of catalogue/search.pl

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Search Patrons - Make patron search more forgiving.

Patron search has only been searching fields for names
that either match or begin with the given search terms.

This is too strict a search.
For example, take the name "Billie Jo Robinson", "Billie Jo"
being the firstname, and "Robinson" being the last name. If one
were to search for "Billie Jo", this patron would not appear in
the search results.

 To remedy this, the search
type has been changed from 'begins_with' to 'contain'.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Circulation - Disable Barcode Field If Item Needs Confirmation

When issuing items, if a confirmation screen comes up,
another item can still be scanned into the barcode field.

If a librarian does not notice the confirmation dialog,
he or she may continue scanning items without confirming
the issue and thus give a patron an item without it actually
being issued to the patron.

This patch disables the barcode field until the confirmation
question is answered.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT2621 : Fix simple barcodes display

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 4271, Offer choice of destination for save biblio operation

A follow-up to the fix for Bug 4121, the user should be able to control whether
the action of saving a bibliographic record takes them to the add item screen
or the detail screen, independently of whether they are adding or modifying the
record.

The addition of a "split button" would leave the default action in place if the
user clicked the "main" part of the button. Clicking on the menu would give the
user specific choices.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4257: Make advance notices actually go out by avoiding null selects

The wrong variable was used to select the number of days-until-due; this fixes it to use
the borrower's preference setting.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fixing Tests on SQLHelper

one test was failing for want of correct parameters
An other one was not correctly described

test on search with a simple string fails because was done with branchcode and start_with includes NULL branches
Removing branchcode from filtercolumns

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4371: Display MARC21 field 773 w/XSLT

Display MARC21 field 773 in the OPAC detail view when XSLT is enabled.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4370: Fix ind1 semantics for MARC21 780/785

When using XSLT stylesheets for display, show MARC21 fields 780 and 785 on the OPAC details page only when ind1=0, as specified by the standard.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT 2553 : Adds a Cancel button

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4404: Untranslated strings in staff search result page

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4248 Rework Cities dropdown for members entry

Garry Collum's patch correctly identified the core flaw in operation
looking closer revealed some other flaws (2 "blank" options, some vars
and documentation belonging to a superceded version, etc.
Changed GetCities to return a structure rather than the messy two returns
Let the template do the concatenation and removed the CGI::Popup

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Revert "Bug 4248: Fixes zip code in cities town pull down list."

This reverts commit 2eac2e7242aa2685df8f511c49bbf50cb55be4fd.

Bug 4248: Fixes zip code in cities town pull down list.

Adds zip code to the GetCities function.  This function is only used in memberentry.pl.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

update circ/fine rules help file

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4210 Acq suggestions don't have LibraryName value passed

Updated to current head.  The various acquisitions suggestion templates have a LibraryName field, but no value is passed for that field.  Added a parameter for that data.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 3782 Update installer info for z39.50 targets

New York Public Library and New York University Library have changed their z39.50 connection information; this patch updates the installer files to the current setups.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 3750 Modify XSLT to display subfield n of 111/711 and remove period after 110/710

If a 111/711 uses a subfield n (Number of part/section/meeting), XSLT wasn't displaying it which made for an incomplete display.

e.g., International Conference on Concurrency Theory 1999 : Eindhoven, Netherlands)
instead of 10th International Conference on Concurrency Theory 1999 : Eindhoven, Netherlands)

I've added it to show before subfield a if present.  I've also removed the period that XSLT puts in the results and display after the last occurrence of a 110/710.  It's fairly standard practice for these entries to have a period in the MARC record, so the XSLT display was showing a double period.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 3788 XSLT does not obey viewISBD syspref

Modified XSLT.pm and MARC21slim2OPACDetail.xsl to obey the viewISBD syspref
setting in the OPAC.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4372 - Fix for bookcount.pl not counting started transfers as seeing an item

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4377 Create default HOLD_PRINT notice file

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT2464 : branch_transfer_limit.pl usability change

Adds branchnames along with branchcodes for display

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

removing AutoCommit=0, the last item is not commited

For an unknown reason, the last item batch updated is NOT updated.
After investigating a LOT (more than 3 hours), it appears the problem comes from the AutoCommit=0
I tried to solve it, but failed. So I remove the AutoCommit=0 option.

this result in a loss of perfs, but it's not a big problem I think. OTH, now everything is updated !

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug Fixing batchmod.pl

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT 2553 : Adds background job for items batch modification

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT 2269, Follow-up : Granular permissions for items batch modification and deletion

Change of permissions names
Change of branch title in guided reports
Add a "Select All / Clean All" on items selection
Add a link to item when a given item cannot be deleted

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT2527 : Adds default empty value for branches when batch modifying items

(so they are not modified by mistake)

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT 2269 : Do not display all the items if there is a large amount of items

Also fix the barcode not found problem (due to empty lines)

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(MT #3007) update hold transfer slip datas

This patch add information to hold transfer slip:
 * itemcallnumber
 * location
 * waiting date

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4353 Add div tags to staff advanced search page

This patch adds div tags to every section of the page, to allow jquery to find them.

Divs added:  basiclimits, language, pubrange, subtype, availability, itemtypelist, currently-avail, select-libs, and sortby

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT3212 : members modification wipeout dateexpiry

Members modification when not in Library information mode would wipe out dateexpiry information

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(BUG #4357) Closing a basket and creating baskets group

when closing a basket without creating a basketgroup, the group is still created => corrected
+ replace "create a basketgroup?" by "create a purchase order now?" in confirmation step.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(BUG #4355) Basket.pl: withdrawal of the "can not close the basket" button

Basket.pl: withdrawal of the "can not close the basket"button. only Uncerain
prices button is displayed. And removing the "??" in the price columns,
replaced by "uncertain" in rrp column

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(BUG #4352) Adding a popup at removing of a basket

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(BUG #4351) The save button don't really save. Renaming in Next button

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(BUG #4350) parcel.tmpl: add Translatable text

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(BUG #4349) In parcel.pl: replaces All x items are displayed by x items are displayed

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(BUG #4348) parcel.pl: list with duplicates orders after a clearfilter

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(BUG #4345) Redirect after closing a basket and creation of a group

after closing a basket and creation of a group we should landed on the "Closed" tab.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(BUG #4344) Basket.pl - remove the line "contract number"

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(BUG #4342) should not be able to close an empty basket

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(BUG #4332) Changed the display of ordered search results

Changing the display of the basket column: changing "basketNumber" to "my
basket (basketNumber)".
Adding the Basket group column with fallowing display: "my group (group
Number)"

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

New Developer to history

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

catalyst-wr68480: better handling of holds re IndependantBranches

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

catalyst-wr68479: Limit holds to the user branch, provide a link to

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4307: Sorts library locations in inventory drop-down list.

Replacement patch for previous patch submitted.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix to enable Amazon cover images in the staff client.

Since this fix enables the previously-missing variable
'AmazonEnabled' in Auth.pm, this will also enable other
Amazon content in the staff client like Product
Descriptions and Amazon Similar Items (See Bug 3863)

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

fix minor typo in preference description

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

bug 4410 - change member to patron to be consistent

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

bug 4412 move staff xslt prefs to staff client tab

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

bug 4425: reorder search options on staff advanced search

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 4424, MARC Framework deletion incorrectly reports usage count

Thanks to Thomas Dukleth for the fix.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

highlighting bug (causes rtl text to move when clicking 'Unhighlight')

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Merge branch 'bug2505_patches' of git://git.catalyst.net.nz/koha into to-push

* 'bug2505_patches' of git://git.catalyst.net.nz/koha: (24 commits)
  Bug 2505 - use strict and warnings in sax_parser_test
  Bug 2505 - enable warnings for link_bibs_to_authorities
  Bug 2505 - add strict and warnings to perlmodule_ls
  Bug 2505 - add strict and warnings to check_sysprefs
  Bug 2505 - Add commented use warnings where missing in *.t
  Bug 2505 - Add commented use warnings where missing in *.pm
  Bug 2505 - Add commented use warnings where missing in the cataloguing/ directory
  Bug 2505 - Add commented use warnings where missing in the misc/ directory
  Bug 2505 - Add commented use warnings where missing in the tools/ directory
  Bug 2505 - Add commented use warnings where missing in the installer/ directory
  Bug 2505 - Add commented use warnings where missing in the rotating_collections/ directory
  Bug 2505 - Add commented use warnings where missing in the C4/ directory
  Bug 2505 - Add commented use warnings where missing in the serials/ directory
  Bug 2505 - Add commented use warnings where missing in the catalogue/ directory
  Bug 2505 - Add commented use warnings where missing in the sms/ directory
  Bug 2505 - Add commented use warnings where missing in the opac/ directory
  Bug 2505 - Add commented use warnings where missing in the virtualshelves/ directory
  Bug 2505 - Add commented use warnings where missing in the suggestion/ directory
  Bug 2505 - Add commented use warnings where missing in the admin/ directory
  Bug 2505 - Add commented use warnings where missing in the circ/ directory
  ...

Conflicts:
C4/Auth_with_cas.pm
acqui/supplier.pl

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 4302, shouldn't have to scroll left to right on z search results

Adds inline pop-up menu for each row of Z39.50 search results, similar to the
one recently added to the additem screen. This allows the user to click
anywhere in the row to display the menu and access the preview/import links.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

removing warn logs

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Followup : d31ac1 reintroducing submit buttons

this reintroduces translation of submit values

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

unconditional warning changed to if $DEBUG

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

follow-up : remove perl error if the librarian choose no budget

the librarian can enter the suggestion without selecting any budget.
Searching the budget name in this case resulted in a Perl error

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT3168 : Suggestion budget management

budget were not saved and not correctly displayed

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(MT3174) acqui/neworderempty.pl: incrementing the quantity when adding an item

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>