[koha-commits] main Koha release repository branch master updated. v3.14.00-309-ge686dd2

Thu Jan 16 23:49:41 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  e686dd25438b66de751551c8f8fecff115b661d4 (commit)
       via  910dff28254b8a93ed03884b0e768fd7c013f449 (commit)
       via  ab2a422af61a3c6df80091a311b64e8448417b13 (commit)
       via  06ef9f11098405cd688f43af39d828a4eb674582 (commit)
       via  ae7431608cfa2f9aa2674079d17f3b96fa12c8c7 (commit)
       via  c62557a4fa239487e128754e79cd3a9427d4f964 (commit)
      from  4e4a33ef5deecc32711092d78d87f2eefe3fbcf5 (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 e686dd25438b66de751551c8f8fecff115b661d4
Author: Galen Charlton <gmc at esilibrary.com>
Date:   Thu Jan 16 23:07:09 2014 +0000

    Bug 10987: (follow-up) fix display of authorities containing only one see from or see also lnik

    This patch fixes an incorrect test on the number of see from and
    see also links in the authority record, as otherwise if a record
    had only one 4XX or 5XX, the linked headings weren't displayed at
    all.  This patch also makes the test consistent across the staff
    theme and the two OPAC themes.

    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 910dff28254b8a93ed03884b0e768fd7c013f449
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Wed Oct 2 15:42:13 2013 -0300

    Bug 10987: remove duplicate 'see also' and 'see from' labels from authority search results

    If an authority record has more than one 4xx$a (or 5xx$a) entries the
    corresponding labels 'see also' and 'used for/see from' werwe repeated.
    This patch removes duplicate instances of the labels, improving
    readability.

    To test:
    - Have authority records with more than one entry on the 4xx (or 5xx)
      fields.
    - Do a search, check "see also:" (or 'used for/see from') appear more
      than once.
    - Apply the patch
    - Reload and check it looks nicer :-D
    - Repeat for: staff auth search, OPAC using ccsr, OPAC using prog
    - Signoff

    Sponsored-by: Universidad Nacional de Cordoba

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

    Work as decribed. No koha-qa errors
    Nice view on staff and both opac

    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit ab2a422af61a3c6df80091a311b64e8448417b13
Author: Galen Charlton <gmc at esilibrary.com>
Date:   Thu Jan 16 22:44:25 2014 +0000

    Bug 10691: (follow-up) replace HTML comments with TT contents

    Comments on the usage of Template Toolkit blocks don't need
    to appear in the rendered HTML, so this patch converts HTML
    comments to TT comments and thereby saves a tiny amount of
    bandwidth.

    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 06ef9f11098405cd688f43af39d828a4eb674582
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Wed Dec 11 11:14:47 2013 -0300

    Bug 10691: (follow-up) restore behaviour for control field ($w)

    As noted by Jared, in the presence of 4XX$w or 5XX$w the display
    got broken by this refactoring. This patch restores the previous
    behaviour in that front.

    The OPAC themes get fixed too, and the Bootstrap one gets this
    fix too.

    Applied the fix that Galen proposed on comment #30 regarding
    Zeno's fix in bug 11174.

    Sponsored-by: Universidad Nacional de Cordoba
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit ae7431608cfa2f9aa2674079d17f3b96fa12c8c7
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Wed Sep 18 11:33:07 2013 -0300

    Bug 10691: use authority ID in see-also links in authority search results (OPAC)

    This patch changes the URL and data used to show the 'see also' links
    on the OPAC's authority search results page.

    Bonus points: makes some strings translatable.

    To test:
    - On your dev setup (master) create some authority records (I created
      personal name authorities).
    - Pick one of them and link 400$a to another one, do the same with 500$a
    - Add some other 400$a and 500$a without linking (i.e. plain text)
    - Make sure zebra is running and changes got indexed.
    - In the OPAC search for the authority that is linked to the others.
    - Check the 'see also:' link points to an authority search
    - Apply the patch
    - Reload/re-do the search
    - Check the 'see also:' link points to the authority id in the case
      of linked authorities, and to an authority search in the case of
      plain text headings.

    Sponsored-by: Universidad Nacional de Cordoba
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>

    Work as described. No errors

    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit c62557a4fa239487e128754e79cd3a9427d4f964
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Wed Sep 18 12:18:20 2013 -0300

    Bug 10691: use authority ID in see-also links in authority search results (staff)

    This patch changes the URL and data used to show the 'see also' links
    on the Staff's authority search results page.

    Bonus points: makes some strings translatable.

    To test:
    - On your dev setup (master) create some authority records (I created
      personal name authorities).
    - Pick one of them and link 400$a to another one, do the same with 500$a
    - Add some other 400$a and 500$a entries with plain text (i.e. no
      linking)
    - Make sure zebra is running and changes got indexed.
    - In the staff interface search for the authority that is linked to the
      others.
    - Check the 'see also:' link points to an authority search
    - Apply the patch
    - Reload/re-do the search
    - Check the 'see also:' link points to the authority id for linked
      authorities,
      and to an authority search result in the case of plain text entries.
    - Check that the authority search from the cataloguing interface still
      works as usual.

    Sponsored-by: Universidad Nacional de Cordoba
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>

    Work as described. No errors

    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

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

Summary of changes:
 .../intranet-tmpl/prog/en/css/staff-global.css     |    6 +-
 .../en/includes/authorities-search-results.inc     |  114 +++++++++++++-------
 .../en/modules/authorities/searchresultlist.tt     |    2 +-
 .../en/includes/authorities-search-results.inc     |   42 ++++++--
 .../en/modules/opac-authoritiessearchresultlist.tt |    4 +-
 koha-tmpl/opac-tmpl/ccsr/en/css/opac.css           |    6 +-
 koha-tmpl/opac-tmpl/prog/en/css/opac.css           |    6 +-
 .../en/includes/authorities-search-results.inc     |   81 +++++++++-----
 .../en/modules/opac-authoritiessearchresultlist.tt |   58 +++++-----
 9 files changed, 210 insertions(+), 109 deletions(-)

hooks/post-receive
-- 
main Koha release repository