Time Nick Message
21:33 cait ok...so they can fix it
21:32 CrispyBRAINS Patron CAN cancel a hold while in transit.
21:31 cait bye Joubu
21:30 Joubu bye #koha!
21:26 CrispyBRAINS Let's see.
21:25 CrispyBRAINS Not sure.
21:24 CrispyBRAINS Currently, I do not think so.
21:24 cait i mean... you send it back and then hte patron realizes... could you still cancel it from the patron account?
21:24 CrispyBRAINS It isn't in transit when it is waiting.
21:23 CrispyBRAINS There is no way to know if the patron picked it up and dropped it off without checking it out, or if a staff member accidently put it in a courier by mistake. So, unless libraries call the patron it is waiting for to find out if they already had it, it seems the logical thing to do is send it to the waiting library.
21:23 cait can we cancel when it's in transit?
21:22 CrispyBRAINS My gut says the only option should be to send it back to the library it is supposed to be waiting at.
21:20 cait we don't have a library doing transport, so a bit out of depth
21:20 cait but if we always offer ignore too... maybe we should
21:20 cait i think sending it
21:20 cait they placed on the item
21:20 CrispyBRAINS You wouldn't. Best case scenario is to put it back where it got off track.
21:20 cait the sytem can't know if it wasn't another hold
21:19 cait how would you know about all this story?
21:19 CrispyBRAINS Yes
21:19 cait so the borrower the hold was for returns it
21:18 CrispyBRAINS When the item is checked in, should the holding library have the ability to ignore the transfer back to the waiting library, or cancel the hold? Or should the item just return to where it is supposed to be?
21:17 CrispyBRAINS I have a fix, but need some input.
21:17 CrispyBRAINS Currently, koha doesn't know what to do with it.
21:16 CrispyBRAINS Right now this is broken. I am working on a fix. An item is at a library on hold, and marked as waiting. A patron picks it up and forgets to check it out. They finish with it and return it to a different library.
21:16 cait how does it end up waiting at the wrong branch?
21:15 cait you lost me
21:15 CrispyBRAINS If a hold is showing as waiting, but at the wrong branch, on check in, should the holding branch be given the option to cancel the hold or ignore the transfer, or should it only set the transfer in motion back to the library it should be waiting at?
21:07 Joubu great, sounds exciting! :)
21:02 rangi ill have to try to switch on more loggin
21:02 rangi nope it just ticks along, then those 2 lines and dies
21:01 Joubu it crashes yes
21:01 Joubu well, one of them
21:01 cait does it die?
21:01 cait what happens? i feel like i missed the beginning of the story
21:01 Joubu do you have the GET request that produces it?
20:59 rangi nope unfortunately
20:59 Joubu nothing else in the logs? Like a line in the splitter code rather than an external lib :)
20:57 rangi its gotta be data related because its quite random
20:57 rangi that is busting it when someone clicks on the page its on
20:57 rangi email
20:57 rangi i think there is an address, or something
20:56 rangi yep
20:56 cait i was wondering too :)
20:55 Joubu are you talking about splitter?
20:55 Joubu hi rangi
20:47 rangi ashimema: re the history, thats not my dept, you will want to ask wizzyrea
20:46 rangi ashimema: i've restarted it, but if you want to track down what is causing it, that would be good
20:45 rangi it crashes at unpredictable times, with this error
20:45 rangi Bad arg length for Socket::sockaddr_family, length is 0, should be at least 2 at /usr/share/perl5/HTTP/Server/Simple.pm line 422.
20:45 rangi Use of uninitialized value in subroutine entry at /usr/share/perl5/HTTP/Server/Simple.pm line 422.
20:45 rangi you can fix if you like :)
20:45 rangi ashimema: there is a bug in it
20:38 * CrispyBRAINS waves
20:34 alexbuckley hi all
20:32 cait josef_moravec++ :)
20:22 cait i saw, thx a lot!
20:00 josef_moravec1 cait: I tried to QA on some ILL patches
20:00 josef_moravec1 hi cait
18:56 ashimema by BOO
18:56 BOOleonard Bye all
18:55 huginn ashimema: The operation succeeded.
18:55 ashimema @later tell rangi also, splitter seems to be down.. oleonard was trying to use it this morning.. I'm happy to help maintain some of this stuff if it would help.. but don't want to step on toes
18:54 Joubu yes, I wanted them to ask on the Spanish list. Sounds like it will be easier for them
18:53 caroline they're nicer without the language barrier :)
18:52 Joubu heh, at least we know where they come from now!
18:52 huginn ashimema: The operation succeeded.
18:52 ashimema @later tell rangi any chance you could let me know how the website history page is kept up to date.. looks like it could do with some utf8 magic applying to it.
18:51 caroline Joubu++ for great reference interview :D
18:49 MiguelV12 Bendiciones.
18:49 MiguelV12 Gracias Joubu me pondre a leer.
18:47 Joubu supongo que existe en castillano tambien, pero no sé donde
18:47 MiguelV12 Excelente :)
18:46 Joubu MiguelV12: hay todos los campos 6xx en este link ^
18:45 caroline full list : http://www.loc.gov/marc/bibliographic/bd6xx.html
18:44 Joubu that was the question
18:44 caroline not sure if that answers the question
18:44 * caroline feels like she's on Jeopardy
18:43 caroline person names in 600, common names in 650
18:43 caroline subjects are in 6xx
18:43 MiguelV12 Si, gracias
18:43 Joubu MiguelV12: preguntes en la lista yahoo :)
18:42 Joubu no son los campos 6xx?
18:42 CREEPYlynn me too ashimema
18:42 MiguelV12 Esta bien!!, jeje gracias.
18:42 Joubu jaja no sé, soy informatico, no bibliotecario!
18:41 * ashimema now wants an irc client with Google translate built in
18:40 MiguelV12 que campos de marc es recomendable
18:40 Joubu Si, es possible con marc
18:39 MiguelV12 Muchas Gracias
18:39 josef_moravec1 evening #koha
18:39 Joubu http://groups.yahoo.com/group/koha-es/
18:39 MiguelV12 es posible con marc21
18:38 MiguelV12 necesito relacionar los titulos a asignaturas aca se le dicen materias, por ejemplo tengo el titulo Filosofia en calidad relacionar ese titulo a las materias Derecho1, Derecho2
18:38 Joubu Quizas que seria mas facil para tu?
18:38 Joubu aca la gente habla ingles, pero si quieres hay una lista en espanol
18:37 MiguelV12 estoy trabajando en Koha
18:37 MiguelV12 Si :)
18:36 Joubu Hablas castillano entonces?
18:34 MiguelV12 EL SALVADOR
18:33 Joubu Where do you come from? De donde eres?
18:31 Joubu How are you doing?
18:31 wahanui bidet, MiguelV12
18:31 MiguelV12 hi
18:31 Joubu Hi MiguelV12
18:30 wahanui i already had it that way, MiguelV12.
18:30 MiguelV12 relate titles to different subjects is possible, thank you
18:29 wahanui i already had it that way, Guest1891.
18:29 Guest1891 relate titles to different subjects is possible, thank you
18:27 Miguel relate titles to different subjects is possible, thank you
18:09 saa if so what measures need to be taken to prevent this happening again
18:09 saa do we will have to reinstall koha
18:09 saa we realised all our koha pages are linked to xmr.omine.org it does not open any page all koha intranet pages are linked to this site how to sort out this issue
16:46 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 normal, P5 - low, ---, andrew.isherwood, Passed QA , Fix API response time on ILL request endpoint
16:46 josef_moravec thanks for quick fix of bug 20996
16:45 josef_moravec AndrewIsh++
16:24 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21717 normal, P5 - low, ---, jonathan.druart, Signed Off , TestBuilder.t is failing randomly
16:24 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21725 normal, P5 - low, ---, nick, Signed Off , Incorrect HAVING in group by in Acquisitions.pm
16:24 Joubu QA needed on bug 21725 and bug 21717 to make Jenkins happy
16:24 reiveune bye
16:21 Joubu C4::Biblio::LinkBibHeadingsToAuthorities around line 587
16:21 caroline1 ugh...
16:20 Joubu $field->delete_subfield( code => '9' );
16:19 caroline1 not in the sense of "link removed"
16:18 caroline1 like, unlinked in the sense of "not linked"
16:18 caroline1 Joubu: does the code say what "unlinked" mean? After I asked the question I thought maybe it was just saying that it couldn't find the authorities for those, not that it actually unlinked them
16:17 Joubu caroline1: weird, after a very quick look at the code I'd say you should not have unlinked if LinkerKeepStale is set
16:14 caroline1 -_-
16:14 caroline1 I'm doing this on my test koha, but I know the client will not be happy if there are unlinked headings
16:13 caroline1 While writing my question, I thought that maybe it's just telling me there are 566 headings for which no authority was found...?
16:12 cait tcohen: there is an omnibus bug i think
16:12 cait maybe talljoy
16:12 cait sorry, no experience :(
16:11 caroline1 *unlink
16:11 caroline1 Did the script unlik them?
16:11 caroline1 I set "LinkerKeepStale" to Keep, but the report when the script finishes says I have 566 headings unlinked
16:10 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17258 major, P5 - low, ---, gmcharlt, NEW , [OMNIBUS] MySQL 5.7
16:10 Joubu blockers for bug 17258
16:09 tcohen Joubu: there isn't a BZ keyword for strict_sql?
16:09 Joubu true wahanui, that's what I was thinking too
16:09 wahanui the question is, like, a bit general
16:09 Joubu what's the question?
16:08 caroline1 Anybody here understand link_bibs_to_authorities.pl ?
16:06 BOOleonard Sorry the case confused me
16:06 CREEPYlynn yeah, I know, BOOleonard
16:06 BOOleonard We're talking about Nov. 1
16:05 CREEPYlynn tho, it was my dad's birthday :)
16:05 CREEPYlynn not a holiday in the US
15:52 Freddy_Enrique Criollo's music day :)
15:52 Freddy_Enrique here... halloween
15:44 caroline not a holiday here and I don't think it's a holiday in the US either
15:44 BOOleonard No holiday here
15:44 caroline makes sense
15:43 caroline ok
15:43 cait All Saints?
15:43 cait in german it's allerheiligen
15:42 caroline cait: what's the holiday?
15:40 cait i will be for a bit in the morning
15:39 ashimema or rather.. will be here
15:39 ashimema I'm here
15:39 cait anyone not on holiday tomorrow?
15:38 cait you too ashimema
15:38 ashimema have a good evening #koha
15:38 * ashimema suddenly realises he needs to go fetch youngest from child minders
15:36 cait but there is lots of scary things today... like all the emails and the to do list
15:36 cait got scared there for amoment :)
15:29 ashimema hehe
15:29 kidclamp While! A while
15:28 ashimema hehe
15:28 ashimema kidclamp going awol for a week!?
15:28 ashimema what!
15:28 kidclamp bbiaw
15:08 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15395 enhancement, P5 - low, ---, julian.maurice, Passed QA , Internationalization: plural forms, context, and more
15:08 tuxayo Thanks for QAing bug 15395 ❤️❤️❤️❤️
15:08 tuxayo Joubu++
15:03 tuxayo :D
15:02 magnuse \o/
15:01 huginn News from kohagit: Bug 18720: Use exception instead of die in GetBasketAsCsv <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=77ea4686395f88c9678c611a78e6db4fd4c04101>
15:01 huginn News from kohagit: Bug 20941: Add Illrequest->getType method <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=cc12509c1e61c0648b57fc39b3428bb2169e303c>
15:01 huginn News from kohagit: Bug 10442: Remove references to non-standard "error" class <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=52028d9932ce1d10ad690261d8f6983b074b2909>
15:01 huginn News from kohagit: Bug 14716: URI-encode URLs in NORMARC xsl templates. <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=a140843721de52477c6460ccca0c1bf0d0aafcfd>
15:01 huginn News from kohagit: Bug 20941: Add unit tests <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=676fbe12ea52e2d3552b7d5300ba278de4ac3437>
15:01 huginn News from kohagit: Bug 20941: (follow-up) Carry out comment #4 advice <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=7cecc27e0152e310a7286ddd9bab96597b91891e>
15:01 huginn News from kohagit: Bug 20941: (QA follow-up) Fix return value of get_type, make templates more consistant <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=aced7b10de6180da18e1cce86d5d020bd9dd4813>
15:01 huginn News from kohagit: Bug 20941: (follow-up) Wrap N/A strings in <span> <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=f7ec3b5ab661cfeb3bdb2a1600ac85baf7c422b7>
15:01 huginn News from kohagit: Bug 20941: (follow-up) Update N/A handling <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=efc92a5ea2fc0aed90cc93dae1a0024c886fd80a>
15:01 huginn News from kohagit: Bug 20941: (follow-up) Switch 'media' -> 'types' <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=74f2a9032475dcfd18141655bd2ef608c4c65801>
15:01 huginn News from kohagit: Bug 21681: Remove C4::Accounts::getcharges <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=655caa61e64af3c1fdb12a3a01f28f8d07752609>
15:01 huginn News from kohagit: Bug 14716: Fix more unencoded url parameters. <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=bffd0d8f11fd68e7706e57c571c9b29d15d800ea>
15:01 huginn News from kohagit: Bug 21694: Add the Koha::Account->lines method <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=00d5ceac1e56edc4ce893c53b0ea6763e9fca5e2>
15:01 huginn News from kohagit: Bug 21694: Keep the context <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=8193d3592d815066943cba5807ef6d80c0f7d366>
15:01 huginn News from kohagit: Bug 21681: Improve the account lines fetch <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=8f6bd9415096924e2648a1a0a5866733f6839760>
15:01 huginn News from kohagit: Bug 21681: Add tests for GetPatronInfo <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=62613b6cf2861c28862ec9a6d55767eff63d2322>
15:01 huginn News from kohagit: Bug 21698: Fix POD of cancel_unfilled_holds.pl <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=b7b6727f92615bcebf2f037202e6647a34c7705a>
15:01 huginn News from kohagit: Bug 21698: (QA follow-up) Fix typo verbosev -> verbose <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=04114d4b835c10bb03b9c092268fc632d5552fed>
15:01 huginn News from kohagit: Bug 21080: Fix patron's attributes display when PA_CLASS is used <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=559e051bb9eecbc92f63aa2941caea6ee373fe0e>
15:01 huginn News from kohagit: Bug 10442: Compiled CSS <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=9f65d3422ee2fe7df556e422b9d1ed3cddc5be38>
14:47 huginn AndrewIsh: The operation succeeded.
14:47 AndrewIsh @later tell josef_moravec I had to obsolete your patches on 20996 as they weren't applying, failing with "error: sha1 information is lacking or useless (Koha/Illrequest.pm)". I un-obsoleted their predecessors and am currently working on rebasing, then I'll attach the rebased patches, including your addition to the POD
14:42 lavamind magnuse: thanks :)
14:41 CrispyBRAINS Thanks BOO. ::)
14:41 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15788 enhancement, P5 - low, ---, koha-bugs, NEW , Split borrowers permission into create/edit and delete
14:41 BOOleonard Bug 15788
14:40 BOOleonard CrispyBRAINS: Doesn't look like it, but that'd be a good idea for a new permission
14:40 magnuse lavamind: too early for dcook, i think, better to leave comments on the bug
14:39 lavamind the fix is to add this in _before_dispatch : https://paste.debian.net/1049851/
14:39 lavamind there was a problem with generating PDF invoices
14:38 lavamind I found another issue with the Mojolicous patch
14:38 lavamind dcook: hello
14:38 CrispyBRAINS Is there a way to prevent staff from deleting patron accounts?
14:33 AndrewIsh Thanks very much :)
14:32 ashimema Show Obsolete -> details -> edit details -> uncheck 'obsolete'
14:32 AndrewIsh Riiiiiight, makes sense ashimema. Should I do it in that order? I'd have thought I'd want to obsolete josef's first wouldn't I?
14:31 ashimema does that help AndrewIsh (IRC)
14:31 ashimema then apply, rebase and re-upload over the top
14:31 ashimema you need to un-obsolete the old patches on bugzilla.. obsolete the ones that replaced them
14:30 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 normal, P5 - low, ---, andrew.isherwood, Signed Off , Fix API response time on ILL request endpoint
14:30 AndrewIsh I'm still having problems with the rebase that josef_moravec did earlier on bug 20996, it won't apply and fails with "error: sha1 information is lacking or useless (Koha/Illrequest.pm)" I'd like to remove his rebased patches and restore to how it was before he did it, so I can rebase myself, anyone got any idea how I can do that?
14:11 andreashm well that was a longshot anyway. gotta go. let me know if anyone has an idea of how this can happen!
14:06 andreashm khall: around? I have an strange accountline behavior for you if you are. ^^ =)
14:04 cait Joubu: thx for adding the sponsored line :)
14:03 ashimema pass
14:03 andreashm Our fines tab is a mess because of this
14:03 andreashm And those that do, when they are checked in the refund is drawn from the total amount (choosing the oldest lines in accountlines for that patron), not the actual lost book.
14:01 andreashm anyone know?
14:01 andreashm How can lost books that are not checked in have accounttype LR?
14:01 Joubu it's not too general IMO, it's for bib framework. We have one for auth framework
14:00 Joubu cait: nope
13:57 cait but it's too general for a real bug... do we have an omnibus?
13:57 cait Joubu: we could mark ti wish list
13:55 Freddy_Enrique caroline _!
13:53 caroline Hi Freddy_Enrique!
13:53 Freddy_Enrique Greetings everyone!
13:52 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21540 normal, P5 - low, ---, koha-bugs, RESOLVED INVALID, Edit item/record pages don't show certain translations from po files
13:52 Joubu cait: I would consider bug 21540 a bug. But mark it WISHLIST or WONTFIX...
13:43 ashimema if it works it's fine with me
13:42 ashimema feel free.. I wasn't aware of prior art
13:42 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21682 major, P5 - low, ---, martin.renvoize, Signed Off , Stock Rotation: Update DB is failing with strict_sql_modes ON
13:42 Joubu ashimema: do you agree with my suggestion on bug 21682? It will avoid the new subroutine
13:40 Archie^ nope just marc21 records about every book
13:40 BOOleonard Archie^: Do you know if the file includes item information?
13:39 Archie^ and i want these all books available only for that library
13:39 BOOleonard Archie^: bibliographic records aren't associated with a specific library. Items are.
13:39 Archie^ i mean, i have that library in koha added
13:38 Archie^ no i mean, if a certain library sent me their marc21xml file and i want to upload it into koha for that one specific library
13:37 caroline Archie^: what do you mean "for a specific library"? You have more than one catalogue?
13:36 Archie^ a question: if im uploading books into koha via Stage MARC records for import does it upload the books just into a catalogue or can i upload for a specific library
13:34 Archie^ hi Koha
13:31 magnuse CREEPYlynn: yay!
13:27 * BOOleonard too
13:26 * CREEPYlynn is jellies
13:26 CREEPYlynn my son is going to norway in April
13:25 * BOOleonard has put a BOO sticker on himself and hopes to be delivered there with the luggage
13:25 magnuse BOOleonard: did you know? BOO is the three letter airport code for Bodø
13:23 Joubu Hi scary #koha
13:23 * ashimema has briefly switched to patch rescue mode as a break from fail qa mode
13:22 ashimema gotta love a bit of bash
13:21 * CREEPYlynn is bash scripting like a boss today
13:21 BOOleonard CREEPYlynn++
13:20 ashimema okies
13:20 CREEPYlynn wouldn't want BOOleonard to be the only one with a costume on
13:20 caroline ashimema: It's on my list, but it'll have to wait until my friday afternoon signoff/docs spree... clients are all over me right now
13:19 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21387 minor, P5 - low, ---, martin.renvoize, Needs Signoff , Receive items from - form should include tax hints the same as the ordering form
13:19 * ashimema hints to caroline that bug 21387 is ready for testing again :)
13:18 caroline guten morgen everyone!
13:16 calire BOOleonard++
13:16 tcohen hola ashimema
13:16 tcohen hi magnuse :-D
13:15 magnuse morning tcohen
13:14 ashimema morning tcohen
13:14 tcohen morning
13:04 BOOleonard Spoooky!!
13:01 * ashimema feels like a zombie
13:01 * ashimema needs lunch
12:57 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8426 enhancement, P5 - low, ---, kyle.m.hall, Needs Signoff , Diacritics not being mapped for searching (Non-ICU)
12:57 kidclamp bug 8426
12:38 AndrewIsh ashimema: Indeed, thanks, I'll try that
12:37 ashimema obviosely replacging the paths to .patch
12:37 ashimema that sometimes works
12:37 ashimema `patch -p1 < example.patch`
12:36 ashimema you could try applying manually
12:36 AndrewIsh Oh, Josef has logged off, poop
12:36 cait josef_moravec++
12:36 AndrewIsh Ah, interesting. Josef has added a patch since I first tried to apply it, but the patch looks fine
12:36 ashimema but in this case.. you're the author right ;)
12:36 ashimema so they need to rebase and repush the patches
12:36 ashimema that tends to need fixing by the original patch author.. it usually means there's history missing
12:36 BOOleonard But perhaps you'd be interested in my new product idea: http://zivotdesign.com/p/#s/15409892513019
12:35 * cait waves
12:35 BOOleonard I usually find that this is an error in the patches rather than my repo
12:35 AndrewIsh It's not on a branch, I'm doing it on master
12:35 AndrewIsh ashimema: It is
12:35 ashimema do you still have the original branch?
12:35 ashimema is this a `git bz apply` ?
12:34 AndrewIsh Does anyone know how to sort out when you've messed up an apply and can then no longer apply that bug? I've tried "git reset --hard origin/master". I'm getting "error: sha1 information is lacking or useless (Koha/Illrequest.pm)." I've seen this before, I just can't remember how to fix it!
12:34 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21241 enhancement, P5 - low, ---, alexbuckley, Signed Off , Set suggestion notices message_transport_type to sms if patrons has no email but smsalertnumber set
12:34 ashimema bug 21241 doesn't feel right to me.. feels to 'one off' ish
12:33 ashimema anyone know if there's prior art to having a fallback sequence for message_transport_types ?
12:28 ashimema that's one heck of a dependancy graph
12:27 ashimema yowsers.. good luck
12:27 ashimema hehe
12:27 ashimema dang
12:27 * ashimema looks at the dashboard stats and realises how much of a machine Katrin is!
12:26 BOOleonard ashimema: I would suggest one of mine but I'm too busy applying dependencies for Bug 18589 :P
12:26 ashimema hehe
12:26 ashimema 'nice easy' and that one don't fall inline ;)
12:26 * andreashm hides
12:25 andreashm ashimema: sorry, couldn't resist! =)
12:25 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10662 new feature, P3, ---, dcook, Failed QA , Build OAI-PMH Harvesting Client
12:25 andreashm bug 10662
12:24 * ashimema is tired of failing qa.. anyone got a nice easy one I can pass ;)
12:22 BOOleonard 👍
12:22 AndrewIsh Ah, it's fixed in 18598 :)
12:22 BOOleonard master
12:22 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18589 enhancement, P5 - low, ---, andrew.isherwood, Signed Off , Show ILLs as part of patron profile
12:22 AndrewIsh BOOleonard: Aha! Is this with bug 18589 applied, or are you talking about master?
12:20 BOOleonard ashimema: A qa failing spree is still a qa spree, so ++
12:20 AndrewIsh BOOleonard: I'll just do a bit of digging first
12:20 BOOleonard AndrewIsh: No rush. Should I file a bug?
12:20 AndrewIsh BOOleonard: Still looking at your query :)
12:19 AndrewIsh josef_moravec: OK, I'll take a look
12:18 * ashimema doesn't like that he's on a qa failing spree :(
12:16 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18591 enhancement, P5 - low, ---, magnus, Pushed to Master , Allow an arbitrary number of comments on ILLs
12:16 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 normal, P5 - low, ---, andrew.isherwood, Signed Off , Fix API response time on ILL request endpoint
12:16 josef_moravec AndrewIsh: could you add follow-up to bug 20996? Bug 18591 touched the Koha::Illrequest->TO_JSON method by adding embed comments
12:16 huginn BOOleonard: The operation succeeded.
12:16 BOOleonard @later tell rangi Could you try to bring back splitter from the dead so that it can walk the earth as a shambling abomination hunting the flesh of the living? (it's still Halloween here)
12:14 AndrewIsh BOOleonard: Ah, interesting. Hang on...
12:11 BOOleonard For me it doesn't do that... Shows all requests.
12:08 AndrewIsh BOOleonard: It needs the borrowernumber to be able to retrieve just that patron's ILL requests
12:05 AndrewIsh BOOleonard: I'm not sure, I'll check...
12:05 BOOleonard The link to ILL from the patron menu includes borrowernumber, but does ill-requests.pl do anything with it?
12:02 AndrewIsh Ah, thanks BOOleonard - that's good to know :)
12:01 AndrewIsh I was a bit scared of asking BOOleonard - he's looking a bit scary with that Halloween costume on
12:01 BOOleonard AndrewIsh: I don't remember off the top of my head but I do know it's not less than 9.
12:00 magnuse BOOleonard probably knows about the minimum IE?
11:58 ashimema but I don't know if that's deliberate or not
11:58 AndrewIsh Anyone know what the minimum IE version supported by the OPAC is? It's not less than 9 is it?
11:58 ashimema and he's not clearly accounted for all cases where LostItem is called
11:57 magnuse yup
11:57 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20844 enhancement, P5 - low, ---, alexbuckley, Signed Off , Reset a hold when it is missing after allocation
11:57 ashimema so.. bug 20844 adds additional functions to C4::Circulation::LostItem
11:57 ashimema it's really easy to forget to add a LostItem call after the actual marking an item as lost
11:56 ashimema well.. I think if changing the state of items.itemlost should always trigger everything in LostItem
11:55 * ashimema adds a bug
11:55 ashimema makes qa'ing hard and error prone
11:55 ashimema that should be handle much more seamlessly
11:55 magnuse you think LostItem should also set items.itemlost?
11:55 ashimema mm
11:54 * ashimema isn't at all impressed by that
11:54 magnuse yeah, because of WhenLostChargeReplacementFee etc
11:54 ashimema else you won't execute a whole bunch of related functions
11:54 ashimema but.. from what I can see you have to always call C4::Circulation::LostItem after doing that update
11:54 * magnuse needs food
11:53 ashimema mmm
11:53 BOOleonard Yes
11:52 magnuse items.itemlost=0 => not lost, items.itemlost>0 => lost, methinks
11:50 ashimema mwah
11:50 ashimema LostItem is something you have to call after marking an item as lost
11:50 * magnuse does an impression of "The scream"
11:50 ashimema ok.. it's just really odd design
11:48 ashimema lol
11:48 BOOleonard Get it?? Scary, right?
11:42 wahanui hi oleopard
11:42 ashimema hi oleonard
11:42 oleonard Hi all
11:39 ashimema am i losing the plot already today?
11:39 ashimema C4::Circulation::LostItem doesn't actually appear to change the item in any way.. i thought it was the routine used to mark an item as lost
11:39 ashimema what does a 'Lost item' look like at the db level?
11:38 ashimema OK.. I'm totally confused
11:18 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14957 enhancement, P5 - low, ---, eivin, Failed QA , Write protecting MARC fields based on source of import
11:18 magnuse getting bug 14957 in would be awesome
11:12 huginn ashimema: The operation succeeded.
11:12 ashimema @later tell cait could you comment on comment 3 on bug 21732
10:05 AndrewIsh josef_moravec: :)
10:05 AndrewIsh josef_moravec: Indeed. Some will be quite tricky, but I'm happy to help when required!
10:05 josef_moravec I have to go, but return later to continue on QA ILL bugs
10:04 josef_moravec AndrewIsh: thanks, this one needed to rebase a bit, but conflicts are easy ones, so i made it myself, but on latter bug it would not be as easy I think...
10:01 AndrewIsh josef_moravec There will almost certainly be some rebasing required as you work through these, so just give me a shout if you need me to rebase anything :)
09:59 AndrewIsh josef_moravec++
09:57 ashimema josef_moravec++
09:55 josef_moravec AndrewIsh++
09:54 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20941 normal, P5 - low, ---, andrew.isherwood, Passed QA , Displaying requests does not display request material type
09:54 josef_moravec bug 20941
09:54 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21615 minor, P5 - low, ---, oleonard, NEW , "Stock rotation" is at the wrong place in the Tools left side menu
09:54 ashimema bug 21615 would be a great one to start with.. both oleonard and I havenot managed to get our kids onto it yet.. but feel free to steal it if you want to have a play :)
09:53 andreashm ashimema: good! it helps indeed!
09:53 andreashm magnuse: sounds about right. göteborg has a few too. We have one. That's about it. =)
09:52 * magnuse learned a little bit of perl in library school
09:52 * ashimema tries to mark up Academy bugs all year round now in the hopes that it's usefull to other fresh devs
09:52 andreashm =)
09:52 magnuse what he said
09:52 ashimema just start small andreashm ;)
09:51 magnuse i think Lund said they have 6 perl people? they have been contributing to the catmandu project, i think
09:51 * andreashm should learn... just not enough hours in the day
09:49 andreashm a few at the big uni:s
09:49 andreashm not so many here either
09:49 josef_moravec ;(
09:49 magnuse the more the merrier!
09:49 josef_moravec We tried to find some perl develpers here to, but with no luck ;)
09:49 magnuse yup
09:49 andreashm ashimema: yes indeed
09:49 andreashm josef_moravec: yup, that one!
09:48 ashimema it was the swedish hackers that cait visisted recently wasn't it?
09:48 josef_moravec magnuse: it would be great, anybody is always welcome, right? ;)
09:48 * magnuse hopes perl hackers at swedish unis will find the time to get more involved in community work
09:48 josef_moravec andreashm: do you mean 10662? The yes ;)
09:47 andreashm josef_moravec has looked at it before right? =)
09:47 ashimema hopeing someone else will get to it before me as it's not an area I'm really up to speed on.. but if they don't.. it's also on my long list
09:47 magnuse that would be awesome!
09:46 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10662 new feature, P3, ---, dcook, Failed QA , Build OAI-PMH Harvesting Client
09:46 magnuse bug 10662
09:46 * ashimema hasn't looked at OAI enhancements yet
09:46 andreashm *holds thumbs and sends cookies around*
09:46 * andreashm hopes to see both ILL and OAI-PMH harvesting for 18.11
09:45 tuxayo I remember about IE 11 being the lowest denominator for the OPAC. What about the staff interface?
09:45 tuxayo Hi, are there written somewhere info about browser support? At least informally.
09:45 magnuse the best to date!
09:45 andreashm sweeeet
09:44 * ashimema thinks 18.11 is gonna be a mahoosive release
09:44 magnuse josef_moravec++
09:44 AndrewIsh :)
09:43 josef_moravec great
09:43 ashimema AndrewIsh is ready and waiting to jump on QA comments :)
09:43 ashimema awesome
09:43 josef_moravec I love the ILL module and we have big plans on using it here in Czech Republic ;)
09:42 ashimema I owe you :)
09:42 ashimema awesome josef :)
09:42 ashimema :)
09:42 josef_moravec then I am going to try to QA through all the ILL queue hopefully
09:42 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20941 normal, P5 - low, ---, andrew.isherwood, Signed Off , Displaying requests does not display request material type
09:42 josef_moravec ashimema: I just continued on QA for bug 20941
09:41 ashimema it's all dried up :(
09:41 ashimema we're sitting here waiting for QAers to start looking again
09:41 ashimema we've hit a QA roadblock for ILL magnuse
09:40 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20600 enhancement, P5 - low, ---, andrew.isherwood, Signed Off , Provide the ability for users to filter ILL requests
09:40 * magnuse keeps his fingers crossed for bug 20600 too
09:28 AndrewIsh No prob :)
09:28 magnuse AndrewIsh++ thank you for signing off on it!
09:28 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21585 minor, P5 - low, ---, koha-bugs, Signed Off , Ignore missing firstname in ILL requests table
09:28 AndrewIsh Thanks for the clarification on bug 21585 magnuse - now signed off :)
09:22 ashimema cait is in meetings all morning right?
09:10 magnuse tjänare andreashm
09:08 andreashm hi magnuse
09:08 magnuse \o/
08:58 * andreashm waves as cait runs by
08:57 cait hi all *runs off to meeting*
08:45 alexbuckley I am going to finish for the night now seeya tomorrow everyone :)
08:28 calire hi andreashm
08:24 andreashm and hi cait
08:24 andreashm hi calire
08:24 andreashm ... ended up with a cinnamon bun instead.
08:22 * andreashm goes to get coffee and a danish
08:22 alexbuckley hehe :)
08:22 andreashm peer pressure! =)
08:22 dcook alexbuckley should go offline too :p
08:22 josef_moravec nice evening dcook
08:22 dcook later!
08:21 alexbuckley bye dcook
08:21 andreashm See ya dcook
08:21 dcook Great way to end a day
08:21 dcook Always nice to chat to you folk ^_^
08:21 josef_moravec ;)
08:21 dcook But on that note, I better head off
08:21 josef_moravec dcook: I understand, so no problem ;)
08:21 dcook Really want to play with all the AI technologies. Keep wondering if I could build a little AI program to help me learn Chinese :p
08:20 dcook Fortunately, I've started studying Chinese 1-2 nights a week so I can't stay too late 1-2 nights a week now heh
08:20 dcook Sorry for the delay. It's been so busy since getting back from North America.
08:20 * andreashm will bribe any potential qa person with cookies!
08:19 josef_moravec dcook++ great job!
08:19 dcook ^_^
08:19 andreashm sounds great
08:19 andreashm dcook: haha, yup
08:19 dcook Just need to do the Koha::Object stuff I think and it shoudl be good
08:19 dcook Woo 10662 passing qa test tools (with a few ignorable warnings)
08:18 ashimema cheers josef_moravec
08:17 dcook Those cheeky developers
08:17 dcook &x-withDeletedData=true
08:17 dcook cheers andreashm :)
08:16 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21002 enhancement, P5 - low, ---, tomascohen, NEW , Add Koha::Account::add_debit
08:16 josef_moravec ashimema: I added few quick comment on bug 21002
08:15 andreashm =)
08:15 ashimema Third time's a charm
08:15 ashimema bug 10662
08:15 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10662 new feature, P3, ---, dcook, Failed QA , Build OAI-PMH Harvesting Client
08:15 ashimema <_oftc_dcook "andreashm: Do you have a URL tha"> bug 10662
08:15 andreashm https://libris.kb.se/api/oaipmh/?verb=ListRecords&metadataPrefix=marcxml_includehold&from=2018-09-10T18:00:00Z&until=2018-09-11T12:00:00Z&x-withDeletedData=true&set=bib:Krh
08:14 * dcook is running the qa test tools just one more time and then it's home time
08:14 dcook I suppose I could just test it on a Koha site..
08:14 dcook andreashm: Do you have a URL that I could use to test the harvester on for marcxml records?
08:13 dcook And look at finishing this off tomorrow morning
08:13 dcook I think I'll head home pretty soon now
08:13 andreashm dcook++
08:13 andreashm oh, super-sweet!
08:13 andreashm I guess I've become your signal to go home. =)
08:13 dcook Nearly through them all
08:13 dcook I've been spending today fixing QA issues for 10662
08:13 dcook Soo late
08:12 andreashm dcook: are you here late again? =)
08:12 dcook Now I've definitely stayed too late ;)
08:12 * dcook waves to andreashm
08:11 andreashm salut!
08:08 dcook Maybe just fix a few more filters..
08:08 * dcook now debates whether or not to go home..
08:06 magnuse ashimema++ josef_moravec++
08:06 dcook josef_moravec++
08:06 dcook ashimema++
08:06 dcook Yuss. datetimepicker back in action
08:06 magnuse dcook++
08:05 * dcook is a persistent bastard
08:05 dcook That could do it..
08:04 dcook Ahh they're supposed to go down below too...
08:03 dcook I've seen this before...
08:03 dcook Maybe something that someone else has already fixed?
08:03 dcook Yikes yeah the problem isn't my javascript it seems..
08:01 dcook It's loading but it is all not happy
08:01 dcook It looks like conflicts between different Javascript libraries or something..
08:01 dcook cheers ashimema. Got all that :)
08:01 * dcook sighs
08:00 ashimema good luck dcook
08:00 * ashimema has to go get breakfast for the kids now
08:00 ashimema all aobve the `[% INCLUDE 'doc-head-open.inc' %]`
07:59 magnuse \o/
07:59 ashimema I needed that full set to get it all to work
07:59 * ashimema sent a long message: < https://matrix.org/_matrix/media/v1/download/matrix.org/JOzAoRLvZXghhSqwZmGNWFaY >
07:58 dcook Oh my... but lots of console errors
07:58 dcook the Asset.js is rendering it in the HTML..
07:58 dcook But not the Asset bit..
07:58 ashimema it was gonna be one of those two.. hehe
07:58 dcook It helped with the jsinclude block
07:58 dcook Or half did it
07:58 dcook Damn it..
07:58 ashimema :)
07:58 dcook SET footerjs = 1 did the trick
07:57 ashimema I was missing it and it meant assets weren't loading at all
07:57 dcook Ahh
07:57 ashimema that was the bug I had
07:57 dcook Already got that one :)
07:56 ashimema I 'think' dcook
07:56 ashimema `[% use Asset %]`
07:55 dcook JS12 says though..
07:54 josef_moravec yes
07:54 * dcook is still skeptical
07:54 dcook SET footerjs = 1
07:54 josef_moravec there is param that says the js is in bottom
07:54 dcook koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc:[% PROCESS jsinclude %]
07:54 dcook vagrant@kohadevbox:kohaclone(bug10662)$ grep -R "PROCESS jsinclude" koha-tmpl/*
07:54 dcook I assume
07:54 dcook That's because it only works on the OPAC
07:54 ashimema let me dig through my notes
07:54 dcook Ahhh
07:53 ashimema oh, that's the exact issue I had
07:53 dcook ashimema: The jsinclude block isn't being included :/
07:52 ashimema there's allot of QA still to do this cycle.. some really nice feature that just need pushing over the edge.. but time is running out :(
07:52 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15516 new feature, P5 - low, ---, julian.maurice, Signed Off , Allow to place a hold on first available item from a group of titles
07:52 * ashimema switches back to bug 15516
07:51 ashimema been there too dcook.. I couldn't for the life of me work out why none of my JS was even loading!
07:51 dcook heh cheers ashimema
07:51 dcook Oh wait there it is..
07:51 wahanui bonjour, alex_a
07:51 alex_a bonjour
07:51 dcook wtf... none of the javascript is working..
07:51 ashimema dcook++
07:50 ashimema love you really dcok
07:50 ashimema dcook.. GO HOME ;)
07:50 dcook It's just 6:50pm (my day ends at 6pm)
07:50 alexbuckley that's ok going to keep preserving with it
07:50 dcook I feel like I'm nearly there
07:50 ashimema stock rotation was a right pain to fix that
07:50 ashimema yeah.. it's a bit challenging to rebase stuff with that move dcook.. I've been there already too
07:50 dcook ^
07:49 ashimema does feels like lots of companies are doing recalls already in different ways.. be nice to get a good solid core feature in to build upon
07:49 dcook Hmm moving the javascript to the bottom of the page has definitely caused some issues..
07:49 ashimema sorry I hadn't got to it earlier in this cycle alexbuckley .. I wasn't even really aware of it untill a few days ago :(
07:48 ashimema you need to check that it's got `| uri` or `| url` correct instead of `| html`
07:48 dcook Yeah my hrefs are the ones that I have left for last
07:48 dcook ashimema: I wish I'd had you around maybe 40 minutes ago heh
07:48 ashimema (currently the main fringe case to be aware of is hrefs)
07:47 alexbuckley Your feedback is really useful for fixing it up for the start of the next release cycle
07:47 ashimema then see what it did and correct anything that looks wrong
07:47 ashimema these days I'd tend to run /misc/devel/add_missing_filters.pl against the files missing filters
07:46 dcook thanks ashimema :)
07:46 ashimema dcook
07:46 wahanui hola, reiveune
07:46 reiveune hello
07:46 ashimema I really wanted to be more positive.. not sure that came accross
07:46 dcook Yikes that's not on the wiki..
07:46 ashimema sorry it's not all good news alexbuckly
07:46 ashimema oh.. your still here
07:46 ashimema `[% USE raw %]`
07:45 alexbuckley hi ashimema, thanks for your feedback on recalls
07:45 dcook Looks like I need to load the raw plugin too..
07:45 * dcook is fixing up 10662
07:45 dcook This one I did by hand :/
07:44 ashimema there's certainly still some issues with the qa script.add filters tool
07:44 dcook I've done custom filters before but it has been a while ago
07:44 dcook How does one load the $raw filter?
07:44 * ashimema can't remember the specifics I'm afraid.. still getting to grips with filters
07:44 dcook That's definitely the line causing the problem
07:44 dcook I didn't see your message heh
07:43 dcook Oh nice one ashimema
07:43 dcook [% Asset.js("lib/jquery/plugins/jquery-ui-timepicker-addon.min.js") | $raw %]
07:43 dcook Huh...
07:43 ashimema we had that chatter yesterday morning after you had left dcook
07:43 ashimema I tihnk
07:43 ashimema that tends to mean you're using an INC that contains a $raw filter.. but you've not imported the raw plugin yet
07:38 dcook Iiiiinteresting
07:38 dcook Template process failed: undef error - : filter not found at /home/vagrant/kohaclone/C4/Templates.pm line 122
07:25 ashimema as much as I'd have loved it in 18.11 I don't think it's ready
07:24 ashimema I have a number of things in that pile already though..
07:24 calire takes some pressure off today
07:24 calire ashimema, sure
07:24 ashimema be nice to regroup next month after release and get a plan together I think
07:24 ashimema I think it's gonna fail simply on code issues calire
07:24 calire ashimema I'm pretty swamped this morning (last day before leave) but feel free to shout if you want me to look at something
07:23 dcook Or rather the default choice?
07:23 dcook Btw, for HTML9... is the safest bet just to use the html filter?
07:23 dcook :(
07:23 ashimema I have a whole page of qa notes and I'm still going :(
07:23 * ashimema is back on the recalls bug this morning.
07:22 ashimema lol
07:22 ashimema Hi josef
07:22 dcook Oh man it's 6:22pm but I'm really craving the breakfast sandwich I had at 10am...
07:22 josef_moravec hi ashimema
07:22 dcook Ignoring them so far heh
07:22 josef_moravec dcook: but yes it would be better to have it ignored in qa script
07:21 josef_moravec dcook: this should be ignored
07:21 ashimema you can fairly safely ignore red qa warnings on the schema files
07:21 dcook yo ashimema
07:21 ashimema morning #koha
07:21 dcook I think maybe another bug in the QA scripts?
07:20 dcook I still get some red for missing pod in Koha/Schema/Result/* files although I'm not sure why as it's there and autogenerated...
07:19 cait bbl
07:19 cait calire is here... hsould be a clue to get ready for work :)
07:14 calire hi dcook
07:12 josef_moravec yes, i like it too ;)
07:12 dcook hey calire
07:12 dcook Love seeing all the red turning to green...
07:12 dcook Also, running the QA scripts is to much fun
07:12 dcook I found myself with some uncommitted time today and before I could commit it to other things I figured I'd just jump into fixing this up
07:10 josef_moravec dcook: it is great move!
07:08 dcook Koha::Object[s] will have to wait until another day, but I want to fix all the JS..
07:08 dcook I updated all the pod though and fixed up some small problems..
07:08 * dcook has a bunch of things to do at home tonight so doesn't really want to stay super late at work..
07:08 dcook Now let's see how fast I can do some of this stuff until I lose the will to continue
07:07 josef_moravec dcook++
07:07 dcook The wiki should be updated now
07:07 dcook josef_moravec: I had a feeling that was probably the case but figured I'd check
07:07 josef_moravec the rule for asset was written before the rule for filters
07:07 dcook heh
07:06 cait lol
07:06 paxed cait: funnily enough, the localization.js has strings that cannot be translated :D
07:06 dcook Ok then I'll update the wiki
07:06 josef_moravec the first one is right
07:06 dcook or: [% Asset.js("lib/jquery/plugins/jquery-ui-timepicker-addon.min.js") %]
07:06 dcook [% Asset.js("lib/jquery/plugins/jquery-ui-timepicker-addon.min.js") | $raw %]
07:06 dcook So the second question..
07:05 huginn Bug 21731: minor, P5 - low, ---, chris, NEW , t::lib::QA::TemplateFilters missing built-in TT filter "xml" causes false warnings
07:05 dcook https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21731
07:05 josef_moravec there is a room to improvement to how we are using filters, but i think we started to move the right direction
07:04 * josef_moravec agrees too
07:04 * dcook agrees
07:04 cait i think it's corret and the test is wrong
07:04 dcook <div style="white-space:pre">[% record | xml %]</div>
07:03 cait i am only about th efilter
07:03 josef_moravec we probably should enable all defalut template toolkit filters?
07:03 dcook cait: Sorry two different questions heh
07:03 cait Same as the html filter, but adds ' which is the fifth XML built-in entity.
07:03 cait raw would make the brwoser parse it
07:03 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14100 new feature, P5 - low, ---, jonathan.druart, CLOSED FIXED, Add base to make item types, branch names, sysprefs values and authorised values translatable
07:03 paxed cait: ah, bug 14100
07:03 dcook Sorry two unrelated questions
07:03 cait i think this wants xml to be displayed
07:03 cait raw would not work for xml
07:03 dcook [% Asset.js("lib/jquery/plugins/jquery-ui-timepicker-addon.min.js") | $raw %]
07:03 cait it is a thing: http://template-toolkit.org/docs/manual/Filters.html#section_html_para
07:02 dcook ?
07:02 cait which file is it?
07:02 cait it could, unless this does something different
07:01 dcook Should that be using $raw as a filter?
07:01 cait could it be osmething we defined?
07:01 dcook Also HTML8: https://wiki.koha-community.org/wiki/Coding_Guidelines#HTML8:_use_Asset_TT_plugin_for_linking_javascript_and_css_files
07:01 cait josef_moravec: odd
07:01 dcook I will open an issue :)
07:00 josef_moravec dcook, cait: there is t::lib::QA::TemplateFilters package, and the xml is not defined here
07:00 cait it got stuck after making the itemtypes translatalbe
07:00 cait paxed: there is one for making authorised values translatable, but not an omnibus i think
06:59 dcook That's what I'm guessing too, but it seems weird? :S
06:59 cait maybe the filter xml is not covered in the test
06:57 dcook missing_filter at line 15 ( <div style="white-space:pre">[% record | xml %]</div>)
06:57 dcook josef_moravec: Any idea why I might be seeing this?
06:55 paxed shown*
06:54 paxed hm. did there exist a bug for "descriptions from database fields show in UI cannot be translated" already?
06:52 cait picking some smaller bugs this morning - only a bit of time
06:51 josef_moravec cait++
06:51 alexbuckley :)
06:50 cait hi alexbuckley!
06:50 josef_moravec hi alexbuckley
06:50 alexbuckley hi cait and josef_moravec
06:34 josef_moravec dcook: great! I am looking forward to your follow-up
06:34 cait just wanted to ask, i am running out of time this week :(
06:34 cait np
06:34 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21678 major, P5 - low, ---, josef.moravec, ASSIGNED , Adding debit "Renewal of Rental Item" and "Reserve Charge" does not create account offset
06:34 josef_moravec cait: great, I will work on ILL patches, just wanted to rework patch on bug 21678 first
06:33 dcook josef_moravec: I've been working on your 10662 feedback today :)
06:32 * dcook waves
06:32 cait could you maybe have another look at the ILL patches?
06:32 cait just applying your patch
06:32 cait good morning josef_moravec
06:32 josef_moravec hi cait
06:31 * cait waves
05:43 * dcook hopes running "dbic" in his kohadevbox was the right thing to do...
05:42 alexbuckley :)
05:32 * dcook waves
05:32 alexbuckley hi again
04:56 dcook As it looks like we're really just using Mojolicious to proxy a PSGI request to an emulated PSGI app
04:55 dcook We could just use 'psgi.input' => $req->env->{'psgi.input'};
04:55 dcook I bet instead of my $body = $c->req->body blah blah
04:54 dcook https://mojolicious.org/perldoc/Mojo/Message/Request#env
04:54 dcook lavamind... I actually have an idea lol
04:46 * lavamind hitting the hay
04:46 lavamind anyway, thanks again, and good night!
04:45 lavamind that put me on the right track :P
04:45 lavamind "dcook Yeah I reckon between the psgi.input and CONTENT_TYPE... it should just be fine"
04:42 dcook hah I don't know that I did too much
04:39 lavamind you lost a bit of time with me on this but you gained $beverage of your choice XD
04:38 lavamind dcook: thanks for your help, really appreciated
04:38 lavamind but I'm not going down that rabbit hoel not tonight
04:37 lavamind I suspect it's probably due to the old Mojolicious version I'm using
04:37 dcook Sounds like a good plan
04:37 lavamind I'll share my findings on the tracker
04:37 lavamind WORKSFORME :p
04:36 lavamind I already spent about 6 hours on this
04:36 lavamind dcook: I'll let the Mojolicious veterans figure that one out
04:36 dcook But on that note, I should really focus on the thing that I should be working on :p
04:35 dcook That's up to you, but I'd keep looking if I were you
04:35 lavamind my $body = $build ? $c->req->build_body : $c->req->body;
04:35 dcook I'm wondering why we're even creating that psgi env hash..
04:29 * dcook finds Mojolicious documentation to be rather inadequate a lot of the time
04:28 dcook "Message content, defaults to a Mojo::Content::Single object."
04:28 dcook Although..
04:27 dcook Mojo::Content::MultiPart should be what $c->req->body is getting I would think..
04:27 dcook Yeah that fix just looks like it makes more efficient
04:26 lavamind I'm using 7.21 because that's whats available in stretch
04:26 dcook "And you don't have to worry about memory usage, because all files above 250KiB will be automatically streamed into a temporary file."
04:26 lavamind dcook: maybe it just has to do with my version of mojolicous
04:25 lavamind but its a pretty recent fix
04:25 lavamind also https://github.com/mojolicious/mojo/commit/2b6078216eee54e4237488e7be7ab79107d219f2
04:25 lavamind seems specific to multipart form content qith lots and lots of parts though
04:24 dcook body() really looks like it should be the right answer
04:23 dcook "That is not something we expect to be efficient i'm afraid and does not qualify. Rebuilding the request body is merely there for debugging purposes."
04:23 dcook Just looking at this: https://github.com/mojolicious/mojo/issues/1206
04:23 dcook used that is
04:23 dcook Err
04:23 dcook Hmm not sure build_body is supposed to be expected
04:22 lavamind POST and GET data gets through
04:22 lavamind seems to work at first glance
04:21 lavamind should it use build_body for allrequests though ?
04:20 dcook Yeah I was wondering about 'Slurp or replace "content".'
04:20 dcook heh
04:20 lavamind https://mojolicious.org/perldoc/Mojo/Message#build_body
04:20 dcook Sounds good to me
04:20 dcook "Render whole body with "get_body_chunk"."
04:20 lavamind that's it
04:20 lavamind with my $body = $c->req->build_body;
04:20 * dcook dislikes the Koha upload code at the best of times..
04:20 lavamind replace my $body = $c->req->body;
04:19 lavamind holy crap
04:19 dcook What was it?
04:19 dcook Nice
04:19 lavamind aaah
04:19 lavamind dcook: GOT IT
04:18 dcook Wth..
04:18 dcook Well let's see... AjaxUpload is where it all starts..
04:13 lavamind but $file->handle isn't that
04:13 dcook Just trying to find where the Koha:: modules get the uploaded file
04:13 lavamind well content_type is multipart/form-data
04:12 lavamind dcook: hrmm
04:10 dcook As for Plack... well we barely use it I think
04:09 dcook Yeah I reckon between the psgi.input and CONTENT_TYPE... it should just be fine
04:07 * lavamind is looking at how Plack might handle this
04:05 * dcook ponders
04:05 dcook Then again..
04:05 dcook Hmm
04:05 dcook Yeah I don't think that'll help you
04:03 lavamind this is invoked when doing a POST to tools/upload-file.pl
04:03 lavamind dcook: this is my latest try http://paste.debian.net/1049803/
04:00 dcook Heh
04:00 lavamind if you help me figure this out I'm gonna owe you a beer :P
04:00 dcook Because it would be $c->req->upload
03:57 dcook Ahh I see what you're talking about now
03:54 lavamind it doesn't crash but it doesn't seem to work any better :/
03:53 lavamind (for the file upload URL only, obvisouly)
03:53 lavamind I tried using an alternate _psgi_env that passes the Mojo::Asset::File handler into psgi.input
03:52 lavamind psgi.input is build with $c->req->body, but Mojo doesn't put any file data there
03:52 lavamind in sub _psgi_env
03:52 lavamind https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74517
03:51 lavamind no, it's got the file
03:51 dcook So $upload->asset is empty?
03:51 dcook "Asset containing the uploaded data, usually a Mojo::Asset::File or Mojo::Asset::Memory object."
03:50 lavamind heh
03:50 dcook Hmm
03:50 dcook Written by the same dude so go figure..
03:50 dcook Ah same in Catalyst
03:50 lavamind $c is the Mojo controller, it has everything
03:50 lavamind my $response = $app->($self->_psgi_env($c));
03:49 lavamind my $app = CGI::Emulate::PSGI->handler($sub);
03:49 lavamind my $sub = CGI::Compile->compile($script);
03:49 lavamind thats what it does
03:49 dcook ikr
03:49 lavamind I wish you could reference a specifc line from a patch that's in the tracker
03:48 lavamind it passes headers, query string and body, but the body doesn't contain any upload data
03:48 lavamind I think it's in the way Mojo builds the PSGI environment
03:47 dcook For the thing you're testing, it's just a different way of implementing Koha.
03:47 dcook Mojolicious is just a framework
03:47 dcook How do you mean?
03:46 dcook Errr
03:46 lavamind the problem is it isn't reaching Koha
03:46 lavamind I can manipulate it as a Mojo::Upload::File object
03:46 dcook Oh I thought you were getting nothing in Mojolicious?
03:45 lavamind dcook: I know the upload reaches Mojolicous though, I can echo the filename and size
03:42 dcook I have a different project where I use Catalyst and it has a plugin for handling static assets in the event that you're not using a reverse proxy (like when you're developing)
03:42 dcook Actually I wonder if it'll work because the reverse proxy handles the static assets..
03:41 dcook Yeah that can be tricky
03:41 dcook heh
03:41 lavamind I have to remember how to do that heh
03:38 lavamind hang on
03:38 lavamind I suppose I could
03:38 dcook That would eliminate one variable
03:37 dcook Why don't you try it without your Apache reverse proxy?
03:37 dcook Oh actually..
03:37 lavamind dcook: alright, I understand
03:37 dcook And the fridge.. and the freezer..
03:37 dcook lavamind: Unfortunately, I probably won't :(. I'm so super busy on all the things currently on my plate
03:36 lavamind dcook: if you'd like to investigate later please feel free to poke me
03:19 dcook Wow QA scripts are way more hardcore than they used to be. But surely lots of things that get through don't actually pass all these...
03:18 * dcook is intrigued but has other things to do himself
03:16 lavamind heh, I was afraid my hours of digging around Mojo and PSGI and stuff woulda been for naught
03:15 wizzyrea at least you can be reasonably sure it isnt any of the easier things now :)
03:14 lavamind (instance specific tmp dirs)
03:13 lavamind I think I've got all the fixed mentioned
03:13 lavamind this is Koha 18.05 btw
03:12 lavamind I put a $log->debug inside the upload hook and it never fires
03:12 lavamind one of the first things I tried was strace morbo, and seeing it doesn't attempt to open or write anything during the upload request I figure the actual file data probably isn't reaching Koka::Uploader
03:11 wizzyrea which was the other bug
03:11 wizzyrea i have a feeling it's still possibly related, if not to that, then to the other things that were done in Koha to work around that
03:11 lavamind reloaded morbo as well
03:10 lavamind I got Apache in front as a reverse proxy
03:10 wizzyrea right, well whatever the "reload all the things" is for mojo
03:10 lavamind wizzyrea: I'm using mojo as an alternative to Plack
03:10 wizzyrea it doesn't hurt anyway
03:10 wizzyrea or maybe you need to reload plack and memcached
03:10 wizzyrea maybe the problem is in mojolicious somewhere
03:09 wizzyrea now, if it's still broken after *that*
03:09 lavamind wizzyrea: yep, I got it, that's what I see
03:09 wizzyrea now
03:09 wizzyrea it's because it's systemd
03:08 wizzyrea this stuff
03:08 wizzyrea PrivateTmp=no
03:08 wizzyrea Result should be:
03:08 wizzyrea #> systemctl show apache | grep PrivateTmp
03:08 wizzyrea #> systemctl restart apache2
03:08 wizzyrea #> systemctl daemon-reload
03:08 lavamind ok
03:08 wizzyrea not just restart apache
03:08 wizzyrea you have to do all of the reloading and everything
03:08 dcook I was thinking more for modules but good point
03:07 wizzyrea are usually reliable.
03:07 lavamind wizzyrea: I switched PrivateTmp in apache2 to flase, but the problem still exists
03:07 wizzyrea cronjobs
03:07 * dcook is looking for examples of good minimal pod..
03:07 dcook Doesn't even have pod..
03:06 dcook heh
03:06 wizzyrea well they have to get moved :P
03:06 dcook Speaking of not liking things. I was taking a look at some Koha:: modules which essentially are exactly the same as C4:: modules and just *sigh*
03:06 wizzyrea i tease there are lots of those.
03:06 wizzyrea it's true it may be one of the only words in the english language that exists in all parts of speech.
03:05 dcook I feel like f can go before/after every other letter there
03:05 lavamind wizzyrea: thanks for the pointer, I'm testing now
03:05 wizzyrea omfg.
03:05 wizzyrea er
03:05 wizzyrea omgf.
03:05 wizzyrea ...what, do we have separate temp locations for all the things now or wht
03:05 dcook :O :O :O
03:05 dcook grep grep grep
03:05 wizzyrea What is this ridiculous directory in /tmp all about
03:04 dcook Why can't I find you, you stupid file...
03:04 dcook heh
03:04 wizzyrea why <bash> aren't <bash> you <bash> working <bash>
03:04 dcook mee too
03:03 wizzyrea (fwiw it took me aaaaaaaagesssssss to track that down the first time)
03:03 * dcook hasn't read the bugs yet
03:02 dcook :(
03:02 wizzyrea and I rather don't like the way we've gone about fixing it but don't think there's another way really
03:02 wizzyrea it's annoying
03:02 dcook PrivateTmp is such a pain..
03:02 dcook Oooh good call wizzyrea
03:01 lavamind hrmmm
03:00 wizzyrea lavamind ^
03:00 huginn Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20428 major, P5 - low, ---, tomascohen, RESOLVED FIXED, MARC import fails on Debian Stretch
03:00 wizzyrea bug 20428 is probably relevant too
02:59 wizzyrea see if that helps
02:59 huginn Bug 19676: enhancement, P5 - low, ---, koha-bugs, RESOLVED DUPLICATE, Debian 9 vs. Plack vs. /tmp -- attention to PrivateTmp for apache2
02:59 wizzyrea https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19676
02:59 wizzyrea I know what that si
02:58 wizzyrea yeah
02:58 wizzyrea aha
02:56 lavamind I think it has to do with Mojo not streaming the file upload data into the psgi handler, but it's all new stuff to me
02:55 lavamind wizzyrea: BTW the app works fine otherwise, I can work in OPAC and Intranetw ithout other issues
02:52 lavamind wizzyrea: Debian 9
02:52 wizzyrea lavamind, what OS are you using
02:42 dcook Ah maybe not. Just confusing output from qa tools..
02:40 dcook Hmm the instructions might be wrong..
02:40 dcook Trying this out for the first time: https://wiki.koha-community.org/wiki/QA_Test_Tools#Use_with_a_KohaDevBox