[koha-commits] main Koha release repository branch master updated. v3.16.00-711-ga732181

Git repo owner gitmaster at git.koha-community.org
Sun Oct 19 16:40:16 CEST 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  a7321813f43083db2e101c2c75dac37990002c65 (commit)
       via  4114495d05a98237d1454845d7851e52178924fb (commit)
       via  8848e72d781d53941446bed36d574d9311ffa0fc (commit)
       via  b4b9e0d21224f0f86fdb006f8071590352012bac (commit)
       via  e44b4cbeeddc1bf4f3c3957e311ca0dea30592c0 (commit)
      from  cf1a63cb0c2585c474ef2916414d936b288012d5 (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 a7321813f43083db2e101c2c75dac37990002c65
Author: simith <simith at inlibro.com>
Date:   Thu Jul 31 09:29:17 2014 -0400

    Bug 8687 - Overdues: Add utf-8 support to HTML and additional fields to CSV output
    
    This patch contains small improvements made in the overdue_notices.pl script.
    
    - UTF-8 support in the HTML output
    - Add the phone, cardnumber, branchname and letternumber columns
      to the csv output
    
    Testing:
    
    I) Create an issue overdue (intranet):
    0) Enter in a patron account, add some characters like
       ユーザー別サイト in his address
    1) Check if in Patron category administration the overdue column
       is YES for his code
    2) In Tools -> Notice triggers define first delay = 7, letter
       Overdue notice. Nothing for second ni third
    3) Check if Tools -> Notices and slips, code overdue is available
    4) checkout a book with a retard more than 7 days.
    5) Check if Adminstration -> Defining circulation and fine rules
       for all libraries has fine rules for his patron category,
       item type book.
    
    II) Before applying the patch
    0) Run misc/cronjobs/overdue_notices.pl -n -html .
    1) Open the file notices_"date".html in a brower
    2) Validate the characters are not correct
    
    III) After applying the patch
    0) Run misc/cronjobs/overdue_notices.pl -n -html .
    1) Open the file notices_"date".html in a brower
    2) Validate the characters are correct
    3) Run misc/cronjobs/overdue_notices.pl -n -csv temp.csv
    4) Validate phone, cardnumber, branchname and letternumber
       columns and values
    
    Sponsored-by: CCSR
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Passes tests and QA script.
    Works as described, no problems found.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    Added the Sponsored-by line.

commit 4114495d05a98237d1454845d7851e52178924fb
Author: Rafal Kopaczka <rkk0 at poczta.onet.pl>
Date:   Tue Oct 7 14:43:36 2014 +0200

    Bug 13035: Overdue notices send notices for due date in future.
    
    Overdue notices, script uses method days_between which returns always
    positive value of days, no mater if due_date is higher or lower from
    date_to_run. This causes overdue notices to be send for real overdue and
    for checkouts with due date in future, which have same days_between as
    in notice triggers.
    
    To reproduce:
    1. Set up overdue notice triggers for eg. 2 day.
    2. Checkout items for 2 borrowers first gets item with due date 2 days
    in past and second gets 2 days in future.
    3. run ./misc/cronjobs/overdue_notices.pl -n -t
    4. watch that notices was generated for both of them.
    
    To test:
    1. Repeat steps 1,2
    2. Apply patch
    3. run ./misc/cronjobs/overdue_notices.pl -n -t
    4. should be 1 notice for borrower with overdue item.
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Passes tests and QA script, just adds an additional test.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 8848e72d781d53941446bed36d574d9311ffa0fc
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>

commit b4b9e0d21224f0f86fdb006f8071590352012bac
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Wed Oct 1 08:35:40 2014 -0400

    Bug 12550 [QA Followup] - Make checking the checkbox load the table of checkouts
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit e44b4cbeeddc1bf4f3c3957e311ca0dea30592c0
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Thu Jul 10 12:37:44 2014 -0400

    Bug 12550 - Add ability to delay loading of patron's checkouts in circulation and patron details
    
    Now that a patron's checkouts are loaded asynchronously, we can further
    improve Koha's performance by not loading the checkouts table when it is
    not needed. For example, if a librarian is checking out 5 items to a
    patron, we really don't need to load the table during the scanning of
    items 1 to 4, just for item 5. Another example would be browsing to the
    patron details table in order to change a patron's password.
    
    Test Plan:
    1) Apply this patch
    2) Browse to circulation.pl for a patron
    3) Note the table is not loaded automatically
    4) Click the "Show checkouts" button
    5) Note the checkouts table loads
    6) Check the "Always show checkouts immediately" checkbox
    7) Reload the page, the checkouts should now load immediately
    8) Repeat steps 3-7 for the patron details page ( moremember.pl )
    
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Amended patch: remove trailing spaces.
    Note: I am not sure the checkbox is at the best place, but I don't have
    a better suggestion.
    
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

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

Summary of changes:
 koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js    |  446 +++++++++++---------
 .../prog/en/modules/circ/circulation.tt            |   13 +-
 .../prog/en/modules/members/moremember.tt          |   69 +--
 misc/cronjobs/overdue_notices.pl                   |   21 +-
 svc/checkouts                                      |    2 +
 5 files changed, 305 insertions(+), 246 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list