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 3.21.00.017 <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 3.21.00.018 <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