Time Nick Message
23:01 koha-jenkins Project Koha_Master_U20 build #114: SUCCESS in 46 min: https://jenkins.koha-community.org/job/Koha_Master_U20/114/
21:21 koha-jenkins Project Koha_Master_U20 build #113: FIXED in 46 min: https://jenkins.koha-community.org/job/Koha_Master_U20/113/
21:21 koha-jenkins Yippee, build fixed!
21:04 tuxayo I'll remember this term to complement another one: «historical reasons»
21:04 tuxayo > organic growth
20:58 ashimema Starting with a metric tonne more tests...
20:57 ashimema AddReturn is a scary place... Let's call it organic growth... One day I'd love to have a solid reason/funding/excuse to work through cleaning it up like you say..
19:40 * philor considers playing the Dumb Librarian Fixing Obvious Bustage In Totally Untested Code card, and just skipping the test entirely
19:33 pastebot "philor" at 127.0.0.1 pasted "Why won't this bloody test fail?" (55 lines) at http://paste.koha-community.org/558
18:31 philor but more immediately, I want a unit test that checks on a lost item charge, so I can copy-paste it without having to figure out how to do it on my own
18:26 philor or maybe what I want it for it not to set $doreturn = 0 for the not-checked-out case, so that $doreturn = 0 only means withdrawn && blockreturnofwithdrawn || lost && blockreturnoflost, so things can just all be if( $doreturn )
18:18 philor and because of where we check $doreturn, for a while I thought I'd then be fixing sending a message to the patron saying the item was checked in when it's blocked by being lost or withdrawn, but we don't do that, we just start to do it and then stop
18:16 philor then next I'll be fixing the way that if you block return of lost items, we'll still fill a hold with your lost item while leaving it lost
18:16 philor I'm currently fixing the unobvious way that if you block return of withdrawn items, and block return of lost items, and AddReturn a lost and withdrawn item, it'll stay lost and withdrawn but the charge for replacementprice will be removed
18:15 philor tcohen: I want all the side-effects to be clear, and to have it be clear what things should and should not happen when $doreturn = 0
18:11 tcohen philor what do you want?
17:44 tuxayo lol nope
17:43 * tuxayo checks AddReturn
17:13 philor changed my mind, once I saw that the current state where comments are below the line they refer to, or have no relationship to any line of code, is actually from the last refactor
17:10 oleonard-away philor: Somebody must be done by now, eh?
16:20 philor could somebody just quickly entirely refactor AddReturn for me? kthx
14:59 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28385 critical, P5 - low, ---, dpavlin, Signed Off , ldap server configuration broken since migration from XML::Simple
14:59 tuxayo ashimema: Hi :) It seems you did the analysis of bug 28385, not sure if all the questions have been resolved. If so, since I signed it off, you might be able to QA it.
14:46 reiveune bye
14:38 tuxayo hi koha o/
14:16 magnuse nah, too tired to fix anything now, see y'all on tuesday!
13:41 ashimema Enabling this may remove modules which are only present for the purposes of preloading or which aren't being detected for other reasons, so use this setting with care.
13:41 ashimema When enabled, unused modules will be removed. Defaults to preserving unused modules.
13:41 ashimema --[no-]preserve-unused
13:36 ashimema agreed
13:36 Joubu README says that it does not remove useless import but an option would be nice actually :)
13:36 Joubu which means Encode should be removed it seems
13:36 Joubu ashimema: +use Encode ();
13:35 Joubu the script decides to chmod...755 to 644
13:35 magnuse oleonard: i see it on a 20.05.07 install
13:35 Joubu ashimema: yes but we need to reach a stable state first
13:34 ashimema I like the idea of tieing it to the QA script ;)
13:34 magnuse oleonard: it is missing when i choose english langauge too, so not a translation problem. nothing in the js console...
13:34 ashimema yeah, would be nice to have both
13:33 Joubu 17600 also dealt with with EXPORT
13:33 ashimema I ran against master
13:33 ashimema oh really.. interesting
13:33 ashimema haha..
13:33 Joubu the script is still running here, I ran on top of 17600
13:33 magnuse oleonard: lucky you :-)
13:33 Joubu ashimema: it's more or less what 17600 did actually :)
13:32 oleonard magnuse: I see it in 20.11.05 so I'm not sure what's going on
13:30 magnuse i see it on other tables, like on pendingreserves.pl
13:29 magnuse i know i can use the form on the left to search, but that little js-search is so much more responsive...
13:28 magnuse oleonard: staff
13:28 ashimema results are interesting using perlimports Joubu.
13:27 oleonard-away magnuse: OPAC or staff interface?
13:25 magnuse on 20.11.05 i am missing the little box for filtering/searching ILL requests in the row of "table controls" right above the main table of ill requests. was that by design, or is it a bug?
12:46 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17600 enhancement, P5 - low, ---, jonathan.druart+koha, In Discussion , Standardize the EXPORT
12:46 Joubu bug 17600
12:46 ashimema it's a pretty new thing, but certainly worth a little look
12:46 ashimema Olaf has a pretty good repuation,, knows perl guts pretty well if memory serves
12:45 Joubu yep
12:45 Joubu worth trying this module
12:45 ashimema I was thinking perlimports might help maintain a cleaner base
12:45 Joubu IIRC there were issues I didn't manage to fix
12:45 ashimema but it's an ongoing battle
12:45 ashimema I seem to remember helping to QA it ;)
12:44 Joubu already, years ago
12:44 Joubu I wrote a patch for that
12:44 ashimema it's meant to help clean up such imports.. I wondered about running it against Koha
12:43 ashimema have you seen https://metacpan.org/pod/distribution/App-perlimports/script/perlimports Joubu ?
12:43 ashimema that reminds me.
12:43 ashimema ooh
12:31 Joubu and those related ones
12:31 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28417 enhancement, P5 - low, ---, jonathan.druart+koha, Needs Signoff , Authen::CAS::Client always loaded even in CAS in not in used
12:31 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28416 enhancement, P5 - low, ---, koha-bugs, In Discussion , Email::Sender::Transport::SMTP is using 10Mo of RAM
12:31 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28415 enhancement, P5 - low, ---, koha-bugs, In Discussion , z3950_responder.pl is running with all the modules in RAM
12:31 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28413 enhancement, P5 - low, ---, jonathan.druart+koha, In Discussion , background job worker is running with all the modules in RAM
12:30 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28411 enhancement, P5 - low, ---, koha-bugs, In Discussion , Investigate memory footprint of the different dependencies
12:30 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28306 enhancement, P5 - low, ---, julian.maurice, Signed Off , Allow to query database with minimal memory footprint
12:30 Joubu bug 28306, bug 28411, bug 28413, bug 28415, bug 28416, bug 28417
12:26 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28410 enhancement, P5 - low, ---, koha-bugs, NEW , Reduce memory footprint
12:26 magnuse hooray for bug 28410
11:49 Joubu yes, a test plan is missing however
11:46 TimothyAlexis_ Yes, I have tested it and it works
11:45 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28388 enhancement, P5 - low, ---, oleonard, NEW , Search result set is lost when viewing the MARC plain view (opac-showmarc.pl)
11:45 oleonard Joubu: Bug 28388 is needs signoff, right?
11:41 tcohen morning
11:35 TimothyAlexis_ This is my whole .gitconfig: http://paste.koha-community.org/554
11:25 oleonard The irc log Google search returns results though
11:24 oleonard Weird, irc log search does seem to be broken
11:23 Joubu TimothyAlexis_: sorry I can't help much, this error means that you have something wrong in your config. Double-check it. Maybe you have another entry that is conflicting with that one
11:07 TimothyAlexis_ It was the same error.
11:07 TimothyAlexis_ It was 2021-04-28 and you helped me fix the .gitconfig file Joubu. Those settings are the same since then.
11:06 TimothyAlexis_ I found it when I removed &q and searched the page for git-bz instead...
11:06 magnuse it doesn't seem to give *any* results, must be something wrong with it
11:05 TimothyAlexis_ I'm not getting any results
11:04 TimothyAlexis_ http://irc.koha-community.org/search.pl?channel=koha&nick=TimothyAlexis&q=git-bz
11:02 magnuse search the logs, if you discussed it here?
10:58 TimothyAlexis_ If I remember correctly, you helped me fix this on that specific machine some weeks ago. :\
10:58 TimothyAlexis_ Joubu: I have that
10:45 Joubu you need that in your gitconfig
10:45 Joubu TimothyAlexis_: http://paste.koha-community.org/553
10:38 TimothyAlexis_ I've also pulled the latest git-bz
10:37 TimothyAlexis_ I have the correct settings for bz-tracker in ~/.gitconfig and the settings are correct for git_config = {... in /usr/local/git-bz/git-bz
10:24 TimothyAlexis_ where I have koha master*
10:24 TimothyAlexis_ Joubu, I want to sign off the patch, but on the machine where I have koha, I'm getting "NameError: global name 'host' is not defined" when trying to do git-bz apply
09:16 TimothyAlexis_ Joubu++
09:16 TimothyAlexis_ Haha, thank you
09:15 Joubu TimothyAlexis_: did you see I attached a patch there 2 hours ago?
09:11 TimothyAlexis_ Something about opac-MARCdetail.pl / .tt allows for the search result to be kept, while when loading opac-showmarc.pl / .tt it is dropped.
09:10 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28388 enhancement, P5 - low, ---, oleonard, NEW , Search result set is lost when viewing the MARC plain view (opac-showmarc.pl)
09:10 TimothyAlexis_ Regarding bug 28388 would it perhaps be so easy as to add [% USE Koha %] to opac-showmarc.tt ?
09:09 fridolin1 Joubu: ok thnaks
09:04 MarkHofstetter1 thx
09:00 MarkHofstetter1 me <= stupid memcache restart did it
08:56 MarkHofstetter1 I really don't like docker in that context
08:56 Joubu Are you aware of the koha-testing-docker project? It's what Koha developers are using
08:56 magnuse MarkHofstetter1: have you considered koha-testing-docker? :-)
08:55 Joubu MarkHofstetter1: did you restart memcached, apache and plack?
08:54 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28302 major, P5 - low, ---, julian.maurice, Pushed to stable , Koha does not work with CGI::Compile 0.24
08:54 Joubu fridolin1: don't forget the follow-up on bug 28302
08:54 MarkHofstetter1 hi, I am trying to use koha-gitify (I want to test some bugs/fixes) but after "gitification" I'm looping endlessly through the upgrade routine - thank in advance
08:36 TimothyAlexis If it's easy for someone to make the patch, that would be great. Otherwise, I'm asking for some guidance so that I can make it.
08:35 TimothyAlexis We have modifications on the opac-details.pl page, so if it would be a patch, I'd still have to understand it to implement it.
08:34 TimothyAlexis Hi Joubu, well either way is fine.
07:35 Joubu \o/
07:34 Joubu TimothyAlexis: do you want a patch or guidance to fix it?
07:32 Joubu ha, it's the "busc" stuff
07:31 Joubu there is something stored in the session
07:30 Joubu TimothyAlexis: yes, let me remember how it works
07:06 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28388 enhancement, P5 - low, ---, oleonard, NEW , Search result set is lost when viewing the MARC plain view (opac-showmarc.pl)
07:06 TimothyAlexis I'm wondering if anyone has any idea how to solve this? bug 28388
06:51 magnuse \o/
06:19 dcook That time of the day already O_O
06:19 dcook salut reiveune
05:51 reiveune hello
01:21 dcook tuxayo: Nicely done