koha.git
10 years agoMerge remote-tracking branch 'origin/new/bug_7408'
Paul Poulain [Tue, 20 Mar 2012 14:03:15 +0000 (15:03 +0100)]
Merge remote-tracking branch 'origin/new/bug_7408'

Conflicts:
installer/data/mysql/sysprefs.sql

10 years agoBug 7408 : Fixing a typo that was breaking the installer
Chris Cormack [Tue, 20 Mar 2012 01:19:31 +0000 (14:19 +1300)]
Bug 7408 : Fixing a typo that was breaking the installer

Signed-off-by: Kristina D.C. Hoeppner <kristina@catalyst.net.nz>
10 years agoMerge remote-tracking branch 'origin/new/bug_8250'
Paul Poulain [Tue, 20 Mar 2012 13:54:10 +0000 (14:54 +0100)]
Merge remote-tracking branch 'origin/new/bug_8250'

10 years agobug 7086 follow-up: removed warn
Paul Poulain [Tue, 20 Mar 2012 12:48:33 +0000 (13:48 +0100)]
bug 7086 follow-up: removed warn

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agobug 7086 follow-up: removed warn
Paul Poulain [Tue, 20 Mar 2012 12:48:33 +0000 (13:48 +0100)]
bug 7086 follow-up: removed warn

10 years agoBug 7086 : "suggested by" fixed for staff members
Adrien Saurat [Thu, 15 Mar 2012 13:59:24 +0000 (14:59 +0100)]
Bug 7086 : "suggested by" fixed for staff members

Now:
- suggested by is stored when a staff member makes a suggestion
- managed by is only updated when a suggestion is edited (not when
it is created)

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7434: Display of issue number for serials in overdues
Adrien Saurat [Mon, 12 Mar 2012 15:34:34 +0000 (16:34 +0100)]
Bug 7434: Display of issue number for serials in overdues

items.enumchron is now displayed when not null

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7719 - Change state of controls based on whether boxes are checked
Owen Leonard [Fri, 30 Dec 2011 20:27:01 +0000 (15:27 -0500)]
Bug 7719 - Change state of controls based on whether boxes are checked

Some OPAC interface controls cannot function unless a checkbox is
checked. Examples:

- Placing a hold on multiple items in the cart window or on a list view page
- Adding tags to multiple items in the cart window or on a list view page
- Removing one or more items from the cart window
- Removing one or more items from a list on the list view page
- Adding one or more items to a list from the cart window

This patch moves these controls into a separate div in which links
are enabled or disabled based on whether checkboxes are checked. It
does so on two pages: The lists view page (when viewing the contents
of a list) and in the cart window.

Links which are in their disabled state should not perform any action.
After checking one or more checkboxes the links should become active
and behave normally.

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Works in chrome and firefox (latest). Passes tests.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 6440: Implement OAI-PMH Sets
Julian Maurice [Mon, 23 Jan 2012 15:25:51 +0000 (16:25 +0100)]
Bug 6440: Implement OAI-PMH Sets

New sql tables:
  - oai_sets: contains the list of sets, described by a spec and a name
  - oai_sets_descriptions: contains a list of descriptions for each set
  - oai_sets_mappings: conditions on marc fields to match for biblio to be
    in a set
  - oai_sets_biblios: list of biblionumbers for each set

New admin page: allow to configure sets:
  - Creation, deletion, modification of spec, name and descriptions
  - Define mappings which will be used for building oai sets

Implements OAI Sets in opac/oai.pl:
  - ListSets, ListIdentifiers, ListRecords, GetRecord

New script misc/migration_tools/build_oai_sets.pl:
  - Retrieve marcxml from all biblios and test if they belong to defined
    sets. The oai_sets_biblios table is then updated accordingly

New system preference OAI-PMH:AutoUpdateSets. If on, update sets
automatically when a biblio is created or updated.

Use OPACBaseURL in oai_dc xslt

10 years agoBug 7696 - NORMARC lacks staff results XSLT
Magnus Enger [Thu, 15 Mar 2012 19:10:47 +0000 (20:10 +0100)]
Bug 7696 - NORMARC lacks staff results XSLT

This patch adds NORMARCslim2intranetResults.xsl, to enable XSLT display
of intreanet search results when marcflavour = NORMARC. Most of the
logic is copied verbatim from NORMARCslim2OPACResults.xsl.

To test:

Make sure you have these system preferences set:
marcflavour = NORMARC
XSLTResultsDisplay  = using XSLT stylesheet

NORMARC and MARC21 is similar enough that testing on MARC21 records should
be OK. (I don't expect anyone to do detailed testing of a lot of the different
positions in leader, 007, 008 etc...)

Before the patch: viewing results in the staff client should give an error
message about a missing XSLT file.

After the patch: Record details should be displayed in the search results,
similar to how it is done for MARC21.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7526 - longoverdue.pl leaves items marked as lost as still checked out to patron
Kyle M Hall [Fri, 10 Feb 2012 15:12:20 +0000 (10:12 -0500)]
Bug 7526 - longoverdue.pl leaves items marked as lost as still checked out to patron

When the longoverdu.pl script is run, and it marks an item as lost ( using
LostItem() ), if fails to remove the item from the borrower record. So, the
item is marked as lost, but is also still listed as checked out to the
borrower.

This commit adds the command line parameter --mark-returned. If used,
longoverdue.pl will remove lost items from the borrowers record.
Functionality will remain the same if it is not used.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
http://bugs.koha-community.org/show_bug.cgi?id=7426
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7724: Skip Linker test if Zebra isn't set up
Jared Camins-Esakov [Fri, 16 Mar 2012 11:38:24 +0000 (07:38 -0400)]
Bug 7724: Skip Linker test if Zebra isn't set up

One of the tests in t/db_dependent/Linker_FirstMatch.t ony works if Zebra is
running and authorities have been indexed. This was causing Jenkins agitas.
The test should be skipped if Zebra isn't running or authorities have not been
indexed at the time the test is run.

