[koha-commits] main Koha release repository branch master updated. v3.16.00-304-g39e9b53
Git repo owner
gitmaster at git.koha-community.org
Tue Jul 29 16:08:42 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 39e9b53b8e123060a3dc2913a3b54882b5b5767e (commit)
via 091124b44e363b5b90a664e1e7fe8fcdcfe89781 (commit)
via 98015e19c804c41cb2e81be32b7d8f6c6906580d (commit)
via 812668861dd11de5ead15924047546f70cdd2eb3 (commit)
via b9c34f6453ccf3724e19695f367b68412abcffea (commit)
via 28b4e4feadf0c1ac64dfc60074dd0d4b7258aaea (commit)
via 22b1113b6bc123d710f48eb9f05688e5fd513765 (commit)
via 2326f1c5f1eb1be0320194269416641309cfb4f8 (commit)
via 9db4eb85f3b84f28766e6e6c98d0928477f123d2 (commit)
via 0d5541d1c84cbf350bbc6d502605aeb5e3aa7988 (commit)
via 0e936ce22062910f1a52f6452d2a1b06e46ff001 (commit)
via 9c138fbfb15b2a1a7f4731686241ba116dcf2bc2 (commit)
from 3e201530ed8e56a4f49e66067a71ad82ad2a558e (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 39e9b53b8e123060a3dc2913a3b54882b5b5767e
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date: Sat Jul 26 11:11:18 2014 -0300
Bug 12649: followup to add Croatian into french
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 091124b44e363b5b90a664e1e7fe8fcdcfe89781
Author: Dobrica Pavlinusic <dpavlin at rot13.org>
Date: Thu Jul 24 14:28:31 2014 +0200
Bug 12649: Add Croatian language
Test scenario:
1. apply the patch
2. run updatedatabase.pl
3. install Croatian language
cd misc/translator
perl translate install hr-HR
4. go to Global system preferences > I18N/L10N preferences
check that Hrvatski(hr-HR) is available
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Works, no koha-qa errors.
Also tested translating to de-DE and on advanced search opac/staff
is listed translated.
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Note: This also makes the language show up in the language pull
down in advanced search. Because the entry is the last in the file,
it will show last in the list. I am noting this here, the list
is not really consistently sorted right now and there is work under
way to change the way we handle this.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 98015e19c804c41cb2e81be32b7d8f6c6906580d
Author: Natalie Bennison <natalieb at catalyst.net.nz>
Date: Fri Jul 25 13:58:17 2014 +1200
Bug 12560: Fixing capitalisation of Item Type
To test:
Go to reports>Statistics Wizard>Acquisition
Notice item type has incorrect capitalisation
Apply Patch
Refresh page
Notice it is now fixed
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Work as described
koha-qa complains for a tab on line 220, fixed.
Natali, don't forget to assign the bug to yourself :)
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Small change, no problems found :)
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 812668861dd11de5ead15924047546f70cdd2eb3
Author: Brian Norris <briann at catalyst.net.nz>
Date: Fri Jul 25 14:16:12 2014 +1200
Bug 12562: fix capitalisation of Save Report button
Test:
go to reports and then create report from SQL
the Save button has wrong capitalisaion of Report
add patch
refresh
should see button being Saved report
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Work as described, no koha-qa errors
Brian, don't forget to assign the bug to yourself and set patch complexity :)
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Small template change, no regressions found.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit b9c34f6453ccf3724e19695f367b68412abcffea
Author: Evonne Cheung <evonne at catalyst.net.nz>
Date: Fri Nov 21 12:59:48 2014 +1300
Bug 12563: capitalisation on detail pages item's tab
Test: go to a record in the catalog, go to detail page, switch to 'items'
tab, look for the 2 'Set status' buttons, check capitalisation should be
lower case for second word.
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Works as described, no koha-qa errors
Evonne, assign the patch to yourself :)
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Small template change, no regressions found.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 28b4e4feadf0c1ac64dfc60074dd0d4b7258aaea
Author: Karl Menzies <karl at catalyst.net.nz>
Date: Fri Jul 25 14:27:32 2014 +1200
Bug 12564: Fixing capitalisation buttons in inventory tool
To Test
1/ Do a search in the inventory tool - view capitalisation errors in lower Buttons
2/ Apply patch
3/ Redo search in the inventory tool
4/ Confirm capitalisation errors are fixed
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Work as described, no koha-qa errors
Fixed capitalization (and typo) on patch bug description :)
Karl, don't forget to assign the bug to yourself :)
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Small template change, no regressions found.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 22b1113b6bc123d710f48eb9f05688e5fd513765
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date: Mon Jul 14 10:01:43 2014 -0300
Bug 12570: test for carped warnings in t/Biblio.t
To test:
- Run
$ prove t/Biblio.t
=> Warnings printed to STDOUT
- Apply patch
- Run
$ prove t/Biblio.t
=> No warnings
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
No more warnings. No koha-qa errors
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Works as described.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 2326f1c5f1eb1be0320194269416641309cfb4f8
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date: Sat Jul 19 14:47:00 2014 -0300
Bug 12612: Remove CGI::scrolling_list from auth_tag_structure.pl
This patch removes the only instance in this file.
Don't know if the feature is of some utility, seems
that does not work (no use of auth values as indicators
on MARC authority editor)
To test:
1. Apply the patch
2. Go to Administration > Authority types
3. Clic on MARC structure of any fw
4. Clic Edit on any tag
5. On tag editor check the pulldown 'Authorised value', that was replaced
6. Check for regressions: select a value, save, verify the value, change
again, etc.
Note: removed a SELECT, using now C4::Koha::GetAuthorisedValueCategories
Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Works as expected, no regressions found.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 9db4eb85f3b84f28766e6e6c98d0928477f123d2
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date: Sat Jul 19 16:44:11 2014 -0300
Bug 12613: Remove CGI::scrolling_list from koha2marclinks.pl
This patch removes all (2) instances from koha2marclinks.pl.
Also changes dropdowns width on mapping editor, from an
ugly ragged right to a fixed width view.
To test:
1. Apply the patch
2. Go to Administration > Koha to MARC mapping
3. Pulldown in this view has been replaced, check
for regressions: change table type and press Ok,
verify you get Koha fields and tag/subfield appropriate
for each table.
4. Now press 'Edit' on any field, go into mapping editor
5. Check that all pulldowns show tag/subtags relevant
for each group ( 000s ... 900s )
6. Check that current mapping is displayed correctly
7. Check for regressions: unmap, remap, press Ok,
verify, change again.
8. If you can, check display of mapping editor on several
browsers, just in case.
Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
I have tested with Firefox/Iceweasel only.
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Passes all tests and QA script.
Also tested in Chromium.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 0d5541d1c84cbf350bbc6d502605aeb5e3aa7988
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date: Sat Jul 19 20:58:17 2014 -0300
Bug 12614: Remove CGI::scrolling_list from auth_subfields_structure.pl
This patch removes all instances on this file. Even removes some commented out instances.
Also re-enable editor feature to show/change kohafield value, can't find when or why it was removed.
Up to QA or RM for consideration, simply to remove.
But is strange to have a non editable field on auth frameworks.
To test:
1. Apply the patch
2. Go to Administration > Authority types
3. Clic MARC structure of any auth fw
4. Clic subfield on any tag, e.g. 031
5. Clic 'Edit subfields' button
Now the tests proper
6. 'Help input' box (editor bottom):
Eight(8) scrolling_list removed, they use to
feed the four pulldowns, so check each one
'Koha field' is new (for me at last)
'Authorised value',
'Thesaurus' (is this valid/useful here?)
'Plugin'
There is a difference between existing subtags and a new subtag,
the 'id' of the pulldowns (so the 8 scrolling list removed)
Check assign/remove/save/load values, search for regressions
If you edit subtag 942, it has a value on kohafield (default fw)
I think that this box could be prettified with fixed width for
pulldowns.
7. No more functional tests, there are 4 scrolling_list removed
that are commented (removed also commented code on tt file),
and two instances that are similar to previously removed
instances, evidently with no effect.
Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
Not sure about the new Koha field entry... All works as expected.
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
All works as expected.
About Koha field: I think it makes sense to have it in the form,
so you can configure the fields for new authority types.
In my database I have mappings on authtypecode and authid.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 0e936ce22062910f1a52f6452d2a1b06e46ff001
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date: Tue Jul 22 12:15:35 2014 +0200
Bug 12622: Fix export report as ods
Bug 11679 introduces an ods export for reports.
It looks quite buggy:
1/ You get a Perl error if the report has no results
2/ The ods file contains 1 lines less than the total results.
Test plan:
After applying this patch, try to export a report which has no results
and verify you get an ods file with headers only.
Try to export a report which has a least 1 result and verify the
generated ods has the right number of lines.
Signed-off-by: Robert Higgins <robert.higgins at catalyst.net.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Fixes the reported problem, passes all tests and QA script.
Added the sign-off line for Robert.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 9c138fbfb15b2a1a7f4731686241ba116dcf2bc2
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date: Fri Jul 25 10:59:11 2014 +0200
Bug 12642: t/SMS.t should not be db dependent
SMS.t should not dependent on the DB.
Test plan:
verify that prove t/SMS.t returns green even if the DB is not there.
Signed-off-by: Robin Sheat <robin at catalyst.net.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Turned off MySQL, tests still pass.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
-----------------------------------------------------------------------
Summary of changes:
admin/auth_subfields_structure.pl | 186 ++++++--------------
admin/auth_tag_structure.pl | 21 +--
admin/koha2marclinks.pl | 28 +--
.../data/mysql/de-DE/mandatory/subtag_registry.sql | 19 ++
.../data/mysql/en/mandatory/subtag_registry.sql | 19 ++
.../data/mysql/es-ES/mandatory/subtag_registry.sql | 20 +++
.../mysql/fr-FR/1-Obligatoire/subtag_registry.sql | 17 ++
.../data/mysql/it-IT/necessari/subtag_registry.sql | 20 +++
.../mysql/nb-NO/1-Obligatorisk/subtag_registry.sql | 20 +++
.../data/mysql/pl-PL/mandatory/subtag_registry.sql | 20 +++
.../data/mysql/ru-RU/mandatory/subtag_registry.sql | 20 +++
.../data/mysql/uk-UA/mandatory/subtag_registry.sql | 20 +++
installer/data/mysql/updatedatabase.pl | 12 ++
.../en/modules/admin/auth_subfields_structure.tt | 53 +++++-
.../prog/en/modules/admin/auth_tag_structure.tt | 18 +-
.../prog/en/modules/admin/koha2marclinks.tt | 137 ++++----------
.../prog/en/modules/catalogue/moredetail.tt | 4 +-
.../prog/en/modules/reports/acquisitions_stats.tt | 2 +-
.../en/modules/reports/guided_reports_start.tt | 4 +-
.../prog/en/modules/tools/inventory.tt | 4 +-
reports/guided_reports.pl | 11 +-
t/Biblio.t | 126 ++++++++++---
t/SMS.t | 2 +
23 files changed, 469 insertions(+), 314 deletions(-)
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list