[koha-commits] main Koha release repository branch master updated. v17.05.00-339-ge359035
Git repo owner
gitmaster at git.koha-community.org
Thu Aug 10 18:55:45 CEST 2017
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 e35903574ce666bca32eb4676f311b99bb268ec2 (commit)
via 61f2ec769073709be3b688654ec355b3f4fbe522 (commit)
via 91dee226cdd089353dc71e559aa9e0b34fddf72e (commit)
via 04966a6203646cae79a48a00ef0873c1f6c64b06 (commit)
via ee6f8e186bc93df9d7943ea38f05615b2e8a813f (commit)
via 4a302b17afee055c5d123d78328d50bdc7531259 (commit)
via efc4caec968dd0394c6d1ee8a2c4be66cc697afb (commit)
via 93366e10fbe2b018caa9d6b388b74cc3ef302a86 (commit)
via 3e5cad0f03d5d90eff2d6cedfb4ff6014c0610c0 (commit)
via ffe659e92ac023d59ebe6dca40a79056d15fae87 (commit)
via 19eb17c7894c4d2d53ecfc1ad4e417aa86ab9410 (commit)
via e57b3a6c4d9bb6319f038909832281dd4a9e39e5 (commit)
from 1da47215516206d51545e5bc662a34e335127ff6 (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 e35903574ce666bca32eb4676f311b99bb268ec2
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date: Thu Jul 6 08:48:18 2017 +0200
Bug 18900: wrong number format in receiving order
Bug 18471 added number formating in order receive.
I found a bug generated by this.
Koha::Number::Price defines methods to (un)format price depending on CurrencyFormat syspref.
The bug is that for CurrencyFormat US, some params are not defined in this module so there are retrieved from system locale. If system locale is not en_US, the params will be wrong. For example fr_FR will define ',' as decimal and so saved unit price is divided by 1000.
To recreate:
- On a server with locale 'fr_FR'
- With Plack mode
- Set CurrencyFormat on 'US'
- Open a basket
- Place an order for an item
- Close basket
- Receive order with changing 'Actual Cost' to '1,234'
- Note the 'Actual Cost' is now '1.23'
- Run unit test t/Number/Price.t without and with patch
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 61f2ec769073709be3b688654ec355b3f4fbe522
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date: Wed Aug 9 12:01:17 2017 -0300
Bug 18900: (QA followup) Make tests conditional to locale existence
It is important to highlight that tests have been skipped instead of having them pass
on the absence of the required fr_FR.UTF-8 locale installed on the system.
This patch does that check and effectively skips them.
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 91dee226cdd089353dc71e559aa9e0b34fddf72e
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date: Thu Jul 6 08:44:51 2017 +0200
Bug 18900: add UT to Number/Price.t
This patch adds unit tests to t/Number/Price.t in order to valid the problem.
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 04966a6203646cae79a48a00ef0873c1f6c64b06
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date: Thu Aug 10 13:15:27 2017 -0300
Bug 5471: Use uri_escape_utf8 to handle utf8 chars correctly
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit ee6f8e186bc93df9d7943ea38f05615b2e8a813f
Author: Owen Leonard <oleonard at myacpl.org>
Date: Fri Jun 9 12:29:58 2017 +0000
Bug 5471 - Quotes in tags fail
This patch makes changes to the tag moderation template and JavaScript
to fix handling of tags with double or single quotes. This patch also
moves the tags moderation JavaScript out of the template and into a
separate JS file.
To test you should have multiple tags awaiting moderation, including
tags which contain double and single quotes.
- Go to Tools -> Tags.
- In the list of tags pending approval, test approving and rejecting
tags, including those containing single or double quotes.
- The state of the "Approve" or "Reject" buttons should correctly
change according to the action you chose.
- The label in the status column should update correctly.
- In the "Check lists" form, submitting approved, rejected, and
unclassified terms should result in the correct message.
Signed-off-by: Lee Jamison <ldjamison at marywood.edu>
Works correctly based on test plan. Tested using single- and
double-quoted tags. Passes QA Tools.
Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 4a302b17afee055c5d123d78328d50bdc7531259
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date: Thu Jun 22 10:14:29 2017 +0200
Bug 18980: Show distinction between shared and private lists in staff
Just as we show this distinction in OPAC, this patch adds a type column
in the Your lists tab that displays Private or Shared. It always contains
Public in the other tab.
Test plan:
[1] Check if you see Shared for a private lists with shares in staff.
[2] Run t/db_dependent/Utils/Datatables_Virtualshelves.t
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Lee Jamison <ldjamison at marywood.edu>
Signed-off-by: Marc Véron <veron at veron.ch>
Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit efc4caec968dd0394c6d1ee8a2c4be66cc697afb
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date: Wed Jun 21 16:53:56 2017 +0200
Bug 18980: Remove js function Check from opac-shelves
This function only serves to check if the shelf name is not empty. It
does not even work since it incorrectly refers to f.addshelf (while it
should test #shelfname).
We can solve this and improve consistency by doing the same as in the
staff template with html5 required.
Test plan:
[1] Without this patch it was possible to add a list without a name in
OPAC (with js error: TypeError: f.addshelf is undefined).
[2] With this patch, verify that you cannot do this anymore.
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Lee Jamison <ldjamison at marywood.edu>
Signed-off-by: Marc Véron <veron at veron.ch>
Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 93366e10fbe2b018caa9d6b388b74cc3ef302a86
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date: Wed Jun 21 16:24:57 2017 +0200
Bug 18980: Add js to dynamically show/hide the Anyone remark
If we we move from Private to Public or vice versa, this should reflect
on the Anyone remark from the first patch.
If we go back to a private list without shares, we should show it.
Handled in a js sub AdjustRemark, triggered by onchange of the category
combo or the permission combo.
Test plan:
[1] Edit a private list without shares in OPAC. Toggle category and/or
permissions. Is the remark shown or hidden accordingly?
Note: Include a test with OpacAllowPublicListCreation enabled.
[2] Edit a shared list or public list in staff. Toggle category/permissions.
You should never see the remark.
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Marc Véron <veron at veron.ch>
Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 3e5cad0f03d5d90eff2d6cedfb4ff6014c0610c0
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date: Wed Jun 21 13:13:17 2017 +0200
Bug 18980: Add remark next to "Anyone seeing this list" permission
This is a follow-up of report 18228 that simplifies the permissions for
lists. It reapplies the idea behind bug 10865 by adding a remark next to
the Anyone permission if it is not actually effective. A next patch will
make it behave dynamically.
Note: The Anyone permission is not relevant if it is a private list that
has no shares. Note that looking at the prefs OpacAllowPublicListCreation
and/or OpacAllowSharingPrivateLists is not decisive. You should look at
list permissions and shares in the database; turning off the Sharing pref
does not automatically remove all shares in the system. It only blocks
creating new shares.
We only need changes in opac-shelves.tt and virtualshelves/shelves.tt.
Test plan:
[1] Verify that you see the remark now in OPAC and staff.
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Marc Véron <veron at veron.ch>
Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit ffe659e92ac023d59ebe6dca40a79056d15fae87
Author: Liz Rea <liz at catalyst.net.nz>
Date: Tue Jun 27 11:42:44 2017 +1200
Bug 18857 - Add buttons to left hand side of circ rules table
To test:
* Apply this patch
* Define some rules in the interface using the left hand buttons
* Define some rules in the interface using the right hand buttons (scroll to see)
* Delete some rules in the interface using the left hand buttons
* Delete some rules in the interface using the right hand buttons (scroll to see)
* Choose a rule to edit using the left hand buttons
* Edit the rule, saving using the left hand button
* Edit a rule, but clear the values using the left hand buttons
* Choose a rule to edit using the right hand buttons (scroll to see)
* Edit the rule, saving using the right hand buttons (scroll to see)
* Edit a rule, but clear the values using the right hand buttons (scroll to see)
Signed-off-by: Guillaume <guillaume_lair at ville-limoges.fr>
Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 19eb17c7894c4d2d53ecfc1ad4e417aa86ab9410
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date: Wed Aug 9 12:59:20 2017 -0300
Bug 19067: Test Makefile.PL mapping
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit e57b3a6c4d9bb6319f038909832281dd4a9e39e5
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date: Wed Aug 9 12:27:00 2017 -0300
Bug 19067: Map clubs/ into INTRANET_CGI_DIR in Makefile.PL
This patch adds the required entry in Makefile.PL so the clubs/ dir is correctly installed
during the make install step.
To test:
- Run:
$ cd kohaclone
$ perl Makefile.PL
- Follow any install method and options
- Run
$ make
$ ls blib/INTRANET_CGI_DIR/ | grep clubs
=> FAIL: No clubs dir
- Run:
$ make clean
- Apply this patch
- Run:
$ perl Makefile.PL
- Follow any install method and options
- Run
$ make
$ ls blib/INTRANET_CGI_DIR/ | grep clubs
=> SUCCESS: The clubs/ dir is correctly picked for inclussion in the intranet cgi dir
- Sign off :-D
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
-----------------------------------------------------------------------
Summary of changes:
C4/Utils/DataTables/VirtualShelves.pm | 1 +
Koha/Number/Price.pm | 3 +
Makefile.PL | 1 +
.../prog/en/modules/admin/smart-rules.tt | 17 +-
.../intranet-tmpl/prog/en/modules/tags/review.tt | 216 ++++----------------
.../prog/en/modules/virtualshelves/shelves.tt | 59 +++++-
.../virtualshelves/tables/shelves_results.tt | 2 +
.../intranet-tmpl/prog/js/pages/tags-review.js | 162 +++++++++++++++
.../opac-tmpl/bootstrap/en/modules/opac-shelves.tt | 59 ++++--
.../opac-article-request-cancel.pl => t/Makefile.t | 49 +++--
t/Number/Price.t | 22 +-
tags/review.pl | 9 +-
12 files changed, 356 insertions(+), 244 deletions(-)
create mode 100644 koha-tmpl/intranet-tmpl/prog/js/pages/tags-review.js
copy opac/opac-article-request-cancel.pl => t/Makefile.t (53%)
mode change 100755 => 100644
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list