Signed-off-by: Ian Walls <koha.sekjal@gmail.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 5596 [REVISED 2] add author to holds to pull report
Owen Leonard [Wed, 14 Mar 2012 14:46:17 +0000 (10:46 -0400)]
Bug 5596 [REVISED 2] add author to holds to pull report

Adding output of author.

Signed-off-by: Kristina D.C. Hoeppner <kristina@catalyst.net.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7301: 'From' field is 1 year ago by default
Jonathan Druart [Wed, 22 Feb 2012 15:30:40 +0000 (16:30 +0100)]
Bug 7301: 'From' field is 1 year ago by default

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7400: Add auto-completion on auth_finder
Matthias Meusburger [Wed, 4 Jan 2012 14:11:03 +0000 (15:11 +0100)]
Bug 7400: Add auto-completion on auth_finder

While typing an authority, will automatically propose authorities (similar to
autocompletion for patron search if activated)

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Tested searching for authorities with and without autocomplete. Note that
this is most useful when used in the "Main entry" box instead of the
"Main entry ($a only)" box.
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Corrected tabs to spaces in auth-finder-search.inc while resolving merge
conflict.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7477: Followup: Fix perlcritic issues
Jonathan Druart [Thu, 8 Mar 2012 11:41:13 +0000 (12:41 +0100)]
Bug 7477: Followup: Fix perlcritic issues

Signed-off-by: Srdjan Jankovic <srdjan@catalyst.net.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
An existing holiday in a branch is not deleted when the branch is overwritten:
* branch B, set today as closed
* switch to branch A, where today is not closed
* copy holidays to branch B
* today is still closed for branch B

That's important to notice for the documentation

10 years agoBug 7477: Copy holidays from one branch to another
Srdjan Jankovic [Fri, 10 Feb 2012 00:09:56 +0000 (13:09 +1300)]
Bug 7477: Copy holidays from one branch to another

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
10 years agoBug 7668 - Improve navigation and toolbar options in guided reports
Owen Leonard [Wed, 7 Mar 2012 19:06:45 +0000 (14:06 -0500)]
Bug 7668 - Improve navigation and toolbar options in guided reports

Creating new include, reports-toolbar.inc for presenting "action"
options to the user, in contrast with "view" options in the left-
hand navigation menu.

In the toolbar: New (guided report, SQL report), Edit, Run.

The presence of the toolbar allows the user to access functions
more flexibly: Getting directly to 'edit' or 'run' from the 'view'
page for instance.

Modifications to guided_reports.pl pass report id and name to
the template for clarity and for the purpose of enabling the
edit/run buttons.

To test: Apply the patch and go through the process of creating
a new saved SQL report. Note that the toolbar is present and
the buttons are functional at appropriate times.

New and Edit options should only be displayed if the user has
permission to create reports. Test with a user who does not
have create permission to confirm.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agobug_7140: Added item description to complement icon to search result and biblio detai...
Srdjan Jankovic [Fri, 27 Jan 2012 03:59:27 +0000 (16:59 +1300)]
bug_7140: Added item description to complement icon to search result and biblio detail pages

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
This seems like a very big improvement, especially for people using screen
readers. I agree that the change to C4::Search is required.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoC4::Members follow-up
Paul Poulain [Thu, 9 Feb 2012 16:42:53 +0000 (17:42 +0100)]
C4::Members follow-up

Overdues already loaded, must set full sub patch

http://bugs.koha-community.org/show_bug.cgi?id=6875
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Fixes issue with circ/branchoverdues.pl
Passes tests.

10 years agoBug 6296 follow-up: DBrev
Paul Poulain [Mon, 19 Mar 2012 16:37:31 +0000 (17:37 +0100)]
Bug 6296 follow-up: DBrev

10 years agoBug 6296: allow users to be authenticated by SSL client certs
Robin Sheat [Thu, 9 Jun 2011 03:11:23 +0000 (15:11 +1200)]
Bug 6296: allow users to be authenticated by SSL client certs

This adds a new syspref: AllowPKIAuth. It can have one of three states:
* None
* Common Name
* emailAddress

If a) this is set to something that's not "None", and b) the webserver
is passing SSL client cert details on to Koha, then the relevant field
in the user's certificate will be matched up against the field in the
database and they will be automatically logged in. This is used as a
secure form of single sign-on in some organisations.

The "Common Name" field is matched up against the userid, while
"emailAddress" is matched against the primary email.

This is an example of what might go in the Apache configuration for the
virtual host:

    #SSLVerifyClient require # only allow PKI authentication
    SSLVerifyClient optional
    SSLVerifyDepth 2
    SSLCACertificateFile /etc/apache2/ssl/test/ca.crt
    SSLOptions +StdEnvVars

The last line ensures that the required details are
passed to Koha.

To test the PKI authentication, use the following curl command:
    curl -k --cert client.crt --key client.key  https://URL/
(look through the output to find the "Welcome," line to indicate that a user
has been authenticated or the "Log in to Your Account" to indicate that a
user has not been authenticated)

To create the certificates needed for the above command, the following series
of commands will work:
    # Create the CA Key and Certificate for signing Client Certs
    openssl genrsa -des3 -out ca.key 4096
    openssl req -new -x509 -days 365 -key ca.key -out ca.crt
    # This is the ca.crt file that the Apache config needs to know about,
    # so put the file at /etc/apache2/ssl/test/ca.crt

    # Create the Server Key, CSR, and Certificate
    openssl genrsa -des3 -out server.key 1024
    openssl req -new -key server.key -out server.csr

    # We're self signing our own server cert here.  This is a no-no in
    # production.
    openssl x509 -req -days 365 -in server.csr -CA ca.crt -CAkey ca.key \
        -set_serial 01 -out server.crt

    # Create the Client Key and CSR
    openssl genrsa -des3 -out client.key 1024
    openssl req -new -key client.key -out client.csr

    # Sign the client certificate with our CA cert. Unlike signing our own
    # server cert, this is what we want to do.
    openssl x509 -req -days 365 -in client.csr -CA ca.crt -CAkey ca.key \
        -set_serial 02 -out client.crt
    openssl pkcs12 -export -in client.crt -inkey client.key -out client.p12
    # In theory we can install this client.p12 file in Firefox or Chrome, but
    # the exact steps for doing so are unclear, and outside the scope of this
    # patch

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Tested with Common Name and E-mail authentication, as well as with PKI
authentication disabled. Regular logins continue to work in all cases when
SSL authentication is set to optional on the server.

