From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 03:28:04 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 02:28:04 +0000
Subject: [Koha-bugs] [Bug 9021] Add SMS via email as an alternative to SMS
services via SMS::Send drivers
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9021
--- Comment #53 from M. Tompsett ---
Also, make sure there are SMS letters (notices and slips).
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 03:43:41 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 02:43:41 +0000
Subject: [Koha-bugs] [Bug 15135] Remove Warning Subroutine HasOverdues
redefined
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15135
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 03:44:34 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 02:44:34 +0000
Subject: [Koha-bugs] [Bug 15135] Remove Warning Subroutine HasOverdues
redefined
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15135
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46083|0 |1
is obsolete| |
--- Comment #10 from Chris Cormack ---
Created attachment 46166
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46166&action=edit
Bug 15135: Get rid of Warning 'Subroutine HasOverdues redefined'
This patch renames the subroutine HasOverdues to WithOverdues in
Koha/Template/Plugin/Borrowers.pm and in members-toolbar.inc (only
place where it is used).
To test:
- First test without patch
- Search for a patron with overdues (I used the 'Overdues' report
on page Home > Cirulation
- Display detail page, open drop down tool 'Print'. Verify that you have
an item 'Print overdues'
- Apply patch
- Repeat steps above. Verify that the item 'Print overdues' still
appears
- Search for remaining occurences of HasOverdues in template files / template
includes with git grep 'Patrons.HasOverdues'. There should not be any
occurences.
Signed-off-by: Chris
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 03:55:41 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 02:55:41 +0000
Subject: [Koha-bugs] [Bug 14133] Print notices generated in special case do
not use print template
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14133
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 03:56:22 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 02:56:22 +0000
Subject: [Koha-bugs] [Bug 14133] Print notices generated in special case do
not use print template
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14133
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46085|0 |1
is obsolete| |
--- Comment #8 from Chris Cormack ---
Created attachment 46167
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46167&action=edit
Bug 14133: Print notices should be generated with the print template
If the overdue notice to send is an email or a SMS and the patron does
not provided such information, a print notice is generated.
But it will be generated with the email or sms template, which is not
useful, especially in the SMS case.
The template to use should be the print one.
Test plan:
0/ Does not apply this patch and Correctly define the overdue rules to generate
an overdue notice to the message_queue table.
Check email + SMS
1/ Define an email address and a SMS number for a patron
2/ Generate the overdue notices (misc/cronjobs/overdue_notices.pl [-t])
3/ 2 notices should have been generated:
mtt=email with the email template used
mtt=sms with the sms template used
4/ Truncate the message_queue table
5/ Remove the email address and repeat 2
6/ Only 1 notice should have been generated:
mtt=print with the email template used
7/ Truncate the message_queue table
8/ Remove the sms alert number and repeat 2
9/ 2 notices should have been generated:
mtt=print with the email template used (or sms I suppose)
10/ Apply this patch and repeat previous steps.
The print notices should be generated with the print template.
IMPORTANT NOTE: This test plan does not take into account the notices
generated for the staff ("These messages were not sent directly to the
patrons."). However the behavior will also change, the print template
will be used in all cases. Is it what we want?
Signed-off-by: Chris
Seems fine by me
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 06:19:58 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 05:19:58 +0000
Subject: [Koha-bugs] [Bug 9021] Add SMS via email as an alternative to SMS
services via SMS::Send drivers
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9021
--- Comment #54 from M. Tompsett ---
HA HA! Email configured well enough. Patch applies cleanly. I believe it tests
well now. I'm signing off both, but since I wrote the Schema patch, I'd like
another sign off.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 06:35:54 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 05:35:54 +0000
Subject: [Koha-bugs] [Bug 9021] Add SMS via email as an alternative to SMS
services via SMS::Send drivers
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9021
--- Comment #55 from M. Tompsett ---
Created attachment 46168
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46168&action=edit
Bug 9021: Flag issues
Check in and check out were failing for me.
Specificly, the $borrower->{flags}->{...} was not accessible as
a hash, so I put a hash ref check around the code that would fail.
TEST PLAN
---------
1) Attempt a checkout
-- blows up with "1" not being allowed as a hash ref.
2) Apply patch
3) Attempt same checkout again
-- success
4) prove -v t/db_dependent/Circulation_dateexpiry.t
-- this triggers CanBookBeIssued
-- this should succeed
5) prove -v t/db_dependent/rollingloans.t
-- mine skipped the tests, but if configured, it
should also trigger and succeed.
6) run koha qa test tools
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 06:47:12 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 05:47:12 +0000
Subject: [Koha-bugs] [Bug 9021] Add SMS via email as an alternative to SMS
services via SMS::Send drivers
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9021
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46163|0 |1
is obsolete| |
--- Comment #56 from M. Tompsett ---
Created attachment 46169
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46169&action=edit
Bug 9021 - Add SMS via email as an alternative to SMS services via SMS::Send
drivers
Nearly all cellular providers allow a person to send an text message to a
cellular
phone by sending an email to phonenumber at provider. We can leverage this
capability
to add the ability for Koha to send sms messages to patrons without the need to
subscribe to an sms gateway server.
Basic plan:
1. Add a table sms_providers to the db to tell Koha what service providers are
available, and what domain emails should be sent to.
2. Add borrowers.sms_provider_id to tell Koha which mobile service the patron
subscribes to for the number given in smsalertnumber
3. Modify Koha to send an email rather than using SMS::Send if the driver is
set to 'Email'
Test plan:
0) Get a mobile phone
1) Apply the patch
2) Run updatedatabase.pl
3) Set the value of SMSSendDriver to 'Email'
4) Go to the admin page, the "Additional parameters" area should now have the
link "SMS cellular providers"
5) On this page, add some providers. Make sure to add the provider for your own
cellular phone service.
Here are some examples:
Sprint phonenumber at messaging.sprintpcs.com
Verizon phonenumber at vtext.com
T-Mobile phonenumber at tmomail.net
AT&T phonenumber at txt.att.net
Only add the domain part in the 'domain' field. So for Verizon, that would be
'vtext.com'
6) Create an account for yourself, add your SMS number, and select your
provider from the dropdown box directly below it.
7) Enable SMS messaging for Item check-in and Item checkout
8) Check out an item to yourself
9) Run process_message_queue.pl
10) Wait! You should receive a text message shortly, when I tested it, I
received my sms message within the minute.
Signed-off-by: Mark Tompsett
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 06:47:20 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 05:47:20 +0000
Subject: [Koha-bugs] [Bug 9021] Add SMS via email as an alternative to SMS
services via SMS::Send drivers
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9021
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46165|0 |1
is obsolete| |
--- Comment #57 from M. Tompsett ---
Created attachment 46170
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46170&action=edit
Bug 9021: Schema update
This ensures that the DBIx structure matches the SQL structure changes.
See comment #48 as to how this was generated.
Also, I had to modify misc/devel/update_dbix_class_files.pl
to have an "overwrite_modifications => 1" option to get it to generate.
This should correct issues raised in comment #45.
Signed-off-by: Mark Tompsett
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 06:47:32 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 05:47:32 +0000
Subject: [Koha-bugs] [Bug 9021] Add SMS via email as an alternative to SMS
services via SMS::Send drivers
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9021
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46168|0 |1
is obsolete| |
--- Comment #58 from M. Tompsett ---
Created attachment 46171
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46171&action=edit
Bug 9021: Flag issues
Check in and check out were failing for me.
Specificly, the $borrower->{flags}->{...} was not accessible as
a hash, so I put a hash ref check around the code that would fail.
TEST PLAN
---------
1) Attempt a checkout
-- blows up with "1" not being allowed as a hash ref.
2) Apply patch
3) Attempt same checkout again
-- success
4) prove -v t/db_dependent/Circulation_dateexpiry.t
-- this triggers CanBookBeIssued
-- this should succeed
5) prove -v t/db_dependent/rollingloans.t
-- mine skipped the tests, but if configured, it
should also trigger and succeed.
6) run koha qa test tools
Signed-off-by: Mark Tompsett
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 06:50:33 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 05:50:33 +0000
Subject: [Koha-bugs] [Bug 9021] Add SMS via email as an alternative to SMS
services via SMS::Send drivers
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9021
--- Comment #59 from M. Tompsett ---
For Canadians using Rogers:
Rogers phonenumber at pcs.rogers.com
See step 5 in comment #4 for wider context.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 07:27:20 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 06:27:20 +0000
Subject: [Koha-bugs] [Bug 15445] DateUtils.t fails on Jenkins due to server
sluggishness
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15445
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 07:27:24 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 06:27:24 +0000
Subject: [Koha-bugs] [Bug 15445] DateUtils.t fails on Jenkins due to server
sluggishness
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15445
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46089|0 |1
is obsolete| |
--- Comment #2 from M. Tompsett ---
Created attachment 46172
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46172&action=edit
Bug 15445 DateUtils.t fails on Jenkins due to server sluggishness
Jenkins server is abnormally slow to perform a task done in less than
1s elsewhere. The test stay valid by giving just a bit more time to Jenkins
server.
TO TEST:
- Confirm that the two test hasn't changed. They the just get 4s to
perform
- Push the patch to master and observe that DateUtils.t doesn't fail
anymore.
NOTE: Also passed koha qa test tools too.
Signed-off-by: Mark Tompsett
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 07:38:31 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 06:38:31 +0000
Subject: [Koha-bugs] [Bug 15283] Switch default ISSUEQSLIP notice to
Template Toolkit
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15283
--- Comment #4 from M. Tompsett ---
Comment on attachment 45336
--> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=45336
Bug 15283 - Switch default ISSUEQSLIP notice to Template Toolkit
Review of attachment 45336:
--> (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=15283&attachment=45336)
-----------------------------------------------------------------
The sample_notices.sql files for non-English files could have been kept closer
to the original in terms of language, rather than changing them to English.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 07:40:17 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 06:40:17 +0000
Subject: [Koha-bugs] [Bug 15283] Switch default ISSUEQSLIP notice to
Template Toolkit
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15283
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mtompset at hotmail.com
--- Comment #5 from M. Tompsett ---
New files could have used the license text found on the wiki.
http://wiki.koha-community.org/wiki/Coding_Guidelines#Licence
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 07:40:59 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 06:40:59 +0000
Subject: [Koha-bugs] [Bug 15283] Switch default ISSUEQSLIP notice to
Template Toolkit
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15283
--- Comment #6 from M. Tompsett ---
You added a Koha level library without any test coverage.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 12:42:06 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 11:42:06 +0000
Subject: [Koha-bugs] [Bug 15344] GetMemberDetails called unecessary
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15344
--- Comment #16 from Marc V?ron ---
(In reply to Fr?d?ric Demians from comment #15)
> This patch has been pushed to 3.20.x, will be in 3.20.8.
Was it pushed without the third patch? - That would be a blocker, because I
still get the error from comment #13 (on master), and it seems that the third
patch that fixes it is not on master.
The offending line is #273 in circ/circulation.pl, we need the member details
here (for useres with staff permission), otherwise we get the software error:
Can't use string ("1") as a HASH ref while "strict refs" in use at
/usr/share/kohaclone/C4/Circulation.pm line 815.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 13:45:01 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 12:45:01 +0000
Subject: [Koha-bugs] [Bug 15344] GetMemberDetails called unecessary
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15344
--- Comment #17 from Fr?d?ric Demians ---
Marc and Johnatan, thanks. I confirm 3rd patch is required. I've pushed it
directly to 3.20.x branch. It still has to be pushed to master/stable.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 14:12:20 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 13:12:20 +0000
Subject: [Koha-bugs] [Bug 14555] Warns in opac-search.pl
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14555
Martin Stenberg changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #42718|0 |1
is obsolete| |
--- Comment #7 from Martin Stenberg ---
Created attachment 46173
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46173&action=edit
Bug 14555: Warns in opac-search.pl
Avoid use of uninitialized value in split
== Test plan ==
1. Go to opac advanced search
2. Click "Search" without any search query
3. Log file will show "Use of uninitialized value in split at ..."
4. Apply this patch
5. Repeat step 1 and 2
6. Log file should no longer show "Use of uninitialized value in split at ..."
Sponsored-by: Regionbibliotek Halland / County library of Halland
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Jan 1 15:18:30 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 14:18:30 +0000
Subject: [Koha-bugs] [Bug 14224] patron notes about item shown at check in
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14224
--- Comment #4 from Martin Stenberg ---
Hi Jonathan,
Do you think it's worth adding a new subroutine GetIssue for this special case
of GetIssues? GetIssues will still return an arrayref when called with
issue_id, so the return type matches the plural "s" at least.
I'm doing an ajax request to minimize overhead when sending the request (i.e.
no need to send note data for all issues when only one has been changed) and
when recieving (i.e. no need to render a new template and send a whole document
when only an "ok" or "nok" is needed). And as a bonus it allows the user to
write a second note while the first one is beeing sent/processed to the server.
To be honest I thought javascript was concidered a requirment for Koha. I'm
using NoScript myself, and am always positivly surprised when a page works
without having to enable javascript.
How about keeping the ajax-version and adding a fallback for browsers without
javascript?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 00:31:38 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 23:31:38 +0000
Subject: [Koha-bugs] [Bug 15454] Bad spanish translation
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15454
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |bgkriegel at gmail.com,
| |katrin.fischer at bsz-bw.de
--- Comment #1 from Katrin Fischer ---
Hi :)
we handle translations via translate.koha-community.org - if you register there
you can suggest the correct translation or even submit it directly (depending
on permissions)
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 00:34:16 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Jan 2016 23:34:16 +0000
Subject: [Koha-bugs] [Bug 15383] Opac: Authority details: Fix translation
issues with tags
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15383
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |katrin.fischer at bsz-bw.de
Summary|Opac: Autority details: Fix |Opac: Authority details:
|translation issues with a |Fix translation issues with
|tags |tags
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 04:29:52 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 03:29:52 +0000
Subject: [Koha-bugs] [Bug 14480] Warns when modifying patron
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14480
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mtompset at hotmail.com
--- Comment #12 from M. Tompsett ---
(In reply to Jonathan Druart from comment #11)
> Anyway, what I have proposed previously was the wrong test indeed, it should
> be
> [% UNLESS debarments %]
Or clearer, swap the IF/ELSE.
You know you are generating the table [% IF (debarments) %], [% ELSE %] you are
putting out that other message.
The advantage of Jonathan's suggestion is smaller patch size. The disadvantage
is that the novice programmer does not usually think in negation logic.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 06:18:01 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 05:18:01 +0000
Subject: [Koha-bugs] [Bug 14480] Warns when modifying patron
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14480
--- Comment #13 from M. Tompsett ---
(In reply to Aleisha Amohia from comment #9)
> Sorry, I made the change to [% IF ( debarments && debarments.size < 1 ) %]
Actually, if you wanted to use an UGLY condition.
[% IF ((! debarments) || (debarments && debarments.size<1)) %]
But let's not go there. :)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 09:54:59 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 08:54:59 +0000
Subject: [Koha-bugs] [Bug 11144] Fix sequence of cronjobs: automatic renewal
- fines - overdue notices
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11144
--- Comment #14 from M. Tompsett ---
Comment on attachment 44283
--> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44283
Bug 11144: Fix sequence of cronjobs: automatic renewal - fines - overdue
notices
Review of attachment 44283:
--> (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=11144&attachment=44283)
-----------------------------------------------------------------
::: misc/cronjobs/crontab.example
@@ +48,4 @@
>
> +# AUTOMATIC RENEWAL + FINES + OVERDUE NOTICES
> +# Sequence is important - renew what can be renewed, charge fines for the rest, print fine amount into overdue notices
> +5 1 * * * __KOHA_USER__ $KOHA_CRON_PATH/automatic_renewals.pl && $KOHA_CRON_PATH/fines.pl && $KOHA_CRON_PATH/overdue_notices.pl -t
I'm pretty sure you want semi-colons. You want to run them sequentially in this
order regardless of the success or failure of script.
echo one; false; echo three
echo one; true; echo three
echo one && false && echo three
echo one && true && echo three
echo one || false || echo three
echo one || true || echo three
Only the first two print one and three for both false and true cases
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 11:00:10 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 10:00:10 +0000
Subject: [Koha-bugs] [Bug 15375] Translatability: Fix issues on OPAC page
'Placing a hold'
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15375
--- Comment #7 from Marc V?ron ---
Still applies on current master.
Note: This patch is about string changes only, it does not change the
functionallity.
If you find an issue or want to suggest an enhancement regarding
functionallity, please file a separate bug.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 11:14:30 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 10:14:30 +0000
Subject: [Koha-bugs] [Bug 14608] HEA : add possibility of sharing usage
statistics in Administration page and Web installer
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14608
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |veron at veron.ch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 11:49:49 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 10:49:49 +0000
Subject: [Koha-bugs] [Bug 15457] New: HEA Display of system preferences cut
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15457
Bug ID: 15457
Summary: HEA Display of system preferences cut
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Websites, Mailing Lists, etc
Assignee: wizzyrea at gmail.com
Reporter: veron at veron.ch
QA Contact: testopia at bugs.koha-community.org
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 11:57:16 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 10:57:16 +0000
Subject: [Koha-bugs] [Bug 15457] HEA Display of system preferences cut
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15457
--- Comment #1 from Marc V?ron ---
Created attachment 46174
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46174&action=edit
HEA: Charts cut for libraries
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 12:01:20 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 11:01:20 +0000
Subject: [Koha-bugs] [Bug 15457] HEA Display of system preferences cut
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15457
--- Comment #2 from Marc V?ron ---
Created attachment 46175
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46175&action=edit
HEA: Charts cut for system preferences
On the Koha community statistics website (http://hea.koha-community.org/), the
charts are cut on small screens.
See screenshots.
A quick solution in hea.css would be to set .chart to a width of 100% (instead
of 48%).
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 12:03:55 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 11:03:55 +0000
Subject: [Koha-bugs] [Bug 15457] HEA : Display of charts is cut on Koha
community statistics website
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15457
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|HEA Display of system |HEA : Display of charts is
|preferences cut |cut on Koha community
| |statistics website
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 12:12:39 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 11:12:39 +0000
Subject: [Koha-bugs] [Bug 15457] HEA : Display of charts is cut on Koha
community statistics website
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15457
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|enhancement |trivial
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 12:19:08 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 11:19:08 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
--- Comment #24 from Marc V?ron ---
Does not apply:
CONFLICT (content): Merge conflict in t/db_dependent/UsageStats.t
Failed to merge in the changes.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 12:21:47 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 11:21:47 +0000
Subject: [Koha-bugs] [Bug 14608] HEA : add possibility of sharing usage
statistics in Administration page and Web installer
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14608
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Patch doesn't apply
--- Comment #9 from Marc V?ron ---
CONFLICT (content): Merge conflict in
koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/admin.pref
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 15:11:01 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 14:11:01 +0000
Subject: [Koha-bugs] [Bug 14060] Remove readonly on date inputs
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14060
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Failed QA
--- Comment #16 from Marc V?ron ---
Hi Jonathan,
Sorry, I found following issues:
1)
Syspref dateformat = dmydot (01.10.2017) is not covered by the function
is_valid_date
Symptom 1:
- Set syspref dateformat to dmydot, clear browser chache
- Try to add a manual restriction on moremember.pl
- manually enter 26.01.2017
- hit 'Add restriction'
Result: restriction is added with empty date = indefinite restriction.
Symptom 2:
- manually enter 1.1.2018
- hit enter
- Message appears "Please enter valid date...", closes automatically after
ca. 1 second
Result: restriction is added with empty date = indefinite restriction.
2)
Software error in following cases
Case 1:
- Set syspref dateformat to dd/mm/yyyy, clear browser cache
- manually enter 1/10/2017
- hit 'Add restriction'
Result: Software error: The given date (1/10/2017) does not match the date
format (metric) at /usr/share/kohaclone/Koha/DateUtils.pm line 153.
Case 2:
- Set syspref to yyyy-mm-dd
- manually enter 2017-2-2
- hit 'Add Restiction'
Result: Software error: The given date (2018-2-2) does not match the date
format (iso) at /usr/share/kohaclone/Koha/DateUtils.pm line 153.
It seems that the timepicker's function parseDate accepts such dates.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 17:38:39 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 16:38:39 +0000
Subject: [Koha-bugs] [Bug 3317] Author displays inconsistently ( and
incorrectly ) in opac-basket.pl (and when emailing cart )
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3317
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |veron at veron.ch
--- Comment #2 from Marc V?ron ---
Is this bug still valid?
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 17:39:28 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 16:39:28 +0000
Subject: [Koha-bugs] [Bug 2710] save buttons should be at both top and
bottom of certain forms
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2710
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |veron at veron.ch
--- Comment #4 from Marc V?ron ---
Is this bug still valid?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 18:06:58 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 17:06:58 +0000
Subject: [Koha-bugs] [Bug 5512] Display Published Date with Received Date
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5512
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |veron at veron.ch
--- Comment #1 from Marc V?ron ---
Is this bug still valid?
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 18:13:19 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 17:13:19 +0000
Subject: [Koha-bugs] [Bug 6847] Bulk userid, password generation and email
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6847
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |veron at veron.ch
--- Comment #1 from Marc V?ron ---
Is this bug still valid?
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 18:22:59 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 17:22:59 +0000
Subject: [Koha-bugs] [Bug 11081] Port Koha::Contrib::Tamil indexer into Koha
code base
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11081
Fr?d?ric Demians changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Passed QA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 18:23:03 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 17:23:03 +0000
Subject: [Koha-bugs] [Bug 11081] Port Koha::Contrib::Tamil indexer into Koha
code base
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11081
Fr?d?ric Demians changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #39419|0 |1
is obsolete| |
--- Comment #16 from Fr?d?ric Demians ---
Created attachment 46176
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46176&action=edit
Bug 11081: Port Koha::Contrib::Tamil indexer into Koha code base
Add two scripts for indexing: koha-index and koha-index-daemon.
Documentation:
perldoc koha-index
perldoc koha-index-daemon
New dependencies:
MooseX::Getopt
MooseX::RW
AnyEvent::Processor
Available as a Debian package: libmoosex-getopt-perl
Test plan:
- Apply the patch
- Install dependencies, if necessary
- Reindex a whole biblio catalog:
koha-index --verbose
- Reindex a whole authorities catalog:
koha-index --source authority --verbose
- Modify 1 biblio record, and index it:
koha-index --select queue
- Modifiy 1 authority record, and index it:
koha-index --select queue --source authority
- Run indexing daemon, with a 15s timeout:
koha-index-daemon --timeout 15
- Modify a biblio record: wait 15s, and see
- Modify an authority record: wait 15s, and see
Signed-off-by: Bernardo Gonzalez Kriegel
Works well, followed test plan without problems
One comment: perhaps an option for koha-index-daemon
to work quietly, without printing information.
No koha-qa errors, but new files don't have license,
fixed in followup
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 18:23:25 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 17:23:25 +0000
Subject: [Koha-bugs] [Bug 11081] Port Koha::Contrib::Tamil indexer into Koha
code base
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11081
Fr?d?ric Demians changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #39420|0 |1
is obsolete| |
--- Comment #17 from Fr?d?ric Demians ---
Created attachment 46177
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46177&action=edit
Bug 11081: (followup) Add license information
Signed-off-by: Fr?d?ric Demians
Signed-off-by: Kyle M Hall
Signed-off-by: Fr?d?ric Demians
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Jan 2 18:24:14 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 02 Jan 2016 17:24:14 +0000
Subject: [Koha-bugs] [Bug 11081] Port Koha::Contrib::Tamil indexer into Koha
code base
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11081
--- Comment #18 from Fr?d?ric Demians ---
Rebased, and revert to Passed QA status.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 11:27:57 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 10:27:57 +0000
Subject: [Koha-bugs] [Bug 15383] Opac: Authority details: Fix translation
issues with tags
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15383
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Pushed to Master |Pushed to Stable
CC| |julian.maurice at biblibre.com
--- Comment #8 from Julian Maurice ---
Patches pushed to 3.22.x, will be in 3.22.2
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 11:29:10 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 10:29:10 +0000
Subject: [Koha-bugs] [Bug 15447] log4perl.conf does not have __LOG_DIR__
replaced when installing
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15447
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Pushed to Master |Pushed to Stable
CC| |julian.maurice at biblibre.com
--- Comment #5 from Julian Maurice ---
Patch pushed to 3.22.x, will be in 3.22.2
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 11:35:25 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 10:35:25 +0000
Subject: [Koha-bugs] [Bug 14397] Typo 'foriegn' in table comments
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14397
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|enhancement |trivial
CC| |julian.maurice at biblibre.com
Status|Pushed to Master |Pushed to Stable
--- Comment #6 from Julian Maurice ---
Patch pushed to 3.22.x, will be in 3.22.2
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 11:54:02 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 10:54:02 +0000
Subject: [Koha-bugs] [Bug 14034] User logged out on refresh after Shibboleth
authentication
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14034
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |julian.maurice at biblibre.com
Status|Pushed to Master |Pushed to Stable
--- Comment #7 from Julian Maurice ---
Patch pushed to 3.22.x, will be in 3.22.2
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 12:16:26 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 11:16:26 +0000
Subject: [Koha-bugs] [Bug 15170] Add 264 field to MARC21*DC.xsl
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15170
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|enhancement |normal
--- Comment #13 from Julian Maurice ---
(In reply to Katrin Fischer from comment #12)
> Hi Julian, I think this is in a grey area - for libraries using RDA it means
> that the publication year, publisher etc. is missing from the export formats
> - which is quite important information.
Sorry, I didn't see comment 8.
So, switching to 'normal' severity again, and I will push it into 3.22.x
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 12:17:04 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 11:17:04 +0000
Subject: [Koha-bugs] [Bug 15170] Add 264 field to MARC21*DC.xsl
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15170
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Pushed to Master |Pushed to Stable
--- Comment #14 from Julian Maurice ---
Patch pushed to 3.22.x, will be in 3.22.2
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 12:21:56 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 11:21:56 +0000
Subject: [Koha-bugs] [Bug 15316] Use Bootstrap modal for authority Z39.50
search results preview
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15316
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |julian.maurice at biblibre.com
Severity|minor |enhancement
--- Comment #9 from Julian Maurice ---
Switching severity to 'enhancement' as the patch does not fix any bug
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 12:22:15 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 11:22:15 +0000
Subject: [Koha-bugs] [Bug 15317] Use Bootstrap modal for MARC and Card
preview when ordering from staged files
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15317
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |julian.maurice at biblibre.com
Severity|minor |enhancement
--- Comment #7 from Julian Maurice ---
Switching severity to 'enhancement' as the patch does not fix any bug
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 12:22:33 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 11:22:33 +0000
Subject: [Koha-bugs] [Bug 15318] Use Bootstrap modal for MARC and Card
preview when ordering an external source
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15318
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|minor |enhancement
CC| |julian.maurice at biblibre.com
--- Comment #6 from Julian Maurice ---
Switching severity to 'enhancement' as the patch does not fix any bug
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 12:22:59 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 11:22:59 +0000
Subject: [Koha-bugs] [Bug 15320] Use Bootstrap modal for MARC preview when
ordering from an existing record
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15320
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |julian.maurice at biblibre.com
Severity|minor |enhancement
--- Comment #6 from Julian Maurice ---
Switching severity to 'enhancement' as the patch does not fix any bug
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 15:53:22 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 14:53:22 +0000
Subject: [Koha-bugs] [Bug 6845] Add default date (today) to view log
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6845
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |veron at veron.ch
--- Comment #1 from Marc V?ron ---
Is this bug still valid?
--
You are receiving this mail because:
You are watching all bug changes.
You are the QA Contact for the bug.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 21:23:17 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 20:23:17 +0000
Subject: [Koha-bugs] [Bug 15170] Add 264 field to MARC21*DC.xsl
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15170
--- Comment #15 from Katrin Fischer ---
Thx Julian!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 21:24:21 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 20:24:21 +0000
Subject: [Koha-bugs] [Bug 11081] Port Koha::Contrib::Tamil indexer into Koha
code base
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11081
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |gmcharlt at gmail.com,
| |katrin.fischer at bsz-bw.de
Keywords| |dependency
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 21:35:27 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 20:35:27 +0000
Subject: [Koha-bugs] [Bug 7143] Bug for tracking changes to the about page
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7143
--- Comment #286 from Chris Cormack ---
Created attachment 46178
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46178&action=edit
Bug 7143 : Adding releases and fixing some missing tabs
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 21:35:48 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 20:35:48 +0000
Subject: [Koha-bugs] [Bug 7143] Bug for tracking changes to the about page
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7143
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Pushed to Master |ASSIGNED
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Sun Jan 3 21:36:01 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 03 Jan 2016 20:36:01 +0000
Subject: [Koha-bugs] [Bug 7143] Bug for tracking changes to the about page
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7143
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 02:35:16 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 01:35:16 +0000
Subject: [Koha-bugs] [Bug 15434] Make search results breadcrumbs human
readable
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15434
--- Comment #8 from Srdjan Jankovic ---
As far as I can see it is a bug either in
* modules/svc/suggestion.tt, line 37, that composes query as q=[indexed query],
or in
* C4::Search::parseQuery() that cannot deal with that kind of query
I think that buildQuery() actually builds an invalid query, but I'd certainly
need an authoritative opinion.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 03:48:31 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 02:48:31 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |chris at bigballofwax.co.nz
Status|Needs Signoff |Patch doesn't apply
--- Comment #43 from Chris Cormack ---
Falling back to patching base and 3-way merge...
Auto-merging reports/issues_stats.pl
CONFLICT (content): Merge conflict in reports/issues_stats.pl
Failed to merge in the changes.
Not a simple conflict
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 04:00:23 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 03:00:23 +0000
Subject: [Koha-bugs] [Bug 11315] Add support for subfield deletion when
doing authority merges.
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11315
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |chris at bigballofwax.co.nz
Status|Needs Signoff |Patch doesn't apply
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 04:08:18 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 03:08:18 +0000
Subject: [Koha-bugs] [Bug 13572] Add onlyactives parameter to
force_borrower_messaging_defaults script
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13572
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 04:08:21 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 03:08:21 +0000
Subject: [Koha-bugs] [Bug 13572] Add onlyactives parameter to
force_borrower_messaging_defaults script
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13572
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #35164|0 |1
is obsolete| |
--- Comment #5 from Chris Cormack ---
Created attachment 46179
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46179&action=edit
Bug 13572 - Add onlyactives parameter to force_borrower_messaging_defaults
script
1) Apply both patches
2) Create a new patron with random values, except for it expiration date, make
it expired (Patrons > New Patron > Student)
3) Enable the system preference called ?EnhancedMessagingPreferences?
4) In ?Administration" > "Patron categories" > Student, modify the "days in
advance", then click "Save"
5) run the script "./misc/maintenance/borrowers-force-messaging-defaults --doit
--onlyactives"
6) Validate that the student created in step 2 hasn't changed (Patrons >
search)
7) Validate that any other student that isn't expired has changed.
Signed-off-by: Chris
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 04:08:48 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 03:08:48 +0000
Subject: [Koha-bugs] [Bug 13572] Add onlyactives parameter to
force_borrower_messaging_defaults script
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13572
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #41235|0 |1
is obsolete| |
--- Comment #6 from Chris Cormack ---
Created attachment 46180
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46180&action=edit
Bug13572 - Changed "borrowers-force-messaging-defaults" --help parameter's
result to better explain the new functionality
New test plan (well, a clearer version):
1) Apply both patches
2) Create a new patron with random values, except for it expiration date, make
it expired (Patrons > New Patron > Student)
3) Enable the system preference called ?EnhancedMessagingPreferences?
4) In ?Administration" > "Patron categories" > Student, modify the "days in
advance", then click "Save"
5) run the script "./misc/maintenance/borrowers-force-messaging-defaults --doit
--onlyactives"
6) Validate that the student created in step 2 hasn't changed (Patrons >
search)
7) Validate that any other student that isn't expired has changed.
Signed-off-by: Chris
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 04:18:15 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 03:18:15 +0000
Subject: [Koha-bugs] [Bug 9316] Add Nginx install options with plack
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9316
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |In Discussion
--- Comment #56 from Chris Cormack ---
With the plack in production as of 3.22.0 this bug needs a rethink, the patches
as they stand will duplicate a lot of stuff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 04:37:20 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 03:37:20 +0000
Subject: [Koha-bugs] [Bug 14751] Allow C4::Context->interface to be set to
'sip' or 'commandline'
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14751
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 04:37:25 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 03:37:25 +0000
Subject: [Koha-bugs] [Bug 14751] Allow C4::Context->interface to be set to
'sip' or 'commandline'
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14751
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #42061|0 |1
is obsolete| |
--- Comment #2 from Chris Cormack ---
Created attachment 46181
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46181&action=edit
Bug 14751 - Allow C4::Context->interface to be set to 'sip' or 'commandline'
Koha::Logger allows us to split up logging to separate files based on
the interface. However, right now the interface is limited to just
'opac' or 'intranet'.
We should extend this to allow the interface to also be set to 'sip'
for SIP server logging, and to 'commandline' for cronjobs and all other
command line scripts.
Signed-off-by: Chris
Does what it says on the tin
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 04:54:40 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 03:54:40 +0000
Subject: [Koha-bugs] [Bug 11807] Add categorycode conversions to LDAP
authentication.
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11807
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #25526|0 |1
is obsolete| |
--- Comment #6 from Chris Cormack ---
Created attachment 46182
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46182&action=edit
Bug 11807 : Add support for categorycode conversions when updating an user
using a LDAP server.
To test
1/ Apply both patches
2/ This patch lets you easily configure mappings for categorycode values.
These mapping will be used when updating the user's account after a successful
LDAP login.
Here is an example configuration :
STU
...
STUDENT
EMPLOYEE
3/ With this configuration, LDAP users with the usertype value "EMP" on the
LDAP server should have the "EMPLOYEE" categorycode in Koha.
Signed-off-by: Chris
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 04:57:22 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 03:57:22 +0000
Subject: [Koha-bugs] [Bug 11807] Add categorycode conversions to LDAP
authentication.
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11807
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 04:57:26 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 03:57:26 +0000
Subject: [Koha-bugs] [Bug 11807] Add categorycode conversions to LDAP
authentication.
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11807
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #42105|0 |1
is obsolete| |
--- Comment #7 from Chris Cormack ---
Created attachment 46183
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46183&action=edit
Bug 11807: (follow-up) remove date conversions
To test
1/ Apply both patches
2/ This patch lets you easily configure mappings for categorycode values.
These mapping will be used when updating the user's account after a successful
LDAP login.
Here is an example configuration :
STU
...
STUDENT
EMPLOYEE
3/ With this configuration, LDAP users with the usertype value "EMP" on the
LDAP server should have the "EMPLOYEE" categorycode in Koha.
Signed-off-by: Chris
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 05:01:59 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 04:01:59 +0000
Subject: [Koha-bugs] [Bug 14571] using_https check for ssl connections
doesn't work in some situations
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14571
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |chris at bigballofwax.co.nz
--- Comment #4 from Chris Cormack ---
Can we get a sign off on this please, I can't sign it off.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:38:39 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:38:39 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
--- Comment #44 from M. Tompsett ---
(In reply to Chris Cormack from comment #43)
> Falling back to patching base and 3-way merge...
> Auto-merging reports/issues_stats.pl
> CONFLICT (content): Merge conflict in reports/issues_stats.pl
> Failed to merge in the changes.
>
>
> Not a simple conflict
Most of it seemed to be whitespace indentation issues.
I'll upload a fix, and then do a comparison to make sure nothing seems lost.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:39:55 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:39:55 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #39250|0 |1
is obsolete| |
--- Comment #45 from M. Tompsett ---
Created attachment 46184
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46184&action=edit
Bug 7679: circulation statistics wizard improvements
- Add patron branch to the list of possible rows, columns, and filters
- Add extended patron attributes to the list of possible rows, columns,
and filters
- Allow to group period by hour
- Allow to translate filter names
NOTE: Removed signed off, since I believe I have dealt with the conflicts.
Retesting required.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:40:01 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:40:01 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #39251|0 |1
is obsolete| |
--- Comment #46 from M. Tompsett ---
Created attachment 46185
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46185&action=edit
Bug 7679: Followup: circulation statistics wizard improvements
Signed-off-by: Brendan
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:40:08 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:40:08 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #39252|0 |1
is obsolete| |
--- Comment #47 from M. Tompsett ---
Created attachment 46186
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46186&action=edit
Bug 7679: Fix criteria display
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:40:15 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:40:15 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #39253|0 |1
is obsolete| |
--- Comment #48 from M. Tompsett ---
Created attachment 46187
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46187&action=edit
Bug 7679: Group different case values in the same row/column
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:40:21 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:40:21 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #39254|0 |1
is obsolete| |
--- Comment #49 from M. Tompsett ---
Created attachment 46188
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46188&action=edit
Bug 7679: Fix several bugs in circulation statistics wizard
- Display values in NULL columns/rows
- Display columns/rows that contain values only in NULL rows/columns
- "To" filter in period row should be inclusive (YYYY-MM-DD should be
YYYY-MM-DD 23:59:59)
- Make it possible to use only the "To" filter in period row (actually
it results in DBI error (2 bind variables instead of 1))
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:40:28 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:40:28 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #39255|0 |1
is obsolete| |
--- Comment #50 from M. Tompsett ---
Created attachment 46189
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46189&action=edit
Bug 7679: Fix two SQL queries when using patron attributes
and remove some Perl warnings
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:40:35 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:40:35 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #39256|0 |1
is obsolete| |
--- Comment #51 from M. Tompsett ---
Created attachment 46190
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46190&action=edit
Bug 7679: Various fixes for circulation statistics wizard
- use SQL TRIM functions to avoid having '' and ' ' considered as
different values
- use Text::Unaccent to remove accents from columns or rows values when
accessing %table. This is required as MySQL consider as equals two
strings that differ only by their accents when using GROUP BY clause.
- Exclude '' values from the list of columns or rows. Otherwise we could
have a row 'UNKNOWN VALUE' and a row 'NULL' which both have the same
values in their cells.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:40:41 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:40:41 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #39257|0 |1
is obsolete| |
--- Comment #52 from M. Tompsett ---
Created attachment 46191
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46191&action=edit
Bug 7679: Display all table headers (case and accent sensitive)
It appears that we can't assume that a lowercase word is equivalent to
the same word in uppercase (or a mix of lowercase and uppercase
letters). And this is also true about accented words.
So display all values even if they differ only by case or by their
accents.
This means that:
- "foobar" and "Foobar" will be in two separate rows/columns
- "Etudiant" and "?tudiant" (french) will be in two separate
rows/columns too
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:57:14 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:57:14 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46184|0 |1
is obsolete| |
--- Comment #53 from M. Tompsett ---
Created attachment 46192
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46192&action=edit
Bug 7679: circulation statistics wizard improvements
- Add patron branch to the list of possible rows, columns, and filters
- Add extended patron attributes to the list of possible rows, columns,
and filters
- Allow to group period by hour
- Allow to translate filter names
NOTE: Removed signed off, since I believe I have dealt with the conflicts.
Retesting required. Found some comments C4::Date that I missed.
line prepatch 194/ postpatch 233 seems to be where the major
conflict is.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:57:21 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:57:21 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46185|0 |1
is obsolete| |
--- Comment #54 from M. Tompsett ---
Created attachment 46193
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46193&action=edit
Bug 7679: Followup: circulation statistics wizard improvements
Signed-off-by: Brendan
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:57:27 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:57:27 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46186|0 |1
is obsolete| |
--- Comment #55 from M. Tompsett ---
Created attachment 46194
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46194&action=edit
Bug 7679: Fix criteria display
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:57:34 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:57:34 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46187|0 |1
is obsolete| |
--- Comment #56 from M. Tompsett ---
Created attachment 46195
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46195&action=edit
Bug 7679: Group different case values in the same row/column
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:57:40 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:57:40 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46188|0 |1
is obsolete| |
--- Comment #57 from M. Tompsett ---
Created attachment 46196
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46196&action=edit
Bug 7679: Fix several bugs in circulation statistics wizard
- Display values in NULL columns/rows
- Display columns/rows that contain values only in NULL rows/columns
- "To" filter in period row should be inclusive (YYYY-MM-DD should be
YYYY-MM-DD 23:59:59)
- Make it possible to use only the "To" filter in period row (actually
it results in DBI error (2 bind variables instead of 1))
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:57:45 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:57:45 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46189|0 |1
is obsolete| |
--- Comment #58 from M. Tompsett ---
Created attachment 46197
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46197&action=edit
Bug 7679: Fix two SQL queries when using patron attributes
and remove some Perl warnings
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:57:51 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:57:51 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46190|0 |1
is obsolete| |
--- Comment #59 from M. Tompsett ---
Created attachment 46198
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46198&action=edit
Bug 7679: Various fixes for circulation statistics wizard
- use SQL TRIM functions to avoid having '' and ' ' considered as
different values
- use Text::Unaccent to remove accents from columns or rows values when
accessing %table. This is required as MySQL consider as equals two
strings that differ only by their accents when using GROUP BY clause.
- Exclude '' values from the list of columns or rows. Otherwise we could
have a row 'UNKNOWN VALUE' and a row 'NULL' which both have the same
values in their cells.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:57:57 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:57:57 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46191|0 |1
is obsolete| |
--- Comment #60 from M. Tompsett ---
Created attachment 46199
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46199&action=edit
Bug 7679: Display all table headers (case and accent sensitive)
It appears that we can't assume that a lowercase word is equivalent to
the same word in uppercase (or a mix of lowercase and uppercase
letters). And this is also true about accented words.
So display all values even if they differ only by case or by their
accents.
This means that:
- "foobar" and "Foobar" will be in two separate rows/columns
- "Etudiant" and "?tudiant" (french) will be in two separate
rows/columns too
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 06:58:21 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 05:58:21 +0000
Subject: [Koha-bugs] [Bug 7679] Statistics wizard: circulation (new filters)
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7679
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Patch doesn't apply |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:25:08 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:25:08 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #38946|0 |1
is obsolete| |
--- Comment #25 from M. Tompsett ---
Created attachment 46200
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46200&action=edit
Bug 14097: Testing C4::UsageStats
DESCRIPTION
-----------
Add unit tests to C4::UsageStats
Subroutine :
(V : tested, X : Not tested)
V NeedUpdate
V BuiltReport
X ReportToCommunity
V _count
TEST PLAN
---------
1. Apply patch
2. prove -v t/db_dependent/UsageStats.t
-- All tests should run successfully
/!\ I have 1 warning but I don't know where it can come /!\
(in cleanup) DBIx::Class::Storage::txn_rollback(): A txn_rollback
in nested transaction is ineffective! (depth 1) at
t/lib/TestBuilder.pm line 308
TEST PLAN OPTIONAL
------------------
Go check bug 13899 to see the coverage of this module.
Coverage BEFORE this patch :
Statement : 40,5%
Branch : 0,0%
Condition : 0,0%
Subroutine : 55,6%
Coverage AFTER this patch :
Statement : 81,0%
Branch : 100,0%
Condition : 100,0%
Subroutine : 88,8%
Signed-off-by: Bernardo Gonzalez Kriegel
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:25:19 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:25:19 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #38947|0 |1
is obsolete| |
--- Comment #26 from M. Tompsett ---
Created attachment 46201
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46201&action=edit
Bug 14097: Delete table issues before testing
If we delete all biblio DB, we could have some troubles with foreign keys
Signed-off-by: Bernardo Gonzalez Kriegel
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:25:30 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:25:30 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #38948|0 |1
is obsolete| |
--- Comment #27 from M. Tompsett ---
Created attachment 46202
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46202&action=edit
Bug 14097: changing verif_systempreferences_values subroutine
It's saving a large number of lines
Signed-off-by: Bernardo Gonzalez Kriegel
All tests run, no koha-qa errors
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:25:55 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:25:55 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #39085|0 |1
is obsolete| |
--- Comment #28 from M. Tompsett ---
Created attachment 46203
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46203&action=edit
Bug 14097: Rollback issue and warning corrected
The rollback issue came from '_count' testing.
It used a 'DROP TABLE' and 'CREATE TABLE' which implicitly do autocommit.
That's why the final rollback could not be.
This patch no longer uses t::lib::TestBuilder either because it was the cause
of the warning on Comment #1 :
"DBIx::Class::Storage::txn_rollback(): A txn_rollback
in nested transaction is ineffective! (depth 1) at
t/lib/TestBuilder.pm line 308"
Signed-off-by: Bernardo Gonzalez Kriegel
Now data is preserved, no errors.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:26:05 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:26:05 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #39435|0 |1
is obsolete| |
--- Comment #29 from M. Tompsett ---
Created attachment 46204
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46204&action=edit
Bug 14097 - Correct noise under Debian
The first four patches do not create any noise under Ubuntu 14.04 LTS.
Running on Debian Jessie:
[SNIP]
keys on reference is experimental at t/db_dependent/UsageStats.t line 88.
keys on reference is experimental at t/db_dependent/UsageStats.t line 106.
keys on reference is experimental at t/db_dependent/UsageStats.t line 119.
keys on reference is experimental at t/db_dependent/UsageStats.t line 136.
keys on reference is experimental at t/db_dependent/UsageStats.t line 153.
keys on reference is experimental at t/db_dependent/UsageStats.t line 162.
keys on reference is experimental at t/db_dependent/UsageStats.t line 587.
[SNIP]
Instead of having 'keys $foo->{bar}', 'keys %{$foo->{bar}}' was used.
TEST PLAN
---------
1) On a DEBIAN system (Jessie gives above messages)
apply the first 4 patches interactively.
2) $ prove t/db_dependent/UsageStats.t
-- trigger noise like above.
3) apply this patch
4) $ prove t/db_dependent/UsageStats.t
-- no noise triggered.
5) koha qa test tools
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:26:13 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:26:13 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #41843|0 |1
is obsolete| |
--- Comment #30 from M. Tompsett ---
Created attachment 46205
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46205&action=edit
Bug 14097 : Avoid issue if the number of syspref shared is modified
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:26:27 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:26:27 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #41844|0 |1
is obsolete| |
--- Comment #31 from M. Tompsett ---
Created attachment 46206
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46206&action=edit
Bug 14097 : Changing AddReserve prototype call
In response of Joubu comment #19
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:26:36 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:26:36 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
--- Comment #32 from M. Tompsett ---
Created attachment 46207
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46207&action=edit
Bug 14097: Fixed missing test cases
Fixed AllowPkiAuth issue caused by 15101.
Added missing AudioAlerts
TEST PLAN
---------
1) prove -v t/db_dependent/UsageStats.t
-- should pass after applying patch
2) Run koha qa test tools
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:30:14 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:30:14 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46200|0 |1
is obsolete| |
--- Comment #33 from M. Tompsett ---
Created attachment 46208
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46208&action=edit
Bug 14097: Testing C4::UsageStats
DESCRIPTION
-----------
Add unit tests to C4::UsageStats
Subroutine :
(V : tested, X : Not tested)
V NeedUpdate
V BuiltReport
X ReportToCommunity
V _count
TEST PLAN
---------
1. Apply patch
2. prove -v t/db_dependent/UsageStats.t
-- All tests should run successfully
/!\ I have 1 warning but I don't know where it can come /!\
(in cleanup) DBIx::Class::Storage::txn_rollback(): A txn_rollback
in nested transaction is ineffective! (depth 1) at
t/lib/TestBuilder.pm line 308
TEST PLAN OPTIONAL
------------------
Go check bug 13899 to see the coverage of this module.
Coverage BEFORE this patch :
Statement : 40,5%
Branch : 0,0%
Condition : 0,0%
Subroutine : 55,6%
Coverage AFTER this patch :
Statement : 81,0%
Branch : 100,0%
Condition : 100,0%
Subroutine : 88,8%
Signed-off-by: Bernardo Gonzalez Kriegel
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:30:21 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:30:21 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46201|0 |1
is obsolete| |
--- Comment #34 from M. Tompsett ---
Created attachment 46209
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46209&action=edit
Bug 14097: Delete table issues before testing
If we delete all biblio DB, we could have some troubles with foreign keys
Signed-off-by: Bernardo Gonzalez Kriegel
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:30:30 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:30:30 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46202|0 |1
is obsolete| |
--- Comment #35 from M. Tompsett ---
Created attachment 46210
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46210&action=edit
Bug 14097: changing verif_systempreferences_values subroutine
It's saving a large number of lines
Signed-off-by: Bernardo Gonzalez Kriegel
All tests run, no koha-qa errors
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:30:34 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:30:34 +0000
Subject: [Koha-bugs] [Bug 15138] typo in reports/borrowers_out.pl -
issues.timestamap
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15138
Fr?d?ric Demians changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |frederic at tamil.fr
--- Comment #6 from Fr?d?ric Demians ---
This patch has been pushed to 3.20.x, will be in 3.20.8.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:30:36 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:30:36 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46203|0 |1
is obsolete| |
--- Comment #36 from M. Tompsett ---
Created attachment 46211
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46211&action=edit
Bug 14097: Rollback issue and warning corrected
The rollback issue came from '_count' testing.
It used a 'DROP TABLE' and 'CREATE TABLE' which implicitly do autocommit.
That's why the final rollback could not be.
This patch no longer uses t::lib::TestBuilder either because it was the cause
of the warning on Comment #1 :
"DBIx::Class::Storage::txn_rollback(): A txn_rollback
in nested transaction is ineffective! (depth 1) at
t/lib/TestBuilder.pm line 308"
Signed-off-by: Bernardo Gonzalez Kriegel
Now data is preserved, no errors.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:30:40 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:30:40 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46204|0 |1
is obsolete| |
--- Comment #37 from M. Tompsett ---
Created attachment 46212
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46212&action=edit
Bug 14097 - Correct noise under Debian
The first four patches do not create any noise under Ubuntu 14.04 LTS.
Running on Debian Jessie:
[SNIP]
keys on reference is experimental at t/db_dependent/UsageStats.t line 88.
keys on reference is experimental at t/db_dependent/UsageStats.t line 106.
keys on reference is experimental at t/db_dependent/UsageStats.t line 119.
keys on reference is experimental at t/db_dependent/UsageStats.t line 136.
keys on reference is experimental at t/db_dependent/UsageStats.t line 153.
keys on reference is experimental at t/db_dependent/UsageStats.t line 162.
keys on reference is experimental at t/db_dependent/UsageStats.t line 587.
[SNIP]
Instead of having 'keys $foo->{bar}', 'keys %{$foo->{bar}}' was used.
TEST PLAN
---------
1) On a DEBIAN system (Jessie gives above messages)
apply the first 4 patches interactively.
2) $ prove t/db_dependent/UsageStats.t
-- trigger noise like above.
3) apply this patch
4) $ prove t/db_dependent/UsageStats.t
-- no noise triggered.
5) koha qa test tools
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:30:45 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:30:45 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46205|0 |1
is obsolete| |
--- Comment #38 from M. Tompsett ---
Created attachment 46213
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46213&action=edit
Bug 14097 : Avoid issue if the number of syspref shared is modified
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:30:49 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:30:49 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46206|0 |1
is obsolete| |
--- Comment #39 from M. Tompsett ---
Created attachment 46214
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46214&action=edit
Bug 14097 : Changing AddReserve prototype call
In response of Joubu comment #19
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 07:30:54 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 06:30:54 +0000
Subject: [Koha-bugs] [Bug 14097] Add unit tests to C4::UsageStats
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14097
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46207|0 |1
is obsolete| |
--- Comment #40 from M. Tompsett ---
Created attachment 46215
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46215&action=edit
Bug 14097: Fixed missing test cases
Fixed AllowPkiAuth issue caused by 15101.
Added missing AudioAlerts
TEST PLAN
---------
1) prove -v t/db_dependent/UsageStats.t
-- should pass after applying patch
2) Run koha qa test tools
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 08:18:42 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 07:18:42 +0000
Subject: [Koha-bugs] [Bug 15304] Norwegian patron database: translatable
strings added to all po files
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15304
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Pushed to Master |Pushed to Stable
CC| |julian.maurice at biblibre.com
--- Comment #17 from Julian Maurice ---
Patch pushed to 3.22.x, will be in 3.22.2
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 08:27:38 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 07:27:38 +0000
Subject: [Koha-bugs] [Bug 15358] merge.pl does not populate values to merge
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15358
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |11315
Referenced Bugs:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11315
[Bug 11315] Add support for subfield deletion when doing authority merges.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 08:27:38 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 07:27:38 +0000
Subject: [Koha-bugs] [Bug 11315] Add support for subfield deletion when
doing authority merges.
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11315
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |15358
Referenced Bugs:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15358
[Bug 15358] merge.pl does not populate values to merge
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 08:32:50 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 07:32:50 +0000
Subject: [Koha-bugs] [Bug 14385] Extend OpacHiddenItems to allow specifying
exempt borrower categories
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14385
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |chris at bigballofwax.co.nz
--- Comment #2 from Chris Cormack ---
Another one I can't sign off on, could someone please test this
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 08:42:01 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 07:42:01 +0000
Subject: [Koha-bugs] [Bug 13022] Hardcoded limit causes records with more
than 20 items to show inaccurate statuses
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13022
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Pushed to Master |Pushed to Stable
CC| |julian.maurice at biblibre.com
--- Comment #29 from Julian Maurice ---
Patches pushed to 3.22.x, will be in 3.22.2
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 08:52:03 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 07:52:03 +0000
Subject: [Koha-bugs] [Bug 7677] New area in subscriptions and new function
when receiving
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7677
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 08:52:08 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 07:52:08 +0000
Subject: [Koha-bugs] [Bug 7677] New area in subscriptions and new function
when receiving
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7677
Chris Cormack changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #45943|0 |1
is obsolete| |
--- Comment #77 from Chris Cormack ---
Created attachment 46216
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46216&action=edit
Bug 7677 [Follow-up]: New areas in subscriptions and new functions when
receiving.
- Various fixes.
Test plan:
Once the makePreviousSerialAvailable syspref is enabled, receive a serial, and
then another, then check that:
- the first received itemtype has been set to the "previous item type" value
(set in the subscription).
- the first received has been made available.
- the last received serial itemtype has been set to the "item type" value (set
in the subscription).
- 995$l is automatically prefilled.
Configure the serialsFieldsToAV syspref. When creating or editing a
subscription, check that:
- the domain and/or origin and/or support fields are correctly displaying the
authorized values configured in the syspref.
Signed-off-by: Chris
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 10:03:32 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 09:03:32 +0000
Subject: [Koha-bugs] [Bug 15442] Checkouts table will not display due to
javascript error
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15442
--- Comment #7 from Jonathan Druart ---
(In reply to Fr?d?ric Demians from comment #6)
> This patch has been pushed to 3.20.x, will be in 3.20.8.
This patch is a bug fix for bug 15289, which is not in 3.20.x.
It was not needed for this branch but won't cause any regression.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 10:27:14 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 09:27:14 +0000
Subject: [Koha-bugs] [Bug 12528] Enable staff to deny message setting access
to patrons on the OPAC
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12528
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46156|0 |1
is obsolete| |
--- Comment #54 from Jonathan Druart ---
Created attachment 46217
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46217&action=edit
Bug 12528: Bug 9254: Followup - Rename pref to EnhancedMessagingPreferencesOPAC
If the new pref is named EnhancedMessagingPreferencesOPAC, it will show
up adjacent to EnhancedMessagingPreferences
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 10:27:20 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 09:27:20 +0000
Subject: [Koha-bugs] [Bug 12528] Enable staff to deny message setting access
to patrons on the OPAC
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12528
--- Comment #55 from Jonathan Druart ---
Created attachment 46218
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46218&action=edit
Bug 12528: redirect to 404 if at least 1 pref is off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 10:28:03 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 09:28:03 +0000
Subject: [Koha-bugs] [Bug 12528] Enable staff to deny message setting access
to patrons on the OPAC
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12528
--- Comment #56 from Jonathan Druart ---
Please keep authorship
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 10:34:18 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 09:34:18 +0000
Subject: [Koha-bugs] [Bug 15190] Bad utf8 decode to unapi and fixing blank
screen error status
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15190
--- Comment #11 from Marcel de Rooy ---
http://code4lib.org/files/unapi_revision_1-14.html
https://www.google.nl/url?sa=t&rct=j&q=&esrc=s&source=web&cd=4&ved=0ahUKEwjFvafUzPbJAhWCDhoKHarGDJUQFgg4MAM&url=https%3A%2F%2Fir.library.oregonstate.edu%2Fxmlui%2Fbitstream%2Fhandle%2F1957%2F2359%2Fintro-unapi-draftV15.doc%3Fsequence%3D8&usg=AFQjCNEGxssbXiRQmzUPElGJkGQd1hjx5w&bvm=bv.110151844,d.bGQ
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 10:38:02 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 09:38:02 +0000
Subject: [Koha-bugs] [Bug 15294] Move the C4::Branch related code to
Koha::Libraries - part 1
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15294
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #45922|0 |1
is obsolete| |
--- Comment #10 from Jonathan Druart ---
Created attachment 46219
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46219&action=edit
Bug 15294: Koha::Libraries - Move existing Koha::Branch[es]
There was already 2 Koha::Branch[es] using Koha::Object[s] before.
For this new rewrite, it seems preferable to start with good basis and
name the new modules Koha::Library and Koha::Libraries.
Tested both patches together, works as expected.
Signed-off-by: Marc V?ron
Signed-off-by: Martin Renvoize
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 10:38:07 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 09:38:07 +0000
Subject: [Koha-bugs] [Bug 15294] Move the C4::Branch related code to
Koha::Libraries - part 1
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15294
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #45923|0 |1
is obsolete| |
--- Comment #11 from Jonathan Druart ---
Created attachment 46220
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46220&action=edit
Bug 15294: Koha::Libraries - Rewrite the admin scripts
This patch rewrites the admin/branches.pl script to use the new modules
instead of C4::Branches.
Test plan:
1/ Create libraries using all the fields available
2/ Create groups of libraries
3/ Assign 1+ libraries to some groups
4/ Delete libraries and groups of libraries
You should not able to delete a library if items or patrons use it.
You should not able to delete a group of libraries if there are still
libraries using it.
5/ Update libraries and groups of libraries
Followed test plan, works as expected.
Signed-off-by: Marc V?ron
Signed-off-by: Martin Renvoize
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 10:40:05 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 09:40:05 +0000
Subject: [Koha-bugs] [Bug 15294] Move the C4::Branch related code to
Koha::Libraries - part 1
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15294
--- Comment #12 from Jonathan Druart ---
(In reply to Brendan Gallagher from comment #9)
> Didn't apply cleanly for me - can you rebase this Jonathan? (ping me when
> you've rebased so I can test it and push it).
>
> Thanks!
Trivial conflicts with bug 15373, bug 13517 and bug 13853 fixed.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 10:41:49 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 09:41:49 +0000
Subject: [Koha-bugs] [Bug 14306] Show URL from MARC21 field 555$u in basket
and detail
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14306
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #42013|0 |1
is obsolete| |
--- Comment #18 from Marcel de Rooy ---
Created attachment 46221
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46221&action=edit
Bug 14306: Remove call to GetMarcNotes from sendshelf
Just as mentioned in the first patch for sendbasket, I discovered that
the call of GetMarcNotes is not really used in sendshelf. The array is
passed to the template, but the template does not use it. (It uses the
information from GetBiblioData.)
Test plan:
[1] Send a list that includes a record with some notes (opac and staff).
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 10:43:40 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 09:43:40 +0000
Subject: [Koha-bugs] [Bug 14306] Show URL from MARC21 field 555$u in basket
and detail
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14306
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Patch doesn't apply |Needs Signoff
--- Comment #19 from Marcel de Rooy ---
Rebased. Who adds a signoff?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 11:03:20 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 10:03:20 +0000
Subject: [Koha-bugs] [Bug 15354] item types aren't showing in default hold
policies
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15354
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 11:03:23 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 10:03:23 +0000
Subject: [Koha-bugs] [Bug 15354] item types aren't showing in default hold
policies
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15354
--- Comment #5 from Jonathan Druart ---
Created attachment 46222
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46222&action=edit
Bug 15354: Sort itemtypes by translated description everywhere on circ rules
admin page
It's better to cmp on lc.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 11:05:56 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 10:05:56 +0000
Subject: [Koha-bugs] [Bug 15354] item types aren't showing in default hold
policies
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15354
--- Comment #6 from Jonathan Druart ---
(In reply to Jesse Weaver from comment #4)
> Minor whitespace issues:
>
> FAIL koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt
> FAIL forbidden patterns
> forbidden pattern: tab char (line 188)
> SKIP spelling
> OK tt_valid
> OK valid_template
>
> (These aren't really your fault; I put a patch to fix all this mess on bug
> 15456 that you can rebase onto if you'd prefer.)
I think it should be fixed asap and not depend on something else to ease a
possible backport.
> More importantly, though, I noticed that while the item type descriptions
> now correctly show, and the list of rules is sorted by the translated
> description, the item type selection dropdown is not sorted by the
> translated description.
Good catch. I have fixed it.
Not that C4::Koha::GetItemTypes should be fixed for a global fix.
It will be easy to do on top of bug 14828.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 11:39:52 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 10:39:52 +0000
Subject: [Koha-bugs] [Bug 14199] Unify all organization code plugins
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14199
--- Comment #11 from Marcel de Rooy ---
(In reply to Katrin Fischer from comment #10)
> Hi Marcel,
>
> can you please fix the patches to work with the renamed files from bug 15181?
The big change that affects this patch, comes from:
b2c302440b7d33e8dc3923b645286a70111dd305 Bug 15097: Update MARC21 de-DE
frameworks to Update 21 (September 2015)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 11:44:56 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 10:44:56 +0000
Subject: [Koha-bugs] [Bug 14199] Unify all organization code plugins
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14199
--- Comment #12 from Katrin Fischer ---
How is that? Maybe it's bug 10963?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 11:57:16 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 10:57:16 +0000
Subject: [Koha-bugs] [Bug 14199] Unify all organization code plugins
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14199
--- Comment #13 from Katrin Fischer ---
Ah sorry, missed looking at the second patch - but I think some more frameworks
updated since (it-IT?)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 12:03:48 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 11:03:48 +0000
Subject: [Koha-bugs] [Bug 14199] Unify all organization code plugins
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14199
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #44020|0 |1
is obsolete| |
--- Comment #14 from Marcel de Rooy ---
Created attachment 46223
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46223&action=edit
Bug 14199: Modify webinstaller sql files for marc21_orgcode plugin
This patch updates the sql installer files for new installations.
Occurrences to the three plugins marc21_field_003, 040c and 040d are
replaced by marc21_orgcode.pl.
This has been generated by one sed statement.
Test plan:
[1] Run a new installation or run some of the changed sql files manually.
[2] Verify that marc21_orgcode is linked to fields previously linked to
the three old plugins.
[3] Do a git grep on the three plugin file names. No occurrences expected.
Signed-off-by: Marcel de Rooy
Have tested the three english sql files:
authorities_normal_marc21.sql: 18 rows with marc21_orgcode (003 and 040a
for 9 auth codes)
marc21_framework_DEFAULT.sql: 1 row with marc21_orgcode (003) -- this patch
does not add them to 040c or 040d; no behavior change..
marc21_simple_bib_frameworks.sql: 8 rows (003 for 8 frameworks)
Signed-off-by: Bernardo Gonzalez Kriegel
Signed-off-by: Marcel de Rooy
The following files do no longer need an adjustment:
installer/data/mysql/de-DE/marcflavour/marc21/optional/marc21_simple_bib_frameworks.sql
installer/data/mysql/es-ES/marcflavour/marc21/optional/marc21_simple_bib_frameworks.sql
installer/data/mysql/it-IT/marcflavour/marc21/optional/marc21_simple_bib_frameworks.sql
For details, check Bug 15097: Update MARC21 de-DE frameworks to Update 21.
And similar reports for es-ES and it-IT.
Verified this change by installing de-DE in the webinstaller. The plugin for
field 003 was correctly set to marc21_orgcode.pl.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 12:04:10 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 11:04:10 +0000
Subject: [Koha-bugs] [Bug 14060] Remove readonly on date inputs
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14060
--- Comment #17 from Jonathan Druart ---
Created attachment 46224
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46224&action=edit
Bug 14060: Add support for dmydot
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 12:05:29 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 11:05:29 +0000
Subject: [Koha-bugs] [Bug 14060] Remove readonly on date inputs
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14060
--- Comment #18 from Jonathan Druart ---
(In reply to Marc V?ron from comment #16)
> Hi Jonathan,
>
> Sorry, I found following issues:
>
> 1)
> Syspref dateformat = dmydot (01.10.2017) is not covered by the function
> is_valid_date
>
> Symptom 1:
> - Set syspref dateformat to dmydot, clear browser chache
> - Try to add a manual restriction on moremember.pl
> - manually enter 26.01.2017
> - hit 'Add restriction'
> Result: restriction is added with empty date = indefinite restriction.
>
> Symptom 2:
> - manually enter 1.1.2018
> - hit enter
> - Message appears "Please enter valid date...", closes automatically after
> ca. 1 second
> Result: restriction is added with empty date = indefinite restriction.
Should be fixed with the last patch.
> 2)
> Software error in following cases
>
> Case 1:
> - Set syspref dateformat to dd/mm/yyyy, clear browser cache
> - manually enter 1/10/2017
> - hit 'Add restriction'
> Result: Software error: The given date (1/10/2017) does not match the date
> format (metric) at /usr/share/kohaclone/Koha/DateUtils.pm line 153.
>
> Case 2:
> - Set syspref to yyyy-mm-dd
> - manually enter 2017-2-2
> - hit 'Add Restiction'
> Result: Software error: The given date (2018-2-2) does not match the date
> format (iso) at /usr/share/kohaclone/Koha/DateUtils.pm line 153.
>
> It seems that the timepicker's function parseDate accepts such dates.
I don't manage to force the leading zeros, suggestions are welcome.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 12:05:44 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 11:05:44 +0000
Subject: [Koha-bugs] [Bug 14199] Unify all organization code plugins
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14199
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Patch doesn't apply |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 12:15:38 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 11:15:38 +0000
Subject: [Koha-bugs] [Bug 14224] patron notes about item shown at check in
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14224
--- Comment #5 from Jonathan Druart ---
(In reply to Martin Stenberg from comment #4)
> Hi Jonathan,
>
> Do you think it's worth adding a new subroutine GetIssue for this special
> case of GetIssues? GetIssues will still return an arrayref when called with
> issue_id, so the return type matches the plural "s" at least.
GetIssue would not return an array, but the matching issue.
> I'm doing an ajax request to minimize overhead when sending the request
> (i.e. no need to send note data for all issues when only one has been
> changed) and when recieving (i.e. no need to render a new template and send
> a whole document when only an "ok" or "nok" is needed). And as a bonus it
> allows the user to write a second note while the first one is beeing
> sent/processed to the server.
>
> To be honest I thought javascript was concidered a requirment for Koha. I'm
> using NoScript myself, and am always positivly surprised when a page works
> without having to enable javascript.
>
> How about keeping the ajax-version and adding a fallback for browsers
> without javascript?
You can add 2 forms in the page, the one needing JS can be hidden with a css
rule.
When the page is loading, a JS code could hide one and display the other one.
I suppose it's the way to go.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 12:17:04 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 11:17:04 +0000
Subject: [Koha-bugs] [Bug 14224] patron notes about item shown at check in
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14224
--- Comment #6 from Jonathan Druart ---
Additional note: the ajax part should be moved to svc/ and some of this code
put in a subroutine.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 12:28:20 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 11:28:20 +0000
Subject: [Koha-bugs] [Bug 14306] Show URL from MARC21 field 555$u in basket
and detail
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14306
--- Comment #20 from Marc V?ron ---
Hi Marcel, a question about the test plan:
Do you expect the values (500$a, 555$a and a URL in 555$u) to be inserted into
the mails? Or should we test only that the mails still work?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 12:36:50 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 11:36:50 +0000
Subject: [Koha-bugs] [Bug 15458] New: Marc21 leader 07 error in translation:
"a- Serial component" part is not in format
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15458
Bug ID: 15458
Summary: Marc21 leader 07 error in translation: "a- Serial
component" part is not in format
Change sponsored?: ---
Product: Koha
Version: 3.20
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: translate.koha-community.org
Assignee: koha-bugs at lists.koha-community.org
Reporter: paivi.knuutinen at jns.fi
QA Contact: testopia at bugs.koha-community.org
Created attachment 46225
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46225&action=edit
a- Monographic component part; b- Serial component part
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 12:39:42 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 11:39:42 +0000
Subject: [Koha-bugs] [Bug 13618] Prevent XSS in the Staff Client and the OPAC
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13618
--- Comment #100 from Jonathan Druart ---
Branches rebased against master.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 12:40:04 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 11:40:04 +0000
Subject: [Koha-bugs] [Bug 13618] Prevent XSS in the Staff Client and the OPAC
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13618
--- Comment #101 from Jonathan Druart ---
Branch rebased against master.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 12:50:53 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 11:50:53 +0000
Subject: [Koha-bugs] [Bug 15443] Re-code RESERVESLIP as HOLD_SLIP
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15443
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Passed QA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 4 12:50:56 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 04 Jan 2016 11:50:56 +0000
Subject: [Koha-bugs] [Bug 15443] Re-code RESERVESLIP as HOLD_SLIP
In-Reply-To:
References:
Message-ID:
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15443
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46153|0 |1
is obsolete| |
--- Comment #4 from Jonathan Druart ---
Created attachment 46226
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46226&action=edit
Bug 15443 - Re-code RESERVESLIP as HOLD_SLIP
It's a bit confusing to have RESERVESLIP named "Hold Slip", we should
finish the job and recode the slip.
Test Plan:
1) Apply this patch
2) Run updatedatabase.pl
3) Try printing a hold slip, you should note no difference
from pre-patch behavior
Followed test plan. The slip is triggered if you want to check out
an item that is reserved / on hold for another patron. Works as expected.
Signed-off-by: Marc V?ron