[koha-commits] main Koha release repository branch 3.12.x updated. v3.12.06-47-gf254801

Git repo owner gitmaster at git.koha-community.org
Wed Nov 13 16:10:25 CET 2013


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, 3.12.x has been updated
       via  f25480111e8a27699693a613755afc5ade2da2d6 (commit)
       via  e7670dc2b690b76175cdb9507985a7ab38eba64d (commit)
       via  815876da1e11e43d016584eb35562a62e7a1f5d5 (commit)
       via  8777c4cd79aaeca1e5ac8fdd4bfa705b52d3efb4 (commit)
       via  973f0460c49f8cad7cb700950ecb0e45f981530e (commit)
       via  9297b61d200d5b94b5609df86ceb42e65f261c49 (commit)
       via  5b38ae5c7f826fae8eb4f270b7fbbfca4e15e2fc (commit)
       via  f0c5bd5efa79ef9441a2ec0cd52296e4ca9647a1 (commit)
       via  5a7e87c2d497d2f98a645a4c7e979df51b24b8cc (commit)
       via  66d1d18cf9177163d8ccc02140693fc7dc57833c (commit)
       via  783d5baeb9a33219a1c5c4085b2fba4d7ede923b (commit)
       via  01cde4055dc7bf19e4918f58c83ff7a4729e8f15 (commit)
       via  6207b60ca850c4ac969c6de70c62945b6c1d1dca (commit)
      from  3f671f71aa106d77cfe25a6aeb2a86433aee785f (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 f25480111e8a27699693a613755afc5ade2da2d6
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Wed Nov 13 12:23:41 2013 -0300

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

commit e7670dc2b690b76175cdb9507985a7ab38eba64d
Author: Galen Charlton <gmc at esilibrary.com>
Date:   Wed Sep 25 15:10:40 2013 +0000

    Bug 10687: (follow-up) remove authority 68a/68b tags from translated MARC21 framework scripts
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit a1c5d280f67a632ab9e1fdd32cf4967c3de6fdd6)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 815876da1e11e43d016584eb35562a62e7a1f5d5
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date:   Fri Aug 2 11:25:56 2013 -0300

    Bug 10687: Delete erroneous tags 68a and 68b on default MARC21 auth framework
    
    This patch deletes information for default
    MARC21 auth framework on tags 68a and 68b.
    Also added update for current installs.
    
    RM note: The 68a and 68b fields were defined in an attempt to include
    both MARC21 classification and MARC21 authority records in the same
    default framework, but I don't think this has turned out to be useful,
    as to my knowledge nobody tries to use MARC21 classification records
    in Koha, and if one were to want to, a distinct record type (or at
    least auth_type) would be a better way to do it.
    
    To test with current authorities_normal_marc21.sql loaded
    
    1) apply the patch
    2) run updatedatabase
    3) check on database that there are no more
    references to tags 68a and 68b
    
    To test for new installs
    1) remove auth frameworks
    2) load authorities_normal_marc21.sql
    3) check that load without problem
    4) check on database that there are no more
    references to tags 68a and 68b
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Paola Rossi <paola.rossi at cineca.it>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 15f28461e09cffe2917f424b8b7e84cd722b1246)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 8777c4cd79aaeca1e5ac8fdd4bfa705b52d3efb4
Author: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date:   Thu Aug 29 22:26:50 2013 +0200

    Bug 10803: make "Make payment" in circ easier to translate
    
    To test:
    - Choose a patron account
    - Add a fine to it
    - Switch to the checkout tab of the patron account
    - Check for the "Make payment" link. Make is not a link, but payment is.
    - Apply patch
    - Check again, now both words are linked.
    
    In translation this string is divided into:
    . %s Make
    payment
    
    Because of this it's difficult to translate it grammatically.
    The patch moves 'Make' into the link text, so that both words
    will show up together in translation.
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Comment: Works as described. No koha-qa errors.
    Not only improves translatability,
    also corrects capitalization.
    
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 39c435e36fec65dbba795214bdde79932bb4edcf)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 973f0460c49f8cad7cb700950ecb0e45f981530e
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Fri Sep 20 11:31:57 2013 -0400

    Bug 10924: fix updating contact note from OPAC
    
    If you update your "Contact note" via the OPAC that change doesn't
    appear in the staff client's "Update patron records" interface. The
    field is missing from members-update.tt
    
    To test, log in to the OPAC and update your details
    (opac-memberentry.pl), modifying the "Contact note" in the "Alternate
    address" section.
    
    In the staff client, view "Patrons requesting modifications"
    (members/memgers-update.pl). The changed contact note field should
    appear in the list of field awaiting approval for that patron.
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Comment: Work as described. No koha-qa errors
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    One line change - contact note now updates perfectly.
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 4713a27280b9ee0956c4f2d9ee0e63b12260db01)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 9297b61d200d5b94b5609df86ceb42e65f261c49
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Wed Sep 18 15:24:08 2013 -0400

    Bug 10911: fix invalid links when displaying records that have 880 fields
    
    Koha intranet search results using XSLT for records with 880 fields have
    the details link created by a code path that incorrectly links to some
    opac pages.
    
    Test Plan:
    1) Enable XSLT for intranet search results
    2) Upload or create a record with at least one 880 field
    3) Search for the item ( must have multiple hits to avoid redirection )
    4) Click the title link, note it is broken
    5) Apply this patch
    6) Click the title link, note it is correct
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Comment: Work as described, no koha-qa errors.
    
    Test
    1) imported two times sample record
    2) reindex
    3) search, find example record, broken links
    4) applied patch
    5) refresh, good links
    
    NOTE: A similar (invalid references) problem is present on
    UNIMARC and NORMAC files
    I'll file a new bug for those cases.
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Works nicely, tested with a different set of sample
    records with Hebrew 880 fields.
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 115a3e6b60cab0e5000483575ca69736bb40ee2c)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 5b38ae5c7f826fae8eb4f270b7fbbfca4e15e2fc
Author: Mathieu Saby <mathieu.saby at univ-rennes2.fr>
Date:   Sun Sep 8 19:38:35 2013 +0200

    Bug 10465: remove useless reference to fields 780 and 785 in UNIMARC XSLT files
    
    UNIMARCslim2OPACDetail.xsl and UNIMARCslim2intranetDetail.xsl contained
    references to 780 and 785 fields, that exist in MARC21 but not in UNIMARC.
    
    This patch remove the code related to 780 and 785 fields. No effect should
    be visible.
    
    To test : in a UNIMARC Koha
    1. Be sure that OPACXSLTDetailsDisplay, OPACXSLTResultsDisplay,
       XSLTDetailsDisplay and XSLTResultsDisplay sysprefs are set to "default"
    2. Apply the patch
    3. Search some records in staff interface : the list view and the detail
       view must be exacty the same as before you apply the patch.
    4. search some records in OPAC : the list view and the detail view must
       be exactly the same as before you apply the patch.
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Comment: No diplay change, no koha-qa errors.
    Removes what can't be displayed.
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Checked detail pages in OPAC and intranet still display correctly
    and without errors.
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 54befcf59ed85be0041d72760eb1771bcce0492e)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit f0c5bd5efa79ef9441a2ec0cd52296e4ca9647a1
Author: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date:   Tue Sep 10 21:18:13 2013 +0200

    Bug 10847: Open link to vendor's website in new window
    
    To test:
    - Create a new vendor or edit an existing one
    - Make sure you have entered a valid URL for the website
    - Click the link from the vendor summary page and verify it opens
      in a new window/tab (depending on your browser configuration)
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit d67ef01814fb4ba1519d66d9ad749dfa64e62a76)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 5a7e87c2d497d2f98a645a4c7e979df51b24b8cc
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Tue May 21 15:58:44 2013 -0400

    Bug 10305 - _increment_barcode is called twice
    
    For some reason, _increment_barcode is called twice in succession in
    cataloguing/additem.pl
    
    Test Plan:
    1) Enable autobarcode = incremental
    2) Catalog an item, not the barcode
    3) Apply this patch
    4) Catalog another item, the barcode should be the next in line
       as previously. No change should be noted.
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    (cherry picked from commit 449fc4e168368cc3b1991152f32a2862b578a04c)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 66d1d18cf9177163d8ccc02140693fc7dc57833c
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Fri Sep 13 14:45:50 2013 -0400

    Bug 10888: Don't hide authorities module from users who lack permission to edit authorities
    
    It should be possible for staff client users to search and view
    authority records even if they don't have permission to edit them.
    
    To test, apply the patch and view the staff client as a user with and
    without "editauthorities" permission.
    
    With "editauthorities" permission, authority search results should show
    both an edit and delete link. Viewing the details of an authority
    record, one should see a toolbar with edit/delete/new options.
    
    The detail view has been altered to use the term "record" in place of
    "biblio" ("Used in X record(s)).
    
    Without "editauthorities" permission, authority search results should
    show no edit or delete link. Viewing the details of an authority, the
    only option shown in the toolbar should be "Save."
    
    On the staff client home page and in the header's "More" menu the link
    to the authorities module should now appear with and without permission
    to edit authorities.
    
    This patch also corrects the permissions in the authority export script
    to allow saving of authority records by users who do not have permission
    to edit.
    
    Signed-off-by: Paola Rossi <paola.rossi at cineca.it>
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 9bd3a44944e6a0cb7920c3c53a88c11a8986b146)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 783d5baeb9a33219a1c5c4085b2fba4d7ede923b
Author: Galen Charlton <gmc at esilibrary.com>
Date:   Sat Sep 21 19:16:56 2013 +0000

    Bug 7813: (follow-up) use descriptive text for delete link
    
    This patch replaces the "times" symbol for the local cover image
    deletion link with more descriptive text.  This was done because
    the original "X" was small, is not necessarily going to be universally
    understood to signify the deletion action, and because "×"
    does not give much context to translators.
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit fe3628070a7affddb691ee589ec0e63639d0a36a)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 01cde4055dc7bf19e4918f58c83ff7a4729e8f15
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Fri Sep 20 13:06:48 2013 +0200

    Bug 7813: (follow-up) improvements for deleting local cover images
    
    This patch:
    - displays a js alert if a problem occurred on deleting the
      image on the server.
    - adds a class for the new span (in order to avoid future problem, if
      someone adds a new span).
    - adds a title for the [X] link.
    - deal with the upload_local_cover_images permission (don't display the
      delete link if the logged in user does not have it).
    - changes the license version.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    I like the ehancements to the original patch and work as expected.
    
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit 1dc7d421bc87e3cf027161b08fbb44875c778f49)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 6207b60ca850c4ac969c6de70c62945b6c1d1dca
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Fri Aug 30 16:43:30 2013 -0300

    Bug 7813: Ability to delete local cover images
    
    This patch adds the ability to delete local cover images from the
    detail page. It adds an 'X' button to trigger the deletion. It occurs
    using a new SVC script called cover_images that accepts the following
    parameters:
    
     - action
     - biblionumber (used for checking imagenumber validity)
     - imagenumber (repeatable)
    
    It then deletes all valid images passed and returns a JSON object
    containing the imagenumbers and the deletion status (1/0). The operation
    is triggered using jQuery.ajax, and with the response the corresponding
    images get deleted.
    
    Note: currently one image is deleted at a time, but the code is there to
    be used.
    
    To test:
    - Open the detail page fr a biblionumber containing one or more cover
      images, go to its images tab.
    - Apply the patch, reload the page, go to the images tab.
    - Thumbnails should look better than before ;-)
    - An 'X' button should show below each cover.
    - Clicking the 'X' button should trigger an alert message asking for
      confirmation.
     a) Cancel: nothing happens, reload the page and check nothing got
        deleted.
     b) Accept: the deleted image should be gone, reload the page to check
        it doesn't exist anymore and is not a browser/DOM manipulation trick.
    - Signoff
    
    Note: check having more cover images than those that fit the width and
    see it wraps fine.
    
    P.S. I chose not to implement every possible action on the svc script, but
    another bug could be filled for more improvements.
    
    Sponsored-by: Universidad Nacional de Cordoba
    Signed-off-by: David Cook <dcook at prosentient.com.au>
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>
    (cherry picked from commit ab362b93a710bddcd385a6a468af6ff682673367)
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

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

Summary of changes:
 authorities/export.pl                              |    2 +-
 cataloguing/additem.pl                             |    5 -
 .../marc21/mandatory/authorities_normal_marc21.sql |   26 -----
 .../marc21/mandatory/authorities_normal_marc21.sql |   26 -----
 .../marc21/mandatory/authorities_normal_marc21.sql |   58 ++++--------
 .../marc21/mandatory/authorities_normal_marc21.sql |   26 -----
 .../Obligatoire/authorities_normal_marc21.sql      |   26 -----
 .../marc21/mandatory/authorities_normal_marc21.sql |   26 -----
 .../marc21/mandatory/authorities_normal_marc21.sql |   26 -----
 .../marc21/mandatory/authorities_normal_marc21.sql |   26 -----
 .../marc21_authorities_DEFAULT_general.sql         |   32 -------
 .../marc21_authorities_DEFAULT_general.sql         |   32 -------
 installer/data/mysql/updatedatabase.pl             |    8 ++
 .../intranet-tmpl/prog/en/css/staff-global.css     |    4 +
 .../prog/en/includes/authorities-toolbar.inc       |   16 +++-
 .../intranet-tmpl/prog/en/includes/header.inc      |    2 -
 .../prog/en/modules/acqui/supplier.tt              |    2 +-
 .../en/modules/authorities/searchresultlist.tt     |   19 ++--
 .../prog/en/modules/catalogue/detail.tt            |   53 +++++++++--
 .../prog/en/modules/circ/circulation.tt            |    2 +-
 .../intranet-tmpl/prog/en/modules/intranet-main.tt |    2 -
 .../prog/en/modules/members/members-update.tt      |    1 +
 .../intranet-tmpl/prog/en/xslt/MARC21slimUtils.xsl |    6 +-
 .../prog/en/xslt/UNIMARCslim2intranetDetail.xsl    |  100 +-------------------
 .../prog/en/xslt/UNIMARCslim2OPACDetail.xsl        |  100 +-------------------
 svc/cover_images                                   |   76 +++++++++++++++
 26 files changed, 184 insertions(+), 518 deletions(-)
 create mode 100755 svc/cover_images


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list