[koha-commits] main Koha release repository branch master updated. v3.20.00-165-ga80d188
Git repo owner
gitmaster at git.koha-community.org
Fri Jun 19 16:43:48 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, master has been updated
via a80d188c83be6131c9274a29c5dcc03893c9e617 (commit)
via a712f5b7bad38cb9f5ad599fdb063cbe6e49c547 (commit)
via a810b1cf59fc0e44ca88b3746b7c0a813703c59c (commit)
via f668a46e23746daa18c5fe8c6005a8661553c645 (commit)
via b335b7299120f642a0f7d7654937521df8f75ba6 (commit)
via 2eaeb708795e7624eb8873b617d4a38d69fa84fc (commit)
via c76b6a148b85b27ec0d3c4bea1f78ab30d3df72f (commit)
via be35039b55a351c97f2c1f9a5b373cb26ac5e0b0 (commit)
via d847b1d92a9df6db2bb5321f032f3ec13d6ba55d (commit)
via b740b1b412e11b1d540b243e7b1767cc0c1cb962 (commit)
via 688452ad7e9131a53a96bd826e6228e73494fa53 (commit)
via 7ab873aaea298c787e93438012fa8792345664f4 (commit)
via 892d374b64fa4eed98955d75b517702f78f1ca40 (commit)
via 7c0c92807f49ef61aa975e84cf26d42f7dfa425f (commit)
from d56a275c0d101e7969192ef326ceaa64cacf83d8 (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 a80d188c83be6131c9274a29c5dcc03893c9e617
Author: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
Date: Fri Jun 19 11:44:16 2015 -0300
Bug 13993: DBRev 3.21.00.009
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit a712f5b7bad38cb9f5ad599fdb063cbe6e49c547
Author: Jonathan Druart <jonathan.druart at koha-community.org>
Date: Tue May 26 18:10:14 2015 +0200
Bug 13993: Clarify test messages
Signed-off-by: Jonathan Druart <jonathan.druart at koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit a810b1cf59fc0e44ca88b3746b7c0a813703c59c
Author: Amit Gupta <amit.gupta at informaticsglobal.com>
Date: Tue Apr 14 14:07:07 2015 +0530
Bug 13993: (3) Transfer order leaves incorrect orderstatus
11) Apply patch (3)
12) Log in to staff client
13) Acquisitions
14) Create a basket for two different vendors
15) Place an order in one vendor's basket.
16) Transfer the order to the other vendor's basket.
17) prove -v t/db_dependent/Acquisition/TransferOrder.t
-- This should succeed without intervention.
18) Run koha qa test tools for the last 3 commits.
Signed-off-by: Indranil Das Gupta <indradg at gmail.com>
Signed-off-by: Indranil Das Gupta (L2C2 Technologies) <indradg at gmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart at koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit f668a46e23746daa18c5fe8c6005a8661553c645
Author: Mark Tompsett <mtompset at hotmail.com>
Date: Thu Apr 16 09:14:54 2015 -0400
Bug 13993: (2) Correct poorly transferred orders
Added Atomic Update to fix poorly transferred orders
TEST PLAN
---------
8) Apply patch (2)
9) Run the database updates
$ ./installer/data/mysql/updatedatabase.pl
-- This should run without error
10) prove -v t/db_dependent/Acquisition/TransferOrder.t
-- This should fail, because the transfer function is still
not fixed.
Signed-off-by: Indranil Das Gupta (L2C2 Technologies) <indradg at gmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart at koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit b335b7299120f642a0f7d7654937521df8f75ba6
Author: Mark Tompsett <mtompset at hotmail.com>
Date: Thu Apr 16 08:54:30 2015 -0400
Bug 13993: (1) Add tests to confirm 'new' and 'cancelled'
This adds 2 tests to t/db_dependent/Acquisition/TransferOrder.t
in order to confirm the order's status is properly marked.
TEST PLAN
---------
1) Log into staff client
2) Acquisitions
3) Create a basket for two differing vendors.
4) Place an order in one of the baskets.
5) Transfer the order from one vendor's basket to the others.
6) Apply this patch (1) only
7) prove -v t/db_dependent/Acquisition/TransferOrder.t
-- should fail one test: not marked as 'cancelled'.
Signed-off-by: Indranil Das Gupta (L2C2 Technologies) <indradg at gmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart at koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit 2eaeb708795e7624eb8873b617d4a38d69fa84fc
Author: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date: Tue Jun 9 02:32:46 2015 +0200
Bug 14215: Change the 'delimiter' syspref description for its wider use
Patch changes 'report files' to 'CSV files' as there are more
options now for downloading and creating CSV files where this
preference is taken into account.
To test:
- Verify the changed system preference description for
'delimiter' is correct.
Signed-off-by: Marc Véron <veron at veron.ch>
Signed-off-by: Jonathan Druart <jonathan.druart at koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit c76b6a148b85b27ec0d3c4bea1f78ab30d3df72f
Author: Frédéric Demians <f.demians at tamil.fr>
Date: Tue May 26 18:23:11 2015 +0200
Bug 13904: Make unimarc_field_4XX displays usefull 200 subfield data
When searching for multivolumes titles, UNIMARC 4xx field plugin
displays the title of the biblios (200$a), without giving info about
volumes (200 $h $i). It neither doesn't display $e (subtitle) info which
could greatly help to disambiguate search result.
The displayed title is supposed to link to a biblio record view (MARC /
normal). It doesn't work.
TO TEST:
- On a UNIMARC Koha, add a new biblio record
- Call the 4XX plugin from 461/463 field
- Search for a biblio record which contains 200$e, and/or 200$h and/or
200$i subfields.
- You get a result list, with two issues:
1. $a, $h & $i aren't displayed
2. Biblio title is not a link
- Apply the patch, and repeat previous steps.
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
I got a link on fulltitle, but in 'ahie' order (not aehi)
Fixed some tabs.
Signed-off-by: Jonathan Druart <jonathan.druart at koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit be35039b55a351c97f2c1f9a5b373cb26ac5e0b0
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date: Tue Mar 24 17:01:30 2015 +0100
Bug 4137: Fix the OPACViewOthersSuggestions behavior
This pref does not work at all, the interface let the user choose to
list all suggestions, but whatever he chooses the suggestion list is the
same.
This patch cleans a bit the suggestedby management.
There are a lot of cases to test, because linked to 2 prefs:
AnonSuggestions and OPACViewOthersSuggestions.
1/ AnonSuggestions = 0 and OPACViewOthersSuggestions = 0
- A non logged in user is not able to make a suggestion.
- A logged in user is not able to see suggestions made by someone else.
2/ AnonSuggestions = 0 and OPACViewOthersSuggestions = 1
- A non logged in user is not able to make a suggestion.
- A logged in user is able to see suggestions made by someone else.
3/ AnonSuggestions = 1 and OPACViewOthersSuggestions = 0
- A non logged in user is able to make a suggestion.
The suggestedby field will be filled with the AnonymousPatron pref value.
He is not able to see suggestions, even the ones made by AnonymousPatron.
- A logged in user is not able to see suggestions made by someone else.
4/ AnonSuggestions = 1 and OPACViewOthersSuggestions = 1
- A non logged in user is able to make a suggestion.
He is able to see all suggestions.
- A logged in user is able to see suggestions made by someone else.
In all cases a logged in user should be able to search for suggestions
(except if he is not able to see them).
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
All use cases tested, work as expected
No errors
Only comment is perhaps (in the future) a gracefull failure
when AnonymousPatron is not set, or has '0' value
Message is DBIx::Class::ResultSet::create(): Column 'suggestedby' cannot be null at ...
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit d847b1d92a9df6db2bb5321f032f3ec13d6ba55d
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date: Wed Apr 22 12:14:24 2015 +0200
Bug 10866: Hide patron's history if intranetreadinghistory is set to not allow
If set to "not allow", the intranetreadinghistory pref prevent staff
members to access patron's checkout history.
But:
1/ The page is still accessible if you know the url
2/ The history can be consulted on the item history page
Test plan:
0/ Don't apply this patch
1/ Set the intranetreadinghistory to allow
2/ Go on a patron's checkout history page
3/ Open a new tab and go on a item's checkout history page
4/ Set the intranetreadinghistory to not allow
5/ Refresh both pages => no change
6/ Apply this patch
7/ Refresh both page.
On the first page, you should see a warning
On the other one, you should see that the patron column is not displayed
anymore.
Followed test plan, results were as expected.
Signed-off-by: Marc Véron <veron at veron.ch>
http://bugs.koha-community.org/show_bug.cgi?id=10886
Signed-off-by: Katrin Fischer <katrin.fischer at bsz-bw.de>
Nice addition!
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit b740b1b412e11b1d540b243e7b1767cc0c1cb962
Author: Magnus Enger <magnus at libriotech.no>
Date: Wed Jun 17 14:36:44 2015 +0200
Bug 14403: Remove warn in Koha::NorwegianPatronDB
Line 99 has an unconditional warn, left over from development:
warn "$combined_username => $combined_password";
This patch deletes the line i question.
To test:
No testing needed, just have a look at the diff and see that
it makes sense to delete the warn.
Signed-off-by: Indranil Das Gupta (L2C2 Technologies) <indradg at gmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart at koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit 688452ad7e9131a53a96bd826e6228e73494fa53
Author: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date: Mon Jun 8 05:04:56 2015 +0200
Bug 13427: jQuery Timepicker is not translated on returns page
The returns page was missing an include with the translated strings.
To test:
- Install an additional language, like de-DE
- Confirm the bug on the returns page
- Make sure SpecifyReturnDate is activated
- Open the datepicker, look at the time settings
- Apply the patch
- Reinstall the language, no update of the po files is needed
- Retest
- Verify, that now the time settings are translated
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Works as expected
Signed-off-by: Jonathan Druart <jonathan.druart at koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit 7ab873aaea298c787e93438012fa8792345664f4
Author: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date: Mon Jun 8 03:18:35 2015 +0200
Bug 11467: Bug Untranslatable srings in opac-detail.tt (IDreamBooks*, OpacBrowseResults)
Patch marks several strings in the Javascript on the OPAC detail
and result page for translation.
1) IDreamBooks*
- Activate the 3 IDreamBooks* system preferences
- Check the 'cloud' and additional content shows up correctly on
the detail and result pages
- Verify everything works as expected and the same as without the patch
2) OpacBrowseResults
- Activate OpacBrowseResults
- Do various searches
- Verify the nex, previous, browse result list features still
work the same as without the patch
Bonus: Check new strings appear in the .po files by updating one
language with the patch applied (perl translate update de-DE)
NOTE: Really should have read the test plan more closely.
I couldn't find the 'Go to detail:' section, until I clicked
'Browse results'.
Signed-off-by: Mark Tompsett <mtompset at hotmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart at koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit 892d374b64fa4eed98955d75b517702f78f1ca40
Author: Aleisha <aleishaamohia at hotmail.com>
Date: Tue Jun 9 18:20:52 2015 +0000
Bug 11011: Rephrasing 'in keyword' in OPAC authority search
Using 'in the complete record' rather than 'in keyword'. I think this fits well as it seems that this means the search looks anywhere in the record.
To test:
1) In the OPAC, click on Authority Search
2) Notice that in the drop-down menu for the 'Where:' field, there is an 'in keyword' option.
3) Apply patch
4) Now says 'in the complete record'
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
Signed-off-by: Jonathan Druart <jonathan.druart at koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
commit 7c0c92807f49ef61aa975e84cf26d42f7dfa425f
Author: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date: Sun Jun 7 23:45:10 2015 +0200
Bug 8686: Raise required version of URI::Escape to 3.31
Raises the minimum required version of URI::Escape from
1.36 to 3.31.
TEST PLAN
---------
1) git branch -b bug_8686 origin/master
2) ./koha_perl_deps.pl -a | grep URI
-- it will list 1.36 required
3) git bz apply 8686
4) ./koha_perl_deps.pl -a | grep URI
-- it will list 3.31 required
5) koha qa test tools
NOTE: Also default in Ubuntu 14.04 LTS,
not just Wheezy as noted in comment #15.
Signed-off-by: Mark Tompsett <mtompset at hotmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart at koha-community.org>
Signoff based on Nicole's comment (bug 9990 comment 6):
"This stops happening if you upgrade URI::Escape to
3.31. We should make it clear in the Perl Modules page that an upgrade
is needed."
Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
-----------------------------------------------------------------------
Summary of changes:
C4/Acquisition.pm | 4 +-
C4/Auth.pm | 1 -
C4/Installer/PerlDependencies.pm | 2 +-
Koha.pm | 2 +-
Koha/NorwegianPatronDB.pm | 1 -
cataloguing/value_builder/unimarc_field_4XX.pl | 8 +++
installer/data/mysql/updatedatabase.pl | 11 ++++
.../prog/en/modules/admin/preferences/admin.pref | 2 +-
.../prog/en/modules/catalogue/issuehistory.tt | 26 +++++----
.../cataloguing/value_builder/unimarc_field_4XX.tt | 4 +-
.../intranet-tmpl/prog/en/modules/circ/returns.tt | 1 +
.../prog/en/modules/members/readingrec.tt | 4 +-
.../opac-tmpl/bootstrap/en/includes/usermenu.inc | 12 ++---
.../bootstrap/en/modules/opac-authorities-home.tt | 2 +-
.../opac-tmpl/bootstrap/en/modules/opac-detail.tt | 8 +--
.../opac-tmpl/bootstrap/en/modules/opac-results.tt | 4 +-
.../bootstrap/en/modules/opac-suggestions.tt | 57 ++++++++++++--------
opac/opac-suggestions.pl | 46 +++++++++++-----
t/db_dependent/Acquisition/TransferOrder.t | 8 ++-
19 files changed, 131 insertions(+), 72 deletions(-)
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list