Time Nick Message
01:12 aleisha hi
06:45 reiveune hello
06:56 Joubu @later tell tcohen "There is no minutes for this meeting - Development_IRC_meeting_5_July_2023"
06:56 huginn Joubu: The operation succeeded.
07:22 cait hello #koha
08:03 krimsonkharne[m] morning #koha
09:02 paulderscheid[m] morning #koha
09:15 * cait waves
09:17 oleonard o/
09:19 cait hi oleonard
09:19 wahanui hi oleopard
09:32 ashimema anyone fancy QAing bug 34269 ?
09:32 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34269 major, P5 - low, ---, martin.renvoize, Signed Off , Regression in circulation rules for 'similar' patron categories
09:33 * ashimema is having a QA morning so happy to return the favour
09:43 ashimema with the new tidying practice
09:43 ashimema how thorough are we being about only tidying the relevant lines vs tidying a nearly tidy file as the whole file?
09:50 ashimema cait around?
09:50 wahanui cait around is cait cait1 cait2 cait3 cait4 cait5
09:51 cait yes
09:51 ashimema the failing test on 33028
09:51 cait yes?
09:51 cait I couldn't look yesterday, it was too late when I saw it
09:51 ashimema I'm trying to work out if the test highlights an issue or if the test needs fixing
09:51 cait and pretty much booked today
09:52 cait one idea i had:
09:52 cait can you verify if the test still fails with Kyle's linked bugs for fixing overduefindescap behavior?
09:52 ashimema the core question is..
09:53 ashimema should we allow an empty string for 'fine'
09:53 cait yes
09:53 cait hm, well
09:53 ashimema emptry string vs undef vs 0 vs 0.00
09:53 ashimema which should be valid
09:53 cait we have the same iissue for overduefinescap
09:53 ashimema that's what this test is catching on
09:53 cait because empty = unlimited
09:53 cait similar discussion
09:54 cait maybe a similar fix too
09:54 cait you can now enter 0 and it will update it to show as empty voer the overdue fines cap
09:55 cait I don't have a strong opinion for the fine one
09:55 cait or not sure what is best, do you have a feeling?
09:56 ashimema I really don't know
09:56 cait I am leaning towards we don't need to allow fine to be 0
09:56 ashimema I'll look at Kyles bugs
09:57 ashimema it was looking at those that reminded me I needed to come back and fix this test π
09:57 cait I wonder how it wroked before the move... and it probably stored 0.000 when the field was still decimal in the db (issuingrules table)
09:57 ashimema it's not just about fine really
09:57 cait so we might be safer with 0 than with empty/undefined
09:57 ashimema it just happens to be what this test picks as a test against
09:57 cait for the overduefinescamp we needed to bring back the empty/0 behavior, because you ened to be able to not limit
09:57 cait and it also caused issues because it reset existing empties to zero, breaking the fine rules
09:58 ashimema yikes
09:58 cait so as we had empty before, you edited a rule, ti would save as 0 and no more fines accruing
09:59 ashimema is there an order to Kyles two bugs
09:59 cait it came up ont he mailing list a lot, I want his fix finally
09:59 ashimema should I look at one before the other?
09:59 cait let me see
09:59 MatthewBlenkinsop[m] o/
09:59 ashimema sorry Matthew Blenkinsop, I'm filling up your queue again π
10:01 cait ashimema: i think bug 32271 and bug 34279 can be tested indepently, but I probably tested as: 33028 - 32271 - 34279
10:01 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32271 major, P5 - low, ---, geraud.frappier, Signed Off , Overdue fines cap (amount) set to 0.00 when editing rule
10:01 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34279 major, P5 - low, ---, kyle, Signed Off , overduefinescap of 0 is ignored, but overduefinescap of 0.00 is enforced
10:01 ashimema ta
10:02 cait I'd realyl love the see the circulation fixes make it, it's a najor udpate headache
10:02 ashimema hmm.. so we drop the price formatting in bug 32271 I see
10:02 ashimema yup
10:02 ashimema that's why I'm devoting time to them now
10:02 ashimema one last push to get them over the line
10:03 ashimema aha, I see
10:03 ashimema he adds 'can_be_blank', which is nice and what I was thinking
10:03 ashimema cool, that works for me
10:03 ashimema I think for 'fines' though, blank is wrong
10:04 ashimema we don't need to distinguish between 0 and blank/undefined
10:04 ashimema 0 means "don't charge anything", blank wouldn't have any meaningful use
10:05 ashimema what does bug me though.. I never wrote unit tests for this module change
10:05 ashimema and no-one caught that!
10:05 ashimema my mistake originally of course.. it got lost in all the work we did
10:05 ashimema but still..
10:16 oleonard cait: Do you remember how to reproduce the problem described in Bug 34072?
10:16 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34072 normal, P5 - low, ---, lucas, Pushed to oldstable , Holds queue search interface hidden on small screens
10:16 oleonard The bug describes "search controls." Does that mean the sidebar filter?
10:36 cait all I did was to squash teh browser window
10:36 cait yes the filters disappeared I think
10:40 oleonard For me the filters just bump down to below the table, which is to be expected
10:40 Annelisterman[m] oleonard: same here with Firefox
10:41 oleonard ...and I don't see any difference before and after the patch
11:15 marcelr o/
11:16 marcelr ashimema: you are welcome to qa 33608 ?
11:18 ashimema I'll add it to the list π
11:18 ashimema just wrapping my head around this circ rules money bug still at the minute
11:20 marcelr ok no worries just trying
11:21 ashimema I will certainly take a look
11:21 ashimema π
11:21 marcelr great
11:42 ashimema how close to circ rules and money and ... are you marcelr
11:43 marcelr i could have a look
11:44 ashimema I wouldn't mind a second pair of QA eyes on bug 32271
11:44 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32271 major, P5 - low, ---, geraud.frappier, Signed Off , Overdue fines cap (amount) set to 0.00 when editing rule
11:44 marcelr i try to have a look today
11:45 ashimema I'm pretty confident in it now.. but I've ended up with three follow-ups.. so I feel like a final run would be beneficial
11:45 marcelr the more follow-ups the more fun :)
11:45 ashimema Unit tests for the win π
11:45 cait at leat if it still works i the end ;)
11:48 cait ashimema++ marcelr++
11:51 marcelr dont forget cait++
11:51 marcelr anyone else ?
11:52 cait possibly, I'll think about it in my next meeting
11:52 marcelr thinking and meetings
11:57 cait ok, yes... maybe not a good idea :)
12:04 tcohen hola #koha o/
12:04 tcohen hi marcelr
12:05 tcohen marcelr, ashimema should we allow passing 1.1 ? or do we enforce 1.10?
12:06 tcohen /^\d+(\.\d{2})?$/
12:06 tcohen -> /^\d+(\.\d{2})?$/
12:09 marcelr hi tcohen
12:09 marcelr i should first have a closer look there
12:09 marcelr but i would tend to 1.10 or something configured for it
12:10 tcohen -> /^\d*(\.\d+)?$/
12:10 tcohen ^^ that one would cover any number
12:10 tcohen with the dot conditional to a decimal
12:10 marcelr yes it covers empty too
12:11 tcohen and the regex is trying to avoid 1,10 if I understand correctly
12:11 tcohen i.e. using an invalid (db-wise) separator
12:11 tcohen ashimema?
12:11 wahanui i guess ashimema is back on security bugs today.. sorry, it's been a bit of a crazy week
12:12 tcohen Koha::Exceptions::CirculationRule::NotDecimal->throw( name => $rule_name, value => $rule_value )
12:12 tcohen - if ( $is_monetary && defined($rule_value) && $rule_value !~ /^\d+(\.\d{2})?$/ );
12:12 tcohen + if ( $is_monetary && defined($rule_value) && $rule_value !~ /^\d*(\.\d+)?$/ );
12:15 tcohen Maybe we should use
12:15 marcelr allowing .3 instead of 0.30 ?
12:15 tcohen perl -MScalar::Util -e 'print "yes\n" if Scalar::Util::looks_like_number("1.10");';
12:15 tcohen perl -MScalar::Util -e 'print "yes\n" if Scalar::Util::looks_like_number("1,10");';
12:15 marcelr looks_like_monetary ?
12:16 tcohen if we force two decimals, then we need to do some trick on the controller (like +0.00 or similar)
12:16 marcelr yeah we should probably not need to do it
12:16 marcelr only in display
12:22 tcohen this is an extra check
12:23 tcohen so any bug in the controller doesn't make the bad value reach the DB
12:23 marcelr do we currently have currencies with more decimals than 2 in Koha ?
12:24 tcohen I don't think so
12:24 tcohen the (tiny) problem as I see it, is that the regex forbids one decimal
12:24 marcelr yes
12:25 Joubu cannot pay fines in bitcoin, too bad
12:25 marcelr but if we do not have monetary > 2, why allow 4 ?
12:25 tcohen Joubu: exactly :-P
12:26 marcelr and the trend is stop with fines altogether
12:30 marcelr we should first define the number of decimals per currency if anyone wants to go there
12:30 marcelr now we dont know
12:30 marcelr just assume 2
12:32 Joubu we always assumed 2
12:32 Joubu git grep decimal_fill Koha/Number/Price.pm
12:33 marcelr yeah no problem for me, but we try to allow .3 and 1.1 too now
12:33 ashimema sorry, was grabbing lunch
12:33 marcelr couldnt we catch missing decimals at the client side ?
12:34 marcelr but we need at server too obviously
12:34 marcelr ashimema: just read what tomas said ^^
12:35 ashimema at some point I think we allowed 3 decimals
12:35 ashimema though I can't remember which currency it was that used that
12:36 marcelr we do not registrate the number of decimals per currency
12:36 ashimema inded
12:36 ashimema indeed.. even
12:36 ashimema as for the regex.. I think I just stuck with the same one we've been introducing on the UI everywhere
12:36 ashimema I didn't come up with it.
12:36 ashimema cait ^
12:37 ashimema where did the pattern match you've been using all over the place come from in the first place?
12:37 marcelr lol
12:37 ashimema Iraqi Dinar has 3 decimals
12:37 Joubu are you aware of bug 34169?
12:37 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34169 enhancement, P5 - low, ---, katrin.fischer, Signed Off , Add validation for monetary input fields in acquisition module
12:37 marcelr they are probably flattened all over
12:37 ashimema as does Jordanian Dinar
12:38 marcelr + number: _("Please add amount in valid format: 0.00"),
12:38 ashimema she's done it differently there
12:38 Joubu this is the correct way
12:39 Joubu it's client-side only however
12:39 ashimema type="text" inputmode="decimal" pattern="^\d+(\.\d{2})?$"
12:39 ashimema that's what's been getting added all over the place
12:39 ashimema I was worried it would block more than it should
12:40 marcelr we shouldnt add them like that
12:40 marcelr we will be mad when the Dinar comes in
12:40 Joubu I am entering into a discussion I don't want to be involved, I am disapearing right now :D
12:40 marcelr same for us all Joubu
12:41 marcelr i sense an In Discussion coming
12:41 ashimema grr.. this is a major bug
12:41 ashimema again
12:41 Joubu you are too many already anyway. Just saying I've talked with cait last week and what is on 34169 seems correct for me.
12:41 ashimema it can't go in discussion.. again
12:42 marcelr i think we could go with it; if you entered .1 and 2.2 twice you will learn it at some point
12:43 marcelr just postponing the whole 3 decimals stuff to the future
12:43 marcelr can we squash those patches ashimema
12:43 marcelr it is like add remove and add etc
12:46 ashimema `/^(?:-?\d+|-?\d{1,3}(?:,\d{3})+)?(?:.\d+)?$/
12:46 ashimema * `/^(?:-?\d+|-?\d{1,3}(?:,\d{3})+)?(?:.\d+)?$/`
12:46 ashimema that's what jQuery validator is using internally
12:46 ashimema yup.. I was leaving that with the RM
12:46 Joubu if we have centralise the checks and the logic, the format will not be difficult to modify later
12:46 ashimema 33028 was a fecking nightmare to work through
12:46 ashimema indeed
12:47 tcohen so, in 330.28, I'll just update the value in Circulation.t so it doens't fail, and we leave it as-is
12:49 marcelr so you can go ahead, ashimema, and pass qa on the 32271 then ?
12:50 tcohen what a twist ROTFL
12:50 marcelr seeing some timethis stuff btw
12:51 ashimema thanks marcelr π
12:51 marcelr tcohen: Koha bends like no other
12:52 marcelr next problem please :)
12:58 tcohen marcelr: show we disable FK checks when changing column names for fks?
12:58 tcohen should
12:59 marcelr i think we should; at some sql versions it could somehow crash
12:59 tcohen mariadb 10.4 => 211200048.pl fails
12:59 marcelr i looked for a report but didnt find it
13:00 tcohen mentioning it on an unrelated bug doesn't generate a new bug report?
13:00 tcohen where's the AI when we need it?
13:01 marcelr right
13:01 marcelr we need a bot here to create bugzilla reports
13:04 Joubu as you are (almost) all around, I would like to know how I could get your attention on those 3 bugs: bug 33568, bug 33940 and bug 30825
13:04 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33568 enhancement, P5 - low, ---, jonathan.druart+koha, Needs Signoff , Use the REST API to display items on the biblio detail view
13:04 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33940 enhancement, P5 - low, ---, jonathan.druart+koha, Needs Signoff , Move get_cardnumber_length and checkcardnumber to Koha
13:04 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30825 enhancement, P5 - low, ---, tomascohen, Needs Signoff , Get rid of GetReservesControlBranch
13:04 ashimema those bugs excite me
13:04 Joubu I am willing to trade SO or QA for them
13:04 ashimema I just haven't had a moment βΉοΈ
13:04 Joubu they are awesome, really
13:04 Joubu there is as well bug 33940
13:05 Joubu I gave it already
13:05 marcelr 33940 sounds intriguing
13:05 Joubu I meant bug 33947
13:05 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33947 enhancement, P5 - low, ---, jonathan.druart+koha, Needs Signoff , Move GetAllIssues to Koha
13:05 Joubu really, please help them, they are big and it won't be easy to rebase endlessly
13:06 Joubu and I am lacking energy for rebases nowadays...
13:07 Joubu 33940 is introducing a concept of a Koha::Policy namespace
13:07 marcelr i should go to that SO and QA trade market one day
13:07 Joubu that I am using in bug 30825 too
13:07 marcelr code related to 33940 is a nightmare iirc
13:07 Joubu make your list! :D
13:08 marcelr just look for my name in NSO and SO :)
13:08 Joubu *was* a nightmare, it's now nice and friendly
13:08 marcelr Joubu++
13:08 Joubu it's using Koha::Result::Boolean and Koha::Object::Messages
13:09 Joubu and Bug 33947: Remove GetAllIssues, did you read that? REMOVE **GetAllIssues**!
13:09 * ashimema looks at bug 33608 as promised now.. then will come to Joubu's list
13:10 Joubu you all should be excited by that! :D
13:10 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33608 enhancement, P5 - low, ---, m.de.rooy, Signed Off , Allow to get statistics about found/recovered books
13:10 tcohen I am, Joubu :-D
13:10 Joubu AND! **Use the REST API to display items on the biblio detail view**
13:11 Joubu really, they are great, lot of code and changes, but there going where we want to go
13:16 * ashimema contemplate
13:16 ashimema is `!$updated_columns{itemlost} && $pre_mod_item->itemlost`
13:16 ashimema really the same as
13:17 ashimema `$updated_columns{itemlost} <= 0 && $pre_mod_item->itemlost > 0`
13:17 ashimema don't we have functionality tied to negative, 0, positive integers there?
13:19 marcelr which patch exactly ashimema
13:19 ashimema https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=33608&attachment=151916
13:19 ashimema Bug 33608: Add UpdateStats to item->store
13:19 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33608 enhancement, P5 - low, ---, m.de.rooy, Signed Off , Allow to get statistics about found/recovered books
13:20 marcelr this is just rewriting what was there already
13:20 ashimema yes.. except I think the logic is changed
13:20 marcelr adding the other situation
13:20 * ashimema wonders if there's good test coverage here
13:22 marcelr the other patches are moving tests
13:22 * ashimema is currently looking at the docs to understand if there's any logical reason for the numerical comparison vs just truthyness
13:22 ashimema the docs feel like we're safe
13:23 ashimema they only talk about 0,1,2,3,4 as values
13:23 ashimema though of course people can add their own
13:23 ashimema I can't see why you'd want to treat -1 differently
13:24 ashimema one question I do have though..
13:24 ashimema did you want to record the itemlost value in your stats.. or just the fact it was lost or found
13:24 marcelr need to get that code back in my mind
13:25 ashimema right now you just add 'item_lost' or 'item_found'
13:25 marcelr the purpose is the lost and found
13:25 marcelr mainly
13:25 marcelr not the value specifically
13:25 ashimema but of course.. itemlost can take various options
13:25 ashimema okies
13:25 ashimema thanks
13:28 Joubu I would not change the logic if it's not the goal of the patch
13:30 marcelr i need to add an else branch there
13:31 marcelr to differentiate lost and found again
13:32 ashimema I think the logic change is sound based on the community manual
13:32 ashimema but I may be missing something behind the scenes.. some hidden functionality
13:32 ashimema i.e. around when we trigger the trigger
13:32 ashimema we will trigger it more often now than before
13:32 ashimema any lost value other than zero will trigger it now
13:32 ashimema before it was only a change to positive integer
13:33 marcelr we ignored the found again situation
13:33 ashimema oop, other way around
13:33 ashimema we ignored negative lost to unlost changes
13:33 ashimema not we catch those as well as positive lost to unlost values
13:34 marcelr i see your point about >0; i am catching <>0 but is just theory we dont have negative values
13:34 marcelr and i think if we have we should catch them too
13:35 marcelr that was the reason
13:35 ashimema I dunno.. was there a use case for it at some point
13:35 ashimema as I said.. I can't see any from the docs right now.. but I'm not a Koha historian and the docs have been known to miss things π
13:35 marcelr negative lost not that i know of
13:36 marcelr negative notforloan is something else
13:36 marcelr if you still miss a test, let me know
13:36 marcelr unit test
13:39 Joubu this is one for cait or kidclamp, I think they know what to do with negative values
13:39 marcelr cant find any other examples so quickly now that test for >0 on itemlost btw normally we just test <>0
13:39 marcelr truthy
13:51 Kolvir Iβve been trying to connect to koha via a ssh tunnel and get the koha maintenance page. I've confirmed that the server is running correctly locally. ssh -L 8080:localhost:80 -N -f -l <user> <ip address> .
13:55 Kolvir Would appreciate any help that would let me connect to the remote server. So close, must be something simple.
13:58 tcohen Feliz cumpleaΓ±os, gmcharlt
14:02 ashimema hmm
14:02 marcelr ?
14:02 ashimema is the tidy check in the qa scripts working well for everyone else?
14:03 marcelr yeah it is not tidy yet probably..
14:03 marcelr written before
14:03 ashimema indeed
14:03 marcelr had no problem with qa tools
14:03 ashimema but I just added a follow-up that should have fixed the tidy issues
14:03 ashimema yet it still fails
14:03 ashimema which is a bit odd
14:03 ashimema second patchset like it for me too
14:04 ashimema I kind of wish you'd split this bug.. cleanup and feature distinctly
14:05 marcelr strange
14:05 ashimema the new + insert methods feel inconsistent with elsewhere in Koha to me
14:05 ashimema not outright wrong.. they work
14:05 ashimema but the inconsistency feels strange
14:05 marcelr the insert is a way to prevent changing lost of calls
14:05 marcelr lots
14:05 ashimema mm
14:06 marcelr it is pragmatic
14:06 ashimema yeah, I see
14:07 marcelr touching code everywhere isnt nice either with backporting etc etx
14:07 Joubu the test is not perfect, and don't do magic. It simply count the number of lines not tidy before and after, then compare. So you can easily cheat it by removing more lines and add some lines that are not tidy, the test won't catch them
14:07 * ashimema is just running the QA tests again but hopefulyl it's all good now
14:07 ashimema then I need a swim
14:07 marcelr qaers fooling qa tools, how bad is that ?
14:07 Joubu and, again, it's a WARN, not FAIL. And it has been pushed to get feedback ;)
14:08 marcelr well there you have it :)
14:09 marcelr you should read it as "we like it " :)
14:17 marcelr perltidy++
14:18 marcelr ashimema++
14:18 marcelr ashimema: i already opened bug 33636
14:18 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33636 enhancement, P5 - low, ---, m.de.rooy, ASSIGNED , Move C4::Stats to Koha namespace
14:19 Kolvir Does anyone know if it is an issue with the tunnel, or is is a setting in koha or apache that is redirecting me to the maintenance page?
14:19 Joubu ashimema, marcelr: why keeping the logic change?
14:20 mtj hi Kolvir: your problem is probably caused by apache
14:20 marcelr why keep it if it is not used in Koha elsewhere Joubu?
14:20 Joubu Kolvir: go to the staff interface and run the update process, on 8081
14:20 mtj hi Joubu :)
14:21 Joubu marcelr: we are talking about Koha::Item->store, be careful there..
14:21 Joubu mtj: hi!
14:21 marcelr a nonzero lost means that it is lost in some way
14:21 marcelr probably negative losts are not very common either
14:21 Joubu you are introducing a behaviour change in a patch that is doing something else
14:22 Joubu *probably*...
14:22 Joubu Are you new in Koha?
14:22 mtj Kolvir: your ssh tunneling is easier if you set koha opac and staff to different port numbers (8080 and 8081)
14:22 marcelr yes i could revert that tiny change if you insist
14:22 ashimema I PQA'd and mentioned for the RM π
14:22 marcelr i will change it and add another test
14:23 marcelr will block it and put it back to pqa
14:23 ashimema okies
14:23 ashimema school run time for me.. bbiab
14:24 marcelr the second concern is harder ashimema
14:24 marcelr move it to the followup
14:24 Joubu commit dd1eec2715b465
14:24 Kolvir I can get to the staff interface via ssh and x11 forwarding, but performance is really bad
14:27 mtj Kolvir: quick fix is to set your staff to port 8081, to access it
14:27 Kolvir what file is that setting in?
14:27 mtj ...or add some entries to your local pc's /etc/host files
14:28 Joubu marcelr: and initially coming from commit 0005f6bad36319e11a1b0c561403e75758a803af, so yes, it does not make a lot of sense
14:28 Kolvir looking in /etc/koha/
14:29 mtj Kolvir: vi /etc/apache2/sites-available/mykoha123.conf
14:29 Kolvir ty
14:31 mtj $ sudo /etc/init.d/apache2 restart
14:31 Kolvir # OPAC 4 <VirtualHost *:80> here or # Intranet 22 <VirtualHost *:80>
14:32 Kolvir 4 and 22 are line numbers
14:34 mtj 22
14:36 mtj Kolvir: you need to add 'Listen 8080' to /etc/apache2/ports.conf
14:38 Kolvir leave listen 80 in there?
14:38 mtj hmm, sure :)
14:39 Kolvir I'm not a network guy, if it hasn't been made clear by my ignorance ;). Am, however a programmer.
14:39 mtj aah, we are setting staff to 8081 yes?
14:39 Kolvir yes
14:39 Kolvir listen 8081 then?
14:39 mtj yeah
14:40 mtj $ ssh -L 9999:localhost:8081 -N -f -l <user> <ip address>
14:40 mtj $ lynx http://localhost:9999
14:41 mtj ^ you should tunnel to staff interface on 8081
14:41 Kolvir there an eaiser way to kill a tunnel than netstat and kill the PID?
14:42 mtj i would kill with pid :)
14:42 Kolvir just seemed clunky...
14:44 Kolvir That did it! Thank you so much. I have been trying to get remote access to this machine for over a year. Volunteer for a church library that I'm setting up with koha. about 6,000 items and not even a manual card catalog. Been a 'process' to say the least.
14:45 Kolvir Thank you so much. Koha community has been great.
14:45 mtj Kolvir: congrats
14:45 Kolvir I started the project with evergreen and community support was silent.
14:46 ashimema everygreen is big and scary for the project your size
14:46 Kolvir So glad I found koha. I know it is overkill for our needs, but we don't have the budget for $600+/year license the commercial offerings
14:46 ashimema I used to support evergreen.. we decided pretty much no-one was big enough to require everygreen here at the time so we dropped support and went Koha only
14:47 mtj Kolvir: if you really need to access many sites on a single port, you can add your hostnames to your local /etc/hosts file
14:47 Kolvir Now if I can find a way to modify the Quick spine label to support my zebra printer, I'd be really rolling.
14:49 mtj echo '127.0.0.1 staff1.mykoha.com' >> /etc/hosts
14:49 mtj echo '127.0.0.1 staff2.mykoha.com' >> /etc/hosts
14:49 mtj lynx http://staff1.mykoha.com:9999
14:49 mtj lynx http://staff2.mykoha.com:9999
14:50 Kolvir It has been a huge project for me. Been really hard to figure out the workflow and organize volunteers. Having no card catalog at all has made it rough, especially as I'm a tech guy, not a librarian.
14:50 Kolvir That for my remote system or the library server?
14:52 mtj Kolvir: add your hostnames to your local /etc/hosts file
14:54 Kolvir Does Koha have any python code ? I'd like to contribute at somnepoint, but perl makes my eyes bleed and I don't know JS yet. I'm an old midrange developer, but my SQL is very rusty.
14:59 Kolvir Again, thank you. This has made my life much easier. I live about 40 min from the library, so this will allow me to get so much more done.
14:59 Joubu no python, but if you are a python developer you will manage to read our code
14:59 Joubu we are trying to make it readable ;)
15:00 Kolvir Not your perl I was referring to but perl in general!
15:00 Kolvir I used to program RPG/ILE, and even, Lord help me, COBOL.
15:03 marcelr cobol++
15:03 marcelr you are in the right channel Kolvir lol
15:05 Kolvir I may have to pick up Java Script soon for a job, between that and SQL I'll see what I can do down the road. Thanks again, I have to head out soon.
15:09 Joubu oleonard++ thanks!!
15:10 oleonard ππ»
15:18 Joubu tcohen: we broke ktd startup, see jenkins
15:18 Joubu 17:16:40 koha_1 | Exception 'Koha::Exceptions::CirculationRule::NotDecimal' thrown 'The circulation rule expected a decimal value' with name => overduefinescap, value =>
15:18 Joubu we need to fix misc4dev
15:18 Joubu I think it's this one:
15:19 Joubu ./insert_data.pl
15:19 Joubu 141 overduefinescap => "",
15:19 Joubu sorry I have to run right now! I will fix it tomorrow if it's not done yet (but it must be done before :D)
15:20 tcohen oook
15:20 tcohen thanks Joubu
15:23 ashimema yowsers.. I didn't even consider that side effect at all
15:23 tcohen I'm wondering...
15:23 tcohen why a reset_all is not failing
15:24 tcohen as it is wiping the DB and loading it all again
15:31 oleonard Idea for Bug 26258: https://snipboard.io/G9fFmo.jpg
15:31 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26258 enhancement, P5 - low, ---, koha-bugs, NEW , Circulation tabs inconsistent with counters
15:33 caroline oleonard, what does the blue and yellow mean?
15:34 caroline I like that if it's 0, there is nothing
15:34 oleonard Blue is neutral, yellow indicates that something might need action (unresolved return claims)
15:35 oleonard I can see using red for the restrictions count
15:36 caroline And were losing the number of resolved claims, is that intentional? Or is it because it's currently 0?
15:37 caroline I definitely like this more that what it was. The inconsistency was bugging me
15:37 oleonard It's intentional because I don't think the count of resolved claims is high-priority information
15:37 caroline +1
15:38 oleonard I was undecided about the count of clubs
15:38 caroline Unless its higher that ClaimReturnedWarningThreshold maybe?
15:39 caroline Yes I'm not sure about the clubs either... is it really useful to know there are 14 clubs going on? There is not limit on patron categories or age for the clubs, so it will always show the total number of clubs
15:40 oleonard I wasn't aware of the ClaimReturnedWarningThreshold preference. That counts both resolved and unresolved?
15:41 caroline I'm not sure... let me check the manual, maybe the info is there
15:44 caroline The manual says nothing, but the patron_messages.inc has
15:44 caroline [% IF return_claims.count > ClaimReturnedWarningThreshold %]
15:44 caroline I think it's all claims no matter the status
15:45 caroline And it would be all claims within a certain time period only, since we can use clean-up database to delete old ones
16:32 oleonard caroline: It could work like this: https://snipboard.io/mJN3rj.jpg
16:33 caroline So did you find out if it was only resolved claims?
16:36 emlam ClaimReturnedWarningTheshold is indeed all claims no matter the status
16:36 oleonard Where does it get used?
16:37 emlam as far as I know, the only thing it does is post a warning message on the patron account page that they have too many claims (but doesn't restrict their account in any way)
16:37 caroline it's meant to show a message. patron_messages.inc
16:37 ashimema i tried to remove that 0 from claims before
16:37 ashimema and got told off
16:37 ashimema can't remember the bug
16:37 ashimema but I tried hard to clean that area once before
16:38 oleonard Yeah I think I remember ashimema
16:38 ashimema I do like your move towards all being labels though.. that's nice
16:50 ashimema can't find the bug though
16:50 ashimema oh well.. I'm sure someone will pipe up
16:50 ashimema anywho.. it's dinner time here..
16:50 ashimema have a good evening all
16:55 caroline ashimema, maybe bug 26264 ?
16:55 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26264 enhancement, P5 - low, ---, oleonard, RESOLVED DUPLICATE, Styling of 'counts' on patron details tabs is inconsistent
17:09 reiveune bye
17:10 ashimema I'm sure there was a fair bit of discussion on whatever bug it was
17:10 ashimema Some that explained why people weren't keen to drop the two numbers
17:19 ashimema Found the discussion
17:19 ashimema https://irc.koha-community.org/koha/2020-08-21#i_2280071
17:19 ashimema From there onwards
17:19 ashimema I think I just gave up
17:43 tcohen @later tell Joubu the jenkins problem was highlighting an issue bug 34279 solves. So not a misc4dev bug
17:43 huginn tcohen: The operation succeeded.
17:43 tcohen tests++
19:44 caroline Does CustomCoverImages work with two placeholders? I'm trying to get PG images to test and the URL has the control number twice (1736 in the example) https://www.gutenberg.org/cache/epub/1736/pg1736.cover.medium.jpg
19:45 caroline So CustomCoverImages should be https://www.gutenberg.org/cache/epub/{001}/pg{001}.cover.medium.jpg
19:45 caroline But I can't get it to work.... I'm wondering if the problem is me or the pref
19:46 caroline oleonard's URL in bug 28235 only has one substitution
19:46 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28235 normal, P5 - low, ---, oleonard, RESOLVED FIXED, Custom cover images are very large in staff search results and OPAC details
20:03 DarusSalamSeminaryLibrary[m] hello
20:06 DarusSalamSeminaryLibrary[m] first time setting up Koha at our library, we are having some trouble with automating the email notification system for patrons. any assistance would be much appreciated
20:19 caroline DarusSalamSeminaryLibrary[m], could you be more specific on what the problem is?
21:50 aleisha hello
22:03 DarusSalamSeminaryLibrary[m] <caroline> "3muskateers, could you be more..." <- For some reason, patrons are not receiving any emails whenever books are checked out or in, even though under "patron messaging preferences" it is set to receive all notices. The "notices" tab also shows that the notice is pending but it remains that way indefinitely. There is no error or failure to send message warning. Hope this clarifies the issue and any help is
22:03 DarusSalamSeminaryLibrary[m] appreciated