[MT2593] Fixed budget amount computation in acqui-home

followup (bug #4020) XSLT unimarc display

C4::Search removing unconditionnal warning

Fixed a major bug in ModReceiveOrder

Orders were not splited correctly.

[MT2593] Added "ordered" column in acqui-home budget display

[MT2590] Removed root budget duplicate checkings

(bug #4004) disallow access for non-logged in users in opac

This doesn't define borrowernumber = 0 if a borrower is NOT logged.
We know borrowernumber 0 is mysqluser... So in virtualshelves, a non logged user have all permissions.

[followup](bug #3667) don't delete if member is guarantor

this fix cleanborrowers to don't delete guanrantors.

[followup](bug #3667) fix dateexpiry filtering

this fix the dateexpiry used to filter users, now we use date specified.

[followup](bug #3667) cleaning borrowers fix

This fix GetBorrowersWhoHaveNotBorrowedSince to select only borrowers that have no issues and have no old_issues more recent than a specified date.

(bug #4020) XSLT unimarc display

When using XSLT Display, and UNIMARC,
since marcFlavour is not used in encoding data, when data is true utf8, as_xml
fails on some subfields.

Moreover, because transformMARCXMLForXSLT edits some values in the marc record
and the PERL UTF8 is not handled by MARC::File::USMARC, it endsup in double
encoding the data.
Sending a patch to fix both issues.

This patch adds
    - two functions in  C4/Charset.pm
        NormalizeString (uses Unicode::Normalize)
        SetUTF8Flag (This function in my opinion belongs to MARC::Record, or at least MARC::File::USMARC)
    - edits C4::XSLT in order to cope with the correct marcflavour
    - edits C4::Search searchResults to use setUTF8Flag

C4::SQLHelper Changing $$ notation to $ ->

Suggestions.pm :Changing $$ notation to $ ->

MT 2285 : enhancement for patronSearch

MT2345 : Changes on contracts

aqbudget display, minor changes

- removed 2 total that means nothing (and were troubling users)
- reindented a little (removed tab)

#4003 default value in PrepareItemrecordDisplay

The default value you can define in cataloguing framework was not used for item created from acq or serial recieve

Bug  :authid empty in Authorities Default summary

(bug #3745) fix deletion of borrowers rights detection

This patch fix the rights checking for borrower deletion.

(bug #3760) fix and improve some reports

This patch fix some problems with dates, and itemtypes level.
And it improve some sql queries.

Test Improvements : Adding Reserves.t

(bug #3667) fix borrowers cleanup

This add a check of borrower expiry date, and don't delete the borrower if he's still active.

bug Fixing: updatedatabase

Here comes a patch for update database.
Can you please test and tell me if it works ?

Thanks a lot

correcting spelling mistake: firstname is surname

MT2408 : UI Changes / CSV Export

Bug 3983 : Needed authorised values list for french frameworks.

[MT2346] Hide basket actions if no basket

Dates : improvements string recognition

Dates can now have one digit for month and day

[MT2346] Supplier display improvements

Used Koha generalized way to display lists of informations.

MT 2426 : Default currency is chosen for price calculation when bookseller's listprice is NULL

MT 1801 : Opac shelves and basket are displayed the same way

This patch also fixes broken location display in both shelves and basket.

followup : auto_truncation 3287252c0

truncated words bore a double * which would create an error

Conflicts solved :
C4/Search.pm

Bug Fix : 3689

Disabling Query_AutoTruncate for NoZebra searches

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

(bug #3584) detect ccl queries

This patch C4::Search::buildQuery to detect ccl queries, and let zebra to parse them.
And set a default index "kw" if not specified.
This improve the detection of ccl queries, and do not duplicate the "ccl=" value.
Adding = as an index sign
Followup to 3037ff9e81149615f94d3c4ff0e5b4c5240f3ce9

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

Translation scripts : Removing trailing / in scripts

Revert "C4/Search.pm followup auto_truncation 3.0.x cherry-pick"

This reverts commit 58406a5e4bb50e18c9620e91b90295d8219842b8.

Conflicts:

C4/Search.pm

Improved guided reports (#3929)

added runtime parameters for guided reports. Online help:

Can I have run-time defined parameters?

Yes, you can: there is a specific syntax that Koha will understand as 'ask for values when running the report'. The syntax is &lt;&lt;Question to ask|authorised_value&gt;&gt;.

The &lt;&lt; and &gt;&gt; are just delimiters. You must put &lt;&lt; at the beginning and &gt;&gt; at the end of your parameter
The Question to ask will be displayed on the left of the string to enter.
The authorised_value can be omitted if not applicable. If it contains an authorised value category, or <strong>branches</strong> or <strong>itemtype</strong> or <strong>categorycode</strong>, a list with the Koha authorised values will be displayed instead of a free field

Note that you can have more than one parameter in a given SQL
Note that entering nothing at run time won't probably work as you expect. It will be considered as "value empty" not as "ignore this parameter". For example entering nothing for : "title=&lt;&lt;Enter title&gt;&gt;" will display results with title='' (no title). If you want to have to have something not mandatory, use "title like &lt;&lt;Enter title&gt;&gt;" and enter a % at run time instead of nothing
Sample :
SELECT surname,firstname FROM borrowers WHERE branchcode=&lt;&lt;Enter patrons library|branches&gt;&gt; AND surname like &lt;&lt;Enter filter for patron surname (% if none)&gt;&gt;

MT 2495 : Fix multiple occurences of itemtype in guided reports criteria

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

Adding optgroup labels to translations

SearchInTable : problem with primary key searches

SearchInTable was filtering primary Key thus returning the whole table when asked a search on a primary key

MT2232 : members/notices.pl and members/messaging.pl are doing the same thing

[MT1234] Huge rewritte to use the API instead of hardcoded SQL in the script

Also fixes a bug in the template.

[MT1234] Code cleaning

[MT1234] Fixed aqcontract breadcrumb

[MT1234] Replaced useless selectbox in the title of contact form

[MT2345] New contract button

[MT2343] Removed HTML heresy from aqplan template

[MT2343] Moved currency up the table

[MT2343] Changed the red X for a green V in the root budget table

[MT2343] Changed the name of the Filter button, and lowered it

[2343] Fix currency not showing in aqbudgets

followup : auto_truncation 3287252c0

truncated words bore a double * which would create an error

C4/Search.pm followup auto_truncation 3.0.x cherry-pick

Auto_truncation is used even though exact search selected.
This patch removes this side effect

Conflicts solved:
C4/Search.pm
Cherry-picked from 3.0.x :
3287252c0

final fix for sysprefs merge, hopefully

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

handle error if no active currency is defined

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

fix silly typo

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

bugfix - fix compilation error

Reflect change of GetLocalCurrency to GetCurrency
in new_acq

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

Merge commit 'pianohacker-koha/prefs-submit' into master

Merge in Jesse Weaver's system preference editor changes.

Conflicts:
C4/Bookfund.pm
koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
koha-tmpl/intranet-tmpl/prog/en/includes/doc-head-close.inc
koha-tmpl/intranet-tmpl/prog/en/modules/intranet-main.tmpl

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

Merge commit 'colin_campbell/dbupgrade3.2' into master

Fix various issues with database updating.

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

Fix for Bug 4065, patron extended attributes error

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

Fixes for Bug 3943, Show add to cart link when lists off in the OPAC

- Also adds some missing conditionals for including javascript
- Also Removes an instance of TMPL_IF EXPR

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

Fix for Bug 4061, Improve OPAC maintenance screen

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

Extend Koha OAI Server to support other metadata formats

Currently Koha OAI server returns records in two formats: marcxml and
oai_dc (Dublin Core). This patch adds a new mode of operation where as
many as necessary metadata formats can be added via XSLT.

Documentation: See the end of oai.pl file to have an explanation of
how it works.

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

Bug 3921 - Add Intranet MARC21 XSL file based on OPAC one

Tweak also intranet CSS file in order to have the same layout than on
OPAC.

For the futur: it would be required to group all XSL in one location.
For now if someone improve OPAC XSL, he/she will have not to forget to
take its modifications back to the intranet XSL.

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

Merge commit 'biblibre/3.2_community' into to-push

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

Fix for Bug 4058, opac-auth.tmpl body id causes layout problems

Removing obsolete help files (Bug 4059)

Fixes for Bug 3958, Standardize vendor/supplier/bookseller terminology

Changing instances of "supplier" or "bookseller" to "vendor."

Fix for Bug 3984, Confusing note on transfers to receive

- Changing 'check this transfer' message to more informative
   'Transfer is XX days late'
 - Adding note in the case of late transfers: 'Transfers are
   considered late after XX [TransfersMaxDaysWarning] days.'

delay dropping of aqbookfund

aqbookfund cannot be dropped before aqorderbreakdown
due to fk cobstraints
remove drop of column bookfundid from aqbudget
column no longer exists at this point

Can only add column reservesallowed to issuingrules once

Remove an invalid db update

Update tries to drop two fk constraints which
don't appear to exist

column was added in two separate updates

3.01.00.068 duplicates the operations of 3.01.00.065
Because one column has already been created, populated and dropped
the rest of the update fails

remove duplicate column add

Change 3.01.00.089 duplicated 3.01.00.070

102 needs to be an update not an insert

remove duplicate addition of casAutherntication preference

Bug 4057 Fix spacing in XSLT OPAC display for 245 subfield f

If a 245 has a subfield f (or any of the displayed fgknps subfields after a, b,
or h) there is no spacing and the display runs together.  This happens in both
the XSLT results list and the XSLT individual title display.  I've added a
space between the a, b, h displays and the beginning of the fgknps set.

Bug 4056: Fixes invalid xhtml in opac-messaging.tmpl if SMS is disabled.

There is a </td> tag which is outside of the block of code which defines checks to see if SMS is enabled.  If SMS is disabled it causes several extra </td> tags.

Bug 3294 Use cn_sort field to sort inventory list output

The inventory tool was using itemcallnumber, title to sort; modified
Items.pm to use cn_sort, itemcallnumber, title so that call numbers
sorted on the padded sort field instead.

Improve behavior of manual history checkbox behavior.

Fix for Bug 3908, Edit subscription page shows manual history block by default

Changing template check for "history" var to check for "manualhistory."

Revised for biblibre/3.2_community: Fix for Bug 3939, Issuing rules clone button looks like a submit

This patch incorporates a drop-down for choosing the "clone to" library
directly from issuing-rules.pl. This eliminates the extra step of choosing
the library after submitting the "clone" action and clarifies the purpose
of the two different controls.

Merge branch 'patroncards-wip' of git://git.foundations.edu/koha into community

Conflicts solved:
Makefile.PL
installer/data/mysql/en/mandatory/sysprefs.sql
installer/data/mysql/fr-FR/1-Obligatoire/unimarc_standard_systemprefs.sql
installer/data/mysql/updatedatabase.pl
kohaversion.pl

Revised for biblibre/3.2_community: Fix for Bug 3780, various OPAC pages have no unique <body> id

OpacAddMastheadLibraryPulldown

In the current OPAC, there is no easy way to search a particular library.
By default, a user searches all libraries, or with SearcMyLibraryFirst turn on,
the user's home library. In dev_week, a library pulldown menu existed in the search masthead.

This commit adds the option to bring that pulldown menu back
by turning on the system preference OpacAddMastheadLibraryPulldown

Rotating Collection > DB upgrade to 102

Rotating Collections

This feature is designed to manage collections of items that move
from library to library periodically. Koha can already track who *has*
and item, and who *owns* and item, but not who *should have* an item.
That is the issue this feature addresses.

It allows a persion to create a collection, and add any number of items
to that collection. The collection can then be transferred from library
to library. If an item shows up at a library that does not currently
'hold' that collection, Koha will ask you to transfer it to the library
that does currently 'hold it. In that way, one can even transfer
collections where some of the items are currently checked out. As soon
as they make it back to a library, they will get transferred to the
current library holding that collection.

The feature consists of 4 main pages.
'Home' Page: The landing page, lists collections and provides access to
the rest of the tools.
  Access is via the Tools page.
Edit Collections: Add/Delete new rotating collections
Add/Remove Items: Add/Remove items from a given collection
Transfer Collection: Set the current 'holder' of a given collection.

Librarian access is controlled by 'CAN_user_tools_rotating_collections'

Revised for biblibre/3.2_community: Reformat of suggestions management interface (Bug 3844)

Changes and fixes include:

- Moving filters forms into left-hand sidebar
- Making filter forms collapsed by default (click to show)
- Converting "check all" to text link to match similar controls on other pages
- Correcting case of tab labels
- Fixing display of custom reason field
- Converting radio buttons to drop-down for more compact display
- Table-sorter column exclusions
- Markup corrections

Revised for biblibre/3.2_community: Putting CSV export sections into tabs

Fix for Bugs 3998, 3968. Includes other markup corrections and reformats

Bug 3998: Confusing warning on budget add page
Bug 3968: Empty Blue Box Pop Up

- Removed warning, made budget amount a required field.
- Added conditional logic to prevent display of tooltip when there is no content
- Moved filter form into sidebar
- Changed Budgets.pm to pass an ID to a form field
- Improvements to breadcrumbs and title tags
- Restructured budgets admin toolbar to match existing toolbar patterns:
  "new," "edit," etc.

Markup tweaks to better style the patron search filter

Adding Extended Attributes management to Auth_with_LDAP

Bugfix: Various Label Creator bugs

This fixes:

* A bug which caused the label template editor to throw
an error when saving when no previous profile was applied.

* A typo which caused a 'fetch without execute' error in Labels.pm

It also comments out several useless warns

Fixes for Bug 3376, Serials Display in OPAC

- Formatting links to different subscription views as tabs like the links to MARC view, ISBD, etc.
- Styling display of brief subscription view for better clarity
- Moving filter form on full subscription view to sidebar
- jQuery-fying some full subscription view scripts
- Adding branch name to output of full subscription information
- Styling year links for visibility and clarity (I'm not styling them as standard tabs because
  they wouldn't wrap well with many many years' worth of subscriptions)

Fix for Bug 3951, Menu a mess in WebKit browsers

- removing inline image in favor of CSS background (inline image
  is used in the OPAC to better work with cart button css
- adding "display:none" to "more" menu. "visibility:hidden" by
  itself creates unwanted whitespace in WebKit browsers
- adding explicit width to toplevelmenu to correct display
  problem in WebKit browsers

(bug #4052) fix mandatory field/subfield deletion

this fix the way to delete repeatable mandatory fields, the minus char is now shown only on repeatable fields, even if they are mandatory or not.
If the last field is mandatory it won't be deleted.

Fixing dbrev 104

There was two variable declaration missing

Bugfix: 4046 Misc SQL Syntax Errors

This patch fixes several SQL syntax errors with the ALTER TABLE pragma

Fix for Bug 4035: Today's checkouts grouped with previous checkouts

Creating separate variable to be output to the template so that
the original issuedate can be used in date comparison.

Add OverdueNoticeBcc syspref required by overdue_notices.pl

Fix for Bug 4024, Search history template problems, and other fixes.

- Markup corrections
- Removing TMPL EXPR
- Converting delete link to FORM ("destructive" actions shouldn't be links)
- Adding "search history" link to logged-in user's sidebar menu