From bugzilla-daemon at bugs.koha-community.org Mon Jan 1 09:48:02 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 01 Jan 2018 08:48:02 +0000
Subject: [Koha-bugs] [Bug 19900] New: edadasdfasd
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19900
Bug ID: 19900
Summary: edadasdfasd
Change sponsored?: ---
Product: Project Infrastructure
Version: unspecified
Hardware: Macintosh
OS: Mac OS
Status: NEW
Severity: blocker
Priority: P5 - low
Component: Hea
Assignee: chris at bigballofwax.co.nz
Reporter: tinotasi at tm2mail.com
QA Contact: jonathan.druart at bugs.koha-community.org
Created attachment 70218
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70218&action=edit
bugrits
asdfasdfasdfasdfasdf
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 1 12:51:13 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 01 Jan 2018 11:51:13 +0000
Subject: [Koha-bugs] [Bug 19900] SPAM
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19900
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|edadasdfasd |SPAM
Status|NEW |RESOLVED
CC| |katrin.fischer at bsz-bw.de
Resolution|--- |INVALID
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 1 15:46:13 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 01 Jan 2018 14:46:13 +0000
Subject: [Koha-bugs] [Bug 19901] New: KOHA process high,
load average over 150
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19901
Bug ID: 19901
Summary: KOHA process high, load average over 150
Change sponsored?: ---
Product: Koha
Version: 17.11
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: OPAC
Assignee: oleonard at myacpl.org
Reporter: mhby87 at gmail.com
QA Contact: testopia at bugs.koha-community.org
Created attachment 70219
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70219&action=edit
opac-detail.pl over process.
My KOHA process is too high over 150 load average. KOHA also hang. Refer
attachment. After disable port 80 for OPAC, KOHA can work for intranet dan load
average is stable
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 1 16:13:46 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 01 Jan 2018 15:13:46 +0000
Subject: [Koha-bugs] [Bug 19901] KOHA process high, load average over 150
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19901
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |INVALID
CC| |katrin.fischer at bsz-bw.de
--- Comment #1 from Katrin Fischer ---
Hi Mohd, this is more likely to be an issue with your setup. For example your
OPAC might have been hit by one of the less nice search engines - in this case
we would recommend to add a robots.txt. Please take your problem to the mailing
list!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 1 16:24:08 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 01 Jan 2018 15:24:08 +0000
Subject: [Koha-bugs] [Bug 19901] KOHA process high, load average over 150
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19901
--- Comment #2 from Mohd Hafiz Yusoff ---
(In reply to Katrin Fischer from comment #1)
> Hi Mohd, this is more likely to be an issue with your setup. For example
> your OPAC might have been hit by one of the less nice search engines - in
> this case we would recommend to add a robots.txt. Please take your problem
> to the mailing list!
Hi Katrin, can I refer this link
https://wiki.koha-community.org/wiki/Koha_Tuning_Guide#Reduce_CPU_usage_of_webcrawlers:_robots.txt
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 1 16:53:03 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 01 Jan 2018 15:53:03 +0000
Subject: [Koha-bugs] [Bug 19901] KOHA process high, load average over 150
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19901
--- Comment #3 from Mohd Hafiz Yusoff ---
I add robots.txt but still overload.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 1 17:10:28 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 01 Jan 2018 16:10:28 +0000
Subject: [Koha-bugs] [Bug 19901] KOHA process high, load average over 150
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19901
--- Comment #4 from Katrin Fischer ---
Sorry, I am not the right person to help you further - please try the mailing
list where a lot more people will read and can help you with your question.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon Jan 1 21:27:44 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 01 Jan 2018 20:27:44 +0000
Subject: [Koha-bugs] [Bug 18783] Allow automatic claims for serials
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18783
--- Comment #6 from Katrin Fischer ---
Would be nice to see this progressing, it's often asked for!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 08:11:24 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 07:11:24 +0000
Subject: [Koha-bugs] [Bug 19902] New: Add column configuration to
bibliographic record checkouts history table
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19902
Bug ID: 19902
Summary: Add column configuration to bibliographic record
checkouts history table
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Staff Client
Assignee: koha-bugs at lists.koha-community.org
Reporter: josef.moravec at gmail.com
QA Contact: testopia at bugs.koha-community.org
CC: gmcharlt at gmail.com
--
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 Tue Jan 2 08:14:42 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 07:14:42 +0000
Subject: [Koha-bugs] [Bug 17527] Hidden settings in MARC framework don' t
work correctly on results list
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17527
--- Comment #28 from Katrin Fischer ---
Marc, could you take another look?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 08:42:07 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 07:42:07 +0000
Subject: [Koha-bugs] [Bug 18477] AR_PENDING notice does not populate values
from article_requests table
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18477
--- Comment #8 from Katrin Fischer ---
This is still quite a problem for users of article requests - can we agree on a
fix?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 08:46:41 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 07:46:41 +0000
Subject: [Koha-bugs] [Bug 19902] Add column configuration to bibliographic
record checkouts history table
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19902
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |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 Tue Jan 2 08:46:44 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 07:46:44 +0000
Subject: [Koha-bugs] [Bug 19902] Add column configuration to bibliographic
record checkouts history table
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19902
--- Comment #1 from Josef Moravec ---
Created attachment 70220
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70220&action=edit
Bug 19902: Add column settings to bibliographic record checkouts history table
Test plan:
0) Apply the patch
1) Go to administration -> Configure columns, note there is new
checkoutshistory-table in Catalogue section
2) Make some configuration in this table settings
3) Go to detail of any record -> Checkout history
4) The columns should be visible according to your configuration from
step 2
5) Play with Column visibility, to ensure it does work as it should
--
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 Tue Jan 2 08:47:13 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 07:47:13 +0000
Subject: [Koha-bugs] [Bug 19902] Add column configuration to bibliographic
record checkouts history table
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19902
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|koha-bugs at lists.koha-commun |josef.moravec at gmail.com
|ity.org |
Patch complexity|--- |Small patch
--
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 Tue Jan 2 09:27:50 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 08:27:50 +0000
Subject: [Koha-bugs] [Bug 19303] Move C4::Members::GetFirstValidEmailAddress
to Koha::Patron-> first_valid_email_address
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19303
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Patch complexity|--- |Small patch
CC| |josef.moravec at gmail.com
QA Contact|testopia at bugs.koha-communit |josef.moravec at gmail.com
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 09:28:13 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 08:28:13 +0000
Subject: [Koha-bugs] [Bug 19303] Move C4::Members::GetFirstValidEmailAddress
to Koha::Patron-> first_valid_email_address
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19303
Josef Moravec 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 Tue Jan 2 09:28:15 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 08:28:15 +0000
Subject: [Koha-bugs] [Bug 19303] Move C4::Members::GetFirstValidEmailAddress
to Koha::Patron-> first_valid_email_address
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19303
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68107|0 |1
is obsolete| |
--- Comment #4 from Josef Moravec ---
Created attachment 70221
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70221&action=edit
Bug 19303: Move C4::Members::GetFirstValidEmailAddress to
Koha::Patron->first_valid_email_address
This new Koha::Patron->first_valid_email_address already exists, it
should be called directly
Test plan:
- With a logged in user who have an email address defined, send a basket.
The email should be marked sent by this email address
- On the "Holds awaiting pickup", the email address must be displayed as well
Signed-off-by: David Bourgault
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 09:46:30 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 08:46:30 +0000
Subject: [Koha-bugs] [Bug 19304] Move C4::Members::GetNoticeEmailAddress to
Koha::Patron-> notice_email_address
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19304
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |josef.moravec at gmail.com
Patch complexity|--- |Small patch
QA Contact|testopia at bugs.koha-communit |josef.moravec at gmail.com
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 09:46:53 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 08:46:53 +0000
Subject: [Koha-bugs] [Bug 19304] Move C4::Members::GetNoticeEmailAddress to
Koha::Patron-> notice_email_address
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19304
Josef Moravec 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 Tue Jan 2 09:46:59 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 08:46:59 +0000
Subject: [Koha-bugs] [Bug 19304] Move C4::Members::GetNoticeEmailAddress to
Koha::Patron-> notice_email_address
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19304
--- Comment #6 from Josef Moravec ---
Created attachment 70223
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70223&action=edit
Bug 19304: (QA follow-up) Fix number of tests in Members.t
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 09:46:56 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 08:46:56 +0000
Subject: [Koha-bugs] [Bug 19304] Move C4::Members::GetNoticeEmailAddress to
Koha::Patron-> notice_email_address
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19304
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68111|0 |1
is obsolete| |
--- Comment #5 from Josef Moravec ---
Created attachment 70222
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70222&action=edit
Bug 19304: Move C4::Members::GetNoticeEmailAddress to
Koha::Patron->notice_email_address
This subroutine is quite trivial and can be replaced easily with a new
method of Koha::Patron
Test plan:
Overdue notices and shelf sharing must be send the to an email address,
according to the value of the pref AutoEmailPrimaryAddress
Signed-off-by: David Bourgault
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 10:33:12 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 09:33:12 +0000
Subject: [Koha-bugs] [Bug 14385] Extend OpacHiddenItems to allow specifying
exempt borrower categories
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14385
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Patch doesn't apply
CC| |josef.moravec at gmail.com
--- Comment #142 from Josef Moravec ---
Hi,
these patchset doesn't apply. So just few comments from quick reading of the
code:
You don't have to change signature of GetHiddenItemnumbers, you can ask for
patron category here... And you can than leave the $search_context as is.
Also the "opac" param in GetMarcBiblio and EmbedItemsInMarcBiblio is not
needed, you can use C4::Context->interface when you need to know the actual
interface
And some squashing would be nice here ;)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 11:28:08 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 10:28:08 +0000
Subject: [Koha-bugs] [Bug 15985] Include transacting library in fines
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15985
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |josef.moravec at gmail.com
|y.org |
CC| |josef.moravec at gmail.com
Status|Signed Off |Patch doesn't apply
--- Comment #5 from Josef Moravec ---
Please rebase.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 11:46:06 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 10:46:06 +0000
Subject: [Koha-bugs] [Bug 12365] Notes for rules
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12365
--- Comment #16 from Josef Moravec ---
Comment on attachment 67802
--> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=67802
Bug 12365 - Notes for rules
Review of attachment 67802:
--> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=12365&attachment=67802)
-----------------------------------------------------------------
::: Koha/Schema/Result/Issuingrule.pm
@@ +300,5 @@
> extra => { list => ["no", "yes", "bib_only", "item_only"] },
> is_nullable => 0,
> },
> + "note",
> + { data_type => "varchar", is_nullable => 1, size => 100 },
The Schema files changes should be generated by update_dbix_class_files.pl
script and should be in separate commit.
::: koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt
@@ +436,5 @@
> Item level holds |
> Article requests |
> Rental discount (%) |
> + Note |
> + |
The colspan parameter is not needed here and should be removed
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 11:46:28 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 10:46:28 +0000
Subject: [Koha-bugs] [Bug 12365] Notes for rules
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12365
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |josef.moravec at gmail.com
Status|Signed Off |Failed QA
QA Contact|testopia at bugs.koha-communit |josef.moravec at gmail.com
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 11:59:01 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 10:59:01 +0000
Subject: [Koha-bugs] [Bug 18183] jQuery append error related to script tags
in cloneItemBlock
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18183
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |josef.moravec at gmail.com
QA Contact|testopia at bugs.koha-communit |josef.moravec at gmail.com
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 11:59:16 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 10:59:16 +0000
Subject: [Koha-bugs] [Bug 18183] jQuery append error related to script tags
in cloneItemBlock
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18183
Josef Moravec 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 Tue Jan 2 11:59:19 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 10:59:19 +0000
Subject: [Koha-bugs] [Bug 18183] jQuery append error related to script tags
in cloneItemBlock
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18183
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #67821|0 |1
is obsolete| |
--- Comment #4 from Josef Moravec ---
Created attachment 70224
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70224&action=edit
Bug 18183: jQuery append error related to script tags in cloneItemBlock
See the detailed explanation of the error on Bugzilla.
As a workaround, we move the append of clone to outeritemblock before the
append of data (long string with item fields and script tags) to clone.
This patch also moves two vars inside the success function, since they are
not used outside of it.
Test plan:
[1] Set AcqCreateItem to receiving.
[2] Receive an order. Verify that adding, updating or deleting an item block
still works as expected.
Signed-off-by: Katrin Fischer
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 12:20:32 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 11:20:32 +0000
Subject: [Koha-bugs] [Bug 15378] Remove 'lost' items from course reserves
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15378
Josef Moravec 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 Tue Jan 2 12:20:36 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 11:20:36 +0000
Subject: [Koha-bugs] [Bug 15378] Remove 'lost' items from course reserves
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15378
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68071|0 |1
is obsolete| |
--- Comment #7 from Josef Moravec ---
Created attachment 70225
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70225&action=edit
Bug 15378: Allow removal of lost items from course reserves
It seems that this restriction is not necessary, it should be possible
to remove lost items from course reserves.
Test plan:
Create a course and add reserve with a lost item
=> Without this patch the "Remove" button is disabled and the item
cannot be removed from the course
=> With this patch applied the button is enable and the item can be
removed
Signed-off-by: S?verine QUEUNE
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 12:20:50 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 11:20:50 +0000
Subject: [Koha-bugs] [Bug 15378] Remove 'lost' items from course reserves
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15378
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Patch complexity|--- |Trivial patch
CC| |josef.moravec at gmail.com
QA Contact|testopia at bugs.koha-communit |josef.moravec at gmail.com
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 12:54:21 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 11:54:21 +0000
Subject: [Koha-bugs] [Bug 19886] Two Factor Authentication: Yubikey
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19886
Jon Knight changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |J.P.Knight at lboro.ac.uk
--- Comment #1 from Jon Knight ---
Is this something that needs to be built into Koha, or is it better to make use
of existing third party systems such as PrivacyIDEA
(https://www.privacyidea.org/) that can be used in conjuction with normal SSO
mechanisms? For example we (Loughborough University) have tested Yubikeys with
PrivacyIDEA and simpleSAMLphp IdP and it works, and we already know that Koha
can use the SAML2.0 authentication assertions from the IdP (as that's what
we're doing in production with PTFS-E). PrivacyIDEA also can also support
Google Authentication as suggested in Bug 19887.
--
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 Tue Jan 2 13:20:06 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 12:20:06 +0000
Subject: [Koha-bugs] [Bug 17047] Mana Knowledge Base : share data
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17047
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Failed QA
--- Comment #187 from Nick Clemens ---
Tried testing, seems to be some issues:
1 - Template code in subscription-add.js file breaks serial biblio search:
620 //[% IF Koha.Preference('Mana') == 1 %]
2 - When I attempt to save a serial:
Variable "$loggedinuser" is not available at
/home/vagrant/kohaclone/serials/subscription-add.pl line 371.
Variable "$template" is not available at
/home/vagrant/kohaclone/serials/subscription-add.pl line 372.
Variable "$loggedinuser" is not available at
/home/vagrant/kohaclone/serials/subscription-add.pl line 371.
Variable "$template" is not available at
/home/vagrant/kohaclone/serials/subscription-add.pl line 372.
Can't use string ("koha_login_context") as a HASH ref while "strict refs" in
use at /home/vagrant/kohaclone/Koha/Subscription/Numberpatterns.pm line 63.
3 - Blank modal when attempting to share report with no notes
4 - Internal server error when trying to share a new report with notes:
Use of uninitialized value $loggedinuser in string ne at
/home/vagrant/kohaclone/Koha/SharedContent.pm line 106.
Can't call method "first_valid_email_address" on an undefined value at
/home/vagrant/kohaclone/Koha/SharedContent.pm line 108.
I stopped testing at this point
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 13:56:30 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 12:56:30 +0000
Subject: [Koha-bugs] [Bug 17374] Make use of fields from syspref
'DefaultPatronSearchFields' in patron search fields dropdown
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17374
--- Comment #9 from Nick Clemens ---
Created attachment 70226
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70226&action=edit
Bug 17374 - Add cardnumber to list of searchable patron fields
To test:
1 - Apply patch
2 - Perform a 'cardnumber' search for patrons
3 - Verify results as expected
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 14:01:10 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 13:01:10 +0000
Subject: [Koha-bugs] [Bug 17374] Make use of fields from syspref
'DefaultPatronSearchFields' in patron search fields dropdown
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17374
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Signed Off
--- Comment #10 from Nick Clemens ---
(In reply to Katrin Fischer from comment #8)
> 1) If you leave the preference empty, the default will show. The default
> doesn't include cardnumber, I think it might be good to include it on top.
>
> 2) When using the preference, the default will still show, so there is no
> way to remove fields. Wouldn't it make more sense to use the non-empty pref
> instead of the default? For example a lot of libraries don't use the sort
> fields and could remove them from the list this way.
>
> The current behaviour seems especially confusing since the preference is not
> empty by default but includes a list of values. Deleting one of them will
> have no effect as long as the value is in the list of default.
>
> Failing for 2).
1 - Attached patch (this may be better as its own report)
2 - I think this is beyond scope here. The preference is intended to simply
control the fields searched when using a 'standard' search. Adding these the
dropdown is a convenience feature. I think removing fields is better done by
javascript for customization. The list of fields here and the list in the pref
didn't have parity before so introducing now would be a bigger change.
Moving back to QA queue for second opinion
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 15:08:18 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 14:08:18 +0000
Subject: [Koha-bugs] [Bug 19903] New: Add Koha::Patron::Restrictions
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19903
Bug ID: 19903
Summary: Add Koha::Patron::Restrictions
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: koha-bugs at lists.koha-community.org
Reporter: tomascohen at gmail.com
QA Contact: testopia at bugs.koha-community.org
We need a Koha::Object-based class to replace (the so-called)
Koha::Patron::Debarments.
--
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 Tue Jan 2 15:08:31 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 14:08:31 +0000
Subject: [Koha-bugs] [Bug 19903] Add Koha::Patron::Restrictions
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19903
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |kyle.m.hall at gmail.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 15:30:11 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 14:30:11 +0000
Subject: [Koha-bugs] [Bug 18183] jQuery append error related to script tags
in cloneItemBlock
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18183
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
Severity|enhancement |normal
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 15:45:00 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 14:45:00 +0000
Subject: [Koha-bugs] [Bug 19807] IntranetCatalogSearchPulldown doesn' t
honor IntranetNumbersPreferPhrase
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19807
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
--- Comment #4 from Jonathan Druart ---
I do not recreate the bug, and I do not understand the fix. Could you detail
please?
(In reply to Victor Grousset/tuxayo from comment #0)
> It's very likely to also affect the OPAC
Does it?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 16:01:49 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:01:49 +0000
Subject: [Koha-bugs] [Bug 18303] Release team omnibus
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18303
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |19904
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19904
[Bug 19904] Release team 18.05
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 16:01:49 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:01:49 +0000
Subject: [Koha-bugs] [Bug 19904] New: Release team 18.05
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19904
Bug ID: 19904
Summary: Release team 18.05
Change sponsored?: ---
Product: Koha
Version: unspecified
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: About
Assignee: koha-bugs at lists.koha-community.org
Reporter: jonathan.druart at bugs.koha-community.org
QA Contact: testopia at bugs.koha-community.org
Blocks: 18303
Updating about team page for 18.05
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18303
[Bug 18303] Release team omnibus
--
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 Tue Jan 2 16:02:52 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:02:52 +0000
Subject: [Koha-bugs] [Bug 19904] Release team 18.05
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19904
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |rel_18_05_candidate
--
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 Tue Jan 2 16:12:40 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:12:40 +0000
Subject: [Koha-bugs] [Bug 15378] Remove 'lost' items from course reserves
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15378
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Passed QA |Pushed to Master
--- Comment #8 from Jonathan Druart ---
Pushed to master for 18.05, thanks to everybody involved!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 16:12:46 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:12:46 +0000
Subject: [Koha-bugs] [Bug 18183] jQuery append error related to script tags
in cloneItemBlock
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18183
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Passed QA |Pushed to Master
--- Comment #5 from Jonathan Druart ---
Pushed to master for 18.05, thanks to everybody involved!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 16:12:51 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:12:51 +0000
Subject: [Koha-bugs] [Bug 19300] Move C4::Reserves::OPACItemHoldsAllowed to
the Koha namespace
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19300
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Passed QA |Pushed to Master
--- Comment #13 from Jonathan Druart ---
Pushed to master for 18.05, thanks to everybody involved!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 16:12:56 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:12:56 +0000
Subject: [Koha-bugs] [Bug 19303] Move C4::Members::GetFirstValidEmailAddress
to Koha::Patron-> first_valid_email_address
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19303
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Passed QA |Pushed to Master
--- Comment #5 from Jonathan Druart ---
Pushed to master for 18.05, thanks to everybody involved!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 16:13:02 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:13:02 +0000
Subject: [Koha-bugs] [Bug 19304] Move C4::Members::GetNoticeEmailAddress to
Koha::Patron-> notice_email_address
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19304
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Passed QA |Pushed to Master
--- Comment #7 from Jonathan Druart ---
Pushed to master for 18.05, thanks to everybody involved!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 16:13:07 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:13:07 +0000
Subject: [Koha-bugs] [Bug 19798] Returns.pl doesn't define itemnumber for
transfer-slip.
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19798
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Passed QA |Pushed to Master
--- Comment #8 from Jonathan Druart ---
Pushed to master for 18.05, thanks to everybody involved!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 16:25:40 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:25:40 +0000
Subject: [Koha-bugs] [Bug 19807] IntranetCatalogSearchPulldown doesn' t
honor IntranetNumbersPreferPhrase
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19807
--- Comment #5 from Katrin Fischer ---
It doesn't effect the OPAC from what I could tell when testing.
It just switches the indexes in the new search options pull down on the simple
search according to the pref from ,wrdl to ,phr for callnumber, copying the
existing behaviour for advanced search.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 16:26:25 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:26:25 +0000
Subject: [Koha-bugs] [Bug 19905] New: Don't append wildcard to query terms
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19905
Bug ID: 19905
Summary: Don't append wildcard to query terms
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: Searching - Elasticsearch
Assignee: koha-bugs at lists.koha-community.org
Reporter: glasklas at gmail.com
Discovered this issue while testing the search links in biblio-details. The
links are hard-coded in the template with queries not using quotes for field
values, which makes QueryBuilder::_truncate_terms append '*' to the search
term. Since wild card searches skips the analyzer-phase, most of our links does
not work (since a lot of field values has word boundaries and/or punctuation
marks, for example the query "Provider:Taschen," will be converted to
"Provider:Taschen,*" and since the indexed value for Provider is "Taschen" this
search will produce no results.
In my opinion (if I'm not misunderstanding the purpose of this code) appending
'*' to words should be removed.
--
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 Tue Jan 2 16:26:39 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:26:39 +0000
Subject: [Koha-bugs] [Bug 19905] Don't append wildcard to query terms
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19905
David Gustafsson changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|koha-bugs at lists.koha-commun |glasklas at gmail.com
|ity.org |
--
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 Tue Jan 2 16:43:21 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:43:21 +0000
Subject: [Koha-bugs] [Bug 9302] Add ability to merge patron records
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9302
Ed Veal changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |eveal at mckinneytexas.org
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 16:52:42 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:52:42 +0000
Subject: [Koha-bugs] [Bug 19905] Don't append wildcard to query terms
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19905
--- Comment #1 from David Gustafsson ---
Created attachment 70227
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70227&action=edit
Bug 19905 - Don't append wildcard to query terms
Appending '*' to query terms makes Elasticsearch skip search
analysis. This will for example cause issues with search links
in biblio details for values other than a just single term, since
the indexed value has been tokenized.
How to test:
1) Find a biblio with a field value containing word boundaries and/or
punctuation marks. For example: "au:Author, Name".
2) Perform a search using this query, this should not produce any results.
3) Apply patch.
4) Search using the same query again, it should now produce the expected
results.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 16:57:10 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:57:10 +0000
Subject: [Koha-bugs] [Bug 19905] Don't append wildcard to query terms
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19905
David Gustafsson changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |Needs Signoff
--- Comment #2 from David Gustafsson ---
I just now discovered that the link I was testing this with (Publisher, using
the query Provider:, is not quoting the value, but most (perhaps all) of
the other links are. So an alternative solution to this issue could be add
quotes in the xsl-template. Still we have received serveral complaints from
libraries failing to perform searches using unquoted field values, and though
we could tell everyone to always use quotes, it feels like Koha would provide a
better user experience if wildcard queries where not the default.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 16:58:19 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 15:58:19 +0000
Subject: [Koha-bugs] [Bug 19905] Don't append wildcard to query terms
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19905
--- Comment #3 from David Gustafsson ---
*librarians, not libraries.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 17:00:44 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 16:00:44 +0000
Subject: [Koha-bugs] [Bug 19807] IntranetCatalogSearchPulldown doesn' t
honor IntranetNumbersPreferPhrase
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19807
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Passed QA |Pushed to Master
--- Comment #6 from Jonathan Druart ---
Note to myself: the variable is not sent from all the scripts that include
search_indexes.inc.
Pushed to master for 18.05, thanks to everybody involved!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 17:08:16 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 16:08:16 +0000
Subject: [Koha-bugs] [Bug 18477] AR_PENDING notice does not populate values
from article_requests table
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18477
--- Comment #9 from Jonathan Druart ---
Created attachment 70228
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70228&action=edit
Bug 18477: Use ->store to update the timestamp
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 17:09:27 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 16:09:27 +0000
Subject: [Koha-bugs] [Bug 18477] AR_PENDING notice does not populate values
from article_requests table
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18477
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Signed Off
--- Comment #10 from Jonathan Druart ---
There is a big lack of test and add a test here will take me too much time. The
fix sounds trivial and the ->store method is obviously missing.
I am willing to push these patches as it, if QA is ok with that.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 17:11:58 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 16:11:58 +0000
Subject: [Koha-bugs] [Bug 19905] Don't append wildcard to query terms
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19905
--- Comment #4 from David Gustafsson ---
Looking through the tests I now saw there is an "QueryAutoTruncate" option that
was enabled in my instance. So turning this off will most likely fix this issue
for us, and I now realize it cannot just be removed (without also removing this
option), making this fix an incorrect one. I have not tested it out myself, but
I think there may be other issues with the current autotruncation, should not
the whole phrase be searched (the "raw" field for example) or else you will get
strange results for wildcard queries containing multiple tokens (since it is
the tokanized values that will be matched against)? But that is another
possible issue, and should perhaps not be addressed here. But for now I'm
disabling the option, and this issues should perhaps be closed.
There is still an issue with the Provider-link not being quoted (and might not
work if "QueryAutoTruncate" is enabled), but better to open another issue for
that one.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 17:13:28 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 16:13:28 +0000
Subject: [Koha-bugs] [Bug 19905] Don't append wildcard to query terms
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19905
David Gustafsson changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |In Discussion
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 17:22:26 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 16:22:26 +0000
Subject: [Koha-bugs] [Bug 19883] Add a selenium script to hit all the pages
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19883
--- Comment #4 from Jonathan Druart ---
(In reply to Marcel de Rooy from comment #3)
> Nice goal on itself. Protecting the database is obviously needed.
> Do several scripts need basic parameters at the beginning already (not only
> what is filled on a form)? So the script needs to know these; could be a
> maintenance headache?
I plan to test with GET parameters only.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 17:26:23 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 16:26:23 +0000
Subject: [Koha-bugs] [Bug 18915] Creating a checkout note (patron note)
sends an incomplete email message
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18915
--- Comment #14 from Jonathan Druart ---
QA: Please make sure bug 19772 is fixed.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 17:28:42 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 16:28:42 +0000
Subject: [Koha-bugs] [Bug 18477] AR_PENDING notice does not populate values
from article_requests table
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18477
Jon Knight changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Failed QA
CC| |J.P.Knight at lboro.ac.uk
--- Comment #11 from Jon Knight ---
(In reply to Jonathan Druart from comment #6)
> (In reply to Jonathan Druart from comment #3)
> > Please provide tests!
>
> Reading the code the change is obvious (the same call exists for other
> methods), but why do we call $self->SUPER::store instead of $self->store?
>
> Back to NQA
I just had a look at this and it would appear to call the parent's store method
because if it calls its own there's a loop: the $self->store() method in
Koha/ArticleRequest.pm can call its $self->open() method. I tested it by doing
the obvious: replacing $self-SUPER::store with $self->store and sure enough its
a memory hogging loop that crashed by dev VM's web server.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 17:38:26 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 16:38:26 +0000
Subject: [Koha-bugs] [Bug 19878] Move template JavaScript to the footer:
UNIMARC editor plugins , part 7
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19878
--- Comment #1 from Owen Leonard ---
Created attachment 70229
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70229&action=edit
Bug 19878: Move template JavaScript to the footer: UNIMARC editor plugins, part
7
This patch updates the last of the UNIMARC plugin templates so that
JavaScript is included in the footer following the same pattern as
other templates which have been modified to fix Bug 17858.
To test you do not need a UNIMARC system.
Apply the patch and for each plugin, configure a MARC subfield (e.g.
100$a) to use that plugin.
>From the MARC edit page, trigger the plugin and confirm that changes
made in the popup window are saved to the corresponding field in the
editor.
- unimarc_field_210c.tt
- unimarc_field_210c_bis.tt
- unimarc_field_225a.tt
- unimarc_field_225a_bis.tt
- unimarc_field_4XX.tt
- unimarc_field_700-4.tt
- unimarc_leader.tt
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 17:38:23 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 16:38:23 +0000
Subject: [Koha-bugs] [Bug 19878] Move template JavaScript to the footer:
UNIMARC editor plugins , part 7
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19878
Owen Leonard changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |Needs Signoff
Patch complexity|--- |Small patch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 17:39:44 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 16:39:44 +0000
Subject: [Koha-bugs] [Bug 19893] Alternative optimized indexing for
Elasticsearch
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19893
David Gustafsson changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #70202|0 |1
is obsolete| |
--- Comment #5 from David Gustafsson ---
Created attachment 70230
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70230&action=edit
Bug 19893 - Alternative optimized indexing for Elasticsearch
Add alternative optimized indexing for Elasticsearch
How to test:
1) Time a full elasticsearch re-index by running the rebuild_elastic_search.pl
with the -d flag: `koha-shell -c "time
rebuild_elastic_search.pl -d"`.
2) Enable ExperimentalElasticSearchIndexing system preference
(found under Global System preferences -> Administration -> Search Engine).
3) Time a full re-index again, it should be about twice at fast (for a
couple of thousand biblios, with fewer biblios results may be more
unpredictable).
Sponsored-by: Gothenburg University Library
--
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 Tue Jan 2 17:41:42 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 16:41:42 +0000
Subject: [Koha-bugs] [Bug 19893] Alternative optimized indexing for
Elasticsearch
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19893
--- Comment #6 from David Gustafsson ---
Fixed an instance of hard-coded Elasticsearch mapping document type in
update_mappings().
--
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 Tue Jan 2 18:03:07 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 17:03:07 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |In Discussion
--- Comment #13 from Jonathan Druart ---
Hi David,
It looks like you find a good place where we can improve speed performance,
good catch.
However it is like we are going backward reintroducing raw SQL queries. I would
prefer to avoid such changes.
Did you try to make a mix of both approaches, like: generate only 1 query with
my $items = Koha::Items->search({ -in => $barcodes }); in place of your my
$items = C4::Items::GetItems($itemnumbers);
I guess we will need to call $items->unblessed and an interesting addition
would be to fill itype according to ->effective_itemtype (so perf will be
impacted if not item-level_itypes, like in your patch). Does it make sense?
Something else, did you make sure serials will not be impacted? GetItem fetches
serialseq and publisheddate. At first glance it looks safe but we need to be
sure.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 18:27:28 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 17:27:28 +0000
Subject: [Koha-bugs] [Bug 19816] output_pref must raised an exception if
called with dateformat => rfc3339 and > dateonly => 1
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19816
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |INVALID
--- Comment #1 from Tom?s Cohen Arazi ---
Even if the RFC doesn't target date-only representations, full-date is proposed
to be used in OpenAPI, so we should support it.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 18:37:35 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 17:37:35 +0000
Subject: [Koha-bugs] [Bug 18477] AR_PENDING notice does not populate values
from article_requests table
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18477
--- Comment #12 from Jonathan Druart ---
Created attachment 70231
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70231&action=edit
Bug 18477: Add tests
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 18:38:12 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 17:38:12 +0000
Subject: [Koha-bugs] [Bug 18477] AR_PENDING notice does not populate values
from article_requests table
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18477
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Needs Signoff
--- Comment #13 from Jonathan Druart ---
Indeed Jon, so we need tests :) Thanks for the catch.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 18:38:26 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 17:38:26 +0000
Subject: [Koha-bugs] [Bug 18477] AR_PENDING notice does not populate values
from article_requests table
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18477
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #70228|0 |1
is obsolete| |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 18:59:04 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 17:59:04 +0000
Subject: [Koha-bugs] [Bug 19890] authorised value branches is not visible
from dropdown in Admin
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19890
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
--- Comment #1 from Jonathan Druart ---
There are 3 authorised value "special categories": branches, itemtypes,
cn_source.
They are used to link a MARC subfield to a branchcode, itemtype or
classification source. They are not proper authorised values (in the sense of
the authorised_values table), but are needed to restrict the input.
We could add them to the category dropdown list and remove the ability to add
value to them (display a friendly message instead to explain the situation).
--
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 Tue Jan 2 18:59:18 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 17:59:18 +0000
Subject: [Koha-bugs] [Bug 19890] authorised value branches is not visible
from dropdown in Admin
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19890
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Version|16.11 |master
--
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 Tue Jan 2 19:11:33 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 18:11:33 +0000
Subject: [Koha-bugs] [Bug 19898] Koha 17.11 - Upload Local Cover Image
Failed - " Cannot process file as an image"
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19898
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
--- Comment #2 from Jonathan Druart ---
The relevant code of the warning is:
1084 my $branches = { map { $_->branchcode => $_->unblessed
} Koha::Libraries->search };
1085 if ( $type ne 'opac' and
C4::Context->boolean_preference('AutoLocation') ) {
1086
1087 # we have to check they are coming from the right
ip range
1088 my $domain =
$branches->{$branchcode}->{'branchip'};
1089 $domain =~ s|\.\*||g;
1090 if ( $ip !~ /^$domain/ ) {
1091 $loggedin = 0;
1092 $cookie = $query->cookie(
1093 -name => 'CGISESSID',
1094 -value => '',
1095 -HttpOnly => 1
1096 );
1097 $info{'wrongip'} = 1;
1098 }
1099 }
It says that you have AutoLocation turned on and the "branchip" field of the
branches table does not match your IP. It also says that you are using a non
superlibrarian account.
Could you confirm this?
If the IP does not match, it is expected that you are rejected, but in that
case it should be the same behaviour with other scripts.
--
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 Tue Jan 2 19:30:08 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 18:30:08 +0000
Subject: [Koha-bugs] [Bug 6918] can't place holds on 'on order' items with
AllowOnShelfHolds off
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6918
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Needs Signoff
--
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 Tue Jan 2 19:30:12 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 18:30:12 +0000
Subject: [Koha-bugs] [Bug 6918] can't place holds on 'on order' items with
AllowOnShelfHolds off
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6918
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68934|0 |1
is obsolete| |
--- Comment #33 from Kyle M Hall ---
Created attachment 70232
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70232&action=edit
Bug 6918 - can't place holds on 'on order' items with AllowOnShelfHolds off
Test Plan:
1) Apply patch
2) Turn off AllowOnShelfHolds
3) Create a bib with one item, mark the item as 'on order'
4) Attempt to place a hold on the item, you should now be able to do so
--
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 Tue Jan 2 19:45:51 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 18:45:51 +0000
Subject: [Koha-bugs] [Bug 18593] Suggestions aren' t updated when one biblio
is merged with another
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18593
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |Needs Signoff
--
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 Tue Jan 2 19:45:54 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 18:45:54 +0000
Subject: [Koha-bugs] [Bug 18593] Suggestions aren' t updated when one biblio
is merged with another
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18593
--- Comment #5 from Jonathan Druart ---
Created attachment 70233
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70233&action=edit
Bug 18593: Move suggestions when bibliographic records are merged
When bibliographic records are merged, the suggestions.biblionumber
field should be replaced with the new record.
Ideally we should have kept an history of the merges, to know what was
the original record. Now we cannot fix the broken links.
Test plan:
1) Create a suggestion in OPAC
2) Accept the suggestion in staff
3) Add order from this suggestion
4) Go to cataloguing search and search for the record and another
5) Merge the suggestion record with the catalog record - catalog record wins
(should be the more common case when a patron suggests something that already
exists)
6) Verify that after merging, the new title displays in the acquisition record
7) Verify that the suggestion info no longer displays in basket
>From Katrin's test plan:
8) Verify that the new title doesn't display in suggestions. The old title will
still show as suggested (not updated)
=> This is the title from suggestions.title, I think it makes sense to show
this one.
Maybe we should open a new bug report to improve the display of the suggestion,
and link to the bibliographic record
--
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 Tue Jan 2 19:46:15 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 18:46:15 +0000
Subject: [Koha-bugs] [Bug 18593] Suggestions aren' t updated when one biblio
is merged with another
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18593
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|koha-bugs at lists.koha-commun |jonathan.druart at bugs.koha-c
|ity.org |ommunity.org
--
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 Tue Jan 2 20:19:33 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 19:19:33 +0000
Subject: [Koha-bugs] [Bug 17858] Omnibus: Move staff client JavaScript to
the footer
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17858
Owen Leonard changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |19906
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19906
[Bug 19906] Move template JavaScript to the footer: NORMARC editor plugins
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 20:19:33 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 19:19:33 +0000
Subject: [Koha-bugs] [Bug 19906] New: Move template JavaScript to the
footer: NORMARC editor plugins
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19906
Bug ID: 19906
Summary: Move template JavaScript to the footer: NORMARC editor
plugins
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: ASSIGNED
Severity: enhancement
Priority: P5 - low
Component: Templates
Assignee: oleonard at myacpl.org
Reporter: oleonard at myacpl.org
QA Contact: testopia at bugs.koha-community.org
CC: testopia at bugs.koha-community.org
Blocks: 17858
NORMARC plugin templates already have JavaScript at the bottom of the template
instead of the top, but they should still be modified to be consistent with
other templates which have been updated as part of Bug 17858.
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17858
[Bug 17858] Omnibus: Move staff client JavaScript to the footer
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 20:58:21 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 19:58:21 +0000
Subject: [Koha-bugs] [Bug 19906] Move template JavaScript to the footer:
NORMARC editor plugins
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19906
--- Comment #1 from Owen Leonard ---
Created attachment 70234
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70234&action=edit
Bug 19906: Move template JavaScript to the footer: NORMARC editor plugins
This patch updates NORMARC plugin templates so that JavaScript is
included in the footer following the same pattern as other
templates which have been modified to fix Bug 17858.
To test you do not need a NORMARC system.
Apply the patch and for each plugin, configure a MARC subfield (e.g.
100$a) to use that plugin.
>From the MARC edit page, trigger the plugin and confirm that changes
made in the popup window are saved to the corresponding field in the
editor.
- normarc_field_007.tt
- normarc_field_008.tt
- normarc_leader.tt
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 20:58:18 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 19:58:18 +0000
Subject: [Koha-bugs] [Bug 19906] Move template JavaScript to the footer:
NORMARC editor plugins
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19906
Owen Leonard changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |Needs Signoff
Patch complexity|--- |Small patch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 20:58:47 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 19:58:47 +0000
Subject: [Koha-bugs] [Bug 19641] Move patron templates JavaScript to the
footer
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19641
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |josef.moravec at gmail.com
|y.org |
CC| |josef.moravec at gmail.com
Status|Signed Off |Failed QA
--- Comment #4 from Josef Moravec ---
Hi Owen, looks good, but few thinks:
So, if I want to update child to adult, and have more patron categories for
adults, than I should see the form for choosing one of them, but I get blank
page...
Also the javascript function confirm_updatechild is defined twice: in
members-menu.js and update-child.tt.
Few buttons in patron toolbar on checkouts screen does not work -
circulation.tt is not part of this patch and so does not include
members-menu.js. I think you will fix this bug 19728, true?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 21:09:58 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 20:09:58 +0000
Subject: [Koha-bugs] [Bug 19682] Move JS to the footer: Two patron-related
tools
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19682
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |josef.moravec at gmail.com
|y.org |
CC| |josef.moravec at gmail.com
Status|Signed Off |Failed QA
--- Comment #6 from Josef Moravec ---
The datepicker in import patrons tool is shown only by dateofbirth field.
Should be also on dateexpiry and dateenrolled.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 22:03:27 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 21:03:27 +0000
Subject: [Koha-bugs] [Bug 11046] Better handling of uncertain years for
publicationyear/ copyrightdate
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11046
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |josef.moravec at gmail.com
|y.org |
CC| |josef.moravec at gmail.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 22:03:48 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 21:03:48 +0000
Subject: [Koha-bugs] [Bug 11046] Better handling of uncertain years for
publicationyear/ copyrightdate
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11046
Josef Moravec 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 Tue Jan 2 22:04:01 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 21:04:01 +0000
Subject: [Koha-bugs] [Bug 11046] Better handling of uncertain years for
publicationyear/ copyrightdate
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11046
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68349|0 |1
is obsolete| |
Attachment #68350|0 |1
is obsolete| |
--- Comment #9 from Josef Moravec ---
Created attachment 70235
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70235&action=edit
Bug 11046: Better handling of uncertain years for publicationyear
This patch makes it possible that uncertain year like 18.. or 197x are
converted to 1800 or 1970 in Koha field copyrightdate (MARC21) or
publicationyear (UNIMARC). (The corresponding MARC record will not be
changed obviously.)
This change will allow for better results when sorting search results or
list contents on copyrightdate. Currently, things like 18.. will sort
together with zero.
Note: The regex now allows four possible uncertain year markers: x or X,
question mark or dot.
Test plan:
[1] Run t/db_dependent/Biblio/TransformMarcToKoha.t
[2] Edit a biblio record. Save 18.. into 260c. Check biblio.copyrightdate.
Signed-off-by: Marcel de Rooy
Followed test plan, patch worked as described, it also passed QA test
tool
Signed-off-by: Alex Buckley
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 22:04:06 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 21:04:06 +0000
Subject: [Koha-bugs] [Bug 11046] Better handling of uncertain years for
publicationyear/ copyrightdate
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11046
--- Comment #10 from Josef Moravec ---
Created attachment 70236
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70236&action=edit
Bug 11046: Add the form YYY-? for uncertain years
This form occurred in Dutch ISBD rules.
The question mark should follow the hyphen(s).
Test plan:
Run t/db_dependent/Biblio/TransformMarcToKoha.t
Signed-off-by: Marcel de Rooy
TransformMarcToKoha tests passed. Also this patch passed QA test tool
Signed-off-by: Alex Buckley
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 22:18:26 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 21:18:26 +0000
Subject: [Koha-bugs] [Bug 18417] Advanced Editor - Rancor - add shortcuts
for copyright symbols (C) (P)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18417
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68509|0 |1
is obsolete| |
Attachment #68510|0 |1
is obsolete| |
--- Comment #8 from Josef Moravec ---
Created attachment 70237
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70237&action=edit
Bug 18417 - Advanced Editor - Rancor - add shortcuts for copyright symbols (C)
(P)
To test:
1 - Apply patch
2 - Open advanced cataloging editor
3 - Try using Alt+P and Alt+C and note symbols are added
4 - Save record and ensure symbols are saved
Signed-off-by: Mark Tompsett
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 22:18:31 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 21:18:31 +0000
Subject: [Koha-bugs] [Bug 18417] Advanced Editor - Rancor - add shortcuts
for copyright symbols (C) (P)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18417
--- Comment #9 from Josef Moravec ---
Created attachment 70238
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70238&action=edit
Bug 18417: (follow-up) Document new shortcuts in dropdown
Signed-off-by: Mark Tompsett
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 22:18:21 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 21:18:21 +0000
Subject: [Koha-bugs] [Bug 18417] Advanced Editor - Rancor - add shortcuts
for copyright symbols (C) (P)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18417
Josef Moravec 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 Tue Jan 2 22:18:47 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 21:18:47 +0000
Subject: [Koha-bugs] [Bug 18417] Advanced Editor - Rancor - add shortcuts
for copyright symbols (C) (P)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18417
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |josef.moravec at gmail.com
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 22:44:26 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 21:44:26 +0000
Subject: [Koha-bugs] [Bug 19801] Internal note added to the detail's user
page
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19801
--- Comment #5 from Liz Rea ---
Created attachment 70239
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70239&action=edit
Bug 19801 - Fixes for QA
- Fixes indentation
- changes messages to patron_messages (even though it's not like that on the
circulation page.)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 22:44:23 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 21:44:23 +0000
Subject: [Koha-bugs] [Bug 19801] Internal note added to the detail's user
page
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19801
Liz Rea changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 22:46:54 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 21:46:54 +0000
Subject: [Koha-bugs] [Bug 16782] Display report URL in staff client report
interface
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16782
--- Comment #18 from Liz Rea ---
Thankfully this still applies. :)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Tue Jan 2 22:50:55 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 02 Jan 2018 21:50:55 +0000
Subject: [Koha-bugs] [Bug 13376] Include a user-editable area/ page for
terms and conditions of use in the opac
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13376
Liz Rea changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |Academy
--- Comment #5 from Liz Rea ---
This bug got derailed, I still think the original feature proposal is a good
one.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 01:21:05 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 00:21:05 +0000
Subject: [Koha-bugs] [Bug 14385] Extend OpacHiddenItems to allow specifying
exempt borrower categories
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14385
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Patch doesn't apply |ASSIGNED
--- Comment #143 from M. Tompsett ---
(In reply to Josef Moravec from comment #142)
> these patchset doesn't apply.
Thanks for looking and noticing this.
> So just few comments from quick reading of the
> code:
>
> You don't have to change signature of GetHiddenItemnumbers, you can ask for
> patron category here... And you can than leave the $search_context as is.
I can't recall the reason off the top of my head, and even after rebasing this
locally -- I have yet to test and put it back up. But I've got a feeling it had
something to so with the nesting of some search functions which were not easily
determined to be opac or intranet otherwise.
> Also the "opac" param in GetMarcBiblio and EmbedItemsInMarcBiblio is not
> needed, you can use C4::Context->interface when you need to know the
> actual interface.
It is better to pass it in my opinion to refactor the parameters list, so that
test functions don't require special mocking to get the C4::Context function to
return the correct value in different cases (like shell script utilities). And
this makes maintenance for other changes in the future easier for others, and
brings the code up to preferred coding style
(https://wiki.koha-community.org/wiki/Coding_Guidelines#PERL16:_Hashrefs_should_be_used_as_arguments)
> And some squashing would be nice here ;)
I was tweaking, testing, fixing, repeating multiple times. Also, smaller
patches are easier to test individually. I will squash as much as possible. I
do, however, not want to lose the fact that Aleisha Amohia put in a massive
effort which helped me see how to solve my C4::Search mess problem that I had
hit while working on originally in bug 10589.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 01:59:16 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 00:59:16 +0000
Subject: [Koha-bugs] [Bug 14385] Extend OpacHiddenItems to allow specifying
exempt borrower categories
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14385
--- Comment #144 from Liz Rea ---
a+ preserving attribution. :)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 02:27:39 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 01:27:39 +0000
Subject: [Koha-bugs] [Bug 19890] authorised value branches is not visible
from dropdown in Admin
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19890
Hugh Rundle changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |koha at hughr.me
--- Comment #2 from Hugh Rundle ---
Yep that would be perfect. That way it's clear they already exist, and also
that authorised_values is not the place to change them. Something like
"[category name] is listed here for information only. To change or add values
for [category name] use [appropriate place].
--
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 Wed Jan 3 02:42:06 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 01:42:06 +0000
Subject: [Koha-bugs] [Bug 19837] Add multiple patrons to a list by cardnumber
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19837
--- Comment #3 from Nick Clemens ---
Created attachment 70240
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70240&action=edit
Bug 19837 - Add error feedback for duplicatred or unfound cardnumbers
To test:
1 - Enter some patrons on a list
2 - 'Enter multiple cardnumbers'
3 - Make sure to add some that are already in list, some that don't
exist, and some new patrons
4 - Verify results are as expected with warnings
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 02:43:11 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 01:43:11 +0000
Subject: [Koha-bugs] [Bug 19837] Add multiple patrons to a list by cardnumber
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19837
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |nick at bywatersolutions.com
--- Comment #4 from Nick Clemens ---
I added some error feedback, it could certainly be made prettier, take a look
please Owen?
Ideally we would be able to add by borrowernumber as well, but I think that is
for another bug
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 02:46:05 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 01:46:05 +0000
Subject: [Koha-bugs] [Bug 19898] Koha 17.11 - Upload Local Cover Image
Failed - " Cannot process file as an image"
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19898
--- Comment #3 from Craig ---
(In reply to Jonathan Druart from comment #2)
> The relevant code of the warning is:
>
> 1084 my $branches = { map { $_->branchcode =>
> $_->unblessed } Koha::Libraries->search };
> 1085 if ( $type ne 'opac' and
> C4::Context->boolean_preference('AutoLocation') ) {
> 1086
> 1087 # we have to check they are coming from the
> right ip range
> 1088 my $domain =
> $branches->{$branchcode}->{'branchip'};
> 1089 $domain =~ s|\.\*||g;
> 1090 if ( $ip !~ /^$domain/ ) {
> 1091 $loggedin = 0;
> 1092 $cookie = $query->cookie(
> 1093 -name => 'CGISESSID',
> 1094 -value => '',
> 1095 -HttpOnly => 1
> 1096 );
> 1097 $info{'wrongip'} = 1;
> 1098 }
> 1099 }
>
>
> It says that you have AutoLocation turned on and the "branchip" field of the
> branches table does not match your IP. It also says that you are using a non
> superlibrarian account.
>
> Could you confirm this?
> If the IP does not match, it is expected that you are rejected, but in that
> case it should be the same behaviour with other scripts.
Hi Jonathan,
Thanks for this. I am indeed using a superlibrarian account, so I'm surprised
that it is reporting otherwise. I have only one patron in the system, me, and
it was initially setup as superlibrarian staff. You are correct about
Autolocation, it was turned on. But I have since disabled it. The IP address
field in the bramches table is blank.
Unfortunately, after turning off Autolocation, restarting Koha, and restarting
memcache, the error still persists.
--
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 Wed Jan 3 03:29:58 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 02:29:58 +0000
Subject: [Koha-bugs] [Bug 16804] Searching can be broken by search terms
containing semicolons in XSLT
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16804
Liz Rea changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 03:30:02 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 02:30:02 +0000
Subject: [Koha-bugs] [Bug 16804] Searching can be broken by search terms
containing semicolons in XSLT
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16804
--- Comment #6 from Liz Rea ---
Created attachment 70241
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70241&action=edit
Bug 16804 - semicolons break series search in 490
To test:
* have OPACSuppression on
* have a record with a 490 field a la:
490 _a something ;
_v 123
* go to the opac and search for this record
* click the Series link from the detail
Without this patch, there will be no results.
With this patch, you will get relevant results (possibly the same record back)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 03:31:05 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 02:31:05 +0000
Subject: [Koha-bugs] [Bug 16804] Searching can be broken by search terms
containing semicolons in XSLT
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16804
Liz Rea changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Failed QA
--- Comment #7 from Liz Rea ---
Hi Katrin,
I put a chopPunctuation on the href attribute, it'll chop of whatever ending
punctuation people feel inclined to put there. What do you think?
Cheers,
Liz
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 03:32:14 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 02:32:14 +0000
Subject: [Koha-bugs] [Bug 16804] Searching can be broken by search terms
containing semicolons in XSLT
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16804
Liz Rea changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #67622|0 |1
is obsolete| |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 03:47:57 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 02:47:57 +0000
Subject: [Koha-bugs] [Bug 16804] Searching can be broken by search terms
containing semicolons in XSLT
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16804
--- Comment #8 from Liz Rea ---
well not *any*, specifically these:
.:,;/ '
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 03:48:05 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 02:48:05 +0000
Subject: [Koha-bugs] [Bug 16804] Searching can be broken by search terms
containing semicolons in XSLT
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16804
Liz Rea changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 10:41:27 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 09:41:27 +0000
Subject: [Koha-bugs] [Bug 19907] New: Email validation on patron add/edit
not working
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19907
Bug ID: 19907
Summary: Email validation on patron add/edit not working
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: Patrons
Assignee: koha-bugs at lists.koha-community.org
Reporter: katrin.fischer at bsz-bw.de
QA Contact: testopia at bugs.koha-community.org
CC: gmcharlt at gmail.com, kyle.m.hall at gmail.com
When entering a new patron or editing an existing one, the email address should
be validated while typing. This is a regression to former behaviour.
The validation still works as expected in other places like for library email
in administration > libraries.
--
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 Wed Jan 3 10:48:45 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 09:48:45 +0000
Subject: [Koha-bugs] [Bug 19908] New: Password should not be mandatory
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19908
Bug ID: 19908
Summary: Password should not be mandatory
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: critical
Priority: P5 - low
Component: Patrons
Assignee: koha-bugs at lists.koha-community.org
Reporter: katrin.fischer at bsz-bw.de
QA Contact: testopia at bugs.koha-community.org
CC: gmcharlt at gmail.com, kyle.m.hall at gmail.com
When password is not set as a mandatory patron field, it should not be
required.
To test:
- make sure BorrowerMandatoryField is empty and doesn't list password
- add a new patron, leave password fields empty
- try to save - you will be forced to enter a password with the field now being
highlighted as required.
I have also tried changing RequireStrongPassword and emptying
minPasswordLength.
Often libraries will leave password and username empty for patrons that won't
ever use the OPAC. For example when they are not using the OPAC functionalities
in general or for internal users. Making the password mandatory is a regression
to former behaviour.
--
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 Wed Jan 3 10:49:46 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 09:49:46 +0000
Subject: [Koha-bugs] [Bug 19908] Password should not be mandatory
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19908
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |josef.moravec at gmail.com
Depends on| |18298
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18298
[Bug 18298] Allow enforcing password complexity
--
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 Wed Jan 3 10:49:46 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 09:49:46 +0000
Subject: [Koha-bugs] [Bug 18298] Allow enforcing password complexity
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18298
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |19908
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19908
[Bug 19908] Password should not be mandatory
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 12:05:55 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 11:05:55 +0000
Subject: [Koha-bugs] [Bug 19908] Password should not be mandatory
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19908
--- Comment #1 from Josef Moravec ---
Created attachment 70242
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70242&action=edit
Bug 19908: Password fields in edit/add patron form should respect
BorrowerMandatoryField preference
Test plan:
0) Do not apply the patch, note the password field is always required
1) Apply the patch
2) Try to add and edit patron with and without "password" in
BorrowerMandatoryField, it should always respect this setting
3) Use "Change password" button in patron toolbar, the password field
should be never required here - when leaved blank, the password is
unchanged
4) Play with minPasswordLength and RequireStrongPassword preferences,
to ensure they work as expected
--
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 Wed Jan 3 12:05:53 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 11:05:53 +0000
Subject: [Koha-bugs] [Bug 19908] Password should not be mandatory
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19908
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |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 Wed Jan 3 12:06:24 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 11:06:24 +0000
Subject: [Koha-bugs] [Bug 19908] Password should not be mandatory
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19908
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|koha-bugs at lists.koha-commun |josef.moravec at gmail.com
|ity.org |
--
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 Wed Jan 3 12:08:50 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 11:08:50 +0000
Subject: [Koha-bugs] [Bug 19907] Email validation on patron add/edit not
working
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19907
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|koha-bugs at lists.koha-commun |josef.moravec at gmail.com
|ity.org |
CC| |josef.moravec at gmail.com
Status|NEW |ASSIGNED
Depends on| |19908
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19908
[Bug 19908] Password should not be mandatory
--
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 Wed Jan 3 12:08:50 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 11:08:50 +0000
Subject: [Koha-bugs] [Bug 19908] Password should not be mandatory
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19908
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |19907
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19907
[Bug 19907] Email validation on patron add/edit not working
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 12:14:29 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 11:14:29 +0000
Subject: [Koha-bugs] [Bug 19909] New: Show attributes in patron search
results
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19909
Bug ID: 19909
Summary: Show attributes in patron search results
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: ASSIGNED
Severity: enhancement
Priority: P5 - low
Component: Patrons
Assignee: julian.maurice at biblibre.com
Reporter: julian.maurice at biblibre.com
QA Contact: testopia at bugs.koha-community.org
CC: gmcharlt at gmail.com, kyle.m.hall at gmail.com
This will add a new column (hidden by default) in the results table to
displays all attributes associated to the corresponding patron.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 12:15:01 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 11:15:01 +0000
Subject: [Koha-bugs] [Bug 19909] Show attributes in patron search results
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19909
--- Comment #1 from Julian Maurice ---
Created attachment 70243
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70243&action=edit
Bug 19909: Show attributes in patron search results
Add a new column (hidden by default) in the results table which
displays all attributes associated to the corresponding patron.
New subroutine Koha::Patron::Attribute::display_value that
returns the authorised value description and defaults to the raw value
if the attribute's type is not associated to an authorised value
category
Test plan:
1. Create some patron attribute types, with and without an associated
authorised value category
2. Choose an existing patron and set a value for each attribute
3. Go to patron search and do a search that will return your patron (and
some others to avoid the redirection)
4. Show column 'Extended attributes' by clicking on 'Column visibility'
button
5. Note that all attributes are displayed correctly
6. prove t/db_dependent/Koha/Patron/Attributes.t
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 12:15:11 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 11:15:11 +0000
Subject: [Koha-bugs] [Bug 19909] Show attributes in patron search results
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19909
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 12:23:34 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 11:23:34 +0000
Subject: [Koha-bugs] [Bug 19698] "Add message" for patron don' t use OPAC
description if message type is OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19698
Sophie MEYNIEUX changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|INVALID |---
Status|RESOLVED |REOPENED
--- Comment #2 from Sophie MEYNIEUX ---
Yes they are displayed in the right place but not with the right description
--
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 Wed Jan 3 12:24:14 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 11:24:14 +0000
Subject: [Koha-bugs] [Bug 19907] Email validation on patron add/edit not
working
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19907
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 12:24:17 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 11:24:17 +0000
Subject: [Koha-bugs] [Bug 19907] Email validation on patron add/edit not
working
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19907
--- Comment #1 from Josef Moravec ---
Created attachment 70244
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70244&action=edit
Bug 19907: Fix email validation in patron edit form
Test plan:
0) Confirm that email validation is not working in add/edit patron form
1) Apply the patch
2) Edit/add patron, the e-mail validation should be working now
3) Ensure the password validation is still working (use test plan from
bug 19908)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 13:23:13 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 12:23:13 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #14 from David Gustafsson ---
Hi! I'm not sure I understand, but where you thinking of skipping the pure-SQL
optimization but keeping the one loading multiple items at once using search
instead of find and then unblessing? Unfortunately that does not make that much
of a difference it seems.
I tried benchmarking with a biblio with 10 items (which is pretty rare, think
most of our biblios has only 1 or 2 items):
Loading each item separately took 372 seconds.
Loading multiple (10) items at once took 47 seconds: This is not bad (about 8
times faster, for 10 items, which is to be expected since results in 1/10 of
DBIx calls). Unfortunately since most libraries probably only have a couple of
items per biblio, averaging below 2, in practice the speedup is probably below
2 times faster.
With the pure SQL variant loading multiple items it took 7.4 seconds.
All benchmarks where performed with item-level_types = 1, and loaded the items
20000 times.
I tried to make sure serials where not affected by adjusting the code for the
fethching of multiple items, and all tests passes (not sure if there are tests
for serials), but I could have another look at this.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 13:30:54 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 12:30:54 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #15 from Tom?s Cohen Arazi ---
(In reply to David Gustafsson from comment #14)
> Hi! I'm not sure I understand, but where you thinking of skipping the
> pure-SQL optimization but keeping the one loading multiple items at once
> using search instead of find and then unblessing? Unfortunately that does
> not make that much of a difference it seems.
DBIx::Class is the same as using plain DBI. So the speed gain doesn't come from
the plain SQL (you could write the same query using the SQL::Abstract syntax
DBIC uses), but from the fact that you are using a more fine-tuned approach on
the query, to get all data at the same time.
What I think Jonathan is saying, is that we need to solve this in a way that
doesn't break our object system for the rest of the functionalities. I will try
to think about it today too.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 13:49:45 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 12:49:45 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #16 from David Gustafsson ---
Just to clarify, the performance gain comes from using DBI directly.
DBIx::Abstract could potentially be faster then DBIx::Class::ResultSet::search,
but in my opinion not much is gained by the abstraction for such a simple at
static query.
If you are saying that the speed gain mainly comes from loading multiple items
at once and not the low-level SQL, that is actually not the case as
demonstrated by the previous benchmark.
Loading multiple items with search (10 at the time) takes 50 seconds.
Loading multiple items (10 at the time) with pure-SQL 7.4 seconds.
And as previously noted, 10 items is pretty rare, so in practice the code using
search will produce an even worse result.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 14:00:43 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 13:00:43 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #17 from Tom?s Cohen Arazi ---
(In reply to David Gustafsson from comment #16)
> Just to clarify, the performance gain comes from using DBI directly.
> DBIx::Abstract could potentially be faster then
> DBIx::Class::ResultSet::search, but in my opinion not much is gained by the
> abstraction for such a simple at static query.
>
> If you are saying that the speed gain mainly comes from loading multiple
> items at once and not the low-level SQL, that is actually not the case as
> demonstrated by the previous benchmark.
>
> Loading multiple items with search (10 at the time) takes 50 seconds.
>
> Loading multiple items (10 at the time) with pure-SQL 7.4 seconds.
>
> And as previously noted, 10 items is pretty rare, so in practice the code
> using search will produce an even worse result.
Can you profile it using ntyprof? I bet the bottleneck comes from
effective_itemtype when unblessing.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 14:17:59 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 13:17:59 +0000
Subject: [Koha-bugs] [Bug 18497] Downloading a report passes the constructed
SQL as a parameter
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18497
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Patch complexity|--- |Small patch
Status|NEW |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 14:18:01 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 13:18:01 +0000
Subject: [Koha-bugs] [Bug 18497] Downloading a report passes the constructed
SQL as a parameter
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18497
--- Comment #4 from Nick Clemens ---
Created attachment 70245
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70245&action=edit
Bug 18947 - Use report id to retrieve saved SQL instead of passing param
This patch takes some of the code when executing report and moves it to
a sub to be reused when downloading
To test:
1 - Run some very long report (see comment #1)
2 - Try to download, erk!
3 - Apply patch
4 - Run report, results hould not have changed
5 - Try to download, success!
6 - Ensure reports work as before
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 14:18:25 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 13:18:25 +0000
Subject: [Koha-bugs] [Bug 18497] Downloading a report passes the constructed
SQL as a parameter
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18497
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|gmcharlt at gmail.com |nick at bywatersolutions.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 14:18:53 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 13:18:53 +0000
Subject: [Koha-bugs] [Bug 19803] SQL report export can fail HTTP 414
Request-URI Too Large
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19803
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |DUPLICATE
CC| |nick at bywatersolutions.com
--- Comment #1 from Nick Clemens ---
*** This bug has been marked as a duplicate of bug 18497 ***
--
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 Wed Jan 3 14:18:53 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 13:18:53 +0000
Subject: [Koha-bugs] [Bug 18497] Downloading a report passes the constructed
SQL as a parameter
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18497
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |fridolin.somers at biblibre.co
| |m
--- Comment #5 from Nick Clemens ---
*** Bug 19803 has been marked as a duplicate of this bug. ***
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 14:33:14 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 13:33:14 +0000
Subject: [Koha-bugs] [Bug 19903] Add Koha::Patron::Restrictions
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19903
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |josef.moravec at gmail.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 14:39:16 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 13:39:16 +0000
Subject: [Koha-bugs] [Bug 19456] Some pages title tag contains html
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19456
Fridolin SOMERS changed:
What |Removed |Added
----------------------------------------------------------------------------
Patch complexity|--- |Small patch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 14:41:55 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 13:41:55 +0000
Subject: [Koha-bugs] [Bug 19456] Some pages title tag contains html
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19456
Fridolin SOMERS changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|Pushed to Stable |RESOLVED
--- Comment #16 from Fridolin SOMERS ---
Ah the patch does not apply easily.
I it not a major problem, I choose not to backport to 17.05.x.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 14:41:56 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 13:41:56 +0000
Subject: [Koha-bugs] [Bug 18403] Hide patron information if not part of the
logged in user library group
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18403
Bug 18403 depends on bug 19456, which changed state.
Bug 19456 Summary: Some pages title tag contains html
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19456
What |Removed |Added
----------------------------------------------------------------------------
Status|Pushed to Stable |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 14:43:45 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 13:43:45 +0000
Subject: [Koha-bugs] [Bug 19602] Add usage statistics link to administration
sidebar menu
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19602
Fridolin SOMERS changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |fridolin.somers at biblibre.co
| |m
--- Comment #7 from Fridolin SOMERS ---
Pushed to 17.05.x, will be in v17.05.08
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 14:58:52 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 13:58:52 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #18 from David Gustafsson ---
When item-level_itypes = 0, the code run is pretty similar to master with patch
applied, the only difference being ->search is used to fetch multiple items
instead of ->find for each one. But this only results in a pretty small
performance gain as demonstrated by the above benchmarks.
So the interesting case to benchmark is when item-level_itypes = 1 (which also
seems to be the most commonly used setting), and the NYTProf screenshots
attached should clearly show that DBIx::Class::ResultSet::find (or
DBIx::Class::ResultSet::search which has similar performance) is the culprit.
But I can generate two new ones for DBIx::Class::ResultSet::search with
multiple items (10 like in the previous example) to show that the same is true
for ->search when loading multiple items.
effective_itemtype() is only expensive when item-level_itypes = 0 the biblio is
fetched though a relation, but this fix is not designed to address the case
when item-level_itypes = 0 since it is much harder to optimize.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 15:07:00 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 14:07:00 +0000
Subject: [Koha-bugs] [Bug 19910] New: Download report as 'Comma separated'
is misleading
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19910
Bug ID: 19910
Summary: Download report as 'Comma separated' is misleading
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Reports
Assignee: koha-bugs at lists.koha-community.org
Reporter: nick at bywatersolutions.com
QA Contact: testopia at bugs.koha-community.org
The system preference 'delimiter' is actually used for determining how the
report will be formatted, we should name it accordingly
--
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 Wed Jan 3 15:07:10 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 14:07:10 +0000
Subject: [Koha-bugs] [Bug 19910] Download report as 'Comma separated' is
misleading
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19910
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|koha-bugs at lists.koha-commun |nick at bywatersolutions.com
|ity.org |
Severity|enhancement |normal
--
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 Wed Jan 3 15:09:17 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 14:09:17 +0000
Subject: [Koha-bugs] [Bug 18497] Downloading a report passes the constructed
SQL as a parameter
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18497
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |19910
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19910
[Bug 19910] Download report as 'Comma separated' is misleading
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 15:09:17 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 14:09:17 +0000
Subject: [Koha-bugs] [Bug 19910] Download report as 'Comma separated' is
misleading
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19910
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |18497
Status|NEW |Needs Signoff
Patch complexity|--- |Trivial patch
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18497
[Bug 18497] Downloading a report passes the constructed SQL as a parameter
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 15:09:19 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 14:09:19 +0000
Subject: [Koha-bugs] [Bug 19910] Download report as 'Comma separated' is
misleading
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19910
--- Comment #1 from Nick Clemens ---
Created attachment 70246
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70246&action=edit
Bug 19910 - Use the delimiter syspref to name download link
To test:
1 - Set delimiter syspref to anything but comma
2 - Donwload report results as comma separated text
3 - They actually follow the syspref
4 - Apply patch
5 - Download link should now match pref selection
6 - Change pref, note link changes
7 - Verify things still work as expected
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 15:13:55 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 14:13:55 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #19 from David Gustafsson ---
Created attachment 70247
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70247&action=edit
DBIX_search
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 15:14:25 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 14:14:25 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #20 from David Gustafsson ---
Created attachment 70248
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70248&action=edit
search_unblessed
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 15:21:58 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 14:21:58 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #21 from David Gustafsson ---
Uploaded the benchmarks. Koha::Object::unblessed takes 3.63s, and
Koha::Schema::Result::Item::effective_itemtype 3.70s (of 157s) when
item-level_itypes = 1, so not much of an impact.
Also: with patch applied, C4::Context::dbh takes an unreasonable 3.71s (of
17.6s), but that can be fixed using this patch:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19819
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 15:25:43 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 14:25:43 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |josef.moravec at gmail.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 15:52:53 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 14:52:53 +0000
Subject: [Koha-bugs] [Bug 19564] Fix extraction of sort order from sort
condition name
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19564
--- Comment #10 from David Gustafsson ---
:) Thanks for the tests! Will apply and test this tomorrow.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 15:58:14 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 14:58:14 +0000
Subject: [Koha-bugs] [Bug 19819] C4::Context->dbh is unreasonably slow
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19819
Josef Moravec 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 Wed Jan 3 15:58:16 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 14:58:16 +0000
Subject: [Koha-bugs] [Bug 19819] C4::Context->dbh is unreasonably slow
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19819
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #69822|0 |1
is obsolete| |
--- Comment #8 from Josef Moravec ---
Created attachment 70249
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70249&action=edit
Bug 19819: C4::Context->dbh is unreasonably slow
Locally cache Koha::Database->schema->storage->dbh in state variable in
C4::Context->dbh to get rid of retrieval overhead
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 16:14:36 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 15:14:36 +0000
Subject: [Koha-bugs] [Bug 19564] Fix extraction of sort order from sort
condition name
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19564
David Gustafsson changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--- Comment #11 from David Gustafsson ---
I had time to do so today. Don't know if I am allowed to sign this off, but can
vouch for that the tests are working.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 16:15:47 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 15:15:47 +0000
Subject: [Koha-bugs] [Bug 19564] Fix extraction of sort order from sort
condition name
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19564
David Gustafsson changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 16:17:10 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 15:17:10 +0000
Subject: [Koha-bugs] [Bug 18984] Remove support for NORMARC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18984
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |josef.moravec at gmail.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 16:22:42 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 15:22:42 +0000
Subject: [Koha-bugs] [Bug 19908] Password should not be mandatory
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19908
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Failed QA
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
--- Comment #2 from Jonathan Druart ---
On members/member-password.pl I can submit the form without confirming the
password.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 16:25:47 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 15:25:47 +0000
Subject: [Koha-bugs] [Bug 19898] Koha 17.11 - Upload Local Cover Image
Failed - " Cannot process file as an image"
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19898
--- Comment #4 from Jonathan Druart ---
(In reply to Craig from comment #3)
> Unfortunately, after turning off Autolocation, restarting Koha, and
> restarting memcache, the error still persists.
Do you still have the warning in the logs?
What is the exact version of Koha you are using?
Did you try on a sandbox?
--
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 Wed Jan 3 16:47:11 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 15:47:11 +0000
Subject: [Koha-bugs] [Bug 10032] Uncertain prices hide 'close basket'
without explanation
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10032
Jon Knight changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #69748|0 |1
is obsolete| |
--- Comment #6 from Jon Knight ---
Created attachment 70250
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70250&action=edit
Bug 10032: Uncertain prices hide 'close basket' without explanation
Added disabled "Close this basket" button when uncertain prices are
present. Note that whilst this has a "title" attribute, there is no
tooltip shown because JQueryUI disabled buttons don't get DOM events.
To test:
1) Before applying the patch, go to staff UI, and select Acquisitions,
pick one of your existing vendors (or create a new one) and then open
a new basket (call it anything you want).
2) Add a new (empty) record order to the basket.
3) Give the new record a made up title, add a single item (no need to
fill in any details), set a price of 10.00 and check the "Uncertain
price" checkbox before saving the record.
4) Confirm that the resulting summary page has a button bar that does
NOT include any button for closing the basket.
5) Modify the record and remove the check mark from the "Uncertain
price" checkbox before saving.
6) Confirm that the resulting basket summary page button bar now does
have a "Close this basket" button (as the basket no longer has
uncertain prices).
7) Apply the patch.
8) Modify the record again to check the "Uncertain price" checkbox
again, then save it.
9) Confirm that the resulting summary page has a button bar that does
include a "Close this basket" button but that this button is greyed
out and inoperative.
10) Modify the record again to remove the check mark in the "Uncertain
price" checkbox, then save the record.
11) Confirm that the resulting summary page has a button bar that does
include a "Close this basket" button which is now active (as the order
no longer has an uncertain price).
--
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 Wed Jan 3 16:55:27 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 15:55:27 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #22 from Jonathan Druart ---
David, I was talking about something like that:
diff --git a/C4/Biblio.pm b/C4/Biblio.pm
index b26c767ef8..373d9f8330 100644
--- a/C4/Biblio.pm
+++ b/C4/Biblio.pm
@@ -2840,7 +2840,7 @@ sub EmbedItemsInMarcBiblio {
&& ( C4::Context->preference('OpacHiddenItems') !~ /^\s*$/ );
require C4::Items;
- my $items = C4::Items::GetItems($itemnumbers);
+ my $items = Koha::Items->search({ itemnumber => { -in => $itemnumbers }
})->unblessed;
if ($opachiddenitems) {
my %hidden_items = map { $_ => undef }
C4::Items::GetHiddenItemnumbers(@{$items});
# Reduce items to non hidden items
diff --git a/Koha/Item.pm b/Koha/Item.pm
index bea05f8317..73306f54a2 100644
--- a/Koha/Item.pm
+++ b/Koha/Item.pm
@@ -235,6 +235,14 @@ sub current_holds {
return Koha::Holds->_new_from_dbic($hold_rs);
}
+sub unblessed {
+ my ($self) = @_;
+ my $itype = $self->effective_itemtype;
+ my $h = $self->SUPER->unblessed;
+ $h->{itype} = $itype;
+ return $h;
+}
Can you share the script you use to benchmark, I could use it to compare
different situations.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 17:07:06 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 16:07:06 +0000
Subject: [Koha-bugs] [Bug 19819] C4::Context->dbh is unreasonably slow
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19819
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
--- Comment #9 from Jonathan Druart ---
Did you try this patch in production? I am expecting regressions on the issues
with had on bug 14778 (like "server has gone away").
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 17:16:29 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 16:16:29 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #23 from David Gustafsson ---
I was using this to benchmark:
https://gist.github.com/gnucifer/5b2119f656592c4d6b3f6dab0b8087f1
Which is more or less equivalent to your code example (when comparing
approaches to improve performance).
And this to run the benchmark:
https://gist.github.com/gnucifer/fa3bb31d49f1258a4d1af270d235d76f
(biblionumber probably needs replacing if running on different instance).
So to replace find with search using this approach will unfortunately not
increase performance very much.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 17:21:20 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 16:21:20 +0000
Subject: [Koha-bugs] [Bug 19698] "Add message" for patron don' t use OPAC
description if message type is OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19698
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |katrin.fischer at bsz-bw.de
--- Comment #3 from Katrin Fischer ---
Hi Sophie,
I think the authorised_value is used in an untypical way here (like MAN_INV or
INVENTORY). It allows you to store a text snippet or template for a text that
then can be changed and edited before saving the message. When the message is
pulled for display, it will pull the stored text from the database, the
authorised_value code is not stored.
Because of this the current behaviour makes sense to me - the OPAC value has no
meaning here.
I think if we wanted it to pull different template texts depending on if you
selected staff or OPAC we'd need to use Ajax similar to what we do for
statistic values depending on fund selection on the acquisition order page. It
might also cause problems if the text will be overwritten after you already
edited it.
--
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 Wed Jan 3 17:21:38 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 16:21:38 +0000
Subject: [Koha-bugs] [Bug 19698] "Add message" for patron don' t use OPAC
description if message type is OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19698
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|normal |enhancement
--
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 Wed Jan 3 17:23:26 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 16:23:26 +0000
Subject: [Koha-bugs] [Bug 19890] authorised value branches is not visible
from dropdown in Admin
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19890
--- Comment #3 from Jonathan Druart ---
I have just tried: If you try to create a AV category "branches", there is a
warning box with "The authorized value category 'branches', 'itemtypes' and
'cn_source' are used internally by Koha and are not valid."
That seems enough to me.
--
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 Wed Jan 3 17:24:45 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 16:24:45 +0000
Subject: [Koha-bugs] [Bug 19819] C4::Context->dbh is unreasonably slow
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19819
--- Comment #10 from David Gustafsson ---
Good catch, I have only run it locally, but it will be included the next
release for us. I'm not very knowledge about the inner working of Plack, but if
the connection is persisted over time, and never closed, I understand how this
can be an issue. I could try too look into this some more trying to figure out
if the current fix if safe, and if not if there is some other way of going
about this.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 17:36:31 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 16:36:31 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #24 from David Gustafsson ---
There is by the way a way of getting equivalent performance (as the SQL-hack)
out of GetItems, if biblios where batch loaded all the way down. (Multiple
biblio ids sent to EmbedItemsInMarcBiblio so that all items could be loaded in
one batch (requireing requiring only one search query for x number of biblios).
But this would be a much more complicated change, and require existing code to
use this new API (loading multiple biblios at once). Still, might be worth
having a look at in the long run.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 18:15:39 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 17:15:39 +0000
Subject: [Koha-bugs] [Bug 19907] Email validation on patron add/edit not
working
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19907
tmcmahon at wlpl.org changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |tmcmahon at wlpl.org
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 18:15:41 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 17:15:41 +0000
Subject: [Koha-bugs] [Bug 19908] Password should not be mandatory
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19908
tmcmahon at wlpl.org changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |tmcmahon at wlpl.org
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 19:16:55 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 18:16:55 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #25 from Jonathan Druart ---
We, of course, know that an ORM will impact performance a bit.
I do not think the script you are testing to compare is fair, you are using the
DBMS cache as the query is always the same.
I think we should use a loop on all X biblionumbers, then retrieve the items
info.
With DBIx::Class::ResultClass::HashRefInflator, I'd expect more x4 instead of
x10, which is still to high, I admit.
If we have to provide a way to fetch a row from its id, I would prefer to
introduce a generic way (at Koha::Object level) to do it and use it only for
batch/heavy operations.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 19:24:07 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 18:24:07 +0000
Subject: [Koha-bugs] [Bug 19907] Email validation on patron add/edit not
working
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19907
Jon Knight changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Patch doesn't apply
CC| |J.P.Knight at lboro.ac.uk
--- Comment #2 from Jon Knight ---
Tried to apply the patch to an up-to-date kohadevbox but it fails:
vagrant at kohadevbox:kohaclone(master)$ git fetch
vagrant at kohadevbox:kohaclone(master)$ git rebase origin/master
Current branch master is up to date.
vagrant at kohadevbox:kohaclone(master)$ git checkout -b bug_19907
Switched to a new branch 'bug_19907'
vagrant at kohadevbox:kohaclone(bug_19907)$ git bz apply 19907
Bug 19907 - Email validation on patron add/edit not working
70244 - Bug 19907: Fix email validation in patron edit form
Apply? [(y)es, (n)o, (i)nteractive] y
Applying: Bug 19907: Fix email validation in patron edit form
Using index info to reconstruct a base tree...
M koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
Falling back to patching base and 3-way merge...
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
CONFLICT (content): Merge conflict in
koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
Failed to merge in the changes.
Patch failed at 0001 Bug 19907: Fix email validation in patron edit form
The copy of the patch that failed is found in:
/home/vagrant/kohaclone/.git/rebase-apply/patch
When you have resolved this problem run "git bz apply --continue".
If you would prefer to skip this patch, instead run "git bz apply --skip".
To restore the original branch and stop patching run "git bz apply --abort".
Patch left in
/tmp/Bug-19907-Fix-email-validation-in-patron-edit-form-LYJrg8.patch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 19:24:38 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 18:24:38 +0000
Subject: [Koha-bugs] [Bug 19907] Email validation on patron add/edit not
working
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19907
Jon Knight 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 Wed Jan 3 19:27:31 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 18:27:31 +0000
Subject: [Koha-bugs] [Bug 19907] Email validation on patron add/edit not
working
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19907
--- Comment #3 from Jon Knight ---
(In reply to Jon Knight from comment #2)
> Tried to apply the patch to an up-to-date kohadevbox but it fails:
Ignore me - I didn't notice the dependency on 19908. However as that bug
currently has an issue with not requiring the second entry of the password,
should testing this patch wait until that is fixed?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 19:27:29 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 18:27:29 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #26 from Jonathan Druart ---
Created attachment 70251
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70251&action=edit
Bug 19884: Add benchmark script to compare DBIC vs plain SQL
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 19:27:51 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 18:27:51 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #27 from Jonathan Druart ---
Still x10 :-/
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 19:38:51 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 18:38:51 +0000
Subject: [Koha-bugs] [Bug 19911] New: Passwords displayed to user during
self-registration are not HTML-encoded
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19911
Bug ID: 19911
Summary: Passwords displayed to user during self-registration
are not HTML-encoded
Change sponsored?: ---
Product: Koha
Version: 17.11
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: OPAC
Assignee: oleonard at myacpl.org
Reporter: library at sll.texas.gov
QA Contact: testopia at bugs.koha-community.org
Created attachment 70252
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70252&action=edit
Example of the generated password not displaying properly due to the less-than
character treated as opening HTML tag
If self-registration is enabled and the PatronSelfRegistrationPrefillForm
system preference is set to "Display and prefill," self-registered users are
shown their password upon successfully registering. If the password contains a
less-than character, browsers treat this as the beginning of an HTML element,
and so the less-than character and anything after it does not display since the
password is not HTML-encoded.
If Koha is set to generate passwords automatically during self-registration
(i.e., users are not allowed or required to enter a password in the
self-registration form), any generated password containing the less-than
character will not display correctly. Users who are expected to copy/save their
password at this time cannot do so, and there is no way to recover that
generated password.
Attached is a screenshot showing what I mean. A solution would to HTML-encode
the passwords when they are displayed as part of the self-registration process,
regardless of whether the user must verify their e-mail address first
(opac-registration-verify.pl).
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 19:57:21 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 18:57:21 +0000
Subject: [Koha-bugs] [Bug 19911] Passwords displayed to user during
self-registration are not HTML-encoded
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19911
Arturo changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |library at sll.texas.gov
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 20:15:33 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 19:15:33 +0000
Subject: [Koha-bugs] [Bug 19911] Passwords displayed to user during
self-registration are not HTML-encoded
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19911
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|enhancement |major
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
Version|17.11 |master
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 20:15:38 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 19:15:38 +0000
Subject: [Koha-bugs] [Bug 19911] Passwords displayed to user during
self-registration are not HTML-encoded
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19911
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|oleonard at myacpl.org |jonathan.druart at bugs.koha-c
| |ommunity.org
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 20:17:04 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 19:17:04 +0000
Subject: [Koha-bugs] [Bug 19911] Passwords displayed to user during
self-registration are not HTML-encoded
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19911
--- Comment #1 from Jonathan Druart ---
Created attachment 70253
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70253&action=edit
Bug 19911: Escape password value during self-registration confirmation
The password must be correctly escape, it can contains html character
and break the display.
Test plan:
Apply first patch and confirm that the display is broken
Apply second patch (this one) and confirm that the display is fixed
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 20:17:00 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 19:17:00 +0000
Subject: [Koha-bugs] [Bug 19911] Passwords displayed to user during
self-registration are not HTML-encoded
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19911
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 20:24:36 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 19:24:36 +0000
Subject: [Koha-bugs] [Bug 13044] Deprecate old book drop mode
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13044
--- Comment #3 from Kyle M Hall ---
I suppose we could make bookdrop mode find the last date open and recalculate
the fine in full from there. That way it looks the same from a user
perspective.
The problem with bookdrop mode is that it is quite fragile. For example, if
fines.pl doesn't run for a night, and does the following night, then using
bookdrop mode will decrement the fine by double the expected amount. The
backdating returns feature does not suffer from this kind of fragility.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 20:34:14 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 19:34:14 +0000
Subject: [Koha-bugs] [Bug 13376] Include a user-editable area/ page for
terms and conditions of use in the opac
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13376
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |katrin.fischer at bsz-bw.de
--- Comment #6 from Katrin Fischer ---
While a general solution for adding pages to the OPAC would be nice, I agree
this would be a good addition.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 20:44:14 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 19:44:14 +0000
Subject: [Koha-bugs] [Bug 19912] New: PrintNoticesMaxLines message should
print after list of items not at end of notice
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19912
Bug ID: 19912
Summary: PrintNoticesMaxLines message should print after list
of items not at end of notice
Change sponsored?: ---
Product: Koha
Version: 17.05
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Notices
Assignee: koha-bugs at lists.koha-community.org
Reporter: michael at bywatersolutions.com
QA Contact: testopia at bugs.koha-community.org
Created attachment 70254
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70254&action=edit
showing message at bottom of notice
PrintNoticesMaxLines prints a message about too many items to list on an
overdue notice, but prints it at the very bottom of the notice. It would be
more clear to the reader if this message could follow the list of items on the
notice, as they might not notice the message at the bottom of the page.
--
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 Wed Jan 3 21:03:54 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 20:03:54 +0000
Subject: [Koha-bugs] [Bug 19884] Improve performance of GetItem
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19884
--- Comment #28 from Jonathan Druart ---
*Thinking loud
Months ago I wrote Koha::Object::Simple (for a different need, it was AV and
caching related).
Now it sounds like we could need something similar here, see
https://github.com/joubu/Koha/commits/koha_object_simple to understand what I
have in mind (this is a very quick draft!)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 22:10:27 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 21:10:27 +0000
Subject: [Koha-bugs] [Bug 18754] Translatability: Get rid of exposed tt
directives in opac-detail.tt
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18754
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |19913
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19913
[Bug 19913] Embedded HTML5 videos are broken
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 22:10:27 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 21:10:27 +0000
Subject: [Koha-bugs] [Bug 19913] New: Embedded HTML5 videos are broken
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19913
Bug ID: 19913
Summary: Embedded HTML5 videos are broken
Change sponsored?: ---
Product: Koha
Version: unspecified
Hardware: All
OS: All
Status: ASSIGNED
Severity: major
Priority: P5 - low
Component: OPAC
Assignee: jonathan.druart at bugs.koha-community.org
Reporter: jonathan.druart at bugs.koha-community.org
QA Contact: testopia at bugs.koha-community.org
Depends on: 18754
The video element is badly generated.
Caused by commit 752aef4593f1aa2d64700bf9738e7e03907eb1cd
Bug 18754: Translatability: Get rid of exposed tt directives in
opac-detail.tt
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18754
[Bug 18754] Translatability: Get rid of exposed tt directives in opac-detail.tt
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 22:15:51 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 21:15:51 +0000
Subject: [Koha-bugs] [Bug 19913] Embedded HTML5 videos are broken
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19913
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Jan 3 22:15:54 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 21:15:54 +0000
Subject: [Koha-bugs] [Bug 19913] Embedded HTML5 videos are broken
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19913
--- Comment #1 from Jonathan Druart ---
Created attachment 70255
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70255&action=edit
Bug 19913: Fix embedded HTML5 videos at the OPAC
Caused by
commit 752aef4593f1aa2d64700bf9738e7e03907eb1cd
Bug 18754: Translatability: Get rid of exposed tt directives in opac-detail.tt
A space is missing between 'source' and 'src'
Test plan:
Set HTML5MediaEnabled to OPAC
Put a video link to 856$u
See the OPAC detail page, tab video, click the video
=> Should play with this patch applied
Note that this does not affect intranet
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Jan 4 00:09:39 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 23:09:39 +0000
Subject: [Koha-bugs] [Bug 18231] Add the missing id-column to
koha.issuingrules-table for DBIC fixtures
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18231
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |katrin.fischer at bsz-bw.de
Status|Signed Off |Failed QA
--- Comment #27 from Katrin Fischer ---
There is a complaint in the QA tools about a file being added and removed -
this is due to the change from .perl to .sql for the file. As all 3 patches
have different authors, this can't be fixed by squashing (not a blocker in my
opinion).
FAIL installer/data/mysql/atomicupdate/Bug-18231-id_to_ussingrules_table.sql
FAIL git manipulation
The file has been added and deleted in the same patchset
t/db_dependent/Koha/IssuingRules.t are ok.
BUT: Cloning isusing rules from one of the library (default to Union for
example) fails:
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Jan 4 00:44:59 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 03 Jan 2018 23:44:59 +0000
Subject: [Koha-bugs] [Bug 12598] New misc/import_borrowers.pl command line
tool
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12598
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Failed QA
--- Comment #165 from Katrin Fischer ---
I've taken a look at the patches for a first QA review.
1) Currently fails QA script:
FAIL t/db_dependent/Koha/Patrons/Import.t
FAIL critic
"$headerrow_0" is declared but not used at line 320, column 5.
Unused variables clutter code and make it harder to read.
2) Tests fail:
kohadev-koha at kohadevbox:/home/vagrant/kohaclone$ perl
t/db_dependent/Koha/Patrons/Import.t
1..124
ok 1 - use Koha::Patrons::Import;
ok 2 - An object of class 'Koha::Patrons::Import' isa 'Koha::Patrons::Import'
# Subtest: test_methods
1..1
ok 1 - Koha::Patrons::Import->can(...)
ok 3 - test_methods
# Subtest: test_attributes
1..1
ok 1 - Koha::Patrons::Import->can(...)
ok 4 - test_attributes
ok 5 - Koha::Patrons::Import->import_patrons carps if no file handle is passed
ok 6 - Got the expected undef from import_patrons with nothing much
ok 7 - Koha::Patrons::Import->import_patrons carps if no file handle is passed
ok 8 - Got the expected undef from import_patrons with no file handle
Use of uninitialized value $matchpoint in string eq at
/home/vagrant/kohaclone/Koha/Patrons/Import.pm line 170, <$handle_1> line 2.
# Looks like your test exited with 255 just after 8.
3) Might be me, but trying to run the script always gives me the help, no error
or anything. Used -c and -f with a random file.
Can you please take a look?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Jan 4 01:30:57 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 04 Jan 2018 00:30:57 +0000
Subject: [Koha-bugs] [Bug 17385] Add custom export formats for bibliographic
records
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17385
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|Add custom export notices |Add custom export formats
| |for bibliographic records
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Jan 4 01:34:07 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 04 Jan 2018 00:34:07 +0000
Subject: [Koha-bugs] [Bug 17006] REST API: add route to change patron's
password
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17006
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |katrin.fischer at bsz-bw.de
Status|Signed Off |Patch doesn't apply
--- Comment #39 from Katrin Fischer ---
Patch doesn't apply, please also take a look at comment#38.
Apply? [(y)es, (n)o, (i)nteractive] y
Applying: Bug 17006: Add API route for changing patron's password
Using index info to reconstruct a base tree...
M Koha/Patron.pm
M Koha/REST/V1/Patron.pm
M api/v1/swagger/paths.json
M api/v1/swagger/paths/patrons.json
M t/db_dependent/api/v1/patrons.t
Falling back to patching base and 3-way merge...
Auto-merging t/db_dependent/api/v1/patrons.t
Auto-merging api/v1/swagger/paths/patrons.json
Auto-merging api/v1/swagger/paths.json
CONFLICT (content): Merge conflict in api/v1/swagger/paths.json
Auto-merging Koha/REST/V1/Patron.pm
Auto-merging Koha/Patron.pm
error: Failed to merge in the changes.
Patch failed at 0001 Bug 17006: Add API route for changing patron's password
The copy of the patch that failed is found in: .git/rebase-apply/patch
When you have resolved this problem run "git bz apply --continue".
If you would prefer to skip this patch, instead run "git bz apply --skip".
To restore the original branch and stop patching run "git bz apply --abort".
Patch left in
/tmp/Bug-17006-Add-API-route-for-changing-patrons-passw-7eNHth.patch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Jan 4 02:03:28 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 04 Jan 2018 01:03:28 +0000
Subject: [Koha-bugs] [Bug 14385] Extend OpacHiddenItems to allow specifying
exempt borrower categories
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14385
--- Comment #145 from M. Tompsett ---
Created attachment 70256
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70256&action=edit
Bug 14385: Extend OpacHiddenItems to allow specifying exempt borrower
categories
Edit: Fixing merge conflicts in
- t/db_dependent/Items.t
- t/db_dependent/Search.t
- C4/Search.pm
Changes the API for calling GetHiddenItems and all the places in the code that
call it. This is to allow borrower categories to be passed in.
Adds an OpacHiddenItemsExceptions syspref to allow certain borrower categories
to be able to see items, even if they are marked hidden by OpacHiddenItems
To test:
1) Make two borrowers, one in a category that should see everything (ie Adult),
and another in a category that should only see certain things (ie Adult -
exceptions)
2) Add the borrower that can see everything (the Adult) to
OpacHiddenItemsExceptions
3) To the OpacHiddenItems syspref, add an item type (ensure that you have some
records that fall under this type in your library).
4) Log in as the borrower that should only see certain things (Adult -
exception)
5) Do a search, filtered to show records which are the item type that you
specified in the OpacHiddenItems syspref. No records should show for this
borrower as this item type is hidden to them.
6) Log in as the borrower that should see everything (Adult)
7) Do the same search. There should be results from this search, as this
borrower category has been specified as an exception to the hidden items
Signed-off-by: Claire Gravely
Signed-off-by: Mark Tompsett
Signed-off-by: Aleisha Amohia
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Jan 4 02:03:34 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 04 Jan 2018 01:03:34 +0000
Subject: [Koha-bugs] [Bug 14385] Extend OpacHiddenItems to allow specifying
exempt borrower categories
In-Reply-To: