[koha-commits] main Koha release repository branch 3.16.x updated. v3.16.06-86-g1d45527

Git repo owner gitmaster at git.koha-community.org
Thu Jan 22 18:47:17 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, 3.16.x has been updated
       via  1d45527578ddf7b21da64e9c0bcd73383a29f042 (commit)
       via  3a8e521641553ecdf7392252f894b890b8306618 (commit)
       via  7f0b3d0bc4868f46ebdd272a5303876a7cbe6110 (commit)
       via  3e7a965a5c620ba28162dbd01bc908c1afb5bef4 (commit)
       via  ffe32fefc9a937c09e30df4fa82860d2dbfab8eb (commit)
       via  2926168aaa44a79efd738221ee4bc942728eae38 (commit)
       via  e94c8136b8b7196db8e49207fcdd5c6093a210c7 (commit)
       via  070c80d31bb021949a5da93518d269294dae00c0 (commit)
       via  cb503984490d6b8c90038a117dd5920cd122b19b (commit)
       via  0da975638a099603771684628096117d14ad7a24 (commit)
       via  7aafba2bbe61b4acb7b6c6acf16857f16a22688d (commit)
       via  9ffc39b8bf6ff76f765af7cc7d8dc42d5fa2c6cf (commit)
       via  cb8332e1a620b1c8b848701edfee947e64d29645 (commit)
       via  2094d1c27e48e61532b58613a1add1c8e760f245 (commit)
       via  07975ec0e02a98709ae90b825d793a02c6407f89 (commit)
       via  e89b490843f91301cc726457290b66e9b62cf2ac (commit)
      from  c60fecc92f003bac0b7d6f0a087ed9fc947173f3 (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 1d45527578ddf7b21da64e9c0bcd73383a29f042
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Thu Jan 8 06:59:26 2015 -0500

    Bug 13537 - Focus should move to barcode after renewing via checkouts table
    
    Koha's checkout screen automatically focuses on the barcode field each
    time it is loaded. Since we've moved the checkouts table functions for
    renewing and returning items to ajax methods, this page is no longer
    refreshed, and thus does not refocus to the barcode after performing
    those actions. This should be fixed so that renewing/returning items via
    the checkouts page moves the focus to the barcode field.
    
    Test Plan:
    1) Find a patron with checkouts.
    2) Renew or return a checkout, note the focus does not move to the
       barcode.
    3) Apply this patch.
    4) Renew or return a checkout, note the focus *does* move to the
       barcode.
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Thomas <tomsStudy at gmail.com>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit 3a8e521641553ecdf7392252f894b890b8306618
Author: Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi>
Date:   Thu Oct 30 14:57:09 2014 +0200

    Bug 13168 - "Today's checkouts" sort improperly because issuedate lacks seconds.
    
    TO REPLICATE:
    
    Prepare a bunch of Items (6+) for checking out, or have a set of barcodes ready for copy-pasting.
    Check-out those items quickly within one minute and observe that the sorting order is not always from the first checkout to the last.
    
    This is because the issuedate doesn't have seconds defined.
    
    AFTER THIS
    
    The bunch of Items is sorted properly.
    
    Tiny patch, works as expected. Passed QA script.
    Signed-off-by: Marc Veron <veron at veron.ch>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit 7f0b3d0bc4868f46ebdd272a5303876a7cbe6110
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Tue Oct 21 04:44:24 2014 -0400

    Bug 13122 - Patron holds table no longer display date item went in transit
    
    In Koha 3.14 and earlier, an item on hold and in transit would display
    the date the item was transferred. This is missing from the new ajax
    holds table.
    
    Test Plan:
    1) Place an item on hold for delivery at a different library
    2) Check the item in, confirm the hold and transfer
    3) View the patron's holds tab on circulation.pl and/or moremember.pl
    4) Note the item is show as in transit, but does not give the "since"
       date
    5) Apply this patch
    6) Note the in transit status now has a "since <date>"
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Small change, works as described.
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit 3e7a965a5c620ba28162dbd01bc908c1afb5bef4
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Tue Nov 18 13:20:19 2014 -0500

    Bug 12971 [QA Followup] - Fix error caused by patron not having any checkouts
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    
    This successfully fixes the problem observed when a patron has no
    checkouts.
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit ffe32fefc9a937c09e30df4fa82860d2dbfab8eb
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Tue Nov 18 07:40:24 2014 -0500

    Bug 12971 [QA Followup]
    
    * Makes the status column display "Overdue!" if overdue
    * Fixes the due date formatting
    * Sorts the checkouts by date due ( oldest to newest )
      Note: I found no evidence that this data was previously sorted,
      so I kept it simple. Sorting based on system preferences could
      be a future enhancement.
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    
    This fixes the issues described for patrons with existing checkouts.
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit 2926168aaa44a79efd738221ee4bc942728eae38
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Mon Nov 17 12:31:05 2014 -0500

    Bug 12971 - Regression: Patron print summary doesn't show checkouts
    
    A patron's print summary should contain a list of checked out items
    as it did in 3.16.2 and earlier.
    
    Please note, as of 3.16.2 reserves were no longer part of the print
    summary and thus are not part of this bug fixing patch.
    
    Test Plan:
    1) Find a patron with checked out items
    2) Choose Print -> Print summary
    3) Note the lack of a list of checkouts
    4) Apply this patch
    5) Reload the page
    5) Print the summary again
    6) Note the list of checkouts
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Works as described, apart from the missing status information
    that Owen already noted on the bug.
    Passes tests and QA script.
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit e94c8136b8b7196db8e49207fcdd5c6093a210c7
Author: Chris Cormack <chris at bigballofwax.co.nz>
Date:   Wed Nov 5 20:00:39 2014 +1300

    Bug 12892 : Alternate fix to restore the deleted code
    
    This doesn't depend on anything other patch.
    
    1. Create a reservation for a Borrower with pickup branch
    2. Check-in the corresponding Item in the pickup branch and confirm the hold
       for the Borrower
    3. Go to circ/circulation.pl for the Borrower and verify that the reservation
       is "Waiting for pickup"
    4. In the Borrower -> Check Out -tab (circ/circulation.pl) on the right of the
       big "Checking out to borrowername" should be a notification about holds ready
       for pickup.
    
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    Signed-off-by: Rafal Kopaczka <rkk0 at poczta.onet.pl>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Passes all tests and QA script, restores the old code and behaviour.
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit 070c80d31bb021949a5da93518d269294dae00c0
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Tue Nov 18 12:40:43 2014 -0500

    Bug 13290 - Add better feedback for returns to the checkouts table
    
    The feedback given when returning an item is only to update the "Check
    in" column. It would be nice if the entire column was highlighted in
    some way so librarians have an easier time spotting which items were
    checked in and which ones failed to be checked in.
    
    Test Plan:
    1) Apply this patch
    2) Browse to a patron with checkouts
    3) Return an item using the checkouts table
    4) Note if the return was successful, the row is highlighted in
       yellow, if the return was not allowed, the row is highlighted
       in red.
    
    Signed-off-by: Christopher Brannon <cbrannon at cdalibary.org>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    (cherry picked from commit 9fcdb0108217fc746731d2d8736a8765427453a0)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit cb503984490d6b8c90038a117dd5920cd122b19b
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Mon Dec 8 07:39:35 2014 -0500

    Bug 13408: Tell librarians they may continue scanning while checkouts table is loading
    
    Some librarians do not realize they can continue scanning items even if
    the checkouts table has not loaded. We should have Koha tell them
    explicitly that they may continue scanning barcodes.
    
    Test Plan:
    1) Apply this patch
    2) Load the checkouts table on circulation.pl, note the loading message is now
       Loading... you may continue scanning.
    3) Load the checkouts table on moremember.pl, note the loading message is still
       Loading...
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    
    This one works too!
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    (cherry picked from commit 28513edb943c255b191c94e0fbc0fe14368e6985)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    
    Conflicts:
    	koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js

