Time  Nick           Message
05:53 dcook          Just came in to say that people have been killing it in Koha over the past while :). Developing in master at the moment, and seeing all sorts of goodies :)
05:53 Francesca      hey dcook
05:53 dcook          yo Francesca
05:53 Francesca      hows aussie
05:53 * dcook        gives his thumbs up to the person who added deleted record support for OAI-PMH
05:53 dcook          Mmm, busy
05:53 dcook          You?
05:53 wahanui        You are not the first to ask that question or a little confused about the function of some installed extensions. or missing the trigger
05:53 Francesca      lol wahanui
05:53 Francesca      I have a migrane
05:54 Francesca      so in bed
05:54 Francesca      @wunder wlg
05:54 huginn         Francesca: The current temperature in Wellington, New Zealand is 8.0°C (5:30 PM NZST on August 11, 2015). Conditions: Light Rain Showers. Humidity: 81%. Dew Point: 5.0°C. Windchill: 3.0°C. Pressure: 30.12 in 1020 hPa (Steady).
07:19 cait           morning #koha
07:35 AmitG          heya cait
08:15 drojf          morning #koha
08:17 cait           morning drojf
08:17 drojf          hi cait
08:20 drojf          @wunder berlin, germany
08:20 huginn         drojf: The current temperature in Berlin Tegel, Germany is 26.0°C (9:50 AM CEST on August 11, 2015). Conditions: Clear. Humidity: 51%. Dew Point: 15.0°C. Pressure: 29.95 in 1014 hPa (Steady).
08:56 * andreashm    waves
09:09 * cait         waves back ;)
09:23 drojf          hi andreashm
09:24 andreashm      hi drojf!
09:36 fridolin       hie
09:37 fridolin       kivilahtio: For bug 14375 I have provided a test module that reproduces the problem
09:37 huginn         04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14375 critical, P5 - low, ---, fridolin.somers, Needs Signoff , DBIx::Connector should be stored in C4::Context instead of dbh
09:38 kivilahtio     fridolin: yes I saw that . It is great news!
09:38 fridolin       it simply runs foreach with a query each 15 min
09:38 kivilahtio     fridolin: unfortunately you have still the changed hash values and subroutine names
09:38 fridolin       this simulates a SIP server with few work
09:38 kivilahtio     fridolin: I will glady sign off your patch if you revert the namin convention
09:39 fridolin       kivilahtio: tell me more
09:39 fridolin       i dont understand
09:39 fridolin       comment 11 ?
09:39 kivilahtio     fridolin: see comment 7 and 11
09:40 kivilahtio     we want you to preserve the dbh and not replace it with DBconn
09:40 kivilahtio     because dbh is directly used from many pplaces
09:40 kivilahtio     also we don't see the point in why dbh needs to be changed to DBconn
09:41 fridolin       its the purpose of this patch
09:42 kivilahtio     fridolin: the purpose of the patch is to change from DBI to DBIx::Connect
09:42 fridolin       with dbix, dbh can not be added to context
09:42 kivilahtio     no I mean the C4::Context->dbh
09:42 kivilahtio     the HASH key dbh
09:42 fridolin       because it is the BDIX::Connector how will cache it
09:42 kivilahtio     the HASH key 'dbh'
09:43 kivilahtio     %context{'dbh'} has been changed to %context{'DBconn'}
09:43 fridolin       no, the purpose is to cache BDIx::Connexion and not dbh
09:44 kivilahtio     fridolin: yes I understand that, but why do you have to change the hash key?
09:44 kivilahtio     $context{'dbh'} = DBIx::Connector
09:44 fridolin       because vars in C4::Context hash are cached
09:44 fridolin       dbh must be given by BCconn that gives the cached one or recreated a new one
09:45 fridolin       you want to keep the hash key name but store the DBIx::Connector object ?
09:46 kivilahtio     fridolin: isn't it compatible with DBI and DBIx?
09:46 kivilahtio     DBIx::Connector should have all the subroutines of DBI and DBIx?
09:46 kivilahtio     so it is a drop-in replacement for DBI
09:46 fridolin       not sure
09:46 kivilahtio     me neither?
09:46 kivilahtio     but I know DBIx::Connector is a drop-in replacement for something :)
09:47 kivilahtio     atleast for DBIx
09:47 kivilahtio     and Imho you can use DBIx handle to do DBI operations
09:48 fridolin       yes for BDconn->dbh, its a DBI object
09:48 fridolin       but DBIx::Connector is not
09:50 kivilahtio     fridolin: I don't understand DBconn -> dbh?
09:51 fridolin       DBIx::Connector as a method dbh() to get the handler
09:51 kivilahtio     ah now i see
09:52 kivilahtio     reading the DBIx::Connector manual helps
09:52 fridolin       it returns a BDI object
09:52 fridolin       I read the code :D
09:52 fridolin       sub _connect for example calls
09:52 fridolin       my $dbh = $self->{_dbh}
09:53 fridolin       so DBIX::Connector does not implement DBI, its more a wrapper around dbh
09:53 kivilahtio     ye
09:54 kivilahtio     so you are saying that your modification adds 'DBconnect' key alongside 'dbh'
09:54 kivilahtio     so you refactor the C4::Context-dbh to utilize DBIx::Connector, and save the connector to 'DBconn' but then save the handle to 'dbh' key?
09:56 fridolin       no, the 'dbh' key is removed
09:56 kivilahtio     hmm
09:57 fridolin       it must be dynamic, given from DBConn
09:57 kivilahtio     I understand
09:57 kivilahtio     I am worried about the backslash
09:57 kivilahtio     but accessing the private methods of C4::Context is bad so they should be burnt for it anyway
09:58 fridolin       indeed
09:58 kivilahtio     they == me
09:58 kivilahtio     :)
09:58 fridolin       i have provided a big patch for test modules that used db mock
09:59 fridolin       why use C4::Context->_new_dbh() instead of C4::Context->dbh() ?
09:59 kivilahtio     fridolin: to work around the issue in dbh()
09:59 fridolin       its a performance killer
09:59 kivilahtio     I cant really remember anymore why
09:59 fridolin       which issue ?
10:00 kivilahtio     maybe you want to have two separate DB connections
10:00 kivilahtio     but I get your point
10:01 kivilahtio     fridolin: have you ran the test suite?
10:01 kivilahtio     fridolin: do some things break?
10:01 fridolin       kivilahtio: you mean all UT ?
10:01 kivilahtio     yes
10:01 kivilahtio     well whatever there is in t/*
10:01 kivilahtio     make test
10:01 kivilahtio     I prseume is the command
10:02 fridolin       nope, only on modules i have changeed
10:02 fridolin       because I do not have a proposer test database
10:02 kivilahtio     you should run it to test against regression
10:02 fridolin       also it is much better if someone else tests ;)
10:02 kivilahtio     fridolin: you can just CREATE DATABSE koha_test;
10:02 kivilahtio     haha
10:03 fridolin       yes but db_dependant needs specific database contents
10:03 kivilahtio     fridolin: yes, the one generated by the web-isntaller
10:03 kivilahtio     it is so difficult I know, and we are so lazy
10:03 fridolin       marc21 with all samples ?
10:03 kivilahtio     fridolin: you dontneed samples
10:04 kivilahtio     fridolin: or if you do, something has done something wrong when creating tests
10:04 fridolin       ah oki, but marc21 sure
10:04 kivilahtio     fridolin: I guess it is ok if you just manage to run it with any marc framework
10:04 fridolin       borrowers categories and branches are mandotory i remember
10:04 fridolin       nope
10:04 fridolin       records creation are using a marc21 framework
10:05 fridolin       like in Search.t
10:10 fridolin       Running UT gives a warning about DBIx::Class::Storage::DBI, but only when using BDI::Mock
10:12 fridolin       All tests successful.
10:12 fridolin       Files=99, Tests=5964, 30 wallclock secs ( 0.75 usr  0.12 sys + 19.79 cusr  1.55 csys = 22.21 CPU)
10:12 fridolin       Result: PASS
10:12 fridolin       kivilahtio: cool, all is OK 8-)
10:12 fridolin       same for xt
10:12 kivilahtio     great
10:14 kivilahtio     fridolin: I will apply it to our dev environemnt and tell you what happens
10:15 fridolin       thanks
10:15 fridolin       kivilahtio: you are working on SIP server right ?
10:16 kivilahtio     fridolin: we  allready have a patch for it so it is working
10:16 kivilahtio     it has worked fro 6 months, but I just discovered the real cause for it few weeks ago
10:17 kivilahtio     and then we ended up to do the same thing differently
10:18 fridolin       kivilahtio: could we have this patch ?
10:18 kivilahtio     yes
10:18 kivilahtio     fridolin: it a the best solution
10:18 fridolin       we also have a temporary solution but its dirty
10:18 kivilahtio     like I said I will take it to our dev/preprod, there we have running dameons
10:18 fridolin       its restoring the dhb->ping call
10:18 kivilahtio     and we  will see how it works out
10:19 fridolin       you said "we already have a patch", with patch ?
12:29 tcohen         morning
12:29 tcohen         marcelr
12:29 tcohen         ?
12:29 tcohen         @wunder cordoba, argentina
12:29 huginn         tcohen: The current temperature in Bo Altos de San Martin - NW, Cordoba city, Cordoba City, Argentina is 11.7°C (9:25 AM ART on August 11, 2015). Conditions: Light Drizzle. Humidity: 88%. Dew Point: 10.0°C. Pressure: 30.18 in 1022 hPa (Steady).
12:29 tcohen         hi cait
12:35 drojf          hi tcohen
12:35 drojf          @wunder berlin, germany
12:35 huginn         drojf: The current temperature in Berlin Tegel, Germany is 31.0°C (1:50 PM CEST on August 11, 2015). Conditions: Clear. Humidity: 38%. Dew Point: 15.0°C. Pressure: 29.95 in 1014 hPa (Steady).
12:35 drojf          i win again ;)
12:35 tcohen         hi drojf
12:35 tcohen         hmpf
12:35 tcohen         :-D
12:35 drojf          @wunder konstanz
12:36 huginn         drojf: The current temperature in Konstanz, Germany is 27.0°C (2:00 PM CEST on August 11, 2015). Conditions: Scattered Clouds. Humidity: 47%. Dew Point: 17.0°C. Pressure: 29.96 in 1014 hPa (Falling).
12:36 tcohen         i assume cait is sunbathing
12:51 Kchris         quit
12:56 tcohen         guys, it is not fair that everyone is on vacation
12:58 drojf          tcohen: so true
12:58 drojf          < 200 "needs signoff"
12:59 drojf          199 is still a scary number :P
12:59 tcohen         sure!
12:59 tcohen         my queue is big too
12:59 tcohen         and some are difficult ones
13:00 * liw          cowers in a corner, afraid of big numbers and ashamed of not being able to help
13:01 drojf          i think it would be awesome if some of the people filling the queue with enhancements would also sign off from time to time
13:01 tcohen         drojf: so true :-D
13:01 drojf          not looking at anyone in particular, kivilahtio :P
13:01 liw            at work I've instituted that code reviews are at #2 priority, new code is #3 (security is #1)
13:02 kivilahtio     drojf: I know
13:02 kivilahtio     drojf: we are working on it :)
13:02 drojf          woohoo :)
13:03 drojf          i miss oleonard and the easy to test html/css-patches
13:03 * tcohen       misses oleonard for several reasons
13:03 kivilahtio     drojf: putti just started here doing sign offs for you
13:04 kivilahtio     drojf: too bad the bug descriptions are contradictionary within themselves it is sometimes hard to grasp those things
13:04 kivilahtio     but we are working on it :)
13:05 drojf          i have the feeling lately that everything i test in koha reveals another 3 things that don't work
13:05 kivilahtio     drojf: pretty much
13:05 kivilahtio     drojf: I wonder hoe our librarians can take it
13:05 kivilahtio     luckily we have top priority for regression tests atm
13:05 kivilahtio     drojf: building some cool ****
13:10 drojf          hm. there is still a lot of reference to the national library of the philippines in the kohacon proposal
13:10 drojf          given that they seem to be no longer involved, that's a little odd
13:12 kivilahtio     drojf: i thought they are using koha and want to host the kohacon?
13:14 Kchris         Any one with expericence using 'SYNC_REPO' option on the kohadevbox?
13:14 Kchris         *experience
13:14 drojf          kivilahtio: as far as i understand there are some people that want to host it, contact with the NLP is not really given at the moment and the place is going to be under construction, or something
13:14 kivilahtio     drojf: yes I know of some people who want to host the KohaCon ;)
13:14 drojf          and they are looking for a new place
13:15 drojf          but those people are not of NLP apparently
13:15 drojf          unlike the proposal suggests
13:15 JRJML_Director is it just me or is the koha staff client's search by callnumber a bit beyond difficult?
13:19 JRJML_Director there's no easy way to get a list of fiction items using "callnum:F" because it pulls up everything with an F in the callnumber. I've tried various iterations.
13:19 JRJML_Director of wildcards and advanced searches and quotes, nothing...
13:19 * tcohen       notices theke is not listed as a support provider
13:21 drojf          [off] 5 months to restore a link, wow
13:24 drojf          tcohen: did you send a request?
13:24 drojf          oh i think you have, how would i have known about it otherwise
13:25 tcohen         who's doing it actually?
13:25 drojf          somebody volunteered to help with the paid support list
13:25 drojf          wizzyrea is doing it, not sure who is helping
13:26 drojf          gmcharlt apparently, as he was addressed in the mail on the list
13:27 JRJML_Director submitting a ticket so i can do a search then
13:31 tcohen         drojf: I'll take it as if I don't qualify :-D
13:33 drojf          tcohen: possible :P
13:33 tcohen         [off] i think they have been gentler to others, but I guess it is vacation
13:35 drojf          either that or it got filtered maybe
13:41 drojf          it was davidnind who volunteered to help out maintaining the paid supportlist
13:53 cait           ah the director is gone... i would have had some hints there
14:06 drojf          @wunder berlin, germany
14:06 huginn         drojf: The current temperature in Berlin Tegel, Germany is 33.0°C (3:20 PM CEST on August 11, 2015). Conditions: Clear. Humidity: 34%. Dew Point: 15.0°C. Pressure: 29.95 in 1014 hPa (Steady).
14:06 drojf          i think hitzefrei is ok
14:31 kivilahtio     oh man the serials module is vulnerable to SQL injection attacks :)
14:32 kivilahtio     luckily very slightly vulnerable
14:32 kivilahtio     I thought the serials module was more new and wouldnt have everything crammed up to the controller script :(
14:32 kivilahtio     would somebody sign off if I rewrote the serials module ?
14:33 kivilahtio     ;D
14:41 tcohen         kivilahtio: i don't think so
14:41 kivilahtio     :)
14:41 tcohen         you cannot rewrite the whole Koha codebase without people flying away :-D
14:41 kivilahtio     tcohen: So the better alternative is just to let it stink?
14:42 tcohen         nah, find a way to work on that, that makes people interested
14:42 tcohen         it is mor a social skill than a technical one
14:42 kivilahtio     i am interested in making writing tests as easy as possible
14:43 tcohen         kivilahtio: that's awesome
14:43 kivilahtio     because I really want to do them but I hate doing them in Koha
14:43 tcohen         make people get involved in your work
14:43 tcohen         ask for opinions, explain how you thought about it, personally
14:43 tcohen         1-1
14:43 kivilahtio     okey
14:44 tcohen         i'm interested in some of your developments, but cannot involve right now
14:45 tcohen         once i end this RM cycle I expect to have more time to involve
14:48 kivilahtio     sounds cool!
15:30 ttt            is it possible to generate list of books by department wise. where can department be added in MARC21
15:51 cait           ttt: maybe put it in the item?
15:51 cait           then if 2 departments have the same book it would work better
16:02 huginn         New commit(s) kohagit: Bug 6874: DBRev <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=8fa214bfa5cc0d46a3e34f4c09bb7635132b0b04> / Bug 6874: (RM followup) DBIx updates <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=03ae8f70c23c0ba5e07e6aa6fdb3c65dafec946e> / Bug 6874: (QA followup) upload_path missing in koha-conf.xml.in <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=7fe69d8e5d09cad
17:00 ttt            which tag 9n 952
17:27 DisneyDave     Quick question.  I cannot find anything on the web.  Can a library order books and resources THROUGH koha and have the record show up in the OPAC as "on order" immediately?
18:16 huginn         New commit(s) kohagit: Bug 13485: DBRev <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=62f3fd6a8f935e61af9199d175ece23baad40878> / Bug 13485: Use the Koha template plugin to access syspref values <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=cb1ac9f36a207a51f8292d96fb4ed52dfe4442b9> / Bug 13485: Add a page to display links to restricted sites <http://git.koha-community.org/gitweb/?p=koha.git;a=comm
18:49 tcohen         bye #koha
20:42 rangi          what?
23:18 tcohen         hi rangi
23:19 tcohen         @later tell rangi i'm worried about bug 14517, i haven't found feedback from potential testers, and it badly affects people using transated templates
23:19 huginn         tcohen: The operation succeeded.
23:20 tcohen         is anyone around?
23:25 tcohen         @wunder cordoba, argentina
23:33 * dac          waves to tcohen
23:34 dac            hmm
23:34 dcook          Interesting...
23:34 wahanui        it has been said that interesting is sometimes good and sometimes bad
23:36 Guest1657      And there's NickServ back up
23:36 tcohen         hi dac dcook
23:37 dcook          tcohen: Haven't seen you around in ages!
23:44 tcohen         dcook: it hasn't been easy at night with the baby unfortunately
23:44 tcohen         how have you been?
23:44 tcohen         did my message to rangi hit the IRC?
23:45 tcohen         @wunder cordoba, argentina
23:45 huginn         tcohen: The current temperature in Bo Altos de San Martin - NW, Cordoba city, Cordoba City, Argentina is 11.4°C (8:45 PM ART on August 11, 2015). Conditions: Drizzle. Humidity: 97%. Dew Point: 11.0°C. Pressure: 30.24 in 1024 hPa (Steady).
23:45 dcook          tcohen: I figured that might be the case
23:45 * dcook        hopes his bub sleeps like dcook did as a kid
23:46 dcook          tcohen: sick pretty much all the time lately :/
23:46 dcook          And I think so re: message for rangi
23:47 tcohen         ah, i missed oceania's slang
23:48 tcohen         dcook: i was thinking of you, as i try to refactor some of the search results/records handling code
23:49 dcook          Power to you!
23:49 dcook          I think about it as well sometimes...
23:49 tcohen         but currently focused on the plack patches,
23:49 dcook          That's probably more important tbh
23:49 tcohen         they are working actually
23:49 dcook          \o/
23:49 tcohen         so, only tiding things now
23:49 tcohen         init scripts, etc
23:50 tcohen         ah, I was missing the —enable and —disable switches implementation
23:50 tcohen         already have —start —stop —restart :-D
23:50 tcohen         with a good configuration for apache
23:50 dcook          you're a legend, tcohen :)
23:51 tcohen         yeah, i don't actually exist LOL
23:51 dcook          Hehe. I had a feeling you were going to say that!
23:51 dcook          I was starting to wonder, but I imagine baby eats up a lot of time
23:51 dcook          At the moment, I'm actually trying to recalibrate things in anticipation of baby
23:52 tcohen         yeah, that's a good idea
23:52 tcohen         i'm spending some time in DSpace
23:52 * dcook        won't be able to work 9+ hour days, but thinks this is probably for the best
23:52 dcook          Oh yeah?
23:53 tcohen         yeah, fortunately i've got a intern, who has some skills and i will be able to have him doing most of the work
23:54 dcook          \o/
23:54 * dcook        needs one of those
23:55 * tcohen       shouldn't have spent time in koha-plack tab-completion lol
23:55 tcohen         the baby asks for attention now
23:55 dcook          Okies. Toodles!
23:57 tcohen         is chris hall here?
23:58 * dcook        doesn't see him