Time Nick Message 01:34 koha-jenkins Project Koha_Master build #1791: FAILURE in 2 min 13 sec: https://jenkins.koha-community.org/job/Koha_Master/1791/ 02:44 koha-jenkins Project Koha_Master build #1792: NOW UNSTABLE in 1 hr 2 min: https://jenkins.koha-community.org/job/Koha_Master/1792/ 06:15 fridolin hi there 06:15 wahanui salut, fridolin 06:15 fridolin salut mon poto 06:15 fridolin https://git.koha-community.org/Koha-community/Koha.git has a pb ? 06:29 ashimema Mornin' #koha 06:31 reiveune hello 06:31 wahanui hi, reiveune 06:32 cait if someone has a moment sometime, the 'random bug' on dashboard picks "Pushed to oldoldoldstable" - possible it doesn't know about the newer status 06:33 cait and with that...g etting ready for work 06:41 fridolin https://git.koha-community.org/Koha-community/Koha.git is back 07:17 ashimema ack 07:17 ashimema 11175 needs yet another rebase 07:17 * ashimema is soooooo tired of rebasing that bug 07:18 ashimema Joubu around yet? 07:21 koha-jenkins Project Koha_20.11_U18 build #138: SUCCESS in 38 min: https://jenkins.koha-community.org/job/Koha_20.11_U18/138/ 07:25 koha-jenkins Project Koha_20.11_U20 build #148: SUCCESS in 42 min: https://jenkins.koha-community.org/job/Koha_20.11_U20/148/ 07:40 koha-jenkins Project Koha_20.11_U21 build #4: SUCCESS in 54 min: https://jenkins.koha-community.org/job/Koha_20.11_U21/4/ 07:42 koha-jenkins Project Koha_20.11_U_Stable build #51: SUCCESS in 56 min: https://jenkins.koha-community.org/job/Koha_20.11_U_Stable/51/ 07:45 marcelr hi there 07:45 marcelr ashimema: around? 07:48 koha-jenkins Project Koha_20.11_D10 build #155: SUCCESS in 1 hr 5 min: https://jenkins.koha-community.org/job/Koha_20.11_D10/155/ 07:49 ashimema yup 07:49 marcelr hi i rebased it, attaching it right now 07:49 marcelr wasnt a hard one 07:49 marcelr just opac-detail and catalogue 07:49 ashimema mm 07:49 marcelr we just confused git a bit :) 07:50 marcelr as we did ourselves 07:50 ashimema mmm 07:50 marcelr all seems to work as before 07:50 ashimema i'm just so tired of that bug 07:50 marcelr but the fetching was really done twice 07:50 ashimema which must have been re-introduced by someone after I stopped working on the bug 😉 07:51 ashimema because I explicitly tested that 07:51 marcelr that might be the case 07:51 marcelr yes, i suspect that was the reason for the caching anyway 07:51 ashimema yup 07:51 marcelr but when i removed the caching, it didnt work already 07:51 ashimema the caching was already broken 07:51 ashimema as I pointed out somewhere 07:51 marcelr but now we fetch once, and all looks good 07:52 ashimema joonas broke that 07:52 ashimema but I'm pretty sure I had it down to one query by that point anyway 07:52 ashimema I'm dissapointed we're reducing test coverage again 07:52 ashimema but at least it's one call instead of a block 07:52 ashimema the block was different between the two controllers already 07:53 marcelr they are on the same page again 07:53 ashimema not sure what you mean ? 07:54 marcelr the two controllers are more or less doing the same again 07:54 marcelr as we should expect them to 07:54 ashimema mm 07:55 ashimema maintaining code twice now instead of once.. but that's fine. 07:55 marcelr profit is in the get_marc_components 07:55 ashimema yup 07:55 ashimema once I moved the query generation and actual search in there we already got most of the benefit 07:56 koha-jenkins Project Koha_20.11_U16 build #127: SUCCESS in 34 min: https://jenkins.koha-community.org/job/Koha_20.11_U16/127/ 07:56 cait hope the basic idea si still there 07:56 ashimema we lose tests for 'when should the link display' 07:56 ashimema but you've also changed the expectation there 07:56 cait i had discussed with Joonas about the logic at some poitn 07:56 cait but trusting you guys (mostly :) ) 07:56 marcelr ashimema: you're grumpy 07:56 ashimema tired 07:58 ashimema you're rebase effectively reverts bug 29284 btw 07:58 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29284 major, P5 - low, ---, nick, Pushed to master , Koha dies when an analytics search fails in Elasticsearch 07:59 marcelr hmm could well be since the patch removes the whole block and readds later 08:00 ashimema yup 08:00 marcelr we'd better do a follow-up, haha 08:00 marcelr there we go 08:00 ashimema indeed 08:00 marcelr i may do it this afternoon or you do it now ? 08:01 ashimema still.. you made that follow-up easier now you moved the code back to the controllers.. 08:01 ashimema silver lining 🙂 08:01 ashimema I can take a look now.. get it over and done with 08:01 marcelr ok 08:02 marcelr you got your FQA score higher martin 08:02 ashimema lol 08:02 marcelr tell your boss :) no offense 08:02 ashimema that's one of the leaderboards I prefer not to top 😉 08:03 ashimema I don't mean to be grumpy 08:03 ashimema we're all in this together 08:03 ashimema just tired and reaching koha burn out 08:03 marcelr well it says more about the quality of the submiiteed code 08:05 koha-jenkins Project Koha_20.11_D9 build #151: SUCCESS in 1 hr 21 min: https://jenkins.koha-community.org/job/Koha_20.11_D9/151/ 08:06 koha-jenkins Project Koha_20.11_U18 build #139: SUCCESS in 41 min: https://jenkins.koha-community.org/job/Koha_20.11_U18/139/ 08:10 ashimema now my brain asks... 08:11 ashimema how does perl evaluate boolean context for an arrayref return value 08:12 marcelr [] is true 08:12 ashimema I thought so 08:12 ashimema so.. 08:12 marcelr so test @{ [] } 08:13 ashimema you will always display the link to components even if there are none now 08:13 ashimema I think 08:13 ashimema bug in both you code and mine I think. 08:13 marcelr no i added a @{ [] } somewhere 08:13 marcelr i tested it too 08:13 ashimema `if ( my $components = $biblio->get_marc_components(C4::Context->preference('MaxComponentRecords')) ) {` 08:14 ashimema yup 08:14 marcelr hmm yeah i added the test somewhere later 08:14 ashimema you indeed add it in the else 08:14 ashimema but not highter up 08:14 ashimema yup 08:14 marcelr you're still sharp 08:14 marcelr thats good 08:14 ashimema shame i missed it before now though 😉 08:15 marcelr have to go now for a while, ready to test your followup in a few hours 08:15 marcelr thx for your help 08:15 ashimema suppose before now it was caught in the template as we test size there 08:15 ashimema thank you 08:15 marcelr ashimema++ 08:15 Joubu Behaviour for the "when to show analytics link" is changed, is that expected? 08:16 ashimema it's changed withing the scope of the bug 08:16 ashimema but not outside of that scope 08:16 ashimema i.e. the change depends on if you're using the new feature 08:16 ashimema if you've disabled the feature the results are as before 08:18 Joubu ok.. 08:18 ashimema but there are no longer tests for that behaviour again now 08:18 ashimema I liked tests 08:19 ashimema but moving code back out of the module and into the controllers kinda makes tests harder 08:19 * ashimema has an important meeting in a few minutes 08:19 ashimema will have to come back to restoring the eval code 08:27 cait meeting here too (not seeing ashimema there tho) bbl 08:27 Joubu good catch for the eval, I would have missed it 08:33 koha-jenkins Project Koha_20.11_D11 build #174: SUCCESS in 46 min: https://jenkins.koha-community.org/job/Koha_20.11_D11/174/ 08:37 koha-jenkins Project Koha_20.11_U20 build #149: SUCCESS in 56 min: https://jenkins.koha-community.org/job/Koha_20.11_U20/149/ 09:30 ashimema @later tell fridolin are you going to try and backport bug 28316 ? 09:30 huginn` ashimema: The operation succeeded. 09:35 cait bug 28316 09:35 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28316 normal, P5 - low, ---, stalkernoid, Pushed to stable , Fix ES crashes related to various punctuation characters 09:36 cait ah 09:36 ashimema indeed 09:36 cait Elasticsearch... can i test Elastic on sandboxes these days? 09:36 ashimema i believe so 09:36 * cait has a couple questions for an upcoming workshop to solve/research 09:36 ashimema we certainly run an ES container and have done for ages 09:37 ashimema I think you just switch the syspref and it works 09:37 cait hm going to try on yours right now :) 09:38 ashimema Joubu.. what do you think to removing the end user display that there's been an analytics search error in the staff client? 09:38 ashimema I'm torn.. I like highlighting issues to the end user.. 09:38 ashimema but I also feel the logs should be enough here and adding the error at the template level again is a little more involved. 09:44 Joubu ashimema: I'd let the error, if it's too verbose we could adjust later 09:44 ashimema it's more passing the error down.. but I've worked out a nice way to do it now 09:44 ashimema so all good 10:12 oleonard o/ 10:24 Joubu tcohen: Can you have a look at the last patch on 19185 please? 10:38 oleonard ashimema: Did Bug 15812 obsolete circ-patron-search-results.inc? 10:38 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15812 normal, P3, ---, nick, Pushed to master , Checkout search with too many results (single character search) causes poor performance or timeout 10:38 ashimema I don't believe it did 10:38 ashimema I think I hinted that I'd love to see a next step to remove the last remaining use case for that.. 10:39 ashimema which I think is around reserves or something maybe? 10:40 ashimema hmm 10:41 ashimema seems I never wrote that on the bug.. just asked Nick about it. 10:42 oleonard I see, I forgot that the holds process had that too 10:42 ashimema 14:33 10:42 ashimema hint hint, wink wink 10:42 ashimema would love to see the same approach taken to the last place that uses that funny circ reults page :wink: 10:42 ashimema 14:33 10:42 ashimema hint hint, wink wink 10:42 ashimema that's what I sent Nick.. 10:43 ashimema seems I totally forgot to add it to the bug 10:43 ashimema do you agree it would be nice to take the same approach if we can and deprecate the include? 10:44 oleonard They way it is right now you can still switch from Patrons to Clubs during that search process. I'm not sure how we'd handle that if we were redirecting to the patron search page. 10:44 ashimema ah, fair 10:45 ashimema I didn't dig very far 10:45 oleonard Maybe we need to generalize the patron search results so that the results can be embedded anywhere 10:45 ashimema perhaps 10:45 ashimema we should turn it into an API component really 😉 10:46 oleonard "we" ;) 10:47 ashimema hehe 10:56 kidclamp oleonard: around? 10:56 oleonard Yup 11:01 kidclamp oleonard++ 11:04 marcelr back and ready for 11175 again 11:06 Joubu if you want a generalize patron search have a look at common/patron_search.tt 11:06 marcelr ashimema: i will fix: Global symbol "$err" requires explicit package name (did you forget to declare "my $err"?) 11:06 marcelr np 11:07 marcelr where is my toolbox? 11:09 marcelr funny when there is an error with the name of $error ! 11:18 Joubu ashimema: 28700 - are we trying to only fix the size log, or something else? 11:18 Joubu if size of the log, I don't think we should mess with unblessed 11:21 tcohen morning 11:22 marcelr hola tcohen 11:34 marcelr Joubu: 11175 is yours now again 11:38 marcelr Joubu: what is currently blocking 28700 ? 11:43 Joubu stuck 11:44 Joubu maybe we should more focus on bug 28692 actually 11:44 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28692 major, P5 - low, ---, stalkernoid, Needs Signoff , Reduce DB action_log table size 11:47 marcelr having a look there 11:47 marcelr remember to have been there? 12:03 ashimema morning tcohen 12:04 ashimema I took bug 28700 for it's title:"Unblessed" method in Object.pm has blessed values for keys in some cases 12:04 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28700 major, P5 - low, ---, stalkernoid, In Discussion , "Unblessed" method in Object.pm has blessed values for keys in some cases 12:04 ashimema as in.. we're trying to actually fix 'unblessed' 12:04 ashimema yet, One of the symptoms is log related.. but it's not the only one. 12:05 ashimema I've hit this issue with API controllers too.. as I know tcohen also has. 12:05 ashimema as such I wanted to fix/warn about the issue at it's route. 12:05 ashimema as for the logs thing.. I prefer that being focused on as a whole in bug 28692 myself 12:05 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28692 major, P5 - low, ---, stalkernoid, Failed QA , Reduce DB action_log table size 12:06 Joubu ashimema: as said on the bug, I am against fixing ->unblessed. We are too close to the release and side-effects are possible 12:10 marcelr ashimema i am thinking about a great solution for the log bug, hahaha 12:10 ashimema I agree 12:10 ashimema it's one for next release 12:10 ashimema the unblessed thing 12:10 marcelr proably 12:11 ashimema but I do think it's worthwhile endeavour during next cycle 12:11 marcelr unblessed is the right word for it 12:11 tcohen Joubu: Can't locate Email/Valid.pm in @INC on KTD startup, I thought it was fixed 12:11 marcelr $object->compactify->logaction 12:12 ashimema I think we either throw exceptions when unblessed is likely to return something that contains blessed stuff.. 12:13 Joubu tcohen: coming from QA tools? 12:13 tcohen stringify? 12:13 ashimema or we fix unblessed to actually return unblessed everything. 12:13 tcohen Joubu: I cannot launch KTD right now 12:13 tcohen what are you using? master-bullseye? 12:13 Joubu yes 12:14 Joubu but it got rebuilt yesterday, not sure I pulled it since 12:14 Joubu trying now 12:16 Joubu well, jenkins is happy so it should be ok 12:20 Joubu tcohen: still working after a pull 12:20 Joubu tcohen: check you qa repo in .env 12:23 tcohen ok, pulling again 12:24 tcohen I was sure I did already 12:24 tcohen but there's a new one it seems... 12:24 Joubu ashimema, marcelr: I am sorry but.. 12:24 Joubu + $template->param( analytics_error => 1 ) if @{$biblio->messages}; 12:25 Joubu this is not robust 12:25 Joubu we need to check if one of the messages is component_search 12:25 marcelr yeah i saw it 12:25 marcelr ashimema let me do it 12:25 marcelr just relax :) 12:26 Joubu marcelr: also, why is it done that way in staff but OPAC? 12:26 ashimema yup 12:26 ashimema I realised 12:26 marcelr opac seems not be interested in errors 12:26 ashimema but also my brain was mid meeting at the time and I wanted to put something down.. I had a feeling you guys would iterate whatever I did 😉 12:26 marcelr but actually that is the other bug that you already pushed Joubu 12:26 Joubu ok, make sense 12:27 marcelr i look now 12:27 ashimema yup.. opac wasn't interested in displaying the errors prior to this patch either.. I think it was a deliberate decision in the preceeding bug 12:27 marcelr right 12:27 Joubu yep 12:32 ashimema marcelr.. i've given a couple of extra things to think about regards that DB action_log reduction bug 12:33 ashimema we need to be careful here.. else we're going to introduce some pretty big functional regressions. 12:33 ashimema think 'diffs' 12:33 ashimema 😉 12:34 Joubu otoh diff is broken now because of the DT inflation 12:34 ashimema yup 12:35 Joubu my point is: we are adding JSON for the new ACQ logs in 21.11 12:35 Joubu so if we need to move everything to JSON it's good timing 12:35 * ashimema would like to see either a fairly consistent json form of diff stored (rather than whole objects at all).. or a minified json stored and a standard inflation before diff called in the viewer. 12:36 ashimema I didn't realise you'd already said to move to json Joubu 🙂 12:36 ashimema I'm totally onboard with that. 12:37 ashimema do you think the patron handling should also move back to whole objects jsonified instead of the custom diff format it currently takes too? 12:37 ashimema then we could have a more consistent diff display throughout... 12:37 ashimema but also would result in larger logs 12:37 ashimema but smaller than current.. as current is a exploded Dumper 12:37 * ashimema needs more tea 12:38 Pavlis Hello, Can you help me with Koha Staff Interface? 12:38 cait Pavlis: you'll have to be a bit more specific :) 12:40 Pavlis I want a 12-digit patron number to be created automatically when creating patrons. In the settings, I set everything according to the instructions. 12:41 Joubu ashimema: If I modify a biblio it logs "biblio BEFORE", not the AFTER... 12:41 ashimema lol 12:41 Pavlis I use 21.05.04.000 of Koha 12:42 Joubu ashimema: If I modify an item, it logs a Dumper representation with the new values 12:42 Joubu all consistent.. 12:42 ashimema yup 12:42 ashimema we are really inconsistent 12:42 ashimema I want to move to consistency 12:42 ashimema we can go either way really.. 12:42 cait Pavlis: the cardnumber will be counted up from the last number used 12:42 ashimema either we log the before and compare to now 12:43 ashimema or we log the after and compare to the previous log line 12:43 cait if you use Automembernum from the system preferences 12:43 Joubu we should sent pre/post to logaction and let it store the diff in JSON 12:43 ashimema I think I preferred the latter when i looked into it all for notices 12:43 cait so if yu set a starting number that is high enough, the next one should be added as you want 12:43 ashimema haha 12:43 ashimema yeah.. 12:43 cait but it probably shoud not be starting with 0 12:43 ashimema that's what I did with borrowers/patrons 12:43 ashimema it stores a json diff 12:43 cait something like 1000 0000 0000 would work 12:43 Pavlis Yes, i Know ... tast number i used is 00000000003 and next auto generate is 7 :( 12:44 cait yeah, that's how it works 12:44 ashimema though not in a "standard" format.. there are a few different ways to represent diffs in json still.. there is no standard so I wasn't sure how to get koha to adopt any of them. 12:44 cait you probably have a 6 in a record somewhere too 12:45 cait that's why it comes up with 7 12:45 Joubu ashimema: think also Koha::Biblio - we don't log the Koha::Object but the MARC 12:45 ashimema indeed 12:45 ashimema that's scary too 12:45 cait might be a deleted patron record too (not sure about that) 12:45 Joubu but that's needed 12:45 ashimema and what marc do we store.. mrc, iso, xml 😉 12:46 ashimema that's where storing whole objects works nicely.. 12:46 ashimema the diff can just be standard text line diff 12:46 ashimema so long as we have the whole picture before and after to compare 12:46 ashimema I can't entirely remember how the output looks.. but i seem to remember it being pretty reasonable 12:46 * ashimema likes it for notices now 12:47 ashimema it's really nice to see the before and after with nice line highlighting of changes 12:47 ashimema I think the diff on it's own is less useful as you lose context 12:47 Joubu yes, but we will end up with a bigger table 12:48 ashimema yup 12:48 Joubu it's what we want to avoid :D 12:48 Joubu pros/cons.. 12:48 Joubu I dont' know, hard call 12:48 ashimema all I'm trying to highlight is that we're inconsistent.. but also that there are pros and cons and there's more to consider than just the table size 12:48 ashimema indeed 12:48 ashimema hard call 12:48 ashimema but certainly needs all aspects considering 12:48 ashimema I totally liked the json diff option when I wrote the patron stuff 12:49 Joubu kidclamp: around? ^ 12:49 ashimema but at the time I hadn't actually seen the log viewer diff view 12:49 * kidclamp waves 12:49 marcelr hi kidclamp 12:49 Pavlis Ah.. i see.. it cant be 00000000001 .. it should be 1000000001 ... start with zero not working 12:49 ashimema my memory of the acq logging stuff was that kidclamp was pretty adamant about wanting the whole object stored every time rather than just the changed fields. 12:50 ashimema but I stay fairly clear on that bug 12:50 Pavlis with number for example 957800924028 works! :-) 12:50 kidclamp I did, but I relented 12:50 ashimema oh? 12:50 wahanui oh are there instructions? 12:50 ashimema I didn't realise you had 12:50 ashimema what was your reasoning for wanting it all 12:50 ashimema vs just changed 12:50 ashimema the logviewer diff view? 12:51 kidclamp just changed would be fine - the bug though only logged certain fields - i.e. you can make changes that are logged but show no difference 12:51 ashimema ah 12:51 ashimema I see 12:51 ashimema I didn't realise that was the case 12:51 kidclamp https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24190#c48 12:51 huginn` Bug 24190: enhancement, P5 - low, ---, andrew.isherwood, RESOLVED FIXED, Add additional Acquisition logging 12:52 ashimema I'm happy with just changes.. but all changes should be treated equally 12:52 ashimema unless of course there's a solid reason to obfiscate them.. like a password field change.. 12:52 ashimema then only the change should be recorded.. without a before/after. 12:52 ashimema in my opinion 12:52 marcelr Joubu i ended up in adding one grep to 11175 and adding $error in the payload 12:52 ashimema thanks marcelr 12:53 Joubu kidclamp: does it mean you are in favor of logging pre and post JSON serialized object? 12:53 marcelr would that be enough for you? 12:53 kidclamp if it is easy to view a diff, yes 12:53 ashimema I considered using this: http://jsonpatch.com/ 12:54 ashimema but I don't think your average Joe would get it 12:54 Joubu marcelr: yes, looks good to me 12:55 Pavlis I have another problem :-D cards with ean not generate ean 12:55 Pavlis i have all modules installed 12:55 ashimema hence I switched to { fieldname => { before => value, after => value } } structure 12:55 ashimema it's not entirely pretty.. but it is clear to all 12:55 Pavlis *patron cards 12:56 Joubu Pavlis: the problem with barcode and zeros is reported there: bug 27426 12:56 marcelr wait Joubu i forgot one 12:56 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27426 enhancement, P5 - low, ---, jonathan.druart+koha, Failed QA , Plugin barcode must keep leading zeros 12:57 Pavlis i see 12:58 marcelr done' 13:00 Joubu ashimema, marcelr: Can I trust you for 11175 or should I retest the whole thing? 13:02 ashimema I'm testing it again right now 13:02 ashimema so far it's looking solid 13:02 marcelr i tested it 13:02 ashimema what I've realised I've missed is adding a test for the message we add to the object on get_marc_components failure 13:02 marcelr no big deal 13:03 ashimema not entirely sure how I'd write a test for that.. it needs simple_search_compat to explode to throw it 13:03 marcelr what would be nice is to extend the tests for get_marc_components a bit 13:03 ashimema but yes.. otherwise I think it's solid 13:03 ashimema that's what I meant marcelr 13:03 ashimema more tests for get_marc_components 13:04 marcelr thats the clou of the thing 13:04 * ashimema kinda would like to eventually have that routine handle unimarc properly too 13:04 marcelr push it Joubu 13:04 marcelr i know you can 13:13 ashimema yup, looks solid to me Joubu.. go for it 13:15 koha-jenkins Project Koha_21.05_U_Stable build #75: SUCCESS in 30 min: https://jenkins.koha-community.org/job/Koha_21.05_U_Stable/75/ 13:16 Joubu marcelr, ashimema: I am writing the missing tests 13:16 ashimema thanks Joubu. 13:20 marcelr Joubu++ 13:20 Joubu ashimema: can you add bug 24850 comment 104 to your list please? 13:20 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24850 major, P5 - low, ---, koha-bugs, Passed QA , Koha::DateUtils ignores offsets in RFC3339 datetimes 13:21 Joubu ashimema: it's for a separate bug I think 13:22 Joubu ashimema: also bug 24609 - at least Koha::Checkout->store is missing test coverage 13:22 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24609 enhancement, P5 - low, ---, andrew.isherwood, Failed QA , Add routes for updating checkouts 13:22 ashimema certainly 13:22 ashimema thanks for all your work the last few days guys 13:22 ashimema I realise I've been a right grumpy git â˜¹ï¸ 13:23 ashimema seeing so much movement has helped.. just a bit scary it's all so last minute and when i have lots of other work on. 13:23 Joubu always last minutes... 13:25 koha-jenkins Project Koha_21.05_U21 build #6: SUCCESS in 40 min: https://jenkins.koha-community.org/job/Koha_21.05_U21/6/ 13:32 ashimema line 104.. I'm not entirely sure I understand your meaning Joubu? 13:33 Joubu ashimema: file? 13:33 wahanui i heard file was probably correct, but something in the process breaks it 13:33 ashimema sorry.. I meant comment 104 13:33 ashimema "due_date": "2021-10-27 23:59",... (full message at https://matrix.org/_matrix/media/r0/download/matrix.org/VIPCDjvhLhwiJrJlHvmaxzuZ) 13:33 ashimema what's it referring to.. a line in a test somewhere? 13:34 Joubu ashimema: It's coming from tests I made. If you PUT with "2021-10-27 23:59" you get the expected "does not match date-time" 13:34 Joubu because it's missing the tz part 13:35 Joubu but if you pass the minute, you get "missing property". I am expecting the same "does not match date time" error 13:35 ashimema ah 13:35 ashimema thanks 13:35 ashimema I see 13:35 ashimema OK.. I'll copy the comment into a new bug and take a quick look 13:39 ashimema ohh.. that's in the JSON::Validator module I think 13:42 koha-jenkins Project Koha_21.05_U16 build #81: SUCCESS in 57 min: https://jenkins.koha-community.org/job/Koha_21.05_U16/81/ 13:44 koha-jenkins Project Koha_21.05_U20 build #85: SUCCESS in 1 hr 0 min: https://jenkins.koha-community.org/job/Koha_21.05_U20/85/ 13:50 koha-jenkins Project Koha_21.05_D10 build #123: SUCCESS in 1 hr 6 min: https://jenkins.koha-community.org/job/Koha_21.05_D10/123/ 13:51 koha-jenkins Project Koha_21.05_D9 build #78: SUCCESS in 35 min: https://jenkins.koha-community.org/job/Koha_21.05_D9/78/ 13:56 oleonard-away kidclamp: Bug 28537 13:56 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28537 enhancement, P5 - low, ---, oleonard, Needs Signoff , Improve HTML generated by OverDrive integration 14:02 koha-jenkins Project Koha_21.05_U18 build #77: SUCCESS in 1 hr 17 min: https://jenkins.koha-community.org/job/Koha_21.05_U18/77/ 14:04 koha-jenkins Project Koha_21.05_D12 build #13: SUCCESS in 38 min: https://jenkins.koha-community.org/job/Koha_21.05_D12/13/ 14:29 koha-jenkins Project Koha_21.05_D9 build #79: SUCCESS in 38 min: https://jenkins.koha-community.org/job/Koha_21.05_D9/79/ 14:35 koha-jenkins Project Koha_21.05_D11 build #98: SUCCESS in 51 min: https://jenkins.koha-community.org/job/Koha_21.05_D11/98/ 14:37 koha-jenkins Project Koha_21.05_D12 build #14: SUCCESS in 54 min: https://jenkins.koha-community.org/job/Koha_21.05_D12/14/ 14:47 huginn` News from kohagit: Bug 14957: DBRev 21.06.00.038 <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=aab18ba056000c27e89f84fdfb16388be77f44f1> 14:47 huginn` News from kohagit: Bug 14957: DBIC schema changes <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=38a71afebf64b96cd60128a28ed7f51c52ecca1e> 14:47 huginn` News from kohagit: Bug 29318: Tidy the code <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=474f228a6986cce5e3da2425db7fbcad444c5fd0> 14:47 huginn` News from kohagit: Bug 29318: Remove permission check from overdrive search page <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=0099cff73967cf459a8a3b6dcf583f5ee01d0394> 14:47 huginn` News from kohagit: Bug 29286: Typo: Librarien will need the manage_auth_values subpermission. <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=93822a8ca77afe2b7230d8f1fc41f56e28204f01> 14:47 huginn` News from kohagit: Bug 29158: (bug 22435 follow-up) Remove ref to account_offset_types.sql in intaller <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=16d2765c3c8575012c437f2c1d343d94f2a79ce1> 14:47 huginn` News from kohagit: Bug 14957: Add tests for subfield order <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=3247eb9a7345bacf3d87894d735d0984cd218a56> 14:47 huginn` News from kohagit: Bug 14957: Fix field order issue <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=8425cf1e13da5ccd7854bf01f3327f1601d3877f> 14:47 huginn` News from kohagit: Bug 14957: Fix record preview <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=f756d9534e7f7e23336635f53493ab04a59179b5> 14:47 huginn` News from kohagit: Bug 14957: Exclude leader from saved/updated record comparison in tests <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=6ab1ca24283d73c4404ef67f3d30651966f4c435> 14:47 huginn` News from kohagit: Bug 14957: Fix regression, rules must be refetched on add/edit/deletion <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=d306da9f8dce5c5266629e5fe376af2804ce3c3b> 14:47 huginn` News from kohagit: Bug 14957: Improve readbility of tests <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=435abd3379e7ffb2e937a58254c473cf0dbf69c8> 14:47 huginn` News from kohagit: Bug 14957: Clean controller <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=10a51487caa86269042e20e37b69b7ba521009c9> 14:47 huginn` News from kohagit: Bug 14957: (QA follow-up) Fix for jQuery undefined <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=78b5cd20158b217728383bf5a69db4a3fd535ef7> 14:47 huginn` News from kohagit: Bug 14957: (QA follow-up) Fix script name <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=c3906bf1dca7cd7d9d2d63fd53df8ba94e3e6365> 14:47 huginn` News from kohagit: Bug 14957: (QA follow-up) Fix template indentation <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=f4b55db9d48a682a5509997d50428ccb1efc6ed3> 14:47 huginn` News from kohagit: Bug 14957: (QA follow-up) Update for bug 27846 <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=7838c8e3078b87b73f5f542329f2fc570c44568c> 14:47 huginn` News from kohagit: Bug 14957: (QA follow-up) Update for bug 17858 <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=28aef3a5257ff7b18a8a798e74d2e1d0aa1b82f5> 14:47 huginn` News from kohagit: Bug 14957: (QA follow-up) Update for bug 26703 <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=a3e113977bd2c1b5422b60291da1d28575d7af5d> 14:47 huginn` News from kohagit: Bug 14957: (QA follow-up) Add missing filters <https://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=342dae89f88f4e89e8cee0c273fe065260950770> 14:49 marcelr ashimema: you must be so happy now 14:50 Joubu nobody is protected from a revert 14:50 Joubu :d 14:50 Joubu kidding! 14:50 ashimema I'll be happy when i see Jenkins come back green 😉 14:50 ashimema shan't count my chickens.. hehe 14:51 ashimema but yes.. very happy in preparation for that 14:53 koha-jenkins Project Koha_21.05_D11 build #99: SUCCESS in 50 min: https://jenkins.koha-community.org/job/Koha_21.05_D11/99/ 14:53 marcelr Koha never reverts ! always going forward ! 14:55 marcelr Joubu: i added a sort of POC patch on bug 29321 for a very simplistic approach on logs 14:55 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29321 enhancement, P5 - low, ---, koha-bugs, ASSIGNED , My new bug 14:55 marcelr in core: dump refs and truncate 14:56 marcelr you can log objects, unblessed stuff, everything 14:57 Joubu marcelr: I don't think we should truncate 14:57 Joubu how would it be useful if the change you want is at the end? 14:57 marcelr you dont unless you active the pref 14:58 oleonard Anyone familiar with the UpdateItemLocationOnCheckin system preference? Is "PROC" an internal setting or does one have to have it defined somewhere? 14:58 marcelr if size is your issue, then truncate is your friend 14:58 Joubu marcelr: don't we actually want a pref to either log either the diff or the pre & post objects? 14:58 marcelr sure but we will not get that in anymore? 14:59 Joubu if you want a short log you chose only the diff, or the whole before/after objects 14:59 marcelr diffs only would be nice 15:00 ashimema I reckon lifting Jonathans 'detect if we have an object in $info' in logcation from one bug into the other is the right appraoch 15:00 cait confetti! 15:00 wahanui o/`'`'`'`'`'`'`'`'`'`'`'` 15:00 cait what a day... 15:00 wahanui somebody said a day was a period of 24 hours, that sense 15:00 ashimema then I'd like to see us iterate on format 15:00 marcelr htg 15:00 ashimema ut that's only a feeling 15:00 marcelr see you 15:00 Joubu marcelr++ 15:01 Joubu nice QA run today :) 15:01 ashimema beers to marcelr 15:01 cait can we do bug 20689 next? 15:01 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20689 enhancement, P5 - low, ---, katrin.fischer, Signed Off , Make it possible to add capital letter subfields to item search fields 15:01 cait *hides* 15:01 cait it's oldestnow... 15:03 Joubu oleonard: it's a location I think 15:03 Joubu so must be a LOC av? 15:03 * ashimema agree 15:04 oleonard Where is "permanent location" defined for an item? 15:05 Joubu items.permanent_location? 15:05 Joubu it's set to items.location when the item is created 15:07 oleonard I'm confused... You can't define an item's current location as "PROC" and its permanent location as something different if you're using the item editor? 15:11 Joubu PROC and CART are supposed to be temporary location, permanent_location is keeping the original location value to be able to swap it back 15:11 Joubu it's my understanding, but kidclamp is the one to ask for that :) 15:12 koha-jenkins Project Koha_Master build #1793: FAILURE in 23 min: https://jenkins.koha-community.org/job/Koha_Master/1793/ 15:13 Joubu mtj: Docker_6 failing to many times with: "koha_memcached_1 exited with code 0"! 15:13 kidclamp some libraries do add that to the edit form 15:15 oleonard kidclamp: I guess I don't understand how you could not add it to the edit form. How else are new items getting the PROC info *and* a correct permanent location? 15:15 kidclamp cataloguing preference 15:16 kidclamp looking 15:16 Joubu NewItemsDefaultLocation 15:16 kidclamp that ting 15:17 oleonard Ah, now it becomes clear. I suspect I knew this at one time. 15:17 Joubu oleonard: the logic is in Koha::Item->store (l.100) 15:18 Joubu for item creation. For item edition it's l.181 15:20 Joubu ok, leaving. I am letting you with Jenkins 15:20 Joubu good luck :D 15:21 koha-jenkins Project Koha_Master_D11_CPAN build #458: UNSTABLE in 33 min: https://jenkins.koha-community.org/job/Koha_Master_D11_CPAN/458/ 15:27 koha-jenkins Project Koha_Master_D11_My8 build #675: UNSTABLE in 39 min: https://jenkins.koha-community.org/job/Koha_Master_D11_My8/675/ 15:47 koha-jenkins Project Koha_Master_U21 build #7: UNSTABLE in 58 min: https://jenkins.koha-community.org/job/Koha_Master_U21/7/ 15:48 koha-jenkins Project Koha_Master_U20 build #221: UNSTABLE in 59 min: https://jenkins.koha-community.org/job/Koha_Master_U20/221/ 15:57 oleonard If I set NewItemsDefaultLocation to some particular shelving location code and I add an item in cataloging, shouldn't it have that default location in items.location? In my test it doesn't. 15:59 * oleonard wonders if his library has never used this feature because it's broken... 16:06 koha-jenkins Project Koha_Master_D11 build #427: UNSTABLE in 38 min: https://jenkins.koha-community.org/job/Koha_Master_D11/427/ 16:09 koha-jenkins Project Koha_Master_D12 build #51: UNSTABLE in 57 min: https://jenkins.koha-community.org/job/Koha_Master_D12/51/ 16:10 oleonard The preference isn't listed in Hea 16:14 koha-jenkins Project Koha_Master_D9 build #1800: UNSTABLE in 1 hr 21 min: https://jenkins.koha-community.org/job/Koha_Master_D9/1800/ 16:33 koha-jenkins Project Koha_Master_U_Stable build #294: UNSTABLE in 34 min: https://jenkins.koha-community.org/job/Koha_Master_U_Stable/294/ 16:45 koha-jenkins Project Koha_Master_D11_MDB_Latest build #710: UNSTABLE in 58 min: https://jenkins.koha-community.org/job/Koha_Master_D11_MDB_Latest/710/ 16:47 koha-jenkins Project Koha_Master_D9 build #1801: STILL UNSTABLE in 41 min: https://jenkins.koha-community.org/job/Koha_Master_D9/1801/ 16:54 oleonard git bisect says that Bug 27545 broke NewItemsDefaultLocation 16:54 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27545 enhancement, P5 - low, ---, jonathan.druart+koha, Pushed to master , NewItemsDefaultLocation is only used from additem.pl 16:54 koha-jenkins Project Koha_Master build #1794: NOW UNSTABLE in 1 hr 5 min: https://jenkins.koha-community.org/job/Koha_Master/1794/ 17:01 oleonard cait: Did you see my comment on Bug 29036? 17:01 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29036 minor, P5 - low, ---, bolshawh, Failed QA , Accessibility: OPAC buttons don't have sufficient contrast 17:02 cait sorry not yet and gone for today 17:02 cait maybe later or tomorrow 17:02 cait wait, i remember - yes i saw, just make it signed off again please? I wil check 17:02 oleonard Okay thank you 17:02 koha-jenkins Project Koha_Master_D12 build #52: STILL UNSTABLE in 52 min: https://jenkins.koha-community.org/job/Koha_Master_D12/52/ 17:05 cait was wondering: is contract for 'disabled' good enough? 17:05 cait if the pale version is inactive 17:07 oleonard I don't know... Sometimes we alert when an inactive button is clicked, sometimes we don't 18:23 koha-jenkins Project Koha_Master build #1795: STILL UNSTABLE in 1 hr 2 min: https://jenkins.koha-community.org/job/Koha_Master/1795/ 18:37 oleonard Is there something about the item modification background job that is different in KTD? 18:37 oleonard I ask because my batch item modifications aren't working 18:38 oleonard Oh, correction, it's just failing to set the LOST status 21:53 koha-jenkins Project Koha_20.05_U20 build #367: FAILURE in 10 min: https://jenkins.koha-community.org/job/Koha_20.05_U20/367/ 21:53 koha-jenkins Project Koha_20.05_U16 build #331: FAILURE in 10 min: https://jenkins.koha-community.org/job/Koha_20.05_U16/331/ 21:53 koha-jenkins Project Koha_20.05_U18 build #206: FAILURE in 10 min: https://jenkins.koha-community.org/job/Koha_20.05_U18/206/ 21:55 koha-jenkins Project Koha_20.05_D10 build #328: FAILURE in 10 min: https://jenkins.koha-community.org/job/Koha_20.05_D10/328/ 21:59 oleonard-away Where is it controlled whether you can batch mod the lost status? 21:59 cait i tihnk it has to be set to visible in the frameworks 22:00 oleonard-away Ah, you're right 22:01 oleonard-away I think Bug 28445 broke the ability to batch update lost status 22:01 huginn` Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28445 enhancement, P5 - low, ---, jonathan.druart+koha, RESOLVED FIXED, Use the task queue for the batch delete and update items tool 22:03 koha-jenkins Project Koha_20.05_D11 build #721: FAILURE in 10 min: https://jenkins.koha-community.org/job/Koha_20.05_D11/721/ 22:03 koha-jenkins Project Koha_20.05_U20 build #368: STILL FAILING in 10 min: https://jenkins.koha-community.org/job/Koha_20.05_U20/368/ 22:32 koha-jenkins Project Koha_20.05_U_Stable build #14: SUCCESS in 49 min: https://jenkins.koha-community.org/job/Koha_20.05_U_Stable/14/ 22:38 koha-jenkins Yippee, build fixed! 22:38 wahanui Congratulations! 22:38 koha-jenkins Project Koha_20.05_U18 build #207: FIXED in 35 min: https://jenkins.koha-community.org/job/Koha_20.05_U18/207/ 22:42 koha-jenkins Project Koha_20.05_D11 build #722: STILL FAILING in 10 min: https://jenkins.koha-community.org/job/Koha_20.05_D11/722/ 22:42 koha-jenkins Project Koha_20.05_U21 build #5: SUCCESS in 49 min: https://jenkins.koha-community.org/job/Koha_20.05_U21/5/ 22:44 koha-jenkins Yippee, build fixed! 22:44 wahanui Congratulations! 22:44 koha-jenkins Project Koha_20.05_U16 build #332: FIXED in 40 min: https://jenkins.koha-community.org/job/Koha_20.05_U16/332/ 22:52 koha-jenkins Project Koha_20.05_D11 build #723: STILL FAILING in 10 min: https://jenkins.koha-community.org/job/Koha_20.05_D11/723/ 22:52 koha-jenkins Project Koha_20.05_D9 build #321: SUCCESS in 1 hr 10 min: https://jenkins.koha-community.org/job/Koha_20.05_D9/321/ 22:55 koha-jenkins Yippee, build fixed! 22:55 wahanui Congratulations! 22:55 koha-jenkins Project Koha_20.05_D10 build #329: FIXED in 1 hr 0 min: https://jenkins.koha-community.org/job/Koha_20.05_D10/329/ 23:02 koha-jenkins Project Koha_20.05_U16 build #333: FAILURE in 10 min: https://jenkins.koha-community.org/job/Koha_20.05_U16/333/ 23:12 koha-jenkins Yippee, build fixed! 23:12 wahanui Congratulations! 23:12 koha-jenkins Project Koha_20.05_U20 build #369: FIXED in 33 min: https://jenkins.koha-community.org/job/Koha_20.05_U20/369/ 23:12 koha-jenkins Project Koha_20.05_D9 build #322: FAILURE in 10 min: https://jenkins.koha-community.org/job/Koha_20.05_D9/322/ 23:22 koha-jenkins Project Koha_20.05_U20 build #370: FAILURE in 10 min: https://jenkins.koha-community.org/job/Koha_20.05_U20/370/ 23:23 koha-jenkins Yippee, build fixed! 23:23 wahanui Congratulations! 23:23 koha-jenkins Project Koha_20.05_D11 build #724: FIXED in 39 min: https://jenkins.koha-community.org/job/Koha_20.05_D11/724/ 23:39 koha-jenkins Project Koha_20.05_D11 build #725: SUCCESS in 46 min: https://jenkins.koha-community.org/job/Koha_20.05_D11/725/ 23:41 koha-jenkins Project Koha_20.05_U18 build #208: SUCCESS in 59 min: https://jenkins.koha-community.org/job/Koha_20.05_U18/208/ 23:48 koha-jenkins Project Koha_20.05_D10 build #330: SUCCESS in 53 min: https://jenkins.koha-community.org/job/Koha_20.05_D10/330/