---
Created attachment 72336
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72336&action=edit
Bug 19532: Extra fixes from Comment 68
This patch fixes the following:
The db uppdate tries to insert after 'notes' in issuing rules, but this
doesn't exist
When placing recall I always get a warning:
Warning: Your library does not allow recalls for
x item types.
Cancelling recall from 'Recall history tab' doesn't work
If patron has no recalls there is 'Patron has no current recalls'
message at bottom of tabs on checkout and details tabs for patron
When confirming a recall if the item is from another branch I get a
transfer generated as well as confirming hold for recall
Feature now checks if the recall branch is the same as the item
holding branch when checking in item to determine if transfer is
required
On recalls awaiting pickup:
need to update recall.borrower to recall.patron - causing
internal server error
When placing a recall - if I hit 'back' after confirming and
confirm again I get a second recall on the same item
Recalls queue should have waiting date information
Recalls queue should have a way to hide cancelled, I would
expect closed recalls to be hidden by default
Recalls queue now hides old (cancelled/expired/finished) recalls by
default and has a checkbox to toggle this
When viewing a patrons account in the staff side we should see
that a checkout has an active recall
Checking out the recall doesn't seem to close the recall
Added an 'F' status for finished/closed recalls
Signed-off-by: Nick Clemens
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 17:02:08 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 16:02:08 +0000
Subject: [Koha-bugs] [Bug 19532] Recalls for Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19532
--- Comment #122 from Nick Clemens ---
Created attachment 72337
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72337&action=edit
Bug 19532 (follow-up) Update characterset for recalls table
Signed-off-by: Nick Clemens
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 17:02:13 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 16:02:13 +0000
Subject: [Koha-bugs] [Bug 19532] Recalls for Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19532
--- Comment #123 from Nick Clemens ---
Created attachment 72338
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72338&action=edit
Bug 19532 - Improve and refine recalls functionality
Only allow 1 recall per item
When checking in only check for holds if no reserves found
Add a T/transferring status for recalls, only confirm them once arrived
Improve message when checking in recall
Fix clashing ids on recall-history
Check if patron can hold item before allowing recall
Prevent multiple recalls when pressing back button
Select/fill only active recall when issuing
To test:
1 - Repeat major test plan - verify all points
2 - Attempt to place multiple recallsusing back button, should fail
3 - Set all item level holds to not allowed, should not be able to place
recall
4 - Check-in recall at branch other than destination, should be in
transit
5 - Confirm at correct branch, should send message and get expiration
date etc.
6 - Return to another branch, should eb in transit again
7 - Play with circ rules and verify only allowed patrons can place
recall
Signed-off-by: Nick Clemens
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 17:02:14 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 16:02:14 +0000
Subject: [Koha-bugs] [Bug 20268] CSS regression: white gap on the top of the
staff pages
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268
Claire Gravely 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 Thu Mar 1 17:02:17 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 16:02:17 +0000
Subject: [Koha-bugs] [Bug 20268] CSS regression: white gap on the top of the
staff pages
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268
Claire Gravely changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #72310|0 |1
is obsolete| |
--- Comment #4 from Claire Gravely ---
Created attachment 72339
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72339&action=edit
Bug 20268: CSS regression: white gap on the top of the staff pages
This patch makes template and CSS changes so that the staff client main
page doesn't get an unwanted top margin from the Bootstrap-grid
conversion.
- The unused "main" class is removed from the login page
- The "main" class on the staff client home page is changed to
"intranet-main."
- The CSS for the staff client home page has been modified accordingly.
To test, apply the patch and clear your browser cache if necessary.
- Open the staff client login page. It should look as it always does.
- Log in and check the style of the main page. There should be no white
margin at the top of the page.
Signed-off-by: Claire Gravely
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 17:05:13 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 16:05:13 +0000
Subject: [Koha-bugs] [Bug 19532] Recalls for Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19532
--- Comment #124 from Nick Clemens ---
Rebased, tested and added some follow-ups to refine the functionality here -
Aleisha can you sign off on my patch?
This implements basic recall functionality and leaves room for improvements,
this is a good starting point and I think can be QAed with an eye toward
building refinement and extending functionality in future patchsets.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 17:33:21 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 16:33:21 +0000
Subject: [Koha-bugs] [Bug 20321] New: C4:XISBN->get_biblionumber_from_isbn
is not used
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20321
Bug ID: 20321
Summary: C4:XISBN->get_biblionumber_from_isbn is not used
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: minor
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: koha-bugs at lists.koha-community.org
Reporter: nick at bywatersolutions.com
QA Contact: testopia at bugs.koha-community.org
Let remove it
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 17:33:27 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 16:33:27 +0000
Subject: [Koha-bugs] [Bug 20321] C4:XISBN->get_biblionumber_from_isbn is not
used
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20321
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|koha-bugs at lists.koha-commun |nick at bywatersolutions.com
|ity.org |
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 17:35:02 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 16:35:02 +0000
Subject: [Koha-bugs] [Bug 20321] C4:XISBN->get_biblionumber_from_isbn is not
used
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20321
--- Comment #1 from Nick Clemens ---
Created attachment 72340
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72340&action=edit
Bug 20321 - Remove get_biblionumber_from_isbn
To test:
1 - grep get_biblionumber_from_isbn
2 - verify all occurences are not actual calls (except for test)
3 - Apply patch
4 - grep get_biblionumber_from_isbn
5 - Verify it is removed
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 17:35:00 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 16:35:00 +0000
Subject: [Koha-bugs] [Bug 20321] C4:XISBN->get_biblionumber_from_isbn is not
used
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20321
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.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 17:43:44 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 16:43:44 +0000
Subject: [Koha-bugs] [Bug 19919] Writing off a Lost Item Fee marks as "Paid
for by patron"
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19919
Barton Chittenden changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |barton at bywatersolutions.com
Severity|enhancement |major
--- Comment #2 from Barton Chittenden ---
Marking as major, simply because this is going to cause a lot of confusion; I
foresee many support tickets.
--
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 Thu Mar 1 18:15:26 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 17:15:26 +0000
Subject: [Koha-bugs] [Bug 20079] Display stack trace for development
installations
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20079
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #70862|0 |1
is obsolete| |
--- Comment #3 from Jonathan Druart ---
Created attachment 72341
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72341&action=edit
Bug 20079: Display stack trace for development installations
"""The default value is development, which causes plackup to
load the middleware components: AccessLog, StackTrace, and Lint unless
--no-default-middleware is set."""
Test plan:
Confirm that the stack trace is displayed when something is wrong (die
somewhere to test)
for dev installations (inside a devbox)
The -E flag must remain deployment for non-dev installs
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 18:22:44 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 17:22:44 +0000
Subject: [Koha-bugs] [Bug 19908] Password should not be mandatory
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19908
Agnes Rivers-Moore changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |arm at hanover.ca
--- Comment #17 from Agnes Rivers-Moore ---
Our Koha as of 17.11.03 does not generate default username for any patron type.
Josef said 'Other categories (than org) have username generated as
"firstname.surname" if you leave it blank' - but that does not seem to be the
case any more.
Previous behaviour also assigned a default password, and that also does not
happen any more.
Result is staff have to add three fields, making them unique/matching, even if
the patron does not want an online account.
Looking forward to this being non-mandatory! Thanks.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 19:21:51 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 18:21:51 +0000
Subject: [Koha-bugs] [Bug 19866] Move template JavaScript to the footer:
UNIMARC editor plugins, part 1
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19866
--- Comment #2 from Owen Leonard ---
Here's a more thorough description of how this patch can be tested even if you
don't have a UNIMARC system:
- Go to Administration -> MARC frameworks -> Default
framework -> MARC structure
- For any tag (I like to use 000 because it's the first one),
choose Actions -> Subfields.
- Edit any subfield you want to configure (choose one
that isn't hidden).
- Expand the "Other options" panel and select the plugin you
want to test under "Plugin."
- Save changes.
- Open an existing or new MARC record for editing in the
standard MARC editor.
- Find the tag and subfield you just modified, and look for
the small plugin trigger icon next to the form field.
- Confirm that clicking this link triggers the correct
plugin (the pop-up window should have a heading with the
correct plugin name).
- Make some changes to the options in the plugin window
in a way that you will be able to confirm the changes
are transferred to the field in the MARC editor. For
instance, choose all the same codes for multiple fields.
- Submit your changes in the plugin's pop-up window.
The window should close.
- Confirm that the values you submitted from the plugin
window have been transferred to the correct field in the
MARC editor.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 20:06:44 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 19:06:44 +0000
Subject: [Koha-bugs] [Bug 19749] Report who show all the reservations
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19749
Magnus Enger changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |magnus at libriotech.no
--- Comment #3 from Magnus Enger ---
Could this report do the job?
https://wiki.koha-community.org/wiki/SQL_Reports_Library#Unfilled_Holds
--
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 Thu Mar 1 20:27:42 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 19:27:42 +0000
Subject: [Koha-bugs] [Bug 17218] New list add attempt while logged in as
dbadmin on staff side should be prevented pre-emptively
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17218
--- Comment #9 from M. Tompsett ---
Comment on attachment 72298
--> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72298
Bug 17218: Stop DB user from clicking 'New list' button
Review of attachment 72298:
--> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=17218&attachment=72298)
-----------------------------------------------------------------
::: koha-tmpl/intranet-tmpl/prog/en/includes/virtualshelves-toolbar.inc
@@ +10,5 @@
> + });
> + [% END %]
> +});
> +
> +[% IF op == 'view' %]
I don't understand this whole IF block's purpose. It seems like new unrelated
code.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 20:31:40 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 19:31:40 +0000
Subject: [Koha-bugs] [Bug 11677] Limit to Only items currently available for
loan or reference not working
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11677
Marjorie Barry-Vila changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |patrick.robitaille at collecto
| |.ca
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 20:33:32 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 19:33:32 +0000
Subject: [Koha-bugs] [Bug 5463] limit to available shows on hold items
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5463
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 Thu Mar 1 20:49:45 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 19:49:45 +0000
Subject: [Koha-bugs] [Bug 17218] New list add attempt while logged in as
dbadmin on staff side should be prevented pre-emptively
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17218
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Failed QA
CC| |mtompset at hotmail.com
--- Comment #10 from M. Tompsett ---
http://localhost:8081/cgi-bin/koha/virtualshelves/addbybiblionumber.pl?newshelf=1&biblionumber=1
When you view the contents of an existing public list, there is a drop-down to
add to cart or a new list. That creates a bunch of possible explosion points,
like the kohadevbox link above.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Thu Mar 1 21:39:11 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Thu, 01 Mar 2018 20:39:11 +0000
Subject: [Koha-bugs] [Bug 14697] Extend and enhance "Claims returned" lost
status
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14697
Jessie Zairo changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |kelly at bywatersolutions.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 03:37:56 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 02:37:56 +0000
Subject: [Koha-bugs] [Bug 20322] New: Circulation page layout and design
update
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20322
Bug ID: 20322
Summary: Circulation page layout and design update
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Circulation
Assignee: koha-bugs at lists.koha-community.org
Reporter: liz at catalyst.net.nz
QA Contact: testopia at bugs.koha-community.org
CC: gmcharlt at gmail.com, kyle.m.hall at gmail.com
I think maybe it's time we refreshed and improved the circulation home page,
don't you?
:)
--
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 Mar 2 03:48:22 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 02:48:22 +0000
Subject: [Koha-bugs] [Bug 20322] Circulation page layout and design update
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20322
--- Comment #1 from Liz Rea ---
Created attachment 72342
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72342&action=edit
Bug 20322: design refresh for circulation page
do note, I don't actually mind if these aren't the changes eventually accepted.
I just know it needs doing and starting it will hopefully get some gears
turning.
To test:
Click all the links and check they work.
Do you like the styling? it's different. :)
--
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 Mar 2 07:38:02 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 06:38:02 +0000
Subject: [Koha-bugs] [Bug 20251] SIP checkout broken in master
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20251
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |m.de.rooy at rijksmuseum.nl
|y.org |
--- Comment #6 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 Mar 2 08:09:29 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 07:09:29 +0000
Subject: [Koha-bugs] [Bug 20271] Merge deleted/old tables with their "alive"
cousins
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20271
--- Comment #5 from Josef Moravec ---
Comment on attachment 72300
--> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72300
Bug 20271: merging delete biblio/items
Review of attachment 72300:
--> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=20271&attachment=72300)
-----------------------------------------------------------------
Just a quick look:
there is missing atomic database update
also the test suite should be adjusted
I am not sure about the changes in OAI, so need to investigate more later
::: C4/Biblio.pm
@@ -3182,5 @@
> - my $bkup_sth = $dbh->prepare($query);
> - $bkup_sth->execute(@bind);
> - $bkup_sth->finish;
> -
> - _koha_delete_biblio_metadata( $biblionumber );
this is missing in your patch
::: C4/Items.pm
@@ +1061,4 @@
> FROM items
> LEFT JOIN branches AS holding ON items.holdingbranch = holding.branchcode
> LEFT JOIN branches AS home ON items.homebranch=home.branchcode
> + LEFT JOIN biblio ON biblio.biblionumber = items.biblionumber AND biblio.deleted_at IS NULL
I don't think this will work, that's condition should be in where clause
::: misc/export_records.pl
@@ +105,5 @@
> push @record_ids, $_->{biblionumber} for @{
> + $dbh->selectall_arrayref(q|
> + SELECT biblionumber
> + FROM biblioitems
> + |, { Slice => {} }, ( $timestamp ) x 4 );
Please include the changes made by Bug 19730
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 08:19:56 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 07:19:56 +0000
Subject: [Koha-bugs] [Bug 20251] SIP checkout broken in master
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20251
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #72291|0 |1
is obsolete| |
Attachment #72308|0 |1
is obsolete| |
--- Comment #7 from Marcel de Rooy ---
Created attachment 72343
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72343&action=edit
Bug 20251: (bug 19280 follow-up) FIX SIP checkout
Signed-off-by: Colin Campbell
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 Mar 2 08:19:53 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 07:19:53 +0000
Subject: [Koha-bugs] [Bug 20251] SIP checkout broken in master
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20251
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 Mar 2 08:38:24 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 07:38:24 +0000
Subject: [Koha-bugs] [Bug 20268] CSS regression: white gap on the top of the
staff pages
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Passed QA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 08:38:27 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 07:38:27 +0000
Subject: [Koha-bugs] [Bug 20268] CSS regression: white gap on the top of the
staff pages
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #72339|0 |1
is obsolete| |
--- Comment #5 from Marcel de Rooy ---
Created attachment 72344
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72344&action=edit
Bug 20268: CSS regression: white gap on the top of the staff pages
This patch makes template and CSS changes so that the staff client main
page doesn't get an unwanted top margin from the Bootstrap-grid
conversion.
- The unused "main" class is removed from the login page
- The "main" class on the staff client home page is changed to
"intranet-main."
- The CSS for the staff client home page has been modified accordingly.
To test, apply the patch and clear your browser cache if necessary.
- Open the staff client login page. It should look as it always does.
- Log in and check the style of the main page. There should be no white
margin at the top of the page.
Signed-off-by: Claire Gravely
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 Mar 2 08:38:52 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 07:38:52 +0000
Subject: [Koha-bugs] [Bug 20268] CSS regression: white gap on the top of the
staff pages
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |m.de.rooy at rijksmuseum.nl
|y.org |
CC| |m.de.rooy at rijksmuseum.nl
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 09:03:11 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 08:03:11 +0000
Subject: [Koha-bugs] [Bug 19547] Maria DB doesn't have a debian.cnf
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19547
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 Mar 2 09:03:20 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 08:03:20 +0000
Subject: [Koha-bugs] [Bug 19547] Maria DB doesn't have a debian.cnf
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19547
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #72283|0 |1
is obsolete| |
--- Comment #9 from Marcel de Rooy ---
Created attachment 72346
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72346&action=edit
Bug 19547: Deal with --create-db missing debian.cnf issue
Followed test plan. Patch functions as described and both commands execute.
Signed-off-by: Dilan Johnpull?
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 Mar 2 09:03:31 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 08:03:31 +0000
Subject: [Koha-bugs] [Bug 19547] Maria DB doesn't have a debian.cnf
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19547
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|testopia at bugs.koha-communit |m.de.rooy at rijksmuseum.nl
|y.org |
CC| |m.de.rooy at rijksmuseum.nl
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 09:03:15 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 08:03:15 +0000
Subject: [Koha-bugs] [Bug 19547] Maria DB doesn't have a debian.cnf
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19547
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #68850|0 |1
is obsolete| |
--- Comment #8 from Marcel de Rooy ---
Created attachment 72345
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72345&action=edit
Bug 19547: MariaDB install has no debian.cnf
This attempts to handle issues arising when running
koha-create on a system that never had MySQL installed.
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 Mar 2 09:05:48 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 08:05:48 +0000
Subject: [Koha-bugs] [Bug 18958] If patron has multiple record level holds
on one record transferring first hold causes next hold to become item level
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18958
--- Comment #11 from Marcel de Rooy ---
QA: Looking here
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 10:00:05 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 09:00:05 +0000
Subject: [Koha-bugs] [Bug 18958] If patron has multiple record level holds
on one record transferring first hold causes next hold to become item level
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18958
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |BLOCKED
--- Comment #12 from Marcel de Rooy ---
Still working on it. Suspect that we are not ready yet.. Moving to BLOCKED
temporarily
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 11:06:17 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 10:06:17 +0000
Subject: [Koha-bugs] [Bug 20287] Move AddMember and ModMember to Koha::Patron
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20287
--- Comment #2 from Josef Moravec ---
Hi Jonathan,
I've read the code and added few comment to commits in gitlab, hope you've
received a notice about that ;)
Also, there are some issues reported by QA tools:
FAIL C4/Members.pm
FAIL pod
*** ERROR:
Spurious =cut command
in file C4/Members.pm
FAIL opac/opac-messaging.pl
FAIL valid
Global symbol "$borrower" requires explicit package name
opac/opac-messaging.pl had compilation errors.
FAIL opac/opac-registration-verify.pl
FAIL valid
"generate_password" is not exported by the Koha::AuthUtils
module
Can't continue after import errors
FAIL tools/modborrowers.pl
FAIL valid
Global symbol "$success" requires explicit package name
tools/modborrowers.pl had compilation errors.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 11:12:25 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 10:12:25 +0000
Subject: [Koha-bugs] [Bug 20323] New: Batch patron modification tool broken
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20323
Bug ID: 20323
Summary: Batch patron modification tool broken
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: major
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: koha-bugs at lists.koha-community.org
Reporter: josef.moravec at gmail.com
QA Contact: testopia at bugs.koha-community.org
>From tools/modborrowers.pl
341 if ( $patron ) {
342 $patron = $patron->unblessed;
343 $patron->{category_description} = $patron->category->description;
This was added by bug 18403 and leads (obviosly) to error:
Can't call method "category" on unblessed reference at
/home/vagrant/kohaclone/tools/modborrowers.pl
--
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 Mar 2 11:18:03 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 10:18:03 +0000
Subject: [Koha-bugs] [Bug 20323] Batch patron modification tool broken
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20323
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|koha-bugs at lists.koha-commun |josef.moravec at gmail.com
|ity.org |
Depends on| |18403
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18403
[Bug 18403] Hide patron information if not part of the logged in user library
group
--
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 Mar 2 11:18:03 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 10:18:03 +0000
Subject: [Koha-bugs] [Bug 18403] Hide patron information if not part of the
logged in user library group
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18403
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |20323
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20323
[Bug 20323] Batch patron modification tool broken
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 11:18:27 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 10:18:27 +0000
Subject: [Koha-bugs] [Bug 20323] Batch patron modification tool broken
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20323
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 11:18:29 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 10:18:29 +0000
Subject: [Koha-bugs] [Bug 20323] Batch patron modification tool broken
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20323
--- Comment #1 from Josef Moravec ---
Created attachment 72347
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72347&action=edit
Bug 20323: Fix batch patron modification tool
Test plan:
1) Go to Tools -> Batch patrons modification
2) Select some patrons and try to change any field
--> without patch - it exploads with log:
Can't call method "category" on unblessed reference at
/home/vagrant/kohaclone/tools/modborrowers.pl
--> with patch - the modification is properly made
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 12:13:19 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:13:19 +0000
Subject: [Koha-bugs] [Bug 18958] If patron has multiple record level holds
on one record transferring first hold causes next hold to become item level
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18958
--- Comment #14 from Marcel de Rooy ---
Created attachment 72349
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72349&action=edit
Bug 18958: (Follow-up) Pass patron to template for confirm hold
Removing the FIXME. Trivial fix.
Not sure where to put it; can be moved on its own.
Test plan:
Confirm a hold and verify that patron details are shown in popup.
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 Mar 2 12:13:14 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:13:14 +0000
Subject: [Koha-bugs] [Bug 18958] If patron has multiple record level holds
on one record transferring first hold causes next hold to become item level
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18958
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #72210|0 |1
is obsolete| |
Attachment #72211|0 |1
is obsolete| |
--- Comment #13 from Marcel de Rooy ---
Created attachment 72348
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72348&action=edit
Bug 18958 - If patron has multiple record level holds on one record
transferring first hold causes next hold to become item level
Imagine you have a record with 3 items, each item is held by LibraryA, LibraryB
and LibraryC respectively. Now, a patron places to record level holds for
pickup at LibraryD. When any of the items are checked in, that item is trapped
to fill the first hold, but a side affect is that the next open hold because
item level for the item that was just checked in!
This is clearly incorrect and prevents the patron from placing more record
level holds on the record.
Test Plan:
1. Create and use a patron that can place multiple record level holds per
record
2. Create a record with X items, each at a different library
3. Place X 'Next available' holds on the record for the patron using the 'Holds
to place' box
4. Rebuild the holds queue
5. Check in LibraryA's copy as LibraryA and confirm the hold
6. Revisit request.pl for the record, notice the next hold in line is now
item-specific
7. Attempt to place another hold for your patron, notice that it requires an
item-specific hold
8. Apply this patch
9. Repeat steps 1-4
10. Revisit request.pl for the record, notice the next hold in line has *not*
become item-specific
Signed-off-by: Kyle M Hall
Signed-off-by: Jason Robb
Signed-off-by: Josef Moravec
Bug 18958 [QA Followup] - Bring back data needed for the diffbranch block of
the template
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 12:18:13 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:18:13 +0000
Subject: [Koha-bugs] [Bug 19933] Move C4::Members::patronflags to the Koha
namespace - part 1
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19933
Kyle M Hall 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 Mar 2 12:18:15 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:18:15 +0000
Subject: [Koha-bugs] [Bug 19933] Move C4::Members::patronflags to the Koha
namespace - part 1
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19933
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #72004|0 |1
is obsolete| |
Attachment #72005|0 |1
is obsolete| |
Attachment #72006|0 |1
is obsolete| |
Attachment #72007|0 |1
is obsolete| |
Attachment #72095|0 |1
is obsolete| |
--- Comment #16 from Kyle M Hall ---
Created attachment 72350
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72350&action=edit
Bug 19933: Remove patronflags - unused vars oldamount/amountold in
circulation.pl
There is oldamound/amountold variable that is sent to the template but
never used later. Let remove it now to avoid useless replacement.
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 12:18:23 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:18:23 +0000
Subject: [Koha-bugs] [Bug 19933] Move C4::Members::patronflags to the Koha
namespace - part 1
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19933
--- Comment #17 from Kyle M Hall ---
Created attachment 72351
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72351&action=edit
Bug 19933: Remove patronflags - In CanBookBeIssued
The change is trivial here, we only want to pass gonenoaddress, lost and
is_debared to the template.
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 12:18:26 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:18:26 +0000
Subject: [Koha-bugs] [Bug 19933] Move C4::Members::patronflags to the Koha
namespace - part 1
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19933
--- Comment #18 from Kyle M Hall ---
Created attachment 72352
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72352&action=edit
Bug 19933: Remove patronflags - easy ones
This patch replaces the easy occurrences of patronflags.
These calls only need the CHARGES->amount value, that is the non issues
charges. Luckily we now have a Koha::Account->non_issues_charges that
deal with that.
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 12:18:29 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:18:29 +0000
Subject: [Koha-bugs] [Bug 19933] Move C4::Members::patronflags to the Koha
namespace - part 1
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19933
--- Comment #19 from Kyle M Hall ---
Created attachment 72353
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72353&action=edit
Bug 19933: Remove patronflags - tricky ones
Here we are, patronflags is used in a couple of places where (almost) all flags
were really useful: C4::SIP::ILS::Patron->new and circulation.pl
This patch only deal with the circulation code as I am not convident
enough with SIP code.
The change does not seems trivial because of the complexity of the
existing code, but the logic is the same. We send a variable to the
template depending on the situation of the patron.
I guess only code eyes ball could catch issue in this patch quickly.
Maybe we need to find a good place in a Koha module to move this code
and provide code coverage (especially when C4::SIP::ILS::Patron will
reuse it).
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 12:18:32 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:18:32 +0000
Subject: [Koha-bugs] [Bug 19933] Move C4::Members::patronflags to the Koha
namespace - part 1
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19933
--- Comment #20 from Kyle M Hall ---
Created attachment 72354
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72354&action=edit
Bug 19933: (follow-up) Remove patronflags - easy ones
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 12:18:35 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:18:35 +0000
Subject: [Koha-bugs] [Bug 19933] Move C4::Members::patronflags to the Koha
namespace - part 1
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19933
--- Comment #21 from Kyle M Hall ---
Created attachment 72355
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72355&action=edit
Bug 19933: Move instantiation of in deletemem.pl
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 12:18:38 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:18:38 +0000
Subject: [Koha-bugs] [Bug 19933] Move C4::Members::patronflags to the Koha
namespace - part 1
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19933
--- Comment #22 from Kyle M Hall ---
Created attachment 72356
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72356&action=edit
Bug 19933: Fix typo in template plugin name
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 12:18:59 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:18:59 +0000
Subject: [Koha-bugs] [Bug 18958] If patron has multiple record level holds
on one record transferring first hold causes next hold to become item level
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18958
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|BLOCKED |Failed QA
--- Comment #15 from Marcel de Rooy ---
Squashed both patches. Follow-up may need to be moved.
Tested this quite a bit. Hard to reproduce at first. (Alternative test plan in
next comment.)
Failing QA because this patch has side-effects when you cancel a hold from the
confirm hold popup. Before we called GetOtherReserves and now we dont. This
changes behavior.
Also I am wondering if the passed patron etc. at lines 160/170 makes any sense.
If you confirm the transfer or cancel, you come back to the default returns.pl
form.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 12:19:20 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:19:20 +0000
Subject: [Koha-bugs] [Bug 18958] If patron has multiple record level holds
on one record transferring first hold causes next hold to become item level
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18958
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
QA Contact|josef.moravec at gmail.com |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 Mar 2 12:21:00 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:21:00 +0000
Subject: [Koha-bugs] [Bug 18958] If patron has multiple record level holds
on one record transferring first hold causes next hold to become item level
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18958
--- Comment #16 from Marcel de Rooy ---
[Without patch]
Three items (CPL MPL RMA)
2 record level patron A and item level from patron B (CPL)
Checkin CPL item at RMA
Confirm
Waiting OK
Checkin this item again and Cancel
Finally reproduced the error: Second patron A hold became Item level (Bug)
Checkin again, confirm
Checkin and cancel
What happened? Priority last hold = 1. Transfer created. No message to staff.
(Bug: transit should be a T hold?)
[With patch]
Three items
2 record level for patron A
1 item level for patron B on CPL item
Goto RMA Checkin CPL item, confirm for patron A Waiting
So far so good. Checkin again and cancel.
Second record level hold at 1. This cancel created no transfer. No message.
(Behavior change?)
Checkin again, confirm. Waiting OK
Checkin again, cancel.
Last item level hold at 1. No transfer. No message.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 12:44:09 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 11:44:09 +0000
Subject: [Koha-bugs] [Bug 20167] Item hold is set to bibliographic hold when
changing pickup location
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20167
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |nick at bywatersolutions.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 13:48:29 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 12:48:29 +0000
Subject: [Koha-bugs] [Bug 18345] Add a more ES based query builder
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18345
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |julian.maurice at biblibre.com
--- Comment #2 from Julian Maurice ---
What is the status of this bug ? Waiting for sign-off ?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:04:26 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:04:26 +0000
Subject: [Koha-bugs] [Bug 9634] Allow for parameters re-use on SQL reports
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9634
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |julian.maurice at biblibre.com
--- Comment #7 from Julian Maurice ---
Is the new option really necessary ?
I can't imagine a use case where someone would like to not use this feature.
And if users really want to repeat themselves they can just rename one of the
parameters.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:05:16 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:05:16 +0000
Subject: [Koha-bugs] [Bug 19546] Can't locate
Mojolicious/Plugin/Koha/REST/Plugin/Pagination.pm breaks opac and staff
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19546
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|REOPENED |ASSIGNED
--- Comment #14 from Tom?s Cohen Arazi ---
I managed to reproduce. This is the same issue introduced by Perl 5.18 we face
with cronjobs. I'll provide a patch.
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:08:05 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:08:05 +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 #66 from Victor Grousset/tuxayo ---
Created attachment 72357
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72357&action=edit
Bug 19522: Label creator: Batches: make "Export Labels" translatable
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:15:18 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:15: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 #67 from Victor Grousset/tuxayo ---
Thanks paxed, I noticed this string but though that I wasn't translatable due
to the Grey Box lib.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:37:06 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:37:06 +0000
Subject: [Koha-bugs] [Bug 9634] Allow for parameters re-use on SQL reports
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9634
--- Comment #8 from Nick Clemens ---
I added it only for old reports, if a library has been using a report with
duplicated inputs this will be a change in behavior and may be
unexpected/unwanted.
Am happy to reconsider or explore alternate options, this seemed the simplest
at the time
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:44:13 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:44:13 +0000
Subject: [Koha-bugs] [Bug 19160] CAS Single Log Out
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19160
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Passed QA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:44:16 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:44:16 +0000
Subject: [Koha-bugs] [Bug 19160] CAS Single Log Out
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19160
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #71831|0 |1
is obsolete| |
Attachment #71832|0 |1
is obsolete| |
Attachment #71833|0 |1
is obsolete| |
Attachment #71834|0 |1
is obsolete| |
Attachment #71835|0 |1
is obsolete| |
--- Comment #24 from Kyle M Hall ---
Created attachment 72358
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72358&action=edit
Bug 19160: CAS Single logout
CAS supports single logout, where if you logout of one application it
logs you out of all of them.
This bug implements this
You will need a CAS server (with single logout configure),
and at least 2 applications (one being Koha)
1/ In Koha login via CAS
2/ Login to the other application via CAS
3/ Logout of the other application
4/ Notice you are still logged into Koha
5/ Log out of Koha
6/ Apply patch
7/ Login to Koha via CAS, login to other app via CAS
8/ Log out of other app
9/ Notice you are logged out of Koha
If you dont have CAS, this patch should be a no op, you could test that
1/ Login and logout normally
2/ Apply patch
3/ Login and logout still work fine
Signed-off-by: Katrin Fischer
Patch works as described, local login still works correctly.
Signed-off-by: Kyle M Hall
Signed-off-by: Katrin Fischer
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:44:24 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:44:24 +0000
Subject: [Koha-bugs] [Bug 19160] CAS Single Log Out
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19160
--- Comment #25 from Kyle M Hall ---
Created attachment 72359
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72359&action=edit
Bug 19160: Move duplicated code to its own private subroutine
Signed-off-by: Katrin Fischer
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:44:27 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:44:27 +0000
Subject: [Koha-bugs] [Bug 19160] CAS Single Log Out
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19160
--- Comment #26 from Kyle M Hall ---
Created attachment 72360
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72360&action=edit
Bug 19160: Move the code from controller to C4::Auth_with_cas
Signed-off-by: Katrin Fischer
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:44:31 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:44:31 +0000
Subject: [Koha-bugs] [Bug 19160] CAS Single Log Out
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19160
--- Comment #27 from Kyle M Hall ---
Created attachment 72361
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72361&action=edit
Bug 19160: Isolate CAS code into its own module
Signed-off-by: Katrin Fischer
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:44:34 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:44:34 +0000
Subject: [Koha-bugs] [Bug 19160] CAS Single Log Out
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19160
--- Comment #28 from Kyle M Hall ---
Created attachment 72362
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72362&action=edit
Bug 19160: (follow-up) Fix problems introduced by renaming logout_required to
logout_if_required
Signed-off-by: Katrin Fischer
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:44:38 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:44:38 +0000
Subject: [Koha-bugs] [Bug 19160] CAS Single Log Out
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19160
--- Comment #29 from Kyle M Hall ---
Created attachment 72363
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72363&action=edit
Bug 19160: Remove tab characters causing qa script to fail
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:44:41 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:44:41 +0000
Subject: [Koha-bugs] [Bug 19160] CAS Single Log Out
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19160
--- Comment #30 from Kyle M Hall ---
Created attachment 72364
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72364&action=edit
Bug 19160: Add POD
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:47:40 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:47:40 +0000
Subject: [Koha-bugs] [Bug 18345] Add a more ES based query builder
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18345
--- Comment #3 from Nick Clemens ---
Currently this is just a placeholder for some code that was lost at some point,
we should pursue in the future, but i think other issues are ahead of this.
Currently we use the built in ES query parser for most things, but in the
future it would be nice to leverage some of the specific ES search endpoints
directly for more refined searches.
Some of that is done here:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19604
Where we need to use more specific searching to mimic what we do in zebra
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:49:21 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:49:21 +0000
Subject: [Koha-bugs] [Bug 19546] Can't locate
Mojolicious/Plugin/Koha/REST/Plugin/Pagination.pm breaks opac and staff
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19546
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |Needs Signoff
Patch complexity|--- |Trivial patch
--
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 Mar 2 14:49:27 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:49:27 +0000
Subject: [Koha-bugs] [Bug 19546] Can't locate
Mojolicious/Plugin/Koha/REST/Plugin/Pagination.pm breaks opac and staff
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19546
--- Comment #15 from Tom?s Cohen Arazi ---
Created attachment 72365
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72365&action=edit
Bug 19546: Run starman from the instance's home dir
This patch makes koha-plack jump into the instance's home directory to
run.
It is required because Perl 5.18 introduced a breaking change that
makes perl die if @INC includes directories for which the user doesn't
have read permission, and need to be traversed when querying for a lib.
This is the case of '.', which is introduced automatically into @INC
until Perl 5.26 (which removes the 'feature').
The Mojolicious::Plugins lib prefixes the plugin names with
'Mojolicious::Plugin' so it first looks (for example) for
Mojolicious::Plugin::Koha::REST::Plugin::Pagination (Pagination is just
the first one on the list). When it looks for it at '.' it dies (because
of Perl's behaviour) so it doesn't query for the right namespace (the
following to try).
I only reproduced it in koha-testing-docker. To test, you just need to
try this patched koha-plack and make sure it doesn't break due to this
permissions issue.
To test:
- In your koha-testing-docker clone, run:
$ docker-compose up -p test
- Open a shell inside the container (on a separate terminal):
$ docker exec -it test_koha_1 bash
- From within the /root directory, restart plack:
$ cd /root
$ koha-plack --restart kohadev
=> FAIL: Logs show plack is broken due to permissions problems trying to
find Mojolicious::Plugin::Koha::REST::Plugin::Pagination
- Use the patched script from this patch:
$ /kohadevbox/koha/debian/scripts/koha-plack --restart kohadev
=> SUCCESS: Plack runs fine, no error in the logs
- Sign off :-D
Note: people who has environments in which the problems is reproducible,
please test this version of koha-plack and stamp your sign-off,
PLEASE.
--
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 Mar 2 14:50:45 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:50:45 +0000
Subject: [Koha-bugs] [Bug 17717] process_message_queue.pl: Can't locate
Authen/CAS/Client/Response/Failure.pm
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17717
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|chris at bigballofwax.co.nz |tomascohen at gmail.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Fri Mar 2 14:55:10 2018
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Fri, 02 Mar 2018 13:55:10 +0000
Subject: [Koha-bugs] [Bug 18345] Add a more ES based query builder
In-Reply-To: