[koha-commits] main Koha release repository branch master updated. v3.16.00-430-g7f8c59e

Tue Aug 19 14:31:56 CEST 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  7f8c59edaa9c790f36fe99552e818cfe8d8f9c8f (commit)
       via  4daaf8067811ac60b4bca891a5379315b5167127 (commit)
       via  6f10c7bbbcb596e79344d15885df8b36eb7d7f51 (commit)
       via  7d203a28fe4f6b8af5df2ef0e67f4b3c9ee2579c (commit)
       via  99e3da58d4e78572003ee232456ccf3652596f62 (commit)
       via  fa678fa241c396eceede9bb1cf6f114d29209c9b (commit)
      from  7706ba9df49794002392597e36f1f4edf3d9e9ab (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 7f8c59edaa9c790f36fe99552e818cfe8d8f9c8f
Author: David Cook <dcook at prosentient.com.au>
Date:   Mon Aug 4 13:49:56 2014 +1000

    Bug 12708 - Unexpected behaviour in IE 9 and lower when using openWindow

    This patch removes the whitespace from arguments to the "name" parameter
    in "window.open". It also adds a trap for the "openWindow" helper
    function, which will use a "null" instead of a "name" if window.open
    isn't able to handle a name with whitespace (i.e. if it's IE <= 9).

    _TEST PLAN_

    1) Switch to an emulated or authentic Internet Explorer version E 9
    or lower
    2) Click "Help"
    3) Note that it opens the page in the current window rather than as a popup

    4) Apply the patch

    5) Shift refresh your page
    6) Click "Help"
    7) Note that it now opens (correctly) as a pop-up

    (If you want to be more thorough, you can test in other browsers as well
    to make sure that it still works.)

    Tested with IE 10 developer tools: IE7, IE 8, IE 9
    Window pops up as expected. No regression found with IE 10 / FF31.0
    Signed-off-by: Marc Véron <veron at veron.ch>

    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Tested for regressions in Firefox and Chromium.
    I checked the help and the duplicate authorities windows
    still work correctly.
    Passes tests and QA script.

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 4daaf8067811ac60b4bca891a5379315b5167127
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Wed Oct 30 10:50:13 2013 +0100

    Bug 9180: All branches should be returned if a default rule exists

    The C4::Overdues::GetBranchcodesWithOverdueRules routine has a bug.
    If a default rule *and* a specific rule exist, only the branchcode for
    the specific rule is returned.

    Test plan:
    prove t/db_dependent/Overdues.t
    and verify the unit tests are consistent.

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 6f10c7bbbcb596e79344d15885df8b36eb7d7f51
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date:   Sun May 4 14:19:02 2014 -0300

    Bug 12180 - Remove HTML from batchMod.pl

    This patch removes HTML code from batchMod.pl

    To test:
    1. Check no regressions on batchMod editor,
    Go to Tools > Batch item modification,
    put a know barcode on barcode list, press continue

    2. Check new line on translation files, update
    a language, look for 'Tag editor'

    Fixed capitalisation

    Signed-off-by: Srdjan <srdjan at catalyst.net.nz>

    Amended patch, a stupid mistake on line 198 of batchMod-edit.tt

    -                    [% javascript %]
    +                    [% mv.javascript %]

    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Works as described, no problems found.
    Passes tests and QA script.

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 7d203a28fe4f6b8af5df2ef0e67f4b3c9ee2579c
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date:   Sat May 3 21:35:43 2014 -0300

    Bug 12177 - Remove HTML from authorities.pl

    This patch removes HTML code from authorities.pl

    To test:
    1. Check no regressions on authority editor,
    new/edit/save
    2. Check new line on translation files, update
    a language, look for 'Tag editor'

    Signed-off-by: Srdjan <srdjan at catalyst.net.nz>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Checked various features of the authority edtior:
    - repeating subfields
    - deleting subfields
    - changing the order of subfields
    - linking a subfield to an authorised value
    - plugins
    No regressions found.
    Passes tests and QA script.

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 99e3da58d4e78572003ee232456ccf3652596f62
Author: Katrin Fischer <katrin.fischer at bsz-bw.de>
Date:   Fri Jul 25 12:23:58 2014 +0200

    Bug 12501: Show dissertation note (MARC21 502) in XSLT results

    Bug 9356 added the dissertation note from MARC21 502 to the staff
    and OPAC detail pages. This patch adds it to the result list.

    1) Catalog a record with 502
       Examples: http://www.loc.gov/marc/bibliographic/bd502.html
    2) Important: Wait until the record got indexed by Zebra or
       index it manually.
    3) Verify your note shows up on the detail pages in staff and OPAC
    4) Verfiy note also shows up on the result pages in staff and OPAC

    5) Hide the dissertation note from display:

    OpacUserCSS:
    .diss_note {
      display:none;
    }

    IntranetUserCSS:
    .diss_note {
      display:none;
    }

    6) Verify CSS works and all looks like before...

    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Work as describes, value can be hidden using css,
    no koha-qa errors

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit fa678fa241c396eceede9bb1cf6f114d29209c9b
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Tue Aug 12 19:34:34 2014 -0300

    Bug 12753: Warnings in t/SIP_Sip.t could be tested

    A C4::SIP warning is expected and should be tested.

    To test:
    - Run
      $ prove -v t/SIP_Sip.t
    => FAIL: A warning is printed to STDOUT
    - Apply the patch
    - Run
      $ prove -v t/SIP_Sip.t
    => SUCCESS: No warnings on STDOUT, a new test verifying the warning is added.
    - It was an easy one, smile and sign off :-D

    Sponsored-by: The Will to Procrastinate
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    Sponsored-by: Sunday TV is boring
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Sponsored-by: Avoiding distance study homework
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

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

Summary of changes:
 C4/Overdues.pm                                     |   18 ++-
 authorities/authorities.pl                         |  136 +++++++++-----------
 koha-tmpl/intranet-tmpl/prog/en/js/staff-global.js |    9 +-
 .../prog/en/modules/authorities/authorities.tt     |   30 ++++-
 .../prog/en/modules/cataloguing/addbiblio.tt       |    2 +-
 .../prog/en/modules/tools/batchMod-edit.tt         |   28 +++-
 .../prog/en/xslt/MARC21slim2intranetResults.xsl    |   13 ++
 .../bootstrap/en/xslt/MARC21slim2OPACResults.xsl   |   13 ++
 t/SIP_Sip.t                                        |   29 ++++-
 t/db_dependent/Overdues.t                          |   51 +++++++-
 tools/batchMod.pl                                  |   76 ++++++-----
 11 files changed, 278 insertions(+), 127 deletions(-)

hooks/post-receive
-- 
main Koha release repository