https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #32 from Jonathan Druart
---
Created attachment 92031
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=92031=edit
Bug 22741: Add a test
Signed-off-by: Jonathan Druart
--
You are receiving this mail
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Jonathan Druart changed:
What|Removed |Added
Attachment #90877|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Jonathan Druart changed:
What|Removed |Added
Status|Needs Signoff |Signed Off
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #30 from Jonathan Druart
---
(In reply to Martin Renvoize from comment #29)
> Hmm.. I don't really like the idea of calling a t/lib function in Koha::
> code.. if we're going to do that I'd suggest we move the compare
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Martin Renvoize changed:
What|Removed |Added
CC||katrin.fisc...@bsz-bw.de
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #29 from Martin Renvoize ---
Hmm.. I don't really like the idea of calling a t/lib function in Koha:: code..
if we're going to do that I'd suggest we move the compare function into koha
proper.
However, I also don't
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #28 from Jonathan Druart
---
(In reply to Martin Renvoize from comment #18)
> {
>"updated_on" : {
>
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #27 from Martin Renvoize ---
To note.. the cause of this in the first place and a patch to try and alleviate
it was written ages ago on bug 20621
--
You are receiving this mail because:
You are watching all bug
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Martin Renvoize changed:
What|Removed |Added
See Also|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Martin Renvoize changed:
What|Removed |Added
Patch complexity|--- |Trivial patch
--
You
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Martin Renvoize changed:
What|Removed |Added
Status|ASSIGNED|Needs Signoff
---
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Martin Renvoize changed:
What|Removed |Added
Attachment #90850|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
M. Tompsett changed:
What|Removed |Added
Status|Needs Signoff |ASSIGNED
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #23 from Martin Renvoize ---
Coolios, so just excluding from the action log trigger would do the trick and
save us a call to date time.. I'll amend the patch in the morning
--
You are receiving this mail because:
You
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #22 from Tomás Cohen Arazi ---
The timezone can be set on connexion time, and we do in Koha/Database:78
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Martin Renvoize changed:
What|Removed |Added
Keywords||RM_priority
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Martin Renvoize changed:
What|Removed |Added
Assignee|koha-b...@lists.koha-commun
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Martin Renvoize changed:
What|Removed |Added
Status|NEW |Needs Signoff
--
You
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #21 from Martin Renvoize ---
Created attachment 90850
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=90850=edit
Bug 22741: Make updated_on timezone aware in Koha::Patron
--
You are receiving this
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #20 from Martin Renvoize ---
Had a quick chat with Kyle and Tomas about this..
We think we should add updated_on to the exclude list for the actionlogs (I
don't think it makes much sense to record such a system
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #19 from Martin Renvoize ---
Wow.. this is a fun one..
The updated_on field is updated by a DB level trigger, but our Koha::Object
store routine does not return freshly populated Koha::Object data (it uses
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #18 from Martin Renvoize ---
OK.. so I threw in some debugging and it looks like it's the updated_on field
that's throwing us.
ok 3 - With BorrowerLogs, one detailed MODIFY action should be logged for
the
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Martin Renvoize changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Jonathan Druart changed:
What|Removed |Added
Severity|enhancement |normal
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #16 from Jonathan Druart
---
(In reply to Jonathan Druart from comment #14)
> I can recreate in a devbox if I change the timezone (ie. tz is different
> between host and box)
It's not that, I can recreate when MySQL
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #15 from Jonathan Druart
---
It was D9 on Node 2 (#742)
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #14 from Jonathan Druart
---
I can recreate in a devbox if I change the timezone (ie. tz is different
between host and box)
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
--- Comment #13 from Tomás Cohen Arazi ---
(In reply to Jonathan Druart from comment #12)
> Jenkins is failing randomly with this error:
Beware that run was launched with COVERAGE=yes, so a very allow run
--
You are receiving
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22741
Jonathan Druart changed:
What|Removed |Added
Summary|Second prove of |Koha/Patrons.t is failing
29 matches
Mail list logo