Signed-off-by: Ian Walls <koha.sekjal@gmail.com>
QA comment: synchronized updatedatabase.pl version of syspref with sysprefs.sql
version, to avoid divergent databases between new and upgrading users.

10 years agoMerge remote-tracking branch 'origin/new/bug_7408'
Paul Poulain [Mon, 19 Mar 2012 15:55:25 +0000 (16:55 +0100)]
Merge remote-tracking branch 'origin/new/bug_7408'

10 years agoBug 7408 Fix ExpireReservesMaxPickUpDelayCharge syspref definition
Frédéric Demians [Fri, 16 Mar 2012 18:52:17 +0000 (19:52 +0100)]
Bug 7408 Fix ExpireReservesMaxPickUpDelayCharge syspref definition

Allow preferences translation

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
This patch fixes another sysprefs problem -
Syck parser (line 300, column -1): syntax error at /usr/lib/perl5/YAML/Syck.pm line 76, <$fh> line 1.
when clicking local use. I could not verify that it fixed translations, but fixing the local use prefs is important too, so I'm signing off.

10 years agoBug 4976 - Status of item returned with process_koc.pl is empty in Intranet
Kyle M Hall [Fri, 9 Mar 2012 21:34:47 +0000 (16:34 -0500)]
Bug 4976 - Status of item returned with process_koc.pl is empty in Intranet

Fix to ensure items returned through a processed koc file have
items.onloan and items.datelastseen set corrrectly.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Uploaded a file with checked in and checked out items. all info
shows properly.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 5698: Followup: Add date picker option to SQL Runtime Parameters
Jonathan Druart [Fri, 16 Mar 2012 10:36:46 +0000 (11:36 +0100)]
Bug 5698: Followup: Add date picker option to SQL Runtime Parameters

Move html

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 5698: Add date picker option to SQL Runtime Parameters
Ian Walls [Sat, 10 Mar 2012 14:03:20 +0000 (09:03 -0500)]
Bug 5698: Add date picker option to SQL Runtime Parameters

Adds a date picker for SQL reports with the 'date' authorised_value.  Updates help
documentation for this new feature, as well.

To test:
1.  Create a new report with two date parameters, using the syntax:
<<Label for this selection of date|date>>
2.  Run the report
3.  Select dates
4.  The SQL generated should format the dates properly in ISO, and keep track of which
date is which

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
10 years agoBug 7670 - make required fields red
Owen Leonard [Thu, 8 Mar 2012 13:53:45 +0000 (08:53 -0500)]
Bug 7670 - make required fields red

Adding "required" style and "Required" note to additem fields,
consistent with the way required fields are displayed
on the add patron form.

This patch also corrects an error in the client-side validation
script which misidentified which fields were required. Leaving
a required field empty should result in an alert and the missing
field being highlighted in yellow.

To test, open the add item form on a record which uses
a framework in which one or more item fields are required.
Required fields should be labeled in red and appended with a
"Required" note.

Submit the form while some required fields are empty. You should
see a javascript alert. The empty mandatory fields should have
a yellow background.

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Works - passes tests.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7656 - "undefined" pop-up message when putting hold on reference item
Kyle M Hall [Tue, 6 Mar 2012 19:08:04 +0000 (14:08 -0500)]
Bug 7656 - "undefined" pop-up message when putting hold on reference item

This is a single line fix. Initializing the msg var to an empty
string solves the problem.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agowelcome David Cook, you're 178th
Paul Poulain [Fri, 16 Mar 2012 10:54:13 +0000 (11:54 +0100)]
welcome David Cook, you're 178th

10 years agoBug 7618 Escape HTML special characters for SQL displayed above results in Report...
David [Mon, 12 Mar 2012 23:17:32 +0000 (10:17 +1100)]
Bug 7618 Escape HTML special characters for SQL displayed above results in Report module

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoMerge remote-tracking branch 'origin/new/bug_7368'
Paul Poulain [Fri, 16 Mar 2012 10:50:42 +0000 (11:50 +0100)]
Merge remote-tracking branch 'origin/new/bug_7368'

10 years ago7368 Typo in cart_to_shelf
Marcel de Rooy [Mon, 12 Mar 2012 11:33:07 +0000 (12:33 +0100)]
7368 Typo in cart_to_shelf

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7644 - Invalid markup in staff client language chooser
Owen Leonard [Mon, 5 Mar 2012 19:47:51 +0000 (14:47 -0500)]
Bug 7644 - Invalid markup in staff client language chooser

This patch removes invalid markup from the language chooser
in the intranet. It also copies the markup and style of
the staff client include to the OPAC so the two are
more consistent. I hope that will make future changes
and debugging a little easier.

I believe this patch also fixes Bug 7366, "Language chooser
display problems."

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Patch fixes bug 7366, but there is still a difference between staff
and OPAC.
Installed languages: en, en-NZ, de-DE, fr-FR
Activated langauges: en, en-NZ, fr-FR
OPC shows English, staff shows en-NZ as label for the list.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7557 follow-up: DBRev number and removed default value
Paul Poulain [Wed, 14 Mar 2012 15:38:07 +0000 (16:38 +0100)]
Bug 7557 follow-up: DBRev number and removed default value

See http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7557#c17,
this follow-up also remove the default value for the syspref

10 years agoBug 7557: Change hardcoded routing list note into a system preference
Katrin Fischer [Sun, 19 Feb 2012 11:31:14 +0000 (12:31 +0100)]
Bug 7557: Change hardcoded routing list note into a system preference

Adds a new system preference RoutingListNote under the Serials tab.

The note will display above the note from the subscription and replace
the current hardcoded note:

"Notes: Please return this item promptly as others are waiting for it."

The patch adds unique ids to all notes and the note in general, so it
can be styled using CSS.

Also corrects the routing slip template to follow the HTML4 rule.

Update 2012-03-12: Fixed problem in updatedatebase.

10 years agoMerge remote-tracking branch 'origin/new/bug_6875'
Paul Poulain [Wed, 14 Mar 2012 15:16:21 +0000 (16:16 +0100)]
Merge remote-tracking branch 'origin/new/bug_6875'

10 years agoBug 6875 Followup: De-nesting C4::Items breaks bulkmarcimport
Ian Walls [Sat, 10 Mar 2012 13:02:26 +0000 (08:02 -0500)]
Bug 6875 Followup: De-nesting C4::Items breaks bulkmarcimport

Removing 'use C4::Branch' in favour of a subroutine-specific 'require C4::Branch'
causes the GetBranchName subroutine to return an error, as it's not defined
in C4::Items.  Adding "C4::Branch::" scoping fixes the error, which is what's done here.

To confirm problem:
1.  Attempt to run bulkmarcimport.pl before applying the patch.  You should get
    ERROR: Adding items to bib 435 failed: Undefined subroutine &C4::Items::GetBranchName
    called at /home/sekjal/kohaclone/C4/Items.pm line 656, <GEN13> line 435.

To test:
2.  apply patch
3.  run bulkmarcimport again.  Error should disappear.

Signed-off-by: Ian Walls <koha.sekjal@gmail.com>
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Note that this problem only appears when importing records with item (952)
fields.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 5749 Fix borrower address display in intranet
Stéphane Delaune [Tue, 6 Mar 2012 15:20:01 +0000 (16:20 +0100)]
Bug 5749 Fix borrower address display in intranet

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
10 years agoMerge remote-tracking branch 'origin/new/bug_7661'
Paul Poulain [Wed, 14 Mar 2012 14:00:50 +0000 (15:00 +0100)]
Merge remote-tracking branch 'origin/new/bug_7661'

10 years ago7661 Followup for resolving moved Record test
Marcel de Rooy [Wed, 14 Mar 2012 12:39:53 +0000 (13:39 +0100)]
7661 Followup for resolving moved Record test

10 years agoMerge remote-tracking branch 'origin/new/bug_7031'
Paul Poulain [Wed, 14 Mar 2012 13:47:34 +0000 (14:47 +0100)]
Merge remote-tracking branch 'origin/new/bug_7031'

10 years agoBug 7031 follow-up typo fix & tab removed
Paul Poulain [Wed, 14 Mar 2012 13:47:21 +0000 (14:47 +0100)]
Bug 7031 follow-up typo fix & tab removed

* s/adsearch/advsearch/ as it was a typo
* replaced tab by 4 spaces

10 years ago7031 Followup: typo
Marcel de Rooy [Fri, 9 Mar 2012 16:49:29 +0000 (17:49 +0100)]
7031 Followup: typo

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
10 years agoBug 7031 Follow-up, More options for AdvancedSearchTypes
Owen Leonard [Tue, 27 Dec 2011 16:18:34 +0000 (11:18 -0500)]
Bug 7031 Follow-up, More options for AdvancedSearchTypes

This follow-up patch corrects some markup errors and removes some
markup which I think is superfluous:

- Using template FILTER command to remove invalid punctuation from
  ID attributes (to quiet the HTML validator)
- Removing <fieldset> since options are bordered by the tab box now
- Correcting the conditional which controls the table markup
  to prevent extra/missing </tr>
- Adding missing ID attribute in the OPAC template

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
10 years agoBug 7031 Follow-up: Staff search displays OPAC description instead of the normal...
Kyle M Hall [Fri, 24 Feb 2012 14:19:08 +0000 (09:19 -0500)]
Bug 7031 Follow-up: Staff search displays OPAC description instead of the normal description for the authorized values

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
10 years agoEnh 7031: More options for Advanced Search
Ian Walls [Wed, 27 Jul 2011 18:08:23 +0000 (14:08 -0400)]
Enh 7031: More options for Advanced Search

Adds the ability to perform advanced searches in both the OPAC and staff client on more than
a single AdvancedSearchType at a time.  Support included for Itemtype, Collection Code and Shelving Location.

AdvancedSearchTypes syspref preference is repurposed; no longer a single value, it can now take
multiple item code fields separated by "|".  The order of these fields will determine the order
of the tabs in the OPAC and staff client advanced search screens.  Values within the search type
are OR'ed together, while each different search type is AND'ed together in the query limits.  The
current stored values are supported without any required modification.

Each set of advanced search fields are displayed in tabs in both the OPAC and staff client.  The
first value in the AdvancedSearchTypes syspref is the selected tab; if no values are present, "itemtypes"
is used.  For non-itemtype values, the value in AdvancedSearchTypes must match the Authorised Value name, and
must be indexed with 'mc-' prefixing that name.

<li> elements in tab are assigned unique IDs, so the text of the tab can be altered to match the
library's needs (using JQuery)

The logic to handle the 5 element row limit has been moved from the Perl to the templates, since Template::Toolkit
has a simple method for extracting the count of an element in a loop and performing 'modulus' on it.

2011-12-21: Incorporated changes recommend by Owen Leonard on bug report.
Signed-off-by: Owen Leonard <oleonard@myacpl.org>
10 years agoBug 7699: Restricted until datepicker broken
Jared Camins-Esakov [Sun, 11 Mar 2012 17:19:00 +0000 (13:19 -0400)]
Bug 7699: Restricted until datepicker broken

To test:
1. Edit a patron, and try to use the datepicker to set the date the patron is
   restricted until.
2. Note that the datepicker does not come up.
3. Apply patch.
4. Repeat step 1.
5. Note that the datepicker does come up.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agobug 7261 follow-up DBRev number
Paul Poulain [Wed, 14 Mar 2012 13:16:01 +0000 (14:16 +0100)]
bug 7261 follow-up DBRev number

10 years ago7261 Followup to make tests consistent
Marcel de Rooy [Mon, 12 Mar 2012 14:23:08 +0000 (15:23 +0100)]
7261 Followup to make tests consistent

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
10 years agoBug 7261 System Preference to select the first day of week
Juan Romay Sieira [Thu, 9 Feb 2012 18:12:31 +0000 (19:12 +0100)]
Bug 7261 System Preference to select the first day of week

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
10 years agoBug 7609: Improving links to find analytics and volumes when using UseControlnumber...
Katrin Fischer [Sun, 26 Feb 2012 21:25:41 +0000 (22:25 +0100)]
Bug 7609: Improving links to find analytics and volumes when using UseControlnumber (MARC21)

To test:
- switch on UseControlnumber
- add a serial record with 001
  - ldr pos 19 = a or
  - 008 (continuing resource), pos 21 = m
- add 2 analytics with with $w subfields containing the 001 of the serial
  - one with ldr pos 7 = a
  - one with ldr pos 7 = b
- add 1-2 monographical records linked to the serial
  - 8xx $w subfield containing the 001 of the serial
  - 490

The serial should show 2 links (in both OPAC and Intranet):
'Show volumes' should only bring up the monographical records.
'Show analytics' should only find the analytical records.

Without the patch both links would find all linked records, no difference
between both links.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoMerge remote-tracking branch 'origin/new/bug_7661'
Paul Poulain [Tue, 13 Mar 2012 17:14:26 +0000 (18:14 +0100)]
Merge remote-tracking branch 'origin/new/bug_7661'

10 years agobug 7661: Moving C4::tests to unit tests
Paul Poulain [Tue, 13 Mar 2012 17:07:54 +0000 (18:07 +0100)]
bug 7661: Moving C4::tests to unit tests

10 years agoBug 5341: Moves the "save" button to the top of the serial receiving
Jonathan Druart [Thu, 8 Mar 2012 09:09:23 +0000 (10:09 +0100)]
Bug 5341: Moves the "save" button to the top of the serial receiving

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 5341: Moves the "save" button to the top of the serial receiving
Jonathan Druart [Thu, 8 Mar 2012 09:09:23 +0000 (10:09 +0100)]
Bug 5341: Moves the "save" button to the top of the serial receiving

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 3216: UNIMARC author facets
Frédéric Demians [Fri, 2 Dec 2011 06:56:27 +0000 (07:56 +0100)]
Bug 3216: UNIMARC author facets

Add 700$b to UNIMARC author facets.

Other facets subfields could be added now. For example, other subjects
subfields.

Following patches are required to handle better MARC21 subfields and choose
other subfields to deal with UNIMARC format.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Tested under both MARC21 and UNIMARC. Does not cause any regressions with
MARC21, and offers the possibility for better faceting there in the future.
Works as advertised with UNIMARC.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7698: Add CHR/ICU Zebra tokenization choice to installation
Frédéric Demians [Mon, 12 Dec 2011 08:39:34 +0000 (09:39 +0100)]
Bug 7698: Add CHR/ICU Zebra tokenization choice to installation

Word search with multi-part facets works properly only with Zebra ICU
tokenization. This patch add a new question to Koha command line
installer:

  Zebra has two methods to perform records tokenization
  and characters normalization: CHR and ICU. ICU is
  recommended for catalogs containing non-Latin
  characters. (chr, icu) [chr]

How to test:

  - perl ./Makefile.PL
  - Try each possible value for new parameter
  - Take a look at zebradb/etc/default.idx file.
    Depending of the parameter you get this line:
      icuchain words-icu.xml
    or this one:
      charmap word-phrase-utf.chr

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
(Note: This patch was previously associated with bug 3216; I moved it to a
separate bug because including ICU is a good idea independent of the fix for
the particular issue described in bug 3216)

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7602: Followup: Spent and ordered pages require full acquisition permissions
Jonathan Druart [Fri, 2 Mar 2012 10:50:34 +0000 (11:50 +0100)]
Bug 7602: Followup: Spent and ordered pages require full acquisition permissions

hide link to neworderempty if user does not have order_manage permission

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7602: Spent and ordered pages require full acquisition permissions
Katrin Fischer [Sun, 26 Feb 2012 06:11:59 +0000 (07:11 +0100)]
Bug 7602: Spent and ordered pages require full acquisition permissions

Changes permissions on these pages so that one acquisition permission
will make it possible to access the pages. Currently pages are only
accessible for users with full (all checkboxes checked) acquisition
permissions.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Tested with only one acquisitions permission (various choices) and
patron can access the ordered and spent pages.

10 years agoBug 7587 follow-up: DBRev number + message changed a little bit
Paul Poulain [Tue, 13 Mar 2012 14:43:24 +0000 (15:43 +0100)]
Bug 7587 follow-up: DBRev number + message changed a little bit

10 years agoBug 7587: Fix table name for biblioimages table
Jared Camins-Esakov [Fri, 9 Mar 2012 12:24:43 +0000 (07:24 -0500)]
Bug 7587: Fix table name for biblioimages table

The kohastructure.sql file had the wrong name for the biblioimages table,
which broke the local cover images feature for new installations, and filled
the log with error messages.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
1) Checked a broken installation with wrong table name.
2) Checked for a new installation
Feature works perfectly.

http://bugs.koha-community.org/show_bug.cgi?id=7487
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7633 - Move display of patron image in self checkout
Owen Leonard [Thu, 1 Mar 2012 15:30:02 +0000 (10:30 -0500)]
Bug 7633 - Move display of patron image in self checkout

This patch moves the display of the patron image in self-checkout
from above the checkout form into a column to the right of the
form.

The patch also corrects an error in the display of the page title
which was preventing correct display of the library name. To test
this change confirm that the page title shows the value of your
LibraryName system preference or "Koha" if the preference is empty.

http://bugs.koha-community.org/show_bug.cgi?id=7631
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Confirmed working as expected. Passes template tests.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 4969: deletion of suppliers
Adrien Saurat [Fri, 2 Mar 2012 13:23:17 +0000 (14:23 +0100)]
Bug 4969: deletion of suppliers

If a vendor/bookseller has no basket/subscription link, it can now
be deleted (after a JS confirmation).

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Verified that I can only delete vendors without linked orders or
subscriptions.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7665 follow-up s/tabs/4 spaces/
Paul Poulain [Tue, 13 Mar 2012 14:11:16 +0000 (15:11 +0100)]
Bug 7665 follow-up s/tabs/4 spaces/

This patch had tabs. I took care of them, and also too care of all tabs in the template

10 years agoBug 7665 - Bibs with no ISBN's show broken images for covers when using Syndetics...
Liz Rea [Wed, 7 Mar 2012 19:29:58 +0000 (13:29 -0600)]
Bug 7665 - Bibs with no ISBN's show broken images for covers when using Syndetics cover images

Patch fixes problem for syndetics + bibs with no ISBN. Also adds "no image found" capability to Syndetics results.

To test (first, contact me and I will let you use my syndetics credentials *for testing this bug only*):
* replicate bug - have a bib with an ISBN, and one without. The one with the ISBN should show a cover, the one without will show a broken image.
* apply patch
* search for the same bib - you should now see "no image available" for the one with no ISBN, and a cover image for your bib with an ISBN.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Ian Walls <koha.sekjal@gmail.com>
QA Comment:  normalized spacing

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7636 - error when trying to email cart when opacuserlogin set to don't allow
Owen Leonard [Thu, 1 Mar 2012 20:06:25 +0000 (15:06 -0500)]
Bug 7636 - error when trying to email cart when opacuserlogin set to don't allow

This patch hides the "send cart" link if the opacuserlogin
preference is set to disallow. Other similar changes included:

- hide the "send list" link
- hide the "your lists" tab
- hide the "log in to create your own lists" link
- hide the "recent comments" link

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Tested with opacuserlogin set to both "Allow" and "Don't allow." With
opacuserlogin set to "Allow," the links showed up, and with opacuserlogin
set to "Don't allow," the links did not.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 6751: Adds a link "Export checkin barcodes" in readingrec
Jonathan Druart [Fri, 10 Feb 2012 10:18:48 +0000 (11:18 +0100)]
Bug 6751: Adds a link "Export checkin barcodes" in readingrec

to export a file containing a list of checkin barcode for the current day.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7645 - System preferences editor save button obscured by language chooser
Owen Leonard [Mon, 5 Mar 2012 17:38:00 +0000 (12:38 -0500)]
Bug 7645 - System preferences editor save button obscured by language chooser

This patch amends the preferences page's CSS file to add
bottom padding to the page container. This gives the bottom-
most submit button some breathing room.

Markup correction: Adding "row" and "col" attributes to
<textareas> to quiet validation errors. This doesn't
affect output at all (dimensions are specified in CSS).

To test, open the system preferences page to a tab like OPAC
which contains many preferences. At the bottom of the page
the last submit button should have about a line's worth of
white space below it.

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoMerge remote-tracking branch 'origin/new/bug_7001'
Paul Poulain [Tue, 13 Mar 2012 11:27:32 +0000 (12:27 +0100)]
Merge remote-tracking branch 'origin/new/bug_7001'

10 years agoBug 7167: Followup: FIX conflicts with Bug 5347
Jonathan Druart [Tue, 13 Mar 2012 08:55:00 +0000 (09:55 +0100)]
Bug 7167: Followup: FIX conflicts with Bug 5347

- Fix sql syntax error
- Fix Encoding
- SendAlerts must return an error if no email is defined
- Get error if no email in memberentry

http://bugs.koha-community.org/show_bug.cgi?id=7001
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agobug_7001: protected are only all libraries letters
Srdjan Jankovic [Mon, 12 Mar 2012 00:00:34 +0000 (13:00 +1300)]
bug_7001: protected are only all libraries letters

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
solves my comment 89: when you have a "mandatory" notice defined for a given library, it's not possible to delete it (and switch back to the "default" notification). isn't this a (small) bug ? was it intended ?

thanks!

10 years agobug 7001 follow-up fixing POD mistake
Paul Poulain [Tue, 13 Mar 2012 10:18:39 +0000 (11:18 +0100)]
bug 7001 follow-up fixing POD mistake

10 years agoMerge remote-tracking branch 'origin/new/bug_7458'
Paul Poulain [Mon, 12 Mar 2012 17:20:36 +0000 (18:20 +0100)]
Merge remote-tracking branch 'origin/new/bug_7458'

10 years agobug_7458: Correct existing existing call numbers retrieval
Srdjan Jankovic [Wed, 7 Mar 2012 04:44:01 +0000 (17:44 +1300)]
bug_7458: Correct existing existing call numbers retrieval

Adjusted to account for duplicate call numbers in the database.  Now retrieves next available call number correctly.

10 years agoBug 7577 follow-up
Paul Poulain [Mon, 12 Mar 2012 13:02:03 +0000 (14:02 +0100)]
Bug 7577 follow-up

The suggestion view was suggested to be able to display the suggestion from a basket line (when it comes from a suggestion, see bug 5346)

