IRC log for #koha, 2020-08-21

All times shown according to UTC.

Time S Nick Message
00:09 khall joined #koha
00:28 GeekRuthie joined #koha
00:29 AndrewFH joined #koha
00:34 cait1 joined #koha
00:44 khall joined #koha
00:50 AndrewFH joined #koha
00:52 inlibro joined #koha
01:11 wizzycray joined #koha
01:52 inlibro joined #koha
02:02 khall joined #koha
02:07 khall joined #koha
02:27 irma_ joined #koha
02:52 inlibro joined #koha
03:53 inlibro joined #koha
03:54 khall joined #koha
04:32 khall joined #koha
04:53 inlibro joined #koha
04:59 chriss joined #koha
05:07 khall joined #koha
05:53 inlibro joined #koha
06:00 lmstrand joined #koha
06:06 khall joined #koha
06:08 TimothyAlexis joined #koha
06:08 TimothyAlexis I've looked at the password system and processes for changing and recovering passwords.
06:11 TimothyAlexis I'm not sure where to begin...
06:17 TimothyAlexis We require users to have a 4 digit PIN as password, so that they can use our automatic lending machines. I have added logic for that. I noticed some things while doing this, which I think need to be corrected in the community version.
06:27 reiveune joined #koha
06:27 reiveune hello
06:30 TimothyAlexis I think the password validation should be consistent in all different situations, and also that the jQuery validator should replace the other stuff, as it makes that other stuff obsolete. Or am I missing something in this conclusion?
06:45 cait joined #koha
06:49 TimothyAlexis related files: password_check.inc, member-password.tt, memberentrygen.tt, password_check.inc, opac-passwd.tt and opac-password-recovery.tt
06:52 Joubu TimothyAlexis: I don't understand your question, but I know there is bug 23816 in the queue that is modifying a bit this code
06:52 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=23816 enhancement, P5 - low, ---, agustinmoyano, Failed QA , Allow to have different password strength and length settings for different patron categories
06:53 inlibro joined #koha
07:02 cait joined #koha
07:03 cait1 joined #koha
07:03 lds joined #koha
07:03 lds hi
07:05 cait1 hi lds and #koha
07:07 cait1 lds++ thx for the new mailing list server
07:12 cait joined #koha
07:13 TimothyAlexis Joubu well, I'm not sure I am ready to ask a question yet.
07:14 TimothyAlexis There is one validation for passwords in forms, using jQuery validator.
07:14 TimothyAlexis Then there is some other validation in Perl.
07:15 Joubu yes
07:15 Joubu because browser validation can be bypassed by the user
07:16 TimothyAlexis OK
07:17 TimothyAlexis If I want to add a criteria to that process, where would I do that?
07:18 Joubu both sides
07:18 TimothyAlexis if $_->isa('Koha::Exceptions::Password::TooShort');
07:18 TimothyAlexis Is this from the Auth module?
07:18 Joubu a JS check and one in the perl module
07:19 Joubu Koha::AuthUtils::is_password_valid
07:21 TimothyAlexis Thank you
07:23 TimothyAlexis Then the two questions are:
07:23 TimothyAlexis Should I add a bug report regarding consistent use of jQuery validator in the forms: member-password.tt, memberentrygen.tt, opac-passwd.tt and opac-password-recovery.tt?
07:23 TimothyAlexis Is it possible to use the same password_check.inc in both opac and intranet? (they have the same content)
07:25 Joubu The password check should be done with the include file "add_password_check", everywhere (once again, don't forget 23816)
07:26 Joubu There is no way to share template files between the 2 interfaces so far
07:27 ashimema[m] https://imgur.com/2BR5QCY
07:28 ashimema[m] why are those tabs all different in the way they display counts
07:28 Joubu for inconsistencies
07:28 ashimema[m] lol
07:28 Joubu (:D)
07:31 TimothyAlexis So is the [% PROCESS 'add_password_check' new_password => 'newpassword' %] is done with Koha::AuthUtils::is_password_valid?
07:31 * ashimema[m] submits bug
07:31 magnuse consistency is boring
07:31 cait1 ashimema[m]: your bug is a duplicate :)
07:31 kohaputti joined #koha
07:32 magnuse kia ora kohaputti
07:32 ashimema[m] grr
07:32 Joubu TimothyAlexis: no! You are template side there
07:32 ashimema[m] I didn't find one
07:32 kohaputti hi
07:32 cait1 hi kohaputti :)
07:32 TimothyAlexis magnuse: ^_^'
07:32 ashimema[m] got a bug number cait1?
07:32 cait1 looking ashimema[m]
07:32 ashimema[m] :)
07:33 Joubu @later tell kidclamp you are aweome!
07:33 cait1 26264
07:33 huginn Joubu: The operation succeeded.
07:33 Joubu @later tell kidclamp you are awesome! (even)
07:33 huginn Joubu: The operation succeeded.
07:33 magnuse bug 26264
07:33 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=26264 enhancement, P5 - low, ---, oleonard, NEW , Styling of 'counts' on patron details tabs is inconsistent
07:33 ashimema[m] that's the one I just added
07:33 cait1 ah tht is yours... there is another
07:33 ashimema[m] hehe
07:33 cait1 yep, still on it
07:34 Joubu what do you suggest ashimema[m]?
07:34 cait1 26258 - although it might be a little different after all
07:34 Joubu they have different meaning
07:34 TimothyAlexis OK, so the add_password_check is a block in password_check.inc
07:34 Joubu there is: 1. X something, 2. x bad y good claims, 3. x out of y clubs
07:34 ashimema[m] lol
07:35 Joubu (not sure about clubs however)
07:35 ashimema[m] looks like they've changed since that bug was added
07:35 ashimema[m] lol
07:35 ashimema[m] I would say make them all postfix and in brackets
07:35 Joubu you cannot write, for 2, x/y claims
07:35 ashimema[m] I'm not close enough to claims to know what they mean
07:36 fridolin joined #koha
07:36 Joubu it's X open claims, Y closed
07:37 cait1 i was planning on hiding the closed ones
07:37 Joubu if you look at 26258, that how claims where displayed, like clubs (x/y) and the last change replaced it with the staring eyes
07:37 cait1 with the id :)
07:37 ashimema[m] Checkouts (1) | Holds (2) | Claims (1/3) | Restrictions (0) | Clubs (1/0)
07:37 Joubu ashimema[m]: yes, but I just explained before why it cannot work for claims ;)
07:37 cait1 i don't see why you need the number of all clubs actually
07:38 ashimema[m] I don't understand your point
07:38 cait1 instead of () we could use the claims formatting for all mybe
07:38 cait1 for holds it could be waiting and total, that wuld make sense
07:38 ashimema[m] the two eyes don't make any sense to em
07:39 ashimema[m] is it the tooltips that are important?
07:39 ashimema[m] is that what you are saying?
07:39 Joubu they get green and red
07:39 ashimema[m] I see
07:39 ashimema[m] so they have functionality tied to them
07:39 ashimema[m] (bad for accessibility functionality though 😉 )
07:40 ashimema[m] not great for the color blind
07:40 Joubu correct
07:41 Joubu bug 25430
07:41 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=25430 enhancement, P5 - low, ---, kyle, Pushed to master , Improve the styling of the claims returned tab
07:41 Joubu that's where it come from
07:43 ashimema[m] I'm happy with caits approach.. though I'm not the biggest fan of that styling to be honest.. I preferred the more minimal digits in perenthesis.
07:43 ashimema[m] but more than anything I'd advocate consistency.. right now it just looks plain messy
07:45 Joubu ashimema[m]: are you planning to work on it?
07:45 cait1 ashimema[m]: it grew on me... but I was also waiting for reactions :)
07:46 cait1 the eyes I mean
07:46 ashimema[m] not sure Joubu
07:46 cait1 i can see why libaries want these to stand out more - and the new markup makes it easy to change/hide
07:46 ashimema[m] I was testing something else when I spotted it and it just bugged me ;)
07:47 ashimema[m] I could do it in a lunch time project perhaps
07:47 ashimema[m] does anyone understand what the clubs two options are..
07:47 cait1 ashimema[m]: did you see the duplicate bug now?
07:47 Joubu ashimema[m]: maybe you could add a summary of the discussion on the new bug report you just created, and we are going to catch Kyle when he will be around?
07:47 cait1 ashimema[m]: i think total and enrolled
07:47 ashimema[m] is it all clubs vs clubs the patron is a member of?
07:47 cait1 that's why i said the total doesn't really make sense there
07:48 ashimema[m] yup.. I marked is as a dupe and cloned my screenshot to it
07:48 Joubu and/or Owen ;)
07:51 ashimema[m] ooh.. thanks for QAing bug 26023 cait
07:51 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=26023 major, P5 - low, ---, martin.renvoize, Passed QA , Incorrect permissions handling for cashup actions on the library level registers summary page
07:51 ashimema[m] I was re-running through the test plan to try and replicate your second points still
07:52 ashimema[m] also.. I think I agree with your final comment.. not sure what to rename though.
07:53 cait1 hm question - in our 20.05 i get 'not found' for pos
07:53 cait1 is it possible we are missing something in the package building process?
07:53 inlibro joined #koha
07:53 ashimema[m] though I think you missed a key difference in the menu name between admin and tools
07:53 cait1 it looks like the file was not there from the message but I can't go server side right now
07:53 ashimema[m] 'Cash registers' vs 'Cashup registers'
07:54 ashimema[m] but I think perhaps 'Cash register summary' might be better in tools as it's no long just about cashing up on that page.
07:54 ashimema[m] hmm
07:54 ashimema[m] not sure why you get a 404
07:54 ashimema[m] that's odd
07:55 cait1 i have this odd memory of a file where we tellwhere to copy things
07:55 cait1 was wondering if pos was missing
07:56 kohaputti In my opinion bug 25532 is not suitable for inclusion in Koha since it feels more like a hack instead of using the koha's search engine to do this instead. What's the process to not accept patches in Koha even though the patch is perfectly working but is not just wanted way of doing things?
07:56 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=25532 enhancement, P5 - low, ---, david.roberts, Signed Off , Add a wikipedia-style search suggestion feature to the OPAC
07:56 ashimema[m] I think you could probably mark is as 'In discussion'.. Joubu commented the same on the bottom of the bug
07:57 ashimema[m] I actually think it's a viable piece of functionality.. though it's a bit mis-sold..
07:57 cait1 kohaputti: i thin it might be debatable
07:57 cait1 i like the idea
07:57 cait1 it might also give you words to suggest and search for in other databases you haven't thoguht of
07:57 kohaputti cait1, why wikipedia instead of the koha database which would in my opinion provide much better results
07:57 ashimema[m] it should be more along the lines of how we get 3rd party cover images or 3rd party biblio data enhancments..
07:57 cait1 but i'd have to take a closer look at the implementaton
07:57 ashimema[m] i.e an opt in for 'Use wikipedia for search autocompletion' as a pref
07:58 cait1 it should definitely be opt in of course
07:58 cait1 because external data
07:58 ashimema[m] well.. if someone wants to write the patch for local search completion that would be great..
07:58 ashimema[m] the wikipedia version would be an option for those not yet on elastic
07:58 ashimema[m] ;)
07:58 ashimema[m] and... it has different functionality
07:59 ashimema[m] we have a University of two using it.. and they really like the results.
07:59 kohaputti I would like to have some evidence it is actually useful to have search keywords suggested that doesn't yield any results
08:00 kohaputti anyway, I'm moving it to discussion and you can all leave your notes there and after that we could decide on this on some dev meeting etc?
08:04 cait1 that's ok for not agreeing
08:05 cait1 but as I said, I see no harm and it's not doable with zebra
08:05 cait1 so it would be really nice for those librraries
08:07 ashimema[m] cait1++
08:07 kohaputti the library could but the code in OPACUserJS if they really wanted this
08:07 Joubu @later tell kidclamp do you think my diag patch on 26031 is enough/makes sense?
08:07 huginn Joubu: The operation succeeded.
08:08 kohaputti I think not everything needs to go to core koha code, some can be plugins, or just code snippets in the *UserJS syspref
08:10 cait1 ok
08:10 cait1 I figured it out: Makefile.pl is missing the pos directory
08:10 cait1 causing issues on installing
08:11 cait1 kohaputti: if it can be done with just jquery maybe documenting woudl be enough, i iwll ty to have a look
08:11 cait1 Joubu: I think we will want to get this into stables ^
08:12 kohaputti cait1, you just copy&paste the javascript from patch to OPACUserJS and it works
08:13 Joubu cait1: I'm RM not RMaint :D
08:13 Joubu Is there a bug report somewhere?
08:14 cait1 Joubu: yes, but you need to go first :)
08:15 cait1 bug 26265 now
08:15 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=26265 blocker, P5 - low, ---, koha-bugs, NEW , Makefile.PL is missing pos directory
08:19 Joubu mtj, mtj[m] are you around still?
08:21 * ashimema[m] quietly wonders if anyone fancies testing bug 24786
08:21 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=24786 enhancement, P5 - low, ---, martin.renvoize, Needs Signoff , Allow setting a cash register for a login session
08:22 cait joined #koha
08:23 cait my coworker confirmas, pos directory doesn't exist on our package installation
08:25 ashimema[m] patch attached
08:26 ashimema[m] I 'think' that's all that's needed.. trivial patch
08:29 cait1 joined #koha
08:53 Joubu @later tell mtj in ktd commit 84d88caa - what was the purpose of "--rules='seq=t/db_dependent/**.t' --rules='par=**'"? My understanding is that the files under t/db_dependent won't be executed in parallel, but *only* the ones with depth 1 (those under t/db_dependent/dirs/* will be in seq however)
08:53 huginn Joubu: The operation succeeded.
08:53 inlibro joined #koha
08:54 Joubu Did this make sense?...
08:54 cait joined #koha
08:59 Joubu ashimema[m]: still not ok for 18501 :-/
09:00 kellym joined #koha
09:00 ashimema[m] man.. that bugs just keeps on giving doesn't it :P
09:01 Joubu that's getting tricky!
09:01 ashimema[m] ooh..
09:01 ashimema[m] 1s interval issues
09:01 ashimema[m] that's interesting
09:01 Joubu yes, this one is random
09:01 Joubu but I got it during the second run
09:02 ashimema[m] I thought my use of Time::Fake would prevent that.. mind you I wasn't coding specifically for that case
09:03 Joubu no time::fake will keep the the time running
09:03 ashimema[m] ah yes..
09:04 Joubu it's not like mocking DateTime->now
09:04 Joubu what we do sometimes in test
09:04 Joubu you should use t::lib::Dates::compare
09:04 Joubu it deals with that (compare with 5s interval)
09:04 ashimema[m] ah yes, of course
09:06 Joubu cait: can you get you coworker confirm ashimema[m]'s patch is correct? 26265
09:06 Joubu your*
09:07 cait1 working on it
09:07 cait1 but we can't test the pacakging
09:07 cait1 but I will sign it
09:08 Joubu @later tell lukeG please add 26265 to your important things to backport into the next 20.05.x
09:08 huginn Joubu: The operation succeeded.
09:09 ashimema[m] maybe I'm being blind.. where did I introduce new Koha::Item->new calls in bug 18501 tests?
09:09 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=18501 major, P5 - low, ---, jonathan.druart, Pushed to master , Automatic refunds need protection from failure
09:10 Joubu "(follow-up) Test undefined userenv behaviour"
09:10 ashimema[m] balls
09:10 ashimema[m] thanks
09:10 ashimema[m] it was a long day yesterday '
09:11 ashimema[m] the previous cases are OK right.. they're already dealt with in the fix es tests bug
09:13 mtj hey #koha
09:14 cait1 hi mtj
09:14 mtj Joubu: re: ktd commit 84d88caa... files under t/db_dependent won't be executed in parallel, everything else will
09:14 mtj hi cait1 :0)
09:15 Joubu mtj: but only files with 1 level depth!
09:16 mtj hmm, no - i think all depths
09:16 Joubu "** is any number of characters, including /, within a pathname"
09:16 Joubu yes, you are right
09:16 mtj snap
09:17 mtj "First match wins". The first rule that matches a test will be the one that applies."
09:17 Joubu `ls t/db_dependent/**.t` in bash only returns 1 level depth files
09:18 ashimema[m] would you suggest t::lib::Dates::compare for the time failure Joubu
09:18 ashimema[m] or do you have anything else in mind
09:18 Joubu yes, Dates::compare!
09:18 ashimema[m] :)
09:18 mtj Joubu:  "We implement our own glob-style pattern matching for --rules. Here are the supported patterns"
09:19 Joubu hum ok
09:19 Joubu I'd like to try and run the selenium tests in seq, they face weird errors and that may be related to // run
09:19 Joubu (// = parallel)
09:21 Joubu but they are run in seq already then!
09:21 mtj hmm, yeah :/
09:22 Joubu next question then :) Do you remember why we prevent all db_dependent tests to be run in parallel?
09:23 magnuse joined #koha
09:25 mtj i think database errors, around locking tables
09:26 mtj one test creating/deleting a table, while another test is currently using the table, etc...
09:27 ashimema[m] 18501 updated patches attached
09:27 ashimema[m] I have no idea where you're final failure is coming from though.. I didn't see it.
09:27 ashimema[m] stuck
09:28 Joubu mtj: right, the deadlock!
09:28 ashimema[m] crap.. now I can replicate your final failure
09:29 ashimema[m] grr.. I wasn't seeing that earlier
09:29 Joubu ashimema[m]: it's not random for me, I even reset_all before retrying
09:29 ashimema[m] yeah.. I get it every time now
09:29 ashimema[m] clearly something was odd at my end that it passed yesterday
09:31 ashimema[m] I have no idea why it's failing though..
09:31 ashimema[m] feels like it may not be linked to 18501 to me
09:34 Joubu do you see it fail on master?
09:34 Joubu It does fail for me, another test..
09:34 Joubu Can't use an undefined value as a HASH reference at /kohadevbox/koha/Koha/Item.pm line 815.
09:35 Joubu silly me
09:35 TimothyAlexis I want to add jQuery validator to member_entry_gen.tt and opac_password_recovery.tt, I also want to make the id's consistently #password and #password2 across files where the validator is used, for consistency.
09:35 Joubu that's the error we are trying to fix :D
09:35 ashimema[m] I fix that one.. but it seems to highlight another one int he same test file
09:35 ashimema[m] but I reckon that second one isn't caused by our bug..
09:35 ashimema[m] just digging to try and understand the test/code
09:36 TimothyAlexis Rather than #NewKey/#Confirm, #password/$password2 and #newpassword/"newpassword2 ... which is confusing...
09:36 ashimema[m] it looks nicely self contained.. so I don't think it's the removal of the prior test that's having a knock on effect
09:37 TimothyAlexis #password2 ... #newpassword2 *
09:37 TimothyAlexis Can I add these as one bug report and patch?
09:38 Joubu newpassword* make sense for password recovery
09:38 Joubu ashimema[m]: git bisect?
09:39 ashimema[m] well.. it does appear to be 18501 :(
09:39 TimothyAlexis I don't care about the id's if noone else does...
09:39 ashimema[m] but I can't for the life of me work out what's goign on at the minute
09:40 Joubu TimothyAlexis: where is #NewKey/#Confirm? git grep NewKey does not return anything on master
09:40 Joubu ha, Newkey
09:41 TimothyAlexis In this version  I'm working with it's in opac-passwd.tt
09:42 * kidclamp waves
09:42 TimothyAlexis Oh it's Newkey :)
09:42 TimothyAlexis Well, I have found another one which is #password/#confirmPassword
09:42 mtj Joubu: i was curious to use  https://metacpan.org/pod/Test::mysqld - as a workaround for deadlock errors in the test-suite
09:43 ashimema[m] found it Joubu
09:43 ashimema[m] I think it's a side effect
09:44 Joubu mtj: maybe we should focus more on a dbix::class oriented solution?
09:45 Joubu kidclamp: could you confirm you agree on the patches I added to 25811 and 26031 this morning?
09:46 mtj Joubu:  i would just be happy with *any* solution :)
09:46 mtj ...dbix::class or other
09:46 ashimema[m] left a comment Joubu
09:47 ashimema[m] as you added the line in question and wrote the test in question I think you may be best to take a stab at fixing it.. I think I've understood the issue but I'm not confident enough to fix it.
09:47 TimothyAlexis Would it be fine to create one bug+patch for adding jQuery validator to both member_entry_gen.tt and opac_password_recovery.tt? Or do they require one for each?
09:54 Joubu ashimema[m]: ha!
09:54 inlibro joined #koha
09:54 Joubu ashimema[m]: should I revert the whole 18501 and forget the last 2 days? :D
09:55 kidclamp Joubu:I commented out the logout and tested before and after your patch.  The STRACE errors I get look the same? for the patron line do we not want the firstname?
09:55 ashimema[m] hell no
09:55 ashimema[m] I like 18501
09:55 kidclamp for 25811^
09:56 ashimema[m] though it highlights yet another area that I think we missed in unit tests...
09:56 ashimema[m] that with this trigger we now also always trigger the paidfor reset whenever itemlost changes
09:56 Joubu the trace is the same, the difference is the debug message to display patron's info
09:56 ashimema[m] I remember querying that actually.. you said that already happened but further down the stack
09:57 kidclamp looks good then, I would just switch to firstname
09:57 Joubu kidclamp: erk yes, I pick the first line of patron-title.inc, but it's for cat 'I' only!
09:57 kidclamp :-)
09:57 Joubu 2     [% patron.surname | html %][% IF patron.othernames %] ([% patron.othernames | html %]) [%- END -%]
09:57 Joubu vs   7     [% patron.firstname | html %] [% patron.surname | html %]
09:57 Joubu so must be firstname and surname
10:01 ashimema[m] kidclamp.. does it make sense for a item to have 'paidfor' set even if 'itemlost' has been 'unset' because the item has been subsequently found?
10:02 ashimema[m] looks like prior to bug 18501 we only reset the paidfor if we actioned a refund.. now even if the refund is unsuccessful (like the patron has left the library or some such), we reset paidfor regardless.
10:02 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=18501 major, P5 - low, ---, jonathan.druart, Pushed to master , Automatic refunds need protection from failure
10:04 Joubu ashimema[m]: fixed, I am cheating but I think it's correct
10:05 ashimema[m] :)
10:05 ashimema[m] awesome.. I think that means we're good on 18501 now then
10:05 kidclamp bug 19919 ashimema
10:05 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=19919 major, P5 - low, ---, martin.renvoize, RESOLVED FIXED, Writing off a Lost Item Fee marks as "Paid for by patron"
10:05 kidclamp we should not use paidfor
10:05 ashimema[m] hehe.. cool
10:05 Joubu commit message for bug 25964 said "We have DB fields that are not mapped with MARC fields, for instance paidfor. They are not handled correctly"
10:05 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=25964 critical, P5 - low, ---, jonathan.druart, Pushed to stable , Data loss possible when items are modified
10:06 ashimema[m] dang.. I should remember code I wrote!
10:07 Joubu but... it's still there?
10:07 ashimema[m] weird
10:08 ashimema[m] I have a feeling that was a QA thing.. we left the field just in case people were using it in reports or had old data in there.
10:08 ashimema[m] I actually think we could kill of the db field entirely now :)
10:09 Joubu ashimema[m]: commit 0f54a1f8c6 "Bug 19919: Stop using paidfor altogether" is setting it in moredetail
10:10 ashimema[m] indeed
10:10 ashimema[m] well..
10:10 Joubu lol
10:10 ashimema[m] building it for display from the data.. rather than setting it in teh db
10:10 Joubu I am at lost :D
10:10 ashimema[m] lets kill it
10:11 ashimema[m] but as a new bug ;P
10:11 Joubu a second time you mean? :)
10:11 Joubu ho ffff!
10:12 Joubu ashimema[m]: commit bbb504c86f5ca "Bug 23463: Replace ModItem with Koha::Item->store"
10:12 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=23463 enhancement, P5 - low, ---, jonathan.druart, Pushed to stable , Move C4::Items CRUD subroutines to Koha::Item
10:12 Joubu +    $accountline->item->paidfor('')->store({ log_action => 0 });
10:12 Joubu I am pretty sure this is a wrong conflict resolution
10:13 Joubu just before you did "-    ModItem( { paidfor => '' }, undef, $itemnumber, { log_action => 0 } );
10:13 Joubu so we are pulling it from C4::Circ since the beginning, but it shouldn't have been there!
10:13 ashimema[m] haha..
10:14 ashimema[m] deffo
10:14 ashimema[m] well sleuthed
10:14 ashimema[m] kill with fire
10:14 Joubu pfiou
10:15 Joubu that's why you should not let me rebase my big stuffs too long!
10:15 ashimema[m] hehe
10:15 ashimema[m] easily done
10:16 ashimema[m] reading my original bug I think I did intend on removing the field entirely but forgot as it went through SO and QA so quick once it started moving
10:16 ashimema[m] lol
10:16 ashimema[m] anywho.. I'll create a bug
10:16 * ashimema[m] likes that we're cleaning stuff up
10:16 ashimema[m] also saves me writing a test for the trigger functionality.
10:18 Joubu ashimema[m]: wondering...
10:18 Joubu should not we remove the superflous line before 18501? :-/
10:19 Joubu nevermind
10:19 Joubu it's still in master
10:20 ashimema[m] I'm writting the new bug to kill it with fire now..
10:21 ashimema[m] lets just make sure we follow 18501 with this one fairly quick ;)
10:21 ashimema[m] and get all those 18501 fixes pushed I reckon
10:21 Joubu kidclamp: ok with 26031?
10:21 Joubu I am about to push it
10:22 kidclamp you are dumping the OPAC file for the intra test? and get_best_xslt_filename is missign lang param
10:22 kidclamp joubu:no
10:22 Joubu it's opac everywhere
10:23 Joubu I am missing lang!
10:23 kidclamp ? the line before your changes says 'intra_text'
10:24 kidclamp 250 # --------------------------------- TEST INTRANET SEARCH
10:24 Joubu right
10:24 Joubu I wanted to dump all the pref values
10:24 Joubu but decided to go with only one
10:24 Joubu that's why, and that's wrong
10:24 khall joined #koha
10:33 huginn News from kohagit: Bug 18501: Don't use paidfor for DB fields not mapped with a MARC field <http://git.koha-community.org/[…]6bc491cfb33c353ec>
10:33 huginn News from kohagit: Bug 18501: (follow-up) Use t::lib::Dates::compare in tests <http://git.koha-community.org/[…]121b9cde06f6712ca>
10:33 huginn News from kohagit: Bug 25811: Add debug info to authentication.t <http://git.koha-community.org/[…]485c40c19151fc47b>
10:33 huginn News from kohagit: Bug 26031: Add diagnostic to search_utf8.t <http://git.koha-community.org/[…]f450d0226a8a71f2c>
10:33 huginn News from kohagit: Bug 18501: (follow-up) Test undefined userenv behaviour <http://git.koha-community.org/[…]f1ccb9ce72cd25c12>
10:33 huginn News from kohagit: Bug 18501: (follow-up) Add fallback to ItemHomeBranch <http://git.koha-community.org/[…]f8ed79af304b4acc3>
10:33 huginn News from kohagit: Bug 18501: (follow-up) Do not rely on userenv->{branchcode} <http://git.koha-community.org/[…]0f4d5833ef7cdd133>
10:33 huginn News from kohagit: Bug 19036: (follow-up) Test credits should be credits <http://git.koha-community.org/[…]6b265748fa1123085>
10:33 huginn News from kohagit: Bug 18501: (follow-up) Remove duplicate test <http://git.koha-community.org/[…]5d565460f439a8320>
10:44 Joubu easy SO on bug 26157
10:44 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=26157 enhancement, P5 - low, ---, jonathan.druart, Needs Signoff , Redirect expected DBI warnings
10:44 Joubu please
10:47 ashimema[m] done
10:54 inlibro joined #koha
11:00 oleonard Busy day so far, eh?
11:15 koha-jenkins Project Koha_Master_D9 build #1413: STILL UNSTABLE in 41 min: https://jenkins.koha-community[…]a_Master_D9/1413/
11:20 koha-jenkins Project Koha_Master_D9_MDB_Latest build #370: STILL UNSTABLE in 46 min: https://jenkins.koha-community[…]9_MDB_Latest/370/
11:20 koha-jenkins Project Koha_Master_U20 build #76: STILL UNSTABLE in 46 min: https://jenkins.koha-community[…]ha_Master_U20/76/
11:24 ashimema[m] Always 😀
11:28 koha-jenkins Project Koha_Master_D10 build #331: STILL UNSTABLE in 53 min: https://jenkins.koha-community[…]a_Master_D10/331/
11:36 oleonard ashimema[m]: Always busy, but not always busy in #koha ;)
11:45 ashimema[m] True
11:52 cait joined #koha
11:52 koha-jenkins Project Koha_Master_D10_Deps build #57: STILL UNSTABLE in 36 min: https://jenkins.koha-community[…]ster_D10_Deps/57/
11:54 inlibro joined #koha
11:55 cait joined #koha
11:55 AndrewFH joined #koha
12:01 koha-jenkins Project Koha_Master_U18 build #875: NOW UNSTABLE in 1 hr 25 min: https://jenkins.koha-community[…]a_Master_U18/875/
12:02 Joubu Anybody familiar with ThingISBN?
12:02 Joubu bug 26270
12:02 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=26270 normal, P5 - low, ---, jonathan.druart, Needs Signoff , XISBN.t is failing since today
12:02 Joubu I think there was a change in their answer
12:04 koha-jenkins Project Koha_Master_U16 build #49: STILL UNSTABLE in 44 min: https://jenkins.koha-community[…]ha_Master_U16/49/
12:07 koha-jenkins Project Koha_Master_D11 build #69: STILL UNSTABLE in 46 min: https://jenkins.koha-community[…]ha_Master_D11/69/
12:08 oleonard Joubu I don't imagine the code on our side has been touched in years
12:09 oleonard Joubu have you tested whether the service is working in Koha?
12:14 cait1 joined #koha
12:15 oleonard It looks to me like it is
12:15 ashimema[m] me too
12:23 Joubu yes, it "works". The change is subtle: my guess is that the service now return the normalized isbn
12:24 Joubu so I am expecting Koha to show the current biblio record when it should not
12:29 koha-jenkins Project Koha_Master_D9_My8 build #394: STILL UNSTABLE in 1 hr 1 min: https://jenkins.koha-community[…]aster_D9_My8/394/
12:33 AndrewFH joined #koha
12:34 cait1 left #koha
12:37 cait joined #koha
12:44 cait joined #koha
12:52 koha-jenkins Project Koha_Master_D11 build #70: STILL UNSTABLE in 44 min: https://jenkins.koha-community[…]ha_Master_D11/70/
12:54 inlibro joined #koha
13:04 caroline joined #koha
13:20 tcohen morning
13:20 oleonard Hi tcohen
13:21 oleonard Sorry I missed your messages last night. Forgot to /away myself.
13:21 tcohen hi! got the car for a service so slowly getting back to work
13:21 tcohen np! thanks for your help yesterday
13:30 * cait waves
13:31 * magnuse waves and wanders off into the weekend
13:41 vfernandes hi #koha
13:41 wizzycray joined #koha
13:43 vfernandes what is the easiest way to block renewals on the same day of the checkout?
13:43 caroline vfernandes: maybe in the circulation rulesl no renewals before
13:45 vfernandes "no renewals before" works in some cases when daysMode is set to not use the calendar
13:46 ashimema[m] still around oleonard?
13:46 vfernandes if daysMode is set to "calendar" the checkout period can be longer than "no renewals before"
13:46 caroline how is it determined what bz is a bug vs an enhancement?
13:47 ashimema[m] @later tell oleonard I'd be interested in your thoughts on my 'Add print option' follow-up on bug 26172.. it feels like a nice generic approach to printing modals..
13:47 huginn ashimema[m]: The operation succeeded.
13:49 Joubu caroline: "Importance"
13:49 Joubu blocker, critical, major, normal, minor, trivial are bugs
13:49 Joubu new ft, enh are enhancements
13:50 caroline but like bug 25555 to me looks like a bug, but it's classified like an enhancement
13:50 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=25555 enhancement, P5 - low, ---, lucas, Pushed to stable , Holds Queue sorts patrons by firstname
13:50 caroline I'm asking all those questions because I'm going through the release notes for the docs and a lot of things that are in the "enhancement" section seem like bugs
13:51 ashimema[m] always nice to clean them up
13:51 caroline also bug 25789 seems like a bug fix
13:51 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=25789 enhancement, P5 - low, ---, emmi.takkinen, Pushed to oldstable , New expiration date on placing a hold in staff interface can be set to a date in the past
13:51 caroline should I clean them up as I see them? I don't want to step on any toes...
13:51 * ashimema[m] would say yes.. but Joubu is not the master of all :)
13:52 Joubu yes please caroline!
13:53 cait caroline++
13:53 Joubu about 25555, aleisha marked it as bug
13:53 caroline ok, I will then
13:53 Joubu as enh
13:53 Joubu "enhancement, not backporting to 19.11.x"
13:54 Joubu she considered it an enhancement
13:54 inlibro joined #koha
13:57 caroline that's why I said I didn't want to step on any toes... I know bug vs enh determined whether it is backported in older versions
13:58 caroline but at the same time, at the point where I come in, this should already have been through the hands of the rmaints
13:58 cait i think most people get emails from bugzilla on changes... so they can still step in
13:59 caroline Yeah, I don't want to spam yall either :)
13:59 oleonard I'm still around ashimema[m]
14:03 ashimema[m] :)
14:10 khall_ joined #koha
14:11 oleonard Joubu, cait: I submitted a followup to Bug 25727 that doesn't address the width issue :/
14:11 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=25727 enhancement, P5 - low, ---, jonathan.druart, Needs Signoff , Update the Select2 JS lib
14:11 oleonard ...but is hopefully an improvement in other ways.
14:13 * ashimema[m] is flagging.. got that friday afternoon feeling
14:14 * ashimema[m] looks forward to a week off however :)
14:26 khall joined #koha
14:43 margaret joined #koha
14:48 oleonard ashimema[m] still around?
14:48 oleonard I'm not sure I understand how to test Bug 26172
14:48 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=26172 enhancement, P5 - low, ---, martin.renvoize, Needs Signoff , Add a cashup summary view
14:49 ashimema[m] yup
14:49 oleonard Am I performing the transactions on the Point of sale page?
14:50 ashimema[m] either there or via a patron (but making sure to select a cash register)
14:50 oleonard How do I do it via a patron?
14:50 ashimema[m] but it's simplest to do it via the point of sale screen (as you then have the links available for the next step)
14:51 ashimema[m] find a patron with fines owed
14:51 ashimema[m] or add a manual debit to a patron
14:51 ashimema[m] then pay it off
14:55 inlibro joined #koha
14:57 oleonard Okay, I got it. I think that solution makes sense.
14:58 oleonard But the print view seems to be too small.
14:59 ashimema[m] Yeah.. I think I need to tweak the css slightly.. but was generally interested in your opinion on the general approach
14:59 ashimema[m] i.e. using media queries and hiding all but the modal content dynamically on the call the print
14:59 ashimema[m] not sure where else we might print modals.. but it felt like a nice generic approach we might like to take elsewhere too
15:00 ashimema[m] the modal content could do with some beautification.. I've not thought about that much yet.. I'm sure I could make it look nicer
15:01 oleonard Seems presentable.
15:01 ashimema[m] thanks
15:08 lisettelatah joined #koha
15:14 reiveune bye
15:14 reiveune left #koha
15:17 fridolin left #koha
15:17 ashimema[m] do we have a fixed width for itemtype images?
15:17 kohaputti joined #koha
15:18 oleonard No
15:19 khall_ joined #koha
15:22 ashimema[m] could make bug 26263 a bit interesting to fix
15:22 huginn Bug http://bugs.koha-community.org[…]_bug.cgi?id=26263 normal, P5 - low, ---, martin.renvoize, ASSIGNED , Style error on OPAC search results when item-level_itypes is set to bibliographic
15:37 ashimema[m] patch attached
15:48 kohaputti joined #koha
15:55 inlibro joined #koha
16:47 khall joined #koha
16:55 inlibro joined #koha
17:18 cait joined #koha
17:55 inlibro joined #koha
18:12 oleonard-away jajm around?
18:18 cait 8pm on a friday..
18:19 oleonard You're here cait :D
18:19 cait yeah, but I have a serious problem
18:19 * oleonard too
18:19 cait koha anonymous?
18:21 khall_ joined #koha
18:23 cait hi khall :)
18:27 khall_ hi cait !
18:55 inlibro joined #koha
19:22 oleonard Is there no other way to access resolved claims other than the tab in circulation?
19:36 cait hm not that i know of
19:39 oleonard Here's the tabs re-done using jajm's plural-handling template syntax: https://zivotdesign.com/p/view[…]?p=15980387474009
19:40 oleonard (and counting only unresolved claims and enrolled clubs)
19:55 inlibro joined #koha
20:38 kellym_ joined #koha
20:56 inlibro joined #koha
21:35 ashimema[m] Interesting
21:53 lukeG1 joined #koha
21:56 inlibro joined #koha
22:56 inlibro joined #koha
23:16 khall joined #koha
23:56 inlibro joined #koha

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