Time Nick Message 00:01 CrispyBran Spoiler alert 00:02 kidclamp you can just "git reset --hard 83e515820abb6e2fa8ddf30f61b3f4972a48ed10" 00:02 kidclamp that si the commit before - but you won't get th eothers 00:03 kidclamp git-revert 18d85cc8d72b55880e88c5364a3819965c8a2831..5acdbb1d2dcb0bd2719f88aa6905c2cf91fab1db 00:03 kidclamp i think that is the other way, but i don't revert often 00:03 CrispyBran At least I found the correct commit id 00:03 CrispyBran :) 00:04 dcook I always do something like "git checkout 83e515820abb6e2fa8ddf30f61b3f4972a48ed10"" 00:04 dcook "git checkout 83e515820abb6e2fa8ddf30f61b3f4972a48ed10"* 00:04 dcook creates a detached head which is easy to get rid of after you're done 00:06 CrispyBran doesn't like the git reset phrase. fatal: ambiguous argument '83e515820abb6e2f8ddf30f61b3f4972a48ed10': unknown revision or path not in the working tree. 00:06 CrispyBran Gotta run for now. Catch you another time. 00:06 kidclamp ah yeah, what dcook said 00:06 CrispyBran Night #koha 01:03 dcook @later tell tcohen You inspired my comment on https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15032 01:03 huginn dcook: The operation succeeded. 01:03 huginn Bug 15032: major, P5 - low, ---, jonathan.druart, In Discussion , [Plack] Scripts that fork (like stage-marc-import.pl) don't work as expected 01:13 alexbuckley hi kathryn :) 01:22 kathryn hi alexbuckley :) 01:37 kidclamp byee 02:53 Bovinity Evenin' 06:14 * magnuse waves 06:14 * dcook waves back 06:15 magnuse yay! 06:15 josef_moravec hi magnuse, dcook 06:15 dcook heya josef_moravec 06:15 wahanui well, josef_moravec is using this "manual" when building packages: https://wiki.koha-community.org/wiki/Building_Debian_Packages_-_The_Easy_Way 06:15 magnuse hiya josef_moravec 06:32 reiveune hello 06:48 jajm hello 06:48 wahanui hi, jajm 06:55 fridolin hi hi 06:59 alex_a bonjour 06:59 wahanui what's up, alex_a 07:00 ashimema Mornin' #koha 07:03 calire hi #koha 07:03 magnuse bonjour europe 08:45 calire matts++ 08:45 calire for being koha-coral king 08:45 andreashm salut! 08:57 ashimema fridolin2: about? 08:57 ashimema oh.. there's a '2' in that nick.. is this the same frido? 09:09 ere So they managed to clone fridolin? 09:09 ere hi #koha! 09:09 * fridolin yes 09:09 ere awesome 09:09 * fridolin and yes 09:09 fridolin ;) 09:09 ere :D 09:10 fridolin_bis I'm tired, fridolin please do my job 09:14 ashimema lol 09:14 andreashm =) 09:15 ashimema sorry for the later than usual release yesterday fridolin , hope I didn't end up putting you behind 09:16 andreashm calire++ 09:17 andreashm matts++ 09:18 fridolin ashimema: no worries, i'm on release right now 09:18 ashimema :) 09:18 fridolin i know how its always difficult to find some calm to release :D 09:18 fridolin i've not all patches but major ones are in 17.11.11 09:19 fridolin i will backport the few missing on next release 09:19 * ashimema has just done some bugzilla tidying.. after my cleanup a couple of months back I hadn't gone back and marked any of my Pushed to Stable as RESOLVED FIXED upon release.. just updated 201 bugs :) 09:19 ashimema brill 09:19 ashimema yeah.. I'm a little behind mater myself 09:20 ashimema master.. even 09:20 ashimema aim to catch up later this week 09:21 ashimema this release seems like it was another fairly big one.. 67 bugs! 09:23 andreashm ashimema: nice! 09:28 tuxayo ashimema++ 09:39 fridolin ashimema++ 09:39 fridolin it contains some old strong bugs 09:39 fridolin Bug 10382 09:39 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10382 critical, P5 - low, ---, alex.arnaud, RESOLVED FIXED, collection not returning to null when removed from course reserves 09:40 ashimema yeah.. I'm really pleased with the bug fix frenzy that's been going on :) 09:40 fridolin indeed, its hunting season on bugzilla ;) 09:45 * Francesca is essay writing 10:16 kohaputti how do you like nested subroutines in Koha code? https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19893#c173 – I have only my personal opinion about this but the argument David has is pretty good about it being a private helper method that is just needed inside of the method. 10:16 huginn Bug 19893: enhancement, P5 - low, ---, glasklas, Needs Signoff , Alternative optimized indexing for Elasticsearch 10:16 kohaputti nested subs are not really used anywhere else in koha so it just caught my eye 10:16 kohaputti and is also a new concept to me 10:17 ashimema personally.. I'm not really a fan of it.. but it was how the code was originally written too so it followed a certain style 10:17 ashimema it's not really especialyl perlish 10:17 kohaputti ashimema, you mean the ES code had nested subs before? 10:18 * kidclamp waves 10:19 ashimema when I asked our more senior perl devs this was their answer: 10:19 ashimema `Dont do it - Perl does not nest subs - I think the lexer rewrites your code as a series of subs --- basically you're fooling yourself into thinking you are writing pascal` 10:20 ashimema and.. that took me back to my pascal days at Uni ;) 10:20 ashimema yeah.. I think allot of the ES code was written in this somewhat strange to a perl programmer style 10:21 kohaputti ashimema, hmm ok, maybe you want to share this in the report? :D 10:21 ashimema I actually braught it up here a week or so ago.. but no-one was around to talk about it ;) 10:21 kohaputti but either way we go, how does the docs work in nested subs? 10:21 ashimema well.. I decided that as the style pre-existed I'd probably pass qa on it (even though I don't really like it) at this point and hope it got tidied up later ;) 10:21 kohaputti this would be maybe something we should put in the coding guidelines 10:22 ashimema kidclamp.. I asked you about that didn't I? 10:22 ashimema did eythian write this stuff in the first place.. I've not dug into the history of the file enough 10:23 kidclamp I believe it was mostly Eythian, Rangi may have gotten his hands dirty at some point too 10:23 kidclamp I don't remember you asking about it, but I agree, it is confusing 10:23 eythian hi what? 10:24 eythian nested subs aren't things that are really done in perl, but closures are. 10:25 eythian See also: https://hop.perl.plover.com/ 10:25 ashimema hehe.. that books on my shelf waiting for me to have a chance to read it. 10:29 kohaputti hmm, today is developer IRC meeting? 10:30 ashimema it is indeed 10:30 kohaputti we could discuss adding nested subs to the coding guidelines 10:30 ashimema feel free to add it as a discussion topic ;) 10:30 eythian my opinion is that I wouldn't do nested subs, but would happily do "my $f = sub { ... };" 10:31 kohaputti ashimema, here https://wiki.koha-community.org/wiki/Development_IRC_meeting_24_October_2018? 10:31 ashimema yup 10:31 eythian really, there's not such a thing as a nested sub, all named sub namespaces are global anyway 10:31 ashimema yeah.. I was thinking of suggesting switching it to coderefs instead too eythian :) 10:32 eythian well, global within the module or main:: that is. 10:32 ashimema I did find it odd https://github.com/colinsc/koha/compare/master...PTFS-Europe:bug_19893_es_fast 10:33 eythian ashimema: do you mean https://github.com/colinsc/koha/compare/master...PTFS-Europe:bug_19893_es_fast#diff-dfafeee731f7694c8ad84cf600da4fbdR165 10:34 eythian because that's not a nested sub, it's an anonymous sub. 10:34 ashimema nope 10:34 ashimema https://github.com/colinsc/koha/compare/master...PTFS-Europe:bug_19893_es_fast#diff-9b2d7c9428adb3f8fd6236eec1297cebR341 10:34 ashimema but there are a few of them scattered around if I remember correctly 10:34 eythian ah 10:34 eythian yeah 10:35 eythian I probably wouldn't do that. 10:35 eythian I suspect it was done by someone who expects subs to be lexically scoped like javascript does. 10:35 ashimema that was exactly what I thought 10:35 ashimema it smells of js ;) 10:35 ashimema hmm 10:36 ashimema not that I'm against JS.. just not writing perl as though it is another language ;) 10:37 eythian I'll happily be against JS :) 10:37 ashimema anywho.. thanks for your input eythian 10:37 eythian np 10:57 ashimema @later tell mtompsett are you aiming to get back to bug 20664 or should I take over on it's QA? 10:57 huginn ashimema: The operation succeeded. 11:33 oleonard Hi #koha 11:35 ashimema Hi own 11:35 ashimema oleonard (IRC): did you say you were going to get your son to do bug 21615? 11:35 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21615 enhancement, P5 - low, ---, oleonard, ASSIGNED , "Stock rotation" is at the wrong place in the Tools left side menu 11:36 oleonard I was going to try and then the weekend got away from me. 11:36 * ashimema 's daughter has a 'coding night' tonight and now she's had her first patch passed qa she's keen to try another ;) 11:37 ashimema I can leave it for you if you like or get her to give it a bash 11:37 oleonard Take it away 11:40 * andreashm waves 11:43 * magnuse waves 11:43 magnuse ashimemasdaughter++ 11:58 oleonard Wow ashimema... so much bug wrangling! 11:59 ashimema sorry ;) 11:59 magnuse ashimema++ 11:59 ashimema I cheated ;) 12:00 * ashimema realised he'd not been diligent about going back after release and setting 'Pushed to stable' to 'RESOLVED FIXED' so he scripted that update 12:10 Joubu hi #koha 12:11 Joubu ashimema: IIRC this is usually done by rangi and cait just after the release (after they turned off the notifications :D) 12:11 ashimema for major release perhaps.. we actualls discussedbz status's and things at the last dev meeting 12:12 ashimema we agreed to have a bit of a tidy up 12:14 Joubu it seems that RMaints now update the status when the patches won't be backported, right? 12:14 Joubu it should not happen for the new ones 12:14 ashimema http://meetings.koha-community.org/2018/development_irc_meeting_10_october_2018.2018-10-10-20.03.html 12:15 ashimema So.. what seems to have been happening (and is what I was advised to to by prior rmaints and cait) 12:16 ashimema is.. when a patch is backported to stable I change 'Pushed to master' to 'Pushed to Stable'.. then Frido changes 'Pushed to Stable' to 'RESOLVED FIXED' for patches being backported further.. which just seemed wrong to me.. 12:17 ashimema so proposition was to have a status for each of the maintained branchs 'Pushed to stable/oldstable/oldoldstable' for rmaints to track what they've pushed.. then for each release set RESOLVED FIXED for patches with the 'Pushed to x' for the version they're releasing 12:18 ashimema Once rangi update the BZ status's I was going to submit the required changes for the dashboard and a script on the release-tools 12:19 ashimema in fact.. I've already got the release-tools addition coded:https://gitlab.com/koha-community/release-tools/merge_requests/3/diffs 12:21 Joubu ashimema: indeed, I remembered I read the meeting minutes. Did you update the wiki page? 12:21 Joubu "ACTION: ashimema to update release maintanence page to reflect the change (ashimema, 20:42:17)" 12:21 ashimema yup 12:22 Joubu https://wiki.koha-community.org/w/index.php?title=Release_maintenance&action=historysubmit&diff=22252&oldid=21691 12:22 Joubu that's all? 12:23 ashimema is it.. untill I can point to how to use the updated koha-release script 12:24 ashimema I wasn't going to update it until we merge that and I wasn't going to ask for that to be merged until the bz update to status's is done 12:24 ashimema does that make sense to you? 12:24 Joubu not sure :D 12:25 Joubu I feel like these kind of changes must be discussed on the mailing list, not voted by few people during a meeting 12:25 Joubu in order to make sure everybody got the info 12:25 Joubu I am not seeing anything on the ML 12:26 ashimema fair enough 12:26 Joubu fridolin: "Has Nginx+Plack users, we at Biblibre are interested by any solution." You are volunteering to write it, that's it? 12:27 fridolin Joubu: sorry i'm not good enought for that 12:27 ashimema I did discus it with fridolin, cait and kidclamp before raising at the meeting and wasn't especially intending on voting on it at that meeting 12:28 fridolin but i can try to get a real developper from biblibre 12:29 fridolin tuxayo: are you interessed ? 12:29 fridolin you can enter in the Koha legend ;) 12:30 Joubu fridolin: I was kidding, we are already 3 on it apparently. But stay tuned, to give your feedback and test the patches when they will be ready! 12:31 fridolin indeed, keep me in touch 12:31 fridolin wa are also testers on Ubuntu 12:39 fridolin mmmm 12:39 fridolin http://catalogue.user01-koha.sandbox.biblibre.eu/ 12:40 fridolin CSS is actually broken in master ? 12:40 fridolin does anyone reproduce ? i missed something in webserver 12:40 fridolin there is rewrite for version in CCS and JS files 12:41 fridolin i dont see 404 errors 12:46 Joubu fridolin: bug 20554 certainly broke your local changes 12:46 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20554 enhancement, P5 - low, ---, claire_gravely, Pushed to Master , New OPAC CSS 12:46 fridolin Joubu: but its for sandboxes, there is no local change 12:47 fridolin is there a syspref to adapt ? 12:47 fridolin OpacAdditionalStylesheet ? 12:48 kidclamp nothing should need an adjust that I can think of, clear memached? it never hurts :-)0 12:49 fridolin there is no on sandboxes 12:51 fridolin opaclayoutstylesheet 12:51 fridolin must contain opac.css 12:56 Joubu fridolin: there is a "Failed" in the JS console 12:56 fridolin kidclamp: something is broken 12:56 fridolin maybe [% Asset.css("css/" _ opaclayoutstylesheet) | $raw %] 12:57 kidclamp hmmm 12:59 Joubu that line looks ok for me 13:01 fridolin Joubu: kidclamp ahhh its above : [% SET opaclayoutstylesheet='opac_' _ KOHA_VERSION _ '.css' UNLESS opaclayoutstylesheet %] 13:01 fridolin KOHA_VERSION is not needed 13:02 fridolin its done in Asset 13:02 fridolin so jsute : [% SET opaclayoutstylesheet='opac.css' UNLESS opaclayoutstylesheet %] 13:02 kidclamp ah, i knew it was arond there, patching fridolin? 13:02 fridolin kidclamp: yep 13:03 Joubu fridolin: but the pref is empty on the sandbox 13:03 fridolin Joubu: indeed, thats why its broken, it should use defautl value opac.css 13:03 fridolin oleonard: are you around ? 13:04 * oleonard slinks away 13:04 fridolin just to confirm that syspref opaclayoutstylesheet is by default empty 13:05 fridolin and code uses opac.css if empty 13:05 fridolin Bug 20554 brakes actually 13:05 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20554 enhancement, P5 - low, ---, claire_gravely, Pushed to Master , New OPAC CSS 13:05 Joubu ho yes you are right, opaclayoutstylesheet=opac.css in DB for MARC21 13:06 Joubu that's why we did not catch this issue 13:06 fridolin kidclamp: Joubu do i followup or new bug ? 13:06 * kidclamp promises he isn't sabotaging UNIMARC installs 13:06 Joubu fridolin: ha but it is for UNIMARC too actaully 13:06 fridolin ;) 13:06 Joubu installer/data/mysql/sysprefs.sql:('opaclayoutstylesheet','opac.css','','Enter the name of the layout CSS stylesheet to use in the OPAC','free'), 13:06 Joubu I would be ok with a follow-up, but kidclamp is the boss 13:07 kidclamp followup works for me, jsut don't let me forget to push it :-) 13:07 fridolin go for followup 13:08 Joubu fridolin: submit it and I will sign it off immediately 13:10 fridolin Joubu: kidclamp 20554 13:11 fridolin Bug 20554 followup 13:11 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20554 enhancement, P5 - low, ---, claire_gravely, Pushed to Master , New OPAC CSS 13:28 tcohen hola 13:28 wahanui que tal, tcohen 13:34 mtompset Greetings, #koha tcohen corilynn laurence Freddy_Enrique Margaret 13:34 corilynn mornin' mtompset! 13:34 Margaret hi mtompset! 13:37 Freddy_Enrique Buenos dias tcohen 13:37 * mtompset hums... 13:37 mtompset https://youtu.be/Yu6--WBPBHo?t=58 13:37 Freddy_Enrique hi Koha fellows :3 13:38 Freddy_Enrique XD 13:40 mtompset That's a good tune. :) 13:41 Freddy_Enrique caroline...you there? 13:41 Freddy_Enrique wahanui, have you seen caroline? 13:41 wahanui i haven't a clue, freddy_enrique 13:41 * Joubu is trying to imagine mtompset doing tap dances 13:45 mtompset Joubu, I'm not coordinated enough to do that. :) 13:45 mtompset Greeting, Joubu. :) 13:45 mtompset Greetings and salutations, even. :) 13:45 Joubu if you want to try anyway, send us a video ;) 13:45 mtompset @seen caroline 13:45 huginn mtompset: caroline was last seen in #koha 15 hours, 42 minutes, and 25 seconds ago: <caroline> bye #koha! 13:46 Freddy_Enrique oh... 13:46 mtompset Freddy_Enrique, the @seen command is useful. :) 13:46 Freddy_Enrique thanks mtompset :) 13:46 Freddy_Enrique still got a lot to learn 13:46 mtompset And if you know they come frequently, but aren't currently around, @later tell <whoever> <message> 13:46 kohaputti meeting in 15 min? 13:46 kohaputti same channel, right? 13:47 oleonard wahanui: What time does caroline usually get here? 13:47 wahanui bugger all, i dunno, oleonard 13:47 tuxayo @seen wahanui 13:47 huginn tuxayo: wahanui was last seen in #koha 1 second ago: <wahanui> bugger all, i dunno, oleonard 13:47 * mtompset goes, "huh? What meeting?" 13:47 kohaputti mtompset, developer 13:47 tuxayo @seen wahanui 13:47 huginn tuxayo: wahanui was last seen in #koha 18 seconds ago: <wahanui> bugger all, i dunno, oleonard 13:47 Freddy_Enrique XD 13:47 mtompset kohaputti, agenda? 13:47 kohaputti https://wiki.koha-community.org/wiki/Development_IRC_meeting_24_October_2018 13:47 tuxayo @seen huginn 13:47 huginn tuxayo: You've found me! 13:47 tuxayo :O 13:48 mtompset kohaputti++ # thanks for the link. 13:48 Freddy_Enrique it's a tech meeting... 13:48 mtompset nested subroutines?! YUCK! 13:49 ashimema I'll be late to the dev meeting.. school run duties 13:49 ashimema please accept this as my apologies 13:49 nuentoter o7 morning all 13:53 fridolin kidclamp: sorry my ethernet cable 13:53 * mtompset mumbles, "If you have something nested, you're doing it wrong. Class it!" 13:53 mtompset Well, except for maybe quick inline'ing. :) 13:55 ashimema Lol 13:55 ashimema Agreed 13:55 fridolin kidclamp: ok tu push Bug 20554 13:55 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20554 enhancement, P5 - low, ---, claire_gravely, Pushed to Master , New OPAC CSS 13:57 JesseM Morning everyone! 13:58 kidclamp on the stack frido, getting what I can in this morning 13:58 kohaputti JesseM, hi 13:58 oleonard Hi JesseM 13:58 kidclamp "morning 13:58 ashimema Interesting.. I thought there was already a guideline for the one they're proposing in SQL 13 13:58 kidclamp I don't think we have one 13:58 kidclamp that was me 13:58 kohaputti ashimema, it is in the source code in a readme 13:58 kidclamp I hate having to mess with them 13:58 kidclamp it's already good form, I just want to make it required 13:58 ashimema I see.. and in a wiki page somewhere but perhaps not a guideline 13:59 ashimema We should deprecate the .SQL file version too 😉 13:59 kohaputti should we wait few more minutes so everyone has a chance to join the discussion? 13:59 ashimema If your saying to always use .perl 13:59 m23 Hi #koha 13:59 ashimema Yes, you should.. I'm just talking whilst I can 😉 14:00 kohaputti ashimema, ah, right you were heading out soon 14:00 ashimema https://wiki.koha-community.org/wiki/Database_updates 14:00 ashimema For where it's already in theory a hard requirement 14:01 ashimema Adding it to the guidelines just clarifies it for those who haven't read that page 14:02 kidclamp #startmeeting Development IRC meeting 24 October 2018 14:02 huginn Meeting started Wed Oct 24 14:02:47 2018 UTC. The chair is kidclamp. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:02 huginn Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:02 huginn The meeting name has been set to 'development_irc_meeting_24_october_2018' 14:02 kidclamp #topic Introductions 14:03 fridolin kidclamp: oki super 14:03 kidclamp #info Nick Clemens, ByWater Solutions 14:03 kidclamp #chair Joubu 14:03 huginn Current chairs: Joubu kidclamp 14:03 fridolin #info Fridolin Somers, Biblibre 14:03 ashimema #info Martin Renvoize - PTFS-E, Uk 14:03 ashimema Ish 14:03 tcohen #info Tomas Cohen Arazi, Theke Solutions 14:03 ashimema Waiting at the school gates 14:03 barton @info Barton Chittenden, ByWater Solutions 14:03 huginn barton: Error: The command "info" is available in the Factoids and RSS plugins. Please specify the plugin whose command you wish to call by using its name as a command before "info". 14:03 josef_moravec #info Josef Moravec, Municipal Library Usti nad Orlici, Czech Republic 14:04 calire #info Claire Gravely, BSZ, Germany 14:04 barton oops. 14:04 mtompset #info Mark Tompsett 14:04 barton #info Barton Chittenden, ByWater Solutions 14:04 thd #info Thomas Dukleth, Agogme, New York City 14:04 oleonard #info Owen Leonard, Athens County Public Libraries, USA 14:04 Joubu #info Jonathan Druart 14:05 JesseM #info Jesse Maseto, BWS 14:05 khall #info Kyle M Hall, ByWater Solutions 14:05 Joubu #link https://wiki.koha-community.org/wiki/Development_IRC_meeting_24_October_2018 14:05 kidclamp full house :-) 14:05 ashimema Wow 14:05 Joubu kidclamp and ashimema distracted me and I forgot to fill the agenda with tons of topics 14:06 kidclamp #topic Announcements 14:06 ashimema Lol.. sorry Joubu 14:06 kidclamp you still can joubu, just let us know to reload 14:06 kidclamp any announcements> 14:07 kidclamp last call for announcements... 14:07 Joubu I am planning a "what's on in koha-devel" email soon 14:07 Joubu https://annuel2.framapad.org/p/What_s_on_in_koha-devel # if you want to your topics 14:08 kidclamp #info Joubu is working on a 'what's on in koha devel' - please add topics to keep this email going 14:08 kidclamp #link https://annuel2.framapad.org/p/What_s_on_in_koha-devel What's On email topic 14:08 Joubu also, bookmark https://frama.link/koha_bz_rel_18_11_candidate - but I already sent an email about that 14:09 kidclamp #info Check the 18.11 bug list too 14:09 kidclamp #link https://frama.link/koha_bz_rel_18_11_candidate 18.11 bugs 14:09 Joubu and "sql modes", but later 14:09 kidclamp #topic Update from the Release manager (18.11) 14:09 kidclamp That's me! 14:09 kidclamp I am working on cleaning out the queue - ES has seen some movement, focus there to get things in before slush is appreciated 14:10 kidclamp slush is coming fast 14:10 kidclamp follow the tag as above for things we need in 18.11 14:10 kidclamp I am excited for the release :-) we did good work, thank you all 14:10 ashimema Scary fast 14:11 kohaputti what is slush? 14:11 wahanui rumour has it slush is coming fast 14:11 kidclamp enhancements not in PQA will not be considered for the release 14:11 ashimema Comes before freeze 14:11 kohaputti ok 14:11 kohaputti thanks 14:11 kidclamp freeze means no more enhancements 14:12 Joubu November 2 - Feature slush - no enhancement/new features that have not passed QA will be considered for release 14:12 mtompset Will all Passed QA make it into the next release? 14:12 Joubu November 9 - Feature freeze - only bugfixes will pushed after this - 1st draft of release notes 14:12 Joubu November 16 - String Freeze - 2nd draft of release notes 14:12 Joubu November 27 - Release 14:12 kidclamp mtompset: they are still subject to review - i iwll do my best barring issues 14:13 kidclamp thanks Joubu 14:13 mtompset okay, thank you, kidclamp. :) 14:13 kidclamp that's all i have for right now 14:14 kidclamp #topic Updates from the Release Maintainers 14:14 kidclamp ashimema? 14:14 wahanui hmmm... ashimema is RMaint for 18.05 ? 14:14 oleonard ashimema said he would be late 14:14 kidclamp fridolin? 14:14 wahanui fridolin is indeed, but maybe not on Rmain 18.11 14:14 fridolin yep 14:14 fridolin I juste released 14:14 fridolin https://koha-community.org/koha-17-11-11-release/ 14:15 ashimema We made release 14:15 ashimema And I have some catching up to do now 14:15 fridolin I'm late on 18.05.x for around 10 bugs I will catchup 14:15 ashimema But otherwise, nothing from me 14:15 kidclamp #info ashimema and frido are releasing releases and being cool 14:15 * fridolin asked help for rebase of Bug 21385 14:15 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21385 major, P2, ---, martin.renvoize, RESOLVED FIXED, Vendor search: Item count is incorrectly updated on partial receive 14:16 fridolin ashimema: is on it 14:16 fridolin oh why resolved 14:16 ashimema Yup 14:16 kidclamp #topic Updates from the QA team 14:17 fridolin juste to finish, Plugins with CSS and JS are in 17.11.x 14:17 fridolin enjoy :) 14:17 kidclamp cait joubu marcelr jajm khall tcohen alex_a josef_moravec 14:17 kidclamp fridolin++ 14:17 Freddy_Enrique fridolin++ 14:17 Joubu kidclamp: sql modes? 14:17 tcohen Joubu: leave that for the begining of the next release 14:18 kidclamp I figured you would bring up now 14:18 Joubu tcohen: I do not understand what you mean 14:18 Joubu are you telling we should not discuss it before the release? 14:19 tcohen no, I mean the changes in Koha::Object 14:19 tcohen this is important, needs to be discussed! 14:19 Joubu ok 14:19 Joubu so 14:19 Joubu "strict sql modes" 14:20 Joubu I am going to create a page on the wiki to explain the situation 14:20 Joubu what we did, where we are, and what need to be done 14:20 Joubu also the problems we are facing 14:20 Joubu to me the priority is to stop the regressions, at least the ones tests can catch 14:21 Joubu and so you should focuss on bug 21613 14:21 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21613 normal, P5 - low, ---, jonathan.druart, Signed Off , Turn strict SQL modes on for tests 14:21 Joubu nothing else for now 14:22 tcohen Joubu++ 14:22 kidclamp #info Joubu will create a wiki page for the strcit sql modes issue 14:22 kidclamp #link http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21613 Bug to add strict mode to tests to prevent regressions 14:22 huginn Bug 21613: normal, P5 - low, ---, jonathan.druart, Signed Off , Turn strict SQL modes on for tests 14:23 kidclamp #topic General development discussion (trends, ideas, ...) 14:24 kidclamp ashimema around? 14:24 mtompset ashimema, said he was going to run late, I think... scrolls up... 14:25 mtompset "<ashimema> I'll be late to the dev meeting.. school run duties 14:25 mtompset <ashimema> please accept this as my apologies" 14:25 kidclamp jsut checking, I ma not sure what is needed for trailers discussion, or just to highlight them 14:25 kidclamp #link https://wiki.koha-community.org/wiki/Commit_messages#Other_trailers Other git trailers 14:25 ashimema I'm sorta around but in the playground 14:26 ashimema Git trailer s was a hang over from last meeting.. 14:26 ashimema Wasn't proposed by me.. I just like the idea 14:26 kohaputti are these *other* trailers a recent thing? 14:26 ashimema It's all documented anyways.. just needed highlighting to sleeve new about it 14:26 oleonard Yes kohaputti 14:27 kidclamp cool 14:27 ashimema Especially the Sponsored-by one which I intend to make more use of for release notes 14:27 Joubu should not they be added by RM (automatically 14:27 Joubu ) 14:27 kohaputti I like them, but I cannot say whether the implementation is correct 14:28 mtompset Just one question: would they change they ... "git bz apply; test; git so; git bz attach" workflow? 14:28 mtompset Oops.. change THE... 14:28 mtompset If not, I don't care. :) 14:28 kohaputti mtompset, I think not because sign-off for example can come after of before, it really doesn't matter or does it? 14:29 * ashimema now has child so disappears again 14:29 kohaputti will it get messy if these different types of trailers are not sorted? 14:30 kidclamp I don't see them causing any issues, they should come before the signoff ones I suppose, but is an easy fix 14:30 ashimema Could end up as an RM thing I suppose Joubu 14:30 ashimema But if it does would be good to script as much as we can 14:30 * mtompset chuckles, "Automate everything!" 14:31 kohaputti I think we can try them, right? It doesn't hurt? 14:32 kidclamp agreed 14:32 kohaputti Would the patch submitter be encouraged to add also "Reported-by"? Or just RM 14:33 Joubu Reported-by can be scripted 14:33 kidclamp I don't think we are suggesting these are required either, just encouraged and allowed? 14:34 Joubu "Co-authored-by" we usually follow-up instead 14:34 Joubu "Mentored-by" cannot be scripted 14:34 Joubu "Thanks-to" must be done manually too 14:34 ashimema Yup 14:34 ashimema Not required 14:34 mtompset Will there be a list of "valid trailer tags" on the wiki? 14:34 kidclamp moving on? 14:35 ashimema I used Mentored-by the other day.. 14:35 Joubu we will certainly not use "Mentored-by" much 14:35 kohaputti would Reported-by be a bit redundant since it is already in bugzilla? 14:35 mtompset kohaputti, but that would allow for it to transfer to release notes easier. 14:35 Joubu mtompset: a link has been sent before 14:35 mtompset okay. 14:36 mtompset I didn't follow the conversation closely. 14:36 kohaputti mtompset, but can't we extract that from bugzilla with some automated fashion? 14:36 mtompset Whatever... moving on, like kidclamp said. :) 14:36 kohaputti does each bug say to which version the patches has been pushed? 14:36 kohaputti okay 14:37 kidclamp these are all good questions :-) I think followup on the email thread and we can develop guidelines et.c if these become widely used or we have a need 14:37 kidclamp #topic Review of coding guidelines 14:37 kidclamp #info Using nested subroutines (as discussed in Bug 19893) 14:37 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19893 enhancement, P5 - low, ---, glasklas, Needs Signoff , Alternative optimized indexing for Elasticsearch 14:38 kohaputti What is your opinion on using nested sub like: https://github.com/colinsc/koha/compare/master...PTFS-Europe:bug_19893_es_fast#diff-9b2d7c9428adb3f8fd6236eec1297cebR341 ? 14:39 kohaputti I think if it is not an closure, i.e. it doesn't refer to outer subroutine it should be moved in top level 14:39 kohaputti on the other hand if it is closure, i.e. refers to something in the outer subroutine I think it is fine for it to be a nested subroutine 14:39 mtompset My question is why isn't this classed? It's written like a class. 14:40 mtompset I really dislike nested routines. 14:40 kohaputti mtompset, that could be a good way to do it but I don't think we should focus on this specific piece of code right now? 14:41 kohaputti mtompset, what are the drawbacks of nested routines in your opinion? 14:41 kohaputti I myself noticed it makes it hard to follow the code because I cannot differentiate the to routines from each other 14:42 kohaputti the two routines* 14:42 Joubu maybe moved to Koha::FieldMappings? 14:42 mtompset our code is not cleanly indented across the project. it makes it harder to eyeball. 14:43 mtompset Anything longer than 3 lines (which is an arbitrary okay inline size) should be it's own class. 14:43 mtompset ^it's^its^ 14:43 kohaputti class? You mean subroutine? 14:44 ashimema I'm against it 14:44 kidclamp I think it does affect readability, quick research seems to indicate it is not 'good' perl unless it is a closure 14:44 ashimema It could be a coderef and be a bit more readable 14:45 * ashimema would like to read more into it before just throwing it out.. see what perl best practice and higher order perl suggests about it 14:45 kohaputti I think why it at least for me affects the readability is because it makes the outer function bigger and there there is more things I need to keep in mind at the same time 14:45 kidclamp what is the benefit of nesting it? 14:45 mtompset class as in OO Perl, like Joubu suggested a Koha::FieldMappings or something. 14:45 tcohen if it was a functor, and we were generating multiple functions to be passed around, maybe but this looks obfuscated as-is, with no clear win 14:46 kohaputti kidclamp, the benefit is when you want to refer to something in the outer function, like some variable 14:46 ashimema I'm also not sure it scopes that way he's expecting 14:46 mtompset tcohen, yes, a functor (if I am understanding your word use) would be okay with a coderef. 14:47 josef_moravec quick reading about it now, and I am against it as it is used here... 14:47 tcohen yes, but you might need to generate functions, but hey, this is not the case 14:47 mtompset kohaputti, if you want to refer to an outer variable, then you shoudl have passed it as a parameter. 14:47 josef_moravec ashimema: yes, the scoping looks like one of the problems 14:47 mtompset -- hashref preferably. :) 14:48 josef_moravec I would like to see some refactoring here - move to better place, as Joube mentioned for example¨ 14:48 Joubu the main issue is that you cannot unit test it 14:48 mtompset -- which you can with a class. :) 14:49 mtompset unit_tests++ :) 14:49 mtompset unittests++ :) 14:49 mtompset tests++ :) 14:49 mtompset regressions_tests++ :) 14:49 kidclamp I think we can vote? 14:49 kohaputti mtompset, yeah I think it can be passed as a parameter too but I guess there is some reason for closure (https://perldoc.perl.org/perlref.html). 14:49 oleonard unitte_ests++ 14:50 kohaputti I think we should not rule out closures completely? but just such nesting where it is actually not a closure 14:50 mtompset oleonard++ # love your spelling. 14:50 mtompset No, no... closures of a SMALL size (which is why I threw out an arbitrary 3 lines) would be okay. 14:51 oleonard mtompset: sts is obviously not a word. 14:51 kidclamp kohaputti: agreed, but this doesn't seem to be a closure as understood, this is a named internal subroutine 14:51 kohaputti mtompset, do you mean with a closure a subroutine that refers to outer subroutines variable? 14:51 kohaputti kidclamp, yes it is not closure in that example 14:52 kidclamp I think we can add the rule and consider exceptions if warranted 14:52 mtompset perhaps... but very limited scope and use. Generally be avoided. 14:52 kohaputti so to put it more clearly: I don't want named internal subroutines, but closure are okay 14:52 kidclamp we are all reasonable people :-) 14:53 kidclamp should we develop a coding guideline? 14:53 kohaputti kidclamp, we have one..? 14:53 Joubu I do not think it's needed 14:53 ashimema Possibly, but perhaps not until a few of us have researched further 14:53 kohaputti kidclamp, you meant for this issue? 14:53 ashimema It's a pretty rare occurrence to come across it 14:54 kidclamp yeah, just wondering if we needed action, or just discussion for this one bug? 14:55 mtompset Well, what do we do when someone else writes a large internal routine like this again? 14:55 ashimema Just the discussion I think.. I'd really like to see that bug moving on but if I'm honest I don't think it's ready for 18.11 myself.. not as is. 14:55 Joubu QA will fail it 14:56 ashimema That's why I think we need a bit more thought before we guideline it 14:56 mtompset -- unless it is a newbie QA'r? ;) 14:56 kohaputti I think we could deal with this now, or if you want have some time to think about it and decide in the next meeting? 14:56 ashimema Right now QA picked it up as a code smell and brought up discussions.. that was the right thing to do 14:56 mtompset if we guideline it, the guideline needs to be flexible and not-rigid, but generally encourage a particular coding style. 14:57 mtompset definitely code smell here. 14:57 kidclamp okay. no guideline for now, will someone respond on the bug to say we don't like it? then we can move on 14:57 kohaputti agreed 14:58 * ashimema is overwhelmed at the moment.. anyone else fancy commenting 14:58 kidclamp #info we do not like nested subroutines - we should avoid them 14:58 kidclamp #info Proposal add guideline SQL13: DB Updates - when preparing DB updates you must use skeleton.perl as the base and ensure updates are idempotent (multiple runs should not cause errors) 14:58 * ashimema certainly wants to re-read a few chapters before he would comment 14:59 mtompset I definitely like the idea of idempotent. :) 14:59 kohaputti I can forward this discussion to the author also, and then work together on refactoring it 14:59 Joubu kidclamp: .sql is useful for "INSERT IGNORE INTO systempreferences" single query 15:00 mtompset Thanks, Joubu. I was going to mention that. :) 15:00 Joubu idenpotent is a guideline already 15:00 thd Nested subroutines when used appropriately can produce less complexity and easier readability. 15:00 ashimema They're already meant to be idempotent 15:00 ashimema It's just not in that guidelines page but in a dB guidelines one 15:01 kidclamp Yes,m I just want to make it easily visible and spelled out, it is threaded through other pages on the wiki 15:01 mtompset Joubu, where is that in the guidelines? 15:01 kidclamp the wiki alsmo mentions .sql files. but I don't want to see those anymore 15:01 kidclamp :-) 15:01 kohaputti thd, I will research on that 15:02 kidclamp essentially now the RM cleans up the update if it is not in the right format, and I don't want to , because I am likely to mess it up 15:02 Joubu if it's not in the wiki it's in my head 15:02 mtompset Ah, I see it: https://wiki.koha-community.org/wiki/Database_updates#Idempotent 15:02 kidclamp just want it easily spelled out and obvious so we all know where it is and I can fail QA and make the author fix :-) 15:02 thd ... but the could be an endless recursion trap more easily. 15:04 reiveune bye 15:04 kidclamp So...you think it is okay to add? or you think we don't need it? 15:05 mtompset I think we should centralize it in the coding guidelines. :) 15:05 kidclamp or, "Updates should be written as directed on the wiki at https://wiki.koha-community.org/wiki/Database_updates" 15:05 kohaputti I think it can be added, also to the README in installer/data/mysql/atomicupdate 15:05 mtompset But not necessarily require the .perl thing. 15:05 kidclamp I want to require the perl 15:05 kohaputti mtompset, what would be a case where skeleton.perl cannot be used? 15:06 mtompset kidclamp, when INSERT IGNORE works? 15:06 mtompset kohaputti, it can always be used. It's just that a person doing it, might just be more comfortable providing just the SQL. 15:06 kidclamp it still gets wrapped in perl when moved to updatedatabase.pl - I just want that tobe the work of the author to avoid copying issues 15:06 kidclamp and having to requote sql statements 15:07 Joubu use q|| instead of quote and you are done ;) 15:07 mtompset I prefer q{} 15:07 Joubu lazy kidclamp 15:07 kidclamp unless the statement has pipes in it joubu 15:07 kidclamp lazy developers? 15:08 kidclamp someone is being lazy in either casee 15:08 mtompset Joubu, write a script to turn .sql atomicupdates to .perl for kidclamp. ;) 15:08 kohaputti well doesn't the RM already has enough things to do? Maybe we should provide the script to do .perl file from just the bare SQL 15:08 kohaputti mtompset, exactly 15:09 kidclamp the sekelton.perl is already there, I just don't see a reason to not use it 15:10 kidclamp but this is not getting the support i hoped :-) 15:11 mtompset kidclamp, whenever I do an atomic update with a .perl, I'll remember to use your name in vain. ;) 15:11 nuentoter lol 15:11 kohaputti mtompset, have you come a cross anybody who was uncomfortable doing the update with the skeleton.perl? 15:12 Joubu I would suggest to update the wiki and tell people we prefer a .perl even for single statement 15:12 mtompset Me. I rather just get the SQL right. :) 15:12 Joubu but we should not FQA for that reason 15:12 mtompset I can support Joubu's suggestion. :) 15:12 kidclamp fair engouh, but I will sigh a lot -- jsut know you make me sigh :-) 15:12 kohaputti mtompset, so your worry is that the perl wrapper creates bugs? 15:12 mtompset Can you make a video of that? ;) 15:13 Joubu kidclamp: rely on QA team to move .sql to .perl 15:14 kidclamp at that point I feel lazy, anyways, I think we can then just add a link tothe update page ont eh coding guidelines and call it a day? 15:14 mtompset kohaputti, nope. Just worried it will do something else magical. Don't trust those inner workings. EEEEEVIL! ;) 15:14 kidclamp mtompset: it is just a wrapper - the rm moving it or you putting it there are the same 15:14 oleonard Is anyone else arguing that this is a bad idea? 15:15 corilynn mtompset++ on getting the SQL right! 15:15 mtompset It's not a BAD idea... just requiring it... Guideline it... 15:15 oleonard Okay, so... RM says do it. 15:15 mtompset not mandate it. 15:16 kidclamp unless there are objections I will add the link to the coding guidelines page, and add to that page that skeleton is STRONGLY encouraged 15:16 kidclamp no guideline 15:16 mtompset okay. 15:16 kohaputti If the RM would convert it to perl then there is nobody reviewing it. I like it more when the author does it so there is the review part included 15:17 mtompset valid point, kohaputti. 15:17 kidclamp kohaputti++ 15:17 mtompset nothing worse than rolling a package which is broken. 15:18 mtompset kidclamp's name will be used less in vain. 15:18 Joubu cannot happen, tests will fail 15:18 kidclamp anyways, I had enoguh discussion, and no objections 15:18 kohaputti Because there are no vanilla SQL atomicupdates in reality then I think we should only allow .perl atomicupdates 15:18 kidclamp #topic Set time of next meeting 15:18 kidclamp November 7, 21 UTC? 15:19 Joubu #info Next meeting: 7 November 2018, 21 UTC 15:19 kidclamp #endmeeting 15:19 huginn Meeting ended Wed Oct 24 15:19:45 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) 15:19 huginn Minutes: http://meetings.koha-community.org/2018/development_irc_meeting_24_october_2018.2018-10-24-14.02.html 15:19 huginn Minutes (text): http://meetings.koha-community.org/2018/development_irc_meeting_24_october_2018.2018-10-24-14.02.txt 15:19 huginn Log: http://meetings.koha-community.org/2018/development_irc_meeting_24_october_2018.2018-10-24-14.02.log.html 15:19 Joubu to make it set correctly ;) 15:20 kidclamp thanks everyone! 15:20 mtompset kidclamp, Sorry if I joked around too hard. :) 15:20 kidclamp no worries mtompset 15:20 kohaputti thank you! 15:21 kidclamp off for a bit 15:26 ashimema Thanks guys 15:27 * ashimema just caught up.. seems I didn't miss much.. I would have +1 the SQL stuff if I were there 15:34 oleonard Joubu: Is [% FILTER html %][% var %][% END %] the same as [% var | html %] ? 15:36 Joubu yes 15:37 Joubu [% FILTER html %] should be used when there are several lines, but the script I wrote to catch missings filters will not considered it as valid 15:37 oleonard So [% FILTER html %][% var | html %][% END %] is SUPER filtered :) 15:37 Joubu yes :) 15:46 oleonard Joubu: Did you write a test to catch new templates with YUI grid markup or did I imagine it? 15:50 * mtompset grins, "KIDCLAAAAAMP!" 15:50 mtompset Just for the humour of it, while I sign off this bug. :) 15:52 Joubu oleonard: I see that: 15:52 Joubu + {pattern => qr{"btn btn-small}, error => "Contains old bootstrap style for buttons (see bug 16239)"}, 15:52 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16239 enhancement, P5 - low, ---, josef.moravec, RESOLVED FIXED, Upgrade Bootstrap in the staff client 15:52 Joubu + {pattern => qr{"btn btn-mini}, error => "Contains old bootstrap style for buttons (see bug 16239)"}, 15:52 Joubu + {pattern => qr{"modal hide"}, error => "Contains old bootstrap style for buttons (see bug 16239)"}, 15:52 Joubu for buttons and modals only it seems 15:52 mtompset kidclamp, signed off bug 21639 for you. ;) 15:52 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21639 normal, P5 - low, ---, nick, Signed Off , Phone notice transports do not exist for new installs 16:03 mtompset @seen oleonard 16:03 huginn mtompset: oleonard was last seen in #koha 16 minutes and 45 seconds ago: <oleonard> Joubu: Did you write a test to catch new templates with YUI grid markup or did I imagine it? 16:04 mtompset oleonard++ # 96 validation errors to 21. Can we get it lower? bug 21645 16:04 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21645 enhancement, P5 - low, ---, oleonard, Needs Signoff , Clean up library groups template 16:04 Freddy_Enrique mtompset, what does sing off mean? 16:04 Freddy_Enrique sign off* 16:05 oleonard mtompset: What tool did you use and do the errors come from the body of the page (as opposed to include files) 16:05 mtompset oleonard, Good point about include files... I just cut and paste page source into validator.w3.org 16:06 mtompset Freddy_Enrique, read through this page and linked to pages. 16:06 mtompset https://wiki.koha-community.org/wiki/Developer_handbook 16:09 mtompset class="edit-group" errors are in the template file. 16:10 mtompset As is the delete-group 16:10 Freddy_Enrique mtompset++ 16:10 Freddy_Enrique nice! 16:11 oleonard mtompset: Fail it and I'll resubmit 16:12 mtompset Nah... just add a patch. 16:12 mtompset no need to resubmit. 16:13 mtompset But I'll fail it, and comment. 16:17 mtompset Greetings, CrispyBran 16:18 CrispyBran hello 16:18 wahanui salut, CrispyBran 16:18 mtompset Have a good lunch, oleonard :) 16:21 * CrispyBran waves 16:31 mtompset Oh bleep. 16:32 mtompset Is anyone familiar with how to untangle the mess that is nested reuses? 16:33 mtompset C4::Circulation uses C4::Members uses C4::Accounts. C4::Circulations also uses C4::Accounts. And then, C4::Circulation also uses C4::Overdues which uses C4::Accounts. 16:33 mtompset Is this where use order becomes important? 16:34 * CrispyBran pulls out the grenade launcher 16:42 * ashimema cries whenever that happens mtompset 16:43 mtompset Sent an email to the devel list. 16:43 mtompset ashimema, cry more.... 16:43 mtompset C4::Circulation -> Koha::Accounts -> C4::Circulation qw (ReturnLostItem); 17:02 mtompset Have a good lunch, oleonard? :) 17:15 oleonard Meh. 17:24 huginn News from kohagit: Bug 15766: DBRev 18.06.00.044 <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=008112025520ba05c162493a55f2f88e70db74d5> 17:24 huginn News from kohagit: Bug 15766: Fix incorrect GROUP BY <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=85da34c94a5a92df78e03a505054a241dc797877> 17:24 huginn News from kohagit: Bug 15486: DBRev 18.06.00.043 <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=0c4ed27244323f970b8b1e2411844ed85511f01b> 17:24 huginn News from kohagit: Bug 20554: (follow-up) KOHA_VERSION is not needed in opaclayoutstylesheet <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=95917f384175008aa67bd87c378f7d469d61219e> 17:24 huginn News from kohagit: Bug 21511: (RM follow-up) Use orders.count instead of new variable <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=679cb3999f01dda76c168bde4e37c5d203348518> 17:24 huginn News from kohagit: Bug 21511: Don't show acquisition details on subscription detail when there is no... <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=777e29e420b96ed7e6d63aff7c6b69ecc9c1d2ee> 17:24 huginn News from kohagit: Bug 15486: (follow-up) Default to unlimited <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=c5641af82a0ac943bea6439a87c5b9b619bf45cf> 17:24 huginn News from kohagit: Bug 21216: (follow-up) Exclude some columns from export <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=023ab5845ac53c9f80ee69562d6c26f85b8b00f4> 17:24 huginn News from kohagit: Bug 21216: Add filter/search options to notices table <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=8960cee9abc3f6a7bd2ffce89a204f273d508926> 17:24 huginn News from kohagit: Bug 15766: (follow-up) Fixing layout of description button <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=461e7375cd9841f6d43e11c273998c3006f04abf> 17:24 huginn News from kohagit: Bug 15766: (follow-up) Correct merge errors <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=f4b7e470e774c43d9b9771f103cefc53d5ecc71f> 17:24 huginn News from kohagit: Bug 15486: Make circ rules UI handle holds_per_day <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=e530107595e1c64a2073ff36a7fc94e07dcc8779> 17:24 huginn News from kohagit: Bug 15766: (follow-up) Hide description field if the batch is empty <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=ba8ec9bfe2c538f4b586f30426e2dffb10b814ae> 17:24 huginn News from kohagit: Bug 15766: (follow-up) Save description link as button on patroncard side <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=39772eca518512ce631ccc22c11d377df82901c4> 17:24 huginn News from kohagit: Bug 15766: (follow-up) Making 'Save description' link a button <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=4a34d9ec0611bb48373809f5c88863a952a62fb0> 17:24 huginn News from kohagit: Bug 15766: Adding descriptions to patron card batches <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=5a4f7c6c0e0d4a6e518415a673abd3c406c89b36> 17:24 huginn News from kohagit: Bug 15486: Modify request.tt to handle the new error string <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=1596818ff6433825cd6071fb692bc508773bbd97> 17:24 huginn News from kohagit: Bug 15486: Extend CanItemBeReserved so it handles daily holds limits <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=fb0ab55fd09457a4d7559d2af9774de9456bb2f2> 17:24 huginn News from kohagit: Bug 15486: Unit tests <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=dad1324b1b25ac018dad84176537c298edffa75d> 17:24 huginn News from kohagit: Bug 15486: DB structure change <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=0d68871836554cf4eb7dd2c1f44aa136fb041157> 17:27 caroline hello everyone! 17:27 * CrispyBran waves 17:30 * oleonard waves goodbye 17:32 caroline now I have the Beatles song in my head... 17:33 corilynn we all live in a yellow submarine, caroline? 17:33 caroline no the hello/goodbye song (not sure of the title) 17:34 corilynn google says it's called hello, goodbye 17:34 caroline hehe! keeping it simple! 17:36 huginn News from kohagit: Bug 15766: (RM follow-up) Schema updates <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=2e3eb44319db63f33ac05a51782c0692aa9a95d8> 17:38 Joubu kidclamp, ashimema, tcohen: bug 21596 comment 19 to review please 17:38 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21596 normal, P5 - low, ---, jonathan.druart, Pushed to Master , Handle default values when storing Koha::Patron 17:49 mtompset Okay.. that was a nasty blip... 17:50 CrispyBran Question for #koha. An item has a hold and has a waiting status. Patron picks up hold without checking it out. Returns it at another branch. Should Koha A) send the item back to the branch it should be waiting at, and wait out a pickup the patron no longer needs or, B) ask the staff member about it? Should the hold be cancelled at that point by staff? Should it automatically be cancelled and trigger the next hold? 17:51 * mtompset cringes. 17:52 caroline CrispyBran: I would say pop-up saying this items is on hold and waiting at library X, with options confirm and transfer, confirm, transfer and print slip, and ignore 17:54 caroline Because the scenario could be different, it could be someone else who brought back the item, not necessarily the person it was meant for. 17:55 CrispyBran The question is, if it could be any number of reasons why it ended up in the wrong place, what is the best way to deal with it. 17:56 CrispyBran How should this get resolved? Call the patron and check if they had read it? 17:57 caroline CrispyBran: I'm not sure how other libraries deal with holds waiting forever on the shelf, but after several days, yes I think a call is in order 18:00 caroline Native English speakers: is "Showed" supposed to be "Shown" in a sentence like 20 orders showed/n? 18:00 caroline Showed sounds weird 18:00 CrispyBran Should be shown 18:01 caroline thx! 18:01 caroline does "Displayed" sound better than shown? 18:04 CrispyBran That or Showing 18:04 CrispyBran Either would work for a singular or plural situation. 18:04 CrispyBran Or Displaying 18:05 nuentoter after 1 week on holds list, we give calls. after 1 more week they go back on shelf. "Displayed" gets my vote 18:05 mtompset orders has nothing to do with holds, right? 18:05 mtompset (making sure I'm parsing the conversations correctly) 18:05 CrispyBran two different topics mtompset 18:06 nuentoter yeah sorry 18:06 caroline Its for a syspref like PatronsPerPage, but I guess they wanted them to start all the same. The dev called it TableMaxShowedOrders, TableMaxShowedItemTypes, etc but it just sounds bad to my ears 18:06 mtompset and these orders are they still in the midst of processing, or are they processed, or is there a mix? 18:06 CrispyBran I'm all for "20 orders revealed for your perusal" 18:07 caroline CrispyBran: lol!! 18:07 nuentoter ^^^ 18:07 mtompset Or the infinitive form. 18:07 nuentoter im with CrispyBran 18:07 mtompset TableMaxOrdersToShow 18:07 mtompset Because this will be used as a limit for showing. 18:08 nuentoter TableMaxOrdersRevealed 18:08 CrispyBran Displayed++ 18:08 nuentoter yeah ^^ 18:08 nuentoter Displayed is better imo tbh 18:09 mtompset ToShow is shorter to type. ;) 18:09 caroline TableMaxOrdersDisplayed 18:09 nuentoter TableMaxOrdersShown 18:09 mtompset And why is the word Table there? 18:09 caroline no idea... 18:10 caroline I was not part of the discussion... I was only SHOWN the result and am not happy with it 18:10 mtompset MaxOrdersToShow (MOTS -- French for Word, right?) 18:10 mtompset come on... have to fit some French in there. ;) 18:11 caroline mtompset: lol!! 18:11 nuentoter Mot is a witty saying 18:11 mtompset And programmers have humour... put in a pun that English speakers will generally miss. :) 18:12 mtompset All the while having a meaningful English phrase. :) 18:13 CrispyBran MaxOrdersDisplayed++ 18:14 CrispyBran It's MOD. ;) 18:14 caroline The things is, it's not only orders, so your word plays won't works with everything :) 18:14 caroline there's suggestions, authorities, funds, etc. 18:15 mtompset Oh... 18:15 mtompset lots of limits. 18:16 mtompset Why not FoosDisplayLimit 18:16 caroline Because of the alphabetic order 18:16 caroline Currently they are all together which is nice 18:17 nuentoter why is the table in there? 18:17 caroline I was told "because it's in a table" 18:17 CrispyBran :/ 18:17 nuentoter oh ... ok good enough 18:18 Joubu PatronsPerPage is almost obsolete as we have DataTables and Ajax to fetch them 18:18 CrispyBran I would rewrite all the pref names. :) 18:18 nuentoter TableMaxShown 18:18 nuentoter TMS 18:19 caroline too many sysprefs? 18:19 nuentoter too much stuff 18:19 nuentoter the S could be changed depending on mood or level of frustration 18:20 Joubu is there a bug number for the "max orders" stuff? 18:20 caroline Joubu: no it's a client development 18:20 caroline I just wanted to know if Showed existed in english 18:20 Joubu which view? 18:20 Joubu ok :) 18:21 Joubu sounds weird to limit the number of results, pagination should be added 18:21 nuentoter showed does, but not in correct usage that way 18:21 caroline Joubu: it's the default to be shown 18:21 caroline Normally it's 20 by default 18:21 nuentoter i showed something to you, ie, past tense. you are being shown is present tense 18:22 caroline but the client wanted to be able to customize 18:22 mtompset oh! Why didn't they YML iy? 18:22 mtompset YML it? 18:22 mtompset { foo: x, bar: y, baz: z } 18:22 mtompset And store that in a DisplayLimits syspref? 18:22 mtompset One systempreference. 18:22 mtompset Oh, never mind.. the de-yml'ing would mean more code. 18:23 mtompset TMS! *laugh* 18:23 nuentoter see, TMS should be it lol 18:24 nuentoter caroline: shown would be the correct english for its use here 18:24 caroline nuentoter: thank you :) 18:25 caroline I didn't know it was a tense thing. I though it was active vs passive 18:29 nuentoter shown is the past participle of the verb show, showed is the past tense 18:35 Joubu bug 21650 # the number of ghost subroutines we have/had is incredible 18:35 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21650 enhancement, P5 - low, ---, jonathan.druart, Needs Signoff , C4::Items::GetLastAcquisitions has never been used and should be removed 18:39 mtompset Joubu, POD coverage failure. 18:39 mtompset Otherwise, I would have signed it off. :) 18:41 Joubu it is expected 18:42 Joubu the POD coverage check is bad 18:42 Joubu it compares % of coverage before and after the patch 18:42 Joubu it only works when sub/methods are added, not when they are removed 18:47 mtompset Fine. Commented as such, and signed off. 18:56 LibTech Hello everyone, I am in need of some assistance. I am trying to set up a koha developer instance but I am encountering a snag. I have installed the koha unstable package and have set it up. It works fine. I then cloned the master respositoy from git and ran koha-gitify. When I try to access koha now, I get the 18.06 setup screen which appears to work, but when it completes, it starts over again rather that going to the koha main screen. Can 18:59 mtompset LibTech, use the kohadevbox. 18:59 Joubu LibTech: which instructions did you follow? 18:59 Joubu you could try to restart plack and memcached 18:59 Joubu (in the other order, first memcached, then plack) 19:00 mtompset https://gitlab.com/koha-community/kohadevbox 19:02 LibTech Restarting memcached did the trick! Thank-you Joubu! 19:05 LibTech mtompset: I have tried kohadevbox before, but for some reason it doesn't seem to want to work on my computer... Thanks for the suggestion though. 19:09 nuentoter i've had issues with kohadevbox too, I just use an old pc that we hadn't had plugged in in forever for any dev stuff. dedicateddevbox 19:18 * cbrannon[A] is now away - Reason : Lunch 19:22 * cait waves 19:22 caroline hi cait! 19:23 caroline training went well? 19:29 cait i hope so :) 19:29 cait tomorrow is the hard part - serials and acq 19:32 nuentoter what the training for? 19:32 nuentoter if you don't mind me asking 19:33 caroline phew, yeah serials and acq are hard to teach (I think it would be hard to teach in any system) 19:35 cait nuentoter: trainign a new library in using Koha 19:36 cait yeah and i think they are not using acq right now, will see ihow it goes tomorrow 19:36 nuentoter oh yeah, thankfully here its only me and one employee that I gotta train on things. can't imagine having to do a full staff 20:06 kidclamp bye koha! 20:09 Joubu @later tell kidclamp follow-up attached on bug 15486 to fix GetHardDueDate.t! <3 20:09 huginn Joubu: The operation succeeded. 20:10 * cait waves 20:12 cait rangi++ 20:15 Joubu Do this bug report makes sense? bug 21651 20:15 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21651 normal, P5 - low, ---, jonathan.druart, Needs Signoff , Force insert of notices related tables during the install process 20:17 cait i thought they already were mandatory 20:17 cait :) 20:22 Joubu they are in "mandatory", but can be unselected 20:22 Joubu I let you imagine what happens if you do not select them but select notices.sql... 20:25 cait but that's true for all the mandatory 20:25 cait that you can unselect 20:26 cait vpn sec 20:27 * CrispyBran is no longer away : Gone for 1 hour 8 minutes 57 seconds 20:27 * CrispyBran waves 21:01 Joubu cait1: others in mandatory are auth_values.sql class_sources.sql sample_frequencies.sql sample_notices.sql sample_numberpatterns.sql 21:02 cait1 hm i think you don't really NEED any of these... maybe notices as some things don't look good when there is no template 21:02 Joubu they need translation anyway, so cannot be move out of this dir 22:01 barton https://i.imgur.com/FUQ8V17.jpg 22:02 caroline barton lol! 22:02 CrispyBran Good luck with that. 22:03 barton Yeah, I should have made it 'REINDEX ZEBRA', but I think 'rebuild' scans better. 22:08 CrispyBran @seen tcohen 22:08 huginn CrispyBran: tcohen was last seen in #koha 7 hours, 21 minutes, and 31 seconds ago: <tcohen> yes, but you might need to generate functions, but hey, this is not the case 22:09 CrispyBran @seen mtompset 22:09 huginn CrispyBran: mtompset was last seen in #koha 3 hours, 9 minutes, and 0 seconds ago: <mtompset> https://gitlab.com/koha-community/kohadevbox 22:09 CrispyBran @seen Joubu 22:09 huginn CrispyBran: Joubu was last seen in #koha 1 hour, 7 minutes, and 34 seconds ago: <Joubu> they need translation anyway, so cannot be move out of this dir 22:11 CrispyBran @seen rangi 22:11 huginn CrispyBran: rangi was last seen in #koha 1 day, 2 hours, 14 minutes, and 45 seconds ago: <rangi> they have them in australia and i have to go there next month, so maybe i can find it then 22:27 caroline bye #koha! 23:03 CrispyBran bye #koha 23:07 Margaret good night #koha 23:11 dcook barton: "This. Is. Koha!" 23:14 dcook Mmm breakfast sandwiches are the best 23:17 barton dcook: I think you need more caps. but it's morning there, so I get it if you're chill. 23:24 dcook So chill right now. Stomach full and quite content. 23:28 mtompset Is there like the THX Dolby Digital Theatre surround sound going on with your "This. Is. Koha!" 23:29 mtompset dcook, what do you consider a breakfast sandwich? 23:29 mtompset English muffin, egg (poached or scrambled) and sausage/bacon? 23:30 mtompset possibly with cheddar cheese? 23:30 mtompset maybe on a homemade biscuit or croissant instead? 23:46 dcook Oh I should have said breakfast wrap 23:47 dcook Fried egg, cheese, sausage, bacon, bbq sauce. Maybe something else I'm forgetting. 23:47 dcook Aussies have tasty breakfasts 23:57 dcook joubu mentions wanting to build a background task daemon and dcook just word vomits all over Bugzilla to him... *facepalm* 23:57 * dcook is just so excited at the idea of having proper background job processing 23:58 dcook Or more proper at least 23:58 dcook properer? :p