[koha-commits] main Koha release repository branch master updated. v3.20.00-1077-g55456b6

Thu Nov 5 13:52:37 CET 2015

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  55456b6e7ff0278f2821a71c38b9a5b67bfd4220 (commit)
       via  25bd92bc3e7271541817660963155468e12b4eda (commit)
       via  daf129f2f24132a271b8d1ca7725744856fece06 (commit)
       via  abe5c30973bb708c1a6e72ff450152c6466fd04b (commit)
       via  1ef0b3c47168c13c98bc839a2de9ef4d76b44560 (commit)
       via  e06c5a89fd539f8cc5f43773bf2aaec5c41e63ed (commit)
      from  35ecddec894ad633e26e8802c7c68864e395fca4 (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 55456b6e7ff0278f2821a71c38b9a5b67bfd4220
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date:   Wed Nov 4 09:24:10 2015 -0300

    Bug 15098: Itemtype description missing from facets for itypes in a search group

    This patch adds a second variable to hold item types info
    just for facets.

    In case we have an item type in a search category, it's description
    is removed from GetItemTypesCategorized, which is good for advanced
    search, but no for facets

    A second var is needed which holds all item type info.

    To test:
    On top of Bug 15092 patches (no really needed, but I write this on top
    of them)
    1) Add an item type to a search group
    2) Do a search in opac, in facets will see only item type code,
    no description, for that item type
    3) Apply the patch
    4) Repeat search, now description must be present

    Followed test plan including translated descriptions. Works as expected.
    Signed-off-by: Marc Véron <veron at veron.ch>

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

commit 25bd92bc3e7271541817660963155468e12b4eda
Author: Marc Véron <veron at veron.ch>
Date:   Wed Nov 4 10:58:50 2015 +0100

    Bug 15123: Acquisition: Better context for "Transferred from..." to avoid ambiguity in translation

    This patch moves the word "basket" out of the <a> tags to have a better context for translation.

    To test:
    - Transfer an order from one basket to another basket
    - Verify that "basket" is detached from the strings "Transferred to" rsp
      "Transferred from" by th <a> tag (See screenshot)
    - Apply patch
    - Verify that the word "basket" is now otuside the <a> tags

    Signed-off-by: Hector Castro <hector.hecaxmmx at gmail.com>
    Works as advertised. Tested with es-ES

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

commit daf129f2f24132a271b8d1ca7725744856fece06
Author: Liz Rea <liz at catalyst.net.nz>
Date:   Tue Nov 3 12:20:25 2015 +1300

    bug 15112: Make greybox consistent size across label/card creator

    Also fixes minor issue with buttons stacking on patroncards/edit-batch
    and labels/label-manage

    To test:

    Export a batch every way you can:

    multiple together from label/label-manage
    individual items from labels/label-edit-batch
    selected items from labels/label-edit-batch
    export full batch from labels/label-edit-batch

    multiple together from patroncards/manage
    individual cards from patroncards/manage
    individual cards from patroncards/edit-batch
    selected cards from patroncards/edit-batch
    export full batch from patroncards/edit-batch

    Also notice that before this patch, the delete and export buttons
    were stacking on the table rows, and  that now they are not doing
    that anymore. It may take a force refresh to see the behaviour and
    the change.

    Signed-off-by: Marc Véron <veron at veron.ch>

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

commit abe5c30973bb708c1a6e72ff450152c6466fd04b
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon Oct 5 11:17:23 2015 +0100

    Bug 14955: Don't double encode chars in the "Show more" link

    Bug 13789 has fixed this issue for the facet links (select/unselect) but
    not for the "Show more" facets link.

    This patch reuses the "url" variable, generated for the facets, for
    these links. A new "local_url" variable is used to build links for each
    entry.

    Test plan (for OPAC and intranet):
    - Search for a very common term
    - Filter the results using facets
    - You should get more than 5 entries for a facet and 1 selected:
    Author
     author 1
     author 2 [x] # Selected
     author 3
     author 4
     author 5
     Show more # you see this link if there is 5+ values for this facet
    - Click on the Show more link
    Without this patch, the url contains double-encoded characters and you
    will certainly don't get any results
    With this patch, you should get results.

    Signed-off-by: Marc Véron <veron at veron.ch>

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

commit 1ef0b3c47168c13c98bc839a2de9ef4d76b44560
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Tue Oct 27 09:21:24 2015 +0000

    Bug 15066: Make transfer rotating collection works under Plack

    This patch the 2 following errors:
    Undefined subroutine &C4::RotatingCollections::transferbook called at
    C4/RotatingCollections.pm line 451.
    Undefined subroutine &C4::RotatingCollections::GetTransfers called at
    C4/RotatingCollections.pm line 450.

    And this warning:
    "my" variable $colId masks earlier declaration in same scope at
    /home/koha/src/rotating_collections/transferCollection.pl line 75.

    Test plan:
    Create a rotating collection
    Transfer it to another branch
    It should work with this patch

    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>

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

commit e06c5a89fd539f8cc5f43773bf2aaec5c41e63ed
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date:   Tue Nov 3 15:15:54 2015 -0300

    Bug 15092: (followup) Descriptions not show for search categories

    If we have some item types in search categories, description
    is not show in opac advanced search

    To test:
    1) Add some item type to a search category
    2) Go to OPAC > Advanced search
    empty description for that category
    3) Apply the patch
    4) Repeat 2), description must be present

    Signed-off-by: Marc Véron <veron at veron.ch>

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

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

Summary of changes:
 C4/RotatingCollections.pm                           |    4 ++--
 koha-tmpl/intranet-tmpl/prog/en/includes/facets.inc |   19 +++++++++----------
 .../intranet-tmpl/prog/en/modules/acqui/basket.tt   |    8 ++++----
 .../prog/en/modules/labels/label-edit-batch.tt      |    9 +++++----
 .../prog/en/modules/labels/label-manage.tt          |    2 +-
 .../prog/en/modules/patroncards/edit-batch.tt       |    7 ++++---
 .../prog/en/modules/patroncards/manage.tt           |    4 ++--
 .../opac-tmpl/bootstrap/en/includes/opac-facets.inc |   19 +++++++++----------
 opac/opac-search.pl                                 |    9 +++++++--
 rotating_collections/transferCollection.pl          |    3 ++-
 10 files changed, 45 insertions(+), 39 deletions(-)

hooks/post-receive
-- 
main Koha release repository