From bugzilla-daemon at bugs.koha-community.org Wed Nov 1 03:52:06 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Nov 2017 02:52:06 +0000
Subject: [Koha-bugs] [Bug 19554] The inventory table should jump to detail
instead of MARCdetail
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19554
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Nov 1 03:52:08 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Nov 2017 02:52:08 +0000
Subject: [Koha-bugs] [Bug 19554] The inventory table should jump to detail
instead of MARCdetail
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19554
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68865|0 |1
is obsolete| |
--- Comment #2 from M. Tompsett ---
Created attachment 68875
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68875&action=edit
Bug 19554: The inventory table should jump to detail instead of MARCdetail
This patch just replaces the link to detail.
Detail provides item information needed in case of inventory problems.
Test plan:
Go to Tools/Inventory.
Create an inventory list without barcode file.
Check if the link goes to detail now.
Signed-off-by: Marcel de Rooy
Signed-off-by: Mark Tompsett
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Nov 1 04:06:15 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Nov 2017 03:06:15 +0000
Subject: [Koha-bugs] [Bug 19551] Cash register report has bad erroneous
results from wrong order of operations
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19551
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68857|0 |1
is obsolete| |
--- Comment #2 from M. Tompsett ---
Created attachment 68876
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68876&action=edit
Bug 19551 - Fix wrong order of operations in cash register report
To test:
1 - Find or create a fine for a patron
2 - Write it off
3 - Home -> Reports -> Cash register
4 - Fill in current date and transaction type 'Write off'
5 - Click Submit
-- see your write off
6 - Fill in previous dates and transaction type 'Write off'
7 - Click Submit
-- see your write off (bad)
8 - Apply patch
9 - Rerun steps 3-7
-- write only visible for current date.
Signed-off-by: Mark Tompsett
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Nov 1 04:06:13 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Nov 2017 03:06:13 +0000
Subject: [Koha-bugs] [Bug 19551] Cash register report has bad erroneous
results from wrong order of operations
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19551
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Nov 1 04:09:31 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Nov 2017 03:09:31 +0000
Subject: [Koha-bugs] [Bug 18645] Creation of Koha feature release change
tool displayed after running Koha update
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18645
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mtompset at hotmail.com
--- Comment #13 from M. Tompsett ---
Please do not do whitespace changes in patches unless necessary for individual
changed lines. White space patches should be separated. Otherwise, it makes
eyeballing the bug more difficult.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Nov 1 11:53:14 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Nov 2017 10:53:14 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
Magnus Enger changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68858|0 |1
is obsolete| |
--- Comment #168 from Magnus Enger ---
Created attachment 68877
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68877&action=edit
Bug 7317: QA followup
This fixes some of the issues reported by the QA script, but not all.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Nov 1 11:53:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Nov 2017 10:53:21 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
Magnus Enger changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68874|0 |1
is obsolete| |
--- Comment #169 from Magnus Enger ---
Created attachment 68878
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68878&action=edit
Bug 7317: Handle backend absense more gracefuly
5/ This patch is WIP, it makes Koha::Illrequest->load_backend raise an
exception
if the passed backend is invalid. This way we will catch more errors
introduced.
The patch also disables the 'New Ill request' when no backends are available.
Gets
rid of a related warning.
TODO: Should add a noticeable warning on the UI, or a tooltip on the button.
TODO: add a couple tests for the exception
4/ This patch fixes the path for the checkboxes jquery plugin, and removes the
include
for tablesorter, as this implementation uses Datatables. This is obviously code
for older
Koha, ported to master.
3/ I know it doesn't look right, but am not sure how to display. This is the
usual empty
datatable display we have everywhere in Koha.
2/ We could fix it with a proper IDE, I use Komodo and refactoring code is
pretty straigh-forward.
Worth a separate bug report.
1/ That TODO is misleading. It refers to their own migration from a different
design. I checked this
code and cannot find occurences of ->status with that annotation. I think this
relates to backends code.
Signed-off-by: Tomas Cohen Arazi
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Nov 1 12:02:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Nov 2017 11:02:36 +0000
Subject: [Koha-bugs] [Bug 16865] Patron self registration: hiding branchcode
hides email and breaks workflow
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16865
--- Comment #5 from Katrin Fischer ---
The primary email field in the form where you enter the patron data.
--
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 Nov 1 12:03:32 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Nov 2017 11:03:32 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
--- Comment #170 from Magnus Enger ---
Trying to summarize the status of this bug now.
*** QA script
** POD coverage
Some FAILS are "POD coverage was greater before, try perl
-MPod::Coverage=PackageName -e666". It sounds like these can be ignored.
** Atomicupdate
FAIL installer/data/mysql/atomicupdate/ill_tables.sql
FAIL git manipulation
The file has been added and deleted in the same patchset
Alex provided ill_tables.sql. Tomas rewrote it as bug_7317_ill.perl. Not sure
there is anything to fix here?
** tt_valid
FAIL koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
OK forbidden patterns
OK git manipulation
OK spelling
FAIL tt_valid
lines 665
FAIL valid_template
: not found
TT is used to set some HTML attributes. Not sure if this can be fixed. Tomas?
*** (In reply to Jonathan Druart from comment #164)
> 1/
> +TODO:
> +
> +- Anything invoking the ->status method; annotated with:
> + + # Old use of ->status !
>
> What do it mean?
OK, according to Tomas.
> 2/ I will not make you rewrite everything, but it would have been better to
> stay consistent and not mix snake_case and camelCase for method names.
OK
> First steps with the GUI:
> 3/ This is what I see when I enable the module:
> https://screenshots.firefox.com/z9dNoRL91NQ5F3cD/pro.kohadev.org
> Looks wrong.
Not sure if this can be fixed?
> 4/ Hit /cgi-bin/koha/ill/ill-requests.pl and got:
> Loading failed for the
koha-tmpl/intranet-tmpl/prog/en/modules/tools/stage-marc-import.tt:
koha-tmpl/intranet-tmpl/prog/en/modules/tools/upload-images.tt:
koha-tmpl/intranet-tmpl/prog/en/modules/tools/upload.tt:
Please adjust
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 10:37:56 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 09:37:56 +0000
Subject: [Koha-bugs] [Bug 18382] action_logs entry for module HOLDS,
action SUSPEND is spammy
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18382
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |m.de.rooy at rijksmuseum.nl
Status|Signed Off |Failed QA
--- Comment #9 from Marcel de Rooy ---
Your solution is not limited to action_logs.
You truncate, so to speak, already at the holds level.
Please add your fix at the hold unblessed level when sending it to the logs.
Squash the patches too.
Thanks.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 10:39:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 09:39:36 +0000
Subject: [Koha-bugs] [Bug 19574] New: Wrong itemtype can be chosen when
placing holds
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19574
Bug ID: 19574
Summary: Wrong itemtype can be chosen when placing holds
Change sponsored?: ---
Product: Koha
Version: unspecified
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Circulation
Assignee: koha-bugs at lists.koha-community.org
Reporter: andreas.hedstrom.mace at sub.su.se
QA Contact: testopia at bugs.koha-community.org
CC: gmcharlt at gmail.com, kyle.m.hall at gmail.com
With the syspref AllowHoldItemTypeSelection set to Allow, it is possible to add
a hold for an itemtype that doesn?t actually exist at a specific branch. Say
that branch A has two itemtypes: books and map, while branch B has three:
books, map and audio book. It is possible to add a hold for branch A with
itemtype audio book although there is no item with such an itemtype at this
branch. This will cause an internal server error if you try to delete this
faulty hold.
To recreate:
1) Enable AllowHoldItemTypeSelection
2) Add, or find, a bibliographic record with at least two branches and at least
two items with two different itemtypes. Branch B should not have all itemtypes.
3) Add a hold for branch B, and choose an itemtype not available at this branch
(there are no such items)
4) Try deleting the hold, note the internal server error.
--
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 Fri Nov 3 10:44:50 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 09:44:50 +0000
Subject: [Koha-bugs] [Bug 10748] Add option to block return of lost items
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10748
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Failed QA
CC| |m.de.rooy at rijksmuseum.nl
--- Comment #7 from Marcel de Rooy ---
Please add a test case for AddReturn.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 10:50:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 09:50:36 +0000
Subject: [Koha-bugs] [Bug 12027] Shibboleth authentication for staff client
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12027
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Patch doesn't apply
--- Comment #27 from Marcel de Rooy ---
Also please add feedback for comment 25.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 10:52:19 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 09:52:19 +0000
Subject: [Koha-bugs] [Bug 19290] Add the ability to filter a result list on
staff client
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19290
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |m.de.rooy at rijksmuseum.nl
Status|Signed Off |Patch doesn't apply
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:00:32 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:00:32 +0000
Subject: [Koha-bugs] [Bug 15524] Set limit on maximum possible holds per
patron by category
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15524
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |m.de.rooy at rijksmuseum.nl
--- Comment #14 from Marcel de Rooy ---
QA: Looking here now
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:02:55 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:02:55 +0000
Subject: [Koha-bugs] [Bug 18904] Advanced editor - Rancor - Add authority
support
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18904
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |19537
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19537
[Bug 19537] Authorities search doesn't correctly populate subfield $2 source of
heading
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:02:55 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:02:55 +0000
Subject: [Koha-bugs] [Bug 19537] Authorities search doesn' t correctly
populate subfield $2 source of heading
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19537
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |18904
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18904
[Bug 18904] Advanced editor - Rancor - Add authority support
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:03:50 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:03:50 +0000
Subject: [Koha-bugs] [Bug 18904] Advanced editor - Rancor - Add authority
support
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18904
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Signed Off
QA Contact|testopia at bugs.koha-communit |m.de.rooy at rijksmuseum.nl
|y.org |
--- Comment #21 from Nick Clemens ---
Missed adding dependency on 19537, that should clear up the error
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:21:41 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:21:41 +0000
Subject: [Koha-bugs] [Bug 12027] Shibboleth authentication for staff client
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12027
--- Comment #28 from Marcel de Rooy ---
Zeno asked me:
===
I think that Matthias on commen7 #26 is quite clear
It is not enough a rebase for this patch ?
===
Yeah I focused on a reply from Martin. So probably true.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:36:13 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:36:13 +0000
Subject: [Koha-bugs] [Bug 14413] map internalnotes to MarcFieldsToOrder
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14413
Koha Team Lyon 3 changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |koha at univ-lyon3.fr
--- Comment #1 from Koha Team Lyon 3 ---
I agree with Nicole, it would be usefull have the possibility to get
order_internalnote thanks to MarcFieldsToOrder syspref, while we download
iso2709 files for order.
Sonia
--
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 Fri Nov 3 11:41:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:41:21 +0000
Subject: [Koha-bugs] [Bug 19420] Improve display of errors from failure of
uploading file during stage import
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19420
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:41:23 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:41:23 +0000
Subject: [Koha-bugs] [Bug 19420] Improve display of errors from failure of
uploading file during stage import
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19420
--- Comment #10 from Nick Clemens ---
Created attachment 68910
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68910&action=edit
Bug 19420 - Add errors on other pages
Eventually these should all be using the same code in a js file
upload.tt already dealt with these errors, but has diff code, made it
work with new error syntax
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:42:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:42:05 +0000
Subject: [Koha-bugs] [Bug 14413] map internalnotes to MarcFieldsToOrder
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14413
--- Comment #2 from Koha Team Lyon 3 ---
It's perhaps fixed bt BZ15503 but it seems that it needs follow-up ?
--
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 Fri Nov 3 11:42:13 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:42:13 +0000
Subject: [Koha-bugs] [Bug 15503] Grab Item Information from Order Files
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15503
Koha Team Lyon 3 changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |koha at univ-lyon3.fr
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:43:11 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:43:11 +0000
Subject: [Koha-bugs] [Bug 15524] Set limit on maximum possible holds per
patron by category
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15524
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |In Discussion
--- Comment #15 from Marcel de Rooy ---
QA Comment
Interesting feature! Limited testing did not reveal problems.
Functionality issue:
In Administration, smart-rules, the section "Default checkout limit by patron
category" contains the item "Maximum total holds allowed (count)".
Not sure if the label should contain total and count btw.
But the section should be renamed. This does not pertain any longer to checkout
limits only.
Furthermore I would expect to have max_holds also under Default checkout, hold
and return policy. It seems consistent to add it also without a patron category
restriction.
Same as current checkouts etc.
Hard to move it to FQA for that reason. At least I want to have some additional
opinions. Setting to In Discussion. Please ask for feedback via IRC or ML.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:48:30 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:48:30 +0000
Subject: [Koha-bugs] [Bug 18798] Use Koha.Preference in OPAC Reading Record
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18798
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |m.de.rooy at rijksmuseum.nl
Status|Signed Off |Failed QA
--- Comment #4 from Marcel de Rooy ---
[% IF GoogleJackets %]
Forgotten ?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:58:10 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:58:10 +0000
Subject: [Koha-bugs] [Bug 18882] Add location code to statistics table for
checkouts and renewals
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |m.de.rooy at rijksmuseum.nl
--- Comment #7 from Marcel de Rooy ---
(In reply to Kyle M Hall from comment #4)
> (In reply to M. Tompsett from comment #2)
> > The code puts '' if there is no location key. But if the location key is
> > NULL, because I hadn't set my shelving location, there is NULL. Do we want
> > NULL's? If so, I'll sign this off. If not, could you tweak that?
>
> I think we should always use NULL is the location is not passed or is set to
> undef. It seems like odd behavior to convert undef values to empty strings.
What we are doing now however, is storing undef only for location. Which is
inconsistent. You change all lines in the second patch, only set undef for
location.
No blocker, but could be improved.
I think that empty string would have been better now.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:59:01 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:59:01 +0000
Subject: [Koha-bugs] [Bug 18882] Add location code to statistics table for
checkouts and renewals
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Passed QA
Patch complexity|--- |Small patch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:59:04 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:59:04 +0000
Subject: [Koha-bugs] [Bug 18882] Add location code to statistics table for
checkouts and renewals
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #64758|0 |1
is obsolete| |
--- Comment #8 from Marcel de Rooy ---
Created attachment 68911
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68911&action=edit
Bug 18882 - Add location code to statistics table for checkouts and renewals
Some libraries wish to track what the current location of items was at the time
they were checked out. This will help libraries track which physical locations
in the library patrons are more likely to check out a given book from.
Test Plan:
1) Apply this patch
2) Run updatedatabase.pl
3) Check out an item that has a location set
4) Renew that item
5) View the checkout and renewal in the statistics table,
verify each has the location column populated correctly
Signed-off-by: Mark Tompsett
Signed-off-by: Marcel de Rooy
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:59:08 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:59:08 +0000
Subject: [Koha-bugs] [Bug 18882] Add location code to statistics table for
checkouts and renewals
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #64759|0 |1
is obsolete| |
--- Comment #9 from Marcel de Rooy ---
Created attachment 68912
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68912&action=edit
Bug 18882 [QA Followup] - Ensure stats with no or undef location are set to
NULL in db
Confirm that only NULL or a string are in the location field of the
statistics table. Test with two items, one with shelving location set,
and one without it set.
Signed-off-by: Mark Tompsett
Signed-off-by: Marcel de Rooy
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 11:59:29 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 10:59:29 +0000
Subject: [Koha-bugs] [Bug 18882] Add location code to statistics table for
checkouts and renewals
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |m.de.rooy at rijksmuseum.nl
|y.org |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 12:24:26 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 11:24:26 +0000
Subject: [Koha-bugs] [Bug 19420] Improve display of errors from failure of
uploading file during stage import
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19420
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Failed QA
--- Comment #11 from Marcel de Rooy ---
Upload local cover image
Upload status: Failed - (2) Root directory for uploads not defined
No, the directory was not writable.
This went wrong since the errcode is 2, but you forgot to add a zero entry in
you errMsgs array. So they are one position out of sync.
Please add a leading entry here (see also upload.tt)
var errMsgs = [ _("File already exists"), _("Directory is not writeable"),
_("Root directory for uploads not defined"), _("Temporary directory for uploads
not defined") ];
Adjust same thing in two other places ?
//Next three errors have no code path to trigger that I see
I am very confident that I tested all codes when I wrote this. They may be hard
to simulate, but I did. Please remove this comment.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 12:25:53 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 11:25:53 +0000
Subject: [Koha-bugs] [Bug 19494] Add reservedate to Holds awaiting pickup
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19494
Simon Pouchol changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68291|0 |1
is obsolete| |
--- Comment #2 from Simon Pouchol ---
Created attachment 68913
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68913&action=edit
Bug 19494: Add reservedate to Holds awaiting pickup
Add reservedate to report.
Change label "Available since" to "Waiting since".
Note: The table (already) crosses the boundaries of its container. Also
the vertical spacing for the first column (already) is not the same as the
others. (Adding the second date only makes it more visible.) These minor
design issues should be solved on another report.
Test plan:
[1] Go to Circulation, Holds awaiting pickup
[2] Note the additional column for reservedate.
Signed-off-by: Marcel de Rooy
Signed-off-by: Simon Pouchol
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 12:26:17 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 11:26:17 +0000
Subject: [Koha-bugs] [Bug 19494] Add reservedate to Holds awaiting pickup
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19494
Simon Pouchol changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |simon.pouchol at biblibre.com
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 13:05:58 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 12:05:58 +0000
Subject: [Koha-bugs] [Bug 19575] New: Use canonical field names and resolve
aliased fields
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
Bug ID: 19575
Summary: Use canonical field names and resolve aliased fields
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: major
Priority: P5 - low
Component: Searching - Elasticsearch
Assignee: koha-bugs at lists.koha-community.org
Reporter: glasklas at gmail.com
The current elastic search fields seems to be a mix between zebra aliases,
field names, some duplicates and new fields. This causes issues with sorting in
UI and hard-coded search-links in XSL-templates for example. I have attempted
to clean things up a bit, which resolves many of these issues. I might not have
the time to do so today, but will attempt to find and mark all current issues
resolved by this fix.
--
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 Fri Nov 3 13:28:33 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 12:28:33 +0000
Subject: [Koha-bugs] [Bug 19575] Use canonical field names and resolve
aliased fields
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19575
--- Comment #1 from David Gustafsson ---
Created attachment 68914
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68914&action=edit
Bug 19575: Use canonical field names and resolve aliased fields
Adjust elastic search mappings to more closely match Zebra equivalents
resolving serveral issues with coded Zebra searches in templates, and
sorting of search results in UI.
--
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 Fri Nov 3 14:12:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 13:12:36 +0000
Subject: [Koha-bugs] [Bug 19548] Search Result Second page not available
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19548
--- Comment #2 from rainer.stowasser at zamg.ac.at ---
Version tested:
Our operational System
16.05.02.000 Linux vbib 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt11-1
(2015-05-24) x86_64
Our test System
17.05.04.000 Linux vbib-dev 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt11-1
(2015-05-24) x86_64
The search string does not matter (tested for space in the brackets)
it works for all quotations
simply try "test" :-)
--
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 Fri Nov 3 14:35:35 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 13:35:35 +0000
Subject: [Koha-bugs] [Bug 19522] Label creator - some strings are not
translatable
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19522
--- Comment #24 from Victor Grousset/tuxayo ---
@Owen it seems that koha-translate doesn't work on a dev install.
http://irc.koha-community.org/koha/2017-11-03#i_1985535
@Dominic how did you installed the language?
https://wiki.koha-community.org/wiki/Installation_of_additional_languages_for_OPAC_and_INTRANET_staff_client#Installing_translations
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 14:56:55 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 13:56:55 +0000
Subject: [Koha-bugs] [Bug 19576] New: opac-detail has multiple Koha::Biblios
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19576
Bug ID: 19576
Summary: opac-detail has multiple Koha::Biblios
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: OPAC
Assignee: oleonard at myacpl.org
Reporter: mtompset at hotmail.com
QA Contact: testopia at bugs.koha-community.org
Marcel noticed this while QA'ing another bug.
Simply remove the duplicate line. Confirm the page loads and works as expected.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 14:57:26 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 13:57:26 +0000
Subject: [Koha-bugs] [Bug 19576] opac-detail has multiple Koha::Biblios
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19576
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|enhancement |trivial
Assignee|oleonard at myacpl.org |mtompset at hotmail.com
Status|NEW |ASSIGNED
CC| |m.de.rooy at rijksmuseum.nl
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 14:57:48 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 13:57:48 +0000
Subject: [Koha-bugs] [Bug 19576] opac-detail has multiple Koha::Biblios
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19576
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |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 Fri Nov 3 15:00:17 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 14:00:17 +0000
Subject: [Koha-bugs] [Bug 19576] opac-detail has multiple Koha::Biblios
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19576
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Patch complexity|--- |Trivial patch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 15:00:19 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 14:00:19 +0000
Subject: [Koha-bugs] [Bug 19576] opac-detail has multiple Koha::Biblios
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19576
--- Comment #1 from M. Tompsett ---
Created attachment 68915
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68915&action=edit
Bug 19576: Remove extra Koha::Biblios
Marcel noticed this while QA'ing another bug.
TEST PLAN
---------
Apply patch and confirm the page still loads and works as expected.
Run Koha QA Test tools
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 15:00:58 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 14:00:58 +0000
Subject: [Koha-bugs] [Bug 13640] Holds To Pull List includes items
unreserveable items
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13640
Marjorie Barry-Vila changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |marjorie.barry-vila at collect
| |o.ca
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 15:06:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 14:06:21 +0000
Subject: [Koha-bugs] [Bug 19576] opac-detail has multiple Koha::Biblios
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19576
M. Tompsett 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 Fri Nov 3 15:16:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 14:16:05 +0000
Subject: [Koha-bugs] [Bug 19576] opac-detail has duplicate 'use
Koha::Biblios; '
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19576
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|opac-detail has multiple |opac-detail has duplicate
|Koha::Biblios |'use Koha::Biblios;'
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 16:12:52 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 15:12:52 +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
Simon Pouchol changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Patch doesn't apply
CC| |simon.pouchol at biblibre.com
--
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 Fri Nov 3 16:35:06 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 15:35:06 +0000
Subject: [Koha-bugs] [Bug 19280] CanBookBeIssued must take a Koha::Patron in
parameter
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19280
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68534|0 |1
is obsolete| |
--- Comment #5 from Jonathan Druart ---
Created attachment 68916
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68916&action=edit
Bug 19280: Pass a Koha::Patron to CanBookBeIssued
We need to make subroutine from C4 use more Koha::Object objects
Seeing bug 19276, starting here is a good start.
Test plan:
The tests should still pass.
Signed-off-by: Jon Knight
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 16:35:00 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 15:35:00 +0000
Subject: [Koha-bugs] [Bug 19280] CanBookBeIssued must take a Koha::Patron in
parameter
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19280
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Patch doesn't apply |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 16:39:58 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 15:39:58 +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
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Patch doesn't apply |Needs Signoff
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 16:59:49 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 15:59:49 +0000
Subject: [Koha-bugs] [Bug 19576] opac-detail has duplicate 'use
Koha::Biblios; '
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19576
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 16:59:54 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 15:59:54 +0000
Subject: [Koha-bugs] [Bug 19576] opac-detail has duplicate 'use
Koha::Biblios; '
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19576
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68915|0 |1
is obsolete| |
--- Comment #2 from Jonathan Druart ---
Created attachment 68917
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68917&action=edit
Bug 19576: Remove extra 'use Koha::Biblios' statement
Marcel noticed this while QA'ing another bug.
TEST PLAN
---------
Apply patch and confirm the page still loads and works as expected.
Run Koha QA Test tools
Signed-off-by: Jonathan Druart
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:00:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:00:36 +0000
Subject: [Koha-bugs] [Bug 19576] opac-detail has duplicate 'use
Koha::Biblios; '
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19576
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |18262
Status|Signed Off |Passed QA
--- Comment #3 from Jonathan Druart ---
Introduced by bug 18262.
Trivial, skipping QA.
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18262
[Bug 18262] Koha::Biblios - Remove GetBiblioData - part 1
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:00:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:00:36 +0000
Subject: [Koha-bugs] [Bug 18262] Koha::Biblios - Remove GetBiblioData - part
1
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18262
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |19576
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19576
[Bug 19576] opac-detail has duplicate 'use Koha::Biblios;'
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:11:11 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:11:11 +0000
Subject: [Koha-bugs] [Bug 19290] Add the ability to filter a result list on
staff client
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19290
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Patch doesn't apply |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:11:16 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:11:16 +0000
Subject: [Koha-bugs] [Bug 19290] Add the ability to filter a result list on
staff client
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19290
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68204|0 |1
is obsolete| |
--- Comment #6 from Jonathan Druart ---
Created attachment 68918
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68918&action=edit
Bug 19290: Browse selected bibliographic records - Staff interface
This patch adds the same feature as bug 10858 for the OPAC interface:
after a search, librarians will be able to browse selected results.
The results can be selected from several pages.
By extension it is possible to add results from several pages to a list
or the cart.
When at least one result is selected, a new "Browse selected records" button
becomes usable and change the behaviour of the existing browser.
The whole feature can be turned off with the pref BrowseResultSelection.
Test plan:
- Launch a search (on the staff interface)
- Check some biblios
- Go on another page
- Check some biblios
- Come back to a page you already check results and confirm that they are
still checked
- Click on the "Browse selected records" button
- Check that you are able to browse results you had checked.
You can also:
- add them to the cart
- add them to a list
QA note: the browsers at the OPAC and the one at the staff interface are
completely different
That's why the code is not mimicking what has been done on bug 10858.
The behaviour must stay the same anyway.
Signed-off-by: S?verine QUEUNE
Signed-off-by: S?verine QUEUNE
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:13:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:13:36 +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
--- Comment #28 from Simon Pouchol ---
Hey Jonathan,
It seems to me the patch isn't applying :
Applying: Bug 6918 - can't place holds on 'on order' items with
AllowOnShelfHolds off
fatal: sha1 information is lacking or useless (C4/Biblio.pm).
error: could not build fake ancestor
Patch failed at 0001 Bug 6918 - can't place holds on 'on order' items with
AllowOnShelfHolds off
--
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 Fri Nov 3 17:27:42 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:27:42 +0000
Subject: [Koha-bugs] [Bug 10748] Add option to block return of lost items
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10748
--- Comment #8 from Jonathan Druart ---
Created attachment 68919
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68919&action=edit
Bug 10748: Add tests
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:27:38 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:27:38 +0000
Subject: [Koha-bugs] [Bug 10748] Add option to block return of lost items
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10748
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:39:12 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:39:12 +0000
Subject: [Koha-bugs] [Bug 10021] Remove dead code related to notifys
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10021
--- Comment #33 from Jonathan Druart ---
(In reply to Marcel de Rooy from comment #30)
> When you remove a sub from a module without full coverage, this is the
> result. Something for QA tools ?
Yes, could be improved. POD failures must be considered as warnings.
(In reply to Marcel de Rooy from comment #31)
> (In reply to Jonathan Druart from comment #8)
> > It appears that has never worked.
> > Could someone confirm?
>
> Did you have any response whatsoever ?
I think we discussed it on IRC, but I do not find the logs.
(In reply to Marcel de Rooy from comment #32)
> QA Comment:
>
> GetOverduesForBranch
> Please add a test. You make some fundamental changes here.
Fundamental? I do not understand what you want me to test.
> koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt
> -
> Not sure. Why do yo remove this line ?
Yes it could have been done somewhere else, it's not directly related to these
changes. There is no "totals*" variables retrieved from the pl script.
> New code:
> + my $accounts = Koha::Account::Lines->search({ borrowernumber =>
> $borrowernumber, }, { order_by => ['accounttype'] });
> + my @accounts;
> + while ( my $account = $accounts->next ) {
> This does not look very nice. Can we get more confusion on $account,
> @accounts or $accounts ?
> Renaming @accounts here might be the least we can do?
> No blocker on itself.
Will fix.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:41:44 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:41:44 +0000
Subject: [Koha-bugs] [Bug 10021] Remove dead code related to notifys
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10021
--- Comment #34 from Jonathan Druart ---
Created attachment 68920
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68920&action=edit
Bug 10021: (QA follow-up) Rename variables
To avoid $account, $accounts and @accounts variables in the same scope
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:45:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:45:21 +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
--- Comment #29 from Jonathan Druart ---
Simon, make sure your branch is up-to-date with origin/master
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:50:09 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:50:09 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68885|0 |1
is obsolete| |
--- Comment #173 from Tom?s Cohen Arazi ---
Created attachment 68921
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68921&action=edit
Bug 7317: Handle backend absense more gracefuly
5/ This patch makes Koha::Illrequest->load_backend raise an exception
if the passed backend is invalid. This way we will catch more errors
introduced.
The patch also disables the 'New Ill request' when no backends are available.
Gets
rid of a related warnings.
Both OPAC and Intranet now display a warning message when no backends
are available.
Tests are added for the load_backend changes.
4/ This patch fixes the path for the checkboxes jquery plugin, and removes the
include
for tablesorter, as this implementation uses Datatables. This is obviously code
for older
Koha, ported to master.
TODO: There's something wrong on the styling. My idea is to get rid
of the custom column visualization tool, and have it display as regular
DataTables. We can then introduce the use of colvis on a separate bug
report.
Note: POD coverage for the exceptions file is wrongly tested. It is a false
positive.
Signed-off-by: Tomas Cohen Arazi
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:50:22 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:50:22 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68886|0 |1
is obsolete| |
--- Comment #174 from Tom?s Cohen Arazi ---
Created attachment 68922
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68922&action=edit
Bug 7317: (QA followup) Get rid of warnings from the tests
Signed-off-by: Tomas Cohen Arazi
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:50:48 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:50:48 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Passed QA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:52:22 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:52:22 +0000
Subject: [Koha-bugs] [Bug 19176] Dates comparison fails on slow server
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19176
--- Comment #27 from Jonathan Druart ---
Created attachment 68923
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68923&action=edit
Bug 19176: (QA follow-up) few cosmetic changes
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:53:18 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:53:18 +0000
Subject: [Koha-bugs] [Bug 19176] Dates comparison fails on slow server
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19176
--- Comment #28 from Jonathan Druart ---
Last patch pushed to master
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 17:54:51 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 16:54:51 +0000
Subject: [Koha-bugs] [Bug 19576] opac-detail has duplicate 'use
Koha::Biblios; '
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19576
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Passed QA |Pushed to Master
--- Comment #4 from Jonathan Druart ---
Pushed to master for 17.11, thanks Mark!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 18:00:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 17:00:05 +0000
Subject: [Koha-bugs] [Bug 11213] GetItemsInfo() called twice
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11213
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Failed QA
--- Comment #128 from Josef Moravec ---
Hi Srdjan,
I did test these patches now, but I am sorry:
The test Search.t is failing after this patchset...
# Failed test 'Warning is raised correctly for invalid tags in MARC::Record'
# at t/db_dependent/Search.t line 681.
there are also so many warnings about preferences missing in mock
virtualshelves on intranet and opac goes to ISE:
DBIx::Class::ResultSet::_construct_results(): Unable to properly collapse
has_many results in iterator mode due to order criteria - performed an eager
cursor slurp underneath. Consider using ->all() instead at
/home/vagrant/kohaclone/Koha/Objects.pm line 209
Can't use string ("1") as an ARRAY ref while "strict refs" in use at
/home/vagrant/kohaclone/C4/XSLT.pm line 290.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 18:29:04 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 17:29:04 +0000
Subject: [Koha-bugs] [Bug 18625] update borrower last seen from SIP
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18625
Jessie Zairo changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jzairo at bywatersolutions.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 18:48:37 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 17:48:37 +0000
Subject: [Koha-bugs] [Bug 19577] New: Secondary email is unable to receive
notices
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19577
Bug ID: 19577
Summary: Secondary email is unable to receive notices
Change sponsored?: ---
Product: Koha
Version: 16.11
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Notices
Assignee: koha-bugs at lists.koha-community.org
Reporter: nick.weingardt at valorchristian.com
QA Contact: testopia at bugs.koha-community.org
Overview: Patrons are able to have multiple email addresses in Koha, but
notices can only be sent to the primary email. The purpose of this bug is to
develop an enhancement that would allow a notice be sent to both emails.
--
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 Fri Nov 3 18:57:47 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 17:57:47 +0000
Subject: [Koha-bugs] [Bug 19559] Elasticsearch QueryAutoTruncate truncate
field names with hyphens if data is quoted
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19559
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68881|0 |1
is obsolete| |
--- Comment #5 from David Bourgault ---
Created attachment 68924
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68924&action=edit
Bug 19559 - Unit tests
Signed-off-by: David Bourgault
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 18:57:49 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 17:57:49 +0000
Subject: [Koha-bugs] [Bug 19559] Elasticsearch QueryAutoTruncate truncate
field names with hyphens if data is quoted
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19559
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68882|0 |1
is obsolete| |
--- Comment #6 from David Bourgault ---
Created attachment 68925
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68925&action=edit
Bug 19559 - Add '-' to list of characters we don't split search terms on
Test plan assumes
QueryAutoTruncate = automatically
SearchEngine = Elasticsearch
To test:
0 - Apply Unit test patch only
1 - prove t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t
2 - Should fail
3 - Apply this patch
4 - prove t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t
5 - should pass
6 - search for 'Local-number:"4"' (or a vlid biblionumber)
7 - should get expected result
Signed-off-by: David Bourgault
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 18:58:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 17:58:21 +0000
Subject: [Koha-bugs] [Bug 19559] Elasticsearch QueryAutoTruncate truncate
field names with hyphens if data is quoted
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19559
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
CC| |david.bourgault at inlibro.com
--- Comment #7 from David Bourgault ---
Works as described in test plan.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 19:09:16 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 18:09:16 +0000
Subject: [Koha-bugs] [Bug 18725] Process_message_queue sends duplicate
emails if message_queue is not writable.
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18725
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |david.bourgault at inlibro.com
--- Comment #8 from David Bourgault ---
It would be easier if you could provide maybe a hint on how to block access to
the message_queue table? Not sure how to test this otherwise.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 19:24:51 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 18:24:51 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68561|0 |1
is obsolete| |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 19:29:22 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 18:29:22 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
--- Comment #175 from Tom?s Cohen Arazi ---
Created attachment 68926
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68926&action=edit
Bug 7317: opac.css update
Signed-off-by: Tomas Cohen Arazi
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 19:29:26 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 18:29:26 +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
Caroline Cyr La Rose changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |caroline.cyr-la-rose at inlibr
| |o.com
--- Comment #9 from Caroline Cyr La Rose ---
The test plan works.
However, is it normal that if I add <> in the notice it comes back
as blank in the message_queue table?
I checked and my notes are in the issues table.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 19:34:01 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 18:34:01 +0000
Subject: [Koha-bugs] [Bug 17656] Irregularities in serial prediction pattern
are planned only for current subscription
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17656
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |david.bourgault at inlibro.com
Status|Needs Signoff |Failed QA
--- Comment #20 from David Bourgault ---
Test plan works as described, but QA tools warns of forbidden patterns.
koha-tmpl/intranet-tmpl/prog/en/includes/serials/subscription_daily_choice.inc
FAIL forbidden patterns
forbidden pattern: Do not use line breaks inside template tags
(bug 18675) (line 5)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 19:40:33 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 18:40:33 +0000
Subject: [Koha-bugs] [Bug 19578] New: TT syntax for notices - There is no
way to pre-process DB fields
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19578
Bug ID: 19578
Summary: TT syntax for notices - There is no way to pre-process
DB fields
Change sponsored?: ---
Product: Koha
Version: unspecified
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: Notices
Assignee: koha-bugs at lists.koha-community.org
Reporter: jonathan.druart at bugs.koha-community.org
QA Contact: testopia at bugs.koha-community.org
Jenkins fails (run 287) on a test in t/db_dependent/Letters/TemplateToolkit.t:
With the historical syntax:
# Your request for an article from tQYRS (c3Av58O0P5xkkIGu) has been
canceled for the following reason:
With the TT syntax:
# Your request for an article from tQYRS_ (c3Av58O0P5xkkIGu) has been
canceled for the following reason:
The last character of the biblio's title has been removed because it's a
punctuation character.
It comes from: C4::Letters::_parseletter
893 $val =~ s/\p{P}$// if $val && $table=~/biblio/;
The same replacement is done for patron's attributes too.
With the TT syntax we retrieve the objects and send them to the template,
without fetching the attributes.
To restore this behaviour we could apply this regex on all the fields of the
biblio objects, but it could be perf killer (we will process fields that we
will certainly not used in the template).
I thought about the TT filter we could apply on all the variables we display,
but we will not know where come from the data, and we need to apply this change
only for the biblio table.
Any other ideas?
--
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 Fri Nov 3 19:41:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 18:41:05 +0000
Subject: [Koha-bugs] [Bug 19578] TT syntax for notices - There is no way to
pre-process DB fields
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19578
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |kyle at bywatersolutions.com,
| |tomascohen at gmail.com
Status|NEW |In Discussion
--
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 Fri Nov 3 19:46:07 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 18:46:07 +0000
Subject: [Koha-bugs] [Bug 19578] TT syntax for notices - There is no way to
pre-process DB fields
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19578
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |m.de.rooy at rijksmuseum.nl
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 19:55:32 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 18:55:32 +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
Dominic Pichette changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |dominic.pichette at inlibro.co
| |m
--- Comment #30 from Dominic Pichette ---
I did a reset --hard on origin/master and still have the same error than
Simon..
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 20:06:49 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:06:49 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68921|0 |1
is obsolete| |
--- Comment #176 from Tom?s Cohen Arazi ---
Created attachment 68927
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68927&action=edit
Bug 7317: Handle backend absense more gracefuly
5/ This patch makes Koha::Illrequest->load_backend raise an exception
if the passed backend is invalid. This way we will catch more errors
introduced.
The patch also disables the 'New Ill request' when no backends are available.
Gets
rid of a related warnings.
Both OPAC and Intranet now display a warning message when no backends
are available.
Tests are added for the load_backend changes.
4/ This patch fixes the path for the checkboxes jquery plugin, and removes the
include
for tablesorter, as this implementation uses Datatables. This is obviously code
for older
Koha, ported to master.
TODO: There's something wrong on the styling. My idea is to get rid
of the custom column visualization tool, and have it display as regular
DataTables. We can then introduce the use of colvis on a separate bug
report.
Note: POD coverage for the exceptions file is wrongly tested. It is a false
positive.
Signed-off-by: Tomas Cohen Arazi
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 20:07:01 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:07:01 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68922|0 |1
is obsolete| |
--- Comment #177 from Tom?s Cohen Arazi ---
Created attachment 68928
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68928&action=edit
Bug 7317: (QA followup) Get rid of warnings from the tests
Signed-off-by: Tomas Cohen Arazi
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 20:07:12 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:07:12 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68926|0 |1
is obsolete| |
--- Comment #178 from Tom?s Cohen Arazi ---
Created attachment 68929
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68929&action=edit
Bug 7317: opac.css update
Signed-off-by: Tomas Cohen Arazi
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 20:09:06 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:09:06 +0000
Subject: [Koha-bugs] [Bug 19014] Patrons should not get a on_reserve
notification if the checkout is due in the future
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19014
Caroline Cyr La Rose changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |caroline.cyr-la-rose at inlibr
| |o.com
Status|Needs Signoff |Failed QA
--- Comment #5 from Caroline Cyr La Rose ---
Hi Jonathan,
I'm having trouble testing this. Which cron job are we supposed to run? Is it
automatic_renewals.pl --send-notices?
Furthermore, I had an item due in the future (in december) and I still received
a notice saying the auto renewal failed because I had fines (which was true).
But why would it try to renew something due in december?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 20:10:17 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:10:17 +0000
Subject: [Koha-bugs] [Bug 18357] On-site checkouts issues with 'Unlimited'
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18357
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
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 Fri Nov 3 20:10:20 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:10:20 +0000
Subject: [Koha-bugs] [Bug 18357] On-site checkouts issues with 'Unlimited'
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18357
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68896|0 |1
is obsolete| |
--- Comment #5 from David Bourgault ---
Created attachment 68930
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68930&action=edit
Bug 18357: Handle unlimited on-site checkouts
If on-site checkouts are set to unlimited (i.e. NULL/undef), they are
currently blocked.
Test plan:
1/ Set All/All rule with Unlimited/unlimited for normal/onsite checkouts
2/ Will be able to perform onsite checkout
3/ Edit rule to be 15/Unlimited normal/onsite
4/ Will be able to perform onsite checkout
=> Without this patch it was blocked
5/ Set rule to 15/15
6/ Onsite checkouts work again
Signed-off-by: David Bourgault
--
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 Fri Nov 3 20:12:10 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:12:10 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68927|0 |1
is obsolete| |
--- Comment #179 from Tom?s Cohen Arazi ---
Created attachment 68931
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68931&action=edit
Bug 7317: Handle backend absense more gracefuly
5/ This patch makes Koha::Illrequest->load_backend raise an exception
if the passed backend is invalid. This way we will catch more errors
introduced.
The patch also disables the 'New Ill request' when no backends are available.
Gets
rid of a related warnings.
Both OPAC and Intranet now display a warning message when no backends
are available.
Tests are added for the load_backend changes.
4/ This patch fixes the path for the checkboxes jquery plugin, and removes the
include
for tablesorter, as this implementation uses Datatables. This is obviously code
for older
Koha, ported to master.
TODO: There's something wrong on the styling. My idea is to get rid
of the custom column visualization tool, and have it display as regular
DataTables. We can then introduce the use of colvis on a separate bug
report.
Note: POD coverage for the exceptions file is wrongly tested. It is a false
positive.
Signed-off-by: Tomas Cohen Arazi
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 20:12:23 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:12:23 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68928|0 |1
is obsolete| |
--- Comment #180 from Tom?s Cohen Arazi ---
Created attachment 68932
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68932&action=edit
Bug 7317: (QA followup) Get rid of warnings from the tests
Signed-off-by: Tomas Cohen Arazi
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 20:12:35 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:12:35 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68929|0 |1
is obsolete| |
--- Comment #181 from Tom?s Cohen Arazi ---
Created attachment 68933
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68933&action=edit
Bug 7317: opac.css update
Signed-off-by: Tomas Cohen Arazi
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 20:22:24 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:22:24 +0000
Subject: [Koha-bugs] [Bug 16787] 'Too many holds' message appears
inappropriately and is missing data
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16787
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Failed QA
CC| |david.bourgault at inlibro.com
--- Comment #7 from David Bourgault ---
Is there a step missing form the test plan? I applied the patch, change my
Circulation rules to set no holds for All/All (I have no other rules for my
test patron's category) and tried to place a hold but got the message "No
available items.", but I can see it as available on the OPAC detail 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 Fri Nov 3 20:26:22 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:26:22 +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
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68898|0 |1
is obsolete| |
--- Comment #31 from Jonathan Druart ---
Created attachment 68934
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68934&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 Fri Nov 3 20:45:25 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:45:25 +0000
Subject: [Koha-bugs] [Bug 19444] Automatic renewal script should not
auto-renew if a patron' s record has expired
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19444
Dominic Pichette changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |dominic.pichette at inlibro.co
| |m
--- Comment #10 from Dominic Pichette ---
I tried it, I started by setting BlockExpiredPatronOpacActions to set, checked
out an item for a particular patron, change the expiration date of that patron
so he is now expired and then executed automatic_renewals.pl. The patch didn't
seem to make a difference, it passed in both cases..
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 20:54:37 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:54:37 +0000
Subject: [Koha-bugs] [Bug 19485] Limiting by title-series (se) broken on ES
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19485
--- Comment #3 from David Bourgault ---
The mappings are supposed to be located in
admin/searchengine/elasticsearch/mappings.yaml
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 20:55:12 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:55:12 +0000
Subject: [Koha-bugs] [Bug 7317] Add an Interlibrary Loan Module to
Circulation and OPAC
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317
--- Comment #182 from Jonathan Druart ---
1/ Error handling on borrowernumber when creating or updating a request
(backend issue?)
2/ Capitalization problems, deprecated words (Branch, borrowers), etc
3/ Use dateformat to display dates
4/ ill/ill-requests.pl?method=generic_confirm&illrequest_id=3
click "send email"
Software error to handle correctly:
No target email addresses found. Either select at least one partner or check
your ILL partner library records. at /home/vagrant/kohaclone/Koha/Illrequest.pm
line 896.
5/ Same for "Your branch has no email address. Please set it. at
/home/vagrant/kohaclone/Koha/Illrequest.pm line 901."
6/ Should not we use message_queue instead of sendmail directly?
7/ Maybe we should add a warning to the about page if the partner_code defined
in the config does not match a category_code (?)
(not finished)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 20:56:17 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 19:56:17 +0000
Subject: [Koha-bugs] [Bug 19444] Automatic renewal script should not
auto-renew if a patron' s record has expired
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19444
--- Comment #11 from Dominic Pichette ---
I tried it, I started by setting BlockExpiredPatronOpacActions to set, checked
out an item for a particular patron, change the expiration date of that patron
so he is now expired and then executed automatic_renewals.pl. The patch didn't
seem to make a difference, it passed in both cases..
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:03:43 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:03:43 +0000
Subject: [Koha-bugs] [Bug 19542] Koha should display Elasticsearch
information in the about page
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19542
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:03:45 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:03:45 +0000
Subject: [Koha-bugs] [Bug 19542] Koha should display Elasticsearch
information in the about page
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19542
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68861|0 |1
is obsolete| |
--- Comment #5 from David Bourgault ---
Created attachment 68935
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68935&action=edit
Bug 19542: Add a check for ES configuration health
This patch adds a new statuc function to Koha::SearchEngine::ElasticSearch
which is instended to replace most of get_elasticsearch_params. This function
reads the configuration from C4::Context->config('elasticsearch') and raises
relevant exceptions when mandatory entries are missing.
Its behaviour is covered by tests.
To test:
- Run:
$ kshell
k$ prove t/Koha/SearchEngine/Elasticsearch.t
=> SUCCESS: Tests pass!
- Sign off :-D
Signed-off-by: David Bourgault
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:03:59 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:03:59 +0000
Subject: [Koha-bugs] [Bug 19542] Koha should display Elasticsearch
information in the about page
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19542
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68862|0 |1
is obsolete| |
--- Comment #6 from David Bourgault ---
Created attachment 68936
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68936&action=edit
Bug 19542: Add Elasticsearch information in the 'About' page
This patch adds Elasticsearch related information to the 'About' page.
The information is gathered and displayed only when the 'SearchEngine' syspref
is set to 'Elasticsearch'. It displays configured nodes, and the status:
- Running
- Not running
In case it is running, it displays the defined indices and the document count
on each.
If there are configuration problems, exceptions are catch and a convenient
warning
message is displayed.
To test:
- Apply this patches
- Run:
$ kshell
k$ prove t/Koha/SearchEngine/Elasticsearch.t
=> SUCCESS: Tests pass!
- Have ES configured in your koha-conf.xml file (by default in kohadevbox)
- Set the 'SearchEngine' syspref to 'Elasticsearch'
- Comment out pieces of the elasticsearch-specific entries (server, index_name,
the whole elasticsearch block). Reload on each change.
=> SUCCESS: Warning messages are displayed and make sense in the context of
your changes.
----> the rest of the tests require having ES running on the dev env. This can
be easily
achieved by creating the kohadevbox using:
$ KOHA_ELASTICSEARCH=1 vagrant up
- Stop the 'elasticsearch' service:
$ sudo service elasticsearch stop
- Reload about.pl
=> SUCCESS: The configured nodes are displayed, and the status is 'not running'
- Start the 'elasticsearch' service:
$ sudo service elasticsearch start
- Reload about.pl
=> SUCCESS: The configured nodes are displayed, the status is 'running' and
created indices info is displayed, along with the document count
on each index.
- Sign off :-D
Signed-off-by: David Bourgault
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:07:47 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:07:47 +0000
Subject: [Koha-bugs] [Bug 19482] Elasticsearch - prevent removal / editing
of required indexes
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19482
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Patch doesn't apply
CC| |david.bourgault at inlibro.com
--- Comment #4 from David Bourgault ---
The 2nd patch does not apply on current master. (Lacking or useless SHA1)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:14:14 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:14:14 +0000
Subject: [Koha-bugs] [Bug 19066] Add branchcode to accountlines
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19066
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Patch doesn't apply
CC| |david.bourgault at inlibro.com
--- Comment #10 from David Bourgault ---
Patch doesnt apply for me on master.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:17:15 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:17:15 +0000
Subject: [Koha-bugs] [Bug 17672] Items table should have a damaged_on column
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17672
Charles Farmer changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |charles.farmer at inlibro.com
--- Comment #7 from Charles Farmer ---
Since we see the date at which the item was damaged on the moredetail.pl,
should we also see this information in the header table of the additem.pl page
along the status? Just pitching the idea out there.
Everything worked fine, though I had to bump up the number of tests from 12 to
13 in order to make it work on the current master.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:21:18 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:21:18 +0000
Subject: [Koha-bugs] [Bug 19522] Label creator - some strings are not
translatable
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19522
--- Comment #25 from Dominic Pichette ---
i translated using misc/translator/translate fr-CA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:24:08 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:24:08 +0000
Subject: [Koha-bugs] [Bug 18948] Reindexing should use aliases to avoid down
time
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18948
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |david.bourgault at inlibro.com
Status|Needs Signoff |Patch doesn't apply
--- Comment #6 from David Bourgault ---
Patch does not apply, SHA1 error for Indexer.pm
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:25:30 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:25:30 +0000
Subject: [Koha-bugs] [Bug 17672] Items table should have a damaged_on column
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17672
Charles Farmer changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #65551|0 |1
is obsolete| |
Attachment #65552|0 |1
is obsolete| |
--- Comment #9 from Charles Farmer ---
Created attachment 68938
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68938&action=edit
Bug 17672 - Schema updates
Signed-off-by: Owen Leonard
Signed-off-by: Charles Farmer
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:25:15 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:25:15 +0000
Subject: [Koha-bugs] [Bug 17672] Items table should have a damaged_on column
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17672
--- Comment #8 from Charles Farmer ---
Created attachment 68937
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68937&action=edit
Bug 17672 - Items table should have a damaged_on column
This patchset adds a 'damaged_on' column to store the date an item is
marked damaged, analogous to withdrawn_on and itemlost_on
To test:
1 - Apply patch
2 - Mark an item damaged via moredetail.pl (Items tab on left in
details)
3 - Note the damaged on date apears below
4 - Unmark the item, the date is removed
5 - Go to the edit items screen (from top bar 'Edit->edit items')
6 - Mark item damaged - check db or moredetails.pl to see damaged_on
date
7 - Unmark item damaged - confirm date is removed
8 - prove t/db_dependent/Items.t
Signed-off-by: Owen Leonard
Signed-off-by: Charles Farmer
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:25:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:25:36 +0000
Subject: [Koha-bugs] [Bug 17672] Items table should have a damaged_on column
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17672
--- Comment #10 from Charles Farmer ---
Created attachment 68939
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68939&action=edit
Bug 17672 - Dumping Items.t expected tests from 12 to 13
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:29:44 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:29:44 +0000
Subject: [Koha-bugs] [Bug 18950] Elasticsearch - Add password access for
admin functions
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18950
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Failed QA
CC| |david.bourgault at inlibro.com
--- Comment #3 from David Bourgault ---
Patch applies, and basic functionality is there, however:
- "Reindex" checked for password correctly but failed since 18948 does not
apply for me (not an issue, just a note)
- The password input field is of type=text but should be of type=password
- The error message when you enter a wrong password should probably more
obvious (like being red, or using the standard koha warning boxes)
- I'd suggest moving the XML node inside the
group, seems cleaner to me.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:30:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:30:21 +0000
Subject: [Koha-bugs] [Bug 18936] Convert issuingrules fields to
circulation_rules
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18936
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |david.bourgault at inlibro.com
--- Comment #8 from David Bourgault ---
No test plan?
--
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 Fri Nov 3 21:38:33 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:38:33 +0000
Subject: [Koha-bugs] [Bug 18213] Add language facets to Elasticsearch
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18213
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Patch doesn't apply
CC| |david.bourgault at inlibro.com
--- Comment #5 from David Bourgault ---
Does not apply.
fatal: sha1 information is lacking or useless
(Koha/SearchEngine/Elasticsearch.pm).
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:39:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:39:21 +0000
Subject: [Koha-bugs] [Bug 19166] Add the ability to add adjustments to an
invoice
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19166
Caroline Cyr La Rose changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Failed QA
CC| |caroline.cyr-la-rose at inlibr
| |o.com
--- Comment #16 from Caroline Cyr La Rose ---
I want this so much to work! I've been asked for this many many times!
But I got a software error.
BIx::Class::Storage::DBI::_dbh_execute(): Table
'koha_master_dev_inlibro.invoice_adjustments' doesn't exist at
/inlibro/git/koha-master-dev-inlibro/Koha/Object.pm line 121
Plus the prove -v ... returned red.
DBD::mysql::st execute failed: Table
'koha_master_dev_inlibro.invoice_adjustments' doesn't exist [for Statement
"SELECT `me`.`adjustment_id`, `me`.`invoiceid`, `me`.`adjustment`,
`me`.`reason`, `me`.`note`, `me`.`budget_id`, `me`.`encumber_open`,
`me`.`timestamp` FROM `invoice_adjustments` `me` JOIN `aqinvoices` `invoiceid`
ON `invoiceid`.`invoiceid` = `me`.`invoiceid` WHERE ( ( `budget_id` = ? AND
`closedate` IS NOT NULL ) )" with ParamValues: 0='6'] at
/usr/share/perl5/DBIx/Class/Storage/DBI.pm line 1765.
DBIx::Class::Storage::DBI::_dbh_execute(): Table
'koha_master_dev_inlibro.invoice_adjustments' doesn't exist at
/inlibro/git/koha-master-dev-inlibro/Koha/Objects.pm line 209
# Looks like your test exited with 255 just after 60.
Dubious, test returned 255 (wstat 65280, 0xff00)
Failed 85/145 subtests
Test Summary Report
-------------------
t/db_dependent/Budgets.t (Wstat: 65280 Tests: 60 Failed: 0)
Non-zero exit status: 255
Parse errors: Bad plan. You planned 145 tests but ran 60.
Files=1, Tests=60, 1 wallclock secs ( 0.02 usr 0.00 sys + 1.49 cusr 0.08
csys = 1.59 CPU)
Result: FAIL
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:40:55 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:40:55 +0000
Subject: [Koha-bugs] [Bug 13188] Mandatory fields are wrong in patron
personal details page
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13188
Ola Andersson changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |ola.andersson at ltu.se
--- Comment #3 from Ola Andersson ---
It is not possible with Koha 17.05.05 to configure a set of mandatory fields in
SelfRegistration and at the same time configure another set of mandatory fields
in SelfModification. This seems to be a design flaw.
E.g. we would like to have Address2 field mandatory in SelfRegistration but
optional in SelfModification.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:49:33 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:49:33 +0000
Subject: [Koha-bugs] [Bug 19454] Script
borrowers-force-messaging-defaults.pl should be able to filter on patron
category
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19454
Charles Farmer changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |charles.farmer at inlibro.com
--- Comment #6 from Charles Farmer ---
Can't really say if the problem is on my end or not, but I couldn't apply your
patches:
> fatal: sha1 information is lacking or useless (misc/maintenance/borrowers-force-messaging-defaults.pl)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:54:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:54:21 +0000
Subject: [Koha-bugs] [Bug 16461] Add some information (branch and barcode)
in fine description for overdue
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16461
David Bourgault changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |david.bourgault at inlibro.com
--- Comment #8 from David Bourgault ---
Missing a test plan
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 21:55:27 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 20:55:27 +0000
Subject: [Koha-bugs] [Bug 18602] SPAM
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18602
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|intuitI8OO3672164 |SPAM
|Quickbooks Payroll Tech |
|Support Phone Number |
CC| |katrin.fischer at bsz-bw.de
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 22:00:23 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 21:00:23 +0000
Subject: [Koha-bugs] [Bug 19452] The -truncate option in
borrowers-force-messaging-defaults.pl should not remove category
preferences
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19452
Charles Farmer changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |charles.farmer at inlibro.com
--- Comment #6 from Charles Farmer ---
Can't really say where the problem lies, but I couldn't apply your patches:
> fatal: sha1 information is lacking or useless (misc/maintenance/borrowers-force-messaging-defaults.pl).
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 22:05:26 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 21:05:26 +0000
Subject: [Koha-bugs] [Bug 19471] Show creation date in patron restrictions
list
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19471
Charles Farmer changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68180|0 |1
is obsolete| |
--- Comment #3 from Charles Farmer ---
Created attachment 68940
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68940&action=edit
Bug 19471: Show creation date in patron restrictions list
Test plan:
1. Go to the "Details" vertical tab of a patron.
2. Click the "Restriction" tab in the bottom and add a manual restriction.
3. Verify a creation date is visible.
4. Edit the same patron.
5. Under "Patron restrictions" verify the creation date of the listed
restiction is visible.
Signed-off-by: Owen Leonard
Signed-off-by: Charles Farmer
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 22:08:40 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 21:08:40 +0000
Subject: [Koha-bugs] [Bug 19549] Reserve Slip prints first found hold when
patron has multiple holds on a biblio
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19549
Caroline Cyr La Rose changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |caroline.cyr-la-rose at inlibr
| |o.com
--- Comment #2 from Caroline Cyr La Rose ---
Is this for multiple holds on the same biblio?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 22:13:37 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 21:13:37 +0000
Subject: [Koha-bugs] [Bug 19549] Reserve Slip prints first found hold when
patron has multiple holds on a biblio
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19549
--- Comment #3 from Jason Palmer ---
(In reply to Caroline Cyr La Rose from comment #2)
> Is this for multiple holds on the same biblio?
Yes. Multiple holds for same bib.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 22:54:53 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 21:54:53 +0000
Subject: [Koha-bugs] [Bug 19579] New:
PatronSelfRegistrationEmailMustBeUnique does not prevent duplicates when
using PatronSelfRegistrationVerifyByEmail
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19579
Bug ID: 19579
Summary: PatronSelfRegistrationEmailMustBeUnique does not
prevent duplicates when using
PatronSelfRegistrationVerifyByEmail
Change sponsored?: ---
Product: Koha
Version: 16.05
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: OPAC
Assignee: oleonard at myacpl.org
Reporter: baselibrary.consortium at nhs.net
QA Contact: testopia at bugs.koha-community.org
With PatronSelfRegistrationVerifyByEmail set to Require and
PatronSelfRegistrationEmailMustBeUnique set to Consider a patron can still self
register many times using the same email address.
Test case
1. Patron submits self registration form using the same email address 3 times
2. Patron receives 3 verification emails
3. Patron clicks on 3 verify token URLs
4. Staff client now shows the 3 patrons
5. Patron logs into 3 accounts on OPAC
6. Patron cannot change personal details because email address already exists
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Nov 3 22:55:47 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 03 Nov 2017 21:55:47 +0000
Subject: [Koha-bugs] [Bug 19579] PatronSelfRegistrationEmailMustBeUnique
does not prevent duplicates when using PatronSelfRegistrationVerifyByEmail
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19579
BASE Library Consortium changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |baselibrary.consortium at nhs.
| |net
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 01:19:39 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 00:19:39 +0000
Subject: [Koha-bugs] [Bug 19420] Improve display of errors from failure of
uploading file during stage import
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19420
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 01:19:41 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 00:19:41 +0000
Subject: [Koha-bugs] [Bug 19420] Improve display of errors from failure of
uploading file during stage import
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19420
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68910|0 |1
is obsolete| |
--- Comment #12 from Nick Clemens ---
Created attachment 68941
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68941&action=edit
Bug 19420 - Add errors on other pages
Fix error array (add '0' value)
Update errors on other places using upload.js
Eventually these should all be using the same code in a js file
upload.tt already dealt with these errors, but has diff code, made it
work with new error syntax
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 01:21:44 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 00:21:44 +0000
Subject: [Koha-bugs] [Bug 19420] Improve display of errors from failure of
uploading file during stage import
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19420
--- Comment #13 from Nick Clemens ---
(In reply to Marcel de Rooy from comment #11)
> Upload local cover image
> Upload status: Failed - (2) Root directory for uploads not defined
> No, the directory was not writable.
> This went wrong since the errcode is 2, but you forgot to add a zero entry
> in you errMsgs array. So they are one position out of sync.
> Please add a leading entry here (see also upload.tt)
> var errMsgs = [ _("File already exists"), _("Directory is not writeable"),
> _("Root directory for uploads not defined"), _("Temporary directory for
> uploads not defined") ];
> Adjust same thing in two other places ?
Adjusted :-)
>
> //Next three errors have no code path to trigger that I see
> I am very confident that I tested all codes when I wrote this. They may be
> hard to simulate, but I did. Please remove this comment.
Removed, I though we only passed 1-4 back in the pm file, my apologies
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 02:27:22 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 01:27:22 +0000
Subject: [Koha-bugs] [Bug 19580] New: Elasticsearch: QueryAutoTruncate
exclude period as splitting character in autotruncation
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19580
Bug ID: 19580
Summary: Elasticsearch: QueryAutoTruncate exclude period as
splitting character in autotruncation
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: nick at bywatersolutions.com
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 02:31:45 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 01:31:45 +0000
Subject: [Koha-bugs] [Bug 19580] Elasticsearch: QueryAutoTruncate exclude
period as splitting character in autotruncation
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19580
--- Comment #1 from Nick Clemens ---
Created attachment 68942
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68942&action=edit
Bug 19580: Unit tests
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 02:31:47 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 01:31:47 +0000
Subject: [Koha-bugs] [Bug 19580] Elasticsearch: QueryAutoTruncate exclude
period as splitting character in autotruncation
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19580
--- Comment #2 from Nick Clemens ---
Created attachment 68943
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68943&action=edit
Bug 19580: Exclude period as splitting character for autotrunction
To test:
0 - Apply Unit test patch only
1 - prove t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t
2 - Should fail
3 - Apply this patch
4 - prove t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t
5 - should pass
6 - search for 'Local-number.raw:"4"' (or a vlid biblionumber)
7 - should get expected result
--
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 Sat Nov 4 02:31:43 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 01:31:43 +0000
Subject: [Koha-bugs] [Bug 19580] Elasticsearch: QueryAutoTruncate exclude
period as splitting character in autotruncation
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19580
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Patch complexity|--- |Trivial patch
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 Sat Nov 4 02:32:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 01:32:36 +0000
Subject: [Koha-bugs] [Bug 19580] Elasticsearch: QueryAutoTruncate exclude
period as splitting character in autotruncation
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19580
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|koha-bugs at lists.koha-commun |nick at bywatersolutions.com
|ity.org |
Depends on| |19559
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19559
[Bug 19559] Elasticsearch QueryAutoTruncate truncate field names with hyphens
if data is quoted
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 02:32:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 01:32:36 +0000
Subject: [Koha-bugs] [Bug 19559] Elasticsearch QueryAutoTruncate truncate
field names with hyphens if data is quoted
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19559
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |19580
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19580
[Bug 19580] Elasticsearch: QueryAutoTruncate exclude period as splitting
character in autotruncation
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 02:33:35 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 01:33:35 +0000
Subject: [Koha-bugs] [Bug 19559] Elasticsearch QueryAutoTruncate truncate
field names with hyphens if data is quoted
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19559
--- Comment #8 from Nick Clemens ---
(In reply to David Gustafsson from comment #4)
> Would be nice if "." could as be included as a valid field characters, since
> it is used for subfields in Elasticsearch. ("Control-number.raw" for
> example).
This could be done, opening a new bug 19580
We may need more adjustments to when we do and do not truncate/split
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 02:42:40 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 01:42:40 +0000
Subject: [Koha-bugs] [Bug 16787] 'Too many holds' message appears
inappropriately and is missing data
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16787
--- Comment #8 from Nick Clemens ---
(In reply to David Bourgault from comment #7)
> Is there a step missing form the test plan? I applied the patch, change my
> Circulation rules to set no holds for All/All (I have no other rules for my
> test patron's category) and tried to place a hold but got the message "No
> available items.", but I can see it as available on the OPAC detail page.
Sorry, I did miss a detail, this patch only affects the staff side, not the
OPAC
--
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 Sat Nov 4 02:42:50 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 01:42:50 +0000
Subject: [Koha-bugs] [Bug 16787] 'Too many holds' message appears
inappropriately and is missing data
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16787
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |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 Sat Nov 4 03:01:03 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 02:01:03 +0000
Subject: [Koha-bugs] [Bug 19166] Add the ability to add adjustments to an
invoice
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19166
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Needs Signoff
--- Comment #17 from Nick Clemens ---
(In reply to Caroline Cyr La Rose from comment #16)
> I want this so much to work! I've been asked for this many many times!
>
> But I got a software error.
>
> BIx::Class::Storage::DBI::_dbh_execute(): Table
> 'koha_master_dev_inlibro.invoice_adjustments' doesn't exist at
> /inlibro/git/koha-master-dev-inlibro/Koha/Object.pm line 121
>
> Plus the prove -v ... returned red.
HI Caroline, Did you run the database updates? It looks like the tables
introduced in this patchset have not been created.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 03:10:42 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 02:10:42 +0000
Subject: [Koha-bugs] [Bug 18213] Add language facets to Elasticsearch
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18213
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #64798|0 |1
is obsolete| |
Attachment #64841|0 |1
is obsolete| |
Attachment #64905|0 |1
is obsolete| |
--- Comment #6 from Nick Clemens ---
Created attachment 68944
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68944&action=edit
Bug 18213 - Add language facets to Elasticsearch
This patch add language as a facet to ES results - it adds
a new template plugin for languages to get the appropriate
description given an iso 639-2 code
To test:
1 - Make sure you have records with differing languages (in the MARC21 008
field characters 35-37 or UNIMARC 101a)
2 - Apply patch
3 - Reload Elasticsearch settings:
http://localhost:8081/cgi-bin/koha/admin/searchengine/elasticsearch/mappings.pl?op=reset&i_know_what_i_am_doing=1
4 - Reindex your records
5 - Search for a phrase that will return results in several languages
6 - Verify you see factes correctly labelled for 'Language'
7 - Verify the facets work
8 - Verify both opac and staff results
9 - prove t/db_dependent/Languages.t
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 03:10:45 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 02:10:45 +0000
Subject: [Koha-bugs] [Bug 18213] Add language facets to Elasticsearch
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18213
--- Comment #7 from Nick Clemens ---
Created attachment 68945
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68945&action=edit
Bug 18213 - DB Update (facet order)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 03:10:48 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 02:10:48 +0000
Subject: [Koha-bugs] [Bug 18213] Add language facets to Elasticsearch
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18213
--- Comment #8 from Nick Clemens ---
Created attachment 68946
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68946&action=edit
Bug 18213 - Add Template plugin and fix for C4/Languages
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 03:10:40 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 02:10:40 +0000
Subject: [Koha-bugs] [Bug 18213] Add language facets to Elasticsearch
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18213
Nick Clemens 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 Sat Nov 4 10:27:58 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 09:27:58 +0000
Subject: [Koha-bugs] [Bug 10708] make fine receipt customizable
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10708
Jamsheer N P changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jamsheer.np at gmail.com
--- Comment #7 from Jamsheer N P ---
There shall be an option for receipt number in the payment slip which can be
reset at beginning of the financial year. So that one need not have to depend
on a manual receipt writing/printing. With the help of a unique receipt number,
payment can be verified later (especially where financial auditing in library
is mandatory).
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 21:27:34 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 20:27:34 +0000
Subject: [Koha-bugs] [Bug 17314] REST API: Add API route to create,
list and delete a purchase suggestion
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17314
--- Comment #21 from Jiri Kozlovsky ---
(In reply to Jon Knight from comment #20)
> I noticed that this doesn't apply cleanly to the latest master HEAD on my
> kohadevbox. I could help rebase if that would be useful to nudge this bug
> along? Unless of course Jiri is working on an updated version?
Well, I've been working on rebasing on top of master HEAD for a little time
already and I'm near completion.
Right now I'm actually solving a really strange bug when GET on an item
(/suggestions/{id}) works perfectly fine, but GET on a list of items
(/suggestions) fails on "Method not implemented" although I've basically copy &
pasted the /patrons implementation.
As soon as I'll solve this strange behaviour, I'll post my work here ;) But
thank you for your effort.(In reply to Jon Knight from comment #19)
> Is development of this patch still ongoing? Also PUT appears to actually be
> a PATCH operation as it doesn't require the whole object data to be
> specified - should the HTTP method be changed to reflect that?
I'm going to fix this too in my next patch.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sat Nov 4 21:37:00 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 20:37:00 +0000
Subject: [Koha-bugs] [Bug 19538] Advanced editor - Rancor - Move syspref
from labs to cataloging and remove experimental note
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19538
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |katrin.fischer at bsz-bw.de
Status|In Discussion |Signed Off
--- Comment #4 from Katrin Fischer ---
We discussed this at the last meeting and agreed to move it out of labs. There
are no open bug reports, just ongoing enhancements. I think it's ripe to leave
the labs.
https://wiki.koha-community.org/wiki/Development_IRC_meeting_11_October_2017
--
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 Sat Nov 4 21:41:40 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 20:41:40 +0000
Subject: [Koha-bugs] [Bug 19538] Advanced editor - Rancor - Move syspref
from labs to cataloging and remove experimental note
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19538
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |In Discussion
--
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 Sat Nov 4 22:51:23 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 04 Nov 2017 21:51:23 +0000
Subject: [Koha-bugs] [Bug 19580] Elasticsearch: QueryAutoTruncate exclude
period as splitting character in autotruncation
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19580
David Gustafsson changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |glasklas at gmail.com
--- Comment #3 from David Gustafsson ---
I think you need to escape that hyphen unless its the last character within the
character class brackets (like this: my @tokens = split
/((?:[\w\-.]+:)?"[^"]+"|\s+)/, $query;)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Nov 5 01:29:43 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 05 Nov 2017 00:29:43 +0000
Subject: [Koha-bugs] [Bug 19481] Elasticsearch - Set default fields for
sorting in mappings.yaml
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19481
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |19482
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19482
[Bug 19482] Elasticsearch - prevent removal / editing of required indexes
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Nov 5 01:29:43 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 05 Nov 2017 00:29:43 +0000
Subject: [Koha-bugs] [Bug 19482] Elasticsearch - prevent removal / editing
of required indexes
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19482
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |19481
Status|Patch doesn't apply |Needs Signoff
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19481
[Bug 19481] Elasticsearch - Set default fields for sorting in mappings.yaml
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Nov 5 02:50:03 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 05 Nov 2017 01:50:03 +0000
Subject: [Koha-bugs] [Bug 19581] New: Elasticsearch - Catmandu split option
adds extra null fields to indexes
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19581
Bug ID: 19581
Summary: Elasticsearch - Catmandu split option adds extra null
fields to indexes
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: Searching - Elasticsearch
Assignee: nick at bywatersolutions.com
Reporter: nick at bywatersolutions.com
This affects sorting, when encountering a null field the title in question will
sort out of place.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Nov 5 03:03:15 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 05 Nov 2017 02:03:15 +0000
Subject: [Koha-bugs] [Bug 19581] Elasticsearch - Catmandu split option adds
extra null fields to indexes
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19581
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |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 Sun Nov 5 03:03:16 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 05 Nov 2017 02:03:16 +0000
Subject: [Koha-bugs] [Bug 19581] Elasticsearch - Catmandu split option adds
extra null fields to indexes
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19581
--- Comment #1 from Nick Clemens ---
Created attachment 68947
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68947&action=edit
Bug 19581: Unit tests
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Nov 5 03:03:19 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 05 Nov 2017 02:03:19 +0000
Subject: [Koha-bugs] [Bug 19581] Elasticsearch - Catmandu split option adds
extra null fields to indexes
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19581
--- Comment #2 from Nick Clemens ---
Created attachment 68948
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68948&action=edit
Bug 19581: Remove 'split' option from Catmandu fixes for marc
This option in combination with append seems to add null entries in the indexes
for fields that don't exist. These fields then affect sorting. Removing it
solves
the issue
To test:
1 - Have an ES setup
2 - Catalog and index some records
3 - View a record in ES:
curl 'localhost:9200/koha_kohadev_biblios/data/4?pretty=yes'
4 - Note there are 'null' entries in the indexes, save this result for
comparison
5 - Apply patch
6 - Reindex, amking sure to delete existing indexes
perl misc/search_tools/rebuild_elastic_search.pl -d
7 - View the record again
8 - Note the null entries are removed
9 - Note indices are otherwise intact
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Nov 5 03:04:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 05 Nov 2017 02:04:36 +0000
Subject: [Koha-bugs] [Bug 19481] Elasticsearch - Set default fields for
sorting in mappings.yaml
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19481
--- Comment #5 from Nick Clemens ---
(In reply to David Bourgault from comment #4)
> Test worked as planned, however sorting by title and author seemed strange.
> I assume the issue wasn't introduced by this bug, but looking at the results
> list, it did not seem alphabetical to me.
See bug 19581 and bug 18827 for ordering issues
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Nov 5 03:14:02 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 05 Nov 2017 02:14:02 +0000
Subject: [Koha-bugs] [Bug 19581] Elasticsearch - Catmandu split option adds
extra null fields to indexes
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19581
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68947|0 |1
is obsolete| |
--- Comment #3 from Nick Clemens ---
Created attachment 68949
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68949&action=edit
Bug 19581: Unit tests
This patch makes a few changes to the tests to take into account the
change to indexing.
To test:
prove t/db_dependent/Koha_Elasticsearch_Indexer.t
Should return green
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Nov 5 06:31:20 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 05 Nov 2017 05:31:20 +0000
Subject: [Koha-bugs] [Bug 19514] No Password restrictions in onboarding tool
patron creation
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19514
Alex Buckley changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |In Discussion
--- Comment #6 from Alex Buckley ---
Hi Jonathan
I have tried implemented the client side validation by doing the following:
1. Adding the following pragma at the start of onboardingstep3.tt:
[% USE Koha %]
2. Adding the following to the bottom of the same file:
[% PROCESS 'password_check.inc' %]
[% PROCESS 'add_password_check' new_password => 'newpassword' %]
3. Added the following to the validation in the ../../../js/onboarding.js
password:
required: true,
password_strong: true
password_no_spaces: true
},?
password2: {
required: true,
password_match: true
}
.
I notice that I get the JS error 'TypeError: _(...).format is not a function.
This error is not thrown by the memberentrygen.tt file which uses the same
client side validation.
Can you give me any suggestions as to how to fix this issue
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Nov 5 08:03:59 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 05 Nov 2017 07:03:59 +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
Alex Buckley changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |alexbuckley at catalyst.net.nz
Status|Needs Signoff |Failed QA
--- Comment #32 from Alex Buckley ---
There's a unnecessary bracket in line 677 of opac-search.pl:
syntax error at /home/vagrant/kohaclone/opac/opac-search.pl line 677, near ")
)"
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun Nov 5 16:41:38 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 05 Nov 2017 15:41:38 +0000
Subject: [Koha-bugs] [Bug 18947] Unexpected Active Directory LDAP
authentication failure mode
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18947
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |nick at bywatersolutions.com
Severity|normal |major
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Sun Nov 5 16:43:00 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 05 Nov 2017 15:43:00 +0000
Subject: [Koha-bugs] [Bug 18992] LDAP fallback behaviour not consistent
In-Reply-To: