IRC log for #koha, 2023-10-10

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

All times shown according to UTC.

Time Nick Message
00:14 alohabot joined #koha
02:17 aleisha joined #koha
02:17 aleisha this is driving me crazy
02:17 FAILKoha/Library/Hour.pm
02:17 FAIL  pod coverage
02:17 POD coverage was greater before, try perl -MPod::Coverage=PackageName -e666
02:17 OKKoha/Library/Hours.pm
02:17 WARNKoha/Schema/Result/Branch.pm
02:17 WARN  tidiness
02:17 The file is less tidy than before (bad/messy lines before: 222, now: 226)
02:17 FAILKoha/Schema/Result/LibraryHour.pm
02:18 FAIL  pod coverage
02:18 POD coverage was greater before, try perl -MPod::Coverage=PackageName -e666
02:18 WARN  tidiness
02:18 The file is less tidy than before (bad/messy lines before: 0, now: 18)
02:18 what am i meant to do??
02:23 sam__ joined #koha
02:39 tcohen[m] If the POD is present and it is a false positive, mention in the bug. If the POD is missing for object_class and friends on schema files, skip that
02:39 About the tidy... There was a post on the devel list
02:39 But basically
02:39 wahanui i guess basically is checks the Bugzilla statuses are correct, then updates the status, adds the 'released in versions' text and adds a comment
02:40 tcohen[m] https://wiki.koha-community.org/wiki/Perltidy
02:40 You need to tidy those 18 lines you added/modified
02:40 aleisha in a schema file?
02:41 its a brand new schema file, from a new table
02:41 tcohen[m] Whichever tool you use (I use VScode) it needs to use the Skype perltidyrc
02:42 As I said, schema files related are usually false positives
02:42 Maybe fix the tool so it is smarter about those
02:43 I can take a look tomorrow at the desk. Way too like here right now and already in bed
02:43 Good luck!
02:44 *too late
02:44 aleisha thanks!
03:34 Adarsh joined #koha
03:34 Adarsh Hello!!
03:34 Does someone is able to run koha on kubernetes cluster?
05:16 ?
05:38 alohabot joined #koha
05:44 bag joined #koha
05:47 reiveune joined #koha
05:47 reiveune hello
06:09 alohabot joined #koha
06:55 cait joined #koha
07:00 thibaud_g joined #koha
07:11 Joubu ashimema: around now
07:12 aleisha: you can ignore tidiness warnings for Koha/Schema/Result files if your patches do not modify them directly
07:14 @later tell tcohen[m] 31199 is used to keep track of selenium/patrons_search.t failures
07:14 huginn Joubu: The operation succeeded.
07:19 alohabot joined #koha
08:08 ashimema I wanted to get your take on bug 34939
08:08 huginn 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=34939 normal, P5 - low, ---, lucas, Pushed to stable , When manually entering dates in flatPickr the hour and minute get set to 00:00 not 23:59
08:09 ashimema Hmm, the bug I linked to it got unlinked somehow
08:11 Joubu isn't that a regression?
08:11 pretty sure 23:59 was the default before
08:11 ashimema Bug 35014
08:11 huginn 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=35014 major, P5 - low, ---, martin.renvoize, Passed QA , Times should only be set for enable-time flatpickrs
08:12 ashimema Improves the specificity of it a bit.. but I'm still not sure about the placement in onchange
08:12 Yeah, the defaults are set, but it appears flatpickr doesn't pick them upset them when manually entering dates instead of using the picker itself
08:13 I think that's what Lucas tries to fix here
08:13 Anywho.. I thought you might have some thoughts.  Other than breaking bookings which I've worked around now in the rebase I couldn't find any other issues.. but it had a slight code smell to me so I wanted the second opinion 😜
08:15 Joubu does it mean 23:59 will be used even if 00:00 is selected?
08:18 ashimema I think so
08:18 Which is bad
08:22 Joubu yes, confirmed
08:29 ashimema: I am not finding a better way to fix the original problem :-/
08:29 ashimema okies
08:30 perhaps we leave it as is and wait and see a bit
08:30 Joubu how does it break bookings?
08:30 ashimema I couldn't find any really obvious breakage.. it was more a bug feeling
08:30 well.. it broke flatpickr entirely
08:30 but I'm going various things after instantiation
08:30 converting to range mode, adding a disable function callback
08:31 Joubu but it's no longer possible to select 00:00, that's a problem I think
08:31 ashimema appending my own onChange
08:31 Yeah.. bookings uses 00:00 -> 23:59 for it's logic
08:31 Joubu yes, that's a problem with flatpickr, we are not merging the constructor's parameters with the default ones
08:31 like we do for datatables for instance
08:32 ashimema indeed
08:32 I use a filter + push approach to merge actually
08:32 Joubu 00:01 is replaced with 23:59
08:32 ashimema oh
08:32 that's not good
08:33 Joubu 75             if ( !parsedDate.getHours() ) {
08:33 alohabot joined #koha
08:33 Joubu that's bad
08:35 and.. I didn't actually see your patch!
08:35 testing now
08:35 ashimema my patch doesn't fix those
08:35 it just binds it so at least we're only executing the time stuff on time enabled flatpickrs
08:35 as apposed to all
08:36 but I agree.. the getHours() is clearly way too loose
08:38 Joubu I just think that... the original bug is not a bug
08:38 neither a feature :D
08:38 but if you allow manual input to specify a date AND time value
08:38 then enter both
08:42 and... flatpickr is not longer maintained? last push is one year old :-/
08:43 ashimema ☹️
08:43 that's upsetting
08:43 davidnind I think I may have "caused" this when testing bug 33703 (see comments 14 and 15) - when manually entering dates, you couldn't enter the time and it defaulted to 00:00. I also mentioned that when FlatPicker was introduced, there were issues with the time and for some, the time was defaulted to 23:59...
08:43 huginn 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=33703 enhancement, P3, ---, lucas, Needs Signoff , Entering dates should be more flexible accepting different entry formats
08:46 ashimema there's a few active forks
10:14 Adarsh Does someone is able to run koha on kubernetes cluster?
10:14 wahanui i already had it that way, Adarsh.
10:17 Adarsh wahanui, can you please help me.  I tried to do it with koha-docker repo but getting errors.
10:17 wahanui Adarsh: sorry...
10:17 paulderscheid[m] Hi Adarsh, I'm currently working on it
10:17 ashimema wahanui is a bot
10:17 paulderscheid[m] wahanui is a bot :D
10:17 Adarsh ok
10:18 Yes, I doubt as it is replying very fast :D
10:18 paulderscheid[m] What was your approach w/ the koha-testing-docker repo?
10:19 Are you using Kompose?
10:20 Joubu koha-testing-docker != koha-docker
10:20 paulderscheid[m] Oh
10:20 Well I'm working on a manifest for ktd
10:20 not koha-docker
10:20 Joubu https://gitlab.com/koha-commun[…]ocker/koha-docker
10:20 paulderscheid[m] You wanna run this in prod Adarsh?
10:20 Adarsh yes, this is the one.
10:21 Yes required for prod.
10:21 paulderscheid[m] Are you using Kompose then?
10:21 khall joined #koha
10:22 Adarsh I tried Kompose but it was not working.
10:25 FATA Unable to load files: validating C:\Users\Adarsh\sw-koha-docker\docker-compose.yml: services.apache2.networks.kohanet.aliases array items[0,1] must be unique
10:27 paulderscheid[m] I ran into that error before when using podman.. Trying to remember how I fixed it
10:30 Adarsh apache2, cron and koha images are given same in docker-compose.
10:30 paulderscheid[m] The first thing that comes to mind is that these two constructed aliases return the same value
10:30 https://gitlab.com/koha-commun[…]type=heads#L52-56
10:31 Have you checked the output?
10:37 Adarsh not getting any output
10:44 paulderscheid[m] It's weird that those aliases aren't unique when I look at the .env in the repo.
10:47 Could you try running compose again with this modification in the `docker-compose.yml`?... (full message at <https://matrix.org/_matrix/med[…]LiKexiQfaqqxTifeo>)
10:48 Sorry, I meant Kompose obv
11:03 * cait waves
11:04 paulderscheid[m] Hi cait
11:04 cait ello
11:04 paulderscheid[m]: I wanted to ask you... are there any plans to get rid of the apache rewrite things for the room reservation plugin?
11:05 I think we discussed it might be possible a longer while ago
11:06 paulderscheid[m] We are currently merging 22.11. When we start w/ compatibility testing I will release a new version using the pages feature.
11:06 But it'll still take some time. Months probably.
11:08 kidclamp joined #koha
11:09 marcelr joined #koha
11:09 marcelr o/
11:10 cait paulderscheid[m]: thx anyway :)
11:10 paulderscheid[m]: if you can, please let me know when the new version is there, but I'll come back to it eventually
11:11 paulderscheid[m] Always glad you're interested!@
11:11 I will
11:11 marcelr Joubu around?
11:12 * marcelr looking for a way to get focus on  a flatpickr input; somehow not trivial
11:14 Joubu yes
11:15 you need to access the input tracked by flatpickr, maybe?
11:15 marcelr i have tried various things including calling flatpickr( id, {} ) etc but to no avail
11:16 do you know the short route? this is a bit trial and error..
11:16 Joubu did you try to access .input?
11:16 marcelr itried also with input.active
11:17 but i think you cannot set focus directly?
11:17 should you go via the flatpickr object?
11:17 there is also an open method but it didnt work out
11:19 alohabot joined #koha
11:19 marcelr opened bug 35000
11:20 huginn 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=35000 enhancement, P5 - low, ---, m.de.rooy, ASSIGNED , OPACMandatoryHoldDates does not work well with flatpickr
11:20 Joubu do you mean you want the calendar to show up directly?
11:21 marcelr i want to focus the date field and that normally means that the calendar opens rightaway
11:22 tcohen[m] hola #koha o/
11:22 marcelr but i saw a recent patch removing readonly attribute
11:22 hola tcohen[m]
11:22 paulderscheid[m] Can't you just target the .flatpickr-input and call .click()?
11:23 marcelr i should have tried it too already, but tried too many things already
11:23 paulderscheid[m] In the little experiment I tried to focus the input, but apparently it has no :focus state so you don't see any change?
11:23 s/?/./
11:24 s/the/my/, s/?/./
11:24 marcelr maybe via controllinbg object
11:25 paulderscheid[m] Could be but I don't see anything in the docs. There's is however the onChange and onOpen hooks where you could manually visually highlight (or whatever).
11:26 Well rather just the onOpen hook
11:27 marcelr i will experiment a bit further (alas)
11:28 cait oleonard: could you amybe have a look at my follow-up on bug 34300
11:28 huginn 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=34300 enhancement, P5 - low, ---, wainuiwitikapark, Signed Off , Add link to place a hold on ordered items in baskets
11:28 cait ?
11:28 I found some more things to fix when adding the permission check but with the datatable it might be good :)
11:32 marcelr Joubu, paulderscheid[m] think i found it ;)
11:32 trigger('focus') seems to work
11:32 paulderscheid[m] Cool
11:33 marcelr a fresh look always provides solution
11:33 paulderscheid[m] True
11:35 alohabot joined #koha
11:36 puphaus[m] Hello together :) is there a possibility to attach an already existing copy of a book to a new order?
11:37 cait the record or the item?
11:37 puphaus[m] the item 😅
11:38 cait if the item was not created in the acq module, there is at the moment no way to attach it manually to an order - I believe I once filed a bug about that
11:38 tcohen[m] Adarsh: the docker images are still a project, targeting the 23.11 release. They are functional but I have a couple design doubts I need to address
11:38 will be working on that this week
11:42 marcelr tcohen[m]: did you wait with 28688 for special reason?
11:44 puphaus[m] @cait ok thanks for the info :)
11:44 huginn puphaus[m]: I've exhausted my database of quotes
11:44 tcohen[m] marcelr just lack of time
11:45 marcelr ok
11:45 common problem
11:46 ashimema: around?
11:55 ashimema hi
11:55 wahanui niihau, ashimema
12:03 tcohen[m] Joubu: the Koha/Biblio.t failing test highlights a slight behavior change in GetNormalizedOCLCNumber
12:04 oleonard Sorry cait was afk but I will take a look
12:04 tcohen[m] should we return '' or undef?
12:04 '' will yield less warning handling
12:05 but I'm not sure what's more correct
12:07 Joubu I don't understand, when I wrote it it was passing...
12:07 magnuse joined #koha
12:07 Joubu or did I forget to run it?...
12:08 after the rewrite, possible..
12:08 cait oleonard: thx! :)
12:10 tcohen[m] Joubu: regardless, get_normalized_upc should have the same behavior as get_normalized_oclc, so we need to pick
12:10 Joubu C4 subs return undef, and the test expects undef
12:10 so I would pick undef
12:11 and so that we could differenciate: "field exists but empty" from "field does not exist"
12:11 tcohen[m] agreed
12:12 Joubu well, I don't understand why the test is failing actually
12:13 marcelr bug 35000 is ready for testing :)
12:13 huginn 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=35000 normal, P5 - low, ---, m.de.rooy, Needs Signoff , OPACMandatoryHoldDates does not work well with flatpickr
12:13 marcelr nice number
12:14 tcohen[m] 53 more to a nicer one
12:14 marcelr right
12:14 what about 44444
12:15 tcohen[m] too far
12:15 marcelr just wait
12:16 Joubu tcohen[m]: patch attached. I should not do that (behaviour change and refactoring), but... I want to :D
12:17 tcohen[m] I'm happy you do it
12:17 you're allowed to
12:17 ✌️
12:28 nikkom Hi koha, I wanted to share my happiness with you. At last I have solved my problem. It was related to COinSinOPACResults syspref. I made it as don't include.
12:29 marcelr nikkom++
12:44 does anyone know whether printing from the cataloguing editor changed from 21.11 to 22.11; i now only see the actual tab instead of all tabs before ?
12:45 magnuse_ joined #koha
12:47 marcelr maybe a result of the restyling of the staff
12:47 probably
12:47 wahanui well, probably is too hard.
12:48 oleonard marcelr: Is that before and after the switch from jQueryUI tabs?
12:48 If so we should update the print stylesheet to make sure tabs are shown
12:49 marcelr not sure, should look further
12:49 but sounds like we should look at print stylesheets
12:52 oleonard cait: What problem did you encounter with Bug 34300 and hiding the column altogether?
12:52 huginn 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=34300 enhancement, P5 - low, ---, wainuiwitikapark, Signed Off , Add link to place a hold on ordered items in baskets
12:56 marcelr oleonard bug 31677 is in 22.11.00 and 22.05.08 (not in 21.11)
12:56 huginn 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=31677 enhancement, P5 - low, ---, oleonard, RESOLVED FIXED, Convert basic MARC editor tabs to Bootstrap
12:57 oleonard marcelr: I wouldn't have guessed that anyone wanted to print the MARC editor but there are certainly other pages where it's obvious that all tabs should be printed.
12:57 marcelr some staff users use it here (as it appears)
12:57 shall i open a new report?
12:58 i guess you will be much faster in solving this than i am ;)
12:58 oleonard Yes if you open a bug you can assign it to me
12:58 marcelr great
13:00 bug 35016
13:00 huginn 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=35016 minor, P5 - low, ---, koha-bugs, NEW , Printing from cataloging editor does no longer print all tabs
13:22 cait oleonard: the modify column started showing in the column config
13:23 oleonard: i had the permission checks around all the th and td elements first, but then when a suser without the reserveforothers viewed the page, the column settings had one too many entries
13:42 magnuse joined #koha
13:47 Dyrcona joined #koha
14:12 oleonard I thought I had it, cait, but no luck :(
14:15 cait yeah the datatable scares me :)
14:16 but as is I think my patch still improves the situation a little?
14:21 at least no link that leads to a permission denied page
15:10 reiveune bye
15:10 reiveune left #koha
15:53 cait bye all!
15:53 cait left #koha
16:33 cait joined #koha
16:58 oleonard I'm getting very grumpy with DataTables.
16:59 ashimema What's up oleonard
17:27 oleonard ashimema: just complaining. I'm sure it'll work out
18:52 jalway joined #koha
19:42 aleisha hello
19:50 magnuse \o/

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

koha1