IRC log for #koha, 2023-06-23

All times shown according to UTC.

Time S Nick Message
05:37 caroline_catlady joined #koha
05:41 caroline_crazycatlady joined #koha
06:00 marcelr joined #koha
06:01 marcelr hi #koha
06:28 reiveune joined #koha
06:30 reiveune hello
06:31 NikolayGospodinov[m] Hello
06:31 NikolayGospodinov[m] may i ask a question
06:32 NikolayGospodinov[m] I am working with a legacy Koha and have no connection to the previous system administrator. The problem is that my user doesn't have all the rights and the Set Permissions option is locked. How can I deal with this problem?
06:45 marcelr NikolayGospodinov[m]: change your borrowers.flags to 1 then you are system librarian
06:45 wahanui marcelr: that doesn't look right
06:45 wahanui joined #koha
06:46 marcelr wahanui
06:46 wahanui yes, marcelr?
06:51 alex_ joined #koha
06:51 alex_ Bonjour
06:51 wahanui what's up, alex_
06:58 thibaud_g joined #koha
07:06 cait joined #koha
07:21 * cait waves
07:21 cait good morning all
07:32 ashimema morning cait
07:34 cait morning
07:34 cait hm no frido
07:44 marcelr hi cait alex_ magnuse
07:44 marcelr and ashimema
07:44 wahanui ashimema is probably back on security bugs today.. sorry, it's been a bit of a crazy week
07:44 cait hi marcelr
07:44 cait it's true
07:44 cait today is CSRF day
07:44 cait ashimema: ? heh
07:44 magnuse hiya marcelr et al
07:44 marcelr oh i just did some major bugs :)
07:46 cait i jumped right into the acq one
07:46 cait but filing 2 new ones I found while testing :(
07:47 marcelr that pattern I know
07:48 marcelr i am not sure how we could do csrf together now with one bug
07:48 marcelr which needs rebasing etc
07:49 cait I am nto sure what tcohen had in mind there - but it sounded like some kind of framwork bug that needed feedback probably?
07:55 marcelr looks like another postpone :)
07:57 ashimema Just on school run
07:57 ashimema Back shortly and will explain
07:59 ashimema I'll rebase once home, there a script in it for identifying possible issues. We need to work through them all
07:59 ashimema I'm royally fed up of this bug..
08:02 cait today is the chance :)
08:02 cait I probably can't help much witht he main one, but when you need me for the smaller ones let me know, I'll be around for QA today
08:17 ashimema hmm
08:18 ashimema the issue is.. it's a huge job.. I need more than just me coding on it
08:18 ashimema if we're going to refuse to push it unless it's 100% coverage then we still have 80% of the work to do
08:20 cait maybe talking it through with marcelr would help? (just think I won't understand much)
08:23 marcelr ashimema: we probably split stuff into patches of a decent size ?
08:24 ashimema I tried that
08:24 ashimema and will try again
08:24 ashimema just opening up the bugs now to remind myself where it's got
08:25 ashimema even split.. there's been a refusal to push any of it without all of it.
08:25 ashimema that's the core issue
08:25 * ashimema is tired
08:26 marcelr hmm i opt for breaking it up and just push incremental
08:26 marcelr better than noting
08:26 marcelr nothing
08:26 marcelr if we would have done so, it would have been ready?
08:27 ashimema bug 30524 discussion is a good starting place
08:27 huginn Bug https://bugs.koha-community.or[…]_bug.cgi?id=30524 is not accessible.
08:34 marcelr ok ashimema it applies, will have a look now
08:34 marcelr lets get that further
08:35 marcelr april 2022
08:35 marcelr and june 2021
08:35 ashimema ho, it still applies
08:35 marcelr yes
08:35 marcelr good work
08:35 ashimema I just went to rebase my local branches and it had conflicts.. haha
08:36 ashimema excellent, David has maintained it 🙂
08:38 marcelr only 397 missing templates
08:38 marcelr we're almost there
08:39 marcelr there is no module test?
08:39 marcelr for Auth or Output ?
08:40 ashimema darn.. you're right
08:40 ashimema I could have swarn I'd written those.. maybe that got lost in the split
08:40 ashimema either way.. your right.. then need writing
08:43 marcelr is there a small follow-up patch for testing this on a staff page
08:43 marcelr >
08:43 marcelr ?
08:43 marcelr i understand we need to probably add output_exit calls ?
08:45 marcelr i am only seeing NEW reports nothing more?
08:46 ashimema 30502 is the one with most of the existing fixes to date
08:46 ashimema it's a bit of a mess ☹️
08:48 ashimema sorry.. I meant but 22990
08:49 marcelr let me try that one
08:49 marcelr are you writing a test now ?
08:50 ashimema I was  just going to check if 22990 actually applies
08:50 ashimema and of course.. it doesn't
08:51 marcelr no but let me rebase the first patch or try
08:52 ashimema sure.. I'll get to writing a test then 🙂
08:52 ashimema thanks
09:27 magnuse joined #koha
09:28 marcelr ashimema: i am going to squash a bit to get more organized on 22990
09:28 ashimema sounds like a plan
09:39 marcelr ok ashimema 22990 is ready
09:39 ashimema nice one
09:40 marcelr whats next
09:40 wahanui i guess next is Koha bibliography in Zotero
09:40 ashimema still working on the tests here
09:40 ashimema output_and_exit_if_error has no tests at all right now ! 🙂
09:40 marcelr so nice :)
09:40 marcelr dont make it to hard now
09:40 marcelr too
09:40 ashimema it's not too bad
09:41 ashimema just rminding myself how it works.. it's been a while
09:41 marcelr there is no other reports with csrf patches yet ?
09:41 marcelr only new entries
09:42 ashimema I think that's the case
09:43 ashimema I have a nagging feeling the script to help identify the issues isn't perfect.. not sure why I have that feeling
09:43 ashimema might be worth a little look to see if you agree with the logic in it
09:44 marcelr ok
09:44 ashimema cheers
09:47 PedroAmorim[m] marcelr: I can give bug 33404 a look if no one else is already looking at it
09:47 huginn Bug https://bugs.koha-community.or[…]_bug.cgi?id=33404 major, P5 - low, ---, m.de.rooy, Signed Off , Authorities imported from Z39.50 in encodings other than UTF-8 are corrupted
09:47 marcelr that would be very nice PedroAmorim[m]
09:56 * cait waves - sorry got pulled away, but iwll be back after a quick lunch break now
09:57 marcelr ashimema: algorithm looks reasonable; add small followup
09:58 marcelr ashimema: should we add a omnibus for those patches since 22990 was meant for that but includes patches itself ?
10:02 ashimema seems sensible to me
10:03 ashimema this test is proving 'fun' to write
10:03 marcelr ok bug 34410 is the new omnibus
10:03 huginn Bug https://bugs.koha-community.or[…]_bug.cgi?id=34410 could not be retrieved: InvalidBugId
10:03 ashimema I misread the code  a bit to start with.. seems I need to mock a Template
10:03 marcelr yeah i would expect some advanced mocking with Auth and Output
10:04 marcelr but you can do it :)
10:07 alex_ joined #koha
10:10 marcelr htg, see your test later ashimema; feel free to change status to PQA on 30524
10:11 ashimema 🙂
10:31 khall joined #koha
10:34 ashimema ok.. trapping exit's is 'hard' to do manually
10:34 ashimema this test may actually require a new test module too
10:34 ashimema Test::Trap
10:41 PedroAmorim[m] marcelr: around?
11:01 cait marcelr++ # just catching up on your morning's work
11:02 ashimema unit test uploaded
11:02 ashimema I'm sure we could cover more.. but it's a basic start
11:19 alex_ joined #koha
11:35 oleonard joined #koha
11:37 oleonard o/
11:40 cait hi oleonard
11:40 wahanui hi oleopard
12:00 tcohen hola #koha o/
12:01 tcohen CSRF day ~o~
12:02 oleonard Oh no I forgot to wear my CSRF hat to work
12:02 tcohen bad oleonard
12:03 * tcohen says while putting his styrofoam crown in his head
12:05 oleonard :D
12:06 tcohen cait: did fridolin mention anything about the multi-receive patches and the 23.05 point release?
12:14 cait he was not around earlier when I looked for him
12:14 cait but I alerted him, maybe he'll realease to night and grab them if you push
12:14 tcohen so I'll push then
12:15 tcohen ashimema, marcelr about the CSRF base bug, are you working on having it PQA? I noticed it got FQA
12:15 * ashimema is failing to write tests
12:15 ashimema well.. i wrote tests.. but the introduced a new test dependancy
12:15 ashimema marcelr came up with an idea on using a mock instrad
12:15 tcohen marcelr: can you help?
12:15 ashimema but my attempts thus far have failed
12:16 tcohen I can bring alfajores to Helsinki if it helps
12:18 ashimema I'm in the middle of an audit here.. and we need to role the release
12:18 tcohen khall, marcelr: about 33847 aren't we supposed to not rely on Koha::* on dbrevs, as they can change in the future and break older dbrevs?
12:18 ashimema so I'm struggling for time again ☹️
12:18 tcohen ashimema: you're gonna do good
12:19 ashimema the audit is going fine so far.. not long until the results
12:19 ashimema I'm just struggling to balance stuff at the minute.. as usual
12:19 ashimema need to step back away from csrf.. we need more team still.. marcelr has been great.. but there's still lots to be done.
12:24 tcohen +1
12:39 emlam joined #koha
13:00 PedroAmorim[m] cait++
13:00 PedroAmorim[m] thanks for amazing input on bug 30719 I'll follow-up with it in the next couple days I hope
13:00 huginn Bug https://bugs.koha-community.or[…]_bug.cgi?id=30719 enhancement, P5 - low, ---, pedro.amorim, Failed QA , ILL should provide the ability to create batch requests
13:02 cait I didn't dare too test much more because of failing tests etc. will start witht he GUI side with the follow-ups
13:07 Dyrcona joined #koha
13:08 tcohen oleonard: around?
13:08 oleonard yep
13:09 tcohen what is the fa-edit replacement in v6?
13:09 tcohen if any
13:10 ashimema yikes: https://fontawesome.com/search?q=edit&o=r
13:11 ashimema pen maybe?
13:12 tcohen I'm looking at a pre v6 bug
13:12 ashimema fa fa-pencil is being used for some other Edit buttons
13:12 tcohen that introduces fa fa-edit
13:12 tcohen and trying to help them
13:12 oleonard In the upgrade patch we switched to fa-pencil-alt, but it looks like the 'alt' might be unnecessary
13:13 tcohen ashimema++
13:13 tcohen oleonard++
13:14 oleonard Looks like fa-pencil-alt may have been a 5.x name only
13:14 * ashimema wonders waht the 'alt' is?
13:14 ashimema hehe
13:14 ashimema makes sense
13:14 tcohen I would expect "fa-alt fa-pencil" with the new style
13:15 tcohen git grep 'fa\-pencil'
13:15 tcohen suspiciously doesn't return anything but the fontawesome css
13:15 tcohen ah no
13:15 tcohen nevermind
13:15 tcohen thanks!
13:19 tcohen https://fontawesome.com/v5/ico[…]f=classic&s=solid
13:19 tcohen when you click on 'latest' it takes you to
13:19 tcohen https://fontawesome.com/icons/[…]f=classic&s=solid
13:20 oleonard I think .fa-solid.fa-pencil is correct, and I'm preparing a patch
13:21 tcohen aweosme
13:21 tcohen there are still a bunch of "fa fa-edit" in the intranet
13:21 tcohen will you take care of those?
13:22 oleonard Yes
13:22 tcohen oleonard++
13:22 tcohen cc me please
13:22 ashimema is our git down
13:23 tcohen I pushed a couple times
13:23 tcohen maybe it is overloaded by jenkins
13:23 tcohen worked for me
13:24 ashimema oh.. back up now
13:24 ashimema weird
13:27 ashimema it's just very very slow
13:28 tcohen ashimema: ....
13:28 tcohen some old decisions will hit us back
13:29 ashimema ?
13:29 tcohen git grep ui_method_icon
13:29 ashimema wow
13:29 ashimema I don't remember that stuff at all
13:29 ashimema yikes
13:29 tcohen the ILL status graph
13:30 tcohen includes a way to specify the icon to be used
13:30 ashimema indeed
13:30 ashimema oop.. meeting time
13:30 ashimema brb
13:30 tcohen and we assume the initial 'fa ' will always be fa
13:36 PedroAmorim[m] https://git.koha-community.org[…]ha-community/Koha
13:36 PedroAmorim[m] this is not opening for me, anyone else?
13:37 tcohen I iwll check the server
13:37 tcohen it has a load of 13
13:38 tcohen :-D
13:38 ashimema interesting
13:38 tcohen something is hitting the web interface
13:38 ashimema doesn't load for me either
13:39 oleonard Maybe Facebook is trying to assimilate us into the Metaverse®
13:39 ashimema lol
13:39 caroline_crazycatlady lol!
13:41 tcohen will do an upgrade of the docker tools
13:41 tcohen and reboot
13:42 ashimema thanks tcohen
13:44 tcohen back up
13:44 PedroAmorim[m] 👏
13:44 PedroAmorim[m] tyvm
13:45 ashimema nice one
13:50 cait2 joined #koha
13:50 cait2 left #koha
13:54 cait2 joined #koha
13:54 cait2 back
13:59 ashimema anyone here played with 'Plane'
14:00 ashimema the OSS Jira alternative?
14:01 cait first time I hear about it :(
14:01 cait but sounds interesting in general
14:01 ashimema it's very new as far as I can tell
14:01 ashimema I'm wondering if it might fit community well..
14:02 cait if it#s very new we might want to let it settle a bit :)
14:02 cait but something to watch maybe?
14:20 ashimema indeed
14:46 oleonard ashimema: Did you ever experiment with Select2 for patron search autocomplete?
14:46 ashimema well
14:47 ashimema I didn't move any further yet
14:47 cait oleonard: there is something partial in our ILL backend maybe
14:47 ashimema but I do have a select2 one in the bookings submission
14:47 cait hm but it might be too specific to our use case, so not as helpful (thinking about it)
14:47 ashimema not added the further adaptations yet
14:47 oleonard it would be so nice to use an asset we already have
14:48 ashimema just discovered more and more issues with our other patron searches though 😜
14:48 oleonard Luckily I only care how it looks ;)
14:48 ashimema yeah
14:48 ashimema it would be nice
14:48 ashimema maybe I can swing back around to it again next week
14:51 cait what's the state of the C... bugs now?
14:51 ashimema ?
14:52 caroline C... sounds like you're swearing XD
14:52 oleonard ashimema: Remember, our plan to rewrite Koha in C?
14:52 ashimema lol
14:52 cait I just never get the sequence right
14:52 cait CSRF?
14:52 ashimema lol
14:53 ashimema I moved on to release
14:55 cait oleonard: shoudl we add the remove of jquery ui to the roadmap?
14:56 oleonard It wouldn't hurt, though it's been a  long road already XD
15:00 bag joined #koha
15:06 cait oleonard: I've added it https://wiki.koha-community.or[…]Roadmap_for_23.11
15:07 oleonard Thanks cait!
15:08 cait kidclamp: kidclamp kidclamp
15:10 oleonard :D
15:10 reiveune bye
15:10 cait felt worth trying :)
15:10 reiveune left #koha
15:16 cait oleonard: found 2 small issues witht he tabs for frameworks - filed 34115
15:20 ashimema cait
15:20 ashimema do you have the power to add users to the website?
15:20 ashimema Pedro has been rolling his release and is at the stage of adding the post to the website in a minute
15:21 ashimema I may just save the announcement for Monday.. running out of work day here
15:21 cait I believe i might not - but I can check
15:21 ashimema I'll ask Liz Monday
15:22 cait i think i can
15:22 cait let me see
15:22 cait PM?
15:28 PedroAmorim[m] cait++
15:28 cait yay karma!
15:28 cait @karma
15:28 huginn cait: Highest karma: "Joubu" (1206), "cait" (1015), and "ashimema" (832).  Lowest karma: "-" (-84), "failed" (-53), and "ie" (-41).  You (cait) are ranked 2 out of 1248.
15:28 cait PedroAmorim++
15:47 oleonard cait: I had no idea you could search a framework with a subfield and get straight to the subfield edit form!
15:49 cait I QA'd it when Joubu added it I think
15:49 cait it is very helpful
15:49 cait maybe we should add a hint :)
15:50 lukeg joined #koha
15:50 lukeg Hi
15:50 cait hi lukeg :)
16:02 h2h2[m] joined #koha
16:06 h2 joined #koha
16:09 philor joined #koha
16:13 khall joined #koha
16:26 cait have a nice weekend #koha!
16:27 cait left #koha
16:32 khall joined #koha
17:10 lukeg joined #koha
17:22 cait joined #koha
17:25 khall joined #koha
18:10 khall joined #koha
18:53 khall joined #koha
19:08 khall_ joined #koha
19:56 lukeg joined #koha
20:18 khall joined #koha
22:02 khall joined #koha
23:06 lukeg joined #koha
23:40 khall joined #koha

| Channels | #koha index | Today | | Search | Google Search | Plain-Text | plain, newest first | summary