[koha-commits] main Koha release repository branch 3.10.x updated. v3.10.05-58-gea3005f

Wed May 8 09:49:30 CEST 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.10.x has been updated
       via  ea3005fa49ffc4699e697d925e6828ed8f602101 (commit)
       via  bdf07b45f9b1593caaf0c65190a3a51c669e51f1 (commit)
       via  ed0298ec0c610214e6fe311d4b6bfe5d50a33375 (commit)
       via  09a227e7e9a6cb19dd1909acbe4b7b19d95c1a7e (commit)
      from  ed918e3248bf2a93759905189d9a0b90cb757665 (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 ea3005fa49ffc4699e697d925e6828ed8f602101
Author: Fridolyn SOMERS <fridolyn.somers at biblibre.com>
Date:   Wed Apr 10 12:37:54 2013 +0200

    Bug 10022 - Multiple tags moderation doesn't work on translated page

    When using a translated intranet, tags moderation using checkboxes and a
    submit button doesn't work. You get a message "Unknown Operation on x
    Term(s)" (translated of course).

    This patch corrects by using name="op-[operation]" instead of name="op"
    in each submit input.

    Test plan :
    - Create a new tag on a record
    - Go to tags moderation in Tools
    => You should see your tag in pending terms
    - Click on checkbox
    - Click on Approve or Reject submit button at the bottom of the table
    => Without patch the tag is still pending
    => With patch operation is well performed, tag is not pending
    - Test "Check lists" form
    - Test Approve and Reject buttons from "Actions" column

    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Passes all tests and QA script.
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit bdf07b45f9b1593caaf0c65190a3a51c669e51f1
Author: Jared Camins-Esakov <jcamins at cpbibliography.com>
Date:   Mon Apr 29 12:32:48 2013 -0400

    Bug 10152: Default sorting wrong in parcels.pl

    When you are receiving an order, the list of existing invoices should
    appear in reverse chronological order. Unfortunately, right now it
    appears in regular chronological order.

    To test:
    1) Make sure you have multiple invoices that have varying shipment
       dates for a vendor. We will assume this vendor is called "Example
       Vendor."
    2) Choose the "Receive shipments" button on the Example Vendor page.
    3) Note that the invoices are sorted by regular chronological order.
    4) Apply patch.
    5) Refresh "Receive shipment" page. Note that invoices are now sorted
       in reverse chronological order.
    6) Sign off.

    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Works as described.
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit ed0298ec0c610214e6fe311d4b6bfe5d50a33375
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Mon Apr 29 12:34:10 2013 -0400

    Bug 10153 - Patron header search form is too wide

    The patron header search form is now too wide to display correctly on
    screens smaller than about 1110px. This patch groups the
    previously shown-by-default choices with the hidden additional
    search parameters.

    To test, apply the patch and view any page which includes the patron
    header search include, for example members/members-home.pl. Additional
    search options should be hidden by default. Clicking "[+]" should show
    all additional options: Search fields, search type, order by, library,
    and category.

    Signed-off-by: Nicole C. Engard <nengard at bywatersolutions.com>
    Works for me.

    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Passes all tests and QA script. I like this change.
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>

    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 09a227e7e9a6cb19dd1909acbe4b7b19d95c1a7e
Author: Edmund Balnaves <ejb at prosentient.com.au>
Date:   Mon Apr 15 11:08:06 2013 +1000

    Bug 8264 - local cover images not working in IE8 [ prog ]

    [Patch submitted by David Cook]
    Edmund, rather than myself, was the author of this patch,
    so I'm not entirely sure about his code choices, but...it
    looks like IE must process images differently than other
    browsers.

    It seems to me that the images are not appearing in IE,
    because they're being regarded as incomplete or having
    0/NULL natural width.

    This patch simply introduces a try/catch hack where it
    tries to add the image and remove any "no-image" elements
    even nwhen the image is considered incomplete/width of
    0/NULL. It won't do this if it catches any errors,
    although I'm not sure that any errors would be caught
    with this set up anyway...

    It's certainly not the ideal solution, but it's a solution.

    To Test:

    Before applying the patch:
    1) Use IE and FF/Chrome/other browser to view a record
       with local images attached
    2) Note that the local images appear in FF/Chrome/other,
       but not in IE

    Apply patch.

    After applying the patch:
    1) Repeat step 1 from above
    2) Note that the local images now appear in all browsers
       as they should.

    --

    As I mentioned before, this seems like a real suboptimal
    solution. Any ideas on why IE might be considering local
    images to be incomplete or having 0/NULL natural width?

    Signed-off-by: David Cook <dcook at prosentient.com.au>
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    I did a regression test using Chromium and Firefox in
    Ubuntu. I checked both prog and CCSR - covers displayed
    nicely.
    All tests and QA script pass.

    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    Changed author of patch to reflect actual authorship.

    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

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

Summary of changes:
 acqui/parcels.pl                                   |    2 +-
 .../prog/en/includes/patron-search.inc             |   73 ++++++++++----------
 koha-tmpl/intranet-tmpl/prog/en/js/localcovers.js  |    7 ++
 .../intranet-tmpl/prog/en/modules/tags/review.tt   |    6 +-
 koha-tmpl/opac-tmpl/prog/en/js/localcovers.js      |    7 ++
 tags/review.pl                                     |    7 ++-
 6 files changed, 61 insertions(+), 41 deletions(-)

hooks/post-receive
-- 
main Koha release repository