Time Nick Message
03:17 dcook Looks like I've been struck by the unusable koha-testing-docker now too...
03:18 davidnind even after a git pull for koha-testing-docker and a ktd pull?
03:18 dcook Yep :/
03:18 dcook And nuking node_modules
03:19 dcook It doesn't like /kohadevbox/koha/cypress.config.ts
03:20 dcook Which I can't even find..
03:20 davidnind I was going to say ashimema had some comments about that last week at https://irc.koha-community.org/koha/2023-09-08#i_2506895
03:22 dcook Yeah no dice :(
03:22 dcook Actually..
03:23 dcook Maybe I was looking at the wrong node_modules..
03:23 pastebot "davidnind" at 127.0.0.1 pasted "My cypress.config.ts - my Koha source folder is up-to-date" (22 lines) at https://paste.koha-community.org/26334
03:23 dcook Aha yes it would help if i was looking at the right git repo..
03:24 davidnind Ignore the paste above, hopefully you have solved it! (Not sure I am any more help.....)
03:24 dcook Thanks davidnind, I'm going to try this again and see how I go..
03:24 dcook Helpful even if only to nudge me to try again hehe
03:24 * dcook is glad he didn't give into his desire to just set the computer on fire...
03:25 dcook Yeah, problem was I was just nuking the node_modules in the wrong git repo. I have a few different Koha git repos for a few different koha-testing-docker projects...
03:25 dcook Thanks for being my rubber duck davidnind :)
03:28 davidnind excellent!
04:59 SunitaBarve[m] after upgrading koha from 21.11.* to 21.11.13.001 the upgrade threw error make issues.borrowernumber and itemnumber NOT NULL ERROR - {UNKNOWN}: DBI Exception: DBD::mysql::db do failed: Cannot change column 'borrowernumber'': used in a foreign key constraint 'issues_ibfk_1' at /usr/share/koha/lib/C4/Installer.pm line 743
04:59 SunitaBarve[m] can anyone help to sort this error
05:37 davidnind This seems similar to bug 32399, so looks like it isn't fixed for 21.11.x
05:37 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32399 major, P5 - low, ---, m.de.rooy, RESOLVED FIXED, Database update for bug 30483 is failing
05:38 davidnind I don't know enough to help, maybe wait a couple of hours and post again, as there should be more people around by then...
05:46 reiveune hello
06:48 matts_ hi !
06:53 Joubu ~o~
07:09 * cait reports that strings have been updated on Pootle
07:18 magnuse_ \o/
08:20 MatthewBlenkinsop[m] o/
08:26 Zahid[m] Hi, i am using koha 22.11 version. In patron card layout, there are 3 fields; the librarian can add only 3 fields in layout, but i require 4 fields in card layout. i have search on internet but cannot find any solution. Can you people help me in this problem. our library wants to print borrower's card with following fields; 1st field... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/brxBXPscbrYhqpAeMdYnSmPf>)
08:43 cait I don't think we can change that
08:44 cait I just had alook and the 3 fields seems to be fix
08:44 cait but you can enter multiple database fields into one text line
08:44 cait does that help?
08:45 cait the example int he manual has <firstname> <surname>
08:45 cait Zahid[m]: https://koha-community.org/manual/22.11/en/html/tools.html#layouts
08:52 SunitaBarve[m] after upgrading koha from 21.11.* to 21.11.13.001 the upgrade threw error make issues.borrowernumber and itemnumber NOT NULL ERROR - {UNKNOWN}: DBI Exception: DBD::mysql::db do failed: Cannot change column 'borrowernumber'': used in a foreign key constraint 'issues_ibfk_1' at /usr/share/koha/lib/C4/Installer.pm line 743
08:52 SunitaBarve[m] can anyone help to sort this error then I tried upgrading to 22.05 but still there is an issue with other bug
08:52 SunitaBarve[m] can someone help to sort out this error
08:52 SunitaBarve[m] can we sort it on production server
08:53 cait there shoudl be something about the version too - that would help locate the problem
08:53 cait hm maybe not in the old versions
08:53 cait you need to find which database update this was, to see what the problem is
08:53 SunitaBarve[m] can u guide me to sort error
08:55 * SunitaBarve[m] uploaded an image: (326KiB) < https://matrix.org/_matrix/media/v3/download/matrix.org/JYcIZVuwNgAkIufKklnTKRDo/Screenshot%20from%202023-09-18%2014-24-47.png >
08:55 SunitaBarve[m] pl. see attached screenshot
08:55 cait ah ok
08:55 cait you need to look at bug 30483
08:55 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30483 normal, P5 - low, ---, m.de.rooy, RESOLVED FIXED, Do not allow NULL in issues.borrowernumber and issues.itemnumber
08:56 cait Ok, what I would do:
08:56 cait make a new SQL report and lok if you have bad data
08:56 cait select * from issues where borrowernumber is null or itemnumber is null;
08:56 cait if you have any, you might need to remove them for the db update to succeed
08:56 SunitaBarve[m] i will run these commands on server with your guidance
08:58 SunitaBarve[m] it does not retrieve any data
09:00 SunitaBarve[m] it says empty set
09:02 SunitaBarve[m] how to change for this bug 30483
09:02 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30483 normal, P5 - low, ---, m.de.rooy, RESOLVED FIXED, Do not allow NULL in issues.borrowernumber and issues.itemnumber
09:03 SunitaBarve[m] where exactly i would need to change can u guide
09:16 cait hm if it's empty the database update should work
09:16 cait sorry, I can't help you more than that
09:17 cait hm ok, that's od
09:17 cait bug 32399 shoudl have fixed your issue
09:17 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32399 major, P5 - low, ---, m.de.rooy, RESOLVED FIXED, Database update for bug 30483 is failing
09:18 cait SunitaBarve[m]: you gave the wrong versions above
09:18 cait you are not updating to 21.11.13
09:18 cait you are updating to a 22.05. version - your screenshot shows it too
09:19 cait please check which version you are really updating to
09:20 cait if you are going to 22.05.xx it shoudl be at least 22.05.12
09:31 Joubu Am I rusted or there is something weird in that: passing variable from pl test => [], then in tt [% test.size %] does not display 0 but empty string
09:33 cait sorry, no idea
09:34 cait have you tried dumping the variable to templates to see how it actually looks like?
09:35 Joubu empty string
09:36 Zahid[m] cait: thanks for the answer, i need these 4 fields in separate lines... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/jBPcooUaHhMrfcXZNfVvNEnd>)
09:37 Zahid[m] *<expirydate> \n or < br > <sort1>
09:37 Zahid[m] will koha knows \n or < br > for separate line
09:39 Zahid[m] * cait: thanks for the answer, i need these 4 fields in separate lines... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/remKTykHdmiYFOLPfEoYfHXD>)
09:54 cait Joubu: does size count correctly if it's not empty?
10:03 AnkeB Hi, we have some orders in a Koha instance that remain from testing and that, unfortunately, have no more corresponding biblios. They are to be deleted, but by clicking on them, we get an HTTP 500. Now we'd like to delete them directly in the database but are not sure what table(s) are involved. Is it sufficient to remove the relevant data from aqbasket?
10:45 oleonard o/
11:20 cait hi Anke, look at aqorders and aqbasket
11:22 cait maybe you can delete the basket and it will cascade
11:22 anke_b[m] Thx cait !
11:28 cait ashimema++ # thx for QA - you did some I was struggling with :)
11:29 anke_b[m] * Thx cait++ !
11:39 marcelr o/
11:40 marcelr ashimema++
11:52 tcohen[m] hola #koha
11:53 marcelr hi tcohen[m]
11:54 tcohen[m] marcelr: is your bug resurrected already?
11:56 ashimema đ
11:57 ashimema just looking at bug 32942 now
11:57 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32942 trivial, P5 - low, ---, frederic, Signed Off , Suggestion API doesn't support custom statuses
11:57 ashimema sorted out the QA issues with follow-ups, but the tests are failing.. just digging into what
12:01 marcelr tcohen[m]: yeah martin did it
12:01 ashimema đ
12:01 tcohen[m] glad to hear that
12:01 ashimema finished off this morning.. it's not PQA đ
12:02 ashimema now
12:02 ashimema not not
12:02 ashimema lol
12:04 ashimema bug 34653 looks trivial.. but I'm not sure what side effects it would have with our package cron et al.
12:04 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34653 minor, P5 - low, ---, evan, Signed Off , Make koha-foreach return the correct status code
12:04 ashimema may need mtj or tcohen on that one
12:05 Zahid[m] using koha 22.11 can someone tell how can i print pdf file for patron card creator with following... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/NslwznXFLpJMXXrThFlTthww>)
12:06 * ashimema needs to get better at failing qa instead of fixing issues
12:09 cait heh
12:09 cait 8 bug bugs in the queue, I like to see that
12:09 cait I got stuck around 11/12
12:14 tcohen[m] ashimema: sounds risky for loops in crons?
12:15 marcelr hmm seeing OpenAPI >>> GET api/v1/items [{"message":"Expected integer - got null.","path":"\/body\/1\/effective_not_for_loan_status"}]
12:15 ashimema yeah
12:16 marcelr `notforloan` smallint(6) DEFAULT NULL COMMENT '1 if the item is not for loan, 0 if the item is available for loan',
12:17 marcelr seems that API is not compliant with kohastructure here
12:17 tcohen[m] the API is highly not compliant with kohastructure heh
12:18 marcelr heh
12:18 tcohen[m] I mean, we did it so we could harmonize all the attribute/column names on the API
12:18 marcelr these things bite you when you are working on something else
12:18 tcohen[m] my $extractor_class = "Koha::MetadataExtractor::MARC::$schema";
12:18 tcohen[m] require $extractor_class;
12:18 tcohen[m] $self->{extractors}->{$schema} = $extractor_class->new;
12:19 tcohen[m] Any ideas why that require could fail saying it cannot find the class?
12:20 marcelr check $schema :)
12:20 tcohen[m] Can't locate Koha::MetadataExtractor::MARC::MARC21
12:20 tcohen[m] but I can call that class directly and it works
12:21 marcelr check @INC
12:22 pastebot "tcohen" at 127.0.0.1 pasted "kohadev-koha@kohadevbox:koha(b" (17 lines) at https://paste.koha-community.org/26344
12:24 pastebot "tcohen" at 127.0.0.1 pasted "Calling it directly" (2 lines) at https://paste.koha-community.org/26345
12:28 marcelr does it compile correctly btw?
12:28 marcelr probably
12:29 marcelr weird
12:37 tcohen[m] can someone try checking out this
12:37 tcohen[m] https://gitlab.com/thekesolutions/Koha/-/commits/bug_metadata_extractor
12:37 tcohen[m] and running perl test.pl
12:37 tcohen[m] on KTD?
12:39 tcohen[m] ashimema: marcelr
12:40 marcelr i have no ktd tcohen[m]
12:40 ashimema in the middle of a rebase right now.. but could try in say half hour?
12:42 marcelr it feels like a configuration issue
12:45 tcohen[m] Joubu: a follow-up on 33406 seems to have broken tests
12:51 tcohen[m] marcelr: it seems I need to RTFM
12:52 tcohen[m] unless ( $self->{extractors}->{$schema} ) {... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/pivHsdOdUnHyjFwUqAgdpFJG>)
12:52 tcohen[m] I'm not sure how the ILL stuff (from where I picked this) is working then
12:52 tcohen[m] :-D
12:52 tcohen[m] ah, got it
12:53 tcohen[m] they are loading a path
12:53 marcelr you got it anyway
13:15 ashimema ack
13:15 ashimema we have some massive holes in our tests still âšī¸
13:15 ashimema there are no tests whatsoever for _send_message_by_email as far as I can see âšī¸
13:16 tcohen[m] hmm
13:18 * ashimema is trying to squeeze bug 12532 through QA
13:18 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12532 new feature, P5 - low, ---, hammat.wele, Signed Off , Redirect guarantee email to guarantor
13:18 ashimema but it worries me there are no tests for the routine we changed
13:21 marcelr did you look at Koha::Email too ?
13:22 ashimema in what sense?
13:25 marcelr that module takes care of emailing ultimately
13:25 ashimema yes of course
13:25 marcelr the last step is being mocked in tests, i think
13:26 ashimema but it's C4::Letters changes that sets the addresses in this case
13:26 marcelr the send_or_die or something from stuffer
13:26 ashimema mm
13:26 ashimema the issue is complete lack of tests
13:26 marcelr yeah that is bad
13:26 ashimema there are literally none for _send_message_by_email
13:27 ashimema so it's not just a case of me adding in tests for the change..
13:27 ashimema it's writing tests for the whole bloomin method đ§
13:51 ashimema marcelr
13:51 ashimema as we're using cc now for the guarantor.. do you think we should record that?
13:51 ashimema i.e. should I add a cc_address field to message_queue and record that at send time we added cc addresses?
14:19 oleonard I feel like someone suggested a smart solution for using template i18n tools to handle this kind of thing: 'ISBD view for: [% biblio.title | html %]'
14:20 oleonard Sorry, not that.
14:20 oleonard The version where biblio.title is [% INCLUDE 'biblio-title.inc' %]
14:20 oleonard ISBD view for: [% INCLUDE 'biblio-title.inc' %]
14:23 oleonard Normally I would do this: [% tx("ISBD view for {title}", { title = biblio.title } ) %]
15:04 caroline Has anyone tried to change the "Hold pickup library match" in the hold policies at the bottom of the circ rules?
15:05 cait i think I played with that once
15:05 cait oleonard: did we file the error for patron batch mod?
15:05 caroline I'm trying to make it so that in a system, only one library can manage holds, I put the "hold policy" to "No holds allowed" for all libraries except the big one, and Hold pickup library match to item's home library
15:06 oleonard cait: I did not
15:06 caroline But when I place a hold in the opac, the pickup library is automatically put to the patron's home library
15:06 cait oleonard: me neither :(
15:06 caroline I want it so that pickup library is only ever the big library
15:07 cait only items from the big library?
15:07 cait or can the other libraries send to the big library?
15:07 caroline no, only the big library can have holds
15:07 caroline on it's items
15:07 cait Hm I think i would set all the itemtypes for the lother libraries to no holds, like you did
15:07 cait ah right
15:08 cait you need to set the admin > library > no pickup location
15:08 caroline the other libraries are small school libraries and they don't have anyone to manage holds on site.
15:08 caroline ah! I'll try that
15:08 cait I think that shoudl take care of it
15:09 cait fingers crossed
15:09 cait oleonard: how do we resolve this? :)
15:09 ashimema man I'm glad I was anal here and am writing the tests
15:09 ashimema the patch plain doesn't work!
15:09 cait which patch?
15:09 cait ... and I had similar on friday
15:09 cait need to change my whole approach after grumping at the tests first
15:09 ashimema bug 12532
15:09 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12532 new feature, P5 - low, ---, hammat.wele, Signed Off , Redirect guarantee email to guarantor
15:09 oleonard cait: The missing bug report? I can file it
15:09 ashimema I've rebased it again
15:10 ashimema simple issue
15:10 cait oleonard++
15:10 ashimema YesNo prefs set value to 1 or 0
15:10 ashimema not 'Yes' or 'yes' or 'No' or 'no'
15:10 ashimema so an equality match aint gonna cut it đ
15:10 ashimema bug 12532
15:12 ashimema Unit tests for the win
15:12 ashimema This is a nice feature..
15:12 ashimema If it works
15:16 caroline Changing the pickup location flag for all libraries except the big one doesn't work. It just blocks the hold
15:17 cait hm it should still allow...
15:17 cait we use that feature in one of our installations
15:17 caroline OPACAllowUserToChooseBranch has to be on
15:17 caroline it seems
15:17 cait ah
15:18 cait interesting
15:18 caroline even if there is only one choice
15:18 cait maybe because it tries to use the patron home library as pre-select?
15:18 cait and that is not allowed?
15:18 caroline I wanted to remove it because it seems it causes a lot of confusion with the teachers, but I'll have to leave it
15:18 cait maybe you can set it to big library and hide it
15:19 caroline hide it with css you mean?
15:19 cait yah, or jquery
15:19 cait but make sur it has the correct value
15:20 caroline we try to avoid such customizations because it's too much to manage with the number of clients we have... so I'll just leave it and they won't have a choice anyway
15:22 reiveune bye
15:23 oleonard cait: Bug 34818
15:23 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34818 major, P5 - low, ---, koha-bugs, NEW , Cannot perform batch patron modification without selecting a patron attribute
15:41 oleonard If something is not backported to stable, shouldn't it be marked as fixed?
15:45 cait yes
15:46 cait caroline: that's fair
16:50 Joubu oleonard: I will have a look as soon as I can
16:59 cait bye all
18:38 Shane-NP[m] Can you add columns to the patrons deatils checkouts tab?
18:39 Shane-NP[m] I see I can pick what is visible, but I want to add last seen to the choices
18:39 cait no, you can't add additional fields there
18:39 cait and the last_seen would match the hceckout date
18:39 cait because last seen is updated whenever something is checked out, so it's there in a way
18:40 Shane-NP[m] ah okay, makes sense
18:41 Shane-NP[m] I was thinking about how last scene updated when checking in...but didn't think that through...checkout would be the last seen if it left
18:41 Shane-NP[m] Logic eludes me at times
18:41 Shane-NP[m] correct words too apparently...seen not scene....
18:43 cait not sure about you, but it's late here :)
18:43 Shane-NP[m] oh does Koha have integration for patrons like Azure or Google or anyone working on that?
18:44 caroline like OIDC or OAuth? It's integrated since 22.11 or 23.05
18:44 caroline In Administration > Identity providers
18:44 Shane-NP[m] Oh, can it import or just auths?
18:45 Shane-NP[m] ie I want to stop importing my patron via CSV
18:45 caroline I'm not sure, I think it's just of authentication
18:45 caroline I haven't used it IRL yet
18:47 caroline But it can create patrons when they first try to log in
18:48 Shane-NP[m] hmm...sadly no one logs in anymore...Librarian left and new person wanted Follett, so its just me and my IT assets still using Koha
18:49 cait I think it can do provisioning, yes
18:50 cait I remember being told it had been tested
18:50 Shane-NP[m] Nice, I will have to check into that more, thank you
18:50 cait there are checkboxes for auto-register (adding an account) and update
18:50 Shane-NP[m] Im on cough18 cough here....Im do for an upgrade
18:51 caroline oof! I have one client still on 18.05... fortunately, they don't ask too many support questions because I would be so lost lol!
18:53 cait I'll be appy when we make it to 22.11.. looking forward for the new staff interface :)
18:54 tcohen[m] Shane-NP: there's no easy way to import, unfortunately
18:54 tcohen[m] it would be great if someone came up with scripts for that, though
18:54 caroline We'll be updating everyone (almost) from 22.05 to 23.05 this fall... I'm not looking forward to them freaking out about the interface :P I tried to prepare them since last year, but I know there will be a lot of handholding and reassuring to do
18:55 Shane-NP[m] Hey all I can say...and maybe it was a VMWare issue...but I installed 22 in my last job to do the same thing...and it was sooo slow looking up stuff and loading checkouts. I am hoping when I install a new version it isn't the same...18 is pretty snappy for me needs
18:56 caroline Shane-NP[m], did you have plack and/or memcached? Koha is very slow if you don't have those enabled
18:56 Shane-NP[m] tcohen: ty
18:57 Shane-NP[m] Yeah we had everything enabled it would load quick with plack...but the backend requests cataloging, were god aweful. We only had 700 chromebook loaded and 500 patrons on a VM with 4 CPUs, 16Gb ram, and 48Gb HD space...but is all lives on a SAN, here it is local storage
18:58 Shane-NP[m] So I am hopeful when I get time and install a new edition it will run just as fast or faster
19:06 Shane-NP[m] Hmm..it can be shared...now to find how to get it... https://support.google.com/a/answer/6343701?hl=en&fl=1&sjid=17815040787555822546-NA
19:10 cait Shane-NP[m]: it shouldn't have been so slow I think, maybe something in the setup was not quite right
19:15 Shane-NP[m] Well maybe this will help me or someone...I can query my own domain and I get all the user data...
19:15 Shane-NP[m] https://developers.google.com/admin-sdk/directory/reference/rest/v1/users/list
19:15 Shane-NP[m] So I guess now its allowing an app, adding it, and then executing the code to extract
19:16 Shane-NP[m] Rabbit hole...when I have a lot of free time
19:16 Shane-NP[m] my youngest is 3 so roughly 18 years from now :P ill have free time
19:16 caroline lol!
19:18 Shane-NP[m] well my work day is over, I will see what more I can dig up, I should review the Oauth stuff to see if the API is already built and we just need extra permissions
19:19 cait would it not work for you to have them log in once?
19:19 cait just wondering
19:19 cait then it auto-creates
19:20 Shane-NP[m] They never use the system, so no real point
19:20 Shane-NP[m] if I library still used it, I would say yes
19:20 Shane-NP[m] if our*
19:21 Shane-NP[m] just picture a dark basement, the whites of my eye, the dim glow of a single laptop and a server's blinking red and green lights...with bats and spiders...I just need a few months to get proper lighting again HAHA!
19:23 cait ah ok, makes sense
19:23 cait sounds cozy ;)
19:28 cait davidnind++ # was just pondering bug 34818 :)
19:28 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34818 major, P5 - low, ---, jonathan.druart+koha, Signed Off , Cannot perform batch patron modification without selecting a patron attribute
19:29 davidnind you'll be able to QA it 8-)
19:29 cait on it :)
19:38 davidnind cait++
19:47 cait Joubu++
19:56 aleisha hello
19:57 caroline good morning aleisha!
19:58 * cait waves
20:07 fridolin Jenkins was angry but looks like there is a fix ouf ;)
20:07 fridolin Bug 33406
20:07 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33406 normal, P3, ---, jonathan.druart+koha, Pushed to stable , Searching for authority with hyphen surrounded by spaces causes error 500 (with ES)
20:35 davidnind hi fridolin!
20:53 davidnind In case it is not just me, I've created an issue for KTD - reset_all no longer seems to work as it should (that is, giving you a fresh setup as though you had just started up KTD) https://gitlab.com/koha-community/koha-testing-docker/-/issues/398
21:00 Joubu davidnind: do you get an error?
21:01 fridolin yey super david ;)
21:05 Joubu davidnind: you need a root shell
21:11 davidnind Joubu: Nothing obvious to me - I will add the log. I tried from using ktd --root --shell, but item was still checked out.
21:14 Joubu davidnind: it should start with https://paste.koha-community.org/26361
21:17 davidnind Joubu: it does...
21:21 Joubu the second line (with 'populate_db.pl') is actually dropping the DB. If you don't see any errors then you are experiencing something very odd!
21:26 davidnind I'm refreshing everything now - local koha git repository and koha-testing-docker - will see if that helps
22:04 davidnind Joubu: Still getting the same. What is weird is it is only in the search results page in the staff interface - showing '1 checked out:', but on the details page it is showing everything as available. In the OPAC, everything looks OK. Thought it might be a browser cache related issue (Firefox developer edition - clears everything when started), but showing in other browsers as well that have cache cleared (Chromium, Google Chrome).
22:12 davidnind Joubu: reindexing everything gets rid of the checkout showing in the staff interface search results - weird! (koha-rebuild-zebra -d -f -v kohadev)