Bug 4133 Ensure that orders have a valid quantity received

quantityreceived drives other parts of acq noticeably budget balances
ensure that a number (usually 0) is set in NewOrder
give field a valid default and set as not null
reinterpret exsting nulls in table as zero

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

Fix for Bug 2869, 'do not delete' marc framework tag - reverts to 'default' framework

- "Do not delete" form was missing hidden framework code input.
- Also added details to breadcrumb navigation

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

Fix for Bug 4136, Invalid XHTML in spinelabel-print view

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

Markup error, closing tag missing

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

Bug 3697 Return due date (SIP Field AH) in item info

Although the due date of an on loan item was being retrieved
the appropriate field in the Item object was not set
causing the due date not to be returned in the item
information response

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

Cleanup date handling in routing.pl

Remove dead code
Clarify the loop assigning dates
Remove the cause of undef warnings in the log

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

Add dependency Authen::CAS::Client

Required CPAN module was missing from deoendency list

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

Fix count in supplier dropdown in serials/claims.pl

Previous patch still used old variable name to get count causing
incorrect value to be returned

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

Correct misleading text

Some koha acq terminology is confusing to new users so lets at least
be consistent in our usages we are adding to the basket here not an order

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

bug 2576 - fix XHTML validation errors generated by previous commit

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

Bug 2576 reopened -- turn off fines elsewhere in OPAC account

The original patch allowed use of the OPACFinesTab syspref to turn off
the Fines tab in the OPAC patron record.  However, a column for fines
still appears in the listing of overdues, and as a tab on the line with
Overdues, Holds, Fines, etc.  This patch turns off the other instances of
fine columns/displays if the syspref is off.

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

set koha_login_context in order to get auto-login for SCO working

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

DB rev 114 - tweak  AutoSelfCheckAllowed, AutoSelfCheckID, and AutoShelfCheckPass

* made AutoSelfCheckID and AUtoSelfCheckPass type free instead of textarea
* consolidated DB updates into one

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

Bug 3738 Automate login to web-based self-check system

Some small, single-branch corporate and special libraries use unattended self-checkout stations, and would like to automate the staff login, so that just going to the sco/sco-main.pl URL would bring the self-check up ready for patron use.  This patch create three sysprefs, AutoSelfCheckAllowed, and AutoSelfCheckID/AutoSelfCheckPass. If the site wants to allow automated login, staff would then need to create the selfcheck user record and enter that login ID and password into the sysprefs.  The kohaclone/opac/sco/sco-main.pl script has been modified to check these sysprefs and pass values (if present and allowed) into the self-check URL.  The URL then bypasses the staff login page and comes up ready for checkout, waiting for the first patron barcode.

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

Bug 3920 - Add 2 syspref to activate XSLT use in OPAC/Intranet

- Add two syspref:
  - OPACXSLTResultsDisplay
  - OPACXSLDetailsDispay
- Add them to .pref files and dispatch them beetween OPAC and Search
  tab.
- Upgrade DB version to 3.2 .113
- Fix something wrong in UNIMARCslim2intranetDetail.xsl
- Display OPAC result and detail pages using those syspref.

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

BZ4104: Removes tagging in staff client

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

Bug 3752: Fixed XHTML errors in opac-MARCdetail.tmpl

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

Bugs 3646 and 2651: Changes the output of compact.xsl from html to xhtml.

Changes the output of compact.xsl to XHTML for catalogue/showmarc.pl?viewas=card.

Adds <title> element in <head>.
Removes link to nonexistent css file (bug 2651).

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

bug 4129 fix display of html on the overdue report

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

Bug 3662: Invalid XHTML in update-child.tmpl - resubmission.

Missing </td> closing tag.

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

bug 3923 OrderPdfFormat default was wrong on update

I went back and updated updatedatabase so that anyone else who updates
doesn't end up with the wrong value, this does not fix those who
upgraded and have the wrong value in there already.

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

Removing noisy warn in C4/XSLT.pm

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

Bugfix: Correcting conditional for branch code check

Also removing needless conditionals on redirects.

Thanks to Braedon Vickers for catching the incorrect conditional.

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

Remove cause of some warnings in order display

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

Bug 4125 - Reformat with perldoc bulkmarcimport.pl doc

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

Fix for Bug 4113, Failed renewal from circulation screen results in malformed table

Renewal failure message should appear in renewal checkbox column just
like it does in moremember.tmpl

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

Bug 4114: Follow-up patch for duplicate id in opac-detail.

Adds the id back in <body>.  Renames the id in <div>.

The only reference to this id was in sanop.css, to which opac-detail.tmpl does not link.

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

Bug: 4124 - Submit button in csv-profile.tmpl floats to the right.

Typo - class='actin' should be class='action' in the <form> tag.

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

bugs 3989, 3990, 3996, 3997 - sys prefs resolved

bug 3989 - RoutingSerials & RenewSerialAddsSuggestion
moved to serials tab and descriptions updated

bug 3990 - add RoutingListAddReserves pref to serials tab

bug 3996 - add intranetbookbag pref to staff client tab

but 3997 - add ReservesControlBranch to circulation tab

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

bug 3993 & 4119 add WebBasedSelfCheck pref to interface

bug 4119 - add WebBasedSelfCheck to sys pref interface
bug 3993 - added quotes around OpacMaintenance description so it appears

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

bug 4033 add missing OverdueNoticeBcc to the interface

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

bug 3994 & 3995 add missing OPAC privacy preferences to interface

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

bug 4087 add missing OpacAddMastheadLibraryPulldown to prefs interface

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

bug 3986 add CAS preferences to interface

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

added subscription.enddate to kohastructure.sql

kohastructure.sql update missed in DB rev 3.01.00.071

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

Bug 4117 Fix system error in routing slip recepients popup

order_by parameter should be an arrayref of hashrefs
Have made order surname, firstname asc (see bug 4067)
do not generate logged warnings by manipulating undef

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

bug 3985 add missing acq preferences

This patch adds AcqCreateItem, CurrencyFormat, gist, and OrderPdfFormat back into the sys prefs interface.

Still missing are acquisitions, OrderPdfTemplate, and emailPurchaseSuggestions.

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

corrected POD for GetMember()

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

fixed remaining calls to GetMember() that used old argument style

New way, correct way to use GetMember is

GetMember(field => fieldvalue[, field2 => field2value ...]);

e.g.,

GetMember(borrowernumber => $borrowernumber);

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

removed unconditional warn

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

Bug 4108 Catch cases when GetMember is generating errors

Fixed two calls where new interface was not used
Arguably new syntax allows more options than we require but it was not catching
error cases (when 0 or undef is passed). It also can now be called in ways
which would validly return multiple hits but explicitly does not
(nor does calling code expect this)
This patch should quieten the flood of errors

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

finished add SpineLabelShowPrintOnBibDetails (DBrev 112)

* improve wording
* add to new system preferences editor

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

Bug 1248 - Add a "quick spine label" print to the bib item page in cataloging

Turn on the system preference SpineLabelShowPrintOnBibDetails and a link for the spinelabel printer will appear for each item on catalogue/detail.pl

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

added two new intranet directories to installation list

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

Bugfix: Fixing a broken link in the patron card layout editor

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

Bug 4114: Fixes duplicate id definition in opac-detail.tmpl

Removes "id=opac-detail" from <body> tag.

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

Bugfix: Adding Graphics::Magick module to About Koha page

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

Bugfix: Implimentation of JpegBlob and AltJpeg methods

Implimentation of JpegBlob and AltJpeg methods in C4::Creators::PDF

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

Bug 3691: Invalid XHTML in about.tmpl

Fixes several missing closing html tags.

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

Bugfix: Correcting hash dereferencing for barcode data

This patch corrects hash dereferencing syntax.

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

Style correction

- Use <div class="dialog alert"> for warnings, not "dialog error"

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

Bugfix: Correcting name of help file in Label Creator Tool

The Label Profile Editor help file was mis-named. This patch corrects
the name.

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

Bugfix: Correcting hash dereferencing for image data

This patch corrects hash dereferencing syntax.

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

Bugfix: Adding error trapping for missing or invalid branch code

This patch adds code to trap, log, and return an intelligent message to the user
when a missing or invalid branch code is passed in. This condition can occur if
the user is logged in as the kohaadmin or the user has multiple instances of the staff
client open and has switched users in some instance.

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

fix 4066 - add rotating collections to toolbar menu

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

Follow up Patch for moving the itemtype icons directory from sfdicons to vokal

Sponsered by vokal

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

Changed the folder location for Vokal icons from sdficons to vokal...

These icons where contributed by Vokal

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

Adding the vokal directory for icons

Contributed by VOKAL

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

More fixes for Bug 2600, HTML tags in titles not escaped in many places

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

Fixes for Bug 2600, HTML tags in titles not escaped in many places

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

added intranetbookbag syspref to install scripts

Had been omitted, so wasn't appearing in fresh
installs.

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

bug 4099: fix adding new patron category

Added categories.enrolmentperioddate to kohastructure.sql, which commit
dc9991 had omitted.

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

fix POD errors reported by xt/author/podcorrectness.t

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

Fix pod markup errors in C4/Budgets.pm

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

Fixes for Bug 3089, Date entry fields should indicate required date format

- Added include, date-format.inc, which shows format, e.g. "(DD/MM/YYYY)"
- Hint shows expected date format based on system preference
- Removing calendar.inc include from templates which don't use it

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

remove mysql5.0 errors in patroncard upgrade

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

Fix for Bug 4086, categorycode on patron search confusing

- Changed 'categorycode' to 'category'
- Formatted additional options to be more compact
- Returning patron search page to wide layout following
  removal of patron filter from the sidebar
