https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #83 from Matt Blenkinsop ---
Nice work everyone!
Pushed to stable for 22.11.x
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Matt Blenkinsop changed:
What|Removed |Added
Version(s)|23.05.00|23.05.00,22.11.02
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #82 from Martin Renvoize ---
Nice work everyone!
Pushed to 23.05.x for the next release
--
You are receiving this mail because:
You are watching all bug changes.
___
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Martin Renvoize changed:
What|Removed |Added
Status|Passed QA |Pushed to master
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #81 from Kyle M Hall ---
Created attachment 145321
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=145321=edit
Bug 14784: (follow-up) Unit tests
Signed-off-by: Kyle M Hall
--
You are receiving this
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Kyle M Hall changed:
What|Removed |Added
Attachment #137727|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Kyle M Hall changed:
What|Removed |Added
Status|Signed Off |Passed QA
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #79 from Kyle M Hall ---
(In reply to Katrin Fischer from comment #78)
> @Kyle: could you maybe weigh in here? IIRC you were the one to write a part
> of this code at least?
Yes, considering Koha::Paron::is_debarred
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #78 from Katrin Fischer ---
@Kyle: could you maybe weigh in here? IIRC you were the one to write a part of
this code at least?
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Katrin Fischer changed:
What|Removed |Added
Status|Failed QA |Signed Off
--- Comment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #76 from Blou ---
So, Signed Off or Failed QA ?
Seems the is still a valid test, per Cate.
--
You are receiving this mail because:
You are watching all bug changes.
___
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #75 from Katrin Fischer ---
Not sure about the date, actually:
If you create an unrestricted debarment you get:
borrowers_debarments.expiration = NULL
borrowers.debarred = -12-31
--
You are receiving this mail
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Martin Renvoize changed:
What|Removed |Added
Status|Signed Off |Failed QA
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #74 from Martin Renvoize ---
Comment on attachment 137727
--> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=137727
Bug 14784: Fix checkin message for restricted patrons
Review of attachment 137727:
-->
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Shi Yao Wang changed:
What|Removed |Added
Status|Failed QA |Signed Off
--- Comment #73
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #72 from Shi Yao Wang ---
Created attachment 137728
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=137728=edit
Bug 14784: (follow-up) Unit tests
--
You are receiving this mail because:
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Shi Yao Wang changed:
What|Removed |Added
Attachment #136327|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Victor Grousset/tuxayo changed:
What|Removed |Added
Status|Signed Off |Failed QA
---
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #69 from Shi Yao Wang ---
The current patch is quite different than the previous one so I marked the
previous tests as obsolete.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #68 from Jonathan Druart ---
Did we lose the tests?
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #67 from David Nind ---
My testing notes (using koha-testing-docker)
Step 1: To add a restriction:
. Select and edit a patron
. Scroll down to the 'Patron restrictions' section
. Add a manual
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
David Nind changed:
What|Removed |Added
Attachment #136040|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
David Nind changed:
What|Removed |Added
Status|Needs Signoff |Signed Off
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Katrin Fischer changed:
What|Removed |Added
Status|REOPENED|Needs Signoff
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Shi Yao Wang changed:
What|Removed |Added
Resolution|WORKSFORME |---
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Shi Yao Wang changed:
What|Removed |Added
Attachment #112724|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Shi Yao Wang changed:
What|Removed |Added
CC||shi-yao.w...@inlibro.com
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #64 from Shi Yao Wang ---
Created attachment 136040
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=136040=edit
Bug 14784: Fix checkin message for restricted patrons
Test plan:
Before
1) Select a user
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #63 from Katrin Fischer ---
Changing status to indicate more clearly that these patches haven't been
pushed.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Katrin Fischer changed:
What|Removed |Added
Resolution|FIXED |WORKSFORME
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Florian changed:
What|Removed |Added
CC||florian.bontemps@biblibre.c
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
The Minh Luong changed:
What|Removed |Added
Status|Failed QA |RESOLVED
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
The Minh Luong changed:
What|Removed |Added
CC||the-minh.lu...@inlibro.com
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
David Nind changed:
What|Removed |Added
Status|Needs Signoff |Failed QA
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Blou changed:
What|Removed |Added
Assignee|charles.far...@inlibro.com |alexis.ripe...@inlibro.com
--
You
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #59 from Arthur Suzuki ---
Hi Sonia,
Just managed to get my Koha testing environment up again.
Unfortunately, after applying both tests patch and fix patchs, some tests are
still failing.
Still a bit of work needed on
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Koha Team University Lyon 3 changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Alexis Ripetti changed:
What|Removed |Added
Status|Failed QA |Needs Signoff
--- Comment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Alexis Ripetti changed:
What|Removed |Added
Attachment #110763|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Alexis Ripetti changed:
What|Removed |Added
Attachment #110764|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #54 from Arthur Suzuki ---
Hi Alexis,
Sorry for the late reply...
After applying both patches I still get a failed tests:
root@kohadevbox:koha(master)$ prove t/db_dependent/Circulation.t
t/db_dependent/Circulation.t ..
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #53 from Alexis Ripetti ---
(In reply to Jonathan Druart from comment #52)
> 1. The second patch "Add tests" adds a change to the module, is that
> expected?
>
> 2. You are modifying a test that is not (at first glance)
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Jonathan Druart changed:
What|Removed |Added
Status|Passed QA |Failed QA
--- Comment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Kyle M Hall changed:
What|Removed |Added
Attachment #106808|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Kyle M Hall changed:
What|Removed |Added
Status|Signed Off |Passed QA
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #51 from Kyle M Hall ---
Created attachment 110764
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=110764=edit
Bug 14784: Add tests
This patch adds tests for PrevDebarred message of AddReturn when
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Kyle M Hall changed:
What|Removed |Added
CC||k...@bywatersolutions.com
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #49 from Alexis Ripetti ---
(In reply to Arthur Suzuki from comment #48)
> Test patch works as expected (there is one test which doesn't pass when test
> patch is applied alone).
> However the test patch seems to change
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #48 from Arthur Suzuki ---
Test patch works as expected (there is one test which doesn't pass when test
patch is applied alone).
However the test patch seems to change things in C4/Circulation as well,
is this due to a
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Arthur Suzuki changed:
What|Removed |Added
Attachment #110485|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Arthur Suzuki changed:
What|Removed |Added
Status|Needs Signoff |Signed Off
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Alexis Ripetti changed:
What|Removed |Added
Status|Signed Off |Needs Signoff
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Alexis Ripetti changed:
What|Removed |Added
Status|Failed QA |Signed Off
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Alexis Ripetti changed:
What|Removed |Added
Attachment #106809|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Kelly McElligott changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Katrin Fischer changed:
What|Removed |Added
Status|Signed Off |Failed QA
--- Comment #43
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Maryse Simard changed:
What|Removed |Added
Status|Needs Signoff |Signed Off
--- Comment #42
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #41 from Katrin Fischer ---
(In reply to Maryse Simard from comment #40)
> Rebased the patch and added a commit for unit tests.
>
> I was unsure which status to set this to, so I've set it back to needs
> signoff.
If
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Katrin Fischer changed:
What|Removed |Added
Attachment #79826|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #39 from Maryse Simard ---
Created attachment 106809
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106809=edit
Bug 14784: Add tests
This patch adds tests for PrevDebarred message of AddReturn when
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Maryse Simard changed:
What|Removed |Added
CC||maryse.sim...@inlibro.com
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #38 from Maryse Simard ---
Created attachment 106808
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106808=edit
Bug 14784: Fix checkin message for debarred patrons
After thinking about it I changed
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Arthur Suzuki changed:
What|Removed |Added
CC||arthur.suz...@biblibre.com
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #37 from Katrin Fischer ---
This makes changes to a core routine - AddReturn - so asking for tests is
normal and they should be provided by the patch author ideally - anyone can,
but it's not QA's job.
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #36 from Fridolin SOMERS ---
> Tests are missing anyhow
This is a very important code, if anyone is willing to provide tests, you are
welcome :)
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Niamh changed:
What|Removed |Added
CC||niamh.walker-headon@it-tall
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Jonathan Druart changed:
What|Removed |Added
Status|Signed Off |Failed QA
--- Comment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Victor Grousset/tuxayo changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Victor Grousset/tuxayo changed:
What|Removed |Added
Attachment #77683|0 |1
is
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Charles Farmer changed:
What|Removed |Added
Attachment #69296|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Charles Farmer changed:
What|Removed |Added
Assignee|david.bourga...@inlibro.com |charles.far...@inlibro.com
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Amit Gupta changed:
What|Removed |Added
CC||amitddng...@gmail.com
--
You
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #31 from Katrin Fischer ---
Hi Frido, maybe you can sign-off again? I think David made changes after my
initial sign-off (would have been better in a little follow-up patch)
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #30 from Fridolin SOMERS ---
It also corrects a small bug, when restriction is infinite, the date
"31/12/" was displayed in return page.
This is because new code always checks for this
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #29 from Fridolin SOMERS ---
(In reply to David Bourgault from comment #28)
> Signed-off-by: Katrin Fischer
Is this patch signed off ?
Or does it need a second
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Fridolin SOMERS changed:
What|Removed |Added
Version|unspecified
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Fridolin SOMERS changed:
What|Removed |Added
Patch complexity|---
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
David Bourgault changed:
What|Removed |Added
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
David Bourgault changed:
What|Removed |Added
Status|Failed QA
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
David Bourgault changed:
What|Removed |Added
Attachment #69295|0
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
David Bourgault changed:
What|Removed |Added
Attachment #68132|0
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Marcel de Rooy changed:
What|Removed |Added
Status|BLOCKED
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Marcel de Rooy changed:
What|Removed |Added
Status|Signed Off
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Marcel de Rooy changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #23 from Katrin Fischer ---
Hi David, thx for the updated test plan. I reset my system since and I could
see the bug and confirm the fix now.
--
You are receiving this mail because:
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Katrin Fischer changed:
What|Removed |Added
Attachment #67102|0
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Katrin Fischer changed:
What|Removed |Added
Status|In Discussion
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
David Bourgault changed:
What|Removed |Added
Status|Failed QA
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Katrin Fischer changed:
What|Removed |Added
Status|Needs Signoff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
David Bourgault changed:
What|Removed |Added
Attachment #58227|0
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Jonathan Druart changed:
What|Removed |Added
Status|Signed Off
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Philippe Audet-Fortin changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Owen Leonard changed:
What|Removed |Added
Status|Needs Signoff |Failed
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
--- Comment #16 from Owen Leonard ---
I don't think I am able to reproduce this bug. Either that or I misunderstand
what the bug is. As far as I can tell I'm getting the correct checkin messages
for restricted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Baptiste changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Bouzid changed:
What|Removed |Added
Assignee|remi.mayrand-provencher@inL
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Bouzid changed:
What|Removed |Added
Status|REOPENED|Needs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Bouzid changed:
What|Removed |Added
Status|CLOSED
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Bouzid changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784
Bouzid changed:
What|Removed |Added
Attachment #49377|0 |1
1 - 100 of 119 matches
Mail list logo