---
QA Comment:
Thx for your proposal. Some points that need attention:
Breeding.pm:
@servers = ();
foreach my $id (@id) {
push @servers, {id => $id};
}
Please explain what you are doing here. Seems unneeded or wrong.
$template->param(
additionalFields => '1'
) if C4::Context->preference('AdditionalFieldsInZ3950ResultSearch');
Please use Koha::Preference in the template.
sub _add_rowdata
Just intuitive, these additions dont look that nice to me. I think you should
add more validation and put this in another subroutine, taking care that you do
not overwrite the default fields.
+++
b/installer/data/mysql/atomicupdate/bug_12747-additional_fields_in_Z3950_search_result.sql
@@ -0,0 +1 @@
+INSERT INTO systempreferences
Please INSERT IGNORE
+('AddressFormat','us','','Choose format to display postal addresses',
'Choice'),
+('AdditionalFieldsInZ3950ResultSearch', '', NULL, 'Determines which MARC
field/subfields are displayed in -Additional field- column in the result of a
search Z3950', 'Free'),
Please respect the sort order in sysprefs.
Note: I saw that a LOT of other new prefs are not in the right order. But you
dont need to correct that here of course. Just add your own one in the right
place.
+ [% IF additionalFields %]
+
+
+ [% FOREACH addnumberfield IN breeding_loo.addnumberfields %]
+ - [% addnumberfield %]:
+ - [% breeding_loo.$addnumberfield %]
+ [% END %]
+
+ |
+ [% END %]
You add all additional fields in one column??
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 10:28:06 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 08:28:06 +0000
Subject: [Koha-bugs] [Bug 17385] Add custom export notices
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17385
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |m.de.rooy at rijksmuseum.nl
|y.org |
--- Comment #12 from Marcel de Rooy ---
QA: Looking here now
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 10:28:42 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 08:28:42 +0000
Subject: [Koha-bugs] [Bug 15924] Coce not enabled on lists
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15924
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #64163|0 |1
is obsolete| |
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |julian.maurice at biblibre.com
|y.org |
CC| |julian.maurice at biblibre.com
--- Comment #6 from Kyle M Hall ---
Created attachment 64323
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64323&action=edit
Bug 15924 - Coce not enabled on lists
To test:
Enable Coce and disable other image services
Load a list and not there are no covers
Apply patch
Load a list and note there are covers
https://bugs.koha-community.org/show_bug.cgi?id=15924
Signed-off-by: Owen Leonard
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 Fri Sep 1 10:43:20 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 08:43:20 +0000
Subject: [Koha-bugs] [Bug 15924] Coce not enabled on lists
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15924
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #64323|0 |1
is obsolete| |
--- Comment #7 from Julian Maurice ---
Created attachment 66691
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66691&action=edit
Bug 15924 - Coce not enabled on lists
To test:
Enable Coce and disable other image services
Load a list and not there are no covers
Apply patch
Load a list and note there are covers
https://bugs.koha-community.org/show_bug.cgi?id=15924
Signed-off-by: Owen Leonard
Signed-off-by: Kyle M Hall
Signed-off-by: Julian Maurice
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 10:43:35 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 08:43:35 +0000
Subject: [Koha-bugs] [Bug 15924] Coce not enabled on lists
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15924
Julian Maurice 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 Fri Sep 1 10:48:17 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 08:48:17 +0000
Subject: [Koha-bugs] [Bug 19007] Allow paypal payments via debit or credit
card again
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19007
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |julian.maurice at biblibre.com
|y.org |
CC| |julian.maurice at biblibre.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 10:50:18 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 08:50:18 +0000
Subject: [Koha-bugs] [Bug 17385] Add custom export notices
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17385
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Failed QA
--- Comment #13 from Marcel de Rooy ---
QA Comment:
Thanks for your work. I think this is not yet ready for inclusion.
sub XSLTParse4Display {
- my ( $biblionumber, $orig_record, $xslsyspref, $fixamps, $hidden_items,
$sysxml, $xslfilename, $lang ) = @_;
+ my ( $biblionumber, $orig_record, $xslsyspref, $fixamps, $hidden_items,
$sysxml, $xslfilename, $lang, $xslFile, $opac ) = @_;
We already have $xslfilename. You add $xslFile ? Does not look good.
The added code seems repetitive.
You add a new routine CustomXSLTExportList. It needs unit tests.
In that routine you are parsing the xsl files for something like +. I am not sure if that is a good idea. Another QA pov
is welcome too.
Note that you are doing it each time again too.
If you provide clear xslt names, that should probably be sufficient. Please
think about another way to enrich that information if needed.
--- a/Koha/XSLT_Handler.pm
+++ b/Koha/XSLT_Handler.pm
@@ -168,7 +168,6 @@ sub transform {
( $xml, $filename, $format ) = @_;
$format ||= 'chars';
}
-
#Initialized yet?
if ( !$self->{xslt_hash} ) {
$self->_init;
No need to remove an empty line if you are not touching this module anyway.
diff --git a/koha-tmpl/intranet-tmpl/prog/en/xslt/biblioexport/README
b/koha-tmpl/intranet-tmpl/prog/en/xslt/biblioexport/README
new file mode 100644
Empty README ?
And another further on.
koha-tmpl/opac-tmpl/bootstrap/en/xslt/biblioexport-samples/MARC21_simple_export.xsl
koha-tmpl/opac-tmpl/bootstrap/en/xslt/biblioexport/MARC21_simple_export.xsl
What is the use of adding two identical files here ?
Actually, all files here are sort of sample files.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 10:52:30 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 08:52:30 +0000
Subject: [Koha-bugs] [Bug 16899] Add ability to disallow overpayments via SIP
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16899
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |m.de.rooy at rijksmuseum.nl
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 10:58:10 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 08:58:10 +0000
Subject: [Koha-bugs] [Bug 16899] Add ability to disallow overpayments via SIP
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16899
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Failed QA
--- Comment #10 from Marcel de Rooy ---
+++ b/Koha/Account/Lines.pm
+sub amount_outstanding {
You add this sub, but you are not using it ? And not testing it ?
This needs feedback. Thanks.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 10:59:50 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 08:59:50 +0000
Subject: [Koha-bugs] [Bug 14353] Show 'damaged' and other status on the
'place holds' page in staff
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14353
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |m.de.rooy at rijksmuseum.nl
|y.org |
CC| |m.de.rooy at rijksmuseum.nl
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:08:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:08:36 +0000
Subject: [Koha-bugs] [Bug 14353] Show 'damaged' and other status on the
'place holds' page in staff
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14353
Marcel de Rooy 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 Fri Sep 1 11:08:39 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:08:39 +0000
Subject: [Koha-bugs] [Bug 14353] Show 'damaged' and other status on the
'place holds' page in staff
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14353
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #64161|0 |1
is obsolete| |
--- Comment #7 from Marcel de Rooy ---
Created attachment 66692
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66692&action=edit
Bug 14353 - Show 'damaged' and other status on the 'place holds' page in staff
This patch adds status 'Damaged' to 'Information' 'Status' in the items
table on 'Place hold' page.
To test:
- Apply patch
- In staff client, try to place an item level hold for items with 'Damaged'
status.
- Verify that the status 'Damaged' appears in the column 'Information'.
Signed-off-by: Owen Leonard
Signed-off-by: Marcel de Rooy
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:11:14 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:11:14 +0000
Subject: [Koha-bugs] [Bug 16486] Display the TIME a fine was
collected/written off
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16486
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |m.de.rooy at rijksmuseum.nl
Status|Signed Off |Failed QA
--- Comment #8 from Marcel de Rooy ---
(In reply to Jonathan Druart from comment #7)
> Would not it make sense to add it everywhere?
>
> % git grep ', thanks' **/*.tt|cut -d':' -f1|uniq
> koha-tmpl/intranet-tmpl/prog/en/modules/members/boraccount.tt
> koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt
> koha-tmpl/intranet-tmpl/prog/en/modules/members/printfeercpt.tt
> koha-tmpl/intranet-tmpl/prog/en/modules/members/printinvoice.tt
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-account.tt
>
> It seems that occurrences in pay.tt are not relevant, we do not display
> payments in this table.
No response on this question.
Changing status to reflect need for feedback.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:11:56 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:11:56 +0000
Subject: [Koha-bugs] [Bug 18796] Allow to print notice while claiming serials
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18796
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Patch doesn't apply
CC| |m.de.rooy at rijksmuseum.nl
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:14:18 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:14:18 +0000
Subject: [Koha-bugs] [Bug 18812] SIP Patron status does not respect
OverduesBlockCirc
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18812
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |m.de.rooy at rijksmuseum.nl
|y.org |
CC| |m.de.rooy at rijksmuseum.nl
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:40:38 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:40:38 +0000
Subject: [Koha-bugs] [Bug 17797] Add XSLT_Handler in opac/unapi
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17797
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |nick at bywatersolutions.com
|y.org |
CC| |nick at bywatersolutions.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:41:15 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:41:15 +0000
Subject: [Koha-bugs] [Bug 19007] Allow paypal payments via debit or credit
card again
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19007
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #65346|0 |1
is obsolete| |
--- Comment #5 from Julian Maurice ---
Created attachment 66693
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66693&action=edit
Bug 19007 - Allow paypal payments via debit or credit card again
A recent change in Paypal has removed the previous default option of paying via
debit or credit card without an account. To bring this option back, we need to
send an additional parameter to the PayPal API.
Test Plan:
1) Enable paypal for your Koha instance
2) Ensure you are not logged in to PayPal
3) Attempt to pay a fine via PayPal
4) Not the the "Pay with Debit or Credit Card" option is missing
5) Apply this patch
6) Refresh opac-account.pl
7) Attempt to make a payment via PayPal again
8) Note the option "Pay with Debit or Credit Card" is now available
Signed-off-by: Kyle M Hall
Signed-off-by: George Williams
Signed-off-by: Julian Maurice
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:43:27 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:43:27 +0000
Subject: [Koha-bugs] [Bug 19007] Allow paypal payments via debit or credit
card again
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19007
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Passed QA
--- Comment #6 from Julian Maurice ---
I can't see the "Pay with Debit or Credit Card" option, but this is probably
because of a misconfiguration in my PayPal account. According to the
documentation at
https://developer.paypal.com/docs/classic/api/merchant/SetExpressCheckout_API_Operation_NVP/
the patch is correct.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:45:31 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:45:31 +0000
Subject: [Koha-bugs] [Bug 19056] Move C4::Reserves::GetReserveCount to the
Koha namespace
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19056
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |julian.maurice at biblibre.com
QA Contact|testopia at bugs.koha-communit |julian.maurice at biblibre.com
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:46:13 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:46:13 +0000
Subject: [Koha-bugs] [Bug 19055] GetReservesToBranch is not used
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19055
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |julian.maurice at biblibre.com
QA Contact|testopia at bugs.koha-communit |julian.maurice at biblibre.com
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:47:55 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:47:55 +0000
Subject: [Koha-bugs] [Bug 19055] GetReservesToBranch is not used
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19055
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #65616|0 |1
is obsolete| |
--- Comment #4 from Julian Maurice ---
Created attachment 66694
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66694&action=edit
Bug 19055: Remove C4::Reserves::GetReservesToBranch
This subroutine is no longer in used and can be removed
Test plan:
git grep GetReservesToBranch
must not return any results
Signed-off-by: Marc V?ron
Signed-off-by: Julian Maurice
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:48:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:48:05 +0000
Subject: [Koha-bugs] [Bug 19055] GetReservesToBranch is not used
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19055
Julian Maurice 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 Fri Sep 1 11:51:11 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:51:11 +0000
Subject: [Koha-bugs] [Bug 17797] Add XSLT_Handler in opac/unapi
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17797
Nick Clemens 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 Fri Sep 1 11:51:15 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:51:15 +0000
Subject: [Koha-bugs] [Bug 17797] Add XSLT_Handler in opac/unapi
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17797
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #63947|0 |1
is obsolete| |
--- Comment #4 from Nick Clemens ---
Created attachment 66695
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66695&action=edit
Bug 17797: Add XSLT_Handler in opac/unapi
Replaces some code by a call to existing module.
Removes the $@->code and $@->message calls.
Test plan
[1] Run /cgi-bin/koha/unapi?id=koha:biblionumber:[number]&format=marcxml
[2] Try some variations.
Signed-off-by: Josef Moravec
Signed-off-by: Nick Clemens
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:52:58 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:52:58 +0000
Subject: [Koha-bugs] [Bug 19056] Move C4::Reserves::GetReserveCount to the
Koha namespace
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19056
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #65617|0 |1
is obsolete| |
--- Comment #3 from Julian Maurice ---
Created attachment 66696
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66696&action=edit
Bug 19056: Replace C4::Reserves::GetReserveCount with
Koha::Patron->holds->count
This subroutine is only used once and can easily be replaced with
Koha::Patron->holds->count
Test plan:
- Set maxreserves=5
- Place 3 holds for a given patron
- Place again 3 holds for this patron
3+3 > 5 => The holds must not be placed
Signed-off-by: Marc V?ron
Signed-off-by: Julian Maurice
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:53:08 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:53:08 +0000
Subject: [Koha-bugs] [Bug 19056] Move C4::Reserves::GetReserveCount to the
Koha namespace
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19056
Julian Maurice 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 Fri Sep 1 11:53:26 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:53:26 +0000
Subject: [Koha-bugs] [Bug 18812] SIP Patron status does not respect
OverduesBlockCirc
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18812
Marcel de Rooy 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 Fri Sep 1 11:53:28 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:53:28 +0000
Subject: [Koha-bugs] [Bug 18812] SIP Patron status does not respect
OverduesBlockCirc
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18812
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #64420|0 |1
is obsolete| |
--- Comment #3 from Marcel de Rooy ---
Created attachment 66697
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66697&action=edit
Bug 18812 - SIP Patron status does not respect OverduesBlockCirc
To test:
1 - Set 'OverduesBlockCirc' to block
2 - Find or create a patron with overdues
3 - Perform a SIP patron lookup on that patron
misc/sip_cli_emulator.pl -a 127.0.0.1 -p 6001 -su term1 -sp term1 -l CPL
--patron {userid or cardnumber} --password {pass} -m patron_information
4 - Note the first character of response is a ' '
5 - Apply patch
6 - Restart memcached, apache, and plack
7 - Perform SIP patron lookup
8 - Note the first character of response is 'Y'
9 - prove t/db_dependent/SIP/Patron.t
10 - Test should return green
Signed-off-by: Chris Kirby
Works as advertised
Signed-off-by: Marcel de Rooy
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 11:55:14 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 09:55:14 +0000
Subject: [Koha-bugs] [Bug 18820] The different parts in the main don' t
automatically adjust with the available space.
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18820
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |m.de.rooy at rijksmuseum.nl
|y.org |
CC| |m.de.rooy at rijksmuseum.nl
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:01:11 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:01:11 +0000
Subject: [Koha-bugs] [Bug 18820] The different parts in the main don' t
automatically adjust with the available space.
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18820
Nick Clemens 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 Fri Sep 1 12:01:14 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:01:14 +0000
Subject: [Koha-bugs] [Bug 18820] The different parts in the main don' t
automatically adjust with the available space.
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18820
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #64447|0 |1
is obsolete| |
--- Comment #4 from Nick Clemens ---
Created attachment 66698
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66698&action=edit
Bug 18820: Adding lines that check if opacuserlogin is enable and if OpacNav
has content.
Signed-off-by: Owen Leonard
Signed-off-by: Nick Clemens
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:05:00 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:05:00 +0000
Subject: [Koha-bugs] [Bug 19209] Koha::Objects should provide ->is_paged
method
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19209
--- Comment #12 from Tom?s Cohen Arazi ---
(In reply to Marcel de Rooy from comment #11)
> Small remark:
>
> + is( ref($patrons->pager), 'DBIx::Class::ResultSet::Pager',
> + 'Koha::Objects->pager returns a valid DBIx::Class object' );
>
> If that is what we really want? Until now we wrapped lots of things in Koha
> objects.
> Btw: pager returns a Data::Page object for the current resultset.
yeah, that looked weird, but I'm not touching it besides moving the tests into
another subtsests. It is probably worth discussing and eventually filling a new
bug.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:08:27 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:08:27 +0000
Subject: [Koha-bugs] [Bug 18820] The different parts in the main don' t
automatically adjust with the available space.
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18820
--- Comment #5 from Marcel de Rooy ---
(In reply to Nick Clemens from comment #4)
> Signed-off-by: Nick Clemens
Please read QA contact field !
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:10:00 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:10:00 +0000
Subject: [Koha-bugs] [Bug 16726] Text in Preferences search box does not
clear
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16726
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |nick at bywatersolutions.com
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:12:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:12:05 +0000
Subject: [Koha-bugs] [Bug 18820] The different parts in the main don' t
automatically adjust with the available space.
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18820
--- Comment #6 from Marcel de Rooy ---
Yes, I was testing it too and would have passed qa on it.
But I could not manange somehow to make this patch show a difference. Followed
the test plan and emptied Nav NavBottom MainUserBlock. Refreshing etc. no
difference?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:14:02 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:14:02 +0000
Subject: [Koha-bugs] [Bug 17797] Add XSLT_Handler in opac/unapi
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17797
--- Comment #5 from Marcel de Rooy ---
(In reply to Nick Clemens from comment #4)
> Signed-off-by: Nick Clemens
Thanks
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:16:48 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:16:48 +0000
Subject: [Koha-bugs] [Bug 17214] Add item in LIST by Record number
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17214
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |m.de.rooy at rijksmuseum.nl
|y.org |
--- Comment #11 from Marcel de Rooy ---
QA: Looking here now.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:27:15 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:27:15 +0000
Subject: [Koha-bugs] [Bug 17214] Add item in LIST by Record number
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17214
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Failed QA
--- Comment #12 from Marcel de Rooy ---
Bug 18260 has been pushed in the meantime and it removed GetBiblio.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:27:51 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:27:51 +0000
Subject: [Koha-bugs] [Bug 17214] Add item in LIST by Record number
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17214
--- Comment #13 from Marcel de Rooy ---
Internal server error:
Undefined subroutine
&CGI::Compile::ROOT::usr_share_koha_masterclone_virtualshelves_shelves_2epl::GetBiblio
called at /usr/share/koha/masterclone/virtualshelves/shelves.pl line 178.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:34:11 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:34:11 +0000
Subject: [Koha-bugs] [Bug 19076] Renewal via Checkout screen is logged as
both a renewal and a checkout
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19076
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |julian.maurice at biblibre.com
|y.org |
CC| |julian.maurice at biblibre.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:35:08 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:35:08 +0000
Subject: [Koha-bugs] [Bug 16726] Text in Preferences search box does not
clear
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16726
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #64072|0 |1
is obsolete| |
Attachment #64876|0 |1
is obsolete| |
--- Comment #19 from Nick Clemens ---
Created attachment 66699
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66699&action=edit
Bug 16726: Clear text in syspref searchbox after submitting
To test:
1) Go to Admin -> search for a system preference
2) Notice your search stays in the search box (this is inconsistent with
search behaviour across Koha)
3) Apply patch and refresh page
4) Make another search
5) Confirm search still works as expected and search terms have been
cleared from search box
6) Confirm search terms show at the top of the results
Sponsored-by: Catalyst IT
Signed-off-by: Nick Clemens
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:35:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:35:05 +0000
Subject: [Koha-bugs] [Bug 16726] Text in Preferences search box does not
clear
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16726
Nick Clemens 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 Fri Sep 1 12:35:11 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:35:11 +0000
Subject: [Koha-bugs] [Bug 16726] Text in Preferences search box does not
clear
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16726
--- Comment #20 from Nick Clemens ---
Created attachment 66700
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66700&action=edit
Bug 16726: [FOLLOW-UP] Putting text in h1
Works as outlined in test plan, search terms now appear at top as h1 as well
Signed-off-by: Dilan Johnpull?
Signed-off-by: Nick Clemens
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:36:48 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:36:48 +0000
Subject: [Koha-bugs] [Bug 19021] Inventory column sorting
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19021
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |nick at bywatersolutions.com
|y.org |
CC| |nick at bywatersolutions.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:42:13 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:42:13 +0000
Subject: [Koha-bugs] [Bug 12747] Add extra column in Z3950 search result
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12747
--- Comment #49 from Blou ---
To the last comment:
<<
You add all additional fields in one column??
>>
Yes. And it's beautiful.
I'll get someone to look at your other points as soon as possible. The one
about Koha.Preference just shows the age of this feature. :)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:42:39 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:42:39 +0000
Subject: [Koha-bugs] [Bug 10112] add test for ./debian/control file
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10112
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact| |kyle at bywatersolutions.com
CC| |kyle at bywatersolutions.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:44:13 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:44:13 +0000
Subject: [Koha-bugs] [Bug 19076] Renewal via Checkout screen is logged as
both a renewal and a checkout
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19076
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #65907|0 |1
is obsolete| |
--- Comment #9 from Julian Maurice ---
Created attachment 66701
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66701&action=edit
Bug 19076 - unit tests
Signed-off-by: Julian Maurice
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:44:17 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:44:17 +0000
Subject: [Koha-bugs] [Bug 19076] Renewal via Checkout screen is logged as
both a renewal and a checkout
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19076
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #65908|0 |1
is obsolete| |
--- Comment #10 from Julian Maurice ---
Created attachment 66702
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66702&action=edit
Bug 19076 - Move issue logging in AddIssue into condtional clause
Currently AddIssue tests if renewal, but logs an issue even if so. This
patch moves the logging into the conditional so a log entry is only
added if we aren't renewing (as renewals are logged separately)
To test:
1 - prove t/db_dependent/Circulation.t - one test should fail
2 - Enable both issue and renewal logs
3 - Checkout an item to a patron
4 - View the logs - the issue is captured
5 - Checkout the item to the patron again and confirm renewal
6 - Both an issue and a renewal are logged
7 - Apply patch
8 Repeat 1-6, tests should pass and only renewal should be logged
Signed-off-by: Julian Maurice
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:44:20 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:44:20 +0000
Subject: [Koha-bugs] [Bug 19076] Renewal via Checkout screen is logged as
both a renewal and a checkout
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19076
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #65916|0 |1
is obsolete| |
--- Comment #11 from Julian Maurice ---
Created attachment 66703
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66703&action=edit
Bug 19076 - followup - delete fines from additional renewal
Signed-off-by: Christopher Brannon
Signed-off-by: Julian Maurice
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:44:30 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:44:30 +0000
Subject: [Koha-bugs] [Bug 19076] Renewal via Checkout screen is logged as
both a renewal and a checkout
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19076
Julian Maurice 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 Fri Sep 1 12:45:35 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:45:35 +0000
Subject: [Koha-bugs] [Bug 10112] add test for ./debian/control file
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10112
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #65654|0 |1
is obsolete| |
--- Comment #10 from Kyle M Hall ---
Created attachment 66704
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66704&action=edit
Bug 10112 - add test for ./debian/control* files
to test, apply patch...
1/ install libparse-debcontrol-perl package
apt-get install libparse-debcontrol-perl
2/ run successful tests
$ perl ./t/00-valid-debian-control-file.t
1..2
ok 1 - parsed ./debian/control file cleanly
ok 2 - parsed ./debian/control.in file cleanly
3/ add some bad syntax to a file
$ sed -i -e '20i -------\' ./debian/control.in
4/ run failed tests
$ perl ./t/00-valid-debian-control-file.t
1..2
ok 1 - parsed ./debian/control file cleanly
not ok 2 - parsed ./debian/control.in file cleanly
Signed-off-by: Katrin Fischer
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 Fri Sep 1 12:45:33 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:45:33 +0000
Subject: [Koha-bugs] [Bug 10112] add test for ./debian/control file
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10112
Kyle M Hall 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 Fri Sep 1 12:48:02 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:48:02 +0000
Subject: [Koha-bugs] [Bug 19021] Inventory column sorting
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19021
--- Comment #5 from Nick Clemens ---
> => With patch first column is not sortable and table is sorted by barcode
When performing an inventory without a barcode file we are essentially
generating a shelf list, shouldn't callnumber (or cn_sort ideally) be the
default sort here?
Even when comparing to a barcode file I think we should probably be sorting by
callnumber in our results
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:52:06 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:52:06 +0000
Subject: [Koha-bugs] [Bug 11580] returnBeforeExpiry and closed day
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11580
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |kyle at bywatersolutions.com
|y.org |
CC| |kyle at bywatersolutions.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:52:52 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:52:52 +0000
Subject: [Koha-bugs] [Bug 11580] returnBeforeExpiry and closed day
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11580
Kyle M Hall 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 Fri Sep 1 12:52:55 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:52:55 +0000
Subject: [Koha-bugs] [Bug 11580] returnBeforeExpiry and closed day
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11580
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #65779|0 |1
is obsolete| |
Attachment #65780|0 |1
is obsolete| |
--- Comment #15 from Kyle M Hall ---
Created attachment 66705
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66705&action=edit
Bug 11580 - If returnBeforeExpiry is ON, use calendar to calculate return date
not on closed day (depending on useDaysMode)
Test plan:
1. ReturnBeforeExpiry is activated
2. useDaysMode is different from "circulation rules only"
3. Set expiry date of a patron to a near date
4. Set a closed day on calendar for this date
5. Do a checkout
Without patch, return date will be patron expiration date
With the patch, return date will be last open day before patron expiration day
Signed-off-by: Claire Gravely
Signed-off-by: Katrin Fischer
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 Fri Sep 1 12:53:03 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:53:03 +0000
Subject: [Koha-bugs] [Bug 11580] returnBeforeExpiry and closed day
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11580
--- Comment #16 from Kyle M Hall ---
Created attachment 66706
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66706&action=edit
BUG 11580 : Added unit test
Added one unit test when the syspref useDaysMode is active.
This does not move code anymore
Signed-off-by: Katrin Fischer
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 Fri Sep 1 12:53:06 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:53:06 +0000
Subject: [Koha-bugs] [Bug 11580] returnBeforeExpiry and closed day
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11580
--- Comment #17 from Kyle M Hall ---
Created attachment 66707
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66707&action=edit
Bug 11580 [QA 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 Fri Sep 1 12:53:35 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:53:35 +0000
Subject: [Koha-bugs] [Bug 18621] After duplicate message system picks
category expiry date rather than manual defined
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18621
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |kyle at bywatersolutions.com
QA Contact|testopia at bugs.koha-communit |kyle at bywatersolutions.com
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:53:50 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:53:50 +0000
Subject: [Koha-bugs] [Bug 19129] Clean up templates for organisation patrons
in staff
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19129
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |kyle at bywatersolutions.com
QA Contact|testopia at bugs.koha-communit |kyle at bywatersolutions.com
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 12:55:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 10:55:05 +0000
Subject: [Koha-bugs] [Bug 18811] Visibility settings inconsistent between
framework and authority editor
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |nick at bywatersolutions.com
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 13:19:29 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 11:19:29 +0000
Subject: [Koha-bugs] [Bug 18811] Visibility settings inconsistent between
framework and authority editor
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811
Nick Clemens 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 Fri Sep 1 13:19:33 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 11:19:33 +0000
Subject: [Koha-bugs] [Bug 18811] Visibility settings inconsistent between
framework and authority editor
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #66309|0 |1
is obsolete| |
Attachment #66310|0 |1
is obsolete| |
Attachment #66311|0 |1
is obsolete| |
Attachment #66312|0 |1
is obsolete| |
--- Comment #18 from Nick Clemens ---
Created attachment 66708
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66708&action=edit
Bug 18811: Change visibility checks in authorities.pl
The check is now <=-4 or >=5, but the framework uses 0 for Show all and
-5 for Hide all. (Note that sql installer scripts also use 8.)
When modifying an authority, the script also showed hidden fields when
filled, since it did not check the hidden field but only the tab field.
NOTE: The proposed solution restores consistency, but will remove hidden
fields from the MARC record.
Test plan:
[1] Set field 942a to Show all in an authority framework.
[2] Open a new record in this framework and verify that you see 942a.
[3] Edit an existing record in this framework and verify again.
[4] Set field 942a now to Hide all in this framework.
[5] Open a new record in this framework and verify that 942a is hidden.
[6] Edit an existing record in this framework and verify again.
Signed-off-by: Marcel de Rooy
Signed-off-by: Josef Moravec
Signed-off-by: Nick Clemens
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 13:19:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 11:19:36 +0000
Subject: [Koha-bugs] [Bug 18811] Visibility settings inconsistent between
framework and authority editor
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811
--- Comment #19 from Nick Clemens ---
Created attachment 66709
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66709&action=edit
Bug 18811: Add Koha Objects for authority tags and subfields
Trivial copy and paste activity.
Will be used in a later patch.
No test plan needed.
Signed-off-by: Marcel de Rooy
Signed-off-by: Josef Moravec
Signed-off-by: Nick Clemens
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 13:19:43 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 11:19:43 +0000
Subject: [Koha-bugs] [Bug 18811] Visibility settings inconsistent between
framework and authority editor
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811
--- Comment #21 from Nick Clemens ---
Created attachment 66711
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66711&action=edit
Bug 18811: Atomic update to print warning at upgrade time
The warning encourages people to run the auth_show_hidden_data script
to check for data in hidden fields and adjust their frameworks.
Signed-off-by: Marcel de Rooy
Signed-off-by: Josef Moravec
Signed-off-by: Nick Clemens
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 13:19:40 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 11:19:40 +0000
Subject: [Koha-bugs] [Bug 18811] Visibility settings inconsistent between
framework and authority editor
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811
--- Comment #20 from Nick Clemens ---
Created attachment 66710
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66710&action=edit
Bug 18811: Add a script for checking authority data in hidden fields
If you edit an authority record while having data in hidden fields or
subfields, that data will be lost now.
This script can help you to unhide some fields and prevent data loss.
Test plan:
[1] Add a PERSO_NAME record. Fill e.g. 100b.
[2] Hide 100b in the PERSO_NAME framework.
[3] Run auth_show_hidden_data.pl and verify that it reports 100b in
the PERSO_NAME framework.
Signed-off-by: Marcel de Rooy
Signed-off-by: Josef Moravec
Signed-off-by: Nick Clemens
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 13:21:24 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 11:21:24 +0000
Subject: [Koha-bugs] [Bug 18897] Koha_Elasticsearch_Indexer.t doesn' t skip
when ES module not installed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18897
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |nick at bywatersolutions.com
Status|Signed Off |Failed QA
--- Comment #7 from Nick Clemens ---
Failing per comment #6
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 13:27:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 11:27:21 +0000
Subject: [Koha-bugs] [Bug 16401] System preference staffClientBaseURL
hardcoded to 'http://'
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16401
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |nick at bywatersolutions.com
|y.org |
CC| |nick at bywatersolutions.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 13:32:11 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 11:32:11 +0000
Subject: [Koha-bugs] [Bug 16401] System preference staffClientBaseURL
hardcoded to 'http://'
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16401
Nick Clemens 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 Fri Sep 1 13:32:14 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 11:32:14 +0000
Subject: [Koha-bugs] [Bug 16401] System preference staffClientBaseURL
hardcoded to 'http://'
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16401
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #66143|0 |1
is obsolete| |
--- Comment #4 from Nick Clemens ---
Created attachment 66712
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66712&action=edit
Bug 16401: System preference staffClientBaseURL hardcoded to 'http://'
Test plan:
1) Insert some value to staffBaseURL preference, without starting http://
2) Apply the patch
3) Update database
4) Go to system preferences adminsitration, find the staffBaseURL
preference, the inserted value should be prepended with 'http://' and
the comment should be "This should be a complete URL, starting with
http:// or https://. Do not include a trailing slash in the URL. (This
must be filled in correctly for CAS, svc, and load_testing to work.)"
5) prove t/db_dependent/Auth_with_cas.t
6) prove t/db_dependent/selenium/basic_workflow.t
7) prove t/db_dependent/check_sysprefs.t
Signed-off-by: Katrin Fischer
Signed-off-by: Nick Clemens
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 13:33:37 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 11:33:37 +0000
Subject: [Koha-bugs] [Bug 19021] Inventory column sorting
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19021
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Failed QA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Sep 1 13:42:18 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 11:42:18 +0000
Subject: [Koha-bugs] [Bug 19129] Clean up templates for organisation patrons
in staff
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19129
Kyle M Hall 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 Fri Sep 1 13:42:20 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 01 Sep 2017 11:42:20 +0000
Subject: [Koha-bugs] [Bug 19129] Clean up templates for organisation patrons
in staff
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19129
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #66257|0 |1
is obsolete| |
Attachment #66258|0 |1
is obsolete| |
--- Comment #8 from Kyle M Hall