From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 00:35:23 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Tue, 31 Jan 2017 23:35:23 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Failed QA
--- Comment #122 from M. Tompsett ---
If you want to have this signed off, I don't want to see:
Software error:
DBIx::Class::Storage::DBI::_dbh_execute(): Incorrect integer value:
'anyunavailable' for column 'onshelfholds' at row 1 at
/home/mtompset/kohaclone/Koha/Object.pm line 120
For help, please send mail to the webmaster (webmaster at ubuntu), giving this
error message and the time and date of the error.
When I attempt a Xenial install.
The other possibility is completely remove onshelfholds and just default it to
something.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 01:16:34 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 00:16:34 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
--- Comment #123 from M. Tompsett ---
Created attachment 59703
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59703&action=edit
Bug 17855: Follow up - Fix onshelfholds
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 01:16:24 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 00:16:24 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 02:00:38 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 01:00:38 +0000
Subject: [Koha-bugs] [Bug 18026] New: URL to database columns link in system
preferences is incorrect
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18026
Bug ID: 18026
Summary: URL to database columns link in system preferences is
incorrect
Change sponsored?: ---
Product: Koha
Version: 16.05
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Staff Client
Assignee: koha-bugs at lists.koha-community.org
Reporter: dilan at calyx.net.au
QA Contact: testopia at bugs.koha-community.org
CC: gmcharlt at gmail.com
Created attachment 59704
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59704&action=edit
Picture of the URL link
Under the Acquisitions tab in Global system preferences, the link to the Koha
items table URL is broken.
The link should be: http://schema.koha-community.org/16_05/tables/items.html
but the link instead is: http://schema.koha-community.org/tables/items.html
Note the "16_05" missing in the latter. This causes a 404 "not found" error to
result.
--
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 Feb 1 02:02:47 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 01:02:47 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
--- Comment #124 from Alex Buckley ---
Created attachment 59705
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59705&action=edit
Bug 17955 - Removed unnecessary use pragmas in the onboarding.pl script
Replaced all sql code in onboarding.pl script with DBIx
Changed the logic of the onshelfholds if statement in onboarding.pl
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 02:04:04 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 01:04:04 +0000
Subject: [Koha-bugs] [Bug 18026] URL to database columns link in system
preferences is incorrect
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18026
dilan at calyx.net.au changed:
What |Removed |Added
----------------------------------------------------------------------------
Priority|P5 - low |P4
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 02:12:45 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 01:12:45 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
Alex Buckley 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 Feb 1 02:12:48 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 01:12:48 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
Alex Buckley changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59705|0 |1
is obsolete| |
--- Comment #125 from Alex Buckley ---
Created attachment 59706
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59706&action=edit
Bug 17955 - Removed unnecessary use pragmas in the onboarding.pl script
Replaced all sql code in onboarding.pl script with DBIx
Changed the logic of the onshelfholds if statement in onboarding.pl
https://bugs.koha-community.org/show_bug.cgi?id=17855
Signed-off-by: Alex Buckley
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 02:29:06 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 01:29:06 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
--- Comment #126 from Alex Buckley ---
Created attachment 59707
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59707&action=edit
bug 17855 - Updated web installer finished
The onboarding tool has been implemented and integrated with the updated
web installer. The onboarding tool has the following functionality:
* After the user has finished the web installer they are automatically
redirected to the onboarding tool with no authentication in between
required
* Ability to create library, patron category, patron (set with
superlibrarian priviliages), item type and circulation rule.
* If the uaser installed sample libraries, patron categories and/or item
types then the step to create one in the onboarding tool will be skipped
and a skipping message will be displayed.
*HTML regxes on all text input fields in all forms in the onboarding
tool this prevents the user from writing in special characters
*Working date picker on the patron category form
Have moved all data values from mandatory to default.
Now all default data values are displayed with a bulletpoint rather than
a selectable checkbox stopping users from unticking neccisary data
values
Test plan:
1. Go through the web installer and onboarding tool with the same steps
as outlined in this below guide and check that the functionality
outlined in the guide exists in the patch:
https://koha-community.org/manual/installation/html/
This is a combination of 67 commits.
Initial 3 HTML screens of the onboarding tool and
the perl onboarding.pl script
Note: The advanced setup option has not yet been
implemented, so what you will see is just the basic setup.
Altered web installer
Moved files from optional to mandatory. Edited
MARC screen so that MARC21 is selected by default.
Changed mandatory to default for MARC values
Changed a few file name descriptions to reduce
spacing in installer step 3 screen.
Changed the radiobuttons to bulletpoints for default settings
removed = sign
Fixed bug in installer where a loop was created if
you're database structure needed to be updated.
Redirected this to the marcflavour screen instead
of logging straight into Koha and skipping over
the basic configuration steps.
Changed the bulletpoints to disabled checkboxes
which are checked by default
Backup of step3.tt
Successfully working insertion of the default
options which are shown as bulletpoints
Removed test text
Working insertion of default and optional options
Correctly selected default marc flavour of marc21
NOTE: Mark Tompsett merged several commits
- Removed html regex on patron password fields. Ran perltidy to
indent and space code appropriately. Ran git bz configuration
commands to remove whitespace. Removed unnecissary uses in
the summary.pl script
- Made arrows in onboarding tool pages
-> for consistency. Additionally removed the 'OR'
- Added the use CGI qw ( -utf8 );
- Removed the second path to creating a library, patron
category, patron, item type, and circulation rule. In onboarding
.tt files and summary.tt
- Fixed typo in summary.tt
- Made consistent single space before and after -> in all
onboarding .tt and summary.tt files.
- Made single
tag before path to create library, patron
category, item type, and circulation rule in the onboarding
.tt files (patron does not need this as its layout is slightly
different)
- Changed button and link names in web installer and onboarding
tool
- Implemented Javascript password check to determine if the values
in the 'password' and 'confirm password' fields are consistent.
If passwords are not consistent a message informing the user
is displayed and they only have to change the password values
rather than re-fill out the whole form.
- Fixed the errors from the koha qa test
- Removed use:C4/Auth from the onboarding.pl
- Moved the "select all options" button down to just above
the selectable options in the basic setup, and positioned at
the top of the page for the advanced setup.
- Removed the "there is nothing to do because the sample data
is already installed" page so if the user has install sample
data in the web installer then the step to create that object
in the onboarding tool is skipped altogether.
- Removed the "We're just going to setup a few more things"
page
- Changed the title of the 'Create patron page'
- Added clarifying information to the "Add a patron category"
page
- Added clarifying information to the "Create item type" page
- Fixed the capitalization in the onboarding tool
- Removed the BackupStep3.tt file which was un-used
- Changed the redirect time displayed to user in step3.tt to 10
- Changed the cardnumber on the create patron administrator
page so that it is only populated when the user installed
sample patrons. Also changed the regex so that it allows
letters and number.
- Used the HTMl validator to identify errors in the template
and have corrected all possible in the onboarding templates.
Signed-off-by: Mark Tompsett
Signed-off-by: Alex Buckley
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 02:47:04 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 01:47:04 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
Alex Buckley changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59707|0 |1
is obsolete| |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 02:47:01 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 01:47:01 +0000
Subject: [Koha-bugs] [Bug 16034] Integration with OverDrive Patron API
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16034
--- Comment #52 from Nick Clemens ---
This is looking very good - the one issue I see remaining is the content in the
OverDrive tab has a lot of html tags showing etc - can that be
processed correctly?
-Nick
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 02:56:18 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 01:56:18 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
Alex Buckley changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59703|0 |1
is obsolete| |
--- Comment #127 from Alex Buckley ---
Created attachment 59708
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59708&action=edit
Bug 17855: Follow up - Fix onshelfholds
Signed-off-by: Alex Buckley
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 02:58:04 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 01:58:04 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
Alex Buckley changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59706|0 |1
is obsolete| |
--- Comment #128 from Alex Buckley ---
Created attachment 59709
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59709&action=edit
Bug 17955 - Removed unnecessary use pragmas in the onboarding.pl script
Replaced all sql code in onboarding.pl script with DBIx
Changed the logic of the onshelfholds if statement in onboarding.pl
https://bugs.koha-community.org/show_bug.cgi?id=17855
Signed-off-by: Alex Buckley
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 03:02:51 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 02:02:51 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59628|0 |1
is obsolete| |
--- Comment #129 from M. Tompsett ---
Created attachment 59710
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59710&action=edit
Bug 17855 - Implemented a retrieval of patron category
enrolmentdate/enrolementdateperiod to set an appropriate dateexpiry for patron
accounts created with the onboarding tool.
Set address and city variables to "" in onboarding.pl
Fixed the creation/filling of the $firstpassword, $secondpassword and
$op variables in the onboarding.pl so they don't cause uninitialised
variable errors.
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 Feb 1 03:03:18 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 02:03:18 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59709|0 |1
is obsolete| |
--- Comment #130 from M. Tompsett ---
Created attachment 59711
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59711&action=edit
Bug 17855 - Follow up: code clean up
Removed unnecessary use pragmas in the onboarding.pl script
Replaced all sql code in onboarding.pl script with DBIx
Changed the logic of the onshelfholds if statement in onboarding.pl
Signed-off-by: Alex Buckley
NOTE: Initially came from 17955
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 Feb 1 03:03:27 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 02:03:27 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59708|0 |1
is obsolete| |
--- Comment #131 from M. Tompsett ---
Created attachment 59712
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59712&action=edit
Bug 17855: Follow up - Fix onshelfholds
Signed-off-by: Alex Buckley
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 Feb 1 03:06:03 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 02:06:03 +0000
Subject: [Koha-bugs] [Bug 17955] Remove the sql code and unnecessary use
pragmas from the onboarding.pl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17955
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mtompset at hotmail.com
Attachment #59639|0 |1
is obsolete| |
--- Comment #6 from M. Tompsett ---
Comment on attachment 59639
--> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59639
Bug 17955 - Removed unnecessary use pragmas in the onboarding.pl script
Patch was moved to bug 17855.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 03:06:57 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 02:06:57 +0000
Subject: [Koha-bugs] [Bug 17955] Remove the sql code and unnecessary use
pragmas from the onboarding.pl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17955
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |ASSIGNED
--- Comment #7 from M. Tompsett ---
Moved back to assigned, so Alex can reuse this or close it as deemed fit.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 03:17:14 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 02:17:14 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
--- Comment #4 from M. Tompsett ---
Comment on attachment 59630
--> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59630
Bug 17982: Fix the use of uniq in sub themelanguage
Review of attachment 59630:
--> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=17982&attachment=59630)
-----------------------------------------------------------------
::: C4/Templates.pm
@@ +256,4 @@
> my $where = $tmpl =~ /xsl$/ ? 'xslt' : 'modules';
> for my $theme (@themes) {
> if ( -e "$htdocs/$theme/$lang/$where/$tmpl" ) {
> + return ( $theme, $lang, [ uniq(@themes) ] );
I'm not sure the parenthesis for uniq are necessary.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 03:58:15 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 02:58:15 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
--- Comment #132 from Alex Buckley ---
*** Bug 17955 has been marked as a duplicate of this bug. ***
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 03:58:15 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 02:58:15 +0000
Subject: [Koha-bugs] [Bug 17955] Remove the sql code and unnecessary use
pragmas from the onboarding.pl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17955
Alex Buckley changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |DUPLICATE
Status|ASSIGNED |RESOLVED
--- Comment #8 from Alex Buckley ---
The commit that was in bug 17955 has been attached to bug 17855. Meaning the
changes required by bug 17955 have been implemented by bug 17855.
*** This bug has been marked as a duplicate of bug 17855 ***
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 03:59:42 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 02:59:42 +0000
Subject: [Koha-bugs] [Bug 17942] Update style of the web installer with
Bootstrap 3
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17942
--- Comment #21 from M. Tompsett ---
(In reply to Owen Leonard from comment #20)
> (In reply to M. Tompsett from comment #14)
> > There, screen shots of what I was talking about. Explain the reasons for the
> > difference.
>
> This is rude.
I apologize for the shortness. I'm sorry. I was wrong.
> (In reply to M. Tompsett from comment #19)
> "Superlibrarian" is red because it is highlighted for visibility in a page
> which is structured with sections topped by headings. There is nothing in
> that section other than text. I think its appropriate to provide a little
> more visibility to that text to help show that it is the focus of that
> section.
>
> "Staff" is plain black because it is informational text being displayed as
> part of a form. As in other places in Koha, when a line in a form displays
> information instead of some kind of input, plain text is paired with class="label">Label: to preserve the structure and flow of the
> form.
> The plain text fits with the structure of the form and does not need to be
> highlighted.
Thank you for the explanation.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 04:04:12 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 03:04:12 +0000
Subject: [Koha-bugs] [Bug 17942] Update style of the web installer with
Bootstrap 3
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17942
--- Comment #22 from M. Tompsett ---
(In reply to Josef Moravec from comment #18)
> > In English it's OK
I can confirm this. Though, OK is so bland. It is quite pretty. Did you get it
working in Czech, Josef? If so, would you like to sign off, or shall I?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 04:31:28 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 03:31:28 +0000
Subject: [Koha-bugs] [Bug 16034] Integration with OverDrive Patron API
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16034
Srdjan Jankovic changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59394|0 |1
is obsolete| |
--- Comment #53 from Srdjan Jankovic ---
Created attachment 59713
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59713&action=edit
bug_16034 Add overdrive info to the users page in the public interface
Signed-off-by: Jesse Weaver
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 04:31:55 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 03:31:55 +0000
Subject: [Koha-bugs] [Bug 16034] Integration with OverDrive Patron API
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16034
Srdjan Jankovic changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59395|0 |1
is obsolete| |
--- Comment #54 from Srdjan Jankovic ---
Created attachment 59714
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59714&action=edit
Bug 16034 follow-up: added WebService::ILS to PerlDependencies
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 05:05:54 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 04:05:54 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
--- Comment #16 from Alex Buckley ---
Created attachment 59715
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59715&action=edit
Bug 17944 - Moved the DBIx code into the ItemType.pm file
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 05:07:56 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 04:07:56 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
Alex Buckley changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 05:20:35 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 04:20:35 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
--- Comment #17 from M. Tompsett ---
Comment on attachment 59715
--> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59715
Bug 17944 - Moved the DBIx code into the ItemType.pm file
Review of attachment 59715:
--> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=17944&attachment=59715)
-----------------------------------------------------------------
::: admin/itemtypes.pl
@@ +140,4 @@
>
> } elsif ( $op eq 'delete_confirm' ) {
>
> + my $overalltotal = Koha::ItemType->get_items_and_biblioitems($itemtype_code);
This is what was asked for in comment #15.
@@ +151,4 @@
>
> } elsif ( $op eq 'delete_confirmed' ) {
>
> + my $itemtype = Koha::ItemType->get_items_and_biblioitems($itemtype_code);
This change confuses me.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 05:23:09 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 04:23:09 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |ASSIGNED
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 05:29:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 04:29:21 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
--- Comment #18 from Alex Buckley ---
Created attachment 59716
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59716&action=edit
Bug 17955 - removed incorrect function call
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 05:29:41 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 04:29:41 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
Alex Buckley changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 05:47:47 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 04:47:47 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
--- Comment #19 from Alex Buckley ---
Created attachment 59717
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59717&action=edit
Bug 17944 - Added additional USE pragma
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 06:02:39 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 05:02:39 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
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 Feb 1 06:02:42 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 05:02:42 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59715|0 |1
is obsolete| |
Attachment #59716|0 |1
is obsolete| |
Attachment #59717|0 |1
is obsolete| |
--- Comment #20 from M. Tompsett ---
Created attachment 59718
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59718&action=edit
Bug 17944 - Moved the DBIx code into the ItemType.pm file
Created get_items_and_biblioitems in Koha::ItemType using
DBIx coding style.
Changed itemtypes.pl to use that method
This addresses comment #15.
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 Feb 1 06:12:04 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 05:12:04 +0000
Subject: [Koha-bugs] [Bug 17993] Do not use modal authentication with CAS
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17993
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59549|0 |1
is obsolete| |
--- Comment #3 from M. Tompsett ---
Created attachment 59719
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59719&action=edit
Bug 17993 - Do not use modal authentication with CAS - lists
Bug 12046 corrected the fact that modal dialog does not allow to use the CAS
authentication in main authentication link.
This must also be corrected in link of lists popup : "Log in to create your own
lists"
Test plan :
- Enable syspref casAuthentication
- Go to OPAC
- Click on Lists > Log in to create your own lists
=> Without patch you get the modal login popup
=> With patch you go to the login page opac-user.pl
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 Feb 1 06:12:09 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 05:12:09 +0000
Subject: [Koha-bugs] [Bug 17993] Do not use modal authentication with CAS
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17993
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59550|0 |1
is obsolete| |
--- Comment #4 from M. Tompsett ---
Created attachment 59720
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59720&action=edit
Bug 17993 - Do not use modal authentication with CAS - tags
Bug 12046 corrected the fact that modal dialog does not allow to use the CAS
authentication in main authentication link.
This must also be corrected in link for tags in detail page : "Log in to add
tags"
Test plan :
- Enable syspref casAuthentication
- Go to OPAC
- Go to a record detail page opac-detail.pl
- Click on "Log in to add tags"
=> Without patch you get the modal login popup
=> With patch you go to the login page opac-user.pl
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 Feb 1 06:13:17 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 05:13:17 +0000
Subject: [Koha-bugs] [Bug 17993] Do not use modal authentication with CAS
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17993
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mtompset at hotmail.com
Status|Needs Signoff |In Discussion
--- Comment #5 from M. Tompsett ---
Missed a spot: opac-tags.pl
Is that fixed elsewhere, or should it be fixed here?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 06:30:41 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 05:30:41 +0000
Subject: [Koha-bugs] [Bug 17886] Don' t show option to add to existing list
if there are no lists
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17886
--- Comment #8 from M. Tompsett ---
(In reply to Owen Leonard from comment #7)
> (In reply to M. Tompsett from comment #5)
>
> > I believe this is the wrong solution. It is much simpler.
>
> Since you're obsoleting a patch from an Academy student in favor of your own
> I think it's worthwhile to give a detailed explanation of your reasoning.
Sorry for taking so long to notice this. Valid point, Owen.
The problem is not the 'Add To' heading section (as hinted at by the 3rd
obsolete patch).
The problem is not around line 40ish (as hinted at by the 1st two obsolete
patches).
The problem is when there are no lists, it makes no sense to show that you can
select a list. The problem is the code correctly checks the count method of the
two shelf types, but fails to do so on the third on line 26.
By simplying adding .count to the third shelf type, the options available are
properly shown. There is no need to add a condition. The existing condition was
broken.
Though, what I do miss about the first two patches are corrections in
indentation.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 07:10:20 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 06:10:20 +0000
Subject: [Koha-bugs] [Bug 18027] New: Added a new item which does not appear
in catalogue search but appears in acquisition report.
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18027
Bug ID: 18027
Summary: Added a new item which does not appear in catalogue
search but appears in acquisition report.
Change sponsored?: ---
Product: Koha
Version: unspecified
Hardware: PC
OS: Windows
Status: NEW
Severity: critical
Priority: P5 - low
Component: Cataloging
Assignee: gmcharlt at gmail.com
Reporter: clouca at cpl.org.au
QA Contact: testopia at bugs.koha-community.org
CC: m.de.rooy at rijksmuseum.nl
Added the item three times (with different barcodes)and these show in report
but cannot be found in catalogue search or by a patron on their platform.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 07:18:59 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 06:18:59 +0000
Subject: [Koha-bugs] [Bug 18027] Added a new item which does not appear in
catalogue search but appears in acquisition report.
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18027
Christalla changed:
What |Removed |Added
----------------------------------------------------------------------------
Priority|P5 - low |P1 - high
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 08:53:30 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 07:53:30 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Failed QA
--- Comment #21 from Jonathan Druart ---
Alex,
There are some wrong things here:
1/ No tests are provided
2/ You do not instanciate a Koha::ItemType object but using
get_items_and_biblioitems as a class method.
3/ get_items_and_biblioitems is badly named, it does not return the items and
the biblioitems, but the number of item and biblioitems.
That's why I suggested ->is_used, which would return true or false.
4/ There is no POD for this new method
5/ Why do not you want to use Koha::Items and Koha::Biblioitems?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 09:03:41 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 08:03:41 +0000
Subject: [Koha-bugs] [Bug 17942] Update style of the web installer with
Bootstrap 3
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17942
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59623|0 |1
is obsolete| |
--- Comment #23 from Josef Moravec ---
Created attachment 59721
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59721&action=edit
[SIGNED-OFF] Bug 17942 - Update style of the web installer with Bootstrap 3
This patch makes style and markup changes to the web installer and new
onboarding tool (Bug 17855).
- Markup has been corrected, improved, and in many places reindented.
- Some page titles have been made more specific.
- Some JavaScript and CSS have been moved to separate files.
In the onboarding tool I have removed form validation from the markup
and JavaScript in many cases where the requirements were not matched
elsewhere in Koha. For instance, we shouldn't limit item type
descriptions to only letters because the database doesn't require such a
limit.
To test, apply the patch and run the web installer with an empty
database. Confirm that the installation process completes correctly and
that each page looks good and works correctly.
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 09:03:34 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 08:03:34 +0000
Subject: [Koha-bugs] [Bug 17942] Update style of the web installer with
Bootstrap 3
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17942
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 09:05:14 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 08:05:14 +0000
Subject: [Koha-bugs] [Bug 17942] Update style of the web installer with
Bootstrap 3
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17942
--- Comment #24 from Josef Moravec ---
(In reply to M. Tompsett from comment #22)
> (In reply to Josef Moravec from comment #18)
> > > In English it's OK
>
> I can confirm this. Though, OK is so bland. It is quite pretty. Did you get
> it working in Czech, Josef? If so, would you like to sign off, or shall I?
Yes, after regenerating templates everything is OK, so my mistake...
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 09:06:38 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 08:06:38 +0000
Subject: [Koha-bugs] [Bug 17942] Update style of the web installer with
Bootstrap 3
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17942
--- Comment #25 from Josef Moravec ---
Signed off, has to say it looks really nice! Good job Owen!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 09:13:52 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 08:13:52 +0000
Subject: [Koha-bugs] [Bug 18022] Empty barcode causes internal server error
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18022
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
Severity|enhancement |major
--- Comment #1 from Jonathan Druart ---
https://lists.katipo.co.nz/pipermail/koha/2017-January/047102.html
--
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 Feb 1 09:14:33 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 08:14:33 +0000
Subject: [Koha-bugs] [Bug 18022] Empty barcode causes internal server error
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18022
--- Comment #2 from Jonathan Druart ---
Note, from the mailing list: I am not able to recreate this issue.
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 09:17:30 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 08:17:30 +0000
Subject: [Koha-bugs] [Bug 17775] Add new user with LDAP not works under Plack
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17775
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Passed QA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 09:17:41 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 08:17:41 +0000
Subject: [Koha-bugs] [Bug 17775] Add new user with LDAP not works under Plack
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17775
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59662|0 |1
is obsolete| |
--- Comment #16 from Jonathan Druart ---
Created attachment 59722
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59722&action=edit
Bug 17775 - Add new user with LDAP not works under Plack
This patch fixes internal server error:
Undefined subroutine &C4::Auth_with_ldap::AddMember called at
/srv/koha_ffzg/C4/Auth_with_ldap.pm line 213.
It occurs only under plack, and it's strange since C4::Members
does EXPORT AddMember and we are importing it into Auth_with_ldap.pm
(and it does work under CGI).
Signed-off-by: Liz Rea
Signed-off-by: Jonathan Druart
I did not test but trust author and signoffer. The change cannot hurt.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 10:01:07 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 09:01:07 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Failed QA
--- Comment #133 from Jonathan Druart ---
Alex,
Few more remarks:
1/ There is something wrong with the sql sample files for translated languages.
The mandatory and optional files for the other languages have to be moved to
the default directory as well.
We will have to be meticulous to avoid regressions.
2/ You should also adapt xt/sample_notices.t, tests are failing after your
patches
3/ Could you double check the module you use in installer/onboarding.pl, some
are uselessly used.
4/ Double check the copyright lines you have added (+# Copyright (C) YEAR
YOURNAME-OR-YOUREMPLOYER)
5/ You copied some existing javascript functions to onboardingstep2.tt. Do not
duplicate existing code.
You should also have a look at this template, the indentation in the javascript
portion is weird.
6/ Same as 5 for onboardingstep3.tt
7/ summary.tt - you should not use
tag, our translation tool does not
handle sentences when they are split by html tag. That make the job really hard
(and sometimes impossible) for translators.
8/ Copyright of summary.pl sounds totally wrong.
9/ Is this Summary info really useful? To me it looks like it should be moved
to the help part. I am not sure it worth a new entry in the "More" menu.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 10:08:15 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 09:08:15 +0000
Subject: [Koha-bugs] [Bug 15030] Certain values in serials' items are lost
on next edit
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15030
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 10:08:20 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 09:08:20 +0000
Subject: [Koha-bugs] [Bug 15030] Certain values in serials' items are lost
on next edit
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15030
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59671|0 |1
is obsolete| |
Attachment #59672|0 |1
is obsolete| |
Attachment #59673|0 |1
is obsolete| |
Attachment #59674|0 |1
is obsolete| |
--- Comment #43 from Josef Moravec ---
Created attachment 59723
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59723&action=edit
[SIGNED-OFF] Bug 15030 - Certain values in serials' items are lost on next edit
When editing serials subscription, we can edit them but some values are not
pulled from the DB correctly to be put in the edit box. If not noticed, the
value will be overwritten on the next save.
Test:
- Create a subscription
- Edit itemcallnumber (952o?) and make sure to have a different value than the
default one.
- Save.
- Edit it again
- The saved value is not there.
This is true for itemcallnumber and a few other fields.
This was caused by calls to ->field($subfield). This would always fail, of
course.
Signed-off-by: Nick Clemens
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 10:08:56 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 09:08:56 +0000
Subject: [Koha-bugs] [Bug 15030] Certain values in serials' items are lost
on next edit
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15030
--- Comment #44 from Josef Moravec ---
Created attachment 59724
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59724&action=edit
[SIGNED-OFF] Bug 15030 - Fixes the serials fields associated with a plugin, to
not overwrite the previously saved value
This fixes the remaining fields from serials-edit.pl that were seeing their
previously entered values
be oblitarated with each new edit. The fields associated to a plugin
(dateaccessioned and barcode) were
always displaying with each new edit, losing the previous effort.
Signed-off-by: Nick Clemens
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 10:09:16 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 09:09:16 +0000
Subject: [Koha-bugs] [Bug 15030] Certain values in serials' items are lost
on next edit
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15030
--- Comment #45 from Josef Moravec ---
Created attachment 59725
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59725&action=edit
[SIGNED-OFF] Bug 15030 - continue. The enumchron value was overwritten by
'header' even when value supplied
Signed-off-by: Nick Clemens
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 10:09:29 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 09:09:29 +0000
Subject: [Koha-bugs] [Bug 15030] Certain values in serials' items are lost
on next edit
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15030
--- Comment #46 from Josef Moravec ---
Created attachment 59726
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59726&action=edit
[SIGNED-OFF] Bug 15030: Add tests
This test will prevent regression on the lost of data when
items.itemcallnumber is linked with a plugin.
Signed-off-by: Nick Clemens
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 10:13:42 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 09:13:42 +0000
Subject: [Koha-bugs] [Bug 17987] Changing
PatronSelfRegistrationDefaultCategory to a dropdown menu syspref
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17987
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
Status|Signed Off |Failed QA
--- Comment #6 from Jonathan Druart ---
Hi Aleisha,
Here are a few remarks:
1/ in the preferences.pl file, I think it would be better to affect the patron
category list to $chunk->{'CHOICES'} if the type of the syspref is
patron-categories, and not depending on its name. That way it will be reusable
more easily.
2/ There is type=PatronCategories"in the DB, choices:patron-categories in the
opac.pref, then choices eq 'patron-categories' in preferences.pl
How is useful this type=PatronCategories?
3/ There are 2 problematic cases:
a. The field is not filled yet: it's not visible on the interface, there is no
blank entry
b. A bit more tricky: if the patron category has been removed, it will not be
displayed.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 10:15:12 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 09:15:12 +0000
Subject: [Koha-bugs] [Bug 17111] Automatic debian/control updates
(oldstable/16.05.x)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17111
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
Version|3.22 |16.05
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 10:37:33 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 09:37:33 +0000
Subject: [Koha-bugs] [Bug 18016] C4::Auth_with_ldap:: ldap_entry_2_hash
inserts 0000-00-00 for invalid dateofbirth
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18016
--- Comment #3 from Jonathan Druart ---
(In reply to Dobrica Pavlinusic from comment #2)
> Seems good, but would it better to do dt_from_string on them to verify they
> are also valid?
It should not happen I think. If that happened, I guess some people would have
raised this problem before.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 10:40:09 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 09:40:09 +0000
Subject: [Koha-bugs] [Bug 18009] IssueSlip.t test fails if launched between
00:00 and 00:59
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18009
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Passed QA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 10:40:13 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 09:40:13 +0000
Subject: [Koha-bugs] [Bug 18009] IssueSlip.t test fails if launched between
00:00 and 00:59
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18009
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59695|0 |1
is obsolete| |
--- Comment #3 from Jonathan Druart ---
Created attachment 59727
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59727&action=edit
Bug 18009 - IssueSlip.t test fails if launched between 00:00 and 00:59
to test patch...
1/ set date between 00:00 and 00:59
$ sudo date -s 'Sun Jan 29 00:41:55 NZDT 2017'
2/ run prove, see fail
$ prove -v t/db_dependent/Members/IssueSlip.t
...
t/db_dependent/Members/IssueSlip.t (Wstat: 65280 Tests: 1 Failed: 0)
Result: FAIL
3/ apply patch
4/ run prove, see pass
$ prove -v t/db_dependent/Members/IssueSlip.t
...
All tests successful.
Result: PASS
NOTE: for code obscurity you could have also done a modulus 24. ;)
Signed-off-by: Mark Tompsett
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 Wed Feb 1 10:48:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 09:48:36 +0000
Subject: [Koha-bugs] [Bug 17111] Automatic debian/control updates
(oldstable/16.05.x)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17111
--- Comment #30 from Jonathan Druart ---
Mirko, do you know what is the install_misc/debian.packages file?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:08:03 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:08:03 +0000
Subject: [Koha-bugs] [Bug 17047] Mana webservice : share and use common
information between Kohas
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17047
--- Comment #18 from Alex Arnaud ---
Created attachment 59728
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59728&action=edit
Bug 17047 - Don't create numberpattern if the same exists
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:08:11 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:08:11 +0000
Subject: [Koha-bugs] [Bug 17047] Mana webservice : share and use common
information between Kohas
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17047
--- Comment #19 from Alex Arnaud ---
Created attachment 59729
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59729&action=edit
Bug 17047 - Remove action column from mana search result table
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:09:19 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:09:19 +0000
Subject: [Koha-bugs] [Bug 17047] Mana webservice : share and use common
information between Kohas
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17047
Alex Arnaud changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:09:47 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:09:47 +0000
Subject: [Koha-bugs] [Bug 17047] Mana webservice : share and use common
information between Kohas
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17047
--- Comment #20 from Alex Arnaud ---
Remember, test plan is here:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17047#c16
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:15:30 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:15:30 +0000
Subject: [Koha-bugs] [Bug 17927] REST API: /holds and /patrons attributes
have wrong types
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17927
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
Status|Passed QA |Failed QA
--- Comment #6 from Jonathan Druart ---
One test is failing after this patch:
# Failed test 'Columns is nullable in DB, not in swagger file for hold:
timestamp'
# at t/db_dependent/api/v1/swagger/definitions.t line 71.
I also think that we should add tests to check data types, don't you think?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:35:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:35:36 +0000
Subject: [Koha-bugs] [Bug 18008] Use templates for generating pagination
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18008
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|oleonard at myacpl.org |josef.moravec at gmail.com
Status|NEW |ASSIGNED
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:37:45 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:37:45 +0000
Subject: [Koha-bugs] [Bug 18028] New: install_dir/*.packages are outdated
and must be removed
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18028
Bug ID: 18028
Summary: install_dir/*.packages are outdated and must be
removed
Change sponsored?: ---
Product: Koha
Version: unspecified
Hardware: All
OS: All
Status: ASSIGNED
Severity: normal
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: jonathan.druart at bugs.koha-community.org
Reporter: jonathan.druart at bugs.koha-community.org
QA Contact: testopia at bugs.koha-community.org
It seems that the install_dir/*.packages files are no longer maintained and not
really useful.
I would suggest to remove them.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:38:00 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:38:00 +0000
Subject: [Koha-bugs] [Bug 18028] install_dir/*.packages are outdated and
must be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18028
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=17626
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:38:00 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:38:00 +0000
Subject: [Koha-bugs] [Bug 17626] INSTALL files are outdated
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17626
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=18028
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:39:16 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:39:16 +0000
Subject: [Koha-bugs] [Bug 18028] install_dir/*.packages are outdated and
must be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18028
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |8840
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8840
[Bug 8840] ubuntu-pkg-check.sh fix and extend functionality
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:39:16 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:39:16 +0000
Subject: [Koha-bugs] [Bug 8840] ubuntu-pkg-check.sh fix and extend
functionality
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8840
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |18028
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18028
[Bug 18028] install_dir/*.packages are outdated and must be removed
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:44:53 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:44:53 +0000
Subject: [Koha-bugs] [Bug 18028] install_dir/*.packages are outdated and
must be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18028
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |Needs Signoff
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:44:59 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:44:59 +0000
Subject: [Koha-bugs] [Bug 18028] install_dir/*.packages are outdated and
must be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18028
--- Comment #1 from Jonathan Druart ---
Created attachment 59730
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59730&action=edit
Bug 18028: Remove outdated install_misc directory
This directory is no longer maintained and contain outdated information
on how to install Koha.
On the Internet there are tutos using them and that can create confusion
to new user (yes it could be confusing as well to execute a file that
does no longer exist).
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:45:19 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:45:19 +0000
Subject: [Koha-bugs] [Bug 18028] install_misc directory is outdated and must
be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18028
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|install_dir/*.packages are |install_misc directory is
|outdated and must be |outdated and must be
|removed |removed
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:49:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:49:21 +0000
Subject: [Koha-bugs] [Bug 17111] Automatic debian/control updates
(oldstable/16.05.x)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17111
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=18028
--- Comment #31 from Jonathan Druart ---
(In reply to Jonathan Druart from comment #30)
> Mirko, do you know what is the install_misc/debian.packages file?
Opened bug 18028 to talk about that.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:49:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:49:21 +0000
Subject: [Koha-bugs] [Bug 18028] install_misc directory is outdated and must
be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18028
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=17111
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 12:51:10 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 11:51:10 +0000
Subject: [Koha-bugs] [Bug 17111] Automatic debian/control updates
(oldstable/16.05.x)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17111
Mirko Tietgen 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 Wed Feb 1 13:01:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:01:05 +0000
Subject: [Koha-bugs] [Bug 17932] Koha::Object should provide a TO_JSON method
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17932
--- Comment #14 from Marcel de Rooy ---
Thanks, Nick. You beat me in signing off :)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 13:03:01 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:03:01 +0000
Subject: [Koha-bugs] [Bug 18014] AddAuthority should respect AUTO_INCREMENT
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18014
--- Comment #14 from Marcel de Rooy ---
(In reply to M. Tompsett from comment #13)
> Didn't test just the first patch by itself, but tested 1&2 together and all
> three together. That's why I commented like I dd in comment #8 and comment
> #9. But the end product is good. :)
Thanks for your feedback and signoff !
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 13:07:01 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:07:01 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
--- Comment #5 from Marcel de Rooy ---
(In reply to M. Tompsett from comment #4)
> Comment on attachment 59630 [details] [review]
> Bug 17982: Fix the use of uniq in sub themelanguage
>
> Review of attachment 59630 [details] [review]:
> -----------------------------------------------------------------
>
> ::: C4/Templates.pm
> @@ +256,4 @@
> > my $where = $tmpl =~ /xsl$/ ? 'xslt' : 'modules';
> > for my $theme (@themes) {
> > if ( -e "$htdocs/$theme/$lang/$where/$tmpl" ) {
> > + return ( $theme, $lang, [ uniq(@themes) ] );
>
> I'm not sure the parenthesis for uniq are necessary.
Seems to be a more a matter of taste.
My personal preference is parentheses, only omitting them with builtins like
open or print.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 13:08:53 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:08:53 +0000
Subject: [Koha-bugs] [Bug 18018] WWW:CSRF is missing in PerlDependencies.pm
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18018
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Passed QA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 13:08:59 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:08:59 +0000
Subject: [Koha-bugs] [Bug 18018] WWW:CSRF is missing in PerlDependencies.pm
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18018
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59694|0 |1
is obsolete| |
--- Comment #3 from Jonathan Druart ---
Created attachment 59731
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59731&action=edit
Bug 18018 - WWW:CSRF is missing in PerlDependencies.pm
WWW:CSRF is missing in PerlDependencies.pm and so
libwww-csrf-perl is missing in PerlDependencies.pm
missing in debian/control.
This patch adds both.
NOTE: Tested on top of 17111.
Checked
./koha_perl_deps.pl -m -u
before and after. Control file eyeballs well.
git diff origin/master -- debian/control
does not differ in this change. :)
Signed-off-by: Mark Tompsett
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 Wed Feb 1 13:15:12 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:15:12 +0000
Subject: [Koha-bugs] [Bug 17927] REST API: /holds and /patrons attributes
have wrong types
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17927
--- Comment #7 from Tom?s Cohen Arazi ---
(In reply to Jonathan Druart from comment #6)
> One test is failing after this patch:
>
> # Failed test 'Columns is nullable in DB, not in swagger file for
> hold: timestamp'
> # at t/db_dependent/api/v1/swagger/definitions.t line 71.
>
> I also think that we should add tests to check data types, don't you think?
I think we might better get rid of definitions.t as there's no obvious reason
why the swagger definition should match the DB. I can take care of adding
'null' as a valid type for 'timestamp' but I think we should better move that
to another bug as the tests are failing without this patches too. See comment
#5
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 13:16:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:16:05 +0000
Subject: [Koha-bugs] [Bug 17927] REST API: /holds and /patrons attributes
have wrong types
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17927
--- Comment #8 from Tom?s Cohen Arazi ---
(In reply to Jonathan Druart from comment #6)
> One test is failing after this patch:
>
> # Failed test 'Columns is nullable in DB, not in swagger file for
> hold: timestamp'
> # at t/db_dependent/api/v1/swagger/definitions.t line 71.
>
> I also think that we should add tests to check data types, don't you think?
Also, are you sure you tested this along with bug 17932?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 13:17:11 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:17:11 +0000
Subject: [Koha-bugs] [Bug 17111] Automatic debian/control updates
(oldstable/16.05.x)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17111
Mirko Tietgen changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59691|0 |1
is obsolete| |
--- Comment #32 from Mirko Tietgen ---
Created attachment 59732
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59732&action=edit
Bug 17111: Update control file for 16.05
https://bugs.koha-community.org/show_bug.cgi?id=17111
NOTE: Correctly adds/removes dependencies in debian/control
Signed-off-by: Mark Tompsett
Signed-off-by: Mirko Tietgen
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 13:25:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:25:05 +0000
Subject: [Koha-bugs] [Bug 17927] REST API: /holds and /patrons attributes
have wrong types
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17927
--- Comment #9 from Jonathan Druart ---
(In reply to Tom?s Cohen Arazi from comment #7)
> I think we might better get rid of definitions.t as there's no obvious
> reason why the swagger definition should match the DB. I can take care of
> adding 'null' as a valid type for 'timestamp' but I think we should better
> move that to another bug as the tests are failing without this patches too.
No, they pass on master.
(In reply to Tom?s Cohen Arazi from comment #8)
> Also, are you sure you tested this along with bug 17932?
Yes I did.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 13:50:08 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:50:08 +0000
Subject: [Koha-bugs] [Bug 17933] Internal software error when searching
patron without birth date
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17933
Dobrica Pavlinusic changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59194|0 |1
is obsolete| |
Attachment #59214|0 |1
is obsolete| |
Attachment #59215|0 |1
is obsolete| |
--- Comment #12 from Dobrica Pavlinusic ---
Created attachment 59733
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59733&action=edit
Bug 17933: Do not instanciate a patron if not needed
Signed-off-by: Dobrica Pavlinusic
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 13:51:53 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:51:53 +0000
Subject: [Koha-bugs] [Bug 17933] Internal software error when searching
patron without birth date
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17933
Dobrica Pavlinusic changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59733|0 |1
is obsolete| |
--- Comment #13 from Dobrica Pavlinusic ---
Created attachment 59734
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59734&action=edit
Bug 17933 - Internal software error when searching patron without birth date
When patrons don't have date of birth (which is not required) patron
search results on moremember page produce internal server error since we
can't convert MySQL invalid date 0000-00-00 to datetime object and
call strfdate on it.
Additionally, since we assign dates to template variables and after
than assign whole $data hash to template, later assigment overrides
previous one, so we see birth date field even for patrons which don't
have one.
This patch fixes both of those problems.
Test:
1. edit patron and remove it's birth date
2. try to search for it, and verify server error
3. apply patch
4. repeat search for patron and verify that it works and doesn't
have enpty birth date field
Signed-off-by: Grace McKenzie
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 13:52:09 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:52:09 +0000
Subject: [Koha-bugs] [Bug 17933] Internal software error when searching
patron without birth date
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17933
--- Comment #15 from Dobrica Pavlinusic ---
Created attachment 59736
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59736&action=edit
Bug 17933: Do not instanciate a patron if not needed
Signed-off-by: Dobrica Pavlinusic
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 13:52:04 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 12:52:04 +0000
Subject: [Koha-bugs] [Bug 17933] Internal software error when searching
patron without birth date
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17933
--- Comment #14 from Dobrica Pavlinusic ---
Created attachment 59735
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59735&action=edit
Bug 17933: Add test and return unless defined dob
Without this patch get_age return actually 0 instead of
Signed-off-by: Dobrica Pavlinusic
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:01:06 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:01:06 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
--- Comment #134 from Owen Leonard ---
(In reply to Jonathan Druart from comment #133)
> 9/ Is this Summary info really useful? To me it looks like it should be
> moved to the help part. I am not sure it worth a new entry in the "More"
> menu.
I agree. I wonder if we could add a persistent shortcut of some kind to the
global help system? I suggest this feature be discussed in a separate bug.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:09:49 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:09:49 +0000
Subject: [Koha-bugs] [Bug 13757] Make patron attributes editable in the opac
if set to ' editable in OPAC'
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757
--- Comment #72 from Tom?s Cohen Arazi ---
Created attachment 59738
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59738&action=edit
Bug 13757: Make Koha::Patron::Modification->store del empty attrs
This patch makes Koha::Patron::Modification->store delete the passed
attributes that contain empty values.
This is the way it currently works, as all opac-editable attributes are
presented to the end-user and they are allowed to delete them, and the
best way I found to represent the deletion on the modification request
is by setting it to the empty string. I chose this way because it is how
the staff interface handles it, so it is consistent.
To test:
- Apply this patch
- Run:
$ prove t/db_dependent/Koha/Patron/Modifications.t
=> SUCCESS: This time tests pass!
- Verify comment #70 on the bug is fixed now
- Sign off :-D
Signed-off-by: Tomas Cohen Arazi
https://bugs.koha-community.org/show_bug.cgi?id=13737
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:09:57 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:09:57 +0000
Subject: [Koha-bugs] [Bug 13757] Make patron attributes editable in the opac
if set to ' editable in OPAC'
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757
--- Comment #73 from Tom?s Cohen Arazi ---
Created attachment 59739
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59739&action=edit
Bug 13757: (QA followup) Make opac-memberentry.pl handle attrs deletion
The original code on this bug skipped empty-valued attributes. But
emptying attributes is the only way to tell the controller script that
the user wants to delete them.
This patch makes opac-memberentry.pl check the existence of attributes
sharing the code of the empty for the given patron, and it stores the
deletion on the Koha::Patron::Modification as needed. Otherwise
deletions got skipped.
To test:
- Verify setting/deleting attributes that are opac-editable and verify
the results are sound.
https://bugs.koha-community.org/show_bug.cgi?id=13737
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:12:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:12:05 +0000
Subject: [Koha-bugs] [Bug 17941] CanBookBeRenewed is very inefficient/slow
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17941
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
--- Comment #2 from Jonathan Druart ---
Hi Petter,
I'd be interested to recreate the perf issue.
I have written this quick script, it generates 20 items, 200 patrons and
generate a hold for each patron.
On the second run, I comment everything but the last 2 lines and execute again
the script. It is executed in 1.5s
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:13:37 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:13:37 +0000
Subject: [Koha-bugs] [Bug 16202] Rental fees can be generated for fractions
of a penny/cent
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16202
--- Comment #6 from Colin Campbell ---
Other uses of sprintf in C4::Circulation no loger have the superfluous 0 - can
we fix the bug now
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:24:14 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:24:14 +0000
Subject: [Koha-bugs] [Bug 13737] [OMNIBUS] Add Koha RESTful Services
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13737
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |tomascohen at gmail.com
--- Comment #5 from Tom?s Cohen Arazi ---
Wrong bug for attaching this patches.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:24:17 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:24:17 +0000
Subject: [Koha-bugs] [Bug 13737] [OMNIBUS] Add Koha RESTful Services
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13737
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59688|0 |1
is obsolete| |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:24:19 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:24:19 +0000
Subject: [Koha-bugs] [Bug 13737] [OMNIBUS] Add Koha RESTful Services
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13737
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59686|0 |1
is obsolete| |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:10:12 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:10:12 +0000
Subject: [Koha-bugs] [Bug 17941] CanBookBeRenewed is very inefficient/slow
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17941
--- Comment #1 from Jonathan Druart ---
Created attachment 59740
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59740&action=edit
Try to recreate CanBookBeRenewed perf issue
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:24:22 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:24:22 +0000
Subject: [Koha-bugs] [Bug 13737] [OMNIBUS] Add Koha RESTful Services
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13737
Tom?s Cohen Arazi changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59687|0 |1
is obsolete| |
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:09:42 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:09:42 +0000
Subject: [Koha-bugs] [Bug 13757] Make patron attributes editable in the opac
if set to ' editable in OPAC'
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757
--- Comment #71 from Tom?s Cohen Arazi ---
Created attachment 59737
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59737&action=edit
Bug 13757: (regression tests) Empty attributes should delete existing
This patch introduces tests for the required functionality.
To test:
- Run:
$ prove t/db_dependent/Koha/Patron/Modifications.t
=> FAIL: The current code doesn't work like that
Signed-off-by: Tomas Cohen Arazi
https://bugs.koha-community.org/show_bug.cgi?id=13737
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:54:28 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:54:28 +0000
Subject: [Koha-bugs] [Bug 16202] Rental fees can be generated for fractions
of a penny/cent
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16202
--- Comment #7 from M. Tompsett ---
(In reply to Colin Campbell from comment #6)
> Other uses of sprintf in C4::Circulation no loger have the superfluous 0 -
> can we fix the bug now
Valid counter point. Okay.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 14:58:48 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 13:58:48 +0000
Subject: [Koha-bugs] [Bug 17941] CanBookBeRenewed is very inefficient/slow
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17941
--- Comment #3 from Petter Goks?yr ?sen ---
Hi Jonathan. Thanks for looking into this!
We were actually working on this problem right now, so I have some information.
The code in question is quite complicated, so I'm not sure I understand it, but
it seems that the pathological cases are when most of the items are checked
out, so it needs to check every permutation of item, hold, and borrower to
check if it can be renewed. But I'm not 100% sure about it.
Rather than trying to understand how all the holds logic work in order to fix
this properly, we have made a patch now, that greatly improves the speed. This
is done by memoizing idempotent DB calls (GetItem, GetMembers and so on),
during the scope of CanBookBeRenewed. It helps a lot, because sometimes the
same item is fetched again and again, 100s to even 1000s of times from the DB!
In one case, by using the patch, the display of checkout tab with 5 items goes
from around 20 seconds to 3 seconds,
We do belive this patch is a good step forward, but we will test it extensivly
in staging environment before we submit it.
I'll try see if I can get some interesting result from your script.
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 15:04:12 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 14:04:12 +0000
Subject: [Koha-bugs] [Bug 13757] Make patron attributes editable in the opac
if set to ' editable in OPAC'
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Failed QA
--- Comment #74 from Jonathan Druart ---
I do not know if it comes from the last patches but I found something else.
if I do not make any changes to the editable attrs, they will appear as
modified anyway (entry in borrower_modifications).
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 15:07:06 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 14:07:06 +0000
Subject: [Koha-bugs] [Bug 13757] Make patron attributes editable in the opac
if set to ' editable in OPAC'
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757
--- Comment #75 from Jonathan Druart ---
Another one:
1/ Set an attr as displayed at the OPAC but not editable
2/ At the OPAC, fill the edit patron form but miss a mandatory field, submit
3/ You get the form again but the attr is not displayed.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 15:07:46 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 14:07:46 +0000
Subject: [Koha-bugs] [Bug 16202] Rental fees can be generated for fractions
of a penny/cent
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16202
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 Feb 1 15:07:49 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 14:07:49 +0000
Subject: [Koha-bugs] [Bug 16202] Rental fees can be generated for fractions
of a penny/cent
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16202
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #49907|0 |1
is obsolete| |
--- Comment #8 from M. Tompsett ---
Created attachment 59741
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59741&action=edit
Bug 16202: GetIssuingCharges should return a collectable amount
Charges should not include elements less than a penny/cent
they are not displayed but can be saved to the database
causing "odd" behaviour down the line
Make the routine round the resultant charge to nearest cent,
so consistent values are returned.
Removed the one case where it was rounded post call.
Although the main danger is values generated by the discount
calculation apply the rounding to all returned charges in case
the item charge is defined using the 3rd or 4th decimal
places.
NOTE: prove -v t/db_dependent/Circulation.t triggers the change.
Though, all the returned amounts are 0.00 only.
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 Feb 1 15:11:35 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 14:11:35 +0000
Subject: [Koha-bugs] [Bug 17941] CanBookBeRenewed is very inefficient/slow
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17941
--- Comment #4 from Jonathan Druart ---
(In reply to Petter Goks?yr ?sen from comment #3)
> We do belive this patch is a good step forward, but we will test it
> extensivly in staging environment before we submit it.
No do not wait, release as soon as you can :)
--
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 Feb 1 15:22:07 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 14:22:07 +0000
Subject: [Koha-bugs] [Bug 18024] Adding a "genres and kinds" search in
Authorities module
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18024
--- Comment #1 from Alexandre ---
A little correction, according to my partner Marjorie, the exact search term is
"genre / form term".
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 15:24:40 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 14:24:40 +0000
Subject: [Koha-bugs] [Bug 17941] CanBookBeRenewed is very inefficient/slow
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17941
--- Comment #5 from Petter Goks?yr ?sen ---
(In reply to Jonathan Druart from comment #4)
> No do not wait, release as soon as you can :)
OK, for the adventurous:
https://gist.github.com/boutros/c3a63904c788d0a6336023042ce22cde
We recently hired a very experienced Perl guy, so he's behind this, not me:)
But it's very simple actually: just by decorating subroutines with ":
PureFunction" - you get memoization for free!
The actuall PureFunctions module is maybe not for the faint of heart, but it's
very small, and I do belive it works out very beautifull. And as this is not a
persistent cache, but only valid during the scope of the request, we don't have
to deal with cache invalidation and such problems.
The solution can easily be extended to other Getter-methods, as long as they
are idempotent (which they should be).
--
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 Feb 1 15:27:08 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 14:27:08 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
--- Comment #135 from M. Tompsett ---
(In reply to Owen Leonard from comment #134)
> I wonder if we could add a persistent shortcut of some kind to the
> global help system? I suggest this feature be discussed in a separate bug.
Yes, points 7-9 are valid issues, but probably removing summary from this, and
discussing perhaps a simplified basics help page on another bug might be in
order.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 15:29:36 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 14:29:36 +0000
Subject: [Koha-bugs] [Bug 18024] Adding a "genres and kinds" search in
Authorities module
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18024
--- Comment #2 from Alexandre ---
Marc reference
marc http://www.loc.gov/marc/authority/ad155.html
In the Koha of Bywater :
http://catalog.bywatersolutions.com/cgi-bin/koha/opac-authorities-home.pl
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 15:33:50 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 14:33:50 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
--- Comment #136 from M. Tompsett ---
Created attachment 59742
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59742&action=edit
Bug 17855: Follow-up on broken xt test
Apologies for missing that. Here is a patch for point 2 in
comment #133.
TEST PLAN
---------
1) apply all except this patch
2) prove xt
-- a failure
3) apply this patch
4) prove xt
-- no failures
5) prove t
-- only prices fails on master, and is unrelated to this patch.
6) run koha qa test tools
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 15:35:03 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 14:35:03 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
--- Comment #137 from M. Tompsett ---
(In reply to M. Tompsett from comment #136)
> Created attachment 59742 [details] [review]
> Bug 17855: Follow-up on broken xt test
>
> Apologies for missing that. Here is a patch for point 2 in
> comment #133.
Though, this will need to be modified further to address point 1 in comment
#133.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 15:50:59 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 14:50:59 +0000
Subject: [Koha-bugs] [Bug 17941] CanBookBeRenewed is very inefficient/slow
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17941
--- Comment #6 from Jonathan Druart ---
Hum, are you sure this is Plack-compatible?
--
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 Feb 1 16:33:56 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 15:33:56 +0000
Subject: [Koha-bugs] [Bug 18008] Use templates for generating pagination in
intranet
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18008
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|Use templates for |Use templates for
|generating pagination |generating pagination in
| |intranet
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 16:34:54 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 15:34:54 +0000
Subject: [Koha-bugs] [Bug 18029] New: Use templates for generating
pagination in opac
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18029
Bug ID: 18029
Summary: Use templates for generating pagination in opac
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: josef.moravec at gmail.com
QA Contact: testopia at bugs.koha-community.org
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 16:35:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 15:35:21 +0000
Subject: [Koha-bugs] [Bug 18029] Use templates for generating pagination in
opac
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18029
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|oleonard at myacpl.org |josef.moravec at gmail.com
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=18008
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 16:35:21 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 15:35:21 +0000
Subject: [Koha-bugs] [Bug 18008] Use templates for generating pagination in
intranet
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18008
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=18029
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 16:35:59 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 15:35:59 +0000
Subject: [Koha-bugs] [Bug 18030] New: Get rid of pagination_bar sub in
C4::Output
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18030
Bug ID: 18030
Summary: Get rid of pagination_bar sub in C4::Output
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: gmcharlt at gmail.com
Reporter: josef.moravec at gmail.com
QA Contact: testopia at bugs.koha-community.org
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 16:36:25 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 15:36:25 +0000
Subject: [Koha-bugs] [Bug 18008] Use templates for generating pagination in
intranet
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18008
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |18030
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18030
[Bug 18030] Get rid of pagination_bar sub in C4::Output
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 16:36:25 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 15:36:25 +0000
Subject: [Koha-bugs] [Bug 18029] Use templates for generating pagination in
opac
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18029
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |18030
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18030
[Bug 18030] Get rid of pagination_bar sub in C4::Output
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 16:36:25 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 15:36:25 +0000
Subject: [Koha-bugs] [Bug 18030] Get rid of pagination_bar sub in C4::Output
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18030
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |18008, 18029
Assignee|gmcharlt at gmail.com |josef.moravec at gmail.com
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18008
[Bug 18008] Use templates for generating pagination in intranet
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18029
[Bug 18029] Use templates for generating pagination in opac
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 16:37:34 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 15:37:34 +0000
Subject: [Koha-bugs] [Bug 18031] New: Update bootstrap in opac
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18031
Bug ID: 18031
Summary: Update bootstrap in opac
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: josef.moravec at gmail.com
QA Contact: testopia at bugs.koha-community.org
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 16:38:01 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 15:38:01 +0000
Subject: [Koha-bugs] [Bug 18029] Use templates for generating pagination in
opac
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18029
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |18031
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18031
[Bug 18031] Update bootstrap in opac
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 16:38:01 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 15:38:01 +0000
Subject: [Koha-bugs] [Bug 18031] Update bootstrap in opac
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18031
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |18029
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18029
[Bug 18029] Use templates for generating pagination in opac
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 17:36:53 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 16:36:53 +0000
Subject: [Koha-bugs] [Bug 16034] Integration with OverDrive Patron API
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16034
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Passed QA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 17:36:56 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 16:36:56 +0000
Subject: [Koha-bugs] [Bug 16034] Integration with OverDrive Patron API
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16034
Nick Clemens changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59392|0 |1
is obsolete| |
Attachment #59393|0 |1
is obsolete| |
Attachment #59713|0 |1
is obsolete| |
Attachment #59714|0 |1
is obsolete| |
--- Comment #55 from Nick Clemens ---
Created attachment 59743
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59743&action=edit
bug_16034 Koha::ExternalContent::OverDrive - a wrapper around
WebService::ILS::Overdrive::Patron
* Using the upstream module for all the heavy lifting
* opac/external/overdrive/auth.pl - 3-legged authentication handler
Signed-off-by: Jesse Weaver
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 Wed Feb 1 17:37:02 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 16:37:02 +0000
Subject: [Koha-bugs] [Bug 16034] Integration with OverDrive Patron API
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16034
--- Comment #56 from Nick Clemens ---
Created attachment 59744
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59744&action=edit
bug_16034 Adding a new syspref - OverDriveCirculation
If set to true, Opac will integrate OverDrive patron services
Signed-off-by: Jesse Weaver
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 Wed Feb 1 17:37:08 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 16:37:08 +0000
Subject: [Koha-bugs] [Bug 16034] Integration with OverDrive Patron API
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16034
--- Comment #57 from Nick Clemens ---
Created attachment 59745
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59745&action=edit
bug_16034 Add overdrive info to the users page in the public interface
Signed-off-by: Jesse Weaver
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 Wed Feb 1 17:37:14 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 16:37:14 +0000
Subject: [Koha-bugs] [Bug 16034] Integration with OverDrive Patron API
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16034
--- Comment #58 from Nick Clemens ---
Created attachment 59746
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59746&action=edit
Bug 16034 follow-up: added WebService::ILS to PerlDependencies
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 Wed Feb 1 17:38:28 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 16:38:28 +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
alfre69 changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |alfre at ibdinternet.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 18:39:37 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 17:39:37 +0000
Subject: [Koha-bugs] [Bug 17941] CanBookBeRenewed is very inefficient/slow
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17941
--- Comment #7 from Petter Goks?yr ?sen ---
(In reply to Jonathan Druart from comment #6)
> Hum, are you sure this is Plack-compatible?
It should be. I've been testing under Plack today without experiencing any
issues, and it certainly seems to work fine. But I haven't profiled for memory
leaks yet.
--
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 Feb 1 19:34:35 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 18:34:35 +0000
Subject: [Koha-bugs] [Bug 18032] New: make test fail
'DBIx::Class::Storage::DBI'
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18032
Bug ID: 18032
Summary: make test fail 'DBIx::Class::Storage::DBI'
Change sponsored?: ---
Product: Koha
Version: 16.05
Hardware: PC
OS: Linux
Status: NEW
Severity: major
Priority: P5 - low
Component: bugs.koha-community.org
Assignee: koha-bugs at lists.koha-community.org
Reporter: jane.chang at cwts.edu
QA Contact: testopia at bugs.koha-community.org
root at debian-w01:/home/kohasysusr/Downloads/koha-16.05.09# make test
PERL_DL_NONLAZY=1 KOHA_CONF=blib/KOHA_CONF_DIR/koha-conf.xml "/usr/bin/perl"
"-MExtUtils::Command::MM" "-MTest::Harness" "-e" "undef
*Test::Harness::Switches; test_harness(0, 'blib/PERL_MODULE_DIR', 'blib/arch')"
t/*.t
t/00-checkdatabase-version.t ........ ok
t/00-deprecated.t ................... ok
t/00-load.t ......................... 1/? DBIx::Class::ResultSet::find(): Your
storage class (DBIx::Class::Storage::DBI) does not set sql_limit_dialect and
you have not supplied an explicit limit_dialect in your connection_info. DBIC
will attempt to use the GenericSubQ dialect, which works on most databases but
can be (and often is) painfully slow. Please file an RT ticket against
'DBIx::Class::Storage::DBI' . at
/home/kohasysusr/Downloads/koha-16.05.09/blib/PERL_MODULE_DIR/Koha/Objects.pm
line 83
t/00-load.t ......................... 196/?
# Failed test 'use Koha::Token;'
# at t/00-load.t line 64.
# Tried to use 'Koha::Token'.
# Error: Can't locate WWW/CSRF.pm in @INC (@INC contains:
/home/kohasysusr/Downloads/koha-16.05.09/blib/PERL_MODULE_DIR
/home/kohasysusr/Downloads/koha-16.05.09/blib/arch /usr/share/koha/lib
/usr/share/koha/lib /etc/perl /usr/local/lib/perl/5.14.2
/usr/local/share/perl/5.14.2 /usr/lib/perl5 /usr/share/perl5 /usr/lib/perl/5.14
/usr/share/perl/5.14 /usr/local/lib/site_perl . /var/lib/koha/plugins
/var/lib/koha/plugins) at
/home/kohasysusr/Downloads/koha-16.05.09/blib/PERL_MODULE_DIR/Koha/Token.pm
line 53.
# BEGIN failed--compilation aborted at
/home/kohasysusr/Downloads/koha-16.05.09/blib/PERL_MODULE_DIR/Koha/Token.pm
line 53.
# Compilation failed in require at t/00-load.t line 64.
# BEGIN failed--compilation aborted at t/00-load.t line 64.
Bailout called. Further testing stopped: ***** PROBLEMS LOADING FILE
'Koha::Token'
FAILED--Further testing stopped: ***** PROBLEMS LOADING FILE 'Koha::Token'
make: *** [test_dynamic] Error 255
--
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 Feb 1 19:34:49 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 18:34:49 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
--- Comment #6 from M. Tompsett ---
Created attachment 59747
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59747&action=edit
Bug 17982: Improve test coverage
This test triggers the three cases that were modified.
prove -v t/db_dependent/Templates.t
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 19:39:39 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 18:39:39 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59747|0 |1
is obsolete| |
--- Comment #7 from M. Tompsett ---
Created attachment 59748
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59748&action=edit
Bug 17982: Improve test coverage
This test triggers the three cases that were modified.
prove -v t/db_dependent/Templates.t
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 19:40:57 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 18:40:57 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59630|0 |1
is obsolete| |
--- Comment #8 from M. Tompsett ---
Created attachment 59749
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59749&action=edit
Bug 17982: Fix the use of uniq in sub themelanguage
Doing uniq( \@themes ) is useless. It will just return to you the only
reference you gave it.
List::MoreUtils::uniq requires a list instead of an arrayref.
So it is a trivial fix that makes sub themelanguage return one theme instead
of three themes like [ 'prog', 'prog', 'prog' ].
Note that Template->new inserts one or two include paths to TT for each of
these three identical themes.
Note: Removed a overcomplete unit test that basically tested uniq with
uniq in order to remove a dependency for a more important bug report.
Test plan:
[1] Run t/db_dependent/Auth.t (triggering themelanguage)
[2] Open a page on OPAC or intranet. (Did you restart Plack?)
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 Feb 1 19:48:53 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 18:48:53 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59748|0 |1
is obsolete| |
--- Comment #9 from M. Tompsett ---
Created attachment 59750
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59750&action=edit
Bug 17982: Improve test coverage
This test triggers the three cases that were modified.
prove -v t/db_dependent/Templates.t
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 20:45:20 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 19:45:20 +0000
Subject: [Koha-bugs] [Bug 18033] New: If/ else sometimes does not make sense
after koha account system refactoring
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18033
Bug ID: 18033
Summary: If/else sometimes does not make sense after koha
account system refactoring
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: gmcharlt at gmail.com
Reporter: josef.moravec at gmail.com
QA Contact: testopia at bugs.koha-community.org
Patch folowing
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 20:46:10 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 19:46:10 +0000
Subject: [Koha-bugs] [Bug 18033] If/ else sometimes does not make sense
after koha account system refactoring
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18033
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |15894
Assignee|gmcharlt at gmail.com |josef.moravec at gmail.com
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15894
[Bug 15894] Unify all account fine payment functions
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 20:46:10 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 19:46:10 +0000
Subject: [Koha-bugs] [Bug 15894] Unify all account fine payment functions
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15894
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |18033
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18033
[Bug 18033] If/else sometimes does not make sense after koha account system
refactoring
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 20:51:52 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 19:51:52 +0000
Subject: [Koha-bugs] [Bug 18028] install_misc directory is outdated and must
be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18028
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59730|0 |1
is obsolete| |
--- Comment #2 from M. Tompsett ---
Created attachment 59751
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59751&action=edit
Bug 18028: Remove outdated install_misc directory
This directory is no longer maintained and contain outdated information
on how to install Koha.
On the Internet there are tutos using them and that can create confusion
to new user (yes it could be confusing as well to execute a file that
does no longer exist).
NOTE: last tweak was mid 2016, but before that 2015. So I agree with
the deletion.
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 Feb 1 20:51:49 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 19:51:49 +0000
Subject: [Koha-bugs] [Bug 18028] install_misc directory is outdated and must
be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18028
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 Feb 1 20:52:18 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 19:52:18 +0000
Subject: [Koha-bugs] [Bug 18033] If/ else sometimes does not make sense
after koha account system refactoring
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18033
--- Comment #1 from Josef Moravec ---
Created attachment 59752
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59752&action=edit
Bug 18033: Remove duplicate code in paycollect.pl
Test plan:
0) apply the patch
1) try to pay individual fee, with full amount and partial amount
it should work the same as before patch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 20:52:55 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 19:52:55 +0000
Subject: [Koha-bugs] [Bug 18033] If/ else sometimes does not make sense
after koha account system refactoring
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18033
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |kyle at bywatersolutions.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 20:53:26 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 19:53:26 +0000
Subject: [Koha-bugs] [Bug 18033] If/ else sometimes does not make sense
after koha account system refactoring
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18033
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 Wed Feb 1 21:08:44 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:08:44 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
Alex Buckley changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |In Discussion
--- Comment #22 from Alex Buckley ---
(in reply to Jonathan Druart in comment 21)
1/ What tests specifically do you want?
3/ I tried '->is_used' but it threw a strict refs error (as this is my first
time writing perl subroutines I tried a different subroutine name
(get_items_and_biblioitems) which worked successfully.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:14:44 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:14:44 +0000
Subject: [Koha-bugs] [Bug 17802] Hea - Add geolocation information
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17802
Nate changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |nate at bywatersolutions.com
--- Comment #1 from Nate ---
I think this is a great idea! The only thing we need to be sure of is that the
interface to add this location information is user friendly enough so that the
librarians and/or support companies can easily add that info in. Great
thinking!
>From a technical standpoint, I cannot comment on the feasibility of this so
hopefully someone else can!
-Nate
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:19:50 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:19:50 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
--- Comment #23 from Alex Buckley ---
(in reply to Jonathan Druart in comment 21)
1/ Presumably you mean test plan rather than the output of a qa test? If so the
test plan is slightly modified from comment 5)
Test plan:
1. In the staff interface, stage and manage MARC records for import
2. Try to delete an itemtype. If there are items of that itemtype in the
database then a message telling you the number of items of that
itemtype there are will be displayed.
3. Record that number
4. View the admin/itemtpes.pl script and confirm that there is sql code
written in this file.
5. Apply this patch
6. View the admin/itemtypes.pl script and observe that there is no sql
in this file. Confirm there is a module call
my $overalltotal = Koha::ItemType->get_items_and_biblioitems($itemtype_code)
7. Go to Koha/ItemType.pm and confirm that there is a function named
get_items_and_biblioitems
8. In the staff interface try to delete the same itemtype
9. Record the number of items there are with that itemtype in the
resulting message
10. The numbers recorded in steps 3 and 9 should match showing that the
DBIx code in the get_items_and_biblioitems subroutine is working as intended
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:23:14 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:23:14 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
--- Comment #10 from Marcel de Rooy ---
Mark,
Thx for signoff but please explain what the second test has to do with the
first patch? You are testing language, but the change is about not duplicating
themes by proper use of uniq.
Marcel
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:24:42 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:24:42 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
Alex Buckley changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |In Discussion
--- Comment #138 from Alex Buckley ---
(in reply to Jonathan Druart and Owen Leonard in comment 133 and 134)
Just to clarify a few of your points:
Re: point 5: "5/ You copied some existing javascript functions to
onboardingstep2.tt. Do not
duplicate existing code."
Do you want me to remove the JavaScript functions and or will fixing the
indentation be sufficient to pass QA?
Re: point 9: What is the final decision on summary info. To get this signed off
and through QA are both of you happy for me to remove summary.pl and
summary.tt. And they can be added in a separate bug?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:25:34 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:25:34 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
--- Comment #11 from Marcel de Rooy ---
Apart from that, it fails too:
# Subtest: Test C4::Templates::themelanguage
1..4
not ok 1 - Expected language: non-en
# Failed test 'Expected language: non-en'
# at t/db_dependent/Templates.t line 68.
# got: 'en'
# expected: 'fr-FR'
ok 2 - Expected language: default en
ok 3 - Expected language: unset
ok 4 - Expected language: not coded for
# Looks like you failed 1 test of 4.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:26:05 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:26:05 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
--- Comment #12 from Marcel de Rooy ---
Do not assume that fr-FR exists. I dont have it :)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:27:18 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:27:18 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
--- Comment #13 from Marcel de Rooy ---
Mock::FileSystem ?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:31:54 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:31:54 +0000
Subject: [Koha-bugs] [Bug 17855] Updated Koha web installer and new
onboarding tool feature to guide users through setting up Koha
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855
--- Comment #139 from Alex Buckley ---
(in reply to Jonathan Druart and M.Tompsett in comment 133, 136 and 137 and
17855)
As I am still learning Koha can I please have a more specific explanation of
the issue(s) which you addressed in points 1 and 2?
And possible solutions thereof
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:37:23 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:37:23 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
M. Tompsett changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mtompset at hotmail.com
--- Comment #14 from M. Tompsett ---
(In reply to Marcel de Rooy from comment #10)
> Thx for signoff but please explain what the second test has to do with the
> first patch? You are testing language, but the change is about not
> duplicating themes by proper use of uniq.
You changed 3 lines of code, this test triggers those three lines of code.
However, you rightly pointed out fr-FR might not exist. Shoot. :(
If it does exist, it triggers the three cases you changed.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:38:53 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:38:53 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
--- Comment #15 from M. Tompsett ---
(In reply to Marcel de Rooy from comment #13)
> Mock::FileSystem ?
That looks promising. I'll try something in about 2 hours.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:41:37 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:41:37 +0000
Subject: [Koha-bugs] [Bug 17982] Fix the use of uniq in sub themelanguage
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |josef.moravec at gmail.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:47:53 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:47:53 +0000
Subject: [Koha-bugs] [Bug 17934] Various issues with auth_values_input_www.t
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17934
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #59173|0 |1
is obsolete| |
--- Comment #3 from Josef Moravec ---
Created attachment 59753
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59753&action=edit
[SIGNED-OFF] Bug 17934: Make t/db_dependent/www/auth_values_input_www.t awesome
There were two lines of code which declared unused variables.
They were removed. Next, the issue of double planning encountered
on bug 17540
(https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17540#c3)
was encountered. So this corrects that and adds a little more
error handling into the whole C4::Context using process so that
perlcritic -4 is achieved.
Also typo on Korean in a comment.
TEST PLAN
---------
2) follow these commands:
$ mv C4/Context.pm C4/Context_backup.pm
$ sudo koha-shell -c "prove t/db_dependent/www/auth_values_input_www.t" kohadev
-- Nasty fail
$ mv C4/Context_backup.pm C4/Context.pm
$ sudo koha-shell -c "export PERL5LIB=\"\";prove
t/db_dependent/www/auth_values_input_www.t" kohadev
-- Nasty fail
$ sudo koha-shell -c "export KOHA_CONF=\"\";prove
t/db_dependent/www/auth_values_input_www.t" kohadev
-- Nasty fail
$ sudo koha-shell -c "export KOHA_INTRANET_URL=\"\";prove
t/db_dependent/www/auth_values_input_www.t" kohadev
$ sudo koha-shell -c "prove t/db_dependent/www/auth_values_input_www.t" kohadev
-- Nasty fail
$ sudo koha-shell -c "export KOHA_INTRANET_URL=\"http://localhost:8080/\";prove
t/db_dependent/www/auth_values_input_www.t" kohadev
-- This should work.
$ perlcritic -4 t/db_dependent/www/auth_values_input_www.t
-- This should be OK
2) apply patch
3) follow these commands:
$ mv C4/Context.pm C4/Context_backup.pm
$ sudo koha-shell -c "prove t/db_dependent/www/auth_values_input_www.t" kohadev
-- This should trigger the C4::Context not working message
$ mv C4/Context_backup.pm C4/Context.pm
$ sudo koha-shell -c "export PERL5LIB=\"\";prove
t/db_dependent/www/auth_values_input_www.t" kohadev
-- This should trigger the C4::Contxt not loading message
$ sudo koha-shell -c "export KOHA_CONF=\"\";prove
t/db_dependent/www/auth_values_input_www.t" kohadev
-- This should trigger the KOHA_CONF not set message
$ sudo koha-shell -c "export KOHA_INTRANET_URL=\"\";prove
t/db_dependent/www/auth_values_input_www.t" kohadev
$ sudo koha-shell -c "prove t/db_dependent/www/auth_values_input_www.t" kohadev
-- These should both trigger the KOHA_INTRANET_URL not set message
$ sudo koha-shell -c "export KOHA_INTRANET_URL=\"http://localhost:8080/\";prove
t/db_dependent/www/auth_values_input_www.t" kohadev
-- This should work.
$ perlcritic -4 t/db_dependent/www/auth_values_input_www.t
-- This should be OK
3) run koha qa test tools
Signed-off-by: Josef Moravec
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 21:47:50 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 20:47:50 +0000
Subject: [Koha-bugs] [Bug 17934] Various issues with auth_values_input_www.t
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17934
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 22:02:43 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 21:02:43 +0000
Subject: [Koha-bugs] [Bug 15529] Process Message Queue may send duplicate
emails if process is launched twice
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15529
Barton Chittenden changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|enhancement |normal
--- Comment #2 from Barton Chittenden ---
I believe that this may also cause fragmenting of CHECKOUT, RENEWAL and CHECKIN
messages. These are digest messages, which should accumulate detail about their
respective circ action in message_queue.metadata until process_message_queue.pl
runs. We have seen evidence of multiple RENEWAL messages going out in very
short order... it would be normal for process_message_queue.pl to be running
while items are being checked in our out, causing two messages to be sent, but
we have had evidence of 3 messages sent at the same time, which isn't possible
unless the message_queue is being processed by two separate instances of
process_message_queue.pl
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 22:03:10 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 21:03:10 +0000
Subject: [Koha-bugs] [Bug 18026] URL to database columns link in system
preferences is incorrect
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18026
Josef Moravec changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |josef.moravec at gmail.com
--- Comment #1 from Josef Moravec ---
References to schema website are in these 3 files:
koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/acquisitions.pref
koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref
koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/patrons.pref
--
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 Feb 1 22:07:22 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 21:07:22 +0000
Subject: [Koha-bugs] [Bug 17944] Remove the sql code from itemtypes.pl
administrative perl script
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17944
--- Comment #24 from Alex Buckley ---
(in reply to Jonathan Druart in comment 21)
Please disregard comment 22, I have fixed the strict refs error issue.
In the commit I am currently working on I have addressed your following points
2, and 3.
Could you please explain what you mean in your point 5? Are you asking why I
havent put the is_used subroutine in the Koha::Items and Koha::Biblioitems
files?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Wed Feb 1 22:08:49 2017
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Wed, 01 Feb 2017 21:08:49 +0000
Subject: [Koha-bugs] [Bug 18034] New: Pseudonym records not found in a
research by author
Message-ID: