Time  Nick           Message
23:51 cait           i know
23:48 rangi          and note fix the syspref while fixing that
23:48 rangi          you can set it to patch doesnt apply
23:48 cait           ... but it keeps coming back if i fail it
23:48 rangi          it wont apply
23:48 rangi          its going to have to
23:47 cait           but... I don't want to fail stuff
23:47 rangi          *snap*
23:47 rangi          well in a minute it will no longer apply
23:47 tcohen         don't worry, it is 2 minutes from not applying anymore
23:46 cait           it also touches prog instead of bootstrap templates :(
23:46 cait           gah.
23:45 cait           hm?
23:45 tcohen         are you ready?
23:42 cait           i agree
23:42 cait           *sigh*
23:41 eythian        yeah, there's already about three conventions, we don't need a fourth
23:41 barton_away    -1
23:40 cait           what is the general opinion to sysprefs with spaces?
23:35 dcook          Documentation ftw!
23:35 * dcook        is really glad that his wife is a citizen
23:35 tcohen         barton++ # we love documented stuff
23:35 dcook          Also need to have my PR before I can get any paid parental leave
23:34 dcook          tcohen: :p
23:34 dcook          no worries, barton_away
23:33 barton_away    thanks for the edjumacation dcook, tcohen.
23:32 barton_away    someday, there will be a wiki article that comes out of this.
23:32 tcohen         ??
23:32 tcohen         isn't it worth all the money you could give them!
23:32 tcohen         aren't you bringing them a new australian?
23:32 barton_away    Ok, I've bookmarked the our conversation starting when dcook started showing me how to view zebra indexes in yaz-client: http://irc.koha-community.org/koha/2014-10-29#i_1588267
23:29 dcook          tcohen: Hope so. Once I send in the application and all the monies.
23:29 dcook          ^awesome
23:29 dcook          http://www.smh.com.au/digital-life/mobiles/apple-responds-to-bending-iphone-6-reports-20140925-10m9z9.html
23:27 tcohen         hey, you will get the permanent residence
23:27 * dcook        anticipates never leaving Australia again
23:26 dcook          I wish!
23:26 dcook          hehe
23:26 tcohen         by barcelona?
23:26 tcohen         is any of you coming to argentina soon?
23:26 dcook          O_o
23:25 barton_away    dcook: http://i.imgur.com/JY1yUWyl.jpg
23:25 dcook          D:
23:25 cait           :/
23:25 dcook          :(
23:25 tcohen         to BArcelona! :-(
23:25 tcohen         yay! my replacement phone has been delivered!
23:23 dcook          While I don't know anything about the iphone6, I'm going to giggle anyway :)
23:22 barton_away    isn't apple having similar issues with the iphone6?
23:21 dcook          cait: Wish I could but I'm fixing stuff due to "flexible" standards
23:20 barton_away    so why aren't we using that to generate the rest of the zebra configs agian?
23:18 tcohen         barton_away: that's the default explain file i'm sure we all have :-D
23:17 barton_away    oh, that's right tcohen ... I did run across something similar somewhere in the guts of koha...
23:16 * tcohen       runs away
23:15 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7673 enhancement, P5 - low, ---, jonathan.druart, Needs Signoff , New patron permissions to manage items (edit, batchmod, delete all)
23:15 cait           but now seriously... i need some sign offs! :) bug 7673 - string corrections from owen?
23:15 tcohen         http://bmayor.biblio.unc.edu.ar:7090/
23:15 barton_away    ... keep me from going any crazier than I am already.
23:15 barton_away    actually ... I have a perl file that reads yaml and spits out various pieces of zebra config.
23:14 tcohen         dcook: you could write a script to generate an SRU explain file from our configuration files
23:14 barton_away    koha community <= first movers.
23:13 dcook          And... that was the start of the apocalypse
23:13 dcook          Unless we had a hive mind O_O
23:13 dcook          Nah, it wouldn't be that efficient in terms of sharing knowledge probably
23:13 tcohen         or two?
23:12 barton_away    yeah.
23:12 tcohen         maybe a clone?
23:12 dcook          :D
23:12 tcohen         now we deprecated grs1, maybe we should start a DOM-only page
23:12 dcook          I wonder if I'd have more time if I had a TARDIS...
23:12 barton_away    yeah, I used martin's as my bible for quite a while.
23:11 dcook          ^
23:11 wizzyrea       yes you do because we can't keep that knowledge locked up in only your brain.
23:11 tcohen         martin's is a good one
23:11 * dcook        really needs to write up some Zebra wiki pages sometime...
23:10 tcohen         rangi++
23:10 tcohen         so much reading should always serve someone
23:10 rangi          and put this all on the wiki barton_away :-)
23:09 barton_away    tcohen++
23:09 barton_away    indeed!
23:09 tcohen         *do
23:09 tcohen         because, after all, that's what zebraidx will to to index your record
23:09 dcook          tcohen++
23:08 tcohen         if the desired new index shows
23:08 dcook          Ooo
23:08 tcohen         and see
23:08 tcohen         running xsltproc .../biblio-zebra-indexdefs.xsl marc_record
23:08 dcook          What tcohen said tripped me up a few times with DOM at first. I was changing the XML but not the XSL
23:08 barton_away    tcohen yes on both.
23:08 tcohen         if that's the case, you could try
23:08 dcook          ^
23:07 tcohen         and generated the koha-zebra-indexdefs.xsl file
23:07 dcook          But if a certain field/subfield tied to an index doesn't exist in the record, it won't appear there
23:07 tcohen         so you have the biblio-koha-indexdefs.xml file, with the new indexes
23:07 dcook          You might have empty indexes but that's probably because the field/subfield in the record was empty
23:07 barton_away    yeah, dom.
23:07 dcook          Or rather..
23:07 tcohen         in DOM, right?
23:07 dcook          Yeah, it'll just be the indexes that have been filled I think
23:06 barton_away    tcohen - I'm trying to add indexes... after a re-index, search on the new indexes doesn't return any results. I'm trying to trouble-shoot that.
23:05 tcohen         what are you trying to do?
23:05 tcohen         happened to me with the facets
23:05 tcohen         because you are querying the internal zebra data structure
23:05 barton_away    that's still a huge leap forward for me...
23:04 barton_away    yeah, I'm thinking so, as well.
23:04 tcohen         i bet the latter
23:03 barton_away    dcook -- Ok, I've been playing with `formatxml` and `elements marcxml` -- it's clear to me that `elements marcxml` returns indexed marc xml entities, and using `show` with `format xml` shows the index names ... but does it display all possible index names, or only those for which marc fields exist?
22:58 * dcook        says in an unrelated comment
22:58 dcook          Hmm, I guess when a spec says "non-normative", they really mean it...
22:54 tcohen         i think we should sitck with utf8::decode until someone has a regression test for a bug introduced by using it
22:53 tcohen         i think it is because utf8::decode and Encode::decode('UTF-8'  do different things with stuff that is not UTF
22:50 cait           ... so why do the tests fail now? :(
22:50 gmcharlt       you're welcome
22:49 tcohen         thanks for the explanation gmcharlt
22:48 gmcharlt       what can I say? the POD for that function explicitly includes a section explaining why it's called IsStringUTF8ish rather than IsStringUTF8
22:46 tcohen         i wrongly interpreted it in the 'loose UTF8 interpretation' the utf8:: namespace tools do
22:45 gmcharlt       the "ish" is basically just an expression of that possibility existing
22:44 gmcharlt       which can happen in malformed MARC records, albeit rarely
22:44 gmcharlt       "appears" in the sense that a string of octets that can encode a UTF8 string, are actually in a different character set
22:43 tcohen         the only broken stuff from the UTF8 patches is that function's unit tests
22:41 tcohen         "appears to be valid UTF-8"
22:41 tcohen         gmcharlt: what confuses me is the "appears"
22:38 gmcharlt       tcohen: IIRC, the general idea is to see if the scalar is either a utf8 string (with the utf8 flag set) or contains a sequence of bytes that is valid UTF-8
22:37 tcohen         gmcharlt: could you help me understand how C4::Charset::IsStringUTF8ish is supposed to work?
22:36 cait           that was days ago :)
22:35 eythian        cait: you quoted the correct name in the bug yourself :)
22:34 cait           installing now
22:33 * cait         blames eythian :)
22:33 eythian        oldstable|main|source: libtemplate-plugin-json-escape-perl 0.2-1kohaoldstable1
22:33 rangi          libtemplate-plugin-json-escape-perl
22:33 rangi          http://debian.koha-community.org/koha/pool/main/libt/
22:32 rangi          you missed plugin
22:32 cait           so... what's wrong?
22:31 cait           not found
22:31 cait           sudo apt-get install libtemplate-json-escape-perl
22:30 cait           OK   http://debian.koha-community.org squeeze-dev/main amd64 Packages
22:30 cait           then i ran sudo apt-get update
22:29 cait           i think so
22:29 eythian        cait: so so far everything's working then?
22:29 tcohen         need gmcharlt basically
22:29 tcohen         but, need information to be sure of the consecquences
22:29 tcohen         so, because of the "ish" thing, I guess is ok to just leave the utf::decode call
22:28 tcohen         but they don't if you use the 'strict' decoding from Encode
22:28 tcohen         it seems that if we stick with the loose utf8 interpretation, (i.e. utf8::decode) tests pass
22:28 tcohen         magnuse: i don't understand the expected semantics of the C4::Charset::IsStringUTF8ish function, that's why I don't know how to patch it
22:27 cait           seems ok
22:27 cait           wget -O- http://debian.koha-community.org/koha/gpg.asc | sudo apt-key add -
22:27 cait           so i created a file koha.list and put that line in
22:27 cait           ok, i noticed the repo was not there
22:27 cait           yes...
22:26 eythian        cait: you know to report with more detail than that :)
22:24 Mexiboy        gracias tcohen
22:24 cait           and it still won't install
22:24 tcohen         hanoi towers
22:24 cait           that i added  deb http://debian.koha-community.org/koha squeeze-dev main
22:24 eythian        cait: what do you mean by "doesn't work"?
22:24 Mexiboy        ok tcohen
22:23 magnuse        tcohen: oh noes! hope it can be fixed in time for 3.18
22:23 tcohen         Mexiboy: hablo español si necesitas
22:22 cait           eythian: it doesn't work :(
22:22 dcook          :D
22:22 barton_away    yeah, that's definitely going into my notes, and will be shared far and wide.
22:22 Mexiboy        hi tcohen
22:22 Mexiboy        Thanks, cait're very good person
22:21 dcook          to*
22:21 dcook          barton_away: Glad ot help :)
22:21 tcohen         so you are from Costa Rica?
22:21 barton_away    yeah.... that will definitely make life easier for me.
22:21 tcohen         hi Mexiboy
22:20 tcohen         barton_away: nice :-D
22:20 tcohen         magnuse: as of me, it is failing-qa because the t/Charset.t tests fail
22:20 cait           Mexiboy: welcome - and don't let them confuse you :)
22:20 barton_away    tcohen: dcook was showing me how  to list existing indexes via yaz.
22:19 tcohen         someone is playing with yaz?
22:19 dcook          barton_away: Iirc, the reason why it shows you "z:index" when using "format xml" is because "index" is the first element set defined in the Zebra config
22:19 wahanui        somebody said dcook was the designated masochist
22:19 tcohen         dcook?
22:19 barton_away    cool...
22:18 tcohen         rangi: awesome news!
22:18 dcook          If you want to see marcxml, you can also try "elements marcxml"
22:18 Mexiboy        I'm really new to Koha, it is new to me, but very interesting
22:18 dcook          :)
22:18 barton_away    dcook++
22:18 tcohen         hicait
22:18 dcook          I was wondering why no one was saying anything... then realized that I had accidentally closed hexchat :p
22:17 wahanui        i heard packages was at http://wiki.koha-community.org/wiki/Debian
22:17 cait           packages
22:17 Mexiboy        than good, excellent
22:17 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11944 major, P5 - low, ---, jonathan.druart, Passed QA , Cleanup Koha UTF-8
22:17 * magnuse      checks up on the status of bug 11944 before falling asleep
22:17 cait           and there is a spanish speaker :)
22:16 cait           intranetuserjs and intranetusercss i think
22:16 cait           also there are preferneces whre you can put custom css and javascript
22:16 cait           some hints about custmizing koha can be found there
22:16 wahanui        rumour has it owen's blog is http://www.myacpl.org/koha/
22:16 cait           owen's blog?
22:15 Mexiboy        etc.
22:14 Mexiboy        I want to customize, change logo, link colors
22:13 cait           what is your problem?
22:13 cait           mostly english speakers this time of the day :)
22:13 Mexiboy        Yes, cait!
22:13 cait           you need help to customize.... a part of administration?
22:13 Mexiboy        ¿Alguna referencia?
22:13 Mexiboy        Alguien me puede ayudar, quiero personalizar la parte administrativa de Koha
22:12 wahanui        what's up, cait
22:12 cait           hola
22:12 Mexiboy        hola
22:12 cait           then i can try again
22:11 cait           waitin gfor another package to finish installing
22:11 cait           yeah thought it was
22:10 eythian        and also make sure you have the k-c repo enabled.
22:09 cait           thx
22:09 cait           i will try again
22:09 eythian        make sure you do apt-get update first
22:08 cait           hmmm
22:08 eythian        cait: sudo apt-get install libtemplate-json-escape-perl
22:08 barton_away    that worked!
22:08 cait           iam tryin to test owen's follow up
22:08 barton_away    sweet!
22:08 cait           eythian: how do I install that JSON dependency?
22:07 dcook          show
22:07 wahanui        i heard format xml was a client side transformation of whatever comes back from the server.
22:07 dcook          format xml
22:06 dcook          f test
22:06 dcook          base biblios
22:06 dcook          yaz-client unix:/dcook/koha-dev/var/run/zebradb/bibliosocket
22:06 dcook          something like "unix:/dcook/koha-dev/var/run/zebradb/bibliosocket"
22:06 dcook          So now you go into koha-conf.xml and look up the socket for the server in the "listen" element
22:05 dcook          Sweet as
22:05 barton_away    looks like it's already there.
22:04 dcook          It might even already be there so you might not need to worry
22:04 dcook          barton_away: Once you've added that... you might need to restart Zebra or rebuild your indexes, but I can't remember for certain
22:03 wahanui        holding library is just the 'place it is currently'
22:03 cait           hm holding library?
22:03 dcook          iirc, the line about "<retrieval syntax="xml" name="index"/> <!-- allow viewing index entries -->" should show you the indexes
22:03 cait           should current location be holding branch?
22:03 cait           hm no oleonard
22:03 pastebot       "dcook" at 127.0.0.1 pasted "zebradb/retrieval-info-bib-dom.xml" (22 lines) at http://paste.koha-community.org/263
22:02 dcook          K, look in your "zebradb/retrieval-info-bib-dom.xml" file
22:02 barton_away    yep.
22:01 dcook          Are you using DOM?
22:01 dcook          Documentation... now that's a very good question :p
22:00 barton_away    or at least, where do I look in the documentation.
22:00 dcook          barton_away: I'm double-checking as we speak ;)
22:00 barton_away    dcook -- how do I query yaz-client to list indexes?
21:59 cait           coudl someone sign off on 12851 and 12505?
21:59 barton_away    I've been meaning to ask #koha if that was possible...
21:59 dcook          You can also see what's being stored in the indexes if you have Zebra configured just so
21:58 dcook          Or a case of an error not making its way back to Koha
21:58 dcook          I'm a fan of using yaz-client to query Zebra more directly in case it's a Koha thing
21:57 barton_away    ... if only it were that easy.
21:57 barton_away    yeah.
21:57 mtompset       Have you started the zebra daemon? :)
21:57 barton_away    good thought, that.
21:56 barton_away    ah.
21:56 eythian        so you know your rebuild is working.
21:55 eythian        I didn't mean config, I meant the actual generated indexes.
21:54 barton_away    I'm not sure that I follow your logic about rolling back the config changes and rebuilding agian eythian ... if the rebuild failed in such a way that existing indexes were working, a roll-back and rebuild would give me exaclty the same results, right?
21:47 barton_away    Sisyphus was a driven damned oscelator <= physics pun.
21:47 cait           not sure about that wahanui!
21:46 cait           heh
21:46 wahanui        it has been said that Sisyphus is spelt neither druthb nor cait.
21:46 eythian        Sisyphus
21:46 cait           had to roll it up again and again...
21:46 eythian        huh?
21:45 cait           how do you spell the name of that man with the hill and the stone again?
21:45 eythian        well you can test b) easily: move your current indexes out of the way and try a rebuild and see what happens.
21:45 cait           now it's 36...
21:45 cait           the queue was around 25 or so
21:45 cait           ... when i went to work this morning
21:44 barton_away    right.
21:44 eythian        oh, so either a) it's not recognising your additions to the index definitions, or b) it's not rebuilding correctly.
21:43 barton_away    => returns expected results.
21:43 barton_away    yes. search on previous indexes works.
21:43 eythian        does it return results from the fields you haven't indexed?
21:43 barton_away    I'm on 3.16.x ... some of the builds are git, others are package with zebra files updated.
21:41 barton_away    when I run a search after the rebuild, the search doesn't return any results from the marc fields that I've indexed.
21:40 barton_away    heh.
21:40 eythian        and what does "not work" mean?
21:39 rangi          different versions of koha and/or perl modules and/or zebra ?
21:39 barton_away    s/rull/full/
21:37 barton_away    I'm having issues adding zebra indexes... the same config files will work for one site, but not another, even after a rull rebuild.
21:37 rangi          yep
21:36 cait           also bernardo has a patch waiting for sign off to break the marc dialects into separate po files *hint* :)
21:36 rangi          breaking it up
21:36 cait           yep
21:36 rangi          smaller po files
21:36 rangi          bernardo was working on something like that
21:36 cait           so you don' thave to do it manually or alter the standard ones
21:36 rangi          yep
21:36 cait           rangi: if we had a separate po file we could easily generate, it hink that could help
21:36 * mtompset     starts humming Lambchop and friends. :)
21:35 mtompset       this is the fun that doesn't end... it goes on and on my friend. ;)
21:35 rangi          cait: thats fixable too
21:35 wizzyrea       oh yes that does sound really painful.
21:35 cait           just saying... cusotm xslt are fine when you are not i a multi language environment... then they get somewhat more painful
21:32 mtompset       wizzyrea: That kind of fit. Weird!
21:30 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11425 new feature, P5 - low, ---, julian.maurice, Signed Off , Search form for items
21:30 eythian        to clarify my slightly ambiguous comments on bug 11425
21:30 eythian        also, Template::JSON::Escape is in the k-c debian repo.
21:29 cait           :)
21:29 wizzyrea       hi
21:29 wizzyrea       *snerk*
21:29 eythian        hi
21:29 huginn         wizzyrea: Quote #34: "<chris> oh except that time .. when katipo decided to write an ils just for fun, then forced HLT to use it" (added by gmcharlt at 06:23 PM, September 16, 2009)
21:29 wizzyrea       @quote random
21:28 huginn         wizzyrea: The operation succeeded.  Quote #312 added.
21:28 wizzyrea       @quote add rangi: babies dont care for your 020$z invalid isbns
21:25 rangi          seems to work ok here .. but would like some more people to try the opac with it installed
21:24 cait           rangi++
21:23 huginn         rangi: The operation succeeded.
21:23 rangi          @later tell tcohen MM-OPAC/theme_dep is rebased, should be ready for merge and testing
20:36 mtj            cool, no hurry
20:35 rangi          yep, if im about
20:34 mtj            rangi, perhaps i could run thru my idea with you, after EOB sometime?
20:33 mtj            a
20:33 mtj            i think i have working solution
20:33 mtj            i was thinking a bit more about the TT/translation/pootle problem last nite...
20:31 mtj            morning all
20:28 cait           tcohen++
20:27 jenkins_koha   Nicole C. Engard: remove firefox addon note
20:27 jenkins_koha   Project Koha_Docs_3.14.x build #86: SUCCESS in 11 sec: http://jenkins.koha-community.org/job/Koha_Docs_3.14.x/86/
20:27 jenkins_koha   Starting build #86 for job Koha_Docs_3.14.x (previous build: SUCCESS)
20:22 jenkins_koha   Nicole C. Engard: remove firefox addon note
20:22 jenkins_koha   Project Koha_Docs build #571: SUCCESS in 2 min 46 sec: http://jenkins.koha-community.org/job/Koha_Docs/571/
20:22 jenkins_koha   Starting build #571 for job Koha_Docs (previous build: SUCCESS)
20:14 jenkins_koha   Nicole C. Engard: remove firefox addon note
20:14 jenkins_koha   Project Koha Docs 3.16.x build #2: SUCCESS in 13 sec: http://jenkins.koha-community.org/job/Koha%20Docs%203.16.x/2/
20:14 jenkins_koha   Starting build #2 for job Koha Docs 3.16.x (previous build: SUCCESS)
20:12 nengard        :)
20:12 jcamins        lol
20:09 nengard        jcamins https://www.facebook.com/nengard/posts/10101110134473516
20:01 rangi          babies dont care for your 020$z invalid isbns
20:00 rangi          shoe even
20:00 rangi          someones show
20:00 rangi          marc records
20:00 wahanui        TT plugins are awesome
20:00 rangi          TT plugins
20:00 rangi          they put everything in their mouth
19:59 jcamins        I think I missed some context... why are babies putting T::T plugins in their mouth?
19:57 jenkins_koha   Project Koha Docs 3.16.x build #1: SUCCESS in 1 min 16 sec: http://jenkins.koha-community.org/job/Koha%20Docs%203.16.x/1/
19:57 jenkins_koha   Starting build #1 for job Koha Docs 3.16.x
19:56 nengard        cool
19:56 rangi          http://jenkins.koha-community.org/job/Koha%20Docs%203.16.x/1/console
19:56 rangi          nengard: koha docs 3.16.x is doing its first run now
19:55 mtompset       Greetings, #koha.
19:53 magnuse        tcohen++ community++
19:41 nengard        hehe
19:38 rangi          tcohen should read back :)
19:38 rangi          you name it, they'll try to eat it
19:38 rangi          fluff off the end of the broom
19:38 rangi          dirt
19:38 rangi          snails
19:37 nengard        oh my
19:37 rangi          if their is dog poop .. and they grab it .. thats where it will go
19:37 rangi          but babies will put anything they get in their mouth
19:37 rangi          hehe not my kids
19:36 nengard        rangi that's horrible!!!
19:35 rangi          cya tcohen
19:35 rangi          yeah, i think you are right, we do double handling
19:35 tcohen         have a nice day #koha
19:35 tcohen         bye
19:31 tcohen         it was
19:31 tcohen         that initializes all his stuff once, and reused
19:31 rangi          that was a great email from olugbenga
19:30 tcohen         was writing an OO TT:Plugin
19:30 tcohen         one thing i did for a customer
19:30 tcohen         to create XML again and then pass to the XSLT
19:29 tcohen         i was thinking that we do too much on the XML that the indexer returns (no point creating MARC::Record objects
19:28 rangi          http://www.workbuffer.org/presentations/kohacon_hackfest/html/index.html  its from 2012 .. we are doing the plugins and filters now
19:28 tcohen         interensting
19:28 rangi          http://www.workbuffer.org/presentations/kohacon_hackfest/html/slide006.html  <-- also we should do this more
19:27 rangi          tcohen: http://www.workbuffer.org/presentations/kohacon_hackfest/html/slide004.html
19:27 rangi          ill do it now
19:26 rangi          either of us can
19:26 * tcohen       hides
19:26 nengard        tcohen or rangi who adds stuff to jenkins? it's missing the 3.16 manual
19:25 rangi          did i ever show you my mental idea for doing MARC display?
19:25 rangi          yeah its a bit tricky
19:21 tcohen         ah, nevermind, opac vs. staff
19:20 tcohen         rangi: do u think marc visibility should be handled at export-for-indexing time? I think doing that after we retrieve the record from zebra is too much overhead
19:19 tcohen         i like the idea rangi
19:19 * rangi        goes off on a tangent
19:19 tcohen         ah
19:19 rangi          so that others could grab them .. like reports, and frameworks etc
19:18 rangi          contribs even (when we set it up)
19:18 rangi          then people who want invalid $z can have it, we could also let htem upload their xslt to the wiki (or to controbs.koha-community.org)
19:18 tcohen         i just said alsmot the same you're saying rangi
19:18 tcohen         he said the $q is new to MARC21, so something we could consider
19:18 rangi          :)
19:17 rangi          maybe a better thing to do is make a good video/tutorial about how to use a custom xslt
19:17 rangi          i think that things like this is why we built it so people could use their own xslt :)
19:15 tcohen         and what about the $q?
19:15 tcohen         i'm happy if i can search for it (if i see it on the back on the book)
19:15 tcohen         i had the same feeling, but am not a cataloguer
19:12 nengard        it's invalid ...
19:12 nengard        I don't think we want to include it at all
19:08 wahanui        do u think is, like, ok to set a cookie name marcdocs_borrowernumber true or false
19:08 tcohen         what do u think?
19:08 tcohen         and he added that ($q) could be shown too
19:07 tcohen         saying that we could consider including $z with an (INVALID) flag
19:07 tcohen         and I noticed the comment from the user
19:07 tcohen         nengard: i'm redoing for the ISSN field
19:06 nengard        tcohen signed off
18:35 nengard        tcohen, on the phone - will look shortly
18:28 tcohen         and now zebra
18:28 wahanui        That'll be $1 for the awesome jar, magnuse
18:28 magnuse        awesome, wahanui
18:28 wahanui        tcohen is obsessed with packages' scripts
18:28 magnuse        tcohen?
18:15 tcohen         should we show invalid isbns at all?
18:15 tcohen         but I think I miss-understood the problem
18:14 tcohen         have you noticed that the same problem happens for ISSN?
18:14 tcohen         i'm about to post a correct version for the patch
18:13 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12829 minor, P5 - low, ---, tomascohen, Needs Signoff , 020z not showing
18:13 tcohen         nengard: you filled bug 12829
18:12 nengard        LOL
18:08 wahanui        hand is asleep again, can't type properly
18:08 tcohen         hand?
18:08 tcohen         maybe you have it installed by habd?
18:08 tcohen         i'm not sure
18:07 oleonard       ubuntu 12.04 LTS
18:07 tcohen         debian 6?
18:07 tcohen         oleonard: what OS version?
18:06 jenkins_koha   * Nicole C. Engard: add FacetMaxCount
18:06 jenkins_koha   * Nicole C. Engard: add id to ReturnpathDefault
18:06 jenkins_koha   * Nicole C. Engard: add ReturnpathDefault
18:06 jenkins_koha   * Nicole C. Engard: add ReplytoDefault
18:06 jenkins_koha   Project Koha_Docs build #570: SUCCESS in 1 min 47 sec: http://jenkins.koha-community.org/job/Koha_Docs/570/
18:06 jenkins_koha   Starting build #570 for job Koha_Docs (previous build: SUCCESS)
18:04 oleonard       When I run t/db_dependent/ReportsGuided.t I get messages like "ok 3 # skip DBD::Mock is too old"
18:03 tcohen         magnuse
18:01 wnickc         I figured it out after a bit of clicking+monkey face+head scratching :)
18:00 oleonard       wnickc: And thanks for testing :)
18:00 tcohen         OMFG
18:00 oleonard       wnickc: It's not a big deal since we don't actually support no-js in the staff client, but if you want to confirm that the HTML5 required attributes are correct you can navigate to the edit page and then disable js
17:58 oleonard       I'm assuming excuse number 2 is "Running tests" https://xkcd.com/303/
17:58 wnickc         oleonard: it's tricky to test reports when javascript is off, all the dropdown buttons stopped working
17:54 oleonard       (beware monster test plan!)
17:51 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11592 normal, P5 - low, ---, mtompset, Needs Signoff , opac detail scripts do not respect MARC tag visibility
17:51 oleonard       Bug 11592
17:51 tcohen         pointer?
17:51 tcohen         not sure?
17:51 oleonard       I don't think it does now, but isn't there a pending patch for that?
17:51 tcohen         i don't see how the XSLT's are doing that
17:50 tcohen         #koha: is anyone aware if hiding from OPAC a subfield in the framework works at all?
17:43 * oleonard     chastises oleonard for adding to the 'Needs signoff' queue again!
16:51 jenkins_koha   Nicole C. Engard: update reports image
16:51 jenkins_koha   Project Koha_Docs build #569: SUCCESS in 1 min 54 sec: http://jenkins.koha-community.org/job/Koha_Docs/569/
16:51 jenkins_koha   Starting build #569 for job Koha_Docs (previous build: SUCCESS)
16:48 * mveron       ...has to run, not walk...
16:46 cait           ... and now i am walking home to do QA :)
16:46 mveron         ...for libraries using Koha... :-)
16:46 cait           mveron: 3.18 will be awesome :)
16:39 mveron         It is a great feature!
16:39 oleonard       mveron++
16:39 oleonard       Thanks mveron!
16:38 jenkins_koha   * Nicole C. Engard: add OpacCustomSearch
16:38 jenkins_koha   * Nicole C. Engard: update opacprivacy
16:38 jenkins_koha   Project Koha_Docs build #568: SUCCESS in 3 min 42 sec: http://jenkins.koha-community.org/job/Koha_Docs/568/
16:38 jenkins_koha   Starting build #568 for job Koha_Docs (previous build: SUCCESS)
16:26 fridolin       bye
16:22 reiveune       bye
16:11 jenkins_koha   * Nicole C. Engard: add missing image label
16:11 jenkins_koha   * Nicole C. Engard: add new images
16:11 jenkins_koha   * Nicole C. Engard: add new patron privacy
16:11 jenkins_koha   Project Koha_Docs build #567: SUCCESS in 6 min 54 sec: http://jenkins.koha-community.org/job/Koha_Docs/567/
16:11 jenkins_koha   Starting build #567 for job Koha_Docs (previous build: SUCCESS)
15:28 oleonard       Okay
15:28 cait           mark it signed off i will test
15:27 oleonard       I don't want to keep the feature from going in because of a minor follow-up
15:27 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=643 enhancement, P2, ---, pasi.kallinen, Passed QA , Allow override of 'debarred' status
15:27 oleonard       My follow-up for Bug 643 is so minor that I hesitate to even mark it 'needs signoff'
15:24 cait           grrrrrr
15:21 * tcohen       messing with cait's guilty-ness
15:21 tcohen         so you're not qaing?
15:20 tcohen         h
15:20 tcohen         a
15:20 cait           writing abstracts
15:20 nengard        tcohen the 3.16 manual isn't on jenkins
15:20 tcohen         what is killing you?
15:20 huginn         04Bug 11425: new feature, P5 - low, ---, julian.maurice, Needs Signoff , Search form for items
15:20 cait           tcohen: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11425#c53
15:19 cait           tcohen: that it's killing me??
15:19 tcohen         nengard_afk: need something on jenkins?
15:19 tcohen         cait: didn't know, that's great
15:09 huginn         nengard_manual: The operation succeeded.
15:09 nengard_manual @later tell gmcharlt the 3.16 manual isn't listed on jenkins
15:01 cait           :(
15:01 cait           well actually writing this abstract for the third time is killing me...
15:00 cait           oleonard: you are killing me :P
15:00 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12944 normal, P5 - low, ---, jonathan.druart, Needs Signoff , There is no way to search orders by creator
15:00 oleonard       Sorry to un-QA another one, but Bug 12944 needed a very minor follow-up
15:00 cait           tcohen: i thought eythian had packed it already - the JSON one
14:59 jenkins_koha   * Nicole C. Engard: add html & css library link
14:59 jenkins_koha   * Nicole C. Engard: update kyle's koha tools link
14:59 jenkins_koha   * Nicole C. Engard: update schema link in appendix
14:59 jenkins_koha   Project Koha_Docs_3.14.x build #85: SUCCESS in 13 sec: http://jenkins.koha-community.org/job/Koha_Docs_3.14.x/85/
14:59 jenkins_koha   Starting build #85 for job Koha_Docs_3.14.x (previous build: SUCCESS)
14:59 nengard_manual The 3.16 manual isn't listed: http://jenkins.koha-community.org/
14:58 nengard_manual who manages jenkins? gmcharlt?
14:54 jenkins_koha   * Nicole C. Engard: update about perl modules image
14:54 jenkins_koha   * Nicole C. Engard: add usage stats to implementation checklist
14:54 jenkins_koha   * Nicole C. Engard: add html & css library link
14:54 jenkins_koha   * Nicole C. Engard: update kyle's koha tools link
14:54 jenkins_koha   * Nicole C. Engard: add HEA link to appendix
14:54 jenkins_koha   * Nicole C. Engard: update schema link in appendix
14:54 jenkins_koha   * Nicole C. Engard: add usage stats preferences
14:54 jenkins_koha   Project Koha_Docs build #566: SUCCESS in 4 min 7 sec: http://jenkins.koha-community.org/job/Koha_Docs/566/
14:53 jenkins_koha   Starting build #566 for job Koha_Docs (previous build: FIXED)
14:40 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11425 new feature, P5 - low, ---, julian.maurice, Needs Signoff , Search form for items
14:40 oleonard       I submitted a template follow-up for Bug 11425 (which was previously Passed QA) I hope someone can take a look.
14:39 jenkins_koha   * Nicole C. Engard: add AcqItemSetSubfieldsWhenReceiptIsCancelled
14:39 jenkins_koha   * Nicole C. Engard: add HoldsInNoissuesCharge
14:39 jenkins_koha   * Nicole C. Engard: add OverdueNoticeCalendar info to notices tool
14:39 jenkins_koha   * Nicole C. Engard: add OverdueNoticeCalendar
14:39 jenkins_koha   * Nicole C. Engard: remove SearchEngine
14:39 jenkins_koha   * Nicole C. Engard: remove OpacShowFiltersPulldownMobile
14:39 jenkins_koha   * Nicole C. Engard: remove references to opacsmallimage
14:38 jenkins_koha   * Nicole C. Engard: remove UseTablesortForCirc
14:38 jenkins_koha   Project Koha_Docs build #565: FIXED in 4 min 4 sec: http://jenkins.koha-community.org/job/Koha_Docs/565/
14:38 wahanui        o/ '`'`'`'`'`'`'`'`'`
14:38 jenkins_koha   Yippee, build fixed!
14:38 jenkins_koha   Starting build #565 for job Koha_Docs (previous build: FAILURE -- last SUCCESS #563 34 min ago)
14:37 nengard_manual WHAT???!!! No failures
14:30 jenkins_koha   Project Koha_Docs build #564: FAILURE in 10 min: http://jenkins.koha-community.org/job/Koha_Docs/564/
14:18 oleonard       Yes, thank you.
14:16 tcohen         oleonard: do i make any sense?
14:15 tcohen         without unmet dependencies that should also be packaged
14:14 tcohen         regardless of that, i have successfully build the .deb on 12.04
14:14 tcohen         squeeze is really old right now, and we could think of shifting to wheeze
14:13 oleonard       Sorry, but what does that comment mean?
14:12 tcohen         ah :-D
14:11 huginn         04Bug 11425: new feature, P5 - low, ---, julian.maurice, Passed QA , Search form for items
14:11 oleonard       http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11425#c53
14:11 tcohen         to see how difficult it would be
14:11 tcohen         oleonard: i'll try to pack it right now, on 12.04
14:08 tcohen         we need eythian to evaluate
14:07 oleonard       Is a backport required before pushing the patch?
14:06 jenkins_koha   * Nicole C. Engard: add updateNotforLoanStatusOnCheckin
14:06 jenkins_koha   * Nicole C. Engard: update implementation checklist with OPACItemHolds
14:06 jenkins_koha   * Nicole C. Engard: update hold section with new OPACItemHolds info
14:06 jenkins_koha   * Nicole C. Engard: add 'force' to OPACItemHolds
14:06 jenkins_koha   Project Koha_Docs build #563: SUCCESS in 1 min 45 sec: http://jenkins.koha-community.org/job/Koha_Docs/563/
14:06 jenkins_koha   Starting build #563 for job Koha_Docs (previous build: FIXED)
14:03 tcohen         i think it shouldn't be hard to backport
14:03 tcohen         it is on 14.04
14:03 tcohen         ah
14:03 oleonard       Template::Plugin::JSON::Escape
14:03 tcohen         which dependency?
14:02 oleonard       I'm wondering whether the Passed QA status is premature because of that issue.
14:02 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11425 new feature, P5 - low, ---, julian.maurice, Passed QA , Search form for items
14:02 tcohen         so it'd depend on how difficult is to do it
14:02 oleonard       I'm wondering because I'm testing Bug 11425, which introduces a dependency not available on Ubuntu <= 12.04.
14:01 tcohen         and packaged
14:01 tcohen         those dependencies need to be maintained
14:01 tcohen         oleonard: negotiate with eythian
14:01 oleonard       And is there a Debian or Ubuntu version cutoff?
14:01 oleonard       Do we have a rule about adding Perl dependencies which are not packaged?
14:00 tcohen         \o
14:00 magnuse        o/
13:59 tcohen         hi magnuse
13:55 nengard_manual not sure I can finish it in time ... but I'll do my best
13:55 nengard_manual :)
13:53 * oleonard     cheers nengard_manual
13:51 jenkins_koha   * Nicole C. Engard: rename  AuthoritySeparator pref
13:51 jenkins_koha   * Nicole C. Engard: add note about closing basket to receive
13:51 jenkins_koha   Project Koha_Docs build #562: FIXED in 1 min 58 sec: http://jenkins.koha-community.org/job/Koha_Docs/562/
13:51 wahanui        o/ '`'`'`'`'`'`'`'`'`
13:51 jenkins_koha   Yippee, build fixed!
13:51 jenkins_koha   Starting build #562 for job Koha_Docs (previous build: FAILURE -- last SUCCESS #560 25 days ago)
13:50 tcohen         morning!
13:45 jenkins_koha   Project Koha_Docs build #561: FAILURE in 11 min: http://jenkins.koha-community.org/job/Koha_Docs/561/
13:25 marcelr        bly
13:25 marcelr        probaly he won't mind
13:25 marcelr        i will update the report
13:25 cait           it's in the logs :)
13:24 cait           marcelr: i think travelling - but i asked him earlier abut the database update patch
13:24 marcelr        ok
13:24 cait           marcelr: he is in paris :)
13:23 marcelr        paul_p: around?
13:19 cait           and jajm++ :)
13:18 jajm           ok
13:12 cait           jajm: hm maybe check your patches? I am not sure right now but I worked through a pile of patches the last 2 weeks
13:04 jajm           cait, done. any other bug you want me to rebase ?
12:58 cait           jajm: thx!
12:51 magnuse        jajm++
12:47 jajm           cait, here! :) i'll rebase 7162 immediately
12:46 jcamins        That would make sense.
12:45 oleonard       I'm guessing the validator doesn't parse CSS so it ignores whatever is in a style attribute
12:44 oleonard       Yup. Flies right past the validator on both HTML5 and XHTML strict settings.
12:44 jcamins        Should be <span style="display: block;"> or <span class="block">
12:44 jcamins        oleonard: looks wrong to me.
12:43 oleonard       Is <span style="block"> actually valid, or just not flagged as invalid by the validator? Looks wrong to me.
12:40 cait           hi fridolin
12:40 cait           heh, i was trying not to say it :)
12:39 fridolin       hie community
12:38 magnuse        must. not. tinker. with. that.
12:38 * magnuse      would like to see a nice grid of the bugs in bz, with different colours for different statuses
12:35 cait           just acq tends to have conflicts more than every other area it hink
12:34 cait           I'd really like to see the 7xxx gone - and it's a nice feature too
12:34 cait           thx
12:34 paul_p         s/#koha/#biblibre/
12:34 paul_p         I asked him on #koha channel
12:34 paul_p         cait = probably afk for lunch
12:33 cait           paul_p: when will he be back?
12:33 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7162 enhancement, P3, ---, julian.maurice, Patch doesn't apply , Factorize code for order cancellation
12:32 cait           paul_p: it woudl be awesome if he could rebase his work on order cancellation  - bug 7162
12:32 paul_p         cait = do you have a specific question for jajm ?
12:32 paul_p         (cait: i'm in Paris, while jajm must be at home. Yes, he's around, but not on koha this week. Maybe the feature freeze can make an exception...)
12:32 druthb         feature freeze is friday :)
12:32 wahanui        druthb: I forgot feature freeze
12:32 druthb         forget feature freeze
12:32 paul_p         jajm = are you around ?
12:32 wahanui        well, feature freeze is 3 October 2013
12:32 druthb         wahanui:  feature freeze?
12:31 paul_p         cait yep, that's in my list...
12:31 cait           andalso, is julian around?
12:31 druthb         no, feature freeze is friday :)
12:31 cait           could you take another look at the database update work?
12:31 wahanui        feature freeze is, like, 3 October 2013
12:31 druthb         wahanui:  feature freeze?
12:31 paul_p         So it mean there are some patches I should sign-off between today and friday...
12:31 druthb         Fun Friday for Feature Freeze!
12:31 cait           yes ):
12:31 paul_p         a lot of F ...
12:30 cait           feature freeze is friday :)
12:30 paul_p         cait yep
12:30 cait           paul_p: around?
12:11 magnuse        cait: that yaml test must be stricter than koha, but yeah
11:57 * akafred      our installation setup is here https://github.com/digibib/ls.ext which uses our koha in docker setup from https://registry.hub.docker.com/u/digibib/koha/
11:54 * akafred      can now log in to our internal work-in-progress koha-installation with my OPL user from the old library system ... yay
11:51 oleonard       Hi everyone
11:51 cait           magnuse: yay for the tests :)
11:43 magnuse        i had "    -" when it should be "     -" ;-)
11:41 cait           cool :)
11:38 magnuse        cait: the yaml issue should be fixed now
11:32 tcohen         hi magnuse
11:32 magnuse        hola tcohen
11:31 magnuse        got it, i'll add that to one of the followups
11:29 cait           thx magnuse
11:24 magnuse        ah, then there was the yaml file
11:12 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11401 enhancement, P5 - low, ---, magnus, Signed Off , Add support for Norwegian national library card
11:12 magnuse        cait: two qa followups attached and status reset to Signed Off on bug 11401
10:54 magnuse        don't we all? ;-)
10:48 mtj            im giving advice about stuff i dont really understand
10:47 magnuse        thanks for your input, mtj. much appreciated!
10:46 mtj            i shut up now... :)
10:46 mtj            -w: Activates some warnings.
10:45 mtj            https://stackoverflow.com/questions/6302025/perl-flags-pe-pi-p-w-d-i-t
10:45 magnuse        dunno
10:43 mtj            perl -w does actually execute, with 'use warnings' forced?
10:42 magnuse        ah :-)
10:40 mtj            oops magnuse, 'perl -c' is the syntax check...  not perl -w
10:40 huginn         magnuse: The current temperature in Bodo, Norway is 5.0°C (11:20 AM CET on October 29, 2014). Conditions: Light Rain Showers. Humidity: 87%. Dew Point: 3.0°C. Windchill: 0.0°C. Pressure: 29.30 in 992 hPa (Steady).
10:40 magnuse        @wunder boo
10:38 huginn         mveron: The current temperature in Wetter Allschwil, Allschwil, Switzerland is 15.1°C (11:38 AM CET on October 29, 2014). Conditions: Overcast. Humidity: 73%. Dew Point: 10.0°C. Pressure: 30.15 in 1021 hPa (Steady).
10:38 mveron         @wunder Allschwil
10:35 mveron         hi magnuse :-)
10:35 magnuse        hiya mveron
10:35 mveron         Hi #koha
10:34 magnuse        something like "perl -c", perhaps?
10:34 mtj            ...it seems to be doing just a basic syntax check, rather than actually executing the code properly
10:33 mtj            the qa-tool sometimes gives false warnings, on stuff like that
10:32 mtj            cool
10:31 mtj            ..but when the file is tested with a basic 'perl -w ./foo.pm' the value is not defined - so we get a warning
10:31 magnuse        yup, a fresh install or running updatedatabase.pl sets it to 0, and the test is ok
10:30 mtj            i think the gotcha here, is that when koha is running properly that syspref value *is* defined ok
10:29 magnuse        kewl
10:28 mtj            "Use of uninitialized value in numeric eq (==)"
10:27 mtj            that warning is a common perl quirk
10:25 mtj            magnuse:  yep - it fixed it
10:24 magnuse        not sure if it explains the complaints about missing modules too
10:24 magnuse        that would explain the "Use of uninitialized value in numeric eq (==)" at least
10:23 magnuse        cait: i *think* it could
10:22 cait           could that be the difference?
10:22 cait           forgot to run the db updates
10:22 cait           maybe hte problem is that i don't have the pref in the db!
10:22 cait           ooh
10:21 magnuse        just to be clear, the "C4::Context->preference('NorwegianPatronDBEnable')   and ..." fixed it?
10:21 magnuse        yeah
10:20 mtj            or 'invalid perl'  even
10:19 mtj            would be a nice tweak for qa-tool to say 'FAIL valid perl'
10:17 mtj            hey magnuse, that did fix the warning ^
10:15 mtj            timmytoady
10:15 magnuse        hehe, yeah
10:15 mtj            looks fcking horrible tho
10:14 magnuse        yeah, i can always do that, if it makes perl happy :-)
10:14 mtj            something like that?
10:14 mtj            if ( C4::Context->preference('NorwegianPatronDBEnable')   and  C4::Context->preference('NorwegianPatronDBEnable') == 1 ) {}
10:14 mtj            i guess what perl wants to make it happy is...
10:13 khall          hi!
10:13 mtj            hiya khall
10:13 mtj            so, very probably an annoying false warning?
10:13 khall          mornin!
10:13 mtj            its just some basic warning error from running 'perl -w'
10:13 magnuse        kia ora khall
10:12 mtj            ...which is not a real test of the syspref value, (i think)
10:11 mtj            $ perl -w C4/Members.pm
10:11 mtj            magnuse:  fyi, i get that error from running ...
10:11 magnuse        cait: do you remember if you ran updatedatabase before the qa script?
10:11 mtj            i might try that now..
10:10 magnuse        that is very similar to what cait reported on the bug, akshuly
10:09 magnuse        'NorwegianPatronDBEnable' would be initialized if you ran updatedatabase.pl, methinks
10:09 mtj            ah, ok..
10:08 magnuse        ah, that line is: if ( C4::Context->preference('NorwegianPatronDBEnable') == 1 ) {
10:08 mtj            qa-tool seems to remove the line number...
10:07 magnuse        ah, at least that explains the FAIL for valid earlier
10:07 mtj            Use of uninitialized value in numeric eq (==) at /home/mason/git/head/C4/Members.pm line 47.
10:06 cait           yeah :(
10:06 magnuse        yup
10:05 mtj            meh, pity there is no line number :/
10:05 mtj            Use of uninitialized value in numeric eq (==)
10:05 mtj            FAIL   valid
10:04 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13088 enhancement, P5 - low, ---, tomascohen, Pushed to Master , Allow the user to specify a max amount of facets to show.
10:04 pastebot       "mtj" at 127.0.0.1 pasted "testing 2 commit(s) (applied to 3541fc3 'Bug 13088: (RM followup) add missing ')" (153 lines) at http://paste.koha-community.org/260
10:04 magnuse        s/leace/leave/
10:04 magnuse        it does seem to leace some *.pc files in /tmp
10:03 mtj            i wonder if its not tiding-up after itself, after each run
10:03 mtj            weird
10:02 magnuse        two different ways of saying "Spurious =cut command  in file C4/Members.pm", then no mention of it
10:01 magnuse        yay, now koha-qa.pl is giving me different results every time i run it
10:01 pastebot       "magnuse" at 127.0.0.1 pasted "Different results from koha-qa.pl" (29 lines) at http://paste.koha-community.org/259
09:57 magnuse        cool
09:56 cait           if it's just syntax and you already found it... a follow up would be accepted :)
09:56 cait           hm you can fix or point out it was not you
09:54 magnuse        cait: the qa tool complains about a couple of pod problems in C4/Members that was not caused by me. should i do a qa followup to fix that, or report it as a separate bug or ignore it?
09:49 magnuse        akafred: nah, perlcritic seems to be happy
09:48 magnuse        nah, C4/Members.pm too
09:48 cait           it also seems to only appear in the nl specific files?
09:48 cait           i think best to look at the code and verify it's ok
09:48 cait           sadly me neither
09:47 magnuse        not sure what "valid" means, though
09:47 magnuse        mtj: perl -c tells me the syntax should be ok
09:46 magnuse        cait: mtj got one fail for pod too, so i'll fix that
09:46 Oak            Guten tag cait
09:46 Oak            :)
09:46 magnuse        Oak
09:46 Oak            magnuse
09:46 * Oak          waves
09:45 cait           boo
09:45 cait           so only failing ofr me?
09:44 akafred        magnuse: perlcritic tripped you up?
09:44 mtj            yeah, im not sure what that means... 'valid'
09:44 akafred        hola amigo :-)
09:44 magnuse        hei akafred
09:44 magnuse        lots of fails on "valid" - that's... interesting...
09:43 magnuse        mtj: thanks a bunch
09:42 mtj            magnuse:  there you go... ^
09:42 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13088 enhancement, P5 - low, ---, tomascohen, Pushed to Master , Allow the user to specify a max amount of facets to show.
09:42 pastebot       "mtj" at 127.0.0.1 pasted "testing 2 commit(s) (applied to 3541fc3 'Bug 13088: (RM followup) add missing ')" (131 lines) at http://paste.koha-community.org/258
09:20 magnuse        mtj: np
09:00 mtj            oops fone..
08:53 mtj            lookng now...
08:51 magnuse        nah, looks like it can read it
08:50 magnuse        s/koha_kohadev/kohadev-koha/
08:49 magnuse        hm, the koha_kohadev user might not be able to read a file in /home/vagrant...
08:48 magnuse        shouldn't this work, then? $ sudo koha-shell -c "PERLCRITIC=/home/vagrant/.perlcriticrc perl -I/home/vagrant/qa-test-tools/ /home/vagrant/qa-test-tools/koha-qa.pl -c 2 -v 2" kohadev
08:48 magnuse        "The default configuration file is called .perlcriticrc. perlcritic will look for this file in the current directory first, and then in your home directory. Alternatively, you can set the PERLCRITIC environment variable to explicitly point to a different file in another location." from https://metacpan.org/pod/perlcritic#CONFIGURATION
08:47 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11401 enhancement, P5 - low, ---, magnus, Failed QA , Add support for Norwegian national library card
08:47 magnuse        mtj: oh, could i bother you to run the qa script on the two patches from bug 11401?
08:46 magnuse        cait and i are getting some different results
08:46 magnuse        yup, but is that affected by the .perlcriticrc config file?
08:46 mtj            its not perlcriric
08:45 mtj            its saying the =head2 section is empty, i think
08:44 magnuse        is that perlcritic or something else?
08:43 magnuse        mtj: when i get a pod fail like that, what is it that is finding the failure?
08:43 pastebot       "magnuse" at 127.0.0.1 pasted "pod fail" (10 lines) at http://paste.koha-community.org/257
08:41 mtj            perhaps warn if it cant find one, or the md5 doesnt match what it expects
08:39 magnuse        yeah
08:38 mtj            magnuse:  would be a good tweak to have the qa-tool show what perlcritic file/options its using
08:20 gaetan_B       hello
08:19 kivilahtio     bye cait
08:19 * cait         waves to #koha
08:18 cait           jajm: around?
08:14 huginn         magnuse: The operation succeeded.
08:14 magnuse        @later tell cait i think my .perlcriticrc does not get picked up, because i run the tests through koha-shell, which means they get run as the koha user, which does not have the .perlcriticrc
07:49 magnuse        hiya alex_a
07:48 alex_a         bonjour
07:47 cait           i go :)
07:47 magnuse        now, off you go! :-)
07:47 magnuse        ok
07:46 cait           yes
07:46 magnuse        cait: you ran the tests on ubuntu?
07:46 magnuse        thanks for investigating cait
07:43 cait           ihave to go to work :(
07:43 cait           then we can see if we can catch someone to repeat the tests
07:42 cait           yaml and back to signed off?
07:42 cait           pf
07:41 * magnuse      thinks he will have to wait a long time if he has to find someone more clever than cait
07:41 cait           Use of uninitialized value in numeric eq (==
07:41 cait           wonder what those mean
07:40 wahanui        privet, reiveune
07:40 reiveune       hello
07:40 cait           someone more clever than me... if they say it's ok maybe the script is dumb :)
07:40 magnuse        worth a try ;-)
07:39 cait           maybe we need someone else to run it
07:39 magnuse        gah!
07:39 cait           everything like before after updating the file
07:39 cait           pod is still there too
07:39 magnuse        about the missing module?
07:39 cait           ah no
07:39 cait           the others remain
07:39 cait           the pod one seems gone onw
07:38 magnuse        i have $ perl -MPerl::Critic -e 'print "$Perl::Critic::VERSION\n"' = 1.122
07:38 cait           hi kivilahtio
07:38 kivilahtio     days
07:37 cait           and the link is a good idea
07:37 cait           hm i am missing the last line
07:37 magnuse        i have a link from the git repo
07:37 cait           i have the perlcriticrc copied to my home directory
07:36 pastebot       "magnuse" at 127.0.0.1 pasted ".perlcriticrc" (14 lines) at http://paste.koha-community.org/256
07:36 cait           not sure why mine is more strict
07:36 cait           dunno
07:36 magnuse        did i mess up some perlcritic config file?
07:35 magnuse        but why for you and not for me?
07:35 magnuse        probably
07:34 cait           =head2 Check sysprefs  =cut
07:34 cait           i think it complains about those
07:34 cait           http://paste.koha-community.org/255
07:34 magnuse        could you paste your misc/cronjobs/nl-sync-from-koha.pl?
07:33 magnuse        yeah
07:33 magnuse        gah!
07:33 cait           21 - below the 18 entry
07:33 cait           ah no, yes, i got it
07:33 magnuse        ah!
07:33 cait           i have 18
07:33 cait           hm no
07:32 cait           it is empty
07:32 magnuse        you do have "Update 2014-10-21" in the big patch?
07:32 cait           *** ERROR: empty =head2  in file misc/cronjobs/nl-sync-from-koha.pl
07:32 cait           FAIL	  pod
07:32 cait           but this look slike a valid one
07:31 * magnuse      neither
07:31 magnuse        $ perl -e "use Crypt::GCrypt" gives me: Can't locate Crypt/GCrypt.pm in @INC
07:31 cait           i am not sure what's going on :(
07:31 cait           hmpf
07:31 magnuse        i'm on a fresh box, without the NL deps
07:30 cait           could you try uninstalling the dependencies added for a test?
07:30 wahanui        interesting is sometimes good and sometimes bad
07:30 cait           hm interesting
07:30 magnuse        for me, qa-test-tools/koha-qa.pl is all green
07:30 cait           but there are some about numeric eq and a POD one too
07:30 cait           i think i see it  - it's in the files that you would only use if you are using the feature
07:29 pastebot       "magnuse" at 127.0.0.1 pasted "koha-qa.pl -c 2 -v 2" (129 lines) at http://paste.koha-community.org/254
07:29 cait           magnuse: http://paste.koha-community.org/253
07:27 cait           do you have that installed?
07:27 cait           magnuse: the qa script fails
07:27 cait           oh hm
07:25 cait           so when the pref one is fixed, switch back to signed off?
07:25 cait           i don't see the other problems any more :)
07:25 wahanui        right now is testing just somebody seeing if it works?
07:25 magnuse        right now
07:25 cait           probably just a tiny typo somewhere
07:25 magnuse        i'll look at it
07:25 magnuse        fails for me too
07:25 magnuse        yup
07:24 cait           can you check the yaml one on your computer?
07:23 cait           but you are right, overall it looks better
07:23 cait           #   at xt/yaml_valid.t line 39.
07:23 cait           #   Failed test 'patrons.pref is YAML'
07:23 cait           not ok 11 - patrons.pref is YAML
07:23 cait           xt/yaml_valid.t
07:23 cait           hm we got a failing test
07:19 * magnuse      keeps his fingers crossed
07:19 cait           tests are running now
07:19 magnuse        yay
07:18 cait           updating master
07:18 cait           but i can try again righ tnow
07:18 cait           hm pretty usre
07:16 magnuse        are you sure you ran it on the latest version of the patch?
07:12 magnuse        that is weird
07:12 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11401 enhancement, P5 - low, ---, magnus, Failed QA , Add support for Norwegian national library card
07:12 magnuse        cait: i just ran qa-test-tools on the two commits on bug 11401 on a fresh debian box (yay kohadevbox) - and got everything green and ok
07:06 magnuse        or fun, your choice
07:06 magnuse        have fum mtj
07:04 mtj            hi magnuse, cait - i keep quiet while i do more .po reading... :)
07:01 magnuse        yay
07:00 cait           some regression testing and then ready to go
07:00 cait           i feel we are close once we resolve that
07:00 cait           hope it was not my mistake
07:00 cait           sorry about that
06:59 magnuse        cait: i was so sure i had the tests working without the modules this time... looking at tit now
06:53 mtj            and the TT blocks cause problems, for translation in pootle
06:52 mtj            it seems the TT blocks are not needed in the .po files, for translation
06:51 mtj            i wonder why TT blocks cannot just be removed completlely,  when building the .po files
06:48 mtj            i have always be aware of this translation limitation/problem with koha -  but i have never really looked closely at it
06:46 mtj            nice paxed, it seems you have the same idea as me
06:43 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12221 enhancement, P5 - low, ---, pasi.kallinen, NEW , Remove TT statement placeholders from translatable strings
06:43 paxed          mtj: something along the lines of bug 12221 perhaps?
06:41 cait           morning magnuse :)
06:41 * magnuse      waves
06:41 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12505 normal, P5 - low, ---, simith.doliveira, Needs Signoff , Variable aqorders.listprice from 'Tools' - 'Notices and Slips' Does not work
06:41 magnuse        bug 12505
06:40 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12851 normal, P5 - low, ---, jonathan.druart, Needs Signoff , order tags are not removed on claiming serials/orders
06:40 cait           it would be nice if someone could take a look at but 12505 and bug 12851
06:13 mtj            aah, i just saw rangi's misc/translator msg ^
06:12 mtj            anyhoo... it might not actually be technically possible? - so i keep quiet now :)
06:09 mtj            which would be a pretty big deal
06:09 mtj            ..but cait - what i am suggesting... is for us to consider a solution, that would allow koha templates to have any type of TT expression
06:08 mtj            yep
06:07 cait           you just have to show that the translated template with a po file that hs the string translated is correct
06:07 cait           i think i'd try the || idea
06:07 cait           hheh
06:06 mtj            i'm not suggesting that anything i have described above woud actually work...
06:05 mtompset       Have a great day, #koha cait mtj rangi
06:05 * mtompset     grins.
06:05 cait           i see bugs in the bigger picture  :)
06:05 mtj            think about the bigger picture here cait
06:04 cait           sounds a bit overly complicated to avoid writing a few more lines
06:03 mtj            ..then replace strings with correct TT blocks after
06:02 mtj            substitute problematic TT blocks with a unique placeholder string, before translation
06:01 cait           perl translate update lang - translate the new string - then install
06:01 mtj            heres what i cant help but wonder... is it somehow possible to substitute TT [% %] blocks in template files, before generating .po files?
06:01 mtompset       I'm going to bed. This is crazy early.
06:00 rangi          bbl
06:00 rangi          you would have to create the .po file
06:00 rangi          nope
05:59 mtompset       So, if I applied a patch with a tweaked TT file, and then installed a new language for koha, that would show me if the sexy version works?
05:59 * rangi        has to go put kids to bed
05:59 rangi          but its all in misc/translator if anyone wants to have a go at making them be able to handle constructs they dont now
05:59 * mtompset     nods.
05:58 rangi          and the needs of the translators outweigh the needs of the developers about 100 million times :)
05:58 rangi          the scripts to generate po files
05:57 mtj            so give me a slap here... which part of koha has a problem with 'complex' TT files ?
05:57 rangi          heya mtj
05:56 mtj            hiya rangi
05:55 rangi          but thats all outside of pootle, pootle is for translating
05:55 rangi          we have scripts that make .po files and then make new tt files from translate po files
05:54 rangi          it handles .po files
05:54 rangi          pootle will never handle tt files
05:52 mtj            wow, it would be so great if we managed to get pootle to handle TT files better...
05:48 mtj            the pootle magic happens on this box  -> http://translate.koha-community.org
05:47 mtj            its an area i dont know much about
05:45 mtj            i didnt much, grepping for pootle/poedit in the repo :/
05:44 mtj            i reckon theres a good chance pootle might handle that syntax ok
05:43 mtj            yeah
05:43 mtompset       Oh... you were talking about it looking sexy. :)
05:40 mtj            hmm, i cant quite remember, right now... :/
05:39 mtompset       Thanks for checking. :)
05:39 mtompset       Okay, then how would I prove/disprove that?
05:39 mtj            and yes, it does :)
05:38 mtj            i guess if we could prove that pootle handled the file correctly, then it might be ok
05:38 mtompset       Because doesn't that second version just look sexy? :)
05:37 mtompset       That's why I asked it. :)
05:36 mtj            hmm, a good question mtompset...
04:50 mtompset       Anyone have a comment on the paste?
04:34 pastebot       "mtompset" at 127.0.0.1 pasted "Coding Guidelines Question" (6 lines) at http://paste.koha-community.org/252
04:31 mtompset       Greetings, #koha.
04:25 kathryn        will perhaps try oleonard about this later. seeyas!
04:12 kathryn        I haven't been in this channel for about a month, sorry to drop a clanger ;)
04:11 kathryn        hi everyone, does anyone know where the discussion got to about which versions of IE are supported in Koha?
03:09 dcook          Once upon a time, I could've told you all sorts of stats about soccer teams though.
03:09 jcamins        More than I'm doing.
03:09 * dcook        is just Googling
03:08 * jcamins      believes you.
03:08 dcook          I think 2005 was during the NHL lockout?
03:08 * jcamins      obviously wasn't paying *that much* attention.
03:06 jcamins        It might've been 2005.
03:05 dcook          Apparently the Carolina Hurricanes won the 2006 stanley cup?
03:05 jcamins        I'm sure they won the hockey equivalent if it was a hockey team.
03:04 dcook          It would be exciting if a hockey team won the world series...
03:04 jcamins        It now strikes me that the person who was so excited was actually more of a hockey fan, so maybe it was a hockey team.
03:04 jcamins        I think.
03:04 jcamins        I understand there was a baseball team in this area that until they won somewhere around 2006 hadn't won a world series since my parents were tiny.
03:03 wizzyrea       yep, and were terrible for 30 years after.
03:03 jcamins        But they won in your lifetime!
03:00 wizzyrea       I was very small when the royals last won the world series.
02:59 wizzyrea       can I have a gratuitous former-kansan moment - the Royals are up 10-0 in a giants led 3-2 series... madness.
02:58 jcamins        It's pretty evil.
02:58 jcamins        Hehe.
02:58 wizzyrea       that label. we hates it, precious.
02:57 wizzyrea       I think I'll add that one to the wiki.
02:57 wizzyrea       thank you very very very much.
02:56 wizzyrea       two heads are way better than one :)
02:56 jcamins        That option didn't occur to me.
02:56 wizzyrea       no, not as crazy :)
02:56 jcamins        Though not nearly as crazy.
02:56 jcamins        Yeah, that's better.
02:56 jcamins        Heh.
02:56 jcamins        Oh.
02:56 wizzyrea       instead of the body append
02:55 wizzyrea       it totally works. Though I just stuck it in the credits in a script tag :P
02:55 jcamins        Maybe we will reach the singularity!
02:55 jcamins        I'm not really sure what will happen.
02:55 wizzyrea       that doesn't seem like it would be necessary lol.
02:54 jcamins        :D
02:54 jcamins        $('body').append("<script>$(document).ready(function () { $('.confirmjs_hold label').contents().filter(function () { return this.nodeType === 3; }).replaceWith(' My awesome text'); });</script>");
02:53 * wizzyrea     chucks it in the credits.
02:53 jcamins        I want to know what happens!
02:53 jcamins        That said, do it!
02:53 wizzyrea       I suspected so.
02:53 wizzyrea       >.>
02:52 jcamins        It's possible that may have been a Freudian slip revealing my true feelings about doing things that crazy.
02:51 jcamins        *wrote
02:51 jcamins        lol
02:51 wizzyrea       keke.
02:51 wizzyrea       we just wrong.
02:51 jcamins        That is, in $(document).ready(...) append a <script> element with the code we just wrong.
02:51 jcamins        There are a variety of increasingly crazy ways to deal with it. I suggest trying to append a script element to the end of the page, 'cause that seems just crazy enough it might work.
02:50 wizzyrea       my noticer has been broken this week.
02:50 * wizzyrea     should have noticed that <facepalm>
02:50 wizzyrea       oh that's rather a problem.
02:50 jcamins        Where that = the labels
02:49 jcamins        That's getting generated in $(document).ready(...) after opacuserjs is already run.
02:49 jcamins        Yeah, that won't work.
02:49 wizzyrea       as is, inside $(document).ready(function(){
02:48 jcamins        How are you running it in opacuserjs?
02:48 wizzyrea       everything else in there works.
02:47 wizzyrea       but not from opacuserjs hmm.
02:47 wizzyrea       well it works in the console
02:47 wizzyrea       hehe there is.
02:47 * dcook        thinks there might be a lot of water and land in the way.
02:46 wizzyrea       I could hug you.
02:46 jcamins        This works for me: $('.confirmjs_hold label').contents().filter(function () { return this.nodeType === 3; }).replaceWith(' My awesome text')
02:46 jcamins        Ohh!
02:44 wizzyrea       I can see one back in my many go's at this that looks extremely similar to what we've just come up with.
02:44 jcamins        Now I'm curious.
02:44 wizzyrea       up to you :)
02:43 jcamins        Maybe I'll try running some code.
02:43 wizzyrea       I'm glad it's not just me, and that this one actually is tricky.
02:43 jcamins        Hm.
02:43 jcamins        Really?
02:43 wizzyrea       yeah both of those still nuke the checkbox
02:38 oleonard       jcamins: It is.
02:37 wizzyrea       thank you both :)
02:37 jcamins        *?
02:37 jcamins        oleonard: isn't it a bit late for you/
02:37 wizzyrea       lol
02:37 * oleonard     runs off to count children
02:37 jcamins        Not children()
02:37 jcamins        contents()
02:36 jcamins        Or contents()
02:36 jcamins        oleonard: oh, no, it would work, you're missing .children()
02:33 * jcamins      still can't remember why he sat down at his computer.
02:32 jcamins        Sorry, didn't test.
02:32 jcamins        oleonard: oh, you're right. You need it to be this.nodeType === 3 && this.textContent...
02:31 oleonard       Oh no, never mind. Checkbox removed.
02:31 oleonard       $('.confirmjs_hold label').filter( function () { return this.textContent === ' Place a hold on ' }).text('My new awesome text yay');
02:31 mtj            ...thats often the approach i use
02:30 mtj            hey wizzyrea, can you get/use the xpath of that element?
02:23 wizzyrea       :)
02:23 wizzyrea       yeah I was looking at that
02:23 jcamins        We can modify that to use my strategy.
02:23 jcamins        Well, how were you doing it that removed the checkbox?
02:23 wizzyrea       no
02:23 jcamins        It doesn't?
02:22 jcamins        You can replace this.nodeType === 3 with something more appropriate like this.textContent === 'Place a hold on ' ... if that's more appropriate.
02:22 wizzyrea       hm no. The version I'm using doesn't have a class on that label.
02:20 wizzyrea       hah
02:20 jcamins        ^^ now with more jQuery
02:19 jcamins        wizzyrea: $('.confirm_label').contents().filter(function () { return this.nodeType === 3 }).text('My new awesome text yay');
02:18 wizzyrea       oh hi jcamins :)
02:18 mtj            cheers for the awesome gwar eythian
02:18 wizzyrea       hrm.
02:17 jcamins        wizzyrea: it's kind of icky... I think you need to not use jQuery, and instead just get the text node.
02:16 wizzyrea       the problem I mostly have, is that trying to change it removes the checkbox.
02:16 oleonard       Oh no the internet can think of so much stuff
02:15 wizzyrea       and most things the internet can think of.
02:15 wizzyrea       because I've tried everything I can think of
02:15 wizzyrea       to change it
02:15 wizzyrea       http://i.imgur.com/aIubvW6.png how can I select that text
02:14 eythian        oh no! the serios face!
02:13 * oleonard     puts on his serious face
02:13 wizzyrea       I'm stumped.
02:13 wizzyrea       well serious.
02:13 wizzyrea       oleonard - i have a serious jquery question for you :/
02:09 jcamins        That matches mine.
02:09 eythian        that looks sane
02:09 jcamins        eythian: (I mean, if it's not there)
02:09 eythian        I'm not sure, I don't know a lot about the XML parsing modules
02:09 oleonard       /usr/share/perl5/XML/SAX/Expat.pm
02:09 oleonard       /usr/lib/perl5/XML/Parser/Expat.pm
02:08 jcamins        eythian: I'm a bit unclear about why Expat should be loaded at all.
02:08 eythian        does locate Expat.pm return something sane?
02:07 oleonard       No
02:07 jcamins        Did you install any XML things using CPAN?
02:05 oleonard       I have them set in .bashrc but I'm not sure they're correct for a gitified package installation
02:03 jcamins        oleonard: did you set the PERL5LIB and KOHA_CONF environment variables manually?
02:03 * oleonard     pokes himself a bit
02:03 oleonard       Myself
02:02 jcamins        oleonard: what user are you running the test as?
02:02 eythian        do you have XML::SAX::Expat installed?
02:02 oleonard       jcamins: No, I don't know about that.
02:02 dcook          Thanks, oleonard :)
02:02 jcamins        Did you use koha-shell to switch to the correct user?
02:01 * jcamins      was thinking "when I try to create a user" or "when I catalog a new record and then delete it immediately."
02:01 jcamins        Sorry...
02:01 jcamins        An actual test.
02:01 jcamins        Oh, right.
02:01 oleonard       t/db_dependent/ReportsGuided.t
02:01 * wahanui      does a testing dance
02:01 jcamins        oleonard: what test?
02:00 oleonard       Hey dcook, congratulations!
02:00 oleonard       So I will inflict my question up on you jcamins: On a package install, when I run a test I get an error about not being able to open XML/SAX/Expat.pm
01:59 * dcook        pokes himself a bit
01:59 dcook          I think so?
01:59 jcamins        oleonard: regretfully, yes.
01:57 oleonard       Anyone awake?
00:36 * ibeardslee   goes back to Andy Williams after that
00:22 eythian        http://www.avclub.com/video/gwar-covers-pet-shop-boys-211000 <-- music for today