[koha-commits] main Koha release repository branch master updated. v3.18.00-487-gd149cd1

Thu Mar 19 19:02:36 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  d149cd1d977ab796edd5da9881f08bac251c54e1 (commit)
       via  bceac94f0a64b826413baecc82a690fca58a46f8 (commit)
       via  8cee27c9155e5aad23dfb3496fec469b2bfec5fd (commit)
       via  88816eebd9a7b3edabdc894a2d7722f5ddec0a38 (commit)
       via  0de27f53ae635a30fcd1e1d2afc4713d9d12807e (commit)
       via  e043b5563f4f4fc038c8857b6f9d54c80f8dbbbe (commit)
       via  6b3fd48f3f0cdf798fc6f28055cc833db94f081c (commit)
       via  a5290b2f9dff755758f85276786ea9b6e534cf85 (commit)
       via  4c196e62c6dc5e85c4c6d3c2582674f94b34abc2 (commit)
       via  7e148240091ab89d268e60fb7d1ff467a0c78c7b (commit)
      from  15468615f9c5c427cc819bf833c2df58d13c5080 (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 d149cd1d977ab796edd5da9881f08bac251c54e1
Author: Dobrica Pavlinusic <dpavlin at rot13.org>
Date:   Fri Feb 13 15:37:12 2015 +0100

    Bug 13709 - Patron cards creator utf-8 encoding and ttf fonts broken

    This change is similar to Bug 8375 which introduced ttf fonts for
    labels printing in order to support diacritics and utf-8 encoding,
    but this change was never implemented for patron cards.

    Test scenario:

    1. make sure that you have <ttf> font mapping in koha-conf.xml
    2. define partron card layout, template, profile and batch
       (with utf-8 chars, probably in patron firstname or surname)
    3. verify that without this patch pdf export file is error message
       Wide character in compress at /usr/share/perl5/PDF/Reuse.pm line 820
    4. apply this patch and verify that generated pdf has correct encoding

    Signed-off-by: Brendan Gallagher <brendan at bywatersolutions.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit bceac94f0a64b826413baecc82a690fca58a46f8
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Mon Jan 19 11:47:46 2015 +0100

    Bug 13021: Reintroduce the name column for guarantor search

    Previous patch removed the name column, it's a regression.
    This patch reintroduces it.

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

commit 8cee27c9155e5aad23dfb3496fec469b2bfec5fd
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Thu Oct 2 09:42:35 2014 +0200

    Bug 13021: guarantor search - guarantor_search.tt is useless

    The guarantor_search.tt is now useless, no script uses it.
    It can be removed safely.

    Signed-off-by: Morag Hills <the.invinnysible.one at gmail.com>

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

commit 88816eebd9a7b3edabdc894a2d7722f5ddec0a38
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Thu Oct 2 09:42:11 2014 +0200

    Bug 13021: guarantor search - main patch

    This patch is the main patch.
    The "common" template is improved to allow different type of picking:
    "add" or "select".
    The first one appends a patron to a list, the second one selects the
    patron and close the result search window.

    The members/guarantor_search.pl has completly changed but is quite the
    same file as acqui/add_user_search.pl. Both should exist: they don't
    belong to the same module (acqui vs members), the picking type is
    different (add vs select) and the columns are not the same.

    The changes in the common template are very powerful, it's now possible
    to list the column we want! This will be very useful for further
    reusability.

    Before this patch, all patrons mathing the pattern were return. Now only
    the first 20 are (depends on the DataTables selected value).

    For QA: This patch introduces a new template plugin "To", for now it
    permits to convert a perl structure to json. In the idea, it could
    permit to convert foo to bar too.

    Test plan:
    1/ Verify there is no regression in the guarantor search. When the
    selection has been done, all data from the guarantor should fill the
    form in the "main address" section.
    Note that the request is done when the search input in not empty and the
    user stop to write for 1 sec.
    2/ Verify there is no regression on the 2 other pages where this patron
    search is used: link a patron to an order and to a basket (in the
    acquisition module).

    Signed-off-by: Morag Hills <the.invinnysible.one at gmail.com>

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

commit 0de27f53ae635a30fcd1e1d2afc4713d9d12807e
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Thu Oct 2 09:36:53 2014 +0200

    Bug 13021: guarantor search - prepare existing code to be reused

    The add_user_search tt file will be reuse in next commits, this commit
    just moves it in a "common" directory.

    Signed-off-by: Morag Hills <the.invinnysible.one at gmail.com>

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

commit e043b5563f4f4fc038c8857b6f9d54c80f8dbbbe
Author: Marc Véron <veron at veron.ch>
Date:   Thu Mar 12 08:04:14 2015 +0100

    Bug 13826 - OPAC: Display RSS icon on list of recent comments

    This patch adds RSS icon to OPAC page 'Recent comments'

    To test:
    1)
    Enable display of 'Recent comments' in OPAC (Syspref OpacShowRecentComments)
    2)
    In OPAC, go to Home > Recent comments
    3)
    Verify that there is a link in the head part of the HTML source code (<link rel="alternate" type="application/rss+xml"....), but no RSS icon on the pate itself (like in search results).
    4)
    Apply patch
    Result: Icon with link to RSS feed is displayed  next to title.

    Bug 13826 - Follow-Up: Close </h3>

    There was a slash missing.

    Signed-off-by: Mirko Tietgen <mirko at abunchofthings.net>
    Signed-off-by for the Follow Up: Marc Veron <veron at veron.ch>

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

