[koha-commits] main Koha release repository branch master updated. v3.16.00-869-g7b7a452

Fri Oct 31 17:12:54 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  7b7a45269a6d949df84b6760857b83da8bda065c (commit)
       via  82c65678bf86686b7cfccc41fe7f921510bfe94c (commit)
       via  0f41f12621659c34f46bb987819f30d1ee8c6d7f (commit)
       via  a8532787f89654570aae9f8581df3b4eae448068 (commit)
       via  86c251c6db5ff7ad14e313d8748318e7f3d51841 (commit)
      from  5e5f0ee283eed34f3defb663f0177e163a854af9 (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 7b7a45269a6d949df84b6760857b83da8bda065c
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Fri Oct 24 09:42:37 2014 -0400

    Bug 13141: (QA Followup) Update number of unit tests

    While updating the number of unit tests from 12 to 13,
    the number of tests to skip due to a outdated version
    of DBD::Mock was missed.

    This patch retains the test number change, and corrects
    the number of tests.

    TEST PLAN
    ---------
    1) prove -v t/db_dependent/ReportsGuided.t
       -- on an UBUNTU git (lacks DBD::Mock >= 1.45)
    2) prove -v t/db_dependent/ReportsGuided.t
       -- on a DEBIAN git (has DBD::Mock >= 1.45)
    3) compare results.
       -- skipped tests should be 8 (2 individual tests + 6 in the loop)
          under Ubuntu.
       -- non-skipped tests should align.
       -- everything should be 13 tests.

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

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 82c65678bf86686b7cfccc41fe7f921510bfe94c
Author: Mark Tompsett <mtompset at hotmail.com>
Date:   Fri Oct 24 00:20:29 2014 -0400

    Bug 13141: Add ability for biblio_framework to be a dropdown in Guided Reports

    By tweaking the GetReservedAuthorisedValues function in
    C4::Reports::Guided, biblio_framework can be added as a dropdown
    list into the guided reports parameters.

    The change in C4/Reports/Guided.pm required a test, which is
    found in t/db_dependent/ReportsGuided.t

    Code was then added to reports/guided_reports.pl to build the
    appropriate hash to trigger the proper dropdown list.

    TEST PLAN
    ---------
    1) Apply patch
    2) prove -v t/db_dependent/ReportsGuided.t
       -- all should pass, this confirms that both
          C4/Reports/Guided.pm and this test file work.
    3) Log into staff client
    4) Reports
    5) Create from SQL
    6) Enter appropriate information like:
    Report name: Test 13141
    -- no need to change Report group or Report is public or Notes or Type
    SQL:
    SELECT CONCAT('<a href=\"/cgi-bin/koha/catalogue/detail.pl?biblionumber=',
         biblio.biblionumber,'\">',biblio.biblionumber,'</a>') AS
         BiblioNumbers, title, author, frameworkcode
    FROM biblio
    WHERE frameworkcode=<<Enter the frameworkcode|biblio_framework>>

    7) Save report
    8) Run report
       -- The parameter entry page should have a drop down of
          framework codes.
    9) Select a framework code, and click Run the report
       -- The displayed SQL should have a "frameworkcode=" portion
          matching the selected framework.
    10) Run the koha qa test tool.

    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Test plan followed successfully.

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

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 0f41f12621659c34f46bb987819f30d1ee8c6d7f
Author: Chris Cormack <chrisc at catalyst.net.nz>
Date:   Fri Oct 31 12:54:04 2014 +1300

    Bug 13174 : XSLT fails if, and only if you have an author, linked to an authority

    And only in MARC21

    To test:

    Only for MARC21 with XSLT on for detail

    1/ Find a record that has an authority linked author in it
    2/ View the detail page in the opac
    3/ Notice bibliographic data is missing
    4/ Apply patch
    5/ Refresh notice bibliographic data is back

    Signed-off-by: David Cook <dcook at prosentient.com.au>

    Works as described.

    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>

    Works as describe, the details are back!

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

commit a8532787f89654570aae9f8581df3b4eae448068
Author: Robin Sheat <robin at catalyst.net.nz>
Date:   Fri Oct 31 11:38:07 2014 +1300

    Bug 13173 - remove some operations that are no longer needed

    Build was failing because some files that it expected had vanished.

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

commit 86c251c6db5ff7ad14e313d8748318e7f3d51841
Author: Chris Cormack <chrisc at catalyst.net.nz>
Date:   Fri Oct 31 11:15:17 2014 +1300

    Bug 13172 t/Koha_Email.t was database dependent

    To test
    1/ Stop your database server
    2/ run t/Koha_Email.t
    3/ Notice the errors
    4/ Apply patch
    5/ run t/Koha_Email.t again
    6/ No errors

    (Don't forget to restart your db server after)

    Signed-off-by: Robin Sheat <robin at catalyst.net.nz>

    Verified that it fixes a package build.

    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Works as described, tests pass now without database connection.

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

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

Summary of changes:
 C4/Reports/Guided.pm                                       |    3 ++-
 debian/rules                                               |    4 ----
 .../opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl  |    2 ++
 reports/guided_reports.pl                                  |   12 +++++++++++-
 t/Koha_Email.t                                             |    5 +++++
 t/db_dependent/ReportsGuided.t                             |    7 ++++---
 6 files changed, 24 insertions(+), 9 deletions(-)

hooks/post-receive
-- 
main Koha release repository