Time  Nick          Message
20:27 cait          there are bugs reported to do user roles and such, but no active development as far as I know
20:26 cait          sorry, not atm
20:16 rcmcdonald    Hello! I've got a quick question for the Koha community. Is there a way to set default permissions to a patron category? I want to define LDAP security groups that contain my librarians that trickles down to patron categories, that in turn sets access permissions automatically
20:03 cait          :)
20:02 kathryn       yup!
20:02 cait          all good, did you have a good day?
20:01 kathryn       thank you for my happy birthday the other day cait, I missed you! :)
20:01 aleisha       thank you cait :)
20:01 cait          happy birthday aleisha :)
19:36 AnnaBoten     oleonard, you missed the duck!
19:36 oleonard      !bang
19:34 wahanui       !bang
19:34 AnnaBoten     \_o< quack!
19:28 AnnaBoten     Best time: oleonard with 5.49 seconds
19:28 AnnaBoten     [('oleonard', 2), ('tcohen', -1)]
19:28 AnnaBoten     \_x< oleonard: 2 (149.98 seconds)
19:28 oleonard      !bang
19:25 wahanui       !bang
19:25 AnnaBoten     \_o< quack!
19:25 AnnaBoten     There was no duck! tcohen: -1 (469.76 seconds)
19:25 tcohen        !bang
19:21 AnnaBoten     \_x< oleonard: 1 (251.55 seconds)
19:21 oleonard      !bang
19:17 wahanui       *click*
19:17 AnnaBoten     \_o< quack!
19:10 AnnaBoten     \_x< oleonard: 0 (5.49 seconds)
19:10 oleonard      !bang
19:10 wahanui       !bang
19:10 AnnaBoten     \_o< quack!
19:02 AnnaBoten     There was no duck! oleonard: -1 (301.05 seconds)
19:02 AnnaBoten     Best time: oleonard with 36.32 seconds
19:02 AnnaBoten     [('tcohen', 2), ('oleonard', 1)]
19:02 oleonard      !bang
19:02 AnnaBoten     \_x< tcohen: 2 (300.41 seconds)
19:02 tcohen        !bang
19:02 AnnaBoten     tcohen, you are reloading... (Reloading takes 5 seconds)
19:02 tcohen        !bang
19:02 AnnaBoten     tcohen, you missed the duck!
19:02 tcohen        !bang
18:57 wahanui       *click*
18:57 AnnaBoten     \_o< quack!
18:54 AnnaBoten     \_x< oleonard: 1 (36.32 seconds)
18:54 oleonard      !bang
18:54 wahanui       *click*
18:54 AnnaBoten     \_o< quack!
18:44 AnnaBoten     \_x< tcohen: 1 (1062.92 seconds)
18:44 tcohen        !bang
18:26 wahanui       *click*
18:26 AnnaBoten     \_o< quack!
18:16 AnnaBoten     Best time: oleonard with 7.34 seconds
18:16 AnnaBoten     \o/ oleonard: 3 ducks out of 3: perfect!!! +5 \o/
18:16 AnnaBoten     \_x< oleonard: 3 (47.50 seconds)
18:16 oleonard      !bang
18:16 AnnaBoten     oleonard, you missed the duck!
18:16 oleonard      !bang
18:15 wahanui       *click*
18:15 AnnaBoten     \_o< quack!
18:14 AnnaBoten     \_x< oleonard: 2 (7.34 seconds)
18:14 oleonard      !bang
18:14 wahanui       oops, slipped over
18:14 AnnaBoten     \_o< quack!
18:11 AnnaBoten     \_x< oleonard: 1 (9.07 seconds)
18:11 oleonard      !bang
18:11 wahanui       *click*
18:11 AnnaBoten     \_o< quack!
18:02 AnnaBoten     oleonard took the lead for the week over corilynn with 8 points.
18:02 AnnaBoten     Best time: oleonard with 4.06 seconds
18:02 AnnaBoten     \o/ oleonard: 3 ducks out of 3: perfect!!! +5 \o/
18:02 AnnaBoten     \_x< oleonard: 3 (84.46 seconds)
18:02 oleonard      !bang
18:00 wahanui       !bang
18:00 AnnaBoten     \_o< quack!
17:51 AnnaBoten     \_x< oleonard: 2 (4.06 seconds)
17:51 oleonard      !bang
17:51 wahanui       oops, slipped over
17:51 AnnaBoten     \_o< quack!
17:43 AnnaBoten     \_x< oleonard: 1 (7.51 seconds)
17:43 oleonard      !bang
17:43 wahanui       !bang
17:43 AnnaBoten     \_o< quack!
17:40 AnnaBoten     The hunt starts now!
17:40 tcohen        !start
17:30 koha-jenkins  Project Koha_Master_D9_MDB_Latest build #100: STILL UNSTABLE in 38 min: https://jenkins.koha-community.org/job/Koha_Master_D9_MDB_Latest/100/
17:13 koha-jenkins  Project Koha_Master_D9_My8 build #121: STILL UNSTABLE in 55 min: https://jenkins.koha-community.org/job/Koha_Master_D9_My8/121/
17:08 koha-jenkins  Project Koha_Master_U18 build #583: NOW UNSTABLE in 53 min: https://jenkins.koha-community.org/job/Koha_Master_U18/583/
16:53 koha-jenkins  Project Koha_Master_D8 build #650: STILL UNSTABLE in 35 min: https://jenkins.koha-community.org/job/Koha_Master_D8/650/
16:52 koha-jenkins  Project Koha_Master_D9 build #1112: STILL UNSTABLE in 38 min: https://jenkins.koha-community.org/job/Koha_Master_D9/1112/
16:48 cait          it#s not in marc
16:48 cait          because of circulation rules
16:48 cait          vfernandes:  we can't really determine available for checkout
16:43 vfernandes    maybe the string needs to be changed
16:42 vfernandes    cait: i know but the term "available" can confuse the libraries (available on the shelf or available for checkout)?
16:34 cait          or in a reference area maybe
16:34 cait          in academic librareis you might find things in a reading room
16:33 cait          vfernandes: it just means 'on the shelf'
16:33 ashimema      it's come back now.. very odd
16:33 cait          available doesn't mean available for checkout
16:33 oleonard      Seems okay to me ashimema
16:30 ashimema      is bugzilla crazy slow for anyone else
16:30 ashimema      erm
16:28 vfernandes    any opinion about this: the link "Limit to currently available items" limits the search to the items where onloan is false... shouldn't list the items where onloan and notforloan are false?
16:21 * oleonard    doesn't know who ashimema is lol'ing at and is afraid to ask
16:15 ashimema      lol
16:13 huginn        News from kohagit: Bug 23493: Remove commented lines <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=a5c82cf0da49b81801d215ee2968754ad655c9cf>
16:13 huginn        News from kohagit: Revert "Bug 23493: (follow-up) Disable rowGroup on due date sort" <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=09b37fb63fc6214a74adc36109fb6212df6e11a0>
16:13 Joubu         just saying but having a script that replace 4 spaces with tab when open a file, then reverse that on write, does not seem complicated. I am certainly missing something.
16:13 huginn        News from kohagit: Revert "Bug 23493: (follow-up) Switch to aDataSort for multi-column sorting" <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=9aaf1e31dfb486e47653af8967fbe6d6bdb573a7>
16:13 oleonard      ashimema: I would have assumed it would be a nightmare commit, but I don't actually know why it would be.
16:13 reiveune      bye
16:09 ashimema      me too.. but it would be a nightmare commit to do such a bulk change :(
16:08 ashimema      indeed
15:56 oleonard      ashimema: I found that argument to be very compelling
15:56 kohaputti     oleonard, probably good to do perltidy at the same time
15:55 oleonard-away ashimema: I'm ready to switch only if we can convert all spaces to tabs in one giant update!
15:53 ashimema      I tweeted it on Jan 5th
15:52 marcelr       bye #koha
15:51 marcelr       and it will be enforced
15:51 ashimema      https://www.reddit.com/r/javascript/comments/c8drjo/nobody_talks_about_the_real_reason_to_use_tabs/?utm_source=share&utm_medium=mweb&_branch_match_id=710523928949887991
15:50 Joubu         ashimema: so you have the coding guideline almost written I bet!
15:50 marcelr       U+205F	MEDIUM MATHEMATICAL SPACE
15:50 Joubu         what was it about? apart of tabs and space...
15:50 ashimema      I was always a spaces man untill I read the accessability arguament
15:50 Joubu         ashimema: nope!
15:49 ashimema      did that come from my tweet about spaces vs tabs the other day Joubu?
15:49 marcelr       maybe some Unicode space alternative?
15:49 marcelr       take your time
15:49 thd           Joubu :)
15:49 Joubu         not sure spaces was the correct choice
15:49 marcelr       i like that one do it on 20UTC please
15:49 Joubu         I think we should discuss tab vs spaces
15:49 marcelr       no need to keep doing that :)
15:48 ashimema      thanks guys.. apologies that was another long one.
15:48 Joubu         marcelr: that's what we are doing for years now :)
15:48 huginn        Log:            http://meetings.koha-community.org/2020/development_irc_meeting_29_january_2020.2020-01-29-14.00.log.html
15:48 huginn        Minutes (text): http://meetings.koha-community.org/2020/development_irc_meeting_29_january_2020.2020-01-29-14.00.txt
15:48 huginn        Minutes:        http://meetings.koha-community.org/2020/development_irc_meeting_29_january_2020.2020-01-29-14.00.html
15:48 huginn        Meeting ended Wed Jan 29 15:48:41 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
15:48 ashimema      #endmeeting
15:48 ashimema      #info Next meeting: 5 February 2020, 20 UTC
15:48 marcelr       heh
15:48 Joubu         endmeeeeeting
15:48 marcelr       Joubu that could be confusing too
15:47 Joubu         that's why I answered you 1 every 2 weeks
15:47 thd           Yes, months are not evenly divided by weeks.
15:47 marcelr       twice a month means every 2 or 3 weeks
15:47 ashimema      also.. 5th gives us another week to remind GBSD on 14th :)
15:46 marcelr       5th and 19th and sticking to first and third
15:46 Joubu         or 5 yes
15:46 ashimema      it can be a nice short one with just a vote on that guidline I'll draft this afternoon
15:46 ashimema      I'm thinking 5th.. to keep the ball rolling on getting those API guidelines done and bringing us back to the off week for general meeting.
15:46 Joubu         I would pick Wed 19th
15:45 marcelr       second tuesday fourth wednesday ?
15:45 marcelr       ok 2
15:45 Joubu         every 2 weeks
15:45 marcelr       do we want 1 or 2 dev meetings a month ?
15:44 Joubu         we need to resync, otherwise you will have the same problem next month
15:44 ashimema      or..
15:44 ashimema      so same time as general meeting, but a day early
15:44 thd           We have been on a second week of month or so pattern for general meetings recently.
15:43 ashimema      how about 11th 8pm
15:43 Joubu         1 or 3 weeks then
15:43 ashimema      8pm on the 12th is the general meeting..
15:43 marcelr       the general should be on the 5th ?
15:42 ashimema      12th.. but that's also a general meeting on the 12th
15:42 marcelr       it always toggles, good for NZ bad for Europe
15:42 ashimema      you're right.. i read the wrong line
15:42 ashimema      oops
15:42 marcelr       no it should be two weeks
15:42 ashimema      2pm or 8pm UTC is the usual switch
15:42 thd           That is one week?
15:41 kohaputti     anything fine for me
15:41 Joubu         add a pref is: atomicupdate, sysprefs.sql, pref.inc
15:41 ashimema      what time shall we make it..
15:41 kohaputti     Joubu, what omission you are talking about?
15:41 Joubu         it does not happen often
15:41 kohaputti     Joubu, not for reading a story or reverting feature
15:40 Joubu         you can spot omission easily
15:40 ashimema      so, 5th
15:40 Joubu         kohaputti: no, 1 patch for the new pref is good
15:40 ashimema      same schedule as ever..
15:40 ashimema      #topic Set time of next meeting
15:39 marcelr       ashimema++
15:39 ashimema      'Guess the RM'
15:39 ashimema      lol
15:39 ashimema      ok.. I think we can probably move on.
15:39 marcelr       your latest patch, Joubu ?
15:39 kohaputti     ashimema, here it would have been better to make use of the syspref in the same patch it is introduced
15:39 wahanui       next meeting is https://wiki.koha-community.org/wiki/Next_IRC_meetings
15:39 Joubu         next meeting?
15:39 Joubu         ;)
15:39 Joubu         what did you mean?
15:39 Joubu         I do not understand that one: "Dobbie is a free elf"
15:39 marcelr       it is just about the title
15:39 marcelr       which is great
15:39 ashimema      I tend to break down bugs into small commits to ease rebasing and backporting personally..
15:38 ashimema      fair
15:38 * ashimema    is looking at this as constructive criticism.. I like :)
15:38 marcelr       Use pref X is a bit cryptic
15:37 marcelr       Introduce pref X to allow enabling or disabling
15:37 ashimema      or you would submit it as one bigger patch
15:37 ashimema      what.. for all three patches you would use that single title?
15:37 kohaputti     body is for more explanation
15:37 Joubu         Add a global switch to turn POS off
15:37 kohaputti     Joubu, read the body if you wanna know more details why it is used
15:36 Joubu         The main patch is "Use `EnablePointOfSale` preference"
15:36 marcelr       it describe what you do
15:36 ashimema      how would you have worded them?
15:36 marcelr       Fix sequence in sysprefs.sql and add missing comma is not that bad
15:36 kohaputti     for me those 24478 ones are really clear
15:36 Joubu         bad
15:36 ashimema      I'm confused.. are you highlighting 24478 as bad or good..
15:36 marcelr       yeah
15:36 Joubu         I think we are done :)
15:35 marcelr       {QA follow-up) Some changes
15:34 ashimema      to me.. the really bad ones are 'Address comment #42' and  'follow-up'
15:33 Joubu         24478: Fix sequence in sysprefs.sql and add missing comma
15:33 Joubu         24478: Use `EnablePointOfSale` preference
15:33 Joubu         24478: Add `EnablePointOfSale` system preference
15:33 Joubu         the following sequence of patch does not tell what the patchset does:
15:32 Joubu         it's the bug, not what does the patch
15:32 Joubu         "Column Configuration for pay-fines-table does not hide Account Type properly"
15:32 marcelr       but leaving scope here
15:32 * ashimema    is really struggling to find a bad one going through master
15:31 marcelr       that is also about more comment in the code
15:30 kohaputti     I don't want to reverse engineer the patch
15:30 kohaputti     for me the issue with non-descriptive commits is I have to use more time to understand the patch
15:30 thd           Without checking I presumed that the issue was more about excessive reference to bug report discussion in commit comments where irrespective of the simplicity of making a simple in comment description.
15:29 ashimema      a reminder is good now and then.. so that would be great Joubu
15:29 marcelr       you always change my titles lol
15:29 ashimema      I would never FQA a bug on just the title without having looked at the rest of the code first.. I would usually fix it for them and bring it up as a point that they should learn from my change
15:28 Joubu         #action Joubu will send an email to the list to remind devs about good/bad commit messages
15:28 marcelr       so actually there is no problem?
15:28 ashimema      so, enforcement at QA level isn't terrible.. but equally use a bit of judgement and generally opt on the side of 'be nice'..
15:28 kohaputti     as a qa person I have no idea how many times this person has made non-descriptive patches, so quick fail or no fail is better in my opinion
15:28 ashimema      having a quick scan of the commitlog we're not straying too far from it all that often
15:27 marcelr       right
15:27 Joubu         but only the first time ;)
15:27 tcohen        when I was RM, I fixed them on push too
15:27 marcelr       new authors are allowed to do a lot
15:27 thd           Joubu++
15:27 tcohen        use social skills and don't be too hard on people, unless they keep their wrong habbit
15:27 marcelr       exception for new authors perhaps
15:26 Joubu         so email to koha-devel ?
15:26 kohaputti     It is just one time to fail the patch, the author should learn for the next times
15:26 ashimema      me too
15:26 tcohen        I agree with Joubu
15:26 Joubu         yes, a lot
15:26 Joubu         and FQA if author stick to their bad habbits ;)
15:26 thd           Have we recently passing patches which match examples of the worst commit messages?
15:26 marcelr       it is always hard to fail a patch for title only
15:26 ashimema      I do sometimes clean up commit messages on push.. though I'm inconsistent
15:26 kohaputti     I vote  to definitely enforce it
15:26 ashimema      agreed I think
15:25 Joubu         it should be done at QA level and RM, not necessarily FQA
15:25 marcelr       selective reinforcement
15:25 marcelr       ah
15:24 Joubu         I had to raise it because kohaputti FQA one of my patches for that reason :)
15:24 tcohen        yeah, I hate that
15:24 Joubu         like: do not c/p the bug title in the commit message, that's 2 different things
15:24 tcohen        You don't always follow it :-P
15:24 Joubu         I'd like to know if we should get back to that or not
15:24 Joubu         I tried to enforce this rule when I was RM. And we voted the guideline. The rule is no longer enforced
15:23 ashimema      it was indeed
15:23 marcelr       note the diff between bug and enhancement here
15:23 ashimema      #link https://wiki.koha-community.org/wiki/Commit_messages#Examples Guidline
15:23 Joubu         that was me
15:23 ashimema      #topic Reinforce good commit messages guideline
15:23 ashimema      shall we move on again then
15:23 ashimema      haha
15:22 tcohen        dont_do_things_too_fast++
15:22 marcelr       +1
15:22 tcohen        +1
15:22 * Joubu       vote yes for rule MEETING01 "Provide a new item with context and example, as well as a full guideline" :D
15:22 tcohen        sorry for that
15:22 tcohen        and didn't notice there was a meeting today
15:21 tcohen        to be honest, I planned to explain it on the previoius meeting
15:21 ashimema      +1
15:21 marcelr       so it needs a bit more context
15:21 Joubu         for the next meeting, could you provide existing examples, good and wrong?
15:20 tcohen        so, to avoid manual handling I proposed a guideline
15:20 tcohen        the OpenAPI plugin will reject things that are supposed to be lists and are undef instead
15:19 tcohen        that one, was a trivial one, so we don't need to check for defined on the result, and just pass the result to the rendering party
15:19 tcohen        and we need conventions so we can nicely prefetch related stuffs, and there will be a need for consistency on function call results
15:18 tcohen        we are building a framework for simplifying retireval of data for rendering on the API
15:18 tcohen        haha
15:18 tcohen        the problem is I'm not on the meeting
15:17 marcelr       never do that again tcohen ;)
15:17 marcelr       lol
15:17 tcohen        I think it was me
15:17 tcohen        PERL27
15:16 marcelr       as a reference https://perlmaven.com/how-to-return-undef-from-a-function
15:16 ashimema      in my opinion it should reply in caller context
15:16 ashimema      where's the context..
15:16 ashimema      indeed.. that's what I want to know
15:16 marcelr       who submitted it?
15:16 Joubu         where does come from PERL27? what was the context?
15:16 * ashimema    reaches for the book
15:15 marcelr       but a return; in list context does the same ?
15:15 marcelr       and obivously [] is not empty list but ()
15:15 marcelr       ProhibitExplicitReturnUndef in best practices
15:14 ashimema      remind me where this came from
15:14 marcelr       this is not recommended
15:14 marcelr       do we mean to write return undef; ?
15:13 ashimema      #topic PERL27: Return values consistency. If a method returns a list, and there are no items, it should return [] (empty list). If it is a scalar, undef.
15:13 ashimema      #topic Review of coding guidelines
15:13 ashimema      #info We will clarify the wording of PERL15 (perhaps even splitting it out into it's own guidline) for Koha::Object relationships
15:12 marcelr       ok i will spend 5 mins on rewording ;)
15:12 ashimema      yeah.
15:12 Joubu         moving on then?
15:11 ashimema      feel free to reword it.. I think we voted on it months ago and didn't come up with better
15:11 marcelr       it is a bit hidden in the examples
15:10 ashimema      PERL15
15:10 ashimema      the guideline already exists I believe
15:09 marcelr       add a guideline too
15:09 Joubu         I will too
15:09 ashimema      I do tend to catch them in new code as QA or Push time
15:09 ashimema      OK, I'm happy to work with someone on a bug to catch outstanding cases..
15:09 Joubu         but easy to eyeball :)
15:09 marcelr       yeah it does not sound easy
15:08 ashimema      indeed.. I tried to work out if it was possible but failed
15:08 marcelr       hoped you could
15:08 Joubu         I do not think it will be possible to write such rule
15:07 marcelr       qa tools?
15:07 marcelr       at least dont allow new ones to come up again
15:07 marcelr       even better
15:07 Joubu         should be quite easy
15:07 marcelr       ok
15:07 Joubu         it's not
15:06 marcelr       i would agree with the principle but it might be a major refactor ?
15:06 ashimema      I think that's the question.. I feel like we should plan to replace them all
15:06 Joubu         ie. are all the related an exhaustive list?
15:06 Joubu         ashimema: is there a "replace them all" plan, or it only applies for the new ones?
15:05 marcelr       and not just before starting
15:04 marcelr       i would suggest to copy stuff from the previous meeting directly after that one
15:04 Joubu         marcelr: refresh
15:04 marcelr       my agenda didnt contain this point too?
15:04 tcohen        it was all of us
15:04 Joubu         was not you?
15:04 ashimema      was this tcohen or Joubu that raised it?
15:03 huginn        Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22407 enhancement, P5 - low, ---, koha-bugs, NEW , OMNIBUS: Use DBIC relations to fetch related object rather than searching for the object
15:03 ashimema      #topic Moving on with Bug 22407
15:03 Joubu         we also need to be consistent, that's certainly the most important bit
15:03 ashimema      #info votes are not holding up development and as such will be postponed untill we have written clear guidlines for the contentious field names (date vs date and timestamp vs date consistency)
15:03 tcohen        list
15:03 tcohen        so, lets move on and I will propose something in the lines of what marcelr said about announcing things on the lise
15:02 tcohen        but such is not the case
15:02 tcohen        development is not held by this, the only problem is if things didn't get pushed/integrated because of this
15:02 tcohen        I will think about it
15:02 * oleonard    leaves for another meeting, test 22880!
15:01 tcohen        lets skip this, and we will think of a better workflow for this guidelines
15:01 marcelr       i agree that doing this on a meeting is not very productive
15:00 thd           I do not see much lack of homework but it may be difficult to elicit proper engaged interactive discussion of nomenclature in this context outside of something as real time as an IRC meeting.
15:00 Joubu         marcelr: I am sure we can do more, it's 5 for the first one, but then we will automate that
14:59 Joubu         stuff is not developped :)
14:59 marcelr       Joubu we can rename 12 times an hour
14:59 ashimema      indeed, that's my point
14:59 cait          Joubu: yeah, but stuff doesn't get pushed before vote
14:59 Joubu         wording will not block development anyway, it's a matter of 5min to adjust the patches to a new word
14:59 marcelr       communicate important decisions or guidelines on the dev list ?
14:58 cait          vote to vote?
14:57 tcohen        during the cycle
14:57 tcohen        we could have things moving on, and have a period of time in which people can counter propose
14:57 ashimema      lets move on
14:57 cait          i feel it's unfiar to hold things up now
14:57 tcohen        maybe
14:57 cait          we have had no additional comments added in the last 2 weeks
14:57 ashimema      indeed
14:57 cait          the rfc are open for commenting
14:56 cait          i am not sure if discussiong things biweekly here is so helpful too - because people don't do homework on this
14:56 tcohen        [off] sorry, grabbed by another meeting
14:56 tcohen        the sooner we sort things the better, but we passed the mappings on the development, we will remap if required
14:55 tcohen        code is being written, that's correct
14:55 cait          exactly
14:55 ashimema      well likewise.. but I don't feel it is being held up.. code is being written and we have time still in the cycle to correct the terms as per a guidline once the guidline it done
14:55 cait          and maybe not backport before we have done that
14:55 cait          i am saying fix according to guielines within this cycle
14:55 thd           Certainly we do not come to understand what guidelines should be without struggling with the questions in a real world context.
14:55 cait          changing timestamp to something else in oen cylce is not hard
14:55 kohaputti     cait, but we cannot fix this without breaking backwards compatibility?
14:55 ashimema      right.. I think so we don't run out of time we should schedule a guideline for next meeting and not vote this week
14:54 cait          and those have been held up already
14:54 cait          1.5+ months is a lot in a cycle
14:54 marcelr       tomas can proceed
14:54 marcelr       guidelines first, fixes later on
14:54 cait          have tomas continue... and then go in and fix according to guidelines at hackfest
14:54 cait          we can still fix that
14:54 cait          my feeling is we shoudl not hold this up forever for one small decision
14:53 thd           ashimema++ #guidelines
14:53 cait          they have both
14:53 cait          like borrowers
14:53 ashimema      then work on making them all consisntent
14:53 cait          but sometimes that's just updated_on
14:53 cait          yes
14:53 ashimema      hense me feeling we need to write a guideline properly first..
14:52 ashimema      there is no 'stay consistent' yet because we are inconsistent
14:52 ashimema      there are updated_on littered around too cait
14:52 marcelr       biblio has a timestamp yes
14:52 cait          timestamp
14:52 cait          i think stay consistent for now
14:52 cait          holds has timestamp too
14:51 marcelr       no it should
14:51 cait          looks like bilbio doesn't have anything
14:51 marcelr       sounds nicer
14:51 ashimema      creation_date, modification_date
14:51 cait          hm items has timestamp
14:51 ashimema      `ed` on `ion`
14:50 cait          what did we do on items?
14:50 cait          heh
14:50 marcelr       to prevent updated_date
14:50 marcelr       what about created_date and modified_date ?
14:49 thd           If we do not always have dates as date times then we should distinguish between them.
14:49 cait          and we have established that we like using _date if it's a date.. but it's a datetime
14:49 cait          so the first 2 are out
14:49 cait          we have verified that it's not creation only
14:49 cait          but back to this one
14:48 ashimema      that's what I suggested
14:48 ashimema      yup
14:48 cait          reducing is easy, but we shoudl stay 'true'
14:48 marcelr       and show date in presentation perhaps
14:48 cait          +1
14:47 marcelr       i think we should store datetimes
14:47 cait          thd: i don't think we do or should do that
14:47 thd           Even if we have only a date and not a time for some source a hypothetical time might be attached as 12.00.00 .
14:47 cait          then we coudl say the last column counts
14:47 cait          i think the only place where we need a decision is date date_created creaton_date updated_on
14:46 Joubu         I think we should have the whole thing/view and vote in one go
14:46 ashimema      and we sebtled on *_date I believe.. but didn't record it
14:46 cait          and also using _id - when we actually store an id
14:46 marcelr       consistent
14:46 marcelr       or trust one dedicated and consitent person to do that for us ;)
14:46 cait          date_ vs. _date
14:45 cait          there was some discussion ont hat last time
14:45 ashimema      we really need to add some guidlines for these rather than voting on them per endpoint
14:44 * ashimema    thinks we should really always store timestamp and then reduce to date only if/when we want to..
14:44 cait          i think we have not been cehcking all the others so far
14:44 cait          good question
14:43 marcelr       or do we differentiate that?
14:43 cait          we have places where update/create are 2 things
14:43 marcelr       does _date also mean datetime ?
14:43 cait          timestamp seems not so bad
14:43 cait          if it#s both
14:42 ashimema      I don't like 'timestamp' on it's own as it's not clear whether it's an initial create or an update that's being recorded
14:42 thd           The fact that it has been previously done in some way does not mean that we should be stuck with what we did previously.  We may want to revisit previous usage for consistency on a reasonable principle.
14:42 Joubu         it's exactly the same
14:42 Joubu         883   `timestamp` timestamp NOT NULL default CURRENT_TIMESTAMP on update CURRENT_TIMESTAMP, -- date and time this item was last altered
14:42 ashimema      ack.. we do indeed also have 'timestamp' in a bunch of places
14:42 kohaputti     cait, it is not the same
14:41 cait          date created and updated?
14:41 cait          but it's not hte same is it?
14:40 Joubu         item already has "timestamp"
14:40 ashimema      we have `updated_by` and `updated_on` in a few places elsewhere in the API.. so that seems sane enough to me
14:40 cait          for suggestions (sorry, slow reading back)
14:40 kohaputti     Joubu, not clear for me
14:40 kohaputti     Joubu, timestamp of what
14:40 Joubu         it should be 'timestamp' I think
14:40 thd           update_date may be nicer for that case.
14:40 kohaputti     we have the patrons endpoint using updated_on
14:40 cait          ashimema: not against people checking - but I tried my best to fill blanks
14:39 marcelr       updated_date is not so nice btw
14:39 thd           Yes.
14:39 cait          sorry had to step out for a sec
14:39 Joubu         in the rest api we use *_date
14:39 ashimema      we discussed the date vs date last meeting.. I think we came to the conclusion we should try to be consistently *_date
14:39 Joubu         it was not a suggestion, just saying what it does
14:38 marcelr       created_on and updated_on are used
14:38 kohaputti     it is used on other endpoints, so let's use updated_on here also
14:38 kohaputti     just checked
14:38 kohaputti     Joubu, updated_on is used elsewhere?
14:37 thd           date_created as opposed to creation_date is inconsistent with other dates as something_date .  We should either have date_something or something_date if it is easy enough to be consistent.
14:37 ashimema      agreed
14:36 Joubu         date => updated_on
14:36 ashimema      so it's not 'date_created' at all
14:36 kohaputti     date_updated would be better
14:36 Joubu         3028   `suggesteddate` date NOT NULL, -- date the suggestion was submitted
14:36 kohaputti     so date_created is a really bad name
14:36 Joubu         3041   `date` timestamp NOT NULL default CURRENT_TIMESTAMP on update CURRENT_TIMESTAMP,  -- date and time the suggestion was updated
14:36 kohaputti     it is the last time the suggestion row was updated
14:35 kohaputti     date contains timestamp
14:35 marcelr       especially if it gets updated or so
14:35 marcelr       suggestion_date is rather vague
14:35 ashimema      all very quiet
14:32 ashimema      whats the difference between suggestion_date and date_created?
14:32 ashimema      er..
14:31 ashimema      #topic RFC /suggestions endpoint
14:31 thd           unfinished business from the previous meeting
14:30 ashimema      I cloned them accross from last meetings agenda just as the meeting started marcelr
14:30 ashimema      #info There are stll allot of blanks in the mappings table, so rather than vote this week I suggest we all try to find time to take a look at it before the next meeting and help fill in those fields.
14:29 marcelr       noting that it wasnt on the agenda?
14:29 ashimema      #link https://wiki.koha-community.org/wiki/Subscriptions_endpoint_RFC /subscriptions
14:28 ashimema      #topic RFC /subscriptions endpoint
14:28 cait          it was a lot at once... and acq
14:27 ashimema      We didn't get through all the votes last meeting
14:27 ashimema      #topic General development discussion
14:27 ashimema      ok.. moving on
14:27 cait          so we can balance that
14:26 * oleonard    agrees that signoffs have been more needed than QA lately
14:26 cait          but qa can still signoff
14:26 ashimema      very true
14:26 cait          it's true
14:26 marcelr       if you remove active signoffers, you have a problem at the other queue
14:26 ashimema      gamification.. remember the leader boards https://dashboard.koha-community.org/
14:26 ashimema      but existing members of the team shouldn't be backwards in coming forwards if you feel there's someone out there who would be a good addition
14:25 cait          andrew too, but not here i think
14:25 cait          yeah davidnind++ again :)
14:25 ashimema      indeed
14:25 ashimema      we have a few great people doing signoff lots at the moment.. David Nind, and Andrew Fuerste-Henry have been storming ahead on that front..
14:25 cait          it's not formally by invite... it just happeens people seem to need a bit of a push
14:24 ashimema      qa team tends to be 'by invite', though if someone wants to volunteer out of the blue it all goes to a vote anyways so we can assess
14:24 ashimema      It would be nice to add one or two more to the team for next cycle.. so keep a look out for people who you think may be capable and willing..
14:23 cait          yeah, that was a good number, harder to get lower
14:23 marcelr       30 is pretty low btw
14:23 cait          anyway, I think i shoudl stop here :)
14:22 cait          we had QA around 30 pretty constantly a while ago - but in general bug activity is quite high right now (which is good :) )
14:22 cait          i am just highlighting a development here - the numbers are much higher which usually indicates longer waiting times too
14:21 marcelr       you need good people
14:21 cait          we need both i'd say
14:21 marcelr       kohaputti: thats not always the answer
14:21 Joubu         we are more lacking testers than QA I'd say
14:21 cait          i think adding someone would not require much... finding the volunteers is the hard bit :)
14:21 cait          kohaputti: always keen to
14:20 kohaputti     could we recruit more qa?
14:20 ashimema      indeed
14:20 cait          testing/confirming can be done by anyone especially
14:20 marcelr       in that list they make me ignore the number
14:20 cait          not only adressing the QA team here, i should say
14:20 cait          yep, but maybe failed qa needs work too? :)
14:19 ashimema      though there are 5 majors in the queue.. I'm sure we should be able to knock them off
14:19 marcelr       includes failed qa etc
14:19 marcelr       this number is not saying that much
14:19 cait          #info Stilll over 100 major and critical  - we need people working on those, retesting, confirming, fixing etc.
14:19 Joubu         [or write bugfixes]
14:19 ashimema      indeed
14:19 kohaputti     cait, on the good side most of those 100 are features and not bugs
14:18 cait          and the hackfest is coming closer...
14:18 cait          we are almost touching 100 in Needs QA atm
14:18 cait          #info Queues are super full - please all balance your patch writing with sign-offs and QA a bit more!
14:17 cait          not much more to say than ashimema did already
14:17 ashimema      cait
14:17 ashimema      #topic Updates from the QA team
14:16 ashimema      #info 19.11.02, 19.05.07 and 18.11.13 were all released since the last meeting
14:15 ashimema      fair enough..
14:15 kidclamp      i think early for most of them :-)
14:13 ashimema      do we have any here today?
14:13 wahanui       it has been said that rmaints is talljoy, lucas, hayley
14:13 ashimema      rmaints?
14:13 ashimema      #topic Updates from the RMaints
14:13 ashimema      #info I am paying close attention to some of the refactoring bugs that are currently making their way through SO/QA and I look forward to pushing them soon.
14:12 ashimema      #info The master branch is moving along nicely at the moment with lots having been pushed over the past couple of weeks.  Thanks go out the QA team who are diligently working through bugs.
14:11 ashimema      #topic Update from the RM
14:11 ashimema      #info Fridolin will be taking a break from Koha for a few months. He wishes us all the best and will be back :)
14:10 ashimema      #info GBSD rescheduled for the 14th February
14:10 ashimema      ok.. lets organise GBSD for the 14th
14:10 cait          fridolin: save travels and all the best
14:10 fridolin      I will ;) and he will learn Perl programming with math and french ;)
14:09 Joubu         13/14 ok for me as well
14:09 Joubu         Enjoy fridolin :)
14:09 ashimema      +1
14:09 oleonard      Send lots of photos fridolin!
14:09 fridolin      see you on WattsApp
14:08 kohaputti     14th sounds nice
14:08 * fridolin    is flying next week to Madagascar
14:08 fridolin      good luck to you, long live the community
14:08 fridolin      just to say that I will be less on Koha for 3 month
14:08 ashimema      how about 13/14th Feb then?
14:08 fridolin      good morning all
14:07 cait          clear your schedule at work :)
14:07 kohaputti     2 weeks to maximise amount of participants
14:07 cait          2 weeks might be enough
14:07 ashimema      thinking next thurs/fri or the following week if we want to give more notice
14:06 ashimema      how much warning do people actually need.. 1 week, 2 weeks enough?
14:06 cait          marseille is coming up end of march, some time before that would be nice
14:06 ashimema      any volunteers to throw a wiki page together and send out a mail to organise one?
14:06 cait          don't beat yourself up, let's set a new date
14:05 ashimema      We need to reschedule the GBSD day as I ran out of time to organise it :(
14:04 ashimema      #topic Announcements
14:04 ashimema      Moving along..
14:03 cait          #info Katrin Fischer, BSZ, Germany
14:03 cait          sorry
14:03 cait          oh
14:02 kohaputti     #info Joonas Kylmälä
14:02 kidclamp      #info Nick Clemens, ByWater Solutions
14:02 tcohen        #info Tomas Cohen Arazi, Theke Solutions
14:02 wahanui       i guess rmaints is talljoy, lucas, hayley
14:02 Joubu         rmaints?
14:02 wahanui       i guess qa_team is cait Joubu marcelr kohaputti josef_moravec tcohen kidclamp khall
14:02 Joubu         qa_team?
14:02 Joubu         #info Jonathan Druart
14:02 thd           #info Thomas Dukleth, Agogme, New York City
14:01 ashimema      #info Martin Renvoize, PTFS Europe
14:01 marcelr       #info Marcel de Rooy Rijksmuseum
14:01 oleonard      #info Owen Leonard, Athens County Public Libraries, Ohio, USA
14:01 ashimema      #info please use "#info" in front of your introduction to have it show up in the automatic minutes
14:01 ashimema      #topic Introductions
14:00 ashimema      #link https://wiki.koha-community.org/wiki/Development_IRC_meeting_29_January_2020 Agenda
14:00 huginn        The meeting name has been set to 'development_irc_meeting_29_january_2020'
14:00 huginn        Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00 huginn        Meeting started Wed Jan 29 14:00:50 2020 UTC.  The chair is ashimema. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00 ashimema      #startmeeting Development IRC meeting 29 January 2020
14:00 ashimema      I forgots again
14:00 ashimema      ooh.. nearly meeting time..
13:56 marcelr       hi tcohen
13:56 tcohen        hi marcelr!
13:55 ashimema      indeed
13:49 Joubu         I thought Log.t was doing it already, but it does not
13:47 ashimema      ta
13:47 Joubu         k
13:47 Joubu         ashimema: will have a loo
13:39 koha-jenkins  Project Koha_Master_D9_MDB_Latest build #99: NOW UNSTABLE in 37 min: https://jenkins.koha-community.org/job/Koha_Master_D9_MDB_Latest/99/
13:38 marcelr       ^^
13:37 ashimema      thoughts Joubu.. ?
13:37 ashimema      I saw it and was going to let it through without a test..
13:37 marcelr       i considered it as out of balance
13:36 marcelr       did you see comment4 on the report btw? it justifies the absence of a test
13:36 marcelr       Joubu: lol that was fast
13:35 marcelr       but this one is a nice example how we can scare anyone away from submitting a patch in the first place
13:34 marcelr       didnt want to distract you from solving all the other bugs :)
13:33 marcelr       on the bug report
13:33 marcelr       No i copied ashimema
13:32 Joubu         marcelr: I was not aware of 23896, also no occurrence in my inbox
13:29 marcelr       it takes just too long
13:29 marcelr       dont like waiting for more than 3 months on reports generally
13:29 marcelr       and i spammed the list for another one
13:28 ashimema      my inbox is pretty perminently full at the minute ;)
13:28 marcelr       23896
13:28 wahanui       bug is, like, that perl code is not activ since its not in PER5LIB
13:28 ashimema      bug?
13:28 marcelr       copied you
13:28 marcelr       ashimema: i copied on a report that is about one line of code and was waiting for 3 months (frustration management here)
13:27 marcelr       o/
13:27 huginn        Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24529 enhancement, P5 - low, ---, martin.renvoize, Needs Signoff , Uninitialised value warnings in C4::Reserves
13:27 ashimema      kohaputti bug 24529 :)
13:23 ere           mtj: thanks
13:23 mtj           ere, many thanks for your mojo fixes... they look promising :0)
13:20 magnuse       jenkins++
13:18 mtj           hmm, jenkins seems to be happy again.. no random java OOM errors now
13:14 magnuse       weird, i have a oai-pmh set mapping with three "rules", all based on a value in the same marc field and subfield (347$b). the first two take effect, but the last one seems to be ignored. tried adding a fourth rule, but it's not just a case of "the last rule being ignored"
13:13 koha-jenkins  Project Koha_Master_U18 build #582: STILL FAILING in 5 min 48 sec: https://jenkins.koha-community.org/job/Koha_Master_U18/582/
13:11 koha-jenkins  Project Koha_Master_D8 build #649: NOW UNSTABLE in 37 min: https://jenkins.koha-community.org/job/Koha_Master_D8/649/
13:03 ashimema      the ones in C4::Reserves are pretty noisy we should fix those
13:02 ashimema      Joubu and marcelr have fixed a fair few already that I've spotted
13:01 ashimema      but I don't believe any of them are critical ones as such
13:01 ashimema      so there are warnings certainly.. and we should get round to fixing them all..
13:01 kohaputti     soo I think my issues with the tests were either that the circ rule refactoring breaks everything or my changes to circ rules and sysprefs break everything
13:01 kohaputti     yup, that's what I see also
13:00 ashimema      https://jenkins.koha-community.org/job/Koha_Master_D9/lastBuild/consoleText
12:58 kohaputti     ok, maybe for me occasional = endless :D
12:58 ashimema      i see occasional warning.. not an enless list
12:57 kohaputti     ashimema, do you see the endless amount of warnings during tests?
12:55 koha-jenkins  Project Koha_Master_D9_MDB_Latest build #98: ABORTED in 49 min: https://jenkins.koha-community.org/job/Koha_Master_D9_MDB_Latest/98/
12:54 ashimema      misc4dev is the key peice.. it's what populates the database with a test database
12:54 ashimema      we get round it by using koha-testing-docker and reset_all and friends
12:53 ashimema      yes.. we should fix that.. but it's been in the long list of things to do since I first started working on koha nearly a decade ago.. ;)
12:53 ashimema      the db_dependant tests really do rely heavily upon the default test dataset ;)
12:52 ashimema      yup
12:45 tcohen        that means some test expects somethings from the data on the DB
12:45 kohaputti     but it should not make a difference, if it does the test should be fixed IMO
12:45 koha-jenkins  Project Koha_Master_D9 build #1111: NOW UNSTABLE in 53 min: https://jenkins.koha-community.org/job/Koha_Master_D9/1111/
12:44 kohaputti     ashimema, I had changed quite a lot circ rules and sysprefs in my run so that could make a difference
12:44 ashimema      so far it's passing.. but obviously take a while to run
12:44 ashimema      i'm running a `prove -vv -r t/db_ependant/` now just to double check kohaputti
12:42 * tcohen      has been abducted by code this morning
12:42 tcohen        not sure?
12:41 mtj           hi tcohen, i added an extra docker4 node to jenkins.. i wonder if its caused jenkins master-node to run out of memory?
12:37 kohaputti     Joubu, maybe do another rebase?
12:36 kohaputti     well, during the past few days there has been bunch of fixes for the tests soo the Joubu's branch was rebased just before those
12:35 koha-jenkins  Project Koha_Master_D9_My8 build #120: NOW UNSTABLE in 39 min: https://jenkins.koha-community.org/job/Koha_Master_D9_My8/120/
12:34 kohaputti     I will now try vanilla master
12:34 kohaputti     ashimema, I'm actually running the tests still with the circ rule patches, but I don't think they can cause such a mess
12:34 ashimema      running inside koha-testing-docker on on your own environment
12:34 ashimema      when was the last time you reset_all?
12:33 oleonard      Hi all
12:33 ashimema      indeed
12:33 kohaputti     ashimema, do you run "prove t/db_dependent" ?
12:32 ashimema      as for testing locally.. I'm not seeing that.. but I am seeing the odd random failure
12:32 ashimema      jenkins isn't happy at the moment.. i believe mtj is working on that
12:31 kohaputti     using ubuntu 18.04 / devbox
12:31 kohaputti     ashimema, it feels like every other db_dependent test fails for me. Something wrong in my environment or is this the status of current master?
12:28 ere           ashimema: my pleasure really, since I've stumbled on this now at least twice and forgotten the solution in between :D
12:28 vfernandes    the link appears on search results facets
12:28 vfernandes    the link "Limit to currently available items" limits the search to the items where onloan is false... shouldn't list the items where onloan and notforloan are false?
12:24 ashimema      :)
12:24 ashimema      thanks for investigating ere
12:23 huginn        Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22522 is not accessible.
12:23 ere           ashimema: I added my proposal to bug 22522. As far as I can see we can support a wide range of Mojolicious::Plugin::OpenAPI versions with quite minor changes, but if I were to make the call I'd up the version requirement to at least 1.x where x is at least 28.
12:18 tcohen        submitting in a minute
12:18 tcohen        I'm done with the _count patch
12:17 Joubu         tcohen: que tal?
12:16 huginn        tcohen: Joubu was last seen in #koha 3 hours, 51 minutes, and 29 seconds ago: <Joubu> so yes
12:16 tcohen        @seen Joubu
12:03 koha-jenkins  Project Koha_Master_U18 build #581: STILL FAILING in 10 min: https://jenkins.koha-community.org/job/Koha_Master_U18/581/
12:02 tcohen        hi ashimema
12:00 ashimema      mornin' tcohen
11:59 tcohen        morning
11:56 koha-jenkins  Project Koha_Master_D8 build #648: STILL FAILING in 4 min 19 sec: https://jenkins.koha-community.org/job/Koha_Master_D8/648/
11:49 koha-jenkins  Project Koha_Master_D9_MDB_Latest build #97: STILL FAILING in 31 sec: https://jenkins.koha-community.org/job/Koha_Master_D9_MDB_Latest/97/
11:49 koha-jenkins  Project Koha_Master_U18 build #580: STILL FAILING in 1 min 30 sec: https://jenkins.koha-community.org/job/Koha_Master_U18/580/
11:47 koha-jenkins  Project Koha_Master_D9_My8 build #118: FAILURE in 31 sec: https://jenkins.koha-community.org/job/Koha_Master_D9_My8/118/
11:46 koha-jenkins  Project Koha_Master_D9 build #1110: STILL FAILING in 1 min 10 sec: https://jenkins.koha-community.org/job/Koha_Master_D9/1110/
11:46 ashimema      ?
11:46 cait          davidnind++ mtj++ ashimema++
11:46 cait          and you are so right :)
11:46 huginn        cait: davidnind was last seen in #koha 1 day, 17 hours, 53 minutes, and 47 seconds ago: <davidnind> You're welcome - didn't quite get it down to under 10 needing sign-off :)
11:46 cait          @seen davidnind
11:45 koha-jenkins  Project Koha_Master_D8 build #647: FAILURE in 32 sec: https://jenkins.koha-community.org/job/Koha_Master_D8/647/
11:45 mtj           ashimema: i havent anticipated a buster error would break the whole pipeline
11:43 mtj           hi cait, nick is davidnind, i think :)
11:43 ashimema      lol
11:43 cait          hm i need to give him karma!
11:43 huginn        cait: I have not seen dnind.
11:43 cait          @seen dnind
11:42 huginn        cait: I have not seen davidn.
11:42 cait          @seen davidn
11:42 huginn        cait: downloading the Perl source
11:42 cait          @dnind
11:42 ashimema      I just wasn't sure how to fix it
11:42 koha-jenkins  Project Koha_Master_D9_MDB_Latest build #96: FAILURE in 32 min: https://jenkins.koha-community.org/job/Koha_Master_D9_MDB_Latest/96/
11:42 koha-jenkins  Project Koha_Master_U18 build #579: FAILURE in 28 min: https://jenkins.koha-community.org/job/Koha_Master_U18/579/
11:42 mtj           ... i'll leave those repos as-is, for now
11:42 ashimema      no worries
11:42 koha-jenkins  Project Koha_Master_D9 build #1109: FAILURE in 25 min: https://jenkins.koha-community.org/job/Koha_Master_D9/1109/
11:42 mtj           hi ashimema, sorry about the error... i fiddled with the dev repos and broken the build
11:34 huginn        News from kohagit: Bug 23493: Replace rowGrouping with rowGroup <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=c35b997c939f2953ee4803de90701493fdbc71b6>
11:34 huginn        News from kohagit: Bug 23493: (follow-up) Switch to aDataSort for multi-column sorting <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=930fedd46282bc46027332f4986f3baaee72b4ad>
11:34 huginn        News from kohagit: Bug 23493: (follow-up) Disable rowGroup on due date sort <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=644ed8860aea4b1297d978700c2966a93e10628a>
11:34 koha-jenkins  Project Koha_19.11_D10 build #6: STILL FAILING in 8 min 54 sec: https://jenkins.koha-community.org/job/Koha_19.11_D10/6/
11:17 koha-jenkins  Project Koha_19.11_D10 build #5: STILL FAILING in 33 sec: https://jenkins.koha-community.org/job/Koha_19.11_D10/5/
11:17 koha-jenkins  Project Koha_Master_D9_My8 build #117: STILL UNSTABLE in 55 min: https://jenkins.koha-community.org/job/Koha_Master_D9_My8/117/
11:14 koha-jenkins  Project Koha_Master_D8 build #646: STILL UNSTABLE in 34 min: https://jenkins.koha-community.org/job/Koha_Master_D8/646/
11:09 koha-jenkins  Project Koha_19.11_D10 build #4: STILL FAILING in 36 sec: https://jenkins.koha-community.org/job/Koha_19.11_D10/4/
11:08 koha-jenkins  Project Koha_19.11_D10 build #3: ABORTED in 2 min 20 sec: https://jenkins.koha-community.org/job/Koha_19.11_D10/3/
10:43 koha-jenkins  Project Koha_19.11_D10 build #2: FAILURE in 10 min: https://jenkins.koha-community.org/job/Koha_19.11_D10/2/
10:39 koha-jenkins  Project Koha_Master_D9 build #1108: STILL UNSTABLE in 49 min: https://jenkins.koha-community.org/job/Koha_Master_D9/1108/
10:34 koha-jenkins  Project Koha_Master_U18 build #578: STILL UNSTABLE in 42 min: https://jenkins.koha-community.org/job/Koha_Master_U18/578/
10:28 koha-jenkins  Project Koha_Master_D10 build #92: ABORTED in 1 min 38 sec: https://jenkins.koha-community.org/job/Koha_Master_D10/92/
10:21 koha-jenkins  Project Koha_Master_D10 build #91: STILL FAILING in 10 min: https://jenkins.koha-community.org/job/Koha_Master_D10/91/
10:20 huginn        News from kohagit: Bug 24305: (follow-up) Remove previous declaration of batch_biblionumbers <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=c2650f577cf78e5c9ae685925876fcff752a5737>
10:08 koha-jenkins  Project Koha_Master_D9_My8 build #116: STILL UNSTABLE in 56 min: https://jenkins.koha-community.org/job/Koha_Master_D9_My8/116/
09:52 koha-jenkins  Project Koha_Master_D9_MDB_Latest build #95: STILL UNSTABLE in 39 min: https://jenkins.koha-community.org/job/Koha_Master_D9_MDB_Latest/95/
09:50 koha-jenkins  Project Koha_Master_D8 build #645: STILL UNSTABLE in 37 min: https://jenkins.koha-community.org/job/Koha_Master_D8/645/
09:12 huginn        News from kohagit: Bug 23856: Split author and ISBN/ISSN out of citation in staged MARC record management <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=12b8c3ac1e01f5a9fbc9221dba9ad44d00c53083>
09:12 huginn        News from kohagit: Bug 23885: Move staff client search results JavaScript into separate file <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=ef4306cc02e2f15aa2466ae65d0b4f9a1fd70ad9>
09:12 huginn        News from kohagit: Bug 23944: Phase out use of jquery.cookie.js in favor of js.cookie.js <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=68c9273fa76cedf8de48d4284693b5058ca5fdbd>
09:12 huginn        News from kohagit: Bug 23944: (follow-up) Update circulation to use new library <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=54c821a9e8df8a0cbc0a74a83fe605cd27cd40ac>
09:12 huginn        News from kohagit: Bug 23947: Phase out jquery.cookie.js: Authority merge <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=d12f50a2e57e8ce84b7761c154b82d42051b9c4a>
09:12 huginn        News from kohagit: Bug 13121: Move search results "action" links ("Place hold," "Add tag," etc) into... <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=07d2a419f36711b5dc9b26a081eafa6a96cfdce9>
09:12 huginn        News from kohagit: Bug 20847: Add more fields to the Batch Patron Modification tool <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=0cdd62600b85721f6f1704138fdf56b94ba15d52>
08:32 fridolin      hihi
08:25 Joubu         so yes
08:25 Joubu         I created a rule and all the empty values were ""
08:22 kohaputti     Joubu, did you check whether smart-rules.pl now adds the rule with value ''?
08:21 Joubu         kohaputti: commit amended
08:20 Joubu         it is $allowedreserves, indeed
08:19 Joubu         let me check that
08:18 kohaputti     Joubu, s/reservecount ne ''/allowedreserves ne ''/ ?
08:18 kohaputti     Joubu, line 439: https://gitlab.com/joubu/Koha/commit/3d922b12bcda5873af8ae13091ab20dd6cdb677f#8f7c9d12a0a1ef2257188d94249f31f73695b1d9_440_439
08:01 alex_a        Bonjour
07:47 reiveune      hello
07:43 huginn        ashimema: The operation succeeded.
07:43 ashimema      @later tell mtj sorry dude, looks like the most recent packaging changes have had unexpected consequences.. we dont seem to be pulling in all our dependancies in koha-testing-docker anymore
07:26 ashimema      Morning #koha