commit 6b3fd48f3f0cdf798fc6f28055cc833db94f081c
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Mon Feb 16 16:33:16 2015 +0100

    Bug 13586: Remove some other occurrences

    Signed-off-by: Larry Baerveldt <larry at bywatersolutions.com>

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

commit a5290b2f9dff755758f85276786ea9b6e534cf85
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Mon Feb 16 16:28:51 2015 +0100

    Bug 13586: Remove the entire block

    Previous patch remove the condition but not the block tested.
    The entire block should be removed.

    Signed-off-by: Larry Baerveldt <larry at bywatersolutions.com>

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

commit 4c196e62c6dc5e85c4c6d3c2582674f94b34abc2
Author: Rochelle <Rochelle_healy at hotmail.com>
Date:   Thu Jan 15 20:51:43 2015 +0000

    Bug 13586: Makefile.PL mentions win32

    Edited Makefile.PL to remove mentions of win32
    To test: Run Makefile.PL

    Signed-off-by: Larry Baerveldt <larry at bywatersolutions.com>

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

commit 7e148240091ab89d268e60fb7d1ff467a0c78c7b
Author: Christophe Croullebois <christophe.croullebois at biblibre.com>
Date:   Tue Mar 10 10:29:15 2015 +0100

    Bug 13768: Correction of double quotes escaped into double quotes

    Without the patch the translated pages for quotes.tt may have the problematic line broken.
    So the js is broken too. The line with bad usage of double quotes escaped :
    var sEmptyTable = _("No quotes available. Please use the \"Add quote\" button to add a quote.");

    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>

    diff on staff PO file

    -msgid ""
    -"No quotes available. Please use the \\\"Add quote\\\" button to add a quote."
    +#, fuzzy
    +msgid "No quotes available. Please use the 'Add quote' button to add a quote."

    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>

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

Summary of changes:
 C4/Patroncards/Patroncard.pm                       |    9 +-
 C4/Utils/DataTables/Members.pm                     |    2 +-
 Koha/{Borrower.pm => Template/Plugin/To.pm}        |   37 ++----
 Makefile.PL                                        |   56 ++-------
 acqui/add_user_search.pl                           |    5 +-
 .../en/modules/acqui/tables/members_results.tt     |    2 +-
 .../prog/en/modules/admin/aqbudget_user_search.tt  |    6 +-
 .../prog/en/modules/admin/aqbudgets.tt             |    8 +-
 .../add_user_search.tt => common/patron_search.tt} |   63 ++++++----
 .../prog/en/modules/members/guarantor_search.tt    |  120 --------------------
 .../prog/en/modules/members/memberentrygen.tt      |   37 ++++++
 .../en/modules/members/tables/guarantor_search.tt  |   23 ++++
 .../intranet-tmpl/prog/en/modules/tools/quotes.tt  |    2 +-
 .../bootstrap/en/modules/opac-showreviews.tt       |    2 +-
 members/guarantor_search.pl                        |   97 ++++------------
 15 files changed, 158 insertions(+), 311 deletions(-)
 copy Koha/{Borrower.pm => Template/Plugin/To.pm} (69%)
 rename koha-tmpl/intranet-tmpl/prog/en/modules/{acqui/add_user_search.tt => common/patron_search.tt} (64%)
 delete mode 100644 koha-tmpl/intranet-tmpl/prog/en/modules/members/guarantor_search.tt
 create mode 100644 koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/guarantor_search.tt

hooks/post-receive
-- 
main Koha release repository