- Corrected some other markup that was causing validation errors

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

Bugfix: Upping the byte size limit of patron images

This brings it more in line with the size limit placed on other images
uploaded for use on patron cards.

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

follow-up to bug 3854: resolve budget period language issues

top level: "Root Budget" becomes "Budget"
second level: "Budget" becomes "Fund"
third level and below: "Sub-Budget" is also a "Fund", but for things
like the "create sub-budget" link, they'll be changed to "create child
fund".

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

bug 4101 patroncards changed to patron card creator

There are references to patroncards in label text through out the patron card creator - I have changed it to 'patron card creator'

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

Bug 3808 Display correct number of records in neworderbiblio screen

New version of patch to apply against 3.2alpha
Also remove related FIXME message that was displaying to the user

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

Bugfix:[3.2] Corrects bad behavior in opac-reserve.pl

When AllowOnShelfHolds is enabled, but maxreserves is NULL, attempting to reserve
an item results in a basically blank page being returned with no error message.

This patch adds a check to see if the check for maxreserves has returned a value and
bypasses the application of it if it has not.

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

(bug #4075) add use of getrecordvalue in opac-detail

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

Bug 4085 - Subscription type length wasn't taken into account

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

remove duplicate lines

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

set DB revision to 3.01.00.111 for 3.2-lapha release

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

Revert "Populated MARC 952$2 field for serials-edit.pl"

This reverts commit 708b4797e3fc253b8dac2a0fa0d16e30839196ef.

Had introduced following error:

Global symbol "$value" requires explicit package name at C4/Biblio.pm line 2381.

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

Rotating Collections Fixes

This patch does the following:
  * Fixes the typos in updatedatabase.pl ( updates colBranchcode to varchar(10)
  * Adds the rotating_collections template files that somehow were not committed
  * Adds a missing sub from RotatingCollections.pm that must been deleted by accident
  * Adds the neccessary hooks in returns.pl to warn that and item needs to be returned
    to the branch that currently holds the collection.

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

Bug 4077 - Tweak UNIMARC default framework

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

Fix updatedatabase.pl mess coming from last merge?

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

Bugfix: Stop C4::Koha from filling the log with pesky warns

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

Bugfix: Stop opac/opac-detail.pl from filling the log with pesky warns

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

Bug 4081: Fixes incorrect html tag in opac-advsearch.tmpl.

Fixes <legend> that is closed with </label>

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

Bug 2505: Adds warnings to opac-ics.pl and opac-search-history.pl

No warnings generated from opac-ics.pl
Fixes warnings for opac-search-history.pl

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

Merge commit 'workbuffer.org-koha/translation' into to-push

Conflicts:
installer/data/mysql/updatedatabase.pl

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

4074 Bugfix: The 'Subject(s)' link(s) are malformed resulting in no search results

This bug was introduced by commit d51332698b2cce43542a58e53fd916df99329158
which removed the inability to do a search on anything contianing a colon (:).

This patch expands the regexp to only normalize true limit operators and leave
all other colons (:) intact.

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

Enhancement: This patch adds a 'cardnumber' parameter and checks

to see if it is present when the script is called. If it is, the patron
information is retrieved and borrowernumber set from that. Otherwise,
the patron information is retrieved by borrowernumber.

The changes in this patch do not require porting to any other Koha code.

Sponsored by www.digital-loom.com

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

Bug 4083 subroutine HasItems needs to be exported

Calls to the unexported subroutine result in system error

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

Fix some code issues in serials scripts

Add use warnings where not present
Remove unused variables
Avoid redeclarations of variables
trailing spaces trimmed

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

Bug 4088 Due Date Not formated in checkout response

due date was not being formatted as a timestamp
but being passed as epoch seconds

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

Bug 4095 Remove incorrect help text

Subscription end date is required before receiving against
the subscription. If not set the subscription needs
renewal before use.

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

Change style of return errors to be more noticable.

Changed the class of the div from message to error.

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

Fix for Bug 3970, Warn user when there is no active currency

- Adds warning to budget admin pages
- Adds warning to currency admin page

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

Markup and formatting corrections.

- XHTML validity
- Can't nest TMPL_IF inside <input>--breaks translator

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

Fix for Bug 4080, OPAC hold screen doesn't obey OPACDisplayRequestPriority pref

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

Adding missing images for collapsible syspref sections

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

Updating history docs

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

Bugfix: Adding Creators system preferences to new system preferences editor

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

Fix for Bug 4089, tools menu floating to bottom of CSV Profiles, and others

- Moving nested TMPL_IFs out of <option> tags (breaks translator)
- Fixing markup to match conventions for form structure
- Removing duplicate form fields (bad merge?)

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

Fix for Bug 4097, Problems with print stylesheet and add item screen

- consolidating some details in print.css
- moving overflow:scroll from additem.tmpl to staff-global.css
- overriding overflow:scroll in print.css

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