Time Nick Message
23:47 dcook tfw you find a bug and really don't want to fix it lol..
23:36 aleisha hi dcook :)
23:35 * dcook just glares at the authority code for a bit heh
23:35 dcook hola aleisha
20:23 aleisha hello
16:52 cait pref thing
16:51 cait without looking at the code... there are cases where holds don't block renewals - if there are other available items
16:43 reiveune bye
15:39 kohaputti you too, bye!
15:39 kohaputti yes, hopefully just that
15:39 Joubu bye kohaputti, have a great evening!
15:39 Joubu or maybe the tests don't init the data correctly
15:39 Joubu yes, that seems bad
15:39 kohaputti well, I will investigate tomorrow -->
15:37 kohaputti or maybe the tests are just missing some circulation rules
15:36 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26659 major, P5 - low, ---, joonas.kylmala, ASSIGNED , An item can be renewed even if there are holds waiting to be filled
15:36 kohaputti Joubu, hmm, if the change from bug 26659 breaks so many tests from Circulation.t then there might be a big issue here
15:31 Joubu yes, silly mistake from me..
15:29 kohaputti it seems to break a whole lot of things
15:29 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26659 major, P5 - low, ---, joonas.kylmala, Needs Signoff , An item can be renewed even if there are holds waiting to be filled
15:29 kohaputti Joubu, issue in my configuration, I ran git revert HEAD~1 instead of git revert HEAD with my own changes for bug 26659
15:27 kohaputti will try reset_all once more
15:26 kohaputti ok, I have that last commit
15:26 Joubu no
15:26 kohaputti Joubu, do I need misc4devel updates?
15:26 Joubu (you need this last commit or Circulation.t will fail)
15:26 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26529 major, P5 - low, ---, nick, Pushed to master , Holds rules enforced incorrectly when not set at library level
15:26 Joubu c12ede482f Bug 26529: (follow-up) Remove warning
15:25 Joubu kohaputti: t/db_dependent/Circulation.t passes for me
15:24 kohaputti I have to go now unfortunately, I will continue debugging tomorrow if still persist, could be something I have messed up in my configs
15:23 kohaputti I have the latest version, nothing new to fetch from git
15:22 Joubu pull
15:21 kohaputti Joubu, are you aware that Circulation.t fails currently?
15:18 koha-jenkins Project Koha_Master_U18 build #914: SUCCESS in 47 min: https://jenkins.koha-community.org/job/Koha_Master_U18/914/
15:15 koha-jenkins Project Koha_Master_U16 build #83: FIXED in 47 min: https://jenkins.koha-community.org/job/Koha_Master_U16/83/
15:15 wahanui Congratulations!
15:15 koha-jenkins Yippee, build fixed!
15:00 koha-jenkins Project Koha_Master_D11 build #107: FIXED in 41 min: https://jenkins.koha-community.org/job/Koha_Master_D11/107/
15:00 wahanui Congratulations!
15:00 koha-jenkins Yippee, build fixed!
14:47 koha-jenkins Project Koha_Master_D10_Deps build #93: FIXED in 53 min: https://jenkins.koha-community.org/job/Koha_Master_D10_Deps/93/
14:47 wahanui Congratulations!
14:47 koha-jenkins Yippee, build fixed!
14:30 koha-jenkins Project Koha_Master_D10 build #369: FIXED in 45 min: https://jenkins.koha-community.org/job/Koha_Master_D10/369/
14:30 wahanui Congratulations!
14:30 koha-jenkins Yippee, build fixed!
14:28 koha-jenkins Project Koha_Master_U18 build #913: FIXED in 48 min: https://jenkins.koha-community.org/job/Koha_Master_U18/913/
14:28 wahanui Congratulations!
14:28 koha-jenkins Yippee, build fixed!
14:25 ashimema[m] cool.. i was hoping that would be the case
14:25 ashimema[m] :)
14:23 cait the current precendence is undcomented... so I feel a release notes comment might be ok in that case?
14:23 cait i think it sounds like a bug - they shoudl follow the same rules
14:18 koha-jenkins Project Koha_Master_U20 build #113: STILL UNSTABLE in 41 min: https://jenkins.koha-community.org/job/Koha_Master_U20/113/
14:18 ashimema[m] the fix is easy.. but it'll change the precedence for existing installs
14:17 ashimema[m] the first comment on the bug is pretty clear
14:17 ashimema[m] basically the circ rules pertaining to IssuingCharges are not consistent, with the way they fallback through defaults etc, with any of the other circ rules
14:16 cait can you explain in a few words?
14:14 ashimema[m] It feels like an accidental error, but I'm wondering if anyone is seeing it as a feature
14:13 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26593 normal, P5 - low, ---, koha-bugs, NEW , Rental discounts are applied in wrong precedence order
14:13 ashimema[m] anyone got any thoughts on bug 26593
13:54 koha-jenkins Project Koha_Master_D9_My8 build #433: FIXED in 1 hr 2 min: https://jenkins.koha-community.org/job/Koha_Master_D9_My8/433/
13:54 wahanui Congratulations!
13:54 koha-jenkins Yippee, build fixed!
13:44 koha-jenkins Project Koha_Master_D9 build #1450: FIXED in 50 min: https://jenkins.koha-community.org/job/Koha_Master_D9/1450/
13:44 wahanui Congratulations!
13:44 koha-jenkins Yippee, build fixed!
13:39 koha-jenkins Project Koha_Master_U2010 build #23: FIXED in 48 min: https://jenkins.koha-community.org/job/Koha_Master_U2010/23/
13:39 wahanui Congratulations!
13:39 koha-jenkins Yippee, build fixed!
13:37 koha-jenkins Project Koha_Master_D9_MDB_Latest build #408: FIXED in 43 min: https://jenkins.koha-community.org/job/Koha_Master_D9_MDB_Latest/408/
13:37 wahanui Congratulations!
13:37 koha-jenkins Yippee, build fixed!
13:29 kohaputti hmm, maybe the cache is cleared on every connection, but it should probably be on every SIP message.
13:24 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23640 major, P5 - low, ---, jonathan.druart, Pushed to oldstable , L1 cache too long in SIP Server
13:24 kohaputti just got report from library that the syspref cache clearing fix in bug 23640 doesn't work anymore, gotta investigate that too
13:22 ashimema[m] that's given me the motivation to pick it back up again.. I had been struggling for a few days to find that motivation again
13:22 ashimema[m] brill, that would be great :)
13:22 kohaputti actually, I will add myself in CC
13:21 kohaputti hmm, ok, I think I want to go the refactoring way, let me know when ready for qa :)
13:21 ashimema[m] I ended up down a rabbit hole.. trying to move transfers code into Koha:: and fix a bunch of bugs in it at the same time
13:21 ashimema[m] it's likely way out of scope.. and I'm happy to rebase atop whatever you do
13:20 ashimema[m] I agree
13:20 kohaputti I will take a look before proceeding, thanks :)
13:20 kohaputti I think I will, I hate that SIP has it's own handling, and that sco and intranet are not handled the same
13:20 ashimema[m] if you were, I'd have said please take a look at my work in progress tree: https://github.com/Koha-Community/Koha/compare/master...mrenvoize:bug_22569
13:19 ashimema[m] I was worried you were going to dig deep into Transfers in general
13:19 ashimema[m] OK
13:19 ashimema[m] ah, yeah..
13:19 kohaputti ashimema[m], the instant notification bug had patch for preventing the checkouts of holds in processing, the same way holds in waiting were blocked, it would be one line change to block now holds in transfer too in SIP
13:17 ashimema[m] one moment
13:17 ashimema[m] indeed
13:17 ashimema[m] ooh
13:17 kohaputti holds in transfer I guess
13:17 kohaputti the transfer handling in sip, sco, intranet
13:17 ashimema[m] or the holds in transfer
13:16 kohaputti yes
13:16 ashimema[m] trasnfers you mean?
13:16 ashimema[m] the more I fix, the more I find broken :(
13:16 kohaputti it needs some refactoring there so I will add some patches for it :)
13:16 ashimema[m] but there's still a little way to go
13:16 ashimema[m] hence I've been working through it on a big tree of bugs
13:16 ashimema[m] transfers has all sorts of issues to be honest..
13:16 ashimema[m] indeed.
13:15 kohaputti now if this get pushed there is a follow-up bug for this regarding holds in transfer state, currently anybody can "steal" others' holds that are on transfer.
13:11 ashimema[m] thanks for taking it on..
13:11 ashimema[m] pleasure
13:11 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12556 enhancement, P5 - low, ---, joonas.kylmala, Passed QA , SelfCheck machine starts the hold instantly with an email sent out
13:11 kohaputti ashimema[m], ooh, huge thanks for the QA in bug 12556 :P
12:54 koha-jenkins Project Koha_Master_D11 build #106: FAILURE in 3 min 2 sec: https://jenkins.koha-community.org/job/Koha_Master_D11/106/
12:53 koha-jenkins Project Koha_Master_U16 build #82: FAILURE in 2 min 28 sec: https://jenkins.koha-community.org/job/Koha_Master_U16/82/
12:50 huginn News from kohagit: Bug 26145: (follow-up 2) DBIC schema changes <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=909a3df890452f0f96ebd29ab571b1b8afaca475>
12:50 huginn News from kohagit: Bug 26529: (follow-up) Remove warning <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=c12ede482f5aad8111a21127e9f42c62f2e3348b>
12:49 koha-jenkins Project Koha_Master_U16 build #81: ABORTED in 9 min 2 sec: https://jenkins.koha-community.org/job/Koha_Master_U16/81/
12:49 koha-jenkins Project Koha_Master_D9_My8 build #432: ABORTED in 44 min: https://jenkins.koha-community.org/job/Koha_Master_D9_My8/432/
12:49 koha-jenkins Project Koha_Master_D10 build #368: ABORTED in 17 min: https://jenkins.koha-community.org/job/Koha_Master_D10/368/
12:49 koha-jenkins Project Koha_Master_D10_Deps build #92: ABORTED in 19 min: https://jenkins.koha-community.org/job/Koha_Master_D10_Deps/92/
12:49 Joubu I've been pretty this morning trying to fix the failures!
12:48 Joubu hum...
12:31 koha-jenkins Project Koha_Master_D9 build #1449: STILL UNSTABLE in 44 min: https://jenkins.koha-community.org/job/Koha_Master_D9/1449/
12:29 koha-jenkins Project Koha_Master_U20 build #112: STILL UNSTABLE in 55 min: https://jenkins.koha-community.org/job/Koha_Master_U20/112/
12:16 koha-jenkins Project Koha_Master_U18 build #912: FAILURE in 6 min 3 sec: https://jenkins.koha-community.org/job/Koha_Master_U18/912/
12:10 koha-jenkins Project Koha_Master_U2010 build #22: UNSTABLE in 48 min: https://jenkins.koha-community.org/job/Koha_Master_U2010/22/
12:05 koha-jenkins Project Koha_Master_D9_MDB_Latest build #407: STILL UNSTABLE in 50 min: https://jenkins.koha-community.org/job/Koha_Master_D9_MDB_Latest/407/
11:51 kohaputti The issue happened in quite complex scenario
11:50 kohaputti Gotta still come up with a test plan.
11:50 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26659 normal, P5 - low, ---, koha-bugs, NEW , An item can be renewed even if there are holds waiting to be filled
11:50 kohaputti filed bug 26659 for the renwal issue
11:47 koha-jenkins Project Koha_Master_D11 build #105: UNSTABLE in 41 min: https://jenkins.koha-community.org/job/Koha_Master_D11/105/
11:33 koha-jenkins Project Koha_Master_U16 build #80: STILL UNSTABLE in 56 min: https://jenkins.koha-community.org/job/Koha_Master_U16/80/
11:21 koha-jenkins Project Koha_Master_D10 build #367: UNSTABLE in 50 min: https://jenkins.koha-community.org/job/Koha_Master_D10/367/
11:14 koha-jenkins Project Koha_Master_D10_Deps build #91: UNSTABLE in 46 min: https://jenkins.koha-community.org/job/Koha_Master_D10_Deps/91/
11:06 koha-jenkins Project Koha_Master_U18 build #911: UNSTABLE in 42 min: https://jenkins.koha-community.org/job/Koha_Master_U18/911/
10:37 huginn News from kohagit: Bug 26145: (follow-up) DBIC schema changes <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=b3b7e07062cf910f293ccfe0dfc8dc56ee5ee4d7>
10:37 huginn News from kohagit: Bug 26145: Add exec flag on .t file <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=4b174113cf88042a0c950a9529e38504125715cf>
10:37 koha-jenkins Project Koha_Master_D9_My8 build #431: UNSTABLE in 1 hr 0 min: https://jenkins.koha-community.org/job/Koha_Master_D9_My8/431/
10:34 Joubu oops
10:30 koha-jenkins Project Koha_Master_U20 build #111: UNSTABLE in 51 min: https://jenkins.koha-community.org/job/Koha_Master_U20/111/
10:30 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15524 enhancement, P5 - low, ---, kyle, RESOLVED FIXED, Set limit on maximum possible holds per patron by category
10:29 ashimema[m] bug 15524
10:28 koha-jenkins Project Koha_Master_D9_MDB_Latest build #406: UNSTABLE in 50 min: https://jenkins.koha-community.org/job/Koha_Master_D9_MDB_Latest/406/
10:25 ashimema[m] didn't realise koha-jenkins could talk
10:25 ashimema[m] ooh
10:25 Joubu kohaputti: link it with 15524
10:25 koha-jenkins Use '!koha-jenkins help' to get help!
10:25 koha-jenkins Joubu did you mean me? Unknown command 'link'
10:25 Joubu koha-jenkins: link it with 15524
10:23 koha-jenkins Project Koha_Master_D9 build #1448: UNSTABLE in 44 min: https://jenkins.koha-community.org/job/Koha_Master_D9/1448/
10:22 ashimema[m] :'(
10:22 kohaputti ashimema[m], gotta file a bug report for that too when I get to it...
10:22 kohaputti ashimema[m], the other day I found code with Koka:: in it (supposed to be Koha::)
10:19 ashimema[m] bet it's a code path that's not tested ;)
10:19 ashimema[m] indeed.. the line is wrong.. and kohaputti's suggested fix is right :)
10:19 kohaputti ok, will file bug and attach the patch :)
10:19 ashimema[m] how strange
10:19 Joubu yes kohaputti, looks like the line is wrong
10:19 ashimema[m] certainly looks like your right to me kohaputti
10:19 ashimema[m] hmm
10:17 kohaputti but this bug has been here for many years so I'm uncertain because why would nobody else have bumped into this?
10:16 kohaputti at least our library user reports doing this code fix blocks now renewals
10:15 * ashimema[m] will read momentarily
10:14 kohaputti We just bumped into this issue that you could renew items even if there was holds and it seems to be caused by this.
10:10 kohaputti Hi, do you also notice that this line looks wrong: https://github.com/Koha-Community/Koha/blob/master/C4/Circulation.pm#L2834 Shouldn't it be next unless CanItemBeReserved($borrowernumber,$itemnumber)->{status} eq 'OK'; ?
10:06 rkrimme1 Thanks alot!
10:06 rkrimme1 Thanks alot1
09:59 cait i think most prefer koha-testing-docker at this point, I am still stuck on my old ways :)
09:59 cait rkrimme1: for testing kohadevbox or koha-testing-docker are great
09:58 cait i got to step away for a bit, will be back later
09:56 cait easy ot miss that in the many comments on the bug
09:56 cait rkrimme1: https://gitlab.com/gnucifer/koha-manual/commit/593c5da008dbc4b074cac9dee167cce9a6dfc250#c5e2c75dc923f21c5b95185b2f350df00fec661f
09:56 cait rkrimme1: did you see the documentation page they wrote? it's really helpful
09:56 ashimema[m] indeed
09:56 cait testing would still give it a push
09:51 ashimema[m] :(
09:51 ashimema[m] making sure there aren't any unintended side effects is hard
09:50 ashimema[m] I think it works.. it's more the dependency that's causing a hold up..
09:48 rkrimme1 cait: We would be willing to test it, but we may need to setup some things first for running a patched master
09:48 * ashimema[m] is tired this morning
09:41 ashimema[m] ere++, kidclamp++
09:41 ashimema[m] I've been following that one fairly closely.. although I'm not an expert in the code there I have been poking the right people to look at it whenever it seems ready to be looked at.. hehe
09:40 ashimema[m] No worries
09:40 cait thx ashimema[m] - i missed the dependency one
09:39 ashimema[m] it's certainly a good one to get working
09:39 ashimema[m] I've poked the dependency to try and get an idea of what's holding it up :)
09:37 huginn News from kohagit: Bug 26145: DB changes - Add biblioimages.imagenumber <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=ff17013a65036200fb0cb208107dc9f70778c49c>
09:37 huginn News from kohagit: Bug 26145: Add the ability to upload a cover image per item <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=c3f2993bdbbd8453c3b604c5d2cf43dc9ee27c88>
09:37 huginn News from kohagit: Bug 26145: Display the cover image at the OPAC <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=549d033cb119687c8eb75e6d2a2bfa178dbd9be6>
09:37 huginn News from kohagit: Bug 26145: Refactoring - Add tests <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=2aa6afd50ac9d806f11d78266a7527c7fd5b17a2>
09:37 huginn News from kohagit: Bug 26145: Refactoring - Rename table biblioimages to cover_images <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=b6f5768145bf21085ecd5024d6e021e4d3c64e15>
09:37 huginn News from kohagit: Bug 26145: Refactoring - Move C4::Images to Koha::CoverImages <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=7bc87ce75ec857292e433c0f6123b2ea02ef6d60>
09:37 huginn News from kohagit: Bug 26152: Remove the use of jquery.checkboxes plugin from serial collection page <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=62a569ea732cf656073e6cbb38ab7ac96df32945>
09:37 huginn News from kohagit: Bug 26154: Remove the use of jquery.checkboxes plugin from batch item deletion and... <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=30fcb92d62e574dbc1ff16c048cb1b12f653f7cf>
09:37 huginn News from kohagit: Bug 26145: Allow multi covers per item <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=a06223a20d8ce9c0dff6c706a73d13f3169d9ab4>
09:37 huginn News from kohagit: Bug 26145: Fix image tag <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=a4ca6c144142115dc3e184ae078d306c104d446d>
09:37 huginn News from kohagit: Bug 26145: Hide 'Upload image' if no CoverImage pref is set <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=176a678fbe7ee1a8484c0123358e97a5d73487c5>
09:37 huginn News from kohagit: Bug 26145: Add the "Cover image" column to the table settings <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=8cc1f02a9481682a8230f2cf57cad6193effa460>
09:37 huginn News from kohagit: Bug 26145: Correctly preserve itemnumber value <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=7478b9430661de4310862f1d29fb2f593d08879d>
09:37 huginn News from kohagit: Bug 26503: Add unit tests <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=bd2087e1cdbeeb8a207fc6881fc2472e8fdb323c>
09:37 huginn News from kohagit: Bug 26503: Add limit for standing orders to acq advanced search <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=352e8d23068caa248415a3081b50978d88e94762>
09:37 huginn News from kohagit: Bug 26145: (QA follow-up) Add missing filters <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=26eadaa7de02e7371323f06c03bfbc1c0f2aac1b>
09:37 huginn News from kohagit: Bug 26145: Add the new column to the column settings yml file <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=f4f03c04e8a1e6afc2160bc24116b6f2a8ed72a1>
09:37 huginn News from kohagit: Bug 26145: DBRev 20.06.00.049 <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=3611a2cd6819fee960aacffd5f5f3dcc09d8a7ee>
09:37 huginn News from kohagit: Bug 26145: DBIC schema changes <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=5ee3b6bbb2f8611b869494e0723918f6bf0bead8>
09:37 huginn News from kohagit: Bug 26519: Compiled CSS <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=d50332c1e57bfe623ce2f7c6ce427dbd12990062>
09:37 huginn News from kohagit: Bug 26145: Compiled CSS <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=d2c59eda8553432df3fae8b001d287eb9210f594>
09:36 ashimema[m] looks like David put that one into discussion.. not sure why
09:36 ashimema[m] hmm
09:35 ashimema[m] so you suggested cait
09:35 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25539 normal, P3, ---, glasklas, In Discussion , Remove AddBiblio "defer_marc_save" option
09:35 ashimema[m] blocked by bug 25539 apparently
09:34 ashimema[m] ah yes..
09:34 cait i think basically it needs the testing - sign off and QA
09:34 ashimema[m] but I can't see what
09:34 ashimema[m] I tried to help get it moving again, but I thought something was blocking it..
09:34 cait testing
09:34 ashimema[m] I can't remember what was holding that one back
09:34 cait i am not sure
09:33 ashimema[m] glasglas.. is that his irc nick
09:33 cait glasglas has been great about rebasing it when asked, but he might be more inclined to do so when someone promises testing
09:32 cait do you mean it doesn't apply cleanly atm?
09:32 cait sadly, not that i know of
09:30 huginn Bug 14957: enhancement, P5 - low, ---, glasklas, Patch doesn't apply , Write protecting MARC fields based on source of import
09:30 rkrimme1 Hi, is anyone currently working on https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14957 ? It says it's not compatible to be merged to current master.
07:16 magnuse hiya ashimema[m]
07:14 ashimema[m] Mornin' #koha
07:13 magnuse ~o~
06:49 wahanui kia ora, alex_a
06:49 alex_a Bonjour
06:35 reiveune hello