[koha-commits] main Koha release repository branch 3.18.x updated. v3.18.06-3-16-g23a326b

Tue May 12 00:09:31 CEST 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.18.x has been updated
       via  23a326b1f67b0f3a51ce45c6241469022d856924 (commit)
       via  51936eb07926864a8ec2a8636215c90297accc54 (commit)
       via  c600d875b5aa19964f50cd3e96f9030661d3ca54 (commit)
       via  fa8d2b7efe29907b00a310b202d3ec51cfdf3c55 (commit)
       via  97efa0d1bc5841485db1a302afa51bae6403cd94 (commit)
       via  f6373d81726eeb570985c9d9d5159993cb7feb30 (commit)
      from  49b5934235b03f44be2240d9c72be59b2e8efe95 (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 23a326b1f67b0f3a51ce45c6241469022d856924
Author: Marc Véron <veron at veron.ch>
Date:   Sun Apr 19 07:37:37 2015 +0200

    Bug 14008: Proposed follow-up: Dialog should appear whenever filters are active

    The first patch displays the value(s) of an active filter if you come back to the page or reload the page via a menu action.
    To make the situation more clear the dialog "No saved reports match your criteria." shoudl always appear when a filter is active.

    Signed-off-by: Mark Tompsett <mtompset at hotmail.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 d1cdbf9d56038f0305f9be90205c0315882ec711)
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>

commit 51936eb07926864a8ec2a8636215c90297accc54
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Fri Apr 17 15:24:36 2015 +0200

    Bug 14008: Display active filters when searching for reports

    On the saved report list, the filters on the left are not displayed.
    It means that some can be set, but the user has not visible feedbacks.

    How filters work here:
    If a user set a filter, it is saved in the session. Next time he will
    come on this search, the previous values will be kept.

    This patch just displayed the filter's values in the input fields.

    Test plan:
    Search for reports by date, author and keyword.
    The filters should be displayed after you have clicked on the submit
    button ("Apply filter").

    Signed-off-by: Mark Tompsett <mtompset at hotmail.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 b05cbc415d2461a3909db593b9b12d5cc506e35d)
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>

commit c600d875b5aa19964f50cd3e96f9030661d3ca54
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Mon Apr 20 16:59:05 2015 +0200

    Bug 14022: Does not reset the subscription length on editing a subscription

    The subscription length is always set to the first/default value
    ('issues') when editing a subscription.
    Even if you selected another value on creating/editing it, the form will
    show you the first option.

    Test plan:
    1/ Add/Edit a subscription, select a subscription length different of
    the first value (issues)
    2/ Edit the subscription and confirm the value is kept

    Bonus: issues, week and months become translatable.

    Works as expected.
    Signed-off-by: Marc Véron <veron at veron.ch>

    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 9d7d645baf35ae8b0b37d5d6c8ad653f8aa55841)
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>

commit fa8d2b7efe29907b00a310b202d3ec51cfdf3c55
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Tue Apr 21 10:44:43 2015 +0200

    Bug 14016: (follow-up) Restore correct date format on issue date (00:00 vs 23:59)

    Same for the overdues.

    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 40543e9c45a9b986c0382cb25e6979a8cc224e56)
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>

commit 97efa0d1bc5841485db1a302afa51bae6403cd94
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Mon Apr 20 10:41:19 2015 +0200

    Bug 14016: Restore correct date format on issue date (00:00 vs 23:59)

    Introduced by bug 13601, and same fix used in bug 10423 and bug 12847:
    the date_due retrieved from the DB is modified.

    There are some problems:
    1/ There is confusion between the iso and sql formats in the codebase.
    2/ Since bug 13601, dt_from_string does not manage the iso format (there
    are occurrences of 'iso' but it assumes that both formats are
    identical).

    To solve the issue, 2 solutions:
    1/ Same as bug 10423 and bug 12847: try to get rid of the change done on
    date_due in C4::Members::GetPendingIssues, it should be kept as the sql
    value.
    2/ Too many errors found and another fallback should be added to
    dt_from_string (if 'iso' is passed, try sql then iso).

    Test plan:
    Go on the checkout list at the OPAC and confirm that the due dates are
    correctly formatted.

    Signed-off-by: Nicolas Legrand <nicolas.legrand at bulac.fr>

    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 9ad8b86643b4a5698851ce4a67122ca35c2a1b61)
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>

commit f6373d81726eeb570985c9d9d5159993cb7feb30
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Tue Mar 10 10:38:50 2015 -0700

    Bug 13817 - Edit links on course items may point to the wrong course!

    If items are assigned to multiple courses, the "edit" links for a course
    item may pass the wrong course id. This is really a cosmetic issue.

    It appears that in Template Toolkit, if you reuse an existing variable
    name in a foreach loop, it will not create a new temporary variable but
    will instead re-use the existing variable leaving assigned at the end to
    the last assignment, just as it would in Perl code.

    This is not a problem when all items are assigned to one and only one
    course, but can cause issues when we loop through the other courses an
    item may be on.

    This is solved easily by renaming the variable used in the foreach loop
    to another unused name.

    Test Plan:
    1) Apply this patch
    2) Try the edit and delete links for an item on reserve for a course
    3) No changes in behavior should be noted

    Signed-off-by: Tom Misilo <misilot at fit.edu>

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    (cherry picked from commit b8da5462b0f72f8035589317f51800394ba700ae)
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>

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

Summary of changes:
 C4/Members.pm                                      |    2 ++
 .../en/modules/course_reserves/course-details.tt   |   12 ++++----
 .../en/modules/reports/guided_reports_start.tt     |   11 +++----
 .../prog/en/modules/serials/subscription-add.tt    |   30 ++++++++++++++++----
 .../opac-tmpl/bootstrap/en/modules/opac-user.tt    |    6 ++--
 reports/guided_reports.pl                          |    1 +
 serials/subscription-add.pl                        |   18 ++++--------
 7 files changed, 45 insertions(+), 35 deletions(-)

hooks/post-receive
-- 
main Koha release repository