← Previous day | Today | Next day → | Search | Index
All times shown according to UTC.
Time | Nick | Message |
---|---|---|
01:59 | dcook joined #koha | |
02:30 | mtj | hi ashimema, prod packages are released using tags pushed by the rmaints |
05:16 | ArvindNaikwadi[m] joined #koha | |
05:17 | ArvindNaikwadi[m] | Koha How to Print Extended Patron Attributes on a Patron Card? |
06:05 | cait joined #koha | |
06:11 | reiveune joined #koha | |
06:12 | reiveune | hello |
06:15 | ArvindNaikwadi[m] | Koha How to Print Extended Patron Attributes on a Patron Card? |
06:42 | cait joined #koha | |
06:46 | Joubu | mtj: did you see the 2 upgrade errors on the ML? |
06:46 | https://lists.katipo.co.nz/pip[…]April/059282.html | |
06:47 | https://lists.katipo.co.nz/pip[…]April/059285.html | |
06:47 | ha, just noticed the error now | |
06:47 | Undefined subroutine &main::DropAllForeignKeys called at /usr/share/koha/intranet/cgi-bin/installer/data/mysql/updatedatabase.pl line 86. | |
06:48 | no that's not the error | |
06:49 | it's a weird one, looks like the DB is empty... | |
06:52 | cait joined #koha | |
06:53 | cait1 joined #koha | |
06:56 | magnuse | did anything come out of the discussion about "limit duration of queries in stat requests" in marseille? |
06:58 | cait1 | hm I think that one was a small group, I didn't hear about the results |
06:58 | alex_ joined #koha | |
07:00 | Joubu | If you are trying to reclaim disk space with docker prune, and notice that some volumes are not deleted, you should use "docker volume prune --filter all=1" (since docker 23) see https://github.com/docker/cli/issues/4028 |
07:01 | magnuse: it happens on bug 33116 | |
07:01 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=33116 enhancement, P5 - low, ---, koha-bugs, NEW , Limit reports execution time with sql variables |
07:02 | cait2 joined #koha | |
07:12 | dcook | After my Docker Desktop Hyper-V crashed today with a corrupt virtual disk, I finally switched over to Docker Desktop WSL 2 heh |
07:12 | Good thing that I schedule regular backups of my Docker volumes, so could restore from last night's backups :D | |
07:12 | The perils of developing on Windows... | |
07:13 | * dcook | waves goodnight |
07:13 | paulderscheid[m] | Morning #koha |
07:13 | Good night dcook | |
07:13 | dcook | paulderscheid[m] I took a look at your wiki instructions again. Interesting idea! |
07:13 | paulderscheid[m] | Did you use the guide or did you DIY? |
07:13 | dcook | I was curious why you didn't go the Docker Desktop route |
07:13 | Joubu | is git fetch working for someone? |
07:14 | dcook | Joubu: I did one fairly recently |
07:14 | I'll try again | |
07:14 | Hmm I'm too up-to-date perhaps | |
07:14 | Joubu | hum ok, it works now. It got stuck for a couple of minutes |
07:14 | paulderscheid[m] | I thought it was a much more complicated setup with Docker Desktop and wanted to simplify things. |
07:14 | dcook | I think it's working |
07:14 | Joubu: It often seems to get stuck for me. I often run "git fetch" and then go away and do something else | |
07:15 | paulderscheid[m]: Fair enough! | |
07:15 | paulderscheid[m] | I wish I could finally get rid of Docker Desktop on macOS :P |
07:15 | dcook | I use Docker for a lot of other projects, so I suppose I like the deep integrations |
07:16 | Faair | |
07:16 | paulderscheid[m] | Me too, but they are not nearly as complicated as Koha :D |
07:16 | dcook | I haven't tried the WSL 2 bind mounts yet.. |
07:16 | Koha certainly has some baggage... | |
07:17 | paulderscheid[m] | Haha ^^ |
07:17 | dcook | The funny thing is that it's the Hyper-V VDHX that got corrupted... |
07:17 | And WSL 2 is just using a VDHX under the hood as well so in theory the same thing could happen again... | |
07:17 | paulderscheid[m] | If you run into roadblocks on WSL2, let me know. I'm interested. |
07:17 | dcook | WSL 2 just hides the virtual machine a lot more than Hyper-V... |
07:17 | Will do. So far so good though. | |
07:18 | I suppose there was the elasticsearch issue.. | |
07:18 | Had to add a .wslconfig file to my ~ in Windows | |
07:18 | Anyway, I really better be running | |
07:18 | paulderscheid[m] | Good Night 👋 |
07:18 | dcook | Ciao ciao |
07:21 | cait1 joined #koha | |
07:27 | magnuse | Joubu: thanks! |
07:43 | perhaps something we could try and keep in mind for future hackfests: to write something down about the discussions we have, so others can have an idea of what direction the discussions took etc? | |
07:52 | ashimema | I was going to write a blog post from my notes magnuse |
07:53 | though I don't have a blog anymore so not sure where to stick it.. the ptfs-e website perhaps | |
07:53 | I know Aude and I both took copious notes 😜 | |
07:53 | so that covers two angles | |
07:53 | magnuse | that would be awesome! |
07:59 | is https://wiki.koha-community.or[…]d_staff_interface up to date? i changed a .scss file and tried to run "yarn build" in ktd, but get "gulp not found" | |
08:01 | ah maybe i missed "yarn install" | |
08:03 | mtj | hi magnuse, gulp should already be installed in ktd |
08:05 | magnuse: hmm, that wiki page might be out-of-date now | |
08:06 | take a look at ./build-resources.PL (and run it too?) | |
08:06 | dpk_ joined #koha | |
08:06 | kidclamp joined #koha | |
08:06 | pastebot joined #koha | |
08:06 | huginn joined #koha | |
08:06 | bshum joined #koha | |
08:06 | nugged joined #koha | |
08:06 | mtj | yarn install --frozen-lockfile |
08:06 | yarn build:prod | |
08:08 | magnuse | "yarn install" + "yarn build" ran ok |
08:08 | error Command "build:prod" not found. | |
08:09 | Joubu | yarn css:build:prod |
08:09 | edit package.json and search for "scripts" to see the options you have | |
08:10 | `yarn build:prod` should indeed work, sorry for the noise.. | |
08:11 | magnuse | "$ KOHA_IMAGE=22.05 ktd up" if that is relevant |
08:14 | Joubu | the new yarn commands are for >= 22.11.04 |
08:51 | mtj | @later tell liliputech: hiya, can you push bug 32668 please |
08:51 | huginn | mtj: The operation succeeded. |
09:04 | mtj | hi Joubu, regarding https://lists.katipo.co.nz/pip[…]April/059285.html |
09:04 | "Undefined subroutine &main::DropAllForeignKeys called at | |
09:04 | /usr/share/koha/intranet/cgi-bin/installer/data/mysql/updatedatabase.pl" | |
09:06 | we need to call DropAllForeignKeys explicitly - or export sub from C4::Installer.pm | |
09:07 | (looks like an actual bug) | |
09:07 | Joubu | we should not reach that line |
09:07 | it's too early in updatedatabase | |
09:07 | it's not a compilation error | |
09:10 | mtj | ah yes, good point :) |
09:10 | i think its a common error.. if the database does not exist? | |
09:11 | Joubu | they are saying they are upgrading, so I assume the DB exists |
09:12 | mtj | mysql is not running. etc? |
09:14 | a db without a syspref table, or no 'Version' row/value? | |
09:15 | Joubu | maybe, but weird that it happens twice in a few days |
09:15 | lds joined #koha | |
09:21 | cait | just a thought I had... |
09:21 | could it be they have multiple instances and for one instance the web isntaller hasn't been run / db not created? | |
09:23 | krimsonkharne[m] | Hi #koha! |
09:23 | anybody getting an error after trying to start ktd? | |
09:23 | I get the following: | |
09:24 | * krimsonkharne[m] | sent a koha code block: https://matrix.org/_matrix/med[…]JRhkNDaHXrecOfchb |
09:24 | krimsonkharne[m] | oops, too many backticks |
09:25 | * krimsonkharne[m] | sent a koha code block: https://matrix.org/_matrix/med[…]bKPWxgXDIrJiqFpgu |
09:27 | Joubu | krimsonkharne[m]: two solutions, either you remove the node_modules directory from your Koha src root dir |
09:27 | or you run `yarn install` to update it | |
09:28 | cait | Joubu: but if you can't get into koha-shell? |
09:28 | Joubu | ? |
09:28 | cait | he said he can't start ktd |
09:28 | Joubu | it's there because you run yarn install outside of ktd |
09:28 | so yarn install outside of ktd | |
09:29 | cait | ah ok, I never tried that (probably good?) |
09:29 | Joubu | remove node_modules :D |
09:30 | node_modules is there because you run yarn install at some point, within ktd or outside. But you should not (unless you are testing patches that require it), ktd is dealing with that and embed the up-to-date node_modules | |
09:30 | in /kohadevbox/node_modules | |
09:30 | BUT /kohadevbox/koha/node_modules has priority over the ktd's shipped one | |
09:31 | and this one is created if you MANUALLY run yarn install at some point | |
09:31 | I have no idea how to make that clear, and put it in a useful place | |
09:31 | (but I am not gonna repeat it again and again every day :D) | |
09:35 | mtj | hi cait, re: multiple instances - i get those errors in the same situation :) |
09:36 | i make a bunch of temp instances when testing 'koha-create' tool - upgrading always generates errors | |
09:41 | cait | Joubu: some kind of FAQ entry maybe on the Readme? |
09:42 | mtj: i was just wondering when reading the mailing list email, but maybe more than one issue there | |
09:43 | he debug site (although I really need to put more keywords on that one) for API has proven really helpful | |
09:45 | Joubu | reported at https://gitlab.com/koha-commun[…]cker/-/issues/371 |
09:53 | magnuse | restart_all in ktd should restart/reload the api too, right? there is no special command? |
09:54 | Joubu | restart_all should be enough |
09:54 | `koha-plack --reload kohadev` is actually enough | |
09:55 | magnuse | thanks |
09:56 | (--restart, not --reload, i think) | |
09:56 | mtj | perhaps the 2 'Upgrade Schema Issue' ML emails are from the same people, as they both use 'library' as instance name |
09:58 | Joubu | magnuse: it's --reload, newly added, not sure it's in stable already |
09:58 | cait | it's common, but I had the same ide |
09:58 | a | |
09:59 | magnuse | Joubu: ah, ok thanks, i'm on 22.05 here anyway |
10:02 | Joubu | bug 21366 that is in 22.11 |
10:02 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=21366 enhancement, P5 - low, ---, dcook, RESOLVED FIXED, Add Plack reload |
10:19 | ashimema | do we not set userenv for ils-di loging? |
10:19 | s/loging/login/ | |
10:28 | wow.. | |
10:28 | is ils-di entirely unauthenticated? | |
10:31 | cait | yes |
10:31 | all you can do is limit by IP | |
10:32 | which... you really shoudl do | |
10:32 | I think that API is internal/trusted use only - supposed to connect to your own discovery or similar | |
10:33 | ashimema | mmm |
10:33 | bug 33504 is unfixable as far as I can tell | |
10:33 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=33504 enhancement, P5 - low, ---, koha-bugs, NEW , ILS-DI does not record renewer_id for renewals |
10:40 | cait | yeah |
10:40 | there is no way to tell.... and no borrower representing the API user | |
10:41 | ashimema | indeed |
10:41 | cait | well, the modal needs to deal with non-existing data then... I guess? |
10:41 | ashimema | mm |
10:41 | that's the conclusion I'm coming to | |
10:41 | but I hate it 😜 | |
10:41 | cait | might also make sense in general, in case of migrated data |
10:41 | hate is such a strong word :) | |
10:41 | ashimema | honestly.. I'm actually surprised I let the field be nullable at all |
10:41 | cait | the renewer could be deleted |
10:42 | staff user leaving | |
10:42 | ashimema | mm |
10:42 | cait | i think nullable makes sense |
10:42 | or you can<'t have a FK, that's worse | |
10:42 | i gues | |
10:44 | paxed | HEA lists countries by number of libraries - is there stats for countries by biblios/auths/items/patrons/issues? |
10:45 | cait | if it's not on the website it could maybe be drawn from the database but hasn#t been done |
10:46 | ashimema | merge requests welcome 😜 |
10:53 | Joubu | paxed: https://paste.koha-community.org/4510 |
10:54 | sum items by coutry | |
10:56 | sorted - https://paste.koha-community.org/4511 | |
10:56 | paxed | Joubu: ohh, thanks! |
10:57 | mtj | hi rangi[m], gmcharlt_, can you sort this -> https://www.mail-archive.com/k[…]org/msg13784.html |
11:02 | paulderscheid[m] | Hi quick question: What do I have to setup for _order_by, _page and _per_page params to work with Koha::Objects within plugins. |
11:05 | Joubu | paulderscheid[m]: the swagger spec |
11:06 | search "parameters/order_by" in api/**/*.yaml | |
11:06 | then you need to use $c->objects->search( $your_rs ) | |
11:07 | see Koha/REST/V1/ERM/Agreements.pm sub list | |
11:09 | paulderscheid[m] | Thanks |
11:11 | cait1 joined #koha | |
11:15 | paulderscheid[m] | Wow! That was so easy :D |
11:16 | paxed | Joubu: is that HEA query really right? 300 million items for finland doesn't sound correct |
11:20 | maybe it counts items multiple times (per consortium? branch?) - 30 million would be more believable number | |
11:22 | krimsonkharne[m] | Thanks Joubu ! Maybe I'm looking in the wrong place, but in my Koha directory there doesn't seem to be a node_modules directory.. |
11:23 | also yarn install gives "bash: yarn: command not found" | |
11:32 | alex_ joined #koha | |
11:32 | alex_ joined #koha | |
11:33 | paxed | krimsonkharne[m]: are you on debian? |
11:33 | krimsonkharne[m] | yes |
11:33 | paxed | on debian yarn is yarnpkg; easiest way is to symlink that to yarn |
11:34 | krimsonkharne[m] | ah :) |
11:34 | * paxed | got bitten by that recently |
11:34 | krimsonkharne[m] | Thanks! :) |
11:37 | Joubu | krimsonkharne[m]: did you pull new ktd images? |
11:38 | marcelr joined #koha | |
11:38 | marcelr | hi #koha |
11:40 | krimsonkharne[m] | Joubu: I don't know. Sorry for stupid questions, I've never done anything with ktd or git before :( |
11:40 | yesterday I ran git pull, switched to my own branch and then ran restart_all and updatedatabase if that helps? | |
11:41 | Joubu | are you inside ktd now? |
11:42 | krimsonkharne[m] | nope, still won't start |
11:42 | Joubu | go to your ktd directory |
11:42 | and run `docker-compose pull` | |
11:42 | go grab a coffee, then try again to start it up | |
11:43 | krimsonkharne[m] | OK, Thanks! :) |
11:51 | Joubu | paxed: maybe this one then? https://paste.koha-community.org/4520 |
11:52 | 11M for Finland? | |
11:54 | paxed: https://paste.koha-community.org/4526 the different infos, items, biblios, borrowers, old_issues, old_reserves | |
11:56 | cait | kidclamp++ |
12:00 | Dyrcona joined #koha | |
12:04 | tcohen | hola #koha o/ |
12:04 | kidclamp++ | |
12:04 | marcelr | hola recovered tcohen |
12:04 | tcohen | hola marcelr! |
12:04 | yes! | |
12:04 | marcelr | great |
12:04 | ready to go again? | |
12:05 | tcohen | sure |
12:05 | I'll just stick to Pastis this time, no mixing | |
12:05 | marcelr | and other viruses |
12:08 | tcohen | yeap |
12:08 | Joubu: do you think a WARNING about the existence of node_modules would be enough? | |
12:09 | Joubu | yes, a warning and how to fix it |
12:09 | tcohen | has anyone checked the cypress failures? |
12:09 | #koha | |
12:09 | Joubu | tcohen: I told you yesterday, wait for the other patch |
12:10 | https://irc.koha-community.org[…]3-04-11#i_2481090 | |
12:10 | tcohen | sureok, I missed that message |
12:25 | Joubu: please later me next time, I think I'm missing messages too often when using znc | |
12:32 | ashimema | hmm |
12:33 | I thought we'd decided js template literals were OK now? | |
12:33 | magnuse | shouldn't this work? "git checkout v22.05.07" in my local koha repo, then "KOHA_IMAGE=22.05 ktd up". the api isn't working and i get this in the logs: Could not load REST API spec bundle: Unable to load schema "file:///kohadevbox/koha/api/v1/swagger/definitions.yaml". at /usr/share/perl5/JSON/Validator/Store.pm line 53. |
12:33 | ashimema | hmm |
12:33 | it should work like that yet magnuse | |
12:33 | sounds like a suspect openapi version or something to me | |
12:34 | magnuse | it shouldn't be looking for api/v1/swagger/definitions.yaml i think? because of bug 30194 |
12:34 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=30194 enhancement, P5 - low, ---, tomascohen, CLOSED FIXED, Update required JSON::Validator version |
12:35 | krimsonkharne[m] | Joubu: Hooray, it worked! Thanks again! |
12:36 | btw for me "docker compose pull" worked, "docker-compose pull" didn't | |
12:36 | Joubu | yes it's now part of docker, it depends on your version of docker I guess |
12:37 | krimsonkharne[m] | ok |
12:37 | mtj | magnuse: make sure your ktd git repo is on 22.05 branch |
12:38 | magnuse | mtj: it is |
12:39 | cait1 joined #koha | |
12:39 | magnuse | ah, the stuff i'm testing is doing something icky |
12:41 | mtj | magnuse: try setting koha repo to latest 22.05 tag, and retest |
12:41 | git checkout v22.05.11 | |
12:43 | magnuse | sorry, the problem was in our local code (and me not being familiar enough with swagger) |
12:44 | cait | my personal laptop has insomnia :( keeps waking up when suspend it |
12:44 | Joubu: are we still cleaning up old entries in Hea automatically? I know we did it once | |
12:44 | ashimema | swagger-cli validate |
12:44 | that's a great command to use to check your specs magnuse | |
12:45 | magnuse | cool, i'll check that out! |
12:46 | Joubu | cait: IIRC there is a cronjob |
12:49 | cait | ok, so the numbers are just rising and rising :) |
12:51 | ashimema | Joubu |
12:51 | changes like bug 33490 | |
12:51 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=33490 normal, P5 - low, ---, jonathan.druart+koha, Signed Off , Agreements - Filter by expired results in error |
12:51 | ashimema | should I be seeing corresponding Cyprus tests for them? |
12:51 | * ashimema | really needs to get closer to more of this |
12:51 | ashimema | the code looks fine to me and works exactly as I expect.. was about to PQA as I have with a bunch of other Vue one's recently.. then wondered about tests |
12:52 | Joubu | tcohen: 33402 - one is left behind |
12:53 | tcohen: git mv t/cypress/integration/Searchbar_spec.ts t/cypress/integration/ERM/Searchbar_spec.ts | |
12:54 | marie-luce joined #koha | |
12:57 | caroline joined #koha | |
13:16 | caroline | Good morning! |
13:20 | cait | hi caroline :) |
13:21 | caroline | cait, did you get an answer from bernardo? |
13:24 | cait | not yet... |
13:25 | caroline | :/ |
13:25 | cait | I'll forward to you, but you are CC too, so I hope he would reply to us both |
13:25 | I am preparing for training and we have manual links everywhere that lead to untranslated manual :( | |
13:27 | caroline | That is my life... I don't have time to translate the manual, so all the links are in english. I put more time in our knowledge base |
13:28 | cait | we were at a pretty good state with 18.11... but now with 22.11 ... not |
13:35 | caroline | If anyone is interested in clubs, hammat did a development to see and change answer to enrolment questions bug 25079 |
13:35 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=25079 enhancement, P5 - low, ---, hammat.wele, Needs Signoff , Show club enrollment question answers in staff client |
13:41 | cait1 joined #koha | |
13:46 | marcelr | if someone could have a look at bug 33360, how nice would that be :) |
13:46 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=33360 enhancement, P5 - low, ---, m.de.rooy, Needs Signoff , SendQueuedMessages: Improve limit behavior and add domain limits |
14:03 | thd joined #koha | |
14:04 | paulderscheid[m] | next meeting? |
14:05 | cait1 | oh right |
14:05 | in an hour I think | |
14:05 | next meetings? | |
14:05 | no wahanui | |
14:05 | paulderscheid[m] | why is this broken? |
14:05 | Ah | |
14:05 | caroline | I was gonna say wahanui is on vacation |
14:06 | I feel like it's been a while since I've seen them (it? him? her?), are they broken for good? | |
14:06 | tcohen | who hosts it? |
14:06 | thd | I had also thought that it was in an hour but the meeting may be set to now. |
14:07 | caroline | In the calendar, it's in a hour |
14:08 | tcohen | can we do it one hour earlier next time? |
14:08 | https://wiki.koha-community.or[…]Next_IRC_meetings | |
14:10 | thd | The time and date converter link is not working correctly for me just now. |
14:10 | caroline | https://wiki.koha-community.or[…]te_Administration I don't see wahaui here, but it says irc is hosted on bugs.k-c.org |
14:10 | thd | It is in an hour as I had expected, the converter link is giving me now. |
14:37 | caroline | probably due to daylight savings nonsense |
14:39 | Dyrcona | I had a look. The converters says EST for me. My current timezone is EDT. So the time zone converter is ignoring daylight savings time. |
14:42 | caroline | Daylight savings time, confusing the heck out of everyone since 1908! |
14:42 | tcohen | we should abolish timezones as well |
14:43 | Dyrcona | I'd rather stay on standard time, but most surveys indicate that people prefer going to daylight savings time in the USA. |
14:43 | UTC FTW! | |
14:45 | tcohen | we could also change the meaning of a second of time to avoid the leap year non-sense |
14:45 | marcelr | got a nice daylight saving time test thing on 33360 btw |
14:45 | caroline | In that case, could we go to a base-10 system? |
14:46 | tcohen | bug 33360 |
14:46 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=33360 enhancement, P5 - low, ---, m.de.rooy, Needs Signoff , SendQueuedMessages: Improve limit behavior and add domain limits |
14:46 | tcohen | huginn: ? |
14:46 | huginn | tcohen: I'll give you the answer as soon as RDA is ready |
14:46 | caroline | lol marcelr! |
14:46 | marcelr | when doing a search back and i hit an unexisting hour |
14:46 | in my timezone | |
14:46 | caroline | here's a better one with DST bug 32232 |
14:46 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=32232 enhancement, P5 - low, ---, koha-bugs, NEW , Koha crashes if dateofbirth is 1947-04-27, 1948-04-25, or 1949-04-24 |
14:47 | marcelr | nice |
14:47 | tcohen | nice one |
14:47 | Joubu | this was a bug in DateTime, that I think was fixed... |
14:48 | it was Sao Paulo at the time.. | |
14:48 | bug 23079 | |
14:48 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=23079 major, P5 - low, ---, jonathan.druart+koha, CLOSED FIXED, Checkouts page broken because of problems with date calculation (TZAmerica/Sao_Paulo) |
14:48 | marcelr | cool |
14:49 | Joubu | we fixed it koha-side, see the floating calculation from 23079 |
14:49 | caroline | Joubu, are you talking about mine or marcelr's? Mine broke between 21.05 and 22.05 (21.05 was fine, 22.05 error) |
14:50 | Joubu | caroline: yours, it's a DateTime issue, not really us. |
14:51 | The solution is on 23079 (at least the way of fixing it, that needs to be adjusted for the method playing with datetime calculation n 32232) | |
14:51 | caroline | ok thanks, I will pass on the info |
14:51 | I knew it was something outside of Koha, but I thought we could maybe not use the time in dateofbirth or something | |
14:52 | Joubu | the "floating" calculation is basically telling to not care about tz, what we want here |
14:54 | caroline | Joubu++ |
14:56 | marcelr | yeah, these things can popup anywhere when adding to dates in searches etc. (just like I had on 33360) |
14:58 | cu | |
14:59 | emlam joined #koha | |
15:02 | Joubu | tcohen: 33408 did you see the previous comment? |
15:03 | @later tell tcohen 33408 did you see the previous comment? | |
15:03 | huginn | Joubu: The operation succeeded. |
15:03 | tcohen | Joubu: are they ok now? |
15:03 | Joubu | I think so, we discussed it recently but I don't remember when/where |
15:04 | tcohen | so no one got assigned the task of removing the QA check? |
15:04 | Joubu | I didn't find an issue either |
15:10 | ashimema | I was going to raise that |
15:10 | whenever the meeting it | |
15:11 | emlam | I think it's supposed to be now, actually... |
15:11 | https://wiki.koha-community.or[…]ing_12_April_2023 | |
15:12 | tcohen | hi, I cannot host it |
15:13 | emlam | tuxayo around? |
15:13 | reiveune | bye |
15:13 | reiveune left #koha | |
15:40 | thd | tuxayo: Are you chairing the meeting? Does the meeting need to be set to an hour earlier for local time changes? |
15:43 | cait1 | hm so now I missed the meeting? |
15:43 | thd | cait1: No |
15:44 | cait1: Perhaps you could call the meeting and chair. | |
15:44 | cait1 | not too keen on it tbh, I need to finish something here and the day is almost over |
15:45 | thd | The change in local time may not be working well. It may be better to have meetings set an hour earlier for the change in local time. |
16:00 | cait1 | does someone have a moment to test isbn search in item search? |
16:00 | this puzzles me | |
16:01 | we are searching for an existing isbn that is stored in biblioitems truncated (there are 2 isbn on the record) %isbn% | |
16:01 | nothing comes up | |
16:04 | callnumber works | |
16:04 | and it works in 22.11.03 but not in 22.11.04 | |
16:04 | caroline | I don't have a problem with ISBN in Item search |
16:04 | (in master) | |
16:04 | Oh wait, two isbn, let me edit my record | |
16:06 | No I still get the search result with two ISBN in the record | |
16:06 | cait1 | I wonder if it could somehow be configuration specific |
16:06 | but they map 020$a to biblioitems.isbn... and the field is filled in the db too | |
16:06 | and it worked pre-update | |
16:07 | caroline | In the db, my isbn field is 9781903436950 | 111222333 |
16:07 | when I search for %9781903436950% in item search it's found | |
16:08 | do I do something different? | |
16:08 | cait1 | no :( |
16:08 | that's exactly what I expect, but it's nto working here | |
16:09 | the record isin the catalog, it has items | |
16:10 | thx for trying to help caroline | |
16:10 | caroline | does it find anything at all or just not the one you're looking for? |
16:11 | cait1 | nothing |
16:11 | caroline | and with other fields? |
16:12 | cait1 | callnumber works |
16:12 | caroline | something not in items? |
16:12 | Like author or something? | |
16:12 | cait1 | yes, had the same idea, sec |
16:14 | caroline | exact search when there is only one isbn? |
16:14 | cait1 | sutck with processing and no response from server |
16:14 | caroline | oops |
16:14 | cait1 | for a title search in a specificlibrary |
16:15 | caroline | do the logs give you any clue? |
16:17 | cait1 | no access to the logs on this machine |
16:17 | I just called, they are looking into it | |
16:17 | tcohen | is anyone available for the dev meeting? |
16:17 | cait1 | I'd be here I guess - waiting for the server to come back (we shoudl be able to do bad things with item search) |
16:21 | paulderscheid[m] | I'm here |
16:39 | Oak joined #koha | |
16:50 | paulderscheid[m] | Should we at least postpone the meeting? |
16:51 | Or do we have the permissions issue again? | |
17:01 | Can I ask you a quick question @tcohen> | |
17:02 | * Can I ask you something real quick tcohen | |
17:02 | cait joined #koha | |
17:02 | cait | i don't know how to runt he script yet |
17:02 | maybe we need to resolve that tomorrow | |
17:04 | cait left #koha | |
17:05 | paulderscheid[m] | You there Pedro Amorim ? |
17:06 | PedroAmorim[m] | yes |
17:08 | paulderscheid[m] | I have a Koha::Object where I want to filter the resultset and then pass to $c->objects->search. |
17:08 | How do I have to format the conditions? | |
17:08 | Like... (full message at <https://matrix.org/_matrix/med[…]KeLTaqGwUPVBrZjZn>) | |
17:09 | Before you ask: I want the $c->objects->search mainly for support of _page, _per_page but the handling of the query params within the second call seems to make a mess. | |
17:18 | PedroAmorim[m] | when you say "how to format the conditions" you're asking how $search_params should look like? |
17:18 | this may be a good reference: | |
17:18 | https://metacpan.org/pod/SQL::[…]parison-operators | |
17:18 | what query exactly are you trying to do | |
17:19 | paulderscheid[m] | Yeah, I was looking at these docs.... (full message at <https://matrix.org/_matrix/med[…]iAISCAQyytaINkRfZ>) |
17:20 | They are added conditionally based on whether they're actually passed. | |
17:23 | And I wanted to reuse that because my frontend components are geared towards plain query params atm | |
17:25 | PedroAmorim[m] | this is how I built a query with ternary: |
17:25 | https://github.com/Koha-Commun[…]llrequests.pm#L65 | |
17:25 | does that help? there are other examples in the code but this is the one on top of my head | |
17:27 | more cool examples here: | |
17:27 | https://github.com/Koha-Commun[…]/ERM/Documents.pm | |
17:28 | sorry if I didn't understand your question fully, others may be of much better help than me! (i have no idea what I'm doing really) and it's been a long day! :D | |
17:29 | paulderscheid[m] | Thanks so much! Sorry for sneaking up on you ^^ |
17:29 | I'm looking at them right now. Maybe I'll find something in there. | |
17:36 | I think the problem is actually the $c->objects->search call | |
17:36 | Without it everything works as expected. | |
17:37 | tcohen | paulderscheid[m]: $c->objects->search does more than pagination |
17:37 | paulderscheid[m] | Yeah, I know |
17:37 | tcohen | it also sets some headers that are required for the datatables wrapper |
17:37 | paulderscheid[m] | I'm trying to grasp what it does |
17:37 | tcohen | so, what you might want to do |
17:37 | is fill the missing request params | |
17:38 | what is the query you're trying to build? | |
17:38 | paulderscheid[m] | > <@paulderscheid:matrix.org> Yeah, I was looking at these docs.... (full message at <https://matrix.org/_matrix/med[…]XLYeyaoMPbgnOzxmC>) |
17:39 | tcohen | Here you can find the helper definition: Koha::REST::Plugin::Objects |
17:39 | It basically uses the helpers from Koha::REST::Plugin::Query | |
17:39 | paulderscheid[m] | I already read it, but I think I misunderstood how it works |
17:39 | tcohen | to extract reserved stuffs from the original request query params |
17:39 | cait joined #koha | |
17:40 | tcohen | we can have a call whenever you want |
17:40 | and we talk about it | |
17:40 | do you know what a 'resultset' is? | |
17:40 | paulderscheid[m] | Yeah, I read the docs |
17:40 | tcohen | so it is not the results from a DB query |
17:41 | but an object representing a query, sort of | |
17:41 | and every time you call ->search | |
17:41 | it generates a new resultset object | |
17:41 | stacking queries one on top of the other | |
17:41 | so $thing->search(ç) | |
17:42 | $things->search({ param => [1,2] })->search({ param => 2 }) will return a new resultset | |
17:43 | so what I think Pedro was saying is | |
17:43 | that you can 'build a resultset' in the controller, and pass it to $c->objects->search | |
17:43 | but in order for things to work for your case, you might need to actually | |
17:44 | do things like $c->req->param( 'q', encode_json({ param => [1,2] }) ) | |
17:45 | paulderscheid[m] | Ah, that's a new angle |
17:45 | tcohen | I'm not saying it is required |
17:45 | but it is a POV | |
17:45 | paulderscheid[m] | Thanks for clearing that up for me |
17:45 | I kind of understood how it works before but that really helped | |
17:45 | tcohen | excellent |
17:46 | hope I didn't mess your understanding heh | |
17:46 | paulderscheid[m] | Nah |
17:47 | tcohen | another example |
17:47 | https://git.koha-community.org[…]/Checkouts.pm#L53 | |
17:49 | paulderscheid[m] | Wait, I think I just got something important!... (full message at <https://matrix.org/_matrix/med[…]eGGriCIlNcWjdkmEh>) |
17:49 | Or is this unrelated? | |
17:50 | tcohen | that's important |
17:51 | paulderscheid[m] | The search call on $c->objects->search uses the query params itself, does it? |
17:51 | tcohen | what we do there, is assess the fact not-reserved query parameters will be passed as 'filters' to the query |
17:51 | so GET /patrons?patron_id=123 | |
17:51 | will imply a Koha::Patrons->search({ patron_id => 123 }) | |
17:52 | paulderscheid[m] | That's exactly the problem that I have! |
17:52 | tcohen | so the trick there, is to pick the 'know-by-the-controller' parameter, remove it from the query, use it to prepare the resultset, and continue |
17:53 | paulderscheid[m] | Thank you so much! That's it! |
17:53 | That was the behaviour I encountered, but I didn't fully understand why it was occuring. | |
17:54 | I owe you one. tcohen++ | |
18:03 | bag joined #koha | |
18:05 | paulderscheid[m] | Works like a charm now! :D Thanks again Pedro Amorim & tcohen |
18:11 | Maybe we can have a call when I advanced a little on my scaffolding thing tcohen. I have so many things to ask ^^ | |
18:11 | Have a nice evening y'all o/ | |
18:33 | tcohen | \o |
18:39 | cait | hm my ktd won't start :( |
18:39 | koha-db-1 | 2023-04-12 18:39:18+00:00 [Note] [Entrypoint]: Entrypoint script for MariaDB Server 1:10.5.19+maria~ubu2004 started. | |
18:39 | Error response from daemon: network 925aa9f4d6d5c79a156c691fa8f9389569111ac0fa87fe8de4fd1c4b3f59f7aa not found | |
18:47 | davidnz | my KTD starts (but I had done git pulls for koha and ktd, and a ktd pull) - maybe disk space issue? |
18:51 | tcohen | cait: it must be something local |
18:51 | ktd down a couple times | |
18:51 | maybe a ktd pull too? | |
19:51 | cait | tcohen: i just did a git pull and a ktd pull before running into that... |
19:52 | looking better without --selenium | |
19:54 | ... so not me looking into the test failures tonight | |
20:03 | * cait | patches bug 33509 instead |
20:03 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=33509 minor, P5 - low, ---, katrin.fischer, Needs Signoff , Staff search result list shows "other holdings" with AlternateHoldingsField when there are no alternate holdings |
20:08 | aleisha joined #koha | |
21:17 | cait | woudl be really nice if someone could take a look at bug 33506 and 33506 |
21:17 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=33506 normal, P5 - low, ---, katrin.fischer, Needs Signoff , Series has wrong index name on scan index page and search option selection is not retained |
21:17 | cait | bug 33506 |
21:18 | bug 33505 | |
21:18 | huginn | 04Bug https://bugs.koha-community.or[…]_bug.cgi?id=33505 minor, P5 - low, ---, katrin.fischer, Needs Signoff , Improve styling of scan index page |
22:01 | caroline | Does someone know what I have to activate in order to see the item_priority column in the holdings table of the OPAC? |
22:01 | I tried choosing priority in OPACShowHoldQueueDetails, but it just shows in the item_holds column with the number of holds, not in the item_priority column | |
22:02 | https://snipboard.io/8LDr5R.jpg | |
22:03 | this is the column I'm looking for https://snipboard.io/5cF39K.jpg | |
22:04 | cait | hm |
22:04 | i think the priority might only show in the patron account | |
22:04 | not in the opac | |
22:04 | oh | |
22:05 | caroline | It does show if you're logged in, but in the same column as the holds |
22:05 | cait | trying something |
22:07 | [% IF holds_count.defined %] | |
22:07 | <th id="item_holds" data-colname="item_holds">Item holds</th> | |
22:07 | [% ELSIF show_priority %] | |
22:08 | <th id="item_priority" data-colname="item_priority">Item hold queue priority</th> | |
22:08 | [% END %] | |
22:08 | I am not sure if that makes sense | |
22:09 | caroline | I tried reading that part, but don't know TT well enough to understand if I understand ;) |
22:09 | cait | when there are holds... the column never shows? but if there are no holds... hter eis no need for priority.. |
22:09 | I thin it never gets into the ELSE | |
22:09 | becuase holds_count is always defined | |
22:10 | caroline | I can file a bug tomorrow. I just wanted to make sure that it was not something completely unrelated to holds that I was missing |
22:11 | cait | yeah i think there is a bug |
22:11 | defined should be count | |
22:12 | it's always defined, but not always more than 1 | |
22:20 | i think i have osmething like a patch if you file now | |
22:22 | hm but maybe better to check that in daylight :) | |
22:22 | bye all | |
23:39 | aleisha | hello all, does anyone do sso and user provisioning with shibboleth for multiple IDPs? weve found some stuff on an embedded discovery service, just want to check we're on the right track |
← Previous day | Today | Next day → | Search | Index