From bugzilla-daemon at bugs.koha-community.org Sun May 1 00:17:08 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 30 Apr 2016 22:17:08 +0000
Subject: [Koha-bugs] [Bug 14695] Add ability to place multiple item holds on
a given record per patron
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14695
--- Comment #131 from Benjamin Daeuber ---
Quick feedback on a few quirks. I don't know if any of these would prevent
closing this and pushing it to master, but nonetheless:
In Circ Client
1. It appears staff can place item level holds when this feature is on,
regardless of whether item level holds are allowed. We tested on Wednesday, and
it appears Nicole has already identified this.
2. The holds to place count (up to three) is set to 3. Select one barcode,
set ?holds to place? dropdown to 3, and it only does one hold on the item. This
seems to be unexpected, but we don?t really know why that dropdown box is there
or what it is for.
3. Put three items on hold on one record. Once you check in one of the
three items you placed on hold, and it is now ?waiting? for the patron, you can
place the fourth hold. You don?t need to check it out to the patron before
being able to place the fourth hold. Is this expected behavior?
In the OPAC
1. Related to point 1 above, this does not appear to affect the OPAC.
Users are blocked from placing item level holds if they are not allowed.
In both
1. Item level holds seem to be disregarded by the hold limit. So if you
have a limit of 10, with 9 items on hold, you can still place 2 item level
holds. We haven?t tested this in more detail.
2. A patron can?t place a bib level and item level hold for the same item.
I?m actually struggling to think what scenario this might be used in, but we?re
nothing if not thorough.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun May 1 00:26:11 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 30 Apr 2016 22:26:11 +0000
Subject: [Koha-bugs] [Bug 16330] REST API: add routes to add,
update and delete patrons
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16330
Benjamin Rokseth changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #50889|0 |1
is obsolete| |
--- Comment #7 from Benjamin Rokseth ---
Created attachment 51066
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51066&action=edit
Bug 16330 - REST API: add routes to add, update and delete patrons
This patch adds support for add, edit and delete patrons via REST API.
GET /api/v1/patrons Get patron list from params
GET /api/v1/patrons/ Get single patron
POST /api/v1/patrons Create a new patron
PUT /api/v1/patrons/ Update data about patron
DEL /api/v1/patrons/ Delete a patron
Test plan:
1) Apply this patch (on top of dependent bug 15126)
2) Run tests perl t/db_dependent/api/v1/patrons.t
3) Add a user with proper rights to use the REST API
4) play with your favourite REST client (curl/httpie, etc.):
Authenticate with the user created above and get a CGISESSION id.
Use the CGISESSION to add, edit and delete patrons via the API.
Please note there is no validation of body input in PUT/POST other
than branchcode,category,userid,cardnumber.
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Sun May 1 00:31:33 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sat, 30 Apr 2016 22:31:33 +0000
Subject: [Koha-bugs] [Bug 16330] REST API: add routes to add,
update and delete patrons
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16330
--- Comment #8 from Benjamin Rokseth ---
> GET /api/v1/patrons Get patron list from params
> GET /api/v1/patrons/ Get single patron
Added possibiblity to search for patrons by any patron properties.
e.g.
GET /api/v1/patrons?cardnumber=
GET /api/v1/patrons?branchcode=&categorycode=S
should give an array of matching patrons or empty array if none.
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Sun May 1 11:14:49 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 01 May 2016 09:14:49 +0000
Subject: [Koha-bugs] [Bug 16383] Making Local Use sysprefs actions buttons
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16383
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #50973|0 |1
is obsolete| |
--- Comment #2 from Marc V?ron ---
Created attachment 51067
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51067&action=edit
Bug 16383: Making Local Use sysprefs actions buttons
To test:
1) Go to Admin -> System Preferences -> Local Use tab
2) Confirm buttons show as font awesome buttons and work as expected
3) Confirm buttons do not wrap on a narrower browser
Sponsored-by: Catalyst IT
Signed-off-by: Marc V?ron
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Sun May 1 11:15:12 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Sun, 01 May 2016 09:15:12 +0000
Subject: [Koha-bugs] [Bug 16383] Making Local Use sysprefs actions buttons
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16383
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |veron at veron.ch
Status|Needs Signoff |Signed Off
Patch complexity|--- |Trivial patch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 02:18:17 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 00:18:17 +0000
Subject: [Koha-bugs] [Bug 15303] Letsencrypt option for Debian package
installations
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15303
--- Comment #61 from David Cook ---
Woo! Hurray Mirko!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 02:31:40 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 00:31:40 +0000
Subject: [Koha-bugs] [Bug 11297] Add support for custom PQF attributes for
Z39.50 server searches
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11297
--- Comment #38 from David Cook ---
(In reply to Marcel de Rooy from comment #37)
> I would agree with you that is not necessary to test string concatenation.
I disagree. I think it is important to test string concatenation, as the query
is just a string, and it's possible that concatenation could mess up the query.
> We should be pragmatic in this regard.
> I guess the point here is not if you can concatenate strings but if you add
> this stuff to the query, will we still have results? Can you predict what
> will happen in a test case?
> This will be hard because we probably do not like to query external sources
> in a unit test.
> I once added some stuff in t/db_dependent/Breeding.t in order to start some
> testing of this module.
Spot on!
Alternatives to querying an external source would be parsing the query. We
might be able to use the ZOOM modules for that.
Example:
$q = new ZOOM::Query::PQF('@attr 1=4 dinosaur');
"Creates a new query object, compiling the query passed as its argument
according to the rules of the particular query-type being instantiated. If
compilation fails, an exception is thrown. Otherwise, the query may be passed
to the Connection method search()."
http://search.cpan.org/~mirk/Net-Z3950-ZOOM/lib/ZOOM.pod#ZOOM::Query
BOOM! Surely that should work?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 04:43:36 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 02:43:36 +0000
Subject: [Koha-bugs] [Bug 16409] New: Remove redundant
'koha-tmpl/opac-tmpl/lib/famfamfam' dir
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16409
Bug ID: 16409
Summary: Remove redundant 'koha-tmpl/opac-tmpl/lib/famfamfam'
dir
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: mtj at kohaaloha.com
QA Contact: testopia at bugs.koha-community.org
Remove redundant 'koha-tmpl/opac-tmpl/lib/famfamfam' dir, from Koha repo
patch to come...
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 04:43:47 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 02:43:47 +0000
Subject: [Koha-bugs] [Bug 16409] Remove redundant
'koha-tmpl/opac-tmpl/lib/famfamfam' dir
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16409
Mason James changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|oleonard at myacpl.org |mtj at kohaaloha.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 08:02:47 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:02:47 +0000
Subject: [Koha-bugs] [Bug 16397] Unable to delete audio alerts
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16397
Katrin Fischer 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 Mon May 2 08:02:50 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:02:50 +0000
Subject: [Koha-bugs] [Bug 16397] Unable to delete audio alerts
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16397
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51047|0 |1
is obsolete| |
--- Comment #3 from Katrin Fischer ---
Created attachment 51068
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51068&action=edit
[PASSED QA] Bug 16397 - Unable to delete audio alerts
This patch corrects a JavaScript error which has arisen due to the
jQuery upgrade. This should have been included in Bug 16321.
To test, apply the patch and go to Administration -> Audio alerts.
- In the list of existing audio alerts, check one or more checkboxes.
The "Delete selected" button should become enabled.
- Uncheck all checkboxes and check that the "Delete selected" button is
now disabled.
- Confirm that deletions are completed correctly.
Signed-off-by: Marc V?ron
Signed-off-by: Katrin Fischer
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 08:10:21 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:10:21 +0000
Subject: [Koha-bugs] [Bug 16363] Use floating toolbar on advanced search
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16363
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |16398
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16398
[Bug 16398] Keep expanded view after clearing the search form
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 08:10:21 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:10:21 +0000
Subject: [Koha-bugs] [Bug 16398] Keep expanded view after clearing the
search form
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16398
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |16363
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16363
[Bug 16363] Use floating toolbar on advanced search
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 08:12:02 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:12:02 +0000
Subject: [Koha-bugs] [Bug 16363] Use floating toolbar on advanced search
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16363
--- Comment #13 from Katrin Fischer ---
Confused - I don't see the floating toolbar in master?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 08:13:29 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:13:29 +0000
Subject: [Koha-bugs] [Bug 16363] Use floating toolbar on advanced search
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16363
--- Comment #14 from Katrin Fischer ---
Never mind - a problem with my branch not being as current as I had thought.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 08:16:03 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:16:03 +0000
Subject: [Koha-bugs] [Bug 16398] Keep expanded view after clearing the
search form
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16398
Katrin Fischer 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 Mon May 2 08:16:06 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:16:06 +0000
Subject: [Koha-bugs] [Bug 16398] Keep expanded view after clearing the
search form
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16398
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51048|0 |1
is obsolete| |
--- Comment #3 from Katrin Fischer ---
Created attachment 51069
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51069&action=edit
[PASSED QA] Bug 16398: Keep the expanded view after clearing the search form
On the advanced search form, the "Clear fields" button should pass
expanded_options
param to preserve the view we use.
Test plan:
- Click on more options
- Fill some fields
- Click on "Clear fields"
=> Without this patch you get the "fewer options" view
=> With this patch, you will keep the "more options" view
Signed-off-by: Marc V?ron
Signed-off-by: Katrin Fischer
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 08:21:07 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:21:07 +0000
Subject: [Koha-bugs] [Bug 7143] Bug for tracking changes to the about page
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7143
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Passed QA
--
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 Mon May 2 08:21:13 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:21:13 +0000
Subject: [Koha-bugs] [Bug 7143] Bug for tracking changes to the about page
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7143
Katrin Fischer changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51056|0 |1
is obsolete| |
--- Comment #298 from Katrin Fischer ---
Created attachment 51070
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51070&action=edit
[PASSED QA] Bug 7143: Moving about to 16.05. Hurray.
Any suggestions to highlight the version number more prominently or
elegantly are welcome. Just add a follow-up.
Test plan:
Go to About. Check Koha team. Hover over the version number.
Signed-off-by: Bernardo Gonzalez Kriegel
16.05, Yeah!
No errors
Signed-off-by: Katrin Fischer
--
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 Mon May 2 08:22:30 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:22:30 +0000
Subject: [Koha-bugs] [Bug 7143] Bug for tracking changes to the about page
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7143
--- Comment #299 from Katrin Fischer ---
Hm, history.txt ends in December last year:
December 26 2015 Koha 3.20.7.1 released releases
New bug or fix that here?
--
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 Mon May 2 08:58:03 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:58:03 +0000
Subject: [Koha-bugs] [Bug 16410] New: koha-remove should optionally remove
an instance's database and db-user
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16410
Bug ID: 16410
Summary: koha-remove should optionally remove an instance's
database and db-user
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Command-line Utilities
Assignee: gmcharlt at gmail.com
Reporter: mtj at kohaaloha.com
QA Contact: testopia at bugs.koha-community.org
CC: robin at catalyst.net.nz
It would be useful if koha-remove optionally removed an instance's database and
db-user
more info to come...
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 08:58:13 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:58:13 +0000
Subject: [Koha-bugs] [Bug 16410] koha-remove should optionally remove an
instance's database and db-user
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16410
Mason James changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|gmcharlt at gmail.com |mtj at kohaaloha.com
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 08:59:21 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:59:21 +0000
Subject: [Koha-bugs] [Bug 6416] koha-remove fails when dropping user
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6416
Mason James changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=16410
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 08:59:21 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 06:59:21 +0000
Subject: [Koha-bugs] [Bug 16410] koha-remove should optionally remove an
instance's database and db-user
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16410
Mason James changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=6416
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 09:00:13 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 07:00:13 +0000
Subject: [Koha-bugs] [Bug 7143] Bug for tracking changes to the about page
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7143
--- Comment #300 from Marcel de Rooy ---
(In reply to Katrin Fischer from comment #299)
> Hm, history.txt ends in December last year:
> December 26 2015 Koha 3.20.7.1 released releases
> New bug or fix that here?
Chris regularly updated that one until now.
--
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 Mon May 2 09:01:34 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 07:01:34 +0000
Subject: [Koha-bugs] [Bug 16405] Fix Circulation/NoIssuesChargeGuarantees.t
test 2
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16405
--- Comment #3 from Marcel de Rooy ---
(In reply to Bernardo Gonzalez Kriegel from comment #2)
> Signed-off-by: Bernardo Gonzalez Kriegel
Thx for your signoffs, Bernardo.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 10:11:34 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 08:11:34 +0000
Subject: [Koha-bugs] [Bug 16411] New: Make Hold.t not depend on two existing
branches
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16411
Bug ID: 16411
Summary: Make Hold.t not depend on two existing branches
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: Test Suite
Assignee: chris at bigballofwax.co.nz
Reporter: m.de.rooy at rijksmuseum.nl
QA Contact: gmcharlt at gmail.com
If you do not have two branches, this test will fail.
Can't call method "branchcode" on an undefined value at t/db_dependent/Hold.t
line 60.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 10:13:26 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 08:13:26 +0000
Subject: [Koha-bugs] [Bug 16411] Make Hold.t not depend on two existing
branches
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16411
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |Needs Signoff
Patch complexity|--- |Trivial patch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 10:13:29 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 08:13:29 +0000
Subject: [Koha-bugs] [Bug 16411] Make Hold.t not depend on two existing
branches
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16411
--- Comment #1 from Marcel de Rooy ---
Created attachment 51071
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51071&action=edit
Bug 16411: Make Hold.t not depend on two existing branches
If you do not have two branches, this test will fail.
Can't call method "branchcode" on an undefined value.
This patch adds a borrower and two branches with TestBuilder.
Test plan:
Run the test.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 10:13:42 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 08:13:42 +0000
Subject: [Koha-bugs] [Bug 16411] Make Hold.t not depend on two existing
branches
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16411
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|chris at bigballofwax.co.nz |m.de.rooy at rijksmuseum.nl
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:04:45 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:04:45 +0000
Subject: [Koha-bugs] [Bug 11088] Patron entry page should use floating
toolbar like cataloging interface
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11088
--- Comment #24 from Jonathan Druart ---
(In reply to Owen Leonard from comment #23)
> (In reply to Jonathan Druart from comment #22)
> > When saving a patron, it will be at the bottom right.
> > Should not we keep a consistency?
>
> Aleisha and I both tried to implement the floating toolbar for this bug in
> the same way that it is implemented elsewhere, but the hidden form fields in
> the search header make it difficult to implement the fixfloat()
> functionality correctly (See Comment 2).
I think I have managed to fix the issue on bug 16400, please have a look.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:06:10 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:06:10 +0000
Subject: [Koha-bugs] [Bug 16167] Remove prefs to drive authorised value
images
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16167
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |16408
See Also|https://bugs.koha-community |
|.org/bugzilla3/show_bug.cgi |
|?id=16408 |
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16408
[Bug 16408] Fix UsageStats.t
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:06:10 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:06:10 +0000
Subject: [Koha-bugs] [Bug 16408] Fix UsageStats.t
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16408
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
Depends on| |16167
See Also|https://bugs.koha-community |
|.org/bugzilla3/show_bug.cgi |
|?id=16167 |
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16167
[Bug 16167] Remove prefs to drive authorised value images
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:07:04 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:07:04 +0000
Subject: [Koha-bugs] [Bug 16408] Fix UsageStats.t
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16408
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51059|0 |1
is obsolete| |
--- Comment #3 from Jonathan Druart ---
Created attachment 51072
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51072&action=edit
Bug 16408: Fix UsageStats.t
Bug 16167 removed some prefs. This impacts the number of tests.
Test plan:
Run the test.
Signed-off-by: Bernardo Gonzalez Kriegel
Test pass
No errors
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 Mon May 2 11:07:02 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:07:02 +0000
Subject: [Koha-bugs] [Bug 16408] Fix UsageStats.t
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16408
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 Mon May 2 11:31:53 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:31:53 +0000
Subject: [Koha-bugs] [Bug 16403] Fix Holds.t (tests 9 and 39)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16403
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 Mon May 2 11:31:56 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:31:56 +0000
Subject: [Koha-bugs] [Bug 16403] Fix Holds.t (tests 9 and 39)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16403
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51054|0 |1
is obsolete| |
--- Comment #4 from Jonathan Druart ---
Created attachment 51073
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51073&action=edit
Bug 16403: Fix holds.t (tests 9 and 39)
Resolves this:
Failed test 'GetReservesFromItemnumber should return a valid borrowernumber'
at t/db_dependent/Holds.t line 97.
got: '2000001890'
expected: '2000001889'
Failed test 'Test AlterPriority(), move to bottom'
at t/db_dependent/Holds.t line 220.
got: '4'
expected: '5'
Test plan:
Run the test before and after applying this patch.
Signed-off-by: Marc V?ron
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 Mon May 2 11:32:00 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:32:00 +0000
Subject: [Koha-bugs] [Bug 16403] Fix Holds.t (tests 9 and 39)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16403
--- Comment #5 from Jonathan Druart ---
Created attachment 51074
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51074&action=edit
Bug 16403: Remove unnecessary undef param
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 Mon May 2 11:34:15 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:34:15 +0000
Subject: [Koha-bugs] [Bug 15534] Add the ability to prevent a patron from
placing a hold on a record with available items
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15534
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |16404
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16404
[Bug 16404] Fix Circulation/Branch.t (tests 8-11)
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:34:15 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:34:15 +0000
Subject: [Koha-bugs] [Bug 16404] Fix Circulation/Branch.t (tests 8-11)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16404
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |15534
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15534
[Bug 15534] Add the ability to prevent a patron from placing a hold on a record
with available items
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:36:07 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:36:07 +0000
Subject: [Koha-bugs] [Bug 16404] Fix Circulation/Branch.t (tests 8-11)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16404
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51052|0 |1
is obsolete| |
--- Comment #3 from Jonathan Druart ---
Created attachment 51075
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51075&action=edit
Bug 16404: Fix Circulation/Branch.t (tests 8-11)
Resolves:
Failed test 'GetBranchitem returns holdallowed and return branch'
at t/db_dependent/Circulation/Branch.t line 226.
Structures begin differing at:
$got->{hold_fulfillment_policy} = 'any'
$expected->{hold_fulfillment_policy} = Does not exist
Failed test 'Without parameters GetBranchItemRule returns the values in
default_circ_rules'
at t/db_dependent/Circulation/Branch.t line 234.
Structures begin differing at:
$got->{hold_fulfillment_policy} = 'any'
$expected->{hold_fulfillment_policy} = Does not exist
Failed test 'With only a branchcode GetBranchItemRule returns values in
default_branch_circ_rules'
at t/db_dependent/Circulation/Branch.t line 239.
Structures begin differing at:
$got->{hold_fulfillment_policy} = 'any'
$expected->{hold_fulfillment_policy} = Does not exist
Failed test 'With only one parametern GetBranchItemRule returns default values'
at t/db_dependent/Circulation/Branch.t line 244.
Structures begin differing at:
$got->{hold_fulfillment_policy} = 'any'
$expected->{hold_fulfillment_policy} = Does not exist
Test plan:
Run test before and after applying this patch.
Signed-off-by: Marc V?ron
Signed-off-by: Jonathan Druart
Note: I do not see the usefulness of the lazy_any variable, I don't
think it make things easier to read.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:36:03 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:36:03 +0000
Subject: [Koha-bugs] [Bug 16404] Fix Circulation/Branch.t (tests 8-11)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16404
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 Mon May 2 11:38:57 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:38:57 +0000
Subject: [Koha-bugs] [Bug 16405] Fix Circulation/NoIssuesChargeGuarantees.t
test 2
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16405
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |14577
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14577
[Bug 14577] Allow restriction of checkouts based on fines of
guarantor/guarantee
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:38:57 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:38:57 +0000
Subject: [Koha-bugs] [Bug 14577] Allow restriction of checkouts based on
fines of guarantor/guarantee
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14577
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |16405
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16405
[Bug 16405] Fix Circulation/NoIssuesChargeGuarantees.t test 2
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:41:20 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:41:20 +0000
Subject: [Koha-bugs] [Bug 16405] Fix Circulation/NoIssuesChargeGuarantees.t
test 2
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16405
--- Comment #4 from Jonathan Druart ---
Created attachment 51076
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51076&action=edit
[ALTERNATIVE-PATCH] Bug 16405: Fix Circulation/NoIssuesChargeGuarantees.t test
2
Resolves:
Failed test 'Patron cannot check out item due to debt for guarantee'
at t/db_dependent/Circulation/NoIssuesChargeGuarantees.t line 63.
got: '10'
expected: '10.00'
Test plan:
Run the test again.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:41:57 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:41:57 +0000
Subject: [Koha-bugs] [Bug 16405] Fix Circulation/NoIssuesChargeGuarantees.t
test 2
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16405
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Needs Signoff
--- Comment #5 from Jonathan Druart ---
Marcel, I think it's better to keep `is`, could you have a look at my
alternative patch please?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:47:45 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:47:45 +0000
Subject: [Koha-bugs] [Bug 16407] Fix Koha_borrower_modifications.t
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16407
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 Mon May 2 11:47:48 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:47:48 +0000
Subject: [Koha-bugs] [Bug 16407] Fix Koha_borrower_modifications.t
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16407
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51060|0 |1
is obsolete| |
--- Comment #3 from Jonathan Druart ---
Created attachment 51077
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51077&action=edit
Bug 16407: Fix Koha_borrower_modifications.t
This test was using hardcoded borrower number, assuming they should be
present. Now we use TestBuilder.
Test plan:
Run the test.
Signed-off-by: Bernardo Gonzalez Kriegel
Test pass before and after patch.
No errors
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 Mon May 2 11:48:10 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:48:10 +0000
Subject: [Koha-bugs] [Bug 16407] Fix Koha_borrower_modifications.t
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16407
--- Comment #4 from Jonathan Druart ---
Created attachment 51078
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51078&action=edit
Bug 16407: Simplify comments
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 Mon May 2 11:56:07 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:56:07 +0000
Subject: [Koha-bugs] [Bug 16104] Warnings "used only once: possible typo"
should be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16104
--- Comment #11 from Jonathan Druart ---
(In reply to Jonathan Druart from comment #10)
> @RM please revert this patch
ping!
Above all, it fails 2 tests in t/db_dependent/Context.t
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:58:38 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:58:38 +0000
Subject: [Koha-bugs] [Bug 16406] Fix Context.t tests 27, 37 and sysprefs.t
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16406
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|normal |minor
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
--- Comment #3 from Jonathan Druart ---
It's a specific case, maybe we should skip the caching tests if no cache is
enabled.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 11:59:57 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 09:59:57 +0000
Subject: [Koha-bugs] [Bug 16384] When canceling 'edit basket',
return to basket summary if you came from there
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16384
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|koha-bugs at lists.koha-commun |katrin.fischer at bsz-bw.de
|ity.org |
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:08:47 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:08:47 +0000
Subject: [Koha-bugs] [Bug 16104] Warnings "used only once: possible typo"
should be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16104
--- Comment #12 from Jonathan Druart ---
(In reply to Jonathan Druart from comment #11)
> (In reply to Jonathan Druart from comment #10)
> > @RM please revert this patch
>
> ping!
>
> Above all, it fails 2 tests in t/db_dependent/Context.t
And t/db_dependent/sysprefs.t
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:10:45 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:10:45 +0000
Subject: [Koha-bugs] [Bug 16104] Warnings "used only once: possible typo"
should be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16104
--- Comment #13 from Julian Maurice ---
Patch reverted in 3.22.x
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:11:11 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:11:11 +0000
Subject: [Koha-bugs] [Bug 16405] Fix Circulation/NoIssuesChargeGuarantees.t
test 2
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16405
Marcel de Rooy 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 Mon May 2 12:11:13 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:11:13 +0000
Subject: [Koha-bugs] [Bug 16405] Fix Circulation/NoIssuesChargeGuarantees.t
test 2
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16405
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51061|0 |1
is obsolete| |
Attachment #51076|0 |1
is obsolete| |
--- Comment #6 from Marcel de Rooy ---
Created attachment 51079
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51079&action=edit
Bug 16405: Fix Circulation/NoIssuesChargeGuarantees.t test 2
Resolves:
Failed test 'Patron cannot check out item due to debt for guarantee'
at t/db_dependent/Circulation/NoIssuesChargeGuarantees.t line 63.
got: '10'
expected: '10.00'
Test plan:
Run the test again.
Signed-off-by: Marcel de Rooy
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:11:27 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:11:27 +0000
Subject: [Koha-bugs] [Bug 13321] Tax and prices calculation need to be fixed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13321
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #48867|0 |1
is obsolete| |
--- Comment #48 from Julian Maurice ---
Created attachment 51080
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51080&action=edit
Bug 13321: Rename variables
This patch renames the variable according to the new DB column names
* gste => tax_excluded
* gsti => tax_included
* gstrate => tax_rate
* gstvalue => tax_value
This patch also modify the ModReceiveOrder subroutine:
* Edit vendor note on receiving is not possible, so the code should not
permit that.
* Update ModReceiveOrder to pass a hashref
And that's all!
git grep on gste, gsti, gstrate and gstvalue should not return any code
that can be executed.
Signed-off-by: Laurence Rault
Signed-off-by: Francois Charbonnier
Signed-off-by: Sonia Bouis
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:11:31 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:11:31 +0000
Subject: [Koha-bugs] [Bug 13321] Tax and prices calculation need to be fixed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13321
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #48868|0 |1
is obsolete| |
--- Comment #49 from Julian Maurice ---
Created attachment 51081
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51081&action=edit
Bug 13321: use the populate_with_prices where needed
Now we need to calculate the prices (ecost, rrp, unitprice) and tax
(tax_rate, tax_value) when the price is set for an order.
This only appends in the 3 files impacted by this patch.
addorder*.pl on ordering
finishreceive.pl on receiving
Signed-off-by: Laurence Rault
Signed-off-by: Francois Charbonnier
Signed-off-by: Sonia Bouis
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:11:35 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:11:35 +0000
Subject: [Koha-bugs] [Bug 13321] Tax and prices calculation need to be fixed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13321
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #48869|0 |1
is obsolete| |
--- Comment #50 from Julian Maurice ---
Created attachment 51082
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51082&action=edit
Bug 13321: Fix the prices calculation method
Well, we have finally arrived here \o/
The method where the prices are calculated uses the equations listed on
the wiki page (http://wiki.koha-community.org/wiki/GST_Rewrite_RFC).
The ecost is calculated from the rrp (using the discount and the tax
rate). That's why we removed the ability to edit this value.
That's why we remove the ability to edit the ecost on ordering in a
previous commit (bug 12840).
The total is now calculated in the scripts. That's why this patch
removes lines in the test file.
In C4::Acquisition::populate_order_with_prices, the calculation on
receiving must depend on the 'invoiceincgst' supplier parameter, and not
listincgst (which is used on ordering).
It also removes the rounding errors, now we store "exact" values in DB
(10^-6).
The values will be displayed using the Price TT plugin it will round the
values for us.
Signed-off-by: Laurence Rault
Signed-off-by: Francois Charbonnier
Signed-off-by: Sonia Bouis
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:11:39 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:11:39 +0000
Subject: [Koha-bugs] [Bug 13321] Tax and prices calculation need to be fixed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13321
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #48870|0 |1
is obsolete| |
--- Comment #51 from Julian Maurice ---
Created attachment 51083
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51083&action=edit
Bug 13321: Tax and prices calculation - DB Changes
This patch adds 7 columns to the aqorders table:
* unitprice_tax_excluded
* unitprice_tax_included
* rrp_tax_excluded
* rrp_tax_included
* ecost_tax_excluded
* ecost_tax_included
* tax_value
It also renames:
* aqorders.gstrate with aqorders.tax_rate
* aqbooksellers.gstrate with aqbooksellers.tax_rate
The new columns are filled with the previous calculation method.
Signed-off-by: Laurence Rault
Signed-off-by: Francois Charbonnier
Signed-off-by: Sonia Bouis
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:11:43 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:11:43 +0000
Subject: [Koha-bugs] [Bug 13321] Tax and prices calculation need to be fixed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13321
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #48871|0 |1
is obsolete| |
--- Comment #52 from Julian Maurice ---
Created attachment 51084
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51084&action=edit
Bug 13321: Csv export - Display rrp and ecost with and without taxes
If a basketgroup is exported as a CSV file, both prices should be displayed to
avoid confusion.
Signed-off-by: Laurence Rault
Signed-off-by: Francois Charbonnier
Signed-off-by: Sonia Bouis
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:11:47 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:11:47 +0000
Subject: [Koha-bugs] [Bug 13321] Tax and prices calculation need to be fixed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13321
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #48872|0 |1
is obsolete| |
--- Comment #53 from Julian Maurice ---
Created attachment 51085
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51085&action=edit
Bug 13321: Fix unitprice rounding
We need to display the rounded value, but insert the exact value.
The unit price can be change on receiving, so the input value can be
modified.
But if it's not, we must insert the exact (not rounded) unit price,
which is the ecost value.
Signed-off-by: Laurence Rault
Signed-off-by: Francois Charbonnier
Signed-off-by: Sonia Bouis
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:14:24 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:14:24 +0000
Subject: [Koha-bugs] [Bug 16405] Fix Circulation/NoIssuesChargeGuarantees.t
test 2
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16405
--- Comment #7 from Marcel de Rooy ---
No problem.
is( $issuingimpossible->{DEBT_GUARANTEES} == 10.00 , 1, '')
and
is( $issuingimpossible->{DEBT_GUARANTEES} + 0, '10.00' + 0, '')
look the same to me btw
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:14:32 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:14:32 +0000
Subject: [Koha-bugs] [Bug 13321] Tax and prices calculation need to be fixed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13321
Julian Maurice changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Signed Off
--- Comment #54 from Julian Maurice ---
Patches rebased on master + 1st patch amended to use $order->{ordernumber}
instead of $ordernumber
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:14:41 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:14:41 +0000
Subject: [Koha-bugs] [Bug 16405] Fix Circulation/NoIssuesChargeGuarantees.t
test 2
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16405
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|m.de.rooy at rijksmuseum.nl |jonathan.druart at bugs.koha-c
| |ommunity.org
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:24:27 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:24:27 +0000
Subject: [Koha-bugs] [Bug 16104] Warnings "used only once: possible typo"
should be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16104
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |m.de.rooy at rijksmuseum.nl
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=16406
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:24:27 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:24:27 +0000
Subject: [Koha-bugs] [Bug 16406] Fix Context.t tests 27, 37 and sysprefs.t
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16406
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=16104
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:30:05 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:30:05 +0000
Subject: [Koha-bugs] [Bug 16412] New: Cache undef in L1 only
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16412
Bug ID: 16412
Summary: Cache undef in L1 only
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: gmcharlt at gmail.com
Reporter: m.de.rooy at rijksmuseum.nl
QA Contact: testopia at bugs.koha-community.org
No need to cache undef in L2.
Prevent numerous warnings like:
Use of uninitialized value in subroutine entry at
/usr/share/koha/masterclone/Koha/Cache.pm line 284
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:31:20 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:31:20 +0000
Subject: [Koha-bugs] [Bug 16412] Cache undef in L1 only
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16412
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |Needs Signoff
Patch complexity|--- |Small patch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:31:23 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:31:23 +0000
Subject: [Koha-bugs] [Bug 16412] Cache undef in L1 only
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16412
--- Comment #1 from Marcel de Rooy ---
Created attachment 51086
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51086&action=edit
Bug 16412: Cache undef in L1 only
No need to cache undef in L2.
Prevent numerous warnings like:
Use of uninitialized value in subroutine entry at
/usr/share/koha/masterclone/Koha/Cache.pm line 284
Test plan:
Run t/Cache.t
Run t/db_dependent/sysprefs.t
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:31:31 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:31:31 +0000
Subject: [Koha-bugs] [Bug 16412] Cache undef in L1 only
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16412
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|gmcharlt at gmail.com |m.de.rooy at rijksmuseum.nl
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:36:34 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:36:34 +0000
Subject: [Koha-bugs] [Bug 16403] Fix Holds.t (tests 9 and 39)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16403
--- Comment #6 from Marcel de Rooy ---
(In reply to Jonathan Druart from comment #5)
> Created attachment 51074 [details] [review]
> Bug 16403: Remove unnecessary undef param
>
> Signed-off-by: Jonathan Druart
I explicitly added the undef to *show* that the date is not defined
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 12:59:07 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 10:59:07 +0000
Subject: [Koha-bugs] [Bug 14695] Add ability to place multiple item holds on
a given record per patron
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14695
--- Comment #132 from Kyle M Hall ---
(In reply to Benjamin Daeuber from comment #131)
> Quick feedback on a few quirks. I don't know if any of these would prevent
> closing this and pushing it to master, but nonetheless:
>
> In Circ Client
>
> 1. It appears staff can place item level holds when this feature is on,
> regardless of whether item level holds are allowed. We tested on Wednesday,
> and it appears Nicole has already identified this.
I'll look into this!
>
> 2. The holds to place count (up to three) is set to 3. Select one barcode,
> set ?holds to place? dropdown to 3, and it only does one hold on the item.
> This seems to be unexpected, but we don?t really know why that dropdown box
> is there or what it is for.
The "Holds to place" does not do anything when placing item specific holds. The
last patch in this series should hide that pulldown when the record-level hold
checkbox is unchecked.
>
> 3. Put three items on hold on one record. Once you check in one of the three
> items you placed on hold, and it is now ?waiting? for the patron, you can
> place the fourth hold. You don?t need to check it out to the patron before
> being able to place the fourth hold. Is this expected behavior?
Yes. That is standard Koha behavior.
>
> In the OPAC
>
> 1. Related to point 1 above, this does not appear to affect the OPAC. Users
> are blocked from placing item level holds if they are not allowed.
>
> In both
>
> 1. Item level holds seem to be disregarded by the hold limit. So if you have
> a limit of 10, with 9 items on hold, you can still place 2 item level holds.
> We haven?t tested this in more detail.
This is also standard Koha behavior. If you want more accurate hold limits, you
need to add a record level itemtype to each record so that Koha will have a
fallback itemtype to check if the hold is a record level hold and not item
level.
>
> 2. A patron can?t place a bib level and item level hold for the same item.
> I?m actually struggling to think what scenario this might be used in, but
> we?re nothing if not thorough.
That is also intentional and expected behavior.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:03:58 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:03:58 +0000
Subject: [Koha-bugs] [Bug 15581] Add a circ rule to limit the auto renewals
given a delay
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15581
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Signed Off
--- Comment #21 from Kyle M Hall ---
(In reply to Jonathan Druart from comment #20)
> (In reply to Kyle M Hall from comment #19)
> > Would you please update the code to pass the object instead of the keys? It
> > makes sense to avoid fetching all this data multiple times when there is no
> > need to do so.
>
> Kyle,
> I have written GetLatestAutoRenewDate following the same pattern as
> GetSoonestRenewDate. I have noticed the inconsistencies in this part of code
> but opted to keep it as it instead of making the code diverge.
> If you agree I'd prefer to fix the two subroutines on another bug report,
> instead of adding a follow-up on this one.
That sounds like a good plan to me!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:14:30 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:14:30 +0000
Subject: [Koha-bugs] [Bug 16104] Warnings "used only once: possible typo"
should be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16104
--- Comment #14 from Jonathan Druart ---
Created attachment 51087
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51087&action=edit
Revert "Bug 16104: Remove warnings "used only once: possible typo""
This reverts commit 6b319a2d487257e8a08bc26af9c6b2e7fa2ece2c.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:14:38 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:14:38 +0000
Subject: [Koha-bugs] [Bug 16406] Fix Context.t tests 27, 37 and sysprefs.t
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16406
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|NEW |RESOLVED
--- Comment #4 from Marcel de Rooy ---
The revert on bug 16104 will resolve this issue too.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:16:19 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:16:19 +0000
Subject: [Koha-bugs] [Bug 16104] Warnings "used only once: possible typo"
should be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16104
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Pushed to Stable |ASSIGNED
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:16:25 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:16:25 +0000
Subject: [Koha-bugs] [Bug 16104] Warnings "used only once: possible typo"
should be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16104
Marcel de Rooy 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 Mon May 2 13:16:32 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:16:32 +0000
Subject: [Koha-bugs] [Bug 16104] Warnings "used only once: possible typo"
should be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16104
Marcel de Rooy 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 Mon May 2 13:22:48 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:22:48 +0000
Subject: [Koha-bugs] [Bug 16104] Warnings "used only once: possible typo"
should be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16104
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Passed QA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:22:51 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:22:51 +0000
Subject: [Koha-bugs] [Bug 16104] Warnings "used only once: possible typo"
should be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16104
Marcel de Rooy changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #49827|0 |1
is obsolete| |
Attachment #51087|0 |1
is obsolete| |
--- Comment #15 from Marcel de Rooy ---
Created attachment 51088
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51088&action=edit
Revert "Bug 16104: Remove warnings "used only once: possible typo""
This reverts commit 6b319a2d487257e8a08bc26af9c6b2e7fa2ece2c.
Signed-off-by: Marcel de Rooy
Tested with memcached and without MEMCACHED_SERVERS.
Cache.t, Context.t and sysprefs.t pass now.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:44:17 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:44:17 +0000
Subject: [Koha-bugs] [Bug 16412] Cache undef in L1 only
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16412
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Needs Signoff |Signed Off
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:44:20 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:44:20 +0000
Subject: [Koha-bugs] [Bug 16412] Cache undef in L1 only
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16412
--- Comment #2 from Jonathan Druart ---
Created attachment 51089
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51089&action=edit
Bug 16412: Make sure Koha::Cache->set_in_cache will not warn anymore
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 Mon May 2 13:44:31 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:44:31 +0000
Subject: [Koha-bugs] [Bug 16412] Cache undef in L1 only
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16412
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51086|0 |1
is obsolete| |
--- Comment #3 from Jonathan Druart ---
Created attachment 51090
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51090&action=edit
Bug 16412: Cache undef in L1 only
No need to cache undef in L2.
Prevent numerous warnings like:
Use of uninitialized value in subroutine entry at
/usr/share/koha/masterclone/Koha/Cache.pm line 284
Test plan:
Run t/Cache.t
Run t/db_dependent/sysprefs.t
Signed-off-by: Jonathan Druart
Amended patch: remove trailing space char and replace 'you' with 'we' to
match existing wordings.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:47:09 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:47:09 +0000
Subject: [Koha-bugs] [Bug 16412] Cache undef in L1 only
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16412
--- Comment #4 from Marcel de Rooy ---
Thanks, Jonathan.
Just to clarify for QA:
If you use memcached, this patch will remove numerous warnings like:
Use of uninitialized value in subroutine entry at
/usr/share/koha/masterclone/Koha/Cache.pm line 284
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:52:48 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:52:48 +0000
Subject: [Koha-bugs] [Bug 16384] When canceling 'edit basket',
return to basket summary if you came from there
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16384
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 Mon May 2 13:52:51 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:52:51 +0000
Subject: [Koha-bugs] [Bug 16384] When canceling 'edit basket',
return to basket summary if you came from there
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16384
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #50994|0 |1
is obsolete| |
--- Comment #3 from Jonathan Druart ---
Created attachment 51091
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51091&action=edit
Bug 16384: Fix cancel link for 'Edit basket'
When you edit the basket from the basket summary page,
saving the change brings you back to the basket summary
page, but cancelling brings you to the baskets page of
the vendor.
To test:
- Add a basket in acq
- Test cancel link returns to baskets page of vendor
- Add a basket and save
- Edit this basket
- Test cancel link now returns to basket summary page
Signed-off-by: Marc V?ron
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 Mon May 2 13:55:35 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:55:35 +0000
Subject: [Koha-bugs] [Bug 16390] Accounts.t does not need MPL
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16390
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
Depends on| |14878, 14402
--- Comment #3 from Jonathan Druart ---
Fix a regression of bug 14878 introduced by bug 14402.
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14402
[Bug 14402] Add --fees option to cleanup_database.pl to purge all 0 balance
fees and payments older than the given number of days
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14878
[Bug 14878] Tests - Create the branchcodes if needed
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:55:35 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:55:35 +0000
Subject: [Koha-bugs] [Bug 14402] Add --fees option to cleanup_database.pl to
purge all 0 balance fees and payments older than the given number of days
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14402
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |16390
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16390
[Bug 16390] Accounts.t does not need MPL
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:55:35 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:55:35 +0000
Subject: [Koha-bugs] [Bug 14878] Tests - Create the branchcodes if needed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14878
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |16390
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16390
[Bug 16390] Accounts.t does not need MPL
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 13:56:33 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:56:33 +0000
Subject: [Koha-bugs] [Bug 16390] Accounts.t does not need MPL
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16390
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 Mon May 2 13:56:36 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 11:56:36 +0000
Subject: [Koha-bugs] [Bug 16390] Accounts.t does not need MPL
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16390
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51049|0 |1
is obsolete| |
--- Comment #4 from Jonathan Druart ---
Created attachment 51092
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51092&action=edit
Bug 16390: Accounts.t does not need MPL
This test fails if you do not have a MPL branch, but a closer look shows
that we only need to replace one occurrence.
Test plan:
Run Accounts.t
Signed-off-by: Marc V?ron
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 Mon May 2 14:08:15 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:08:15 +0000
Subject: [Koha-bugs] [Bug 16369] Clean up and improve plugins template
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16369
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 Mon May 2 14:08:18 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:08:18 +0000
Subject: [Koha-bugs] [Bug 16369] Clean up and improve plugins template
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16369
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #50910|0 |1
is obsolete| |
--- Comment #8 from Jonathan Druart ---
Created attachment 51093
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51093&action=edit
Bug 16369 - Clean up and improve plugins template
This patch makes multiple changes to the plugins home page template to
bring it up to date with current interface patterns.
Test the following changes:
- Breacrumb links have been corrected to include "Tools" in the path.
Verify that this link is correct.
- A toolbar has been added for an "Upload plugin" button. Uploading is
an action, not a view, so it should be displayed in a toolbar. Verify
that the button works correctly.
- Messages are now formatted as messages rather than as headings. To
test, trigger a message by, for instance, uninstalling all plugins or
passing an invalid "method" parameter with the URL.
- Incorrect capitalization corrected.
- Plugin actions are moved to a single "Actions" dropdown menu. This
includes 'Run report,' 'Run tool,' 'Configure,' and 'Uninstall.' Test
that all these menu options work correctly.
- The standard "Tools" sidebar menu has been added.
- An "onclick" attribute has been removed in favor of defining the event
in JavaScript. Test by choosing the 'Uninstall' menu item for a
plugin. Test both confirm and cancel actions.
Also changed:
- Corrected capitalization on the tools home page.
- Adding missing plugins link to the tools sidebar menu.
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 Mon May 2 14:08:35 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:08:35 +0000
Subject: [Koha-bugs] [Bug 16411] Make Hold.t not depend on two existing
branches
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16411
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51071|0 |1
is obsolete| |
--- Comment #2 from Marc V?ron ---
Created attachment 51094
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51094&action=edit
Bug 16411: Make Hold.t not depend on two existing branches
If you do not have two branches, this test will fail.
Can't call method "branchcode" on an undefined value.
This patch adds a borrower and two branches with TestBuilder.
Test plan:
Run the test.
Signed-off-by: Marc V?ron
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:08:44 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:08:44 +0000
Subject: [Koha-bugs] [Bug 16411] Make Hold.t not depend on two existing
branches
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16411
Marc V?ron changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |veron at veron.ch
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 Mon May 2 14:09:23 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:09:23 +0000
Subject: [Koha-bugs] [Bug 16369] Clean up and improve plugins template
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16369
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
Severity|enhancement |normal
--- Comment #9 from Jonathan Druart ---
I'd consider this patch as a bugfix.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:10:45 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:10:45 +0000
Subject: [Koha-bugs] [Bug 16383] Making Local Use sysprefs actions buttons
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16383
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 Mon May 2 14:10:48 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:10:48 +0000
Subject: [Koha-bugs] [Bug 16383] Making Local Use sysprefs actions buttons
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16383
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #51067|0 |1
is obsolete| |
--- Comment #3 from Jonathan Druart ---
Created attachment 51095
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51095&action=edit
Bug 16383: Making Local Use sysprefs actions buttons
To test:
1) Go to Admin -> System Preferences -> Local Use tab
2) Confirm buttons show as font awesome buttons and work as expected
3) Confirm buttons do not wrap on a narrower browser
Sponsored-by: Catalyst IT
Signed-off-by: Marc V?ron
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 Mon May 2 14:13:47 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:13:47 +0000
Subject: [Koha-bugs] [Bug 11089] Item edit screen should use floating
toolbar like cataloging interface
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11089
--- Comment #5 from Jonathan Druart ---
Please see bug 16400 before QAing this patch.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:18:30 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:18:30 +0000
Subject: [Koha-bugs] [Bug 15562] Make Koha more suitable for shared hosting
with plack
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15562
--- Comment #41 from Kyle M Hall ---
(In reply to Jesse Weaver from comment #40)
> I think that may be a devbox-specific problem, Kyle; this works for me, at
> least in a single-site (OPAC/intranet) git installation.
>
> Though note that I used koha-multi.psgi, after adding:
>
> use Koha::Handler::Plack::CGI;
Excellent! Do you think you can sign off on the patch set?
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:22:22 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:22:22 +0000
Subject: [Koha-bugs] [Bug 10459] borrowers should have a timestamp
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10459
--- Comment #35 from Kyle M Hall ---
(In reply to Katrin Fischer from comment #33)
> It looks like we have a 'ton' of timestamp in the database. Apart from that
> there seems to be no existing other schema for timestamps columns that I
> could spot. I am not opposed to changing it - I think that's what Marcel
> wanted to do?
Indeed! I think we should standardize on a single format. I think "updated_on"
for timestamps that are set on each update and "created_on" for timestamps that
are only set at the time of creation would be appropriate. We could have a vote
at the next dev meeting to formalize this ( or another proposed format ) and
add it to the coding guidelines.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:33:27 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:33:27 +0000
Subject: [Koha-bugs] [Bug 10459] borrowers should have a timestamp
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10459
--- Comment #36 from M. Tompsett ---
(In reply to Kyle M Hall from comment #35)
> I think we should standardize on a single format.
+1
> I think "updated_on" for timestamps that are set on each update
> and "created_on" for timestamps ... only set at the time of creation
+1
> We could have a vote at the next dev meeting to formalize this ( or another
> proposed format ) and add it to the coding guidelines.
+1 -- Just incase I don't make it to the next dev meeting.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:39:32 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:39:32 +0000
Subject: [Koha-bugs] [Bug 16170] Pseudo foreign key in Items
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16170
--- Comment #16 from Kyle M Hall ---
> Kyle: You most probably did not. If I do, these test pass with me. Note that
> Holds.t does not pass (in current master). And a fix is in PQA for
> ILSDI_Services.t.
> See comment 12, giving you an q-a-d alternative too for running
> update_dbix_class_files.pl (for testing only).
Indeed, that was my issue!
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:41:09 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:41:09 +0000
Subject: [Koha-bugs] [Bug 10459] borrowers should have a timestamp
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10459
--- Comment #37 from Marc V?ron ---
(In reply to Kyle M Hall from comment #35)
> (In reply to Katrin Fischer from comment #33)
> > It looks like we have a 'ton' of timestamp in the database. Apart from that
> > there seems to be no existing other schema for timestamps columns that I
> > could spot. I am not opposed to changing it - I think that's what Marcel
> > wanted to do?
>
> Indeed! I think we should standardize on a single format. I think
> "updated_on" for timestamps that are set on each update and "created_on" for
> timestamps that are only set at the time of creation would be appropriate.
> We could have a vote at the next dev meeting to formalize this ( or another
> proposed format ) and add it to the coding guidelines.
+1 for standardizing, "created_on", "updated_on"
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:46:08 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:46:08 +0000
Subject: [Koha-bugs] [Bug 14395] Two different ways to calculate 'No renewal
before' (days or hours)
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14395
--- Comment #50 from bondiurbano ---
(In reply to Holger Mei?ner from comment #44)
> Don't worry, bondiurbano. Everything is okay. We're just waiting to pass QA.
> The QA team also has to check a lot of other bugs, so this may take some
> time. That's all.
Holger, please look at bug #15581 by Jonathan Druart
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:52:17 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:52:17 +0000
Subject: [Koha-bugs] [Bug 16170] Pseudo foreign key in Items
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16170
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Passed QA
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:52:19 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:52:19 +0000
Subject: [Koha-bugs] [Bug 16170] Pseudo foreign key in Items
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16170
Kyle M Hall changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #49723|0 |1
is obsolete| |
Attachment #49724|0 |1
is obsolete| |
Attachment #49771|0 |1
is obsolete| |
Attachment #49772|0 |1
is obsolete| |
--- Comment #17 from Kyle M Hall ---
Created attachment 51096
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51096&action=edit
Bug 16170: Pseudo foreign key in Items
While many of us would like to get rid of biblioitems one day, the current
scheme includes a biblioitemnumber and a biblionumber in Items.
(Which is not so great..)
But also note that biblionumber is NOT defined as a foreign key in Items,
although a belongs_to relation has been added to the DBIx scheme!
This inconsistency should be resolved. The "remove biblioitem table"
operation is a large one, but in the meantime we better make biblionumber
a regular FK not a 'pseudo' one.
Note: If in an (very) exceptional case biblionumbers are found in items,
that do not exist in biblio, this patch prints a warning at upgrade
time and does not add the constraint.
@RM: Please update the DBIx scheme accordingly.
Test plan:
[1] Run the upgrade. Check if the FK constraint has been added.
[2] Remove the FK constraint. Change the biblionumber of one item to an
unexisting record. Run the upgrade again. Notice the warning.
Signed-off-by: Marcel de Rooy
Tested both cases: constraint added as well as warning printed.
Signed-off-by: Mark Tompsett
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:52:29 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:52:29 +0000
Subject: [Koha-bugs] [Bug 16170] Pseudo foreign key in Items
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16170
--- Comment #18 from Kyle M Hall ---
Created attachment 51097
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51097&action=edit
Bug 16170 - Corrected to make work more smoothly
Signed-off-by: Mark Tompsett
Signed-off-by: Marcel de Rooy
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:52:31 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:52:31 +0000
Subject: [Koha-bugs] [Bug 16170] Pseudo foreign key in Items
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16170
--- Comment #19 from Kyle M Hall ---
Created attachment 51098
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51098&action=edit
Bug 16170: [QA Follow-up] Adjust Items.t
Items.t inserts an item via biblioitem not correctly linked to biblio.
The new foreign key constraint does not allow that.
Actually, we should be choosing to either remove biblioitems from Koha or
remove biblionumber from items.
Note: This seems to be the only case where an item is added this way.
Test plan:
Run Items.t
Signed-off-by: Marcel de Rooy
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 14:52:34 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 12:52:34 +0000
Subject: [Koha-bugs] [Bug 16170] Pseudo foreign key in Items
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16170
--- Comment #20 from Kyle M Hall ---
Created attachment 51099
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51099&action=edit
Bug 16170: [QA Follow-up] Fix embeded typos
Just noticed this typo three times here.
Signed-off-by: Marcel de Rooy
Signed-off-by: Kyle M Hall
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 15:09:38 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 13:09:38 +0000
Subject: [Koha-bugs] [Bug 14530] 856$u URL checker ( check-url.pl )
complains about ftp based URLs
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14530
Jesse Lambertson changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jesse at meta21st.com
--- Comment #7 from Jesse Lambertson ---
This is mostly a question about URL checking in general.
This script checks for good links in 856 $ u?
Cab it be run anywhere while logged into the server, or does it need to be run
from a specific part in the file system?
Just want to make sure.
Thank you,
Jesse
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 15:39:57 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 13:39:57 +0000
Subject: [Koha-bugs] [Bug 12509] Untranslatable "Restriction added by
overdues process"
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12509
Marjorie Barry-Vila changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |marjorie.barry-vila at ccsr.qc
| |.ca
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 15:49:53 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 13:49:53 +0000
Subject: [Koha-bugs] [Bug 16413] New: Prototype for GetLatestAutoRenewDate
and GetSoonestRenewDate should be changed
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16413
Bug ID: 16413
Summary: Prototype for GetLatestAutoRenewDate and
GetSoonestRenewDate should be changed
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: ASSIGNED
Severity: enhancement
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
Depends on: 16344
Currently the 2 GetLatestAutoRenewDate and GetSoonestRenewDate subroutines
takes $borrowernumber and $itemnumber in parameter.
They refetch the patron, item and issue information but they already are
available from where they are called.
It would make sense to change the prototype of these 2 subroutines to accept
the patron, item and issue information directly to avoid unnecessary refetches.
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16344
[Bug 16344] Add a circ rule to limit the auto renewals given a specific date
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 15:49:53 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 13:49:53 +0000
Subject: [Koha-bugs] [Bug 16344] Add a circ rule to limit the auto renewals
given a specific date
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16344
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |16413
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16413
[Bug 16413] Prototype for GetLatestAutoRenewDate and GetSoonestRenewDate should
be changed
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 15:53:13 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 13:53:13 +0000
Subject: [Koha-bugs] [Bug 16413] Prototype for GetLatestAutoRenewDate and
GetSoonestRenewDate should be changed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16413
--- Comment #1 from Jonathan Druart ---
Created attachment 51100
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51100&action=edit
Bug 16413: Change prototype GetLatestAutoRenewDate
Currently the 2 GetLatestAutoRenewDate and GetSoonestRenewDate
subroutines takes $borrowernumber and $itemnumber in parameter.
They refetch the patron, item and issue information but they already are
available from where they are called.
It would make sense to change the prototype of these 2 subroutines to
accept the patron, item and issue information directly to avoid
unnecessary refetches.
Test plan:
Make sure this change does not introduce regressions on bug 15581 and bug
16344.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 15:53:10 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 13:53:10 +0000
Subject: [Koha-bugs] [Bug 16413] Prototype for GetLatestAutoRenewDate and
GetSoonestRenewDate should be changed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16413
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 Mon May 2 15:53:17 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 13:53:17 +0000
Subject: [Koha-bugs] [Bug 16413] Prototype for GetLatestAutoRenewDate and
GetSoonestRenewDate should be changed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16413
--- Comment #2 from Jonathan Druart ---
Created attachment 51101
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51101&action=edit
Bug 16413: Change prototype of GetSoonestRenewDate
Same change as previous patch for the GetSoonestRenewDate subroutine.
Test plan:
Make sure this change does not introduce regressions on bug 7413.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 15:53:47 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 13:53:47 +0000
Subject: [Koha-bugs] [Bug 15581] Add a circ rule to limit the auto renewals
given a delay
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15581
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=16413
--- Comment #22 from Jonathan Druart ---
(In reply to Kyle M Hall from comment #21)
> (In reply to Jonathan Druart from comment #20)
> > (In reply to Kyle M Hall from comment #19)
> > > Would you please update the code to pass the object instead of the keys? It
> > > makes sense to avoid fetching all this data multiple times when there is no
> > > need to do so.
> >
> > Kyle,
> > I have written GetLatestAutoRenewDate following the same pattern as
> > GetSoonestRenewDate. I have noticed the inconsistencies in this part of code
> > but opted to keep it as it instead of making the code diverge.
> > If you agree I'd prefer to fix the two subroutines on another bug report,
> > instead of adding a follow-up on this one.
>
> That sounds like a good plan to me!
Patches submitted on bug 16413.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 15:53:47 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 13:53:47 +0000
Subject: [Koha-bugs] [Bug 16413] Prototype for GetLatestAutoRenewDate and
GetSoonestRenewDate should be changed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16413
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=15581
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 16:01:24 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 14:01:24 +0000
Subject: [Koha-bugs] [Bug 16402] Web Installation fails - Step 3 Importing
the Database Structure
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16402
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=14637
CC| |jonathan.druart at bugs.koha-c
| |ommunity.org
--- Comment #1 from Jonathan Druart ---
Hello Gopalakrishna,
Please try to comment the sql_mode=STRICT_TRANS_TABLES line in your
MySQL/MariaDB config file.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 16:01:24 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 14:01:24 +0000
Subject: [Koha-bugs] [Bug 14637] Add patron category fails with MySQL 5.6.26
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14637
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=16402
--
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 Mon May 2 16:03:36 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 14:03:36 +0000
Subject: [Koha-bugs] [Bug 16414] New: aqorders.budgetgroup_id has never been
used and can be removed
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16414
Bug ID: 16414
Summary: aqorders.budgetgroup_id has never been used and can be
removed
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: ASSIGNED
Severity: normal
Priority: P5 - low
Component: Acquisitions
Assignee: jonathan.druart at bugs.koha-community.org
Reporter: jonathan.druart at bugs.koha-community.org
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 Mon May 2 16:03:55 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 14:03:55 +0000
Subject: [Koha-bugs] [Bug 16414] aqorders.budgetgroup_id has never been used
and can be removed
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16414
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=13669
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 16:03:55 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 14:03:55 +0000
Subject: [Koha-bugs] [Bug 13669] Webinstaller fails on MySQL 5.6
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13669
Jonathan Druart changed:
What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugs.koha-community
| |.org/bugzilla3/show_bug.cgi
| |?id=16414
--- Comment #7 from Jonathan Druart ---
(In reply to Jonathan Druart from comment #6)
> (In reply to Humberto Blanco from comment #5)
> > Tom?s,
> > I have similar problem with mysql enterprise end i need to modify the
> > database structure.so:
> >
> > ALTER TABLE aqorders ALTER COLUMN budgetgroup_id SET DEFAULT 0;
>
> This column does not seem to be used and could be removed.
> In the meanwhile it would be better to set it to NULL (and remove the NOT
> NULL constraint) instead of set it to 0 (which does not make sense).
See bug 16414.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 16:07:48 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 14:07:48 +0000
Subject: [Koha-bugs] [Bug 16415] New: Layout problem on staff client detail
page if local cover images are enabled
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16415
Bug ID: 16415
Summary: Layout problem on staff client detail page if local
cover images are enabled
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
URL: /cgi-bin/koha/catalogue/detail.pl?biblionumber=X
OS: All
Status: ASSIGNED
Severity: minor
Priority: P5 - low
Component: Templates
Assignee: oleonard at myacpl.org
Reporter: oleonard at myacpl.org
QA Contact: testopia at bugs.koha-community.org
Created attachment 51102
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51102&action=edit
Screenshot showing layout problem
If you have local cover images enabled in the staff client but not Amazon
images, local covers are not displayed in a column on the right like they
should be.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 16:16:16 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 14:16:16 +0000
Subject: [Koha-bugs] [Bug 16415] Layout problem on staff client detail page
if local cover images are enabled
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16415
Owen Leonard changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |Needs Signoff
Patch complexity|--- |Trivial patch
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 16:16:18 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 14:16:18 +0000
Subject: [Koha-bugs] [Bug 16415] Layout problem on staff client detail page
if local cover images are enabled
In-Reply-To:
References:
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16415
--- Comment #1 from Owen Leonard ---
Created attachment 51103
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51103&action=edit
Bug 16415 - Layout problem on staff client detail page if local cover images
are enabled
This patch fixes a layout problem which appears on the staff client
bibliographic detail page if local covers are enabled but Amazon covers
are not.
To test, apply the patch and turn off AmazonCoverImages.
- Enable the LocalCoverImages system preference and locate or create a
record which has a local cover image associated with it.
- View the bibliographic detail page. The bibliographic details should
display in two columns, with text on the left and local cover image on
the right.
- Test with AmazonCoverImages and LocalCoverImages enabled.
- Test with AmazonCoverImages enabled and LocalCoverImages disabled.
--
You are receiving this mail because:
You are watching all bug changes.
From bugzilla-daemon at bugs.koha-community.org Mon May 2 16:22:13 2016
From: bugzilla-daemon at bugs.koha-community.org (bugzilla-daemon at bugs.koha-community.org)
Date: Mon, 02 May 2016 14:22:13 +0000
Subject: [Koha-bugs] [Bug 16416] New: insecure connection for bug report
search
Message-ID:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16416
Bug ID: 16416
Summary: insecure connection for bug report search
Change sponsored?: ---
Product: Project Infrastructure
Version: unspecified
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: Project website
Assignee: wizzyrea at gmail.com
Reporter: wosch at freebsd.org
On the page https://koha-community.org/support/search-bugs/
I searched for a bug report. I put in the string "boston" and pressed the
submit button.
In Firefox and Safari I get a popup warning from the browser:
"The information you have entered on this page will be sent over an insecure
connection and could be read by a third party.
Are you sure you want to send this information?"
In the source code I see: