[koha-commits] main Koha release repository branch master updated. v3.16.00-957-gaef500f

Git repo owner gitmaster at git.koha-community.org
Fri Nov 7 15:42:15 CET 2014


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".

The branch, master has been updated
       via  aef500f6a3168be665d2a38e9aa2c745c6e24c78 (commit)
       via  4824ddba418be2e3d41412e7cbe740c12005c712 (commit)
       via  1aab0ff3e39a7be8855c70ba984520a56a121a85 (commit)
       via  71c0b8226e8fd4633831b81c9f9f6f1d1c84bbbb (commit)
       via  301b96e66de444fb4c03a76ef59842185489e2bc (commit)
       via  5f58431380c2ec753f4e69628c4edce82223ae9d (commit)
       via  81445f9e889c5f87a922fbfd7e7156d220a62cd7 (commit)
       via  499745460604cde6bf7f7afe4b4a8f2459c5aa4e (commit)
       via  6f17c68a24eb56976b684472ffdae2ce77e6955c (commit)
       via  694b56a9cef96095f21bcf9435cd05e8fa415b89 (commit)
       via  696c9feee25bf6ce15d850dc80dd0da034288471 (commit)
       via  1fc8916e2e944af608970ea418ac3fe9db92f0aa (commit)
       via  3fb57f34ea5644683e18262186dbba09e51a6c92 (commit)
      from  1cabaa32b40f225dedfc969db02af596d7d22994 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit aef500f6a3168be665d2a38e9aa2c745c6e24c78
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Fri Nov 7 11:37:27 2014 -0300

    Bug 12448: DBRev 3.17.00.048
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 4824ddba418be2e3d41412e7cbe740c12005c712
Author: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date:   Fri Aug 15 21:35:45 2014 +0200

    Bug 12448: Follow-up: Only ask confirmation when rental charge > 0
    
    To test:
    - Check out an item with no rental charge
    - Verify no confirmation message appears
    - Check out an item with rental charge
    - Verify rental charge shows
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 1aab0ff3e39a7be8855c70ba984520a56a121a85
Author: mbeaulieu <mbeaulieu at inlibro.com>
Date:   Tue Jul 29 16:20:39 2014 -0400

    Bug 12448: Ask for confirmation when checking out an item with rental fees
    
    Sponsored-by: Ville de Victoriaville, QC
    
    Confirmation box contents:
    
    "Please confirm checkout"
    "-Rental charge for this item: n"
    [Yes, check out (Y)] [No, Don't Check Out (N)]
    
    Test case A: Confirm checkout
    1) Go to checkout user "X"'s checkout page.
    2) Enter barcode for an item with rental fees.
    3) Click the "Check out" button.
    4) Confirmation box appears.
    5) Click on the "Yes" button.
    6) Item is added to checkout list.
    7) Fees are added to the patron's account.
    
    Test case B: Decline checkout
    1) Go to checkout user "X"'s checkout page.
    2) Enter barcode for an item with rental fees.
    3) Click the "Check out" button.
    4) Confirmation box appears.
    5) Click the "No" button.
    6) Checkout page goes back to its initial state.
    7) Patron has no item checked out and no fees to pay.
    
    Signed-off-by: Marc Véron <veron at veron.ch>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    With the system preference RentalFeesCheckoutConfirmation
    set to "don't ask" there is no change in behaviour.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 71c0b8226e8fd4633831b81c9f9f6f1d1c84bbbb
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Thu Aug 14 15:39:07 2014 +0200

    Bug 12760 - add restrictions purge to cleanup_database.pl (followup 2)
    
    Formatting :
    perltidy
    use q{} for SQL queries
    add use Modern::Perl
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 301b96e66de444fb4c03a76ef59842185489e2bc
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Thu Aug 14 15:15:48 2014 +0200

    Bug 12760 - add restrictions purge to cleanup_database.pl (followup 1)
    
    Remove unused $query var
    Correct redefined $count var
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 5f58431380c2ec753f4e69628c4edce82223ae9d
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Thu Aug 14 14:56:26 2014 +0200

    Bug 12760 - add restrictions purge to cleanup_database.pl
    
    This patch adds to the database cleanup script an option to purge expired patron restrictions (debarments in code).
    
    Test plan :
    - Select a borrower
    - Create a restriction with expiration date in the futur
    - Create a restriction expired since 7 days
    - Create a restriction expired since 14 days
    - run "misc/cronjobs/cleanup_database.pl -v --restrictions 14"
    => no restriction is removed for this borrower
    - run "misc/cronjobs/cleanup_database.pl -v --restrictions 13"
    => restriction expired since 14 days is removed
    - run "misc/cronjobs/cleanup_database.pl -v --restrictions 6"
    => restriction expired since 7 days is removed
    - run without argument "misc/cronjobs/cleanup_database.pl"
    => You see help text for restrictions option
    - run without days "misc/cronjobs/cleanup_database.pl -v --restrictions"
    => You get a purge on 30 days
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 81445f9e889c5f87a922fbfd7e7156d220a62cd7
Author: Mark Tompsett <mtompset at hotmail.com>
Date:   Sun Oct 26 22:43:08 2014 -0400

    Bug 9312: Resolve error triggered by mutli-type return value.
    
    $results is 1 upon success or a hash on failure.
    Rather than check %$results || %error, I changed it to
    $results!=1 || %error. Strangely, this works without
    messy warnings. I believe this resolves the problem Katrin
    mentions in comment #43.
    
    TEST PLAN
    ---------
    0) Make sure the system preference patronimages is set to Allow.
    1) Apply all the patches
    2) Login to staff client
    3) Tools -> Upload patron images (it's in the bottom left)
    4) Choose an image file, browse for a photo, enter a patron #.
    5) Click Upload
    6) Click the card number link
       -- the uploaded photo should be visible on the left side of
          the screen.
    7) Run koha qa test tools.
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    
    Image uploaded without problems
    No koha-qa errors
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 499745460604cde6bf7f7afe4b4a8f2459c5aa4e
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Thu Aug 22 14:26:45 2013 +0200

    Bug 9312: Followup for one tab character
    
    To satisfy qa tools, removing one tab somewhere..
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    
    Sometimes our QA tools are pretty annoying, in cases like this, when its a
    tab in a comment, .. I don't think we really need to hold up a patch
    set for it
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 6f17c68a24eb56976b684472ffdae2ce77e6955c
Author: Galen Charlton <gmc at esilibrary.com>
Date:   Wed Jan 16 08:12:31 2013 -0800

    Bug 9312: fix a couple more uninitialized variable warnings
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 694b56a9cef96095f21bcf9435cd05e8fa415b89
Author: Fridolyn SOMERS <fridolyn.somers at biblibre.com>
Date:   Wed Jun 12 15:29:26 2013 +0200

    Bug 9312: Perltidying picture-upload.pl
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 696c9feee25bf6ce15d850dc80dd0da034288471
Author: Fridolyn SOMERS <fridolyn.somers at biblibre.com>
Date:   Fri Dec 21 14:31:52 2012 +0100

    Bug 9312: strict perl for picture-upload.pl
    
    Testing - this is a code-cleanup patch, so the purpose
    of testing is to ensure that patron image upload functionality
    still works.  To test:
    
    - Turn on the patronimages system preference
    - Go to Tools | Upload patron images and import a suitable image
      for a test patron.
    - Bring up the test patron and verify that the image is attached.
    - From the patron details page, upload a replacement image.
    - Create a patron image ZIP file containing at least two images (
      per the documentation of the patron image feature) and load it
      via the patron image import tool.
    - Verfiy that the test patrons now have images.
    - Verify that tools/picture-upload.pl didn't report any errors
      in the Apache error log.
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 1fc8916e2e944af608970ea418ac3fe9db92f0aa
Author: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date:   Wed Nov 5 22:13:48 2014 +0100

    Bug 12829: Follow-up - fix small display issue in UNIMARC
    
    UNIMARC XSLT uses <li> instead of <span>.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 3fb57f34ea5644683e18262186dbba09e51a6c92
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Wed Oct 29 15:16:57 2014 -0300

    Bug 12829: properly skip invalid ISBN when rendering for normal record display
    
    Currently, 020$z (and 010$z fr UNIMARC) occurences are not shown, but the XSLT logic
    used introduces punctuation characters for those $z occurences.
    
    This patch adds a check for the existence of subfield a, and only loops
    on $a subfields.
    
    To test:
    - Create/have a record with 020$z (or 010$z on UNIMARC) but no 020$a (no 010$a on UNIMARC).
    - Open the detail page for the record (on both OPAC and staff).
    => FAIL: the ISBN label and ';' and '.' characters incorrectly shown.
    - Repeat mixing with 020$a occurences to notice the bug.
    - Apply the patch, reload
    => SUCCES: ISBN label shows only on the presence of a $a, and $z are skipped.
               no matter how many ISBN fields you add.
    - Sign off :-D
    
    Regards
    To+
    
    http://bugs.koha-community.org/show_bug.cgi?id=12901
    Signed-off-by: Nicole C. Engard <nengard at bywatersolutions.com>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

-----------------------------------------------------------------------

Summary of changes:
 C4/Circulation.pm                                  |   15 +-
 installer/data/mysql/sysprefs.sql                  |    1 +
 installer/data/mysql/updatedatabase.pl             |   11 +-
 .../en/modules/admin/preferences/circulation.pref  |    9 +-
 .../prog/en/modules/circ/circulation.tt            |    6 +-
 .../prog/en/xslt/MARC21slim2intranetDetail.xsl     |   25 +-
 .../prog/en/xslt/NORMARCslim2intranetDetail.xsl    |   25 +-
 .../prog/en/xslt/UNIMARCslim2intranetDetail.xsl    |   26 +-
 .../bootstrap/en/xslt/MARC21slim2OPACDetail.xsl    |   26 +-
 .../bootstrap/en/xslt/NORMARCslim2OPACDetail.xsl   |   25 +-
 .../bootstrap/en/xslt/UNIMARCslim2OPACDetail.xsl   |   26 +-
 kohaversion.pl                                     |    2 +-
 misc/cronjobs/cleanup_database.pl                  |  209 +++++----
 tools/picture-upload.pl                            |  458 ++++++++++++--------
 14 files changed, 538 insertions(+), 326 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list