commit 0da975638a099603771684628096117d14ad7a24
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Fri Nov 21 08:13:46 2014 -0500

    Bug 13293 - Regression: Override renewal limit option broken with AJAX circ
    
    The "Override renewal limit" checkbox no longer works to un-hide the
    renewal checkboxes in the table of checkouts. I assume this is because
    the script works in such a way that it is looking for the checkboxes
    before they are part of the document.
    
    Test Plan:
    1) Apply this patch
    2) Check the "Override renewal limit" checkbox
    3) Note the checkbox now appears
    
    Note, this change allows the "X of Y renewals remaining" to be displayed
    for non-renewable items. I left this as it is for two reasons: 1) it
    gives the librarian more information that may be useful, and 2) it looks
    nicer and more uniform.
    
    Signed-off-by: Frederic Demians <f.demians at tamil.fr>
    
    I confirm the bug and the solution.
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Passes tests and QA script, works as described.
    Exception: on-site checkouts can't be renewed.
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit 7aafba2bbe61b4acb7b6c6acf16857f16a22688d
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Tue Nov 18 09:17:40 2014 -0500

    Bug 13286 - Checkouts table due date not updated when an item is renewed.
    
    When an item is renewed from the checkouts table, the new due date is
    displayed in the renewal column, but the old date due is still displayed
    in the due date column. This should be updated as well.
    
    Test Plan:
    1) Apply this patch
    2) Renew an item from the checkouts table
    3) Note the date due is updated in the "Due date" column
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    
    This works well in my test. My librarians will be grateful for the fix
    -- they were complaining about this bug.
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Works as described, passes tests and QA script.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 9ffc39b8bf6ff76f765af7cc7d8dc42d5fa2c6cf
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Thu Nov 13 11:04:17 2014 -0500

    Bug 13195 - Regression: Circulation checkouts table no longer shows item type description
    
    Another regression caused by Bug 11703: The list of checkouts on the
    circulation and patron detail page shows item type codes instead of the
    full description.
    
    Test Plan:
    1) View a patron's checkouts, note the Item type column displays the
       code rather than the description.
    2) Apply this patch
    3) Refresh the page, note the description is now displayed
    
    Signed-off-by: Frederic Demians <f.demians at tamil.fr>
    
    Works as described.
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    
    Works as described, no problems found.
    Passes tests and QA script.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    
    Conflicts:
    	koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js

commit cb8332e1a620b1c8b848701edfee947e64d29645
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Tue Nov 11 20:01:59 2014 -0500

    Bug 13236 - Regression: Table of checkouts no longer preselects overdue items for rewewal
    
    Before Bug 11703, overdue items in the list of a patrons checkouts had
    the renewal checkbox preselected so that librarians could quickly renew
    only those items which required it. This is not longer the case.
    
    This patch corrects it. To test, apply the patch and clear your browser
    cache. Check out to a patron who has overdues and confirm that the
    overdue items have the "renew" checkbox preselected. Check that items
    which are not overdue are not preselected.
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Works as described, no problems found.
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit 2094d1c27e48e61532b58613a1add1c8e760f245
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Thu Nov 13 12:03:42 2014 -0500

    Bug 12778 - Regression: Item lost status doesn't show in list of checkouts
    
    When using the longoverdue script it's possible that items marked lost
    remain on the patron account. I think it's important for staff to see
    that some items are marked lost - currently the list of checkouts
    doesn't show any sign of the lost status.
    
    Test Plan:
    1) Find a patron with a checked out lost item
    2) Note the lost status is not displayed in the checkouts table
    3) Apply this patch
    4) Refresh the page, note the lost status now displays
    5) Repeat this test plan for a damaged item
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Tested successfully with damaged and multiple lost values.
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Works as described, no problems found.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    
    Conflicts:
    	koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js

commit 07975ec0e02a98709ae90b825d793a02c6407f89
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Thu Nov 13 20:15:22 2014 -0500

    Bug 13253 - Unnecessary white space above checkouts table in circulation
    
    On the new checkouts page there is some padding above the checkouts,
    relatives' checkouts, and holds tables caused by extra markup in the
    table's sDom configuration (http://legacy.datatables.net/ref#sDom):
    
    <'row-fluid'<'span6'><'span6'>r>t<'row-fluid'>t
    
    This creates several empty <div>s which don't serve any purpose. This
    patch simplifies it to:
    
    rt
    
    To test, apply the patch, clear your browser cache, and check out to a
    patron who has items checked out, holds on their account, and child
    records attached which also have checkouts.
    
    The padding above the table of checkouts, the table of relatives'
    checkouts, and the table of holds should match that on the sides.
    
    Signed-off-by: Liz Rea <liz at catalyst.net.nz>
    Checked per plan on both Check Out and Details pages, spacing appears correct.
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Works as described, no problems found.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    
    Conflicts:
    	koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js

commit e89b490843f91301cc726457290b66e9b62cf2ac
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Wed Oct 1 08:25:02 2014 -0400

    Bug 13020 - Checkouts table default ordering is incorrect
    
    The checkouts table is no longer keeping "today's checkouts" in the
    order they were scanned.
    
    Test Plan:
    1) Create 3 records "Test 1", "Test 2" and "Test 3" each with one item
    2) Check out the items to a patron in the order "Test 2", "Test 1", "Test 3".
    3) Note the order is incorrect.
    4) Apply this patch
    5) Refresh the page
    6) Note the order is now correct
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Note that the bug already exists before bug 12550. I checked on a 3.14.x
    branch (without the ajax circ stuff), and the order is the same as with
    this patch.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    
    Conflicts:
    	koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js

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

Summary of changes:
 C4/Circulation.pm                                  |    6 +-
 circ/circulation.pl                                |   29 ++-
 .../intranet-tmpl/prog/en/css/staff-global.css     |    6 +
 .../prog/en/includes/members-toolbar.inc           |    2 +-
 .../intranet-tmpl/prog/en/includes/strings.inc     |    2 +-
 koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js    |   69 +++++--
 koha-tmpl/intranet-tmpl/prog/en/js/holds.js        |    4 +-
 .../prog/en/modules/circ/circulation.tt            |    1 +
 .../prog/en/modules/members/moremember-print.tt    |  196 +++++++++-----------
 members/summary-print.pl                           |  104 +++++++++++
 svc/checkouts                                      |   31 +++-
 11 files changed, 306 insertions(+), 144 deletions(-)
 create mode 100755 members/summary-print.pl


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list