IRC log for #koha, 2014-08-26

← Previous day | Today | Next day → | Search | Index

All times shown according to UTC.

Time Nick Message
00:01 dpk left #koha
00:08 wnickc joined #koha
00:17 indradg joined #koha
00:19 Mahdi joined #koha
00:24 dcook Sweet. I didn't know you could pin packages.
00:25 eythian++
00:25 tcohen++
00:31 Mahdi joined #koha
00:36 wnickc @wunder 05048
00:36 huginn wnickc: The current temperature in Lebanon, New Hampshire is 21.0°C (7:53 PM EDT on August 25, 2014). Conditions: Clear. Humidity: 81%. Dew Point: 17.0°C. Pressure: 30.11 in 1020 hPa (Rising).
01:02 Mahdi joined #koha
01:04 Mahdi_ joined #koha
01:06 Mahdi joined #koha
01:36 mtompset Have a great day, #koha dcook eythian rangi
01:36 Mahdi joined #koha
01:39 Mahdi_ joined #koha
01:42 kmlussier joined #koha
01:51 tcohen joined #koha
01:53 tcohen @wunder cordoba, argentina
01:53 huginn tcohen: The current temperature in Cordoba, Argentina is 8.0°C (10:00 PM ART on August 25, 2014). Conditions: Clear. Humidity: 50%. Dew Point: -2.0°C. Windchill: 7.0°C. Pressure: 30.24 in 1024 hPa (Rising).
02:05 indradg @wunder Calcutta
02:05 huginn indradg: Error: No such location could be found.
02:06 indradg @wunder Kolkata
02:06 huginn indradg: The current temperature in Kolkata, India is 30.0°C (7:20 AM IST on August 26, 2014). Conditions: Haze. Humidity: 84%. Dew Point: 27.0°C. Pressure: 29.71 in 1006 hPa (Rising).
02:09 Mahdi joined #koha
02:12 tgoat joined #koha
02:16 tcohen do we have bootstrap3 already?
02:16 dcook I don't think so. I think we're still on 2
02:27 tcohen http://snag.gy/SgGM5.jpg
02:28 dcook That's cool :)
02:29 tcohen I was thinking of vertical tabs actually, so it can grow
02:31 Mahdi joined #koha
02:36 dcook Hmm, that might be a better idea
02:36 Or expandable sections
02:36 Rather than tabs per se
02:38 tcohen i'm not sure what it is
02:38 like the accordion we use ?
02:39 oh, it might be a carrousel!
02:40 eythian http://shouldiuseacarousel.com/ <-- a reference on carousels
02:41 Mahdi joined #koha
02:42 tcohen heh
02:42 we don't need an auto-scrolling tedious carousel
02:50 * dcook isn't a fan of carrousels :p
02:52 Mahdi joined #koha
03:09 tcohen night
03:15 wizzyrea you know, I think new books displays are mighty fine.
03:15 but I am not sold on carousels.
03:20 eythian carousels are boring. You want ferris wheels or those things that shoot you up really fast.
03:24 pianohacker joined #koha
03:36 rocio_afk left #koha
03:53 Mahdi joined #koha
03:57 indradg hi!
03:57 eythian I assume you don't sleep, indradg
03:57 indradg do I need to add a patch's details and test plans in a commit message?
03:57 eythian: affirmative
03:57 eythian indradg: it's preferred
03:58 indradg thanks... was doing that, but thought better to ask, than to be hauled up :D
04:39 Mahdi joined #koha
05:03 cait joined #koha
05:05 dcook hey cait :)
05:06 "<indradg> eythian: affirmative"
05:06 hehe
05:06 indradg hehe
05:32 * cait waves
05:40 Mahdi joined #koha
05:43 barton_away joined #koha
05:54 indradg joined #koha
06:13 * magnuse waves
06:23 cait :)
06:23 bbiab
06:24 cait left #koha
06:38 reiveune joined #koha
06:39 reiveune hello
06:39 wahanui privet, reiveune
06:39 indradg hi magnuse
06:39 hello reiveune
06:40 reiveune \o_
06:41 Mahdi joined #koha
07:15 paul_p joined #koha
07:41 Mahdi joined #koha
07:45 ashimema joined #koha
07:46 indradg joined #koha
07:57 indradg ashimema: ping
07:58 need to about ptfs sandboxes
08:00 magnuse hiya indradg
08:00 indradg hi magnuse
08:00 you might be about to help
08:00 magnuse @wunder boo
08:00 huginn magnuse: The current temperature in Bodo, Norway is 14.0°C (9:50 AM CEST on August 26, 2014). Conditions: Partly Cloudy. Humidity: 72%. Dew Point: 9.0°C. Pressure: 29.86 in 1011 hPa (Rising).
08:01 indradg i'm trying to use a Koha sandbox (first time)... with master and web-installer
08:01 what is the userid/passwd
08:01 test/test doesn
08:01 doesnt seem to do the trick :(
08:05 magnuse i don't know anything beside what is documented here, i'm afraid
08:07 http://wiki.koha-community.org[…]es#Step_4_:_login
08:11 Joubu indradg: you have to setup the sandbox before
08:11 pick one of the 5 available sandboxes
08:12 the " URL to setup the sandbox"
08:12 indradg Joubu: hi... i did send a request for "master" got the email
08:12 Joubu indradg: on which one?
08:12 (sorry didn't read the backlog)
08:13 indradg: the sandbox 1 looks setup
08:13 http://sandbox1-staff.ptfs-europe.co.uk/
08:13 the credentials (test/test) work
08:14 "Sandbox setup by martin.renvoize" but not setup by you...
08:14 indradg hmmm
08:15 Joubu: what I wish to do (first time, so may be I've got it all wrong) - have a box setup to master, apply the patch for bug 12815 and check if it applies
08:15 huginn 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=12815 enhancement, P3, ---, indradg, Needs Signoff , Adding browser based multi-lingual jquery.ime input method support to Koha
08:16 Joubu indradg: if you applied it today (in the last 10 hours), something wrong append.
08:16 indradg: I don't have an access to the ptfs-e sandboxes. Try with a BibLibre sandbox please.
08:17 indradg Joubu: uploaded that patch into bugzilla just about 3 hours back fyi
08:18 * indradg is not even sure if following the sandboxes workflow correctly :(
08:18 Joubu indradg: if you got an email, it should be good
08:19 indradg: but there is only 1 sandbox setup, and it has been setup 20 days ago...
08:28 ashimema Hi indradg.. i'm about now.
08:29 dcook good night all
08:31 ashimema 62 | sandbox  | Indranil Das Gupta | indradg@gmail.com               | 2014-08-26 06:34:05 | apply  |        0 | sandboxes.ptfs-europe.co.uk
08:31 indradg Joubu: I think I was doing it all wrong
08:32 ashimema: yep that was me
08:32 ashimema I can see your sandbox insteadded as master..
08:32 I don't record what db you use... it's the DB that gives you the login etc..
08:33 hmm.. Joubu.. how do you guys handle people walking through the installer?
08:33 indradg ok... clueless question time... I picked up one of Joubu's sandboxes... instead of master, I picked out the bug id this time... and bingo~ the patch applied and works
08:33 ashimema possibly, I've got somthing wrong in the config of db passwords here.?
08:34 Joubu ashimema: the updatedb is executed by sandbox script
08:34 ashimema that's right indradg,, sandboxes are always based on master + the bug you ask for..
08:34 Joubu ashimema: when the email is sent, the updatedb should be executed
08:34 indradg ashimema then what is the purpose of using the "master" option ?
08:34 checks for installer related scripts?
08:35 Joubu indradg: if you choose "master", you won't test any patch
08:35 ashimema basically so have somthing to test against..
08:35 i.e.  a before bug, and after bug case.
08:35 Joubu: of course it is..
08:35 indradg let me get this, the master is essentially the "control" in terms of a science experiment?
08:35 ashimema silly me
08:36 exactly :)
08:36 * indradg has a lightbulb moment!
08:36 indradg :D
08:37 ashimema :)
08:37 indradg ashimema++ / Joubu++ :D
08:42 Mahdi joined #koha
08:52 cait joined #koha
08:54 indradg_ joined #koha
09:19 ashimema any dbic pros about?
09:39 drojf joined #koha
09:43 Mahdi joined #koha
09:48 * cait waves
09:51 drojf hi cait :)
10:05 magnuse ashimema: ask, don't ask to ask ;-)
10:06 * magnuse waves to cait and the shadow of drojf
10:15 goes searching for lunch
10:17 ashimema sorted
10:18 Seems my row object weren't unique enough to be ->update'able...
10:18 had to combine the search and update into one action
10:20 Joubu ashimema: you don't have a pk on the table?
10:21 ashimema primary key is set to the combination of two columns..
10:21 it's a linking table.
10:21 and.. the primary key isn't proply set at database level.. only in dbic schema..
10:21 (it's a nasty old project, that I'm bringing up to date)
10:22 but yeah.. I've got there in the end.
10:22 i was doing search.. and update on separate lines.. with a loop on the search results.. but doing the search and update in the same statement dbic 'gets it right'
10:23 now I just need to sort out my ->copy issue and I'll be happy..
10:24 the copy issue is just plain bad db design rather than dbic though.. we've got a qhole bunch of dangling data with no parent in the database.. so the object to copy just plain doesn't exist sometimes :(.. thinking that wrapping in a transaction is the best way around that..
10:30 indradg joined #koha
10:38 indradg wizzyrea: around?
10:38 hello #koha
10:38 @seen rangi
10:39 huginn indradg: rangi was last seen in #koha 10 hours, 57 minutes, and 11 seconds ago: <rangi> pin
10:39 indradg @seen wizzyrea
10:39 huginn indradg: wizzyrea was last seen in #koha 7 hours, 23 minutes, and 11 seconds ago: <wizzyrea> but I am not sold on carousels.
10:39 indradg @seen ashimema
10:39 huginn indradg: ashimema was last seen in #koha 15 minutes and 24 seconds ago: <ashimema> the copy issue is just plain bad db design rather than dbic though.. we've got a qhole bunch of dangling data with no parent in the database.. so the object to copy just plain doesn't exist sometimes :(.. thinking that wrapping in a transaction is the best way around that..
10:43 Mahdi joined #koha
10:47 indradg ashimema: around?
10:56 kmlussier joined #koha
11:03 Mahdi joined #koha
11:05 ashimema am now..
11:42 drojf joined #koha
11:49 collum joined #koha
12:01 meliss joined #koha
12:02 oleonard joined #koha
12:04 oleonard Hi #koha
12:10 Did we ever agree on a time for the next Koha general meeting?
12:10 "#agreed Next meeting is going to take place on September 17th - time to be agreed on on the mailing list"
12:15 Mahdi joined #koha
12:21 tgoat joined #koha
12:24 tcohen joined #koha
12:24 tcohen mornin
12:24 ashimema he tcohen
12:29 cait oleonard: we also forgot about the dev meeting it looks like
12:29 oleonard: i see your comment on the overdue bug - do they show in red for you?
12:38 oleonard They do not. And the markup doesn't seem to have a class to tell it to be red.
12:39 I've checked on two different test installations running master
12:39 cait oleonard: agreed
12:39 i am not sure what kyle does to make it work :)
12:39 khall khall khall...
12:39 khall mornin!
12:40 cait hah! :)
12:40 morning
12:40 khall yeah, I heard it that time ; )
12:40 cait :)
12:40 NateC joined #koha
12:40 cait oleonard also fails to see red overdues :(
12:41 khall weird, maybe it's because I'm using chrome? Let me check
12:42 oleonard khall: Not unless there is a class at work here which cait and I don't recognize as the one which should be making it look overdue
12:43 cait oleonard: i think the date comparison in the checkout js should set it - but i couldn't see what was going wrong there
12:44 nengard joined #koha
12:45 khall confirmed, works in Chrome, not in Firefox. Should have a class of "overdue"
12:46 no js errors either
12:47 oleonard: the code that should do it is in js/checkouts.js starting at line 151
12:47 does firefox not support Date objects?
12:49 cait oleonard: trying to get runreport.pl to send an emali now next - cross fingers? :)
12:49 to see if i can reproduce the problem you had
12:56 drojf oleonard: i noticed that video embedded in the opac looks funny with the bootstrap theme when used on a small screen. in which css file would i put something like video { width: 100%; max-width: 480px;}? opac.css?
12:58 khall oleonard: looks like FF is more picky about date formats than chrome: http://stackoverflow.com/quest[…]e-but-not-firefox
12:58 oleonard drojf: OPACUserCSS
12:58 wahanui OPACUserCSS is an existing system preference in Koha
13:00 drojf oleonard: not sure should be changed locally as it looks funny for everyone ;)
13:01 tcohen hi cait khall oleonard
13:01 oleonard drojf: If you think you have CSS which will make embedded video work for everyone you should modify opac.less and submit a patch for that and a separate one with the recompiled opac.css
13:02 khall mornin tcohen!
13:02 drojf oleonard: thanks
13:03 talljoy joined #koha
13:04 * tcohen is generating dbic schemas after applying 10212
13:05 cait does someone know what to do when you can't schedule a report?
13:05 i have "Failed to add scheuled task" in the gui
13:06 and in the logs:  you do not have permission to use atq
13:06 my /etc/at.deny file is empty
13:09 oleonard khall: Any idea why some checkouts show a time in the "due date" column and some don't?
13:09 Joubu tcohen: looking at your followup patch (10212), cait asked me to show/display the panel the user is modifying
13:09 tcohen i'm not sure I understand
13:10 khall oleonard: is it not because of changes to Koha's date formatting that drop the time when it's 23:59?
13:10 Joubu tcohen: you are modiying the panel/tab 'acquisition', you save. The page displays the acquisition tab
13:10 tcohen ah
13:10 oleonard khall: I guess so, I was not aware of that.
13:10 Dyrcona joined #koha
13:10 cait i really need help to get the scheduler working - all suggestions welcome
13:11 could it be i am missing something in my koha_conf file? i am testing with bug 12031
13:11 huginn 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=12031 normal, P5 - low, ---, liz, Passed QA , Task Scheduler not sending mail
13:11 tcohen Joubu: let me think a bit, I understand what is wrong, thou
13:11 Joubu tcohen: I think the "panel" variable is only set if a save has been done, not when you come for the first time. So maybe a [% if panel %][% panel %][% else %] false [% end %] will do the trick
13:11 tcohen exactly
13:11 i'll fix it right now
13:13 cait ColinC++
13:13 Joubu tcohen: not that all occurrences of "panel" have been introduced for this behavior.
13:16 * tcohen is aware
13:18 cait ok, got a step further... www-data was in at.deny...
13:21 nice_coworker++
13:23 Mahdi joined #koha
13:24 paul_p joined #koha
13:28 cait ... and received the scheduled report by email :)
13:30 akafred joined #koha
13:31 chris_n http://tinyurl.com/l87jw5e
13:31 cait oleonard-away: i can reproduce your problem with the replyto adn return-path address for runreport
13:33 huginn New commit(s) kohagit: Bug 12808: remove useless diags from t/db_dependent/Context.t <http://git.koha-community.org/[…]3bb0b50446e0e80b2> / Bug 12222: members-update.tt has a giant hash variable <http://git.koha-community.org/[…]ab77c70a58e8cbc2a> / Bug 10212: DBRev 3.17.00.015 <http://git.koha-community.org/[…]git;a=commitdiff;
13:34 tcohen Joubu++
13:36 Joubu Thanks tcohen!
13:37 I will try to add the table configuration to acquisition tables on rewriting the gst/vat code
13:37 rocio joined #koha
13:37 tcohen did u get feedback on that?
13:37 drojf joined #koha
13:38 Joubu tcohen: no
13:38 tcohen hmpf
13:38 Joubu tcohen: francharb asked me to add something to the rfc.
13:39 and some feedbacks on the french ML
13:39 So yes, I got feedbacks :)
13:39 tcohen did u get feedback from nz and us users?
13:40 i'm worried about different legislations being covered
13:41 cait tcohen: worried too, bti sad not more feedback
13:41 Joubu:could you try to send an email to the koha list again maybe?
13:42 tcohen @later tell rangi we need nz feedback on GST/VAT use cases
13:42 huginn tcohen: The operation succeeded.
13:42 cait i stil have my print out to read through too :(
13:42 tcohen @later tell bag we need US feedback on GST/VAT use cases
13:42 huginn tcohen: The operation succeeded.
13:42 akafred koha-create --request-db mydbname gives "awk: fatal: cannot open file `/etc/mysql/koha-common.cnf' for reading (No such file or directory) " Anyone have any idea why? Do I have to install mysql-server even if i am not going to use to run it on the koha-server?
13:43 tcohen akafred: its a design problem koha-create has
13:43 cait Joubu: i got feedback from a german library
13:43 tcohen you need to use --use-db instead
13:43 cait I'd like to add that to the report
13:43 they send it today - about what is required here
13:43 tcohen and have /etc/koha/passwd instead
13:43 cait maybe i can get to it tomorrow or the day after
13:44 tcohen akafred: put instance:db_user:db_pass:db_name
13:44 on the file
13:46 akafred ok. I'll try. Is it documented somewhere (e g like: http://wiki.koha-community.org[…]rate_MySQL_server ) in case I need more instructions?
13:46 tcohen create /etc/koha/passwd
13:46 and use --use-db (without any other parameter)
13:46 akafred And it will populate?
13:47 tcohen nope
13:47 unless...
13:47 wahanui unless is probably meant to give code that reads more like natural language? but i'm not a fan either
13:47 tcohen you use --default-sql file.gz
13:47 :=D
13:47 akafred Ok... is there a bug report on the design issue?
13:48 tcohen there isn't
13:48 Viktor joined #koha
13:48 Joubu cait: ok great
13:49 akafred tcohen: Thanks ... I'll have to investigate more tomorrow...
13:49 tcohen it works for us, we use that daily
13:50 no worries
13:52 carmenh joined #koha
14:25 jcamins Joubu++ # apparently some people weren't around when I gave you karma
14:34 cait rocio++ # like your blog post!
14:35 rocio thanks cait!
14:35 cait jcamins: so the question is... does the karma still count if noone sees it? :)
14:35 jcamins cait: I don't know, that's why I gave Joubu more.
14:35 cait :)
14:37 reiveune bye
14:37 reiveune left #koha
14:38 Joubu :)
14:55 cait Joubu: working on bug 12717 now :)
14:55 huginn 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=12717 critical, P5 - low, ---, jonathan.druart, Signed Off , Library no longer receiving Overdue email for patrons without email address
14:55 cait thx! Joubu++
14:58 Joubu cait: If you have a clear vision on how fix the current inconsistencies when sending print notices, let me now.
14:58 cait the solution sounds good - but i think we need to work on the gather_* next
14:58 Joubu cait: I think all print notices should use the same script, it's why I add new parameter to this script (don't remember the bz number)
14:59 cait Joubu: the problem is that you will have to add a lot of entries to the cron for that
14:59 i think maybe it needs an option for a combined email by library or something like that
14:59 not sure yt
14:59 Joubu cait: ok, but at least 2 patches (modifying this script) NSO
14:59 cait and the file should not be named holdnotices :)
14:59 yep
14:59 Joubu cait: will be easy to fix
15:00 cait i think so too, but I think we need to put some thought into how it will be easy to use
15:00 we didn't use it so far, so not sure yet
15:01 Joubu cait: I don't know how people/librarians use these script, so not sure to understand all workflows
15:01 cait i thik until the mtt change sit was only used for hold_print
15:02 Joubu yep, but I meant print notices in general
15:02 cait and there was an additional printoverdues.sh
15:02 i wonder what shoudl happen with that now - i can still see it on master
15:03 it crates a pdf from html files... that you generated with overdues.pl i think
15:03 Joubu maybe printoverdues could be removed after the new patches I submit (but not sure...)
15:03 cait I guess we need a good way to announce such changes
15:04 Joubu yes, release notes :p
15:04 cait i'd not be unhappy to see it go, but maybe after some deprecation tme - it's using a python library too
15:05 Joubu: but noone reads them! :P
15:06 Joubu everybody should!
15:06 cait :)
15:07 i think maybe not remove immediately - but urge people to switch to the other script
15:07 as a first step and make sure it  can offer a similar result
15:07 or better :)
15:10 Joubu cait: On a major release, it could be safe to describe a good and new workflow and explain that old way/script are removed
15:10 anyway, I don't a good and new workflow to propose :)
15:11 cait, tcohen(, others?): what could be the best way to "highlight" plumbing patches (like bug 12827 I am going to submit). Generally it's quite easy patches, but a routine prototype will be changed. So the integration time should be short, otherwise regression will be introduced easily.
15:11 huginn 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=12827 enhancement, P5 - low, ---, jonathan.druart, ASSIGNED , NewOrder returns basketno
15:11 cait hm maybe a tag in the title?
15:12 Joubu I think I still asked and the answer was : "set the component to plumbing"
15:12 cait [API] or similar?
15:12 Joubu s/still/already
15:12 cait and set the component to plumbing? :)
15:12 Joubu cait: yes, but not easy to filter on
15:12 cait you could use a keyword - tags seem to be for personal use only
15:12 but not sure how the keywords work
15:13 Joubu: should ther ebe a fallback to another notice text if you haven't defined one? i am testing overdues and have checked email and print for mtt
15:13 Joubu cait: yes, good idea. It could be good to have several keywords defined somewhere
15:14 datadoctor joined #koha
15:14 Joubu cait: Actually, my idea was to fallback using the "print" notice.
15:14 cait ok
15:15 makes sense
15:15 Joubu cait: If the patch, I think the fallback is : sent an email with notice in failure + using the print notice
15:15 pff
15:15 s/If/with
15:15 cait hm
15:15 Joubu it's not what you are expecting for?
15:16 datadoctor we like a notice priority of SMS > Email > Phone notification > Print Notice for cost efficiency
15:16 cait we could work with it - but i think it won't solve the problem of people suddenly getting no email - because they might not have deifned a print notice yte
15:16 i am still testing
15:16 datadoctor We don't have default notices for all transport_types like we used to
15:17 if you don't fill one in, I'm assuming you can't send it?
15:17 cait hm well the option to have different notices for different types is in itself new
15:17 so before you had to use one template for all
15:17 datadoctor I like that feature a lot.
15:17 Joubu datadoctor: if no template is defined for a mtt, the notice won't be sent
15:17 cait with the exception of hold_print
15:18 Joubu (won't be generated)
15:18 datadoctor thanks for the clarification Joubu
15:18 I noticed that is functioning using an unless clause
15:18 in the overdue_notices.pl
15:20 cait Joubu: when is a print notice set to sent?
15:20 datadoctor if there is no email defined, then prepare a print notice
15:21 Joubu cait: when it's sent ? :)
15:21 cait: and the only wait to sent print notices is the gather_print_notices
15:21 s/wait/way
15:22 cait ok
15:22 so we won't interfere with that
15:23 it loosk like it#s working to me
15:24 i think what I'd like is an option for gather_print to outout all holds and all overdues for a branch in one email each
15:24 i think right now you can't do that - because it doesn't accept multiple letter codes?
15:27 francharb hi all
15:27 do you know if I have to reindex zebra if I turn on the querystemming preference?
15:27 Joubu cait: note that bug 11603 removes the "holdnotices-" prefix for files generated by gather_print_notices
15:27 huginn 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11603 enhancement, P5 - low, ---, jonathan.druart, Needs Signoff , Gather print notices: add csv and ods export
15:27 Joubu s/removes/replaces
15:27 cait cool
15:28 Joubu http://bugs.koha-community.org[…]ment.cgi?id=30983
15:28 cait i was hoping it dd
15:28 i hope to test it next
15:28 francharb Joubu, j'ai jamais vu autant de substitions de ma vie
15:28 ;)
15:28 Joubu francharb: sorry...
15:28 francharb s/;)/:)
15:28 ;)
15:28 je rigoooole
15:28 ;)
15:29 Joubu francharb: to push the enter key is compulsive...
15:29 oleonard khall still around?
15:30 francharb Joubu, ahah
15:31 Mahdi joined #koha
15:43 cait left #koha
16:16 indradg joined #koha
16:28 oleonard indradg around?
16:30 cait joined #koha
16:32 wnickc joined #koha
16:36 indradg joined #koha
16:36 Mahdi joined #koha
16:43 cait oleonard++ # thx for suggestion a solution for the transl problem
16:44 oleonard yeah I'm glad i was able to find a way
16:48 cait me too :)
17:06 pastebot "oleonard" at 127.0.0.1 pasted "khall: Why this markup preceding the checkouts table?" (4 lines) at http://paste.koha-community.org/174
17:07 khall oleonard: not sure at this point. it's been too long ; ) I'm guessing either for whitespace, or it's just dead markup that got left in there that got missed during cleanup.
17:09 oleonard khall: I noticed because the whitespace seemed unnecessary.  The "processing" div, since it is absolutely positioned, doesn't need to be placed in a "row" div.
17:10 khall yeah, that markup should be removed
17:23 Mahdi joined #koha
17:38 huginn New commit(s) kohagit: Bug 10402: (QA followup) amend unit tests <http://git.koha-community.org/[…]d1fdb41983d316346> / 10402: DBIx::Class schema update <http://git.koha-community.org/[…]65af64931fc85ef85> / 10402: DBRev 3.17.00.016 <http://git.koha-community.org/[…]401aca9d674222c55> /
17:41 jcamins Yay!
17:41 tcohen++
17:42 tcohen :-D
17:54 indradg joined #koha
17:57 * cait waves
17:58 tcohen hi cait
18:00 indradg hi all
18:00 hi tcohen
18:00 tcohen hi indradg
18:02 indradg I have a question: I tried out my patch for bug 12815 on one of biblibre's sandboxes. it applied and the patch seemed to behave as per test plan. afterwards, I got into my ~/kohaclone and ran `prove t` it came out FAILED
18:02 huginn 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=12815 enhancement, P3, ---, indradg, Failed QA , Adding browser based multi-lingual jquery.ime input method support to Koha
18:03 indradg http://pastebin.com/aM3Rr7AA that's the `prove t` output
18:03 can someone clue me in what i'm missing?
18:03 wahanui well, missing is usualy library only, an item that's not out
18:03 jcamins That's expected.
18:03 You added a database update, right?
18:03 indradg jcamins: yes
18:04 jcamins One of the tests checks that all database updates have been numbered.
18:04 However, only the RM can number a database update.
18:04 tcohen indradg: you can safely ignore *that* error
18:04 indradg yes... that was the first error... i think
18:05 jcamins indradg: that's the only error, too.
18:11 indradg ok! I see oleonard has tested out the patch. it seems to run for him as too, but he has marked it 'Failed QA' with certain inputs and guidance.
18:12 how do I processed next: address the issue - like adding the jQuery.IME license info into about.tt and mark it back in 'In Discussion'
18:13 jcamins, tcohen what is the expected protocol here?
18:13 cait mark it back needs sign-off i think
18:13 indradg cait: thanks :)
18:13 oleonard indradg: http://bugs.koha-community.org[…]ment.cgi?id=29887
18:13 indradg: For example
18:13 indradg oleonard: checking
18:13 yes I got that that part
18:16 oleonard: what i did not get was your point #1 in http://bugs.koha-community.org[…]g.cgi?id=12815#c4
18:16 huginn 04Bug 12815: enhancement, P3, ---, indradg, In Discussion , Adding browser based multi-lingual jquery.ime input method support to Koha
18:16 indradg can you please explain, so that I can avoid this
18:16 oleonard Simple: Take out those comments
18:17 "+<!-- Bug 12815 - Adding browser based multi-lingual jquery.ime input method support to Koha -->"
18:17 huginn 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=12815 enhancement, P3, ---, indradg, In Discussion , Adding browser based multi-lingual jquery.ime input method support to Koha
18:18 oleonard indradg: We use git. Git can track when any particular line of code was modified. We don't need comments in the markup which track changes.
18:19 indradg oleonard: you mean this - http://pastebin.com/JutLyzD9
18:20 oleonard No, I mean "+<!-- Bug 12815 - Adding browser based multi-lingual jquery.ime input method support to Koha -->"
18:21 * indradg been using git only for like a week. trying to get up to speed with the 'jargon', but obviously not there yet :-(
18:21 oleonard indradg: Just take out those HTML comments.
18:23 indradg oleonard: blimey! i got it! that was just for my own tracking. found out the hard way that in the opac, calling in the lib was not going to work from doc-close-head.inc, rather from opac-bottom.inc
18:23 will remove those bogus artifacts
18:23 Mahdi joined #koha
18:24 tcohen indradg: you can use template::toolkit comments if you consider them important
18:25 -> [%# This is a comment ]
18:25 oleonard But not for adding bug information!
18:28 cait hm for some time those showed up in the translations
18:28 not sure if that got fixed
18:28 i hope it did
18:29 indradg tcohen: copy that!
18:30 tcohen cait: there are some improvements to tmpl_process3.pl on my pipe
18:30 cait it be in already - i haven't checked
18:30 maybe osmething i need to check next time i see a comment in qa :)
18:44 Viktor joined #koha
18:48 huginn New commit(s) kohagit: Bug 12325 - Marc and Card view in Z39.50 Results doesn't work properly in IE 8 <http://git.koha-community.org/[…]1a4366df29cdba833> / Bug 12176 - Remove HTML from additem.pl <http://git.koha-community.org/[…]34d8dd3e03e56ed24> / Bug 12717: Library no longer receiving Overdue email for patrons without email address <ht
18:52 tcohen bye #koha
18:54 mtompset joined #koha
18:54 mtompset Greetins, #koha.
18:55 I'm trying to build the ultimate test set, but I need to determine the columns of a table, and if they are string, number, or date. Anyone give ideas on a good way to do that?
19:00 oleonard Bye #koha
19:00 huginn New commit(s) kohagit: Bug 12626: SQLHelper replacement - C4::Acquisition <http://git.koha-community.org/[…]fba8d2cfb7efce069>
19:01 bag @weather 93102
19:01 huginn bag: The current temperature in Westside (Gillespie), Santa Barbara, California is 25.8°C (12:01 PM PDT on August 26, 2014). Conditions: Clear. Humidity: 53%. Dew Point: 16.0°C. Pressure: 29.99 in 1016 hPa (Steady). High surf advisory in effect from 4 PM this afternoon to 3 am PDT Thursday...
19:18 mtompset NICE! Found a DBIx way to do it nicely.
19:23 kmlussier joined #koha
19:24 Mahdi joined #koha
19:32 indradg joined #koha
20:12 Mahdi joined #koha
20:22 tcohen joined #koha
20:22 tcohen @wunder cordoba, argentina
20:22 huginn tcohen: The current temperature in Cordoba, Argentina is 16.0°C (5:00 PM ART on August 26, 2014). Conditions: Clear. Humidity: 15%. Dew Point: -11.0°C. Pressure: 30.21 in 1023 hPa (Steady).
20:25 nengard left #koha
20:27 wnickc joined #koha
20:47 eternalsword patron notification emails, can the delay be negative to say how many days before? It doesn't make sense to remind patrons that their book is due after the fact.
20:50 talljoy left #koha
20:51 eternalsword or I guess it's advance notice of item due. where is the default number of days prior defined?
20:52 cait i think you are looking at overdues
20:53 eternalsword okay looks like it's a cronjob argument
20:53 cait adcance notices required that the extendedmessagingpreferences are turned on
20:53 and then you can configure a standard per category
20:53 and change in the patron account individually
20:53 wahanui cait: that doesn't look right
20:53 wahanui joined #koha
20:54 cait eternalsword: are you looking at overdues.pl or at advance_notices.pl?
20:54 maybe you can do something with arguments, but i have never had to :)
21:10 indradg Hi
21:11 I noticed something, that out of 522 .tt files under intranet-tmpl/prog/en/modules, only 15 includes [% USE Koha %] in their declaration. In contrast, out of 72 .tt files under opac-tmpl/bootstrap/en/modules, 49 files uses the declaration.
21:11 Any specific reason for this low ratio among the intranet templates?
21:12 cait bootstrap was redone after the tt plugin got available :)
21:12 well it was done after.. i think
21:12 and oleonard used it everywhere it hink
21:12 indradg hmmm
21:12 cait it reflects that the bootstrap templates are newer and less files than the staff interface, where changes get reflected slower
21:13 Mahdi joined #koha
21:13 cait indradg: we are still in a kind of transition phase from htp to tt, because we don't use the nice new options everywhere yet
21:13 htp being the old template system
21:13 indradg cait: just wanted to be sure that there were no design decisions I might be missing out on
21:14 cait i think it's mostly a historic thing
21:14 mtompset indradg: You could always fix that with a mega patch. ;)
21:14 cait please don't :) heh
21:14 indradg coz, oleonard comment against bug 12815 that I should prolly not use a separate .inc for the jquery.ime
21:14 huginn 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=12815 enhancement, P3, ---, indradg, In Discussion , Adding browser based multi-lingual jquery.ime input method support to Koha
21:15 indradg cait ^^^ and I can't do that as cleanly as in opac bootstrap templates
21:15 hence the question
21:16 for now I can plug a [% USE Koha %] inside the doc-head-close.inc, but that looks ugly IMHO
21:16 cait hm not sure what's best there
21:17 indradg jcamins gave me a friendly warning shot about not going around messing across a lot of templates in a single shot :)
21:17 mtompset Wow that is a LONG patch file.
21:17 I felt my blood sugar drop scrolling through that.
21:18 indradg mtompset: thats mostly due to the jquery.ime lib inclusion
21:18 mtompset Yes. :)
21:18 * indradg was swallowing hard at the 1.1 mb size of it :P
21:18 mtompset Perhaps you might want to split your patch to have the jquery.im lib inclusion stuff as a separate patch.
21:19 So that your specific Koha changes are in another commit.
21:21 indradg mtompset: doable! how about the leaving out the [% USE Koha %] out of 12815, and adding a fresh patch and *adding* in this second patch as the "Depends on" for 12815?
21:21 cait what do you suggest ^^
21:21 cait i think too tired for good suggestions :)
21:22 kinda late here and about to go to sleep
21:22 jcamins indradg: if you are saying you'd like to make 12815 OPAC-only, fine. If you're saying you'd like to add a bunch of files but no functionality... no.
21:22 Just add two patches on bug 12815.
21:22 huginn 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=12815 enhancement, P3, ---, indradg, In Discussion , Adding browser based multi-lingual jquery.ime input method support to Koha
21:22 mtompset I'm with jcamins on that.
21:22 cait opac/staff sounds good to me
21:23 jcamins Right, I'd prefer it be both OPAC and staff client, but that's indradg's decision.
21:24 cait ok, confused, i should have gone to bed :)
21:24 will now - good night all :)
21:24 cait left #koha
21:25 Mahdi joined #koha
21:25 indradg jcamins: hmmmm... how about a fresh patch that adds [% USE Koha %] to koha-tmpl templates; then break up this 12815 patch into separate OPAC and staff client patches, with the staff client patch being made "Depends on" the [% USE Koha %] patch
21:26 mtompset tcohen: You will fear the day you told me to look more in depth into the tests. ;)
21:26 indradg and obsolete this current patch
21:26 tcohen mtompset: ?
21:26 mtompset I'm adding 700+ test cases to NewChannels, just to run through every possible data case! ;)
21:26 tcohen omfg
21:27 isn't it possible to generalize a bit? :-P
21:27 jcamins indradg: "Depends on" is only relevant for bugs.
21:28 The fact that there is no [% USE Koha %] declaration is not a bug.
21:28 At least, IMO.
21:28 tcohen: may disagree.
21:28 s/://
21:28 So what I would prefer is three patches on bug 12815. One that adds the [% USE Koha %], one for the OPAC, and one for the staff client.
21:28 huginn 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=12815 enhancement, P3, ---, indradg, In Discussion , Adding browser based multi-lingual jquery.ime input method support to Koha
21:29 jcamins Maybe even 5 patches, splitting the added libs into their own patches, too.
21:29 1.1MB is *big*.
21:29 For a patch.
21:29 Small for a feature film.
21:29 indradg heh
21:29 tcohen heh
21:30 mtompset tcohen: I did generalize. I only use two branch codes, 3 numbers (undef, 0, 1), 3 dates (undef, 0000-..., 3999-...), the some possible values for lang (undef, '', 'koha', 'slip', 'en'), ... it gets large fast when you cover all the possible general cases for each field. ;)
21:30 jcamins mtompset: how many of the tests are handwritten?
21:31 (versus pattern tests in a loop, for example)
21:31 mtompset I wanted to build the tests automatically, but that was too hard. :)
21:31 well, it isn't 700 tests... it's 700 data points.
21:32 Just wanted to freak tcohen out a bit. ;)
21:33 indradg jcamins: in that scenario, the actual feature enabled patch for OPAC and staff client's test plan should mention as #1 'install lib add patch' and in case of staff client - 'install lib patch followed by "USE Koha" patch'
21:33 ?
21:33 jcamins No.
21:33 Step one of the test plan is "apply patches."
21:33 It is your responsibility to upload them to bugzilla in the correct order.
21:35 mtompset -- sometimes the test plan has steps to demonstrate the problem before the "apply patches" step.
21:36 indradg jcamins: so *one* bug filing number followed by a) lib add patch upload ; then USE Koha patch upload and finally the OPAC and staff client patches... all uploaded against bug 12815?
21:36 huginn 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=12815 enhancement, P3, ---, indradg, In Discussion , Adding browser based multi-lingual jquery.ime input method support to Koha
21:36 jcamins Yes.
21:36 indradg got it!
21:36 jcamins indradg: there's no useful way to test the first two patches.
21:36 indradg i was thinking about that
21:37 jcamins Sure, you could add a test plan "add a call to the Koha TT plugin in every single template," but no one is going to test that.
21:37 Ever.
21:37 Likewise, you could add a test plan "try to download jquery.ime plugin using your web browser," but it's hard to see why anyone would want to do that.
21:37 indradg makes sense
21:39 tcohen indradg: always focus on lowering the barrier for potential testers
21:40 indradg tcohen: aye!
21:40 * indradg takes the patch back to chop shop!
22:07 eythian hi
22:14 wizzyrea whaddup
22:20 papa joined #koha
22:21 indradg hi... can anyone throw somelight on this - http://wiki.koha-community.org[…]s#Translatability
22:22 any examples?
22:22 eythian there's examples in koha
22:26 Mahdi joined #koha
22:33 indradg eythian: in /intranet-tmpl/prog/en/modules/a​dmin/preferences/i18n_l10n.pref it is given as http://pastebin.com/n0kSHX6u
22:34 but shouldn't it be as http://pastebin.com/xkLXtcCa
22:34 eythian those two pastes are exactly identical, I think
22:34 but I don't actually know what your question is
22:35 indradg eythian: no there are not exactly same
22:35 my question is w.r.t. - Single quote (') and double quotes (") must not be used. Use glyphs: « » ‘ ’ “ ”
22:35 ^^^
22:35 from the wiki link above
22:37 eythian oh, the quote
22:37 I missed that
22:37 it's probably an error then
22:37 indradg so no need to worry on that?
22:37 eythian or it's the case that it sometimes works, sometimes doesn't
22:37 I don't know
22:37 I mean, it is probably the .pref that's in error, the wiki is most likely right
22:38 indradg who might? fredericd ?
22:38 eythian who?
22:38 err
22:38 who what?
22:41 indradg http://wiki.koha-community.org[…]f=11425&oldid=843
22:42 eythian: that specific line in the wiki was added by Frédéric Demians aka fredericd
22:43 eythian yes, he's probably correct
22:44 indradg @seen fredericd
22:44 huginn indradg: fredericd was last seen in #koha 26 weeks, 0 days, 7 hours, 43 minutes, and 22 seconds ago: <fredericd> #info Frédéric Demians, Tamil
23:02 mtompset indradg: EEEEEW! Smart quotes.
23:03 tcohen joined #koha
23:03 indradg mtompset: good for health? :P
23:03 mtompset No, for shorter life. Because you waste yours debugging them. :P
23:06 indradg lol, so can ignore that "must" for now? :D
23:06 eythian well, it might cause problems sometimes, or it might not.
23:13 Mahdi joined #koha
23:14 indradg sounds like Schrödinger's cat :P
23:16 tcohen eythian: --enable-sru vs. --enable-z3950 ?
23:17 eythian tcohen: they're different things, aren
23:17 't they?
23:18 tcohen doesnt' zebra speak both?
23:19 eythian I think it does, but assuming they're different protocols that are configured differently, what does the speaking doesn't have much of an influence on the option naming.
23:26 Topic for #koha is now Koha 3.16.3 is available! Next general meeting is 13 and 14 August 2014 at 22:00 and 15:00 UTC. Welcome to the IRC home of Koha http://koha-community.org. Please use http://paste.koha-community.org for pastes.
23:28 gmcharlt http://koha-community.org/koha-3-16-3-released/
23:28 tcohen wb!
23:28 gmcharlt++

← Previous day | Today | Next day → | Search | Index

koha1