This tiny follow-up introduce this link

10 years agoBug 7577: Adds a display page for suggestions
Jonathan Druart [Wed, 22 Feb 2012 14:30:38 +0000 (15:30 +0100)]
Bug 7577: Adds a display page for suggestions

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
10 years agobug 7408 follow-up fixing trailing `
Paul Poulain [Fri, 9 Mar 2012 15:28:38 +0000 (16:28 +0100)]
bug 7408 follow-up fixing trailing `

10 years agoBug 7408 - Don't cancel found reserves.
Kyle M Hall [Thu, 8 Mar 2012 19:45:56 +0000 (14:45 -0500)]
Bug 7408 - Don't cancel found reserves.

At the moment, a reserve will be canceled if it has passed its
expiration date, even if the item is waiting or in the process
of being transferred. The situation could arise where someone
has a hold filled, but it is canceled while in transit, or before
the borrower can pick it up.

10 years agobug 7408 follow-up DBRev number
Paul Poulain [Fri, 9 Mar 2012 15:13:35 +0000 (16:13 +0100)]
bug 7408 follow-up DBRev number

10 years agoBug 7408 - Expire holds that have been waiting too long
Kyle M Hall [Thu, 23 Feb 2012 17:09:06 +0000 (12:09 -0500)]
Bug 7408 - Expire holds that have been waiting too long

If the new syspref ExpireReservesMaxPickUpDelay is enabled,
this will cancel holds that have been waiting for longer than the
number of days specified in the syspref ReservesMaxPickupDelay.

If ExpireReservesMaxPickUpDelayCharge is set, the borrower charged the fee set therein.

Signed-off-by: Ian Walls <koha.sekjal@gmail.com>
Altered circulation.pref to include currency class and [% local_currency %] param

10 years agoRemove a merge marker.
Marc Balmer [Fri, 9 Mar 2012 13:39:49 +0000 (14:39 +0100)]
Remove a merge marker.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agobug 7001 follow-up for DBRev
Paul Poulain [Fri, 9 Mar 2012 10:11:21 +0000 (11:11 +0100)]
bug 7001 follow-up for DBRev

10 years agobug_7001: Issue and Reserve slips are notices.
Srdjan Jankovic [Fri, 9 Mar 2012 00:22:17 +0000 (13:22 +1300)]
bug_7001: Issue and Reserve slips are notices.

Branches can have their own version of notices - added branchcode to
letter table.
Support html notices - added is_html to letter table.
Support for borrower attributes in templates.
GetPreparedletter() is the interface for compiling letters (notices).
Sysprefs for notice and slips stylesheets
Added TRANSFERSLIP to the letters

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 6086 follow-up Uppercase initial coding guideline fix
Paul Poulain [Thu, 8 Mar 2012 17:20:17 +0000 (18:20 +0100)]
Bug 6086 follow-up Uppercase initial coding guideline fix

The patch did not add invalid uppercases, but did not fix them either. I fixed them

10 years agoBug 6086: Followup: Replace \t characters with 4 spaces
Jonathan Druart [Thu, 8 Mar 2012 10:53:49 +0000 (11:53 +0100)]
Bug 6086: Followup: Replace \t characters with 4 spaces

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 6086 - adding a dynamic filter to pending holds
Alex Arnaud [Fri, 2 Mar 2012 15:21:07 +0000 (16:21 +0100)]
Bug 6086 - adding a dynamic filter to pending holds

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
10 years agoBug 7631 - Self checkout renewal fails because of reference to non-existent subroutin...
Owen Leonard [Thu, 1 Mar 2012 15:00:14 +0000 (10:00 -0500)]
Bug 7631 - Self checkout renewal fails because of reference to non-existent subroutine in sco-main.pl

Changing "CanBookBeIssuedCheckout" to "CanBookBeIssued"

To test, try to renew an item which has no renewals left. Before the patch
you'll get an error:

Undefined subroutine &main::CanBookBeIssuedCheckout called at /opac/sco/sco-main.pl line 135.

After the patch you'll get the correct message about having no renewals left.
Other tests: checking out a barcode which doesn't exist, checking out an
item which is on hold for another patron.

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoMerge remote-tracking branch 'origin/new/bug_6488'
Paul Poulain [Thu, 8 Mar 2012 17:08:44 +0000 (18:08 +0100)]
Merge remote-tracking branch 'origin/new/bug_6488'

10 years agoBug 6488: Display OPAC search result count conditional to OpacHiddenItems syspref
Srdjan Jankovic [Wed, 7 Mar 2012 03:53:09 +0000 (16:53 +1300)]
Bug 6488: Display OPAC search result count conditional to OpacHiddenItems syspref

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Works as advertised. Amended this patch for a small spelling issue: supress->suppress. Could not suppress it ;)

10 years agoBug 7603: Change description of 952$3 in German frameworks to reflect new functionality
Katrin Fischer [Sun, 26 Feb 2012 08:58:19 +0000 (09:58 +0100)]
Bug 7603: Change description of 952$3 in German frameworks to reflect new functionality

952$a can be used for a note about accompanying materials. The note will now
show up on checkout, on return and on staff's catalog detail page.
Patch changes the field description in the German frameworks,
attempting to make it a bit more clear what the field can be used for.

To test:
Run the webinstaller selecting German and make sure all bibliographic
Frameworks (Default, Fast add, sample frameworks) install without errors.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
I can't comment on the translations, but the installation seems to work.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7629 - Update sco.css to include CSS Gradients and prevent error upon missing...
Owen Leonard [Thu, 1 Mar 2012 14:40:01 +0000 (09:40 -0500)]
Bug 7629 - Update sco.css to include CSS Gradients and prevent error upon missing background image file

Copying CSS gradients for input buttons and dialogs from opac.css. Also copied missing
CSS for language-picker footer. Markup corrected in a couple of places in order
for the right style to be applied.

To test corrections to dialog CSS, enter an invalid patron barcode.
Multiple languages will have to be enabled in order to see the footer
correction working.

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Dialog CSS looks correct, footer also looks correct with multi languages.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7271 - change sort in items.pm to sort by home library instead of holding branch
Liz Rea [Mon, 28 Nov 2011 17:39:33 +0000 (11:39 -0600)]
Bug 7271 - change sort in items.pm to sort by home library instead of holding branch

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7517: Revised Patch. Patron category types not displaying <optgroup> in dropdown.
Garry Collum [Sat, 25 Feb 2012 20:09:45 +0000 (15:09 -0500)]
Bug 7517: Revised Patch. Patron category types not displaying <optgroup> in dropdown.

The patron category drop-down box does not display the general category groups as it once did.  This patch fixes the display of the optgroups.

To test. Before applying the patch, go to a patron record.  Edit the patron. Select the category drop-down in Library Management to see if the general categories appear. After the patch is applied the dropdown will show the categories.  'Child' - child categories, 'Adult' - adult categories, etc.

Fixes XHTML errors.
1. Invalid end tag for cardnumber input.
2. Extra quote in textarea.
3. The label for debarredcomment did not have a corresponding id.  There are two instances of this id field because of an 'IF' statement, this patch failed QA in the prior patch because only one of these ids was changed.

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7493 follow-up DBRev number
Paul Poulain [Thu, 8 Mar 2012 16:02:00 +0000 (17:02 +0100)]
Bug 7493 follow-up DBRev number

10 years agoBug 7493 [REVISED] Deleting a record with comments breaks "Recent Comments"
Owen Leonard [Mon, 6 Feb 2012 14:05:18 +0000 (09:05 -0500)]
Bug 7493 [REVISED] Deleting a record with comments breaks "Recent Comments"

Adding foreign key constraint on biblio and borrowers:

- delete reviews when bibliographic records are deleted
- set set reviews.borrowernumber to NULL when patron records are deleted.

Before these constraints are set the database update script will
clean up existing instances of these problems by deleting reviews of
deleted biblios and setting to NULL reviews.borrowernumber for deleted
patrons.

In comments moderation, the list of comments will indicate "(deleted patron")
if the review was submitted by a patron who has since been deleted.

In the OPAC display of comments will omit the patron name altogether
if the patron has since been deleted.

To test:

1. CONFIRM THAT THE DATABASE UPDATE RUNS CORRECTLY

Before applying the patch:

 - delete the record for a title which has a review
 - delete a patron who has submitted a review (on a different title).

Viewing /cgi-bin/koha/opac-showreviews.pl should result in an error.

Apply the patch and make sure the database update runs. Viewing
the recent comments page in the OPAC should no longer result in an
error. The title with the comment from a deleted patron should
show no patron name on these pages:

 - Recent comments (opac-showreviews.pl)
 - Recent comments RSS (opac-showreviews.pl?format=rss)
 - Detail page (opac-detail.pl)

Comments from other patrons should show correctly.

2. CONFIRM THAT NEW DELETIONS PERFORM CORRECTLY

After applying the patch:

 - delete the record for a title which has a review
 - delete a patron who has submitted a review (on a different title).

Viewing /cgi-bin/koha/opac-showreviews.pl should NOT result in an error.
The review of the title which you deleted should no longer appear in the
list of recent comments.

The title with the comment from a deleted patron should
show no patron name on these pages:

 - Recent comments (opac-showreviews.pl)
 - Recent comments RSS (opac-showreviews.pl?format=rss)
 - Detail page (opac-detail.pl)

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7615 - Give option to use description for homebranch/holding branch in label...
Chris Nighswonger [Wed, 29 Feb 2012 00:21:11 +0000 (19:21 -0500)]
Bug 7615 - Give option to use description for homebranch/holding branch in label creator instead of the branchcode

This patch adds the ability to print the name of the item's homebranch on labels

Thanks to Shane Sammons <ssammons-at-npelem.com> for the modified SQL SELECT statement.

Document Manager: The documentation will need to be updated to reflect the added field 'branchname' to the list
of available fields for label printing.

To test:

1. Create a new label layout or modify an existing one to include 'branchname'
2. Create a new label batch or using an existing one, export the batch.
3. Verify that the resulting labels contain the home branch name for the respective items.

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Tested with the plan - works

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7616 - Remove unused template markup for css_libs, css_module, js_libs, etc.
Owen Leonard [Tue, 28 Feb 2012 20:23:02 +0000 (15:23 -0500)]
Bug 7616 - Remove unused template markup for css_libs, css_module, js_libs, etc.

Removing references to unused template variables and markup.

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7584 - Update cart and lists buttons style using CSS3 features
Owen Leonard [Wed, 22 Feb 2012 20:23:12 +0000 (15:23 -0500)]
Bug 7584 - Update cart and lists buttons style using CSS3 features

Replacing styling of cart and lists buttons with CSS3 features:
border-radius, background-gradient, multiple backgrounds. While
the CSS is a little more complicated, the markup is much simpler
which should make customization simpler.

Browsers tested:

Firefox 10 on Windows 7
Chrome 17 on Windows 7
Opera 11 on Windows 7
Internet Explorer 8 on Windows 7 (no rounded corners or background images)

Firefox 9 on Ubuntu 11.10
Chrome 16 on Ubuntu 11.10
Opera 11 on Ubuntu 11.10

Internet Explorer 6 on Windows XP (looks acceptably bad)

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Also tested in:

Safari on Mac OS X
Firefox 10 on Mac OS X
Chrome on Mac OS X

All the above tests passed

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
10 years agoBug 7284: Fix unit test broken by erroneous POD
Jared Camins-Esakov [Wed, 7 Mar 2012 23:55:52 +0000 (18:55 -0500)]
Bug 7284: Fix unit test broken by erroneous POD

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>