Time Nick Message 03:53 Francesca hi 03:53 wahanui hola, Francesca 03:57 wizzyrea hi Francesca 03:57 Francesca sup 03:58 wizzyrea not a whole lot really, worky work work work 03:59 wizzyrea how about you? 03:59 Francesca worky worky work uni style 04:00 Francesca and bad asthma 04:05 wizzyrea bleh 04:07 Francesca good description 04:16 wizzyrea I feel very thankful that I have never had more than a wee bit of allergy sniffles 04:16 wizzyrea no asthma or other "oh god I can't breathe" issues. 04:17 mtj hey Francesca, wizzyrea ;0) 04:17 wizzyrea hi mtj! 04:17 Francesca hi mtj 04:17 wizzyrea gosh it seems like ages since I've talked to you 04:17 Francesca I just took a mouse off sirius 04:17 Francesca hes not happy 04:17 wizzyrea eh let him eat it :) 04:18 Francesca in this house we rescue mice 04:18 mtj yeah, april already - how did that happen?! 04:18 wizzyrea I don't know! 04:18 wizzyrea Francesca: I question your methods, but to each their own. :) 04:18 wizzyrea me personally, I don't like having mice around. :P 04:19 Francesca i caught it in a net and let it go outside 04:19 wizzyrea so a caught mouse is always a "good kitty" moment for me 04:19 wizzyrea well, I like them if they are in cages and friendly and not bitey and scrobbling around in my pantry 04:19 Francesca do you have a cat? 04:19 wizzyrea not anymore :( 04:20 Francesca sad 04:20 wizzyrea yeah a b ti 04:20 ibeardslee it's called a calendar, and each time the world spins does a full rotation, we add cross off another day we have done, there are about 30 in a month and if depending where you started, you eventually cycle through 12 of those months to get to April. 04:20 mtj hey #koha...does anyone know what the process is to fix a pootle/translation typo? 04:20 wizzyrea bit* 04:20 Francesca i need to email thatcher about guiitar - i keep forgetting 04:20 wizzyrea ibeardslee: ilu 04:20 Francesca lol 04:23 ibeardslee gah .. too much changing thought process mind sentence 04:25 mtj bug 16144, re: that pootle Q 04:25 huginn 04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16144 minor, P5 - low, ---, koha-bugs, NEW , Patron search fails when using Korean (pootle/translate error) 04:38 mtj ibeardslee: how about magnets? 04:38 mtj :p 04:38 wizzyrea yes, how do they work 06:39 marcelr hi #koha 06:48 Joubu morning 06:48 reiveune hello 06:48 wahanui que tal, reiveune 06:48 marcelr hi Joubu reiveune 06:49 reiveune salut marcelr Joubu 06:58 alex_a bonjour 07:28 cait morning #koha 07:34 LibraryClaire morning cait :) 07:35 cait morning LibraryClaire :) 07:46 cait Joubu: only email 07:46 cait that bug worries me a great deal 07:46 Joubu cait: just tested and works for me.... 07:46 Joubu I have only email too 07:46 cait when i entered an e-mail address, all was good 07:47 cait but without... 07:47 cait i think it should have created an email to the library and a print notice? 07:48 ashimema morning #koha 07:48 cait morning ashimema :) 07:50 LibraryClaire hi ashimema 07:50 Joubu starting a week debugging overdue_notices is like self-flagellation 07:50 cait sorry for that :( 07:51 cait ah, something 07:51 cait there was a line printed by overdue_notices.pl 07:51 cait something about MTT and print? 07:51 cait possible i only have an email template, no print template - could that be it? 07:52 Joubu ha, yes 07:53 cait shoudl there be a fallback or is it my fault? 07:53 cait thought there used to be one :) 07:54 * ashimema runs away before he gets roped into the self-flagellation too 07:54 cait lol 07:55 * magnuse waves 07:55 Joubu cait: I don't remember, but I though email was the fallback 07:56 Joubu I don't understand, I cannot recreate the bug on master......... 07:56 Joubu an email notice is required for a patron without email address and a print notice is generated 08:04 cait it hink what it used to do was 2 things 08:04 cait generated a print entry in message queue 08:04 cait and an entry with no borrowernumber witht he email for the library 08:05 cait can you reproduce the problem without the patch? 08:05 Joubu cait: added a note 08:05 Joubu the patch should fix the email for the library 08:05 Joubu not the print generated for the patrons 08:06 Joubu the print is sent even without the patch 08:06 cait i didn't get either 08:06 cait and no idea why that would be 08:07 cait if the date was wrong, adding the email address shouldn't have generated a notice either 08:07 Joubu cait: so maybe we have another bug related to the feedback template to use 08:07 * ashimema hands Joubu a bigger stick 08:08 Joubu :d 08:08 cait Joubu: what does your overdue template look like? 08:08 cait which have you set? 08:10 Joubu The default ODUE 08:11 Joubu for the email 08:11 cait that has only email i think? 08:11 Joubu and print is "cardnumber:patron:email:item 08:11 Joubu <<borrowers.cardnumber>>:<<borrowers.firstname>> <borrowers.surname>>:<<borrowers.email>>:<<items.barcode>>" 08:12 Joubu If I remove the contain of the print template, I get 08:12 Joubu No circulation ODUE letter transported by print at C4/Letters.pm line 637. 08:12 Joubu and I don't get the notices are not generated 08:13 Joubu and the notices are not generated 08:36 cait aha! 08:36 cait Joubu: that's exactly what i saw 08:37 cait but ithink they used to be generated - there was no print template added by default when we started I thnk 08:53 Joubu cait: I will confirm that, but it should be done on another bug reporgt 08:53 cait makes sense 08:54 cait i have switched it to signed off already 08:54 cait with a print template, if it works, we got at least a first work around 08:59 cait hi drojf 08:59 drojf morning #koha 08:59 drojf hi cait. that was fast 09:01 LibraryClaire back in a bit' 09:17 Joubu cait: with 15967 the email for the library should still be generated if the print templates are not defined for a given overdue notice 09:26 cait can only say that it didn't happen for me 09:27 cait I don't have access to that installation from here - so can only do more tests tonight 09:27 cait does it work when you empty out your print template? 09:46 Joubu cait: ok back a bit on that: if you check print, you have to have a print template, otherwise you will get a warning when launching the overdue_notices script 09:46 cait i didn't check print 09:47 cait only email and empty print layout... 09:47 Joubu I don't think we got another behaviour before 09:47 cait yes we did 09:47 cait i am seeing it live in 3.18.x 09:47 cait no print template ever added - and the notices to the library are generated 09:48 cait there is some mismatch/misunderstanding somewhere 09:48 Joubu will be hard for me to test, I do not have a valid 3.18 DB 09:48 cait i have plenty... but no mater atm :) 09:49 Joubu the code looks the same between 3.18 and master 09:49 cait checking letter - i only have 1 print line 09:49 cait and that's for holds 09:49 cait the table letter 09:50 cait select * from message_queue where borrowernumber is null shows a lot of notices to the library - last ones from march 09:51 cait if we require a print template now for this to work, it's a change in behaviour - and i think a problematic one, as you won't notice it for a long time possibly 09:51 cait did the print line get deleted when you emptied your print layout? 09:53 Joubu If I remove the print template, the notices for the patrons are not generated and no email for librarians neither 09:53 cait hm ok so you can reproduce 09:53 cait and we don't agree if this is an issue? 09:53 Joubu it's a configuration issue :) 09:53 cait i disagree 09:53 Joubu you check "print" but do not have it defined 09:53 Joubu it cannot work 09:54 cait because i don't want to generate both 09:54 cait i only want a print, if there is no email 09:54 cait and not even a print, but an information to the library, that there is an overdue, where the patron could not be informed 09:54 cait if i check print - i'd sent both to the patron, that's very different 09:55 cait from the intended behaviour here - this is a fallback for users without email adresses 09:56 Joubu ok got it 09:56 cait sorry for being difficult 09:57 Joubu but it was a bug actually :p 09:57 cait hm? 09:57 Joubu that's a "regression" from 14133, the goal was to use the print template for all print notices 09:57 Joubu before 14133 the email or sms template was used for the fallback print notice 09:57 Joubu which was wrong 09:58 Joubu you do not want to generate a sms (so very short) template for printing 09:58 cait hm 09:58 cait first thing... id' say use print when print is available... fall back to email 09:58 Joubu but I understand what you said, I am not saying we should keep this behavior 09:58 cait falling back to sms seems a little odd ye 09:58 cait s 09:59 cait the problem with chaging the behaviour is backwards compatibility 09:59 cait for example we'd have to edit all the notices on update for the libraries... or copy with SQL - if we changed behaviour lease would be an automatic db update i think 09:59 cait so it keeps working 09:59 Joubu but I think we should force the libraries to define correct templates instead of trying to fix configuration issues in the code 09:59 cait i think the email to the library in this case is special 10:00 cait it's kind of print/not print... 10:00 cait it's emails that could not be delivered 10:01 Joubu do you agree with 14133: print templates should be used for print notices? 10:01 cait reading 10:01 cait i think i'd have built in a fallback 10:03 cait ah i see 10:03 cait IMPORTANT NOTE: This test plan does not take into account the notices generated for the staff ("These messages were not sent directly to the patrons."). However the behavior will also change, the print template will be used in all cases. Is it what we want? 10:03 cait it does make sense... the problem is the change in behaviour that is kinda unexpected because undocumented 10:03 cait the easiest would be to say... if no print, try email... if none... error 10:04 cait or we could clena up the database by adding print templates when they don't exist - but that seems a bit messy 10:04 Joubu yes but should we do that for any letters? 10:05 Joubu i.e. at the C4::Letters::GetPreparedLetter? (that's the easy and global solution) 10:05 cait it woudl make sense to me at first glance... 10:05 Joubu (and the one which may introduce regressions) 10:05 cait but i might need to have lunch and think about it more 10:05 cait the problem is ee is that that the change has been backportet 10:05 cait backported :( 10:06 cait so this is already a problem for some libraries - it would definitely be for us 10:06 cait so we should try to figure this out somehow quickly :( 10:06 Joubu Yes but I have fixed a bug and written a big IMPORTANT NOTE, as you notice 10:06 cait i didn't realize the problem until i tested yesterday 10:06 Joubu I don't know what I can do more.... 10:06 cait yeah - but the libraries don't look at that 10:07 Joubu As always, I have tons of solutions, but never know which one should be picked 10:08 Joubu I really think that should be fixed at the DB level (cp email to print if print does not exist) 10:08 Joubu I am a bit afraid to patch C4::Letters to be honest... 10:08 cait might also be confusing... but maybe 10:09 cait i am leaving an note on the initial bug 10:09 cait could we patch only the overdues email to the library? 10:09 cait keeping it a bit more granular? 10:10 Joubu I don't think so, we need the same behaviour for all letters 10:10 Joubu why would it be more granular? 10:10 cait just because the email to the library is kind of a special case 10:10 cait it's the only notice we do that for 10:13 cait Joubu: will try to ask around, but have to eat some lunch first 10:13 Joubu yep, enjoy :) 11:38 LibraryClaire hello 11:38 wahanui what's up, LibraryClaire 11:41 magnuse howdy LibraryClaire 11:42 LibraryClaire hey magnuse :) 12:23 oleonard Hi #koha 12:23 cait hi oleonard! :) 12:27 kidclamp morning 12:27 cait morning kidclamp 12:27 * LibraryClaire waves 12:30 drojf hello again 12:33 druthb drojf! 12:33 druthb cait! 12:33 drojf druthb! 12:34 druthb :) 12:39 magnuse drojf! druthb! NateC! tcohen! 12:40 NateC magnuse!! 12:40 wahanui magnuse is a Norwegian giant. 12:40 druthb magnuse!! 12:40 wahanui magnuse is a Norwegian giant. 12:40 drojf magnuse! 12:40 wahanui magnuse is a Norwegian giant. 12:40 drojf heh 12:41 cait druthb! 12:41 drojf cait! 12:43 tcohen hi #koha, good morning! 12:46 marcelr hi tcohen 12:46 magnuse if you say so 12:52 tcohen what exciting stuff will we work on today? 13:08 cait tcohen: do you wnat me to pick something for you? ;) 13:10 cait bug 14633? 13:10 huginn 04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14633 major, P4, ---, John.Seymour, Signed Off , apache2-mpm-itk depencency makes Koha uninstallable on Debian Stretch 13:10 tcohen cait: that cannot be fixed, unfortunately 13:11 cait there is a patch 13:11 drojf a magical patch 13:11 cait dunno :) 13:11 cait but asking QA, not miracles :) 13:11 drojf if it fixes unfixable problems :) 13:12 tcohen i thought itk wasn't available anymore 13:12 tcohen i'll QA it 13:23 magnuse tcohen++ 13:24 cait wow 160 registrations for kohacon 13:24 cait new record? 13:24 tcohen :/ 13:25 cait why the sad face? 13:26 cait oh 13:26 tcohen cait: can u take a look at 15263? 13:26 tcohen I'm not sure I can QA it as I signed part of it already 13:26 cait and jonathan too 13:26 cait tsk :) 13:26 cait yeah i can try 13:27 cait maybe not today, but sometime this week? 13:28 magnuse bug 15263 13:28 huginn 04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15263 enhancement, P5 - low, ---, mirko, Signed Off , XSLT display fetches sysprefs for every result processed 13:28 cait ther are also signed patches for the REST API now 13:28 wahanui okay, cait. 13:28 cait forget there? 13:28 wahanui cait: I forgot there 13:28 magnuse nasty xsltses 13:28 oleonard ther? 13:28 wahanui hmmm... ther is any patche to solve it? or a feature in the queue to make the default value configurable 13:29 cait the? 13:29 wahanui the is a stop word 13:29 magnuse STOP! 13:29 oleonard wahanui: forget ther 13:29 wahanui oleonard: I forgot ther 13:29 cait forget the 13:29 wahanui cait: I forgot the 13:29 cait the is not a stop word 13:29 cait the? 13:29 wahanui the is, like, not a stop word 13:29 cait also... 11904 in terms of performance would be good 13:30 cait bug 13216 13:30 huginn 04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13216 enhancement, P5 - low, ---, liz, Signed Off , fix koha-shell to work with git/dev installs 13:35 magnuse bug 11904 13:35 huginn 04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11904 enhancement, P5 - low, ---, julian.maurice, Signed Off , Proposal for a uniform way to send messages to user interface 13:53 drojf are there problems regarding independentbranches in OPAC, or only in staff client? does independentbranches touch OPAC at all, or is only singlebranchmode relevant there? 13:55 tcohen isn't singlebranch being deprecated? 13:56 drojf oh true, pushed to master 13:57 cait hm i think it doesn't have consequences for the opac 13:57 cait maybe reserveforothers has 14:16 tcohen bug 11904 14:16 huginn 04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11904 enhancement, P5 - low, ---, julian.maurice, Signed Off , Proposal for a uniform way to send messages to user interface 14:20 drojf so to seperate the opacs in one koha instance there are only the env variables (opac_search_limit etc.) and no syspref? or am i missing something? never thought about it before 14:21 cait drojf: nope i think you are right 14:21 cait the only way is the apache config things 14:22 cait and not sure if that works with plack? 14:22 drojf the real way is to use seperate instances ;) 14:22 cait ... i know 14:25 ccordova hi, guys :-) 14:26 cait hi ccordova 14:31 ccordova I have a problem OPAC result, I pb 3.22.5 and plack when conducted a search shows the results but when passing a resulltado page to another page of results shows no result. 14:32 ccordova *pb = Koha 14:34 ccordova my opac 14:34 ccordova http://201.215.95.132/ 14:36 ccordova sorry for my English :( 14:39 cait where are you from? 14:39 ccordova Santiago 14:39 cait i like the colors :) 14:40 cait tcohen: around maybe? 14:40 tcohen yes 14:40 cait ccordova: i see what you mean, but not sure what may causing this 14:41 cait tcohen: do you have an idea maybe? 14:41 wahanui maybe is a momentaneous error 14:41 cait paging seems broken 14:41 cait hm it looks like there is something odd about the sorting param 14:41 cait http://201.215.95.132/cgi-bin/koha/opac-search.pl?q=der%20%23relevance_dsc&offset=5&sort_by=relevance_dsc 14:41 cait it appears twice 14:42 tcohen something breaking JS? 14:43 cait ccordova: hm wild guess... but could you try turning off the did you mean? 14:45 cait the paging is really odd 14:45 cait shows 10 a page, but only jumps by 5? 14:45 cait http://201.215.95.132/cgi-bin/koha/opac-search.pl?q=espanol&offset=5&sort_by=relevance_dsc works but the generated 14:45 cait http://201.215.95.132/cgi-bin/koha/opac-search.pl?q=espanol%20%23relevance_dsc&offset=5&sort_by=relevance_dsc doesn't 14:46 cait hm don't see the problem in our 3.20.3 without plack 14:48 ccordova to remove the same thing happens plack 14:50 cait ccordova: i am sorry, i am not sue what would causing this - it's odd 14:52 cait ccordova: did you change the search parameters maybe? 14:52 cait are you running any local changes? 14:54 cait kidclamp++ thx for commenting 14:57 Joubu cait, drojf: OVERRIDE_SYSPREF_ works under plack 14:57 cait Joubu++ thx 14:57 cait so the separate opacs thing shoudl work too? 14:58 Joubu afaik yes 14:58 drojf thanks Joubu 14:59 drojf are these things documented somewhere besides bz and/or rfc in the wiki? 15:01 kidclamp Joubu - with no extra work? I nkow we were hitting some issues trying to implement 15:04 tcohen Joubu: charconv is just... useless 15:04 Joubu kidclamp: please open a bug report if it does not 15:05 Joubu tcohen: you don't like useless subroutine? 15:05 tcohen Joubu: have you read it? 15:06 tcohen it always returns what was passed 15:06 Joubu tcohen: yes, it's magic 15:06 tcohen without touching it .-D 15:06 ccordova cait : parameters of search http://snag.gy/HuNDt.jpg 15:06 tcohen if smth -> return @_ 15:06 Joubu else return @_ :) 15:06 tcohen if always_false_statement -> touch it 15:06 tcohen else return @_ 15:06 tcohen AWESOME 15:07 cait ccordova: hm not sure because of the spanish, but try turning UseQueryParser OFF if is ON 15:07 kidclamp @joubu++ 15:07 huginn kidclamp: downloading the Perl source 15:07 Joubu tcohen: see 12357 comment 53 15:07 kidclamp joubu++ 15:07 Joubu kidclamp: what did I do? 15:07 cait Joubu: he is going to file lots of bugs for you now ;) 15:08 kidclamp lots of things joubu :-) general qa and helpfulness 15:08 cait Joubu++ 15:09 cait btw, how should we move on bug 15967 now 15:09 huginn 04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15967 major, P5 - low, ---, jonathan.druart, Needs Signoff , Print notices are not generated if the patron cannot be notified 15:09 cait i don't want to sign if off currently 15:09 cait we are a bit low on qa people 15:09 tcohen hey, someone please pass 14598 15:10 cait bug 14598 15:10 huginn 04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14598 major, P5 - low, ---, tomascohen, Signed Off , itemtype is not set on statistics by C4::Circulation::AddReturn 15:10 tcohen it doesn't make any sense not to have it already 15:10 Joubu cait: I think my follow-up is ok 15:10 kidclamp I will SO 15967 if noone beats me to it today 15:10 reiveune bye 15:10 Joubu cait: we may fix it better later, but to make it backportable easily I'd prefer to have a quick&easy patch 15:11 cait ok 15:11 cait not opposing that :) 15:11 cait i will be stand by for QA 15:18 cait thx for the help kidclamp 15:19 pianohacker Joubu: *blink* OVERRIDE_SYSPREF_ works under Plack? How? 15:20 pianohacker also hi y'all 15:22 Joubu pianohacker: I though it worked.... it does not? :) 15:23 pianohacker Joubu: my understanding was that it couldn't, because SetEnvs aren't passed through the reverse-proxy to starman 15:25 Joubu pianohacker: I am on something else, but will try to have a look of someone confirm and open a bug report 15:26 pianohacker okey doke. No rush :) 15:26 Joubu maybe tcohen already had a look 15:33 bag morning 15:36 cait morning bag 15:37 pianohacker the most fearsome of bags 15:37 bag roar! 15:38 drojf uberbag 15:39 bag uberROAR! 15:39 drojf :D 15:48 magnuse bag: HI 15:48 wahanui que tal, magnuse 15:56 drojf meeting, later #koha 16:33 Joubu bye #koha! 16:58 cabillman If I need to delete a ton of items. What would be the best method? Is it safe to just delete them from the items table? 16:58 cabillman or should i select the barcodes out and paste them into the batch item deletion catalog tool? 16:59 cabillman I have to research what caused the problem but a librarian was trying to catalog something and koha generated 114,000 items 17:15 oleonard cabillman: All items on the same record? 17:16 oleonard If so it might be simpler just to delete that record and re-do it. 17:16 cabillman oleonard: same biblionumber but it has a number of valid items as well 17:17 oleonard cabillman: I don't know what the practical limit is on the batch item deletion tool, but it works. 17:18 cabillman there are 400 valid items so deleting that doesn't seem practical 17:19 cabillman oleonard: so deleting them from the sql table isn't an option? 17:26 cabillman quickly looking at the code it looks like it calls _koha_delete_item which just copies the data to deleted items before performing the "delete from" query on the items table 17:35 cait you would also have to reindex 17:35 cait if you delete item data 17:35 cait or change it 17:35 wahanui cait: that doesn't look right 17:36 cait i missed the beginning of the conversation- but why not use the batch delete tool as oleonard suggested? 17:36 cabillman cait: there are 114,000 items to delete 17:36 cait hm ok 17:36 cait that#sa bit much for the tool 17:36 cabillman cait: but they were all created today after 12:00 so i have a sql query to select the items 17:37 cait i think the batch edit works with up to 1000 . but might be mistaken 17:37 cait you could try it with a few first 17:37 cait but as i said - full reindex after 17:37 cait or at least reindexing the records you changed - but that might be about all? 17:38 cabillman cait: thanks for the info 17:38 cabillman cait: i'll wait until the day is over and do a database dump before running the delete 17:38 cabillman and then do a full index 17:38 cait that's also a good idea :) 17:39 cait how did you accidentally add so many items? 17:39 cabillman i have no idea... i got a phone call from a librarian who wasn't sure why all their barcodes were in use 17:40 cabillman if I can figure it out i will certainly open a bug report :) 17:44 kidclamp we did recently see a library do that with about 3 million items 17:55 cabillman oleonard, cait: thanks for your help 18:55 Brooke o/ 19:00 oleonard Bye #koha! 19:02 Brooke @seen nengard 19:02 huginn Brooke: nengard was last seen in #koha 1 week, 0 days, 1 hour, 7 minutes, and 2 seconds ago: <nengard> bug 16151 reported 19:02 Brooke nutty fudgekins. Email it is 20:08 ibeardslee mtj: elves 20:44 huginn New commit(s) kohagit: Bug 7736: (QA followup) Add missing body id and class <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=7e1b0c25cdc9906cc32dbee721236b9d934a19a7> 20:54 drojf evening 20:55 cait hi drojf 20:56 cait meeting finished? 20:58 drojf finished for me, there are stil some people moving server stuff around 21:20 eythian drojf: leave approved for Berlin and Greece. 21:20 eythian Now just need to book things. 21:21 eythian I think that'll be tomorrow night's project. 21:21 wizzyrea eythian: that museum looks weird and amazing 21:21 cait yay:) 21:22 eythian wizzyrea: the museum is normal, the artist was a bit weird 🙂 21:22 wizzyrea well yes. 21:22 wizzyrea that 21:25 eythian cait: I'll also get a couple of days around things to sight see 21:27 rangi florent: Koha::Patron (not borrower :)) its borrower on 3.22.x patron on master 21:27 florent rangi: I was just going to ask. Thank you 21:27 cait :) 21:28 cait we had both at some point - now it's fixed 21:31 cait eythian: cool, when do you arrive in Berlin? 21:31 eythian Dunno yet, Saturday or Sunday perhaps. 21:32 cait ah cool 21:32 eythian You? 21:32 wahanui You are welcome 21:32 cait monday, but quite early 21:32 eythian Cool 21:32 cait my flight is around 9 or so 21:32 eythian I'm going to take the train. 21:33 cait almost 10 hours form here i think 21:33 eythian Is Greece shengen? 21:34 eythian Ah, it is, good. 21:34 drojf eythian: cool! i won't be able to make amsterdam in april, have to postpone that after kohacon 21:35 eythian Oh well, it'll be summeryer then anyway. 21:35 drojf indeed 21:36 eythian Though it was like 19 the yesterday. 21:36 eythian *here 21:36 eythian Too hot. 21:36 eythian @wunder ams 21:36 huginn eythian: The current temperature in Schiphol, Badhoevedorp, Netherlands is 11.9°C (11:30 PM CEST on April 04, 2016). Conditions: Clear. Humidity: 79%. Dew Point: 8.0°C. Pressure: 29.68 in 1005 hPa (Steady). 21:43 drojf @wunder berlin, germany 21:43 huginn drojf: The current temperature in Berlin Tegel, Germany is 14.0°C (11:20 PM CEST on April 04, 2016). Conditions: Clear. Humidity: 67%. Dew Point: 8.0°C. Pressure: 29.74 in 1007 hPa (Steady). 21:44 drojf @wunder thessaloniki 21:44 huginn drojf: Error: _ssl.c:475: The handshake operation timed out 21:44 drojf @wunder thessaloniki 21:44 huginn drojf: The current temperature in Thessaloniki, Greece is 11.0°C (12:20 AM EEST on April 05, 2016). Conditions: Scattered Clouds. Humidity: 82%. Dew Point: 8.0°C. Pressure: 29.98 in 1015 hPa (Steady). 21:44 drojf come on 21:46 * drojf should go to be 21:46 drojf good night #koha 21:46 rangi cya drojf 22:23 florent rangi: based on DBIx::Class documentation, there is a parameter called 'disable_sth_caching' which can be set to 1 to disable SQL statements caching via DBI 'prepare_cached' 22:23 rangi right, so that implies they should all be cached by default eh? 22:23 florent rangi: while I can't find the positive statement about caching, it ... 22:24 florent I think so. 22:24 florent and it is not set in Koha::Schema->connect() 22:24 rangi right 22:24 florent so all SQL statements using Koha->schema->connect are already cached 22:25 florent step 2: isolate some code in an object and write many tests 22:26 rangi sounds good to me 22:26 florent me too :) 22:28 * cait likes 'many tests' :) 22:33 florent cait: me too! We never have enough tests. 22:33 cait so true :) 22:38 rangi florent: http://splitter.koha-community.org/ <--- this might come in handy too 22:38 rangi it helps spot any patches that are waiting that might be touching the same thing 22:41 rangi eg I found 22:41 rangi https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12598 22:41 huginn 04Bug 12598: enhancement, P5 - low, ---, kyle, Patch doesn't apply , New misc/import_borrowers.pl command line tool 22:41 rangi which has this patch 22:41 rangi https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48755&action=diff 22:41 rangi amongst others 22:41 florent Indeed. Thank you. I might even figure out what @bad_dates once did. The variable is declared but not - apparently - used. Or it is and I might break something soon. 22:41 rangi :) 22:47 * Francesca waves 23:05 florent rangi: sorry to bother you but what does 'patch does not apply' mean? Part of bug 12598 involves migrating all the import code into Koha::Patron::Import, which is my next step 23:05 huginn 04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12598 enhancement, P5 - low, ---, kyle, Patch doesn't apply , New misc/import_borrowers.pl command line tool 23:06 rangi it means one of those patches doesn't apply cleanly on master 23:06 rangi it might be a simple conflict, or it might be a big one, but I reckon trying to apply them is a good starting point 23:07 cait khall is not around at the moment - i think if it oculd be fixed/rebased that would be nice 23:07 cait bye all :) 23:07 Francesca bye cait 23:10 florent I will look into it. 23:10 rangi cool