← Previous day | Today | Next day → | Search | Index
All times shown according to UTC.
Time | Nick | Message |
---|---|---|
00:13 | inlibro joined #koha | |
00:22 | khall joined #koha | |
00:42 | aleisha_ joined #koha | |
00:59 | lukeG joined #koha | |
01:10 | khall joined #koha | |
01:13 | inlibro joined #koha | |
02:13 | inlibro joined #koha | |
02:16 | aleisha_ joined #koha | |
03:11 | kathryn joined #koha | |
03:13 | inlibro joined #koha | |
03:55 | aleisha_ joined #koha | |
04:14 | inlibro joined #koha | |
04:42 | khall joined #koha | |
04:59 | chriss joined #koha | |
05:13 | dcook | Wow checkauth() is 800+ lines of code... |
05:14 | inlibro joined #koha | |
05:15 | khall joined #koha | |
05:58 | wshealy joined #koha | |
06:14 | inlibro joined #koha | |
06:19 | did joined #koha | |
06:27 | kohaputti joined #koha | |
06:40 | cait joined #koha | |
06:59 | alex_a joined #koha | |
06:59 | alex_a | Bonjour |
07:01 | cait1 joined #koha | |
07:02 | Joubu | dcook: git-bz crashes with a timeout if you wait too long before the confirmation (with -e). Otherwise it never crashes for me |
07:02 | cait2 joined #koha | |
07:02 | reiveune joined #koha | |
07:02 | dcook | Joubu: I think in my case it's usually when attaching multiple patches |
07:02 | Joubu | hum no, the confirmation is even without -e |
07:02 | reiveune | hello |
07:02 | dcook | allo alex_a, reiveune (and obviously Joubu) |
07:03 | Joubu | I do that several times per day and no crashes |
07:03 | dcook | Hmm weird |
07:03 | lds joined #koha | |
07:03 | dcook | I do have an odd networking setup |
07:03 | A little 3G modem likes injecting a 2nd default route into my route table | |
07:03 | Very irritating | |
07:03 | Often ends up slowing down network traffic so maybe that's the culprit.. | |
07:04 | * dcook | shrugs |
07:04 | dcook | Trying to write a patch before finishing up for the day.. |
07:04 | Joubu | ha, that's somethign else then :) |
07:04 | about 26231 - only for 19.11? | |
07:05 | fridolin joined #koha | |
07:10 | dcook | Joubu: Hmm not sure. I see that it says 19.11 but I'm not sure why. Not sure if me or someone else set that. I might've set it by accident, as I've seen it in 19.11 although I checked in master so it should be master too... |
07:13 | cait2 | bug 26231 |
07:13 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=26231 critical, P5 - low, ---, koha-bugs, Needs Signoff , bulkmarcimport.pl does not import authority if it already has a 001 field |
07:14 | cait2 | the history mmight tell, but often people set the version they tested with |
07:14 | inlibro joined #koha | |
07:14 | dcook | I probably set it to 19.11 on my first prod test |
07:14 | Then didn't think to unset it after writing and testing against master | |
07:15 | cait2 | https://bugs.koha-community.or[…]vity.cgi?id=26231 it was filed 19.11 - no changes since initial filing |
07:17 | alexbuckley joined #koha | |
07:20 | dcook | Makes sense |
07:21 | Just changed it to master :p | |
07:23 | tcohen joined #koha | |
07:25 | Joubu | kohaputti: on bug 16357, "To truly prevent any issues we probably would have to hard code the log4perl.conf". What do you mean? |
07:25 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=16357 normal, P3, ---, dcook, Failed QA , Plack error logs are not time stamped |
07:25 | kohaputti | Joubu, the sysadmin could make thing log to /dev/null if they wanted |
07:25 | we have no way to prevent that :D | |
07:26 | dcook | The sysadmin could just turn off Koha :p |
07:27 | Joubu | yes, sysadmins can do whatever they want :) |
07:28 | * dcook | almost has a fix for bug 26301 |
07:28 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=26301 normal, P5 - low, ---, dcook, NEW , Self-checkout blocks renew for overdues even when OverduesBlockRenewing allows it in opac-user.pl |
07:29 | kohaputti | Joubu, dcook so the best we can do I think is that we check there is some kind of configuration for these log4perl logging categories, and then inform about those issues in about.pl page (which as I commented should be another bug report) |
07:30 | dcook | I think so |
07:30 | Basically, my patch checks to see if there is any log appenders | |
07:30 | If there isn't, don't use the middleware, so just output to stderr which daemon puts into plack-error.log | |
07:31 | If there are Log4perl appenders (whatever they are), then use the middleware to use those appenders for the output | |
07:31 | They could actually be appenders that output to STDERR which puts them in plack-error.log still | |
07:31 | Joubu | yes, that's exactly what we need |
07:32 | kohaputti | dcook, reading https://metacpan.org/pod/Log::Log4perl#Appenders appender seems the right place to check actually, it decides whether something is outputted from log4perl according to that documention |
07:32 | dcook | But the problem with that is that the middleware uses categories |
07:32 | So you could have PLACKOPAC defined as an appender, but if no category uses it, you're still screwed | |
07:33 | I think the patch I used in the end is probably the best a person can do, although I'm open to alternatives | |
07:34 | kohaputti | dcook, hmm, I thought this checked the appenders for the given category so I don't see the issue |
07:34 | Log::Log4perl->get_logger('plack-opac')->has_appenders | |
07:34 | it only returns true if plack-opac category has appenders | |
07:35 | dcook, is there some way to define a appender for category and not use it? | |
07:36 | dcook | "Log::Log4perl->get_logger('plack-opac')->has_appenders " is what I used in the patch and that should be solid |
07:37 | kohaputti | yes I think so too |
07:37 | dcook | "dcook, is there some way to define a appender for category and not use it?" appenders are defined for categories. Appenders are defined independently and then attached to categories. |
07:37 | ack | |
07:37 | "appenders are not defined for categories"* | |
07:37 | magnuse joined #koha | |
07:40 | kohaputti | dcook, yeah, I see it now, thanks for the explanation |
07:41 | magnuse | \o/ |
07:41 | dcook | No worries :D |
07:41 | On that note, I should really go home heh | |
07:42 | magnuse | noooo, stay a little longer and fix all the things! |
07:43 | dcook | But hopefully I already fixed all the things O_O |
07:43 | No no one has time for that.. | |
07:44 | I thought it was good timing talking about dropping support for IE 11 lately since Microsoft itself plans to drop support in 3 months heh | |
07:45 | Ok really going now | |
07:45 | Hope you all have a good day :D | |
07:45 | magnuse | have fun, dcook |
07:46 | khall joined #koha | |
07:56 | Joubu | is bugzilla super slow, or it's just me? |
08:06 | kohaputti | Joubu, seems to work alright for me, browsing and searching |
08:07 | Joubu | yes it's back to normal now |
08:14 | inlibro joined #koha | |
09:03 | kathryn joined #koha | |
09:14 | inlibro joined #koha | |
09:17 | Joubu | @later tell kidclamp please add bug 25758 on top of your list |
09:17 | huginn | Joubu: The operation succeeded. |
09:31 | Joubu | cait2: bug 25630 depends on bug 24399, is that correct? |
09:31 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=25630 enhancement, P5 - low, ---, katrin.fischer, Signed Off , More capitalization and terminology fixes for system preferences |
09:31 | Bug http://bugs.koha-community.org[…]_bug.cgi?id=24399 enhancement, P5 - low, ---, koha-bugs, NEW , New permission to edit "system-level" news items | |
09:31 | hibbes joined #koha | |
09:39 | hibbes | Hello! We are trying to migrate our SchoolLibrary to koha. We imported all Books with MarcEdit and they appear in Koha. But we need to generate one item for every book (over fivethousend). But we dont't know how. :-/. |
09:45 | hibbes left #koha | |
09:53 | cait2 | Joubu: looks like one of my usual typos :( |
09:53 | i don't think there is a rela depedency | |
09:54 | probably another interface one i linked that is already pushed, I will remove the link | |
09:54 | gone for lunch, available a bit later | |
10:00 | cait joined #koha | |
10:02 | lmstrand joined #koha | |
10:15 | inlibro joined #koha | |
10:27 | khall joined #koha | |
10:37 | cait joined #koha | |
10:38 | fridolin joined #koha | |
10:41 | alexbuckley joined #koha | |
10:46 | khall joined #koha | |
10:46 | Joubu | kohaputti: 26015 - what's the value of $KOHA_INTRANET_URL env var? |
10:46 | kohaputti | empty |
10:46 | I used docker exec -it koha_koha_1 bash | |
10:46 | to get to the container | |
10:46 | and then I ran prove after that | |
10:48 | hmm, the kohadev user also has it empty | |
10:48 | Joubu | that's not expected |
10:48 | are you on ktd's master? | |
10:49 | kohaputti | it's probably few months since I downloaded ktd, I will fetch the latest version and let you know |
10:49 | khall_ joined #koha | |
10:49 | Joubu | export KOHA_INTRANET_URL=http://koha:8081 |
10:49 | should do it | |
10:52 | kohaputti | Joubu, I understand why it is not set for me, I run it using the instruction in # Manually of README, instead of those fancy scripts/aliases :) |
10:54 | Joubu | I don't use them either |
11:04 | tcohen | why self punishment? |
11:05 | :P | |
11:05 | good morning | |
11:05 | cait2 | back |
11:09 | Joubu | tcohen: ctrl-R is my friend |
11:09 | tcohen | kd is handy to start over |
11:09 | Joubu | and it does not set the koha-image, so it's not super useful imo |
11:09 | tcohen | ku + ku is a gret combo |
11:10 | I have it set in my .bashrc | |
11:10 | Joubu | I have a one line in my history with cd + down + up. It works great : |
11:10 | :) | |
11:13 | tcohen | Joubu about your bug 22343 question |
11:13 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=22343 new feature, P5 - low, ---, tomascohen, Needs Signoff , Add SMTP configuration options to Administration |
11:14 | tcohen | I think it was because some exceptions propagated and raised more warnings that I needed to test |
11:14 | I'm not sure that still stands as there's no unique constraint on the smtp_servers.library_id column | |
11:15 | inlibro joined #koha | |
11:15 | Joubu | we should not get more warnings |
11:15 | or you are cheating :D | |
11:17 | tcohen | I will check |
11:17 | thanks for the feedback | |
11:42 | fridolin joined #koha | |
11:45 | magnuse | do we have an example of a koha with lots of arabic records? someone asked about it on facebook... |
11:47 | janncis__ | hello I get this wierd error while importing patrons wia import_patrons.pm |
11:47 | use of uninitialized values $borrowers{"cardnumber"} in string eq at Koha/Patrons/Import.pm line 137 | |
11:48 | oleonard joined #koha | |
11:49 | oleonard | o/ |
11:50 | janncis__ | that error is for all entries |
11:59 | magnuse | janncis__: do you have a cardnumber in your data? |
11:59 | janncis__ | yes, I double checked it |
12:02 | lds joined #koha | |
12:04 | wizzyrea joined #koha | |
12:09 | marcelr joined #koha | |
12:09 | marcelr | hi #koha |
12:09 | oleonard++ # signoff | |
12:10 | oleonard | You arrived just in time :) |
12:11 | marcelr | you were not leaving ? |
12:15 | inlibro joined #koha | |
12:16 | Dyrcona joined #koha | |
12:39 | caroline joined #koha | |
12:50 | Joubu | wiki is down |
12:51 | so cannot see who is hosting it | |
12:51 | gmcharlt maybe? | |
12:51 | hum not "down" actually | |
12:51 | but dying | |
12:52 | jzairo joined #koha | |
12:52 | Joubu | back to normal now, sorry for the noise |
13:03 | tcohen | @later tell oleonard is it ok to have bootstrap4.5.2 on your patchset? |
13:03 | huginn | tcohen: The operation succeeded. |
13:07 | oleonard | tcohen as far as I know it is |
13:07 | tcohen | it actually is |
13:07 | I reset the package.json file to current master | |
13:07 | and yarn add bootstrap | |
13:07 | and things are going smooth so far | |
13:15 | inlibro joined #koha | |
13:18 | oleonard | I have to leave for a while but I'll be back to look again tcohen |
13:18 | tcohen | cool |
13:22 | lukeG1 joined #koha | |
13:24 | huginn | News from kohagit: Bug 23653: Add license information in about.pl <http://git.koha-community.org/[…]175d646b29558fe71> |
13:24 | News from kohagit: Bug 23653: Remove uneeded cond test <http://git.koha-community.org/[…]28dfab2d7f7048375> | |
13:24 | News from kohagit: Bug 26175: Remove warn if undefined barcode in misc/export_records.pl <http://git.koha-community.org/[…]983fdf839467bfdf4> | |
13:24 | News from kohagit: Bug 26014: Add publication year and edition to Z39.50 results in acquisition <http://git.koha-community.org/[…]96b08465defa59aff> | |
13:24 | News from kohagit: Bug 26013: Fix formatting for dates on "Manage stage MARC records" page <http://git.koha-community.org/[…]c435ac8bea6550c82> | |
13:24 | News from kohagit: Bug 23653: use local copy of swagger v2 schema <http://git.koha-community.org/[…]45d4f9647c2b3b428> | |
13:24 | News from kohagit: Bug 25129: Update German (de-DE) web installer sample files for 20.05 <http://git.koha-community.org/[…]08c0f676e93fd5566> | |
13:25 | tcohen | JOubu++ |
13:25 | Joubu++ | |
13:28 | cait2 | Joubu++ |
13:35 | wizzyrea joined #koha | |
13:36 | magnuse | Joubu++ |
13:39 | gmcharlt | Joubu: I checked the wiki server, don't see any immediately reason for downtime; will chalk it up to a glitch for now |
13:41 | Joubu | thanks gmcharlt for having a look! |
13:49 | dev meeting in 10min! | |
13:55 | tcohen | (y) |
14:00 | Joubu | #startmeeting Development IRC meeting 26 August 2020 |
14:00 | huginn | Meeting started Wed Aug 26 14:00:54 2020 UTC. The chair is Joubu. Information about MeetBot at http://wiki.debian.org/MeetBot. |
14:00 | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
14:00 | Topic for #koha is now (Meeting topic: Development IRC meeting 26 August 2020) | |
14:00 | huginn | The meeting name has been set to 'development_irc_meeting_26_august_2020' |
14:00 | Joubu | #topic Introductions |
14:00 | Topic for #koha is now Introductions (Meeting topic: Development IRC meeting 26 August 2020) | |
14:01 | Joubu | #info Jonathan Druart |
14:01 | tcohen | #info Tomas Cohen Arazi, Theke Solutions |
14:01 | Joubu | qa_team |
14:01 | qa_team? | |
14:01 | wahanui | qa_team is cait Joubu marcelr kohaputti josef_moravec tcohen kidclamp khall ashimema alex_a jajm matts |
14:02 | Joubu | rmaints? |
14:02 | wahanui | lukeG, aleisha, and tuxayo, I SUMMON THEE!! |
14:02 | talljoy joined #koha | |
14:02 | kidclamp | #info Nick Clemens, ByWater Solutions |
14:02 | marcelr | #info Marcel de Rooy |
14:02 | kohaputti | #info Joonas Kylmälä |
14:04 | Joubu | #topic Announcements |
14:04 | Topic for #koha is now Announcements (Meeting topic: Development IRC meeting 26 August 2020) | |
14:04 | Joubu | Anyone have something? |
14:04 | tcohen | can we self promote our devs? |
14:04 | he | |
14:05 | marcelr | good idea |
14:05 | tcohen | Bug 22343 is ready for testing |
14:05 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=22343 new feature, P5 - low, ---, tomascohen, Needs Signoff , Add SMTP configuration options to Administration |
14:05 | marcelr | smtp again |
14:05 | tcohen | kudos to khall, ashimema[m] and JOubu for QA feedback |
14:06 | Joubu | #info the SMTP configuration admin page feature is ready for testing! bug 22343 |
14:06 | marcelr, do you have one? | |
14:06 | I have 3, but I have a dedicated section for that ;) | |
14:06 | marcelr | every bug that i submitted :) |
14:07 | tcohen | I have another announcement |
14:07 | Joubu | #info Marcel has patches waiting for signoff! |
14:07 | tcohen | I'm working on merging the Koha2Koha ILL backend into core Koha |
14:07 | once I have it working I'll file a bug and submit the patches, the challenges are translatability | |
14:08 | on the current layout | |
14:08 | koha-jenkins | Project Koha_Master_U20 build #80: SUCCESS in 44 min: https://jenkins.koha-community[…]ha_Master_U20/80/ |
14:08 | Joubu | why translatibility? |
14:09 | tcohen | there are some bad design decisions on the ILL stuff, that prevent it |
14:09 | Joubu | #info Tomas is going to adjust the Koha2Koha ILL backend to have it into Koha core |
14:09 | k | |
14:09 | something else? | |
14:09 | tcohen | shortcuts |
14:09 | amoyano joined #koha | |
14:09 | Joubu | #topic Update from the Release Manager 20.11 |
14:09 | Topic for #koha is now Update from the Release Manager 20.11 (Meeting topic: Development IRC meeting 26 August 2020) | |
14:09 | Joubu | I forgot: |
14:09 | #link https://wiki.koha-community.or[…]ng_26_August_2020 | |
14:10 | koha-jenkins | Project Koha_Master_D9 build #1416: SUCCESS in 46 min: https://jenkins.koha-community[…]a_Master_D9/1416/ |
14:10 | Joubu | I am harassing the QA team to have them on the^Mmy 3 top priority bugs: task queue, merge deleted* table, and the move to bootstrap 4 |
14:10 | kidclamp | i am going over the merge |
14:10 | Joubu | You must watch regularly and put your attention on https://frama.link/koha_bz_RM_priority |
14:10 | there are 2 major bugs that are waiting for too long already | |
14:10 | tcohen | I'm looking at the bootstrap one and communicating with oleonard about the things I find |
14:10 | Joubu | yep, thanks kidclamp! |
14:11 | I don't have anything to say apart that it's now or... maybe never | |
14:11 | at least not for 20.11 if not ready very soon | |
14:11 | marcelr | push the merge one now |
14:12 | Joubu | and I will certainly drop some of them (tired of rebasing them) |
14:12 | tcohen | yeah, push it |
14:12 | haha | |
14:12 | Joubu | I don't want to bypass QA for such big changes |
14:12 | tcohen | Nick is on it |
14:12 | marcelr | i try the task q friday? |
14:12 | Joubu | you will like it marcelr! |
14:12 | tcohen | Joubu about the holds table, does your message imply that you expect a regression test for that? |
14:13 | Joubu | the task queue opens door for a lot of great feature, and will fix lot of the long standing issues we have with the background jobs |
14:13 | cait2 | #info Katrin Fischer, BSZ, Germany |
14:13 | Joubu | tcohen: I was thinking about it, and had to drop the idea somehwere, nothing else |
14:13 | tcohen | ok, that concern is covered |
14:13 | I can take a look at tests, just in case | |
14:13 | cait2 | sorry for being a little late |
14:14 | Joubu | cait++ |
14:14 | for being late | |
14:14 | :D | |
14:14 | cait2 | that's a first - what did you sign me up for already? |
14:14 | Joubu | #topic Updates from the Release Maintainers |
14:14 | Topic for #koha is now Updates from the Release Maintainers (Meeting topic: Development IRC meeting 26 August 2020) | |
14:14 | Joubu | I don't think we have any |
14:14 | #topic Updates from the QA team | |
14:14 | Topic for #koha is now Updates from the QA team (Meeting topic: Development IRC meeting 26 August 2020) | |
14:15 | Joubu | you are actually right in time for your topic :) |
14:15 | marcelr | just in time cait2 |
14:15 | cait2 | oh oops |
14:15 | marcelr | she cut the queue remarkably |
14:15 | Joubu | #info Stable releases are coming soon! |
14:15 | cait2 | #info QA queue is in pretty good shape right now, but needs work to stay that way, big patches waiting |
14:15 | Joubu | FQA is pretty bad :D |
14:15 | inlibro joined #koha | |
14:15 | khall joined #koha | |
14:15 | cait2 | yes, but some complicated things remain, i hope to get your help with that |
14:16 | marcelr | and how about In discussion :) |
14:16 | cait2 | and some will jump back into the queue when their devs return from holidays probably :) |
14:16 | i especially woudl be happy about help with SIP patches | |
14:16 | and the aformentioned big 3 | |
14:17 | Joubu | bug 21979 |
14:17 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=21979 enhancement, P5 - low, ---, kyle, Signed Off , Add option to SIP2 config to send arbitrary item field in CR instead of collection code |
14:17 | Joubu | bug 25541 |
14:17 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=25541 enhancement, P5 - low, ---, kyle, Signed Off , Add ability to prevent checkin via SIP of items with holds |
14:17 | cait2 | also someone else running the Elasticsearch.t and see if they can reproduce my failure |
14:17 | Joubu | Who around is familiar with SIP? |
14:17 | cait2 | bug 26250 |
14:17 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=26250 major, P5 - low, ---, jonathan.druart, Signed Off , Test suite does not pass if Elastic is used as search engine |
14:17 | kohaputti | cait2, I can help with SIP |
14:18 | Joubu | thanks kohaputti! |
14:18 | cait2 | that would be great |
14:18 | note: bug 25261 also has a sip component | |
14:18 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=25261 enhancement, P5 - low, ---, martin.renvoize, Signed Off , Multiple parts handling - confirmation alert |
14:18 | cait2 | so if someone could take care of that, i am happy to test in the GUI |
14:18 | koha-jenkins | Project Koha_Master_D10 build #335: SUCCESS in 53 min: https://jenkins.koha-community[…]a_Master_D10/335/ |
14:18 | Project Koha_Master_U18 build #879: ABORTED in 53 min: https://jenkins.koha-community[…]a_Master_U18/879/ | |
14:19 | Joubu | Also, I need some QA help on bug 25758, more brainstorming. |
14:19 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=25758 major, P5 - low, ---, jonathan.druart, In Discussion , Items scheduled for automatic renewal do not show that they will not renew due to a hold |
14:19 | cait2 | i probably forgot something, but that's it from me basically :) |
14:19 | Joubu | kidclamp is going to ask some Bywater educators, but additional feedback would be great |
14:19 | cait2 | ah, it seemed to work well on staff side, more issues found? have to check on that later |
14:20 | Joubu | cait2: I cheat, lot of changes from the patch were not advertised |
14:20 | so it's not only opac-user that needs to be tested | |
14:20 | ok, moving on | |
14:20 | #Actions from last meeting | |
14:20 | #topic Actions from last meeting | |
14:20 | Topic for #koha is now Actions from last meeting (Meeting topic: Development IRC meeting 26 August 2020) | |
14:21 | Joubu | #topic ashimema is going to add his stamp on 20168 |
14:21 | Topic for #koha is now ashimema is going to add his stamp on 20168 (Meeting topic: Development IRC meeting 26 August 2020) | |
14:21 | Joubu | It's done! |
14:21 | cait2 | bug 20168 |
14:21 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=20168 enhancement, P5 - low, ---, oleonard, ASSIGNED , Update of the OPAC bootstrap template to bootstrap v4 |
14:21 | Joubu | #info ashimema signed off 20168, thanks! |
14:21 | #topic ashimema is collecting notes about bug 15522 and is going to review and dig a bit more into the code | |
14:21 | Topic for #koha is now ashimema is collecting notes about bug 15522 and is going to review and dig a bit more into the code (Meeting topic: Development IRC meeting 26 August 2020) | |
14:21 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=15522 enhancement, P5 - low, ---, jonathan.druart, Signed Off , New interface for revamped circulation rules |
14:21 | Joubu | still in progress |
14:21 | #action ashimema is collecting notes about bug 15522 and is going to review and dig a bit more into the code (postponed) | |
14:21 | #topic ashimema to propose a guideline ragarding using direction object notation rather than indirect object notation (postponed) | |
14:21 | Topic for #koha is now ashimema to propose a guideline ragarding using direction object notation rather than indirect object notation (postponed) (Meeting topic: Development IRC meeting 26 August 2020) | |
14:22 | cait2 | wrong tag? |
14:22 | Joubu | #info A guideline for remove indirect object notation has been written and available at https://wiki.koha-community.or[…]t_Object_Notation |
14:22 | I don't think we need to vote on it | |
14:23 | #info there is a bug ready for testing: Bug 25898 - Prohibit indirect object notation | |
14:23 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=25898 enhancement, P5 - low, ---, julian.maurice, Needs Signoff , Prohibit indirect object notation |
14:23 | marcelr | We are not going to change all occurrences ? |
14:23 | Joubu | yes we are |
14:23 | the patch is there already | |
14:23 | marcelr | rebase party? |
14:23 | Joubu | easy conflicts :) |
14:23 | marcelr | conflicts-- |
14:23 | Joubu | most of the occurrences are new CGI |
14:24 | better now than later IMO | |
14:24 | we will certainly have to do it anyway | |
14:24 | marcelr | perl 7 |
14:24 | cait2 | is it just style or does/will have it an effect? |
14:25 | ah, deprecation? | |
14:26 | marcelr | but we vote for the guideline not for the bulk change |
14:26 | Joubu | I did not plan to vote on that |
14:26 | should we? | |
14:26 | marcelr | well not for me, it is obvious better |
14:26 | Joubu | :) |
14:27 | marcelr | just making the difference |
14:27 | tcohen | let's fix it |
14:27 | Joubu | I opened a perl 7 omnibus, in case we have other things to deal with |
14:27 | bug 26303 | |
14:27 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=26303 enhancement, P5 - low, ---, koha-bugs, NEW , [OMNIBUS] Preparing for Perl 7 |
14:27 | tcohen | we could make a rule: you touch a file with indirect calls, you're oblighed to fix it inline |
14:27 | Joubu | I would prefer a fix all at once patch |
14:28 | cait2 | maybe we could add a link to the perl 7 thing to the cuideline |
14:28 | Joubu | let's continue the discussion on the bug report if you want to |
14:28 | cait2 | as a reminder of why |
14:28 | Joubu | there is a link to https://www.effectiveperlprogr[…]-object-notation/ |
14:29 | cait2 | oh sorry, i missed that |
14:29 | i was expecting it on top | |
14:29 | can we have the QA tools look for it as well please? | |
14:29 | Joubu | #info tuxayo integrate the RMaint-handover-draft into the "release maintenance" wiki page (postponed) |
14:29 | I talked with him, he will get back to it soon | |
14:29 | #action tuxayo integrate the RMaint-handover-draft into the "release maintenance" wiki page (postponed) | |
14:30 | #info davidnind Find and update relevant places to record Perl version required, including release notes, manual and wiki (one source of the truth): update - still working on this (postponed) | |
14:30 | and I don't have news from this | |
14:30 | #action davidnind Find and update relevant places to record Perl version required, including release notes, manual and wiki (one source of the truth): update - still working on this (postponed) | |
14:30 | #topic General development discussion (trends, ideas, ...) | |
14:30 | Topic for #koha is now General development discussion (trends, ideas, ...) (Meeting topic: Development IRC meeting 26 August 2020) | |
14:31 | Joubu | ha cait2, got it now :) |
14:31 | I am slow you now | |
14:31 | indeed, I mixed info and topic | |
14:31 | subtopic is missing | |
14:31 | #subtopic | |
14:31 | anyone, an additional discussion? | |
14:31 | ideas? | |
14:31 | wahanui | ideas are good! |
14:32 | cait2 | not right now |
14:32 | maybe just a reminder: please write nice commit messages | |
14:32 | the trend is to minimalistic | |
14:32 | marcelr | i have two patches underway dealing with availability on opac xslt results |
14:33 | always candidates for In Discussion | |
14:33 | cait2 | tricky area, haven't read yet |
14:34 | Joubu | marcelr: bug numbers? |
14:34 | marcelr | bug 21260 |
14:34 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=21260 normal, P5 - low, ---, m.de.rooy, Needs Signoff , Improve the Availability line of OPAC XSLT search results |
14:34 | marcelr | bug 26302 |
14:34 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=26302 enhancement, P5 - low, ---, m.de.rooy, ASSIGNED , OPAC XSLT Results: List variable number of itemcallnumbers |
14:34 | marcelr | some nasty xslt there |
14:34 | with keys and generate etc | |
14:35 | cait2 | also quite a bit of configuration options at play whne i remember correctly |
14:35 | Joubu | I wrote that few weeks ago and pushed yesterday, maybe related? bug 26139 |
14:35 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=26139 normal, P5 - low, ---, jonathan.druart, Pushed to master , 'Place hold' button isn't hidden in all detail views if there are no items available for loan |
14:35 | Joubu | it's not about xslt, but availability |
14:35 | cait2 | always have refused to try and customize that |
14:36 | Joubu | not really |
14:37 | marcelr: how do you want to promote them? | |
14:37 | marcelr | good question; an old problem |
14:37 | Joubu | maybe you could detail the gain it will bring, and ask the general ML for a signoff? |
14:38 | marcelr | my impression is that mailing the regular ML does not trigger signoffs |
14:38 | Joubu | it depends on how you sell it :) |
14:38 | marcelr | good developers cannot sell |
14:38 | Joubu | haha, true |
14:38 | marcelr | and the opposite around |
14:39 | Joubu | I think it could help to list the different situations |
14:39 | the bad old ones, and how your patch will fix them | |
14:39 | even for us, developers | |
14:40 | or maybe it's the test plan from comment 8 | |
14:40 | marcelr | we'll see |
14:40 | Joubu | ok |
14:40 | I think a before vs after could help | |
14:40 | kidclamp | I only looked quickly marcel, but it seems you remove some info from non-available items, I think that should be optional |
14:41 | rather hide dsipaly than take it away entirely | |
14:41 | marcelr | good observation |
14:41 | Joubu | #info |
14:42 | #info Please have a look at bug 21260 (Improve the Availability line of OPAC XSLT search results) | |
14:42 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=21260 normal, P5 - low, ---, m.de.rooy, Needs Signoff , Improve the Availability line of OPAC XSLT search results |
14:42 | Joubu | ok moving on |
14:42 | #topic Review of coding guidelines | |
14:42 | Topic for #koha is now Review of coding guidelines (Meeting topic: Development IRC meeting 26 August 2020) | |
14:42 | Joubu | we have something! |
14:42 | marcelr | kidclamp: there is even an old report asking for callnumbers on all statuses |
14:42 | Joubu | #info Don't use input type "number" in favor of using a pattern check instead |
14:43 | #link https://technology.blog.gov.uk[…]type-for-numbers/ | |
14:43 | #link https://bugs.koha-community.or[…]_bug.cgi?id=17098 | |
14:43 | huginn | Bug 17098: normal, P5 - low, ---, gmcharlt, NEW , Validation problems with form field type "number" (decimal separator) |
14:44 | Joubu | cait2: did you add that? |
14:44 | was the idea to vote or propose a guideline at the next meeting? | |
14:44 | marcelr | how often did we use that ? |
14:44 | Joubu | it happens |
14:44 | twice last week ;) | |
14:45 | marcelr | 10-15 files |
14:45 | 20 | |
14:45 | Joubu | #action Joubu open a bug report for the QA script to catch input type number |
14:46 | I read the .gov.uk link when ashimema[m] sent it and it made sense | |
14:46 | marcelr | yes, it makes sense |
14:46 | Joubu | now I have to admit I cannot remember what was the arguments |
14:47 | ok, Martin is on vacation so... | |
14:47 | marcelr | could be coming from the accessiblity community |
14:47 | Joubu | #action ashimema write a guideline to forbid usage for type "number" |
14:48 | and we are good I think | |
14:48 | it said "review of the coding guideline" but there is no guideline | |
14:48 | anything more, anyone? | |
14:48 | #topic Set time of next meeting | |
14:48 | Topic for #koha is now Set time of next meeting (Meeting topic: Development IRC meeting 26 August 2020) | |
14:49 | Joubu | #info Next meeting: 9 September 2020, 20 UTC |
14:50 | @later tell rangi the next dev meeting is scheduled for 9 September 2020, 20 UTC. Can you confirm 20 UTC is good for nz/au? | |
14:50 | huginn | Joubu: The operation succeeded. |
14:50 | Joubu | #endmeeting |
14:50 | Topic for #koha is now Welcome to #koha this channel is for discussion of the Koha project and software https://koha-community.org | |
14:50 | huginn | Meeting ended Wed Aug 26 14:50:13 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) |
14:50 | Minutes: https://meetings.koha-communit[…]-08-26-14.00.html | |
14:50 | Minutes (text): https://meetings.koha-communit[…]0-08-26-14.00.txt | |
14:50 | Log: https://meetings.koha-communit[…]26-14.00.log.html | |
14:52 | koha-jenkins | Project Koha_Master_D10_Deps build #60: SUCCESS in 43 min: https://jenkins.koha-community[…]ster_D10_Deps/60/ |
14:54 | Project Koha_Master_D11 build #73: UNSTABLE in 44 min: https://jenkins.koha-community[…]ha_Master_D11/73/ | |
14:56 | cait2 | ugh sorry, phone call |
14:57 | Joubu: i wrote "New" proposing a new guideline so can tell people to stop using number | |
14:57 | see also the bug i linked, we had tons of troubel with it in context of I18N | |
14:58 | also accessibility, which ashimema's report focuses on | |
14:58 | so multiple reasons | |
14:58 | kohaputti | I just noticed Joubu's QA team email regarding those big bugs needing QA. I just wanted to remind you all what I said earlier about that I have not found the follow-up patch style of fixes good for doing QA quickly. It takes so much longer for me to get into knowing the true intention of the commits when I have to switch between multiple commits messages instead of reading one story at a time. I think doing revision of patches is |
14:58 | better way and adding note on the commit about what was changed since previous revision so everybody remembers what is fixed already. | |
14:58 | margaret joined #koha | |
14:59 | cait2 | hm the first commit shoudl have the full story |
14:59 | kohaputti | just saying this aloud again in case some other people join my single person movement towards revision based patches :P |
14:59 | cait2 | but not sure what revision based would mean |
14:59 | kohaputti | cait2, no, the commit message should be about the patch content |
14:59 | cait2 | i don't want to review all the code again when i just want to know how they fixed the bug i found |
14:59 | that's why i really like follow-up patches | |
15:00 | but maybe i am misudnerstanding how it would work your way | |
15:01 | i don't usually reveiw one after the other, but the patches altogether, just checking in the second, third... round the follow-ups for requested changes specifically | |
15:01 | kohaputti | cait2, you probably understood right, here you have to do the review again every time |
15:02 | cait2, if you review all together you need to use something like git diff HEAD~X..HEAD for that and it misses the inteded story in the commit messages, you only get what is the story in the code which might be accidentally wrong | |
15:03 | cait2 | it depends on the patch |
15:03 | but yes, i use git diff HEAD~2.. or so | |
15:03 | kohaputti | the bug Joubu linked in QA team email, https://gitlab.com/joubu/Koha/commits/bug_20271, would benefit of the revision, v1, v2, v3, etc. based patches |
15:04 | cait2 | yes.... but one big patch? |
15:04 | kohaputti | one patch per logical change |
15:05 | cait2 | but even there... I find it worthwhile to see what was fixed how and where of the bugs found |
15:05 | it gives clues on where to look next | |
15:05 | kohaputti | like "Merge deleted_items to items table", second one "Merge deleted_biblioitems to biblioitems", etc. |
15:06 | cait2 | i would have loved for this to be a bit more granular... like do one table after theother, but i was told it was not good to do that because it all touches the same code |
15:06 | kohaputti | cait2, you have the discussion in bugzilla where you point out any mistakes found. |
15:07 | cait2 | the commit messages n the follow up shoudl have some info too ideally |
15:07 | i guess there are pros and cons to both approaches | |
15:07 | lisettelatah joined #koha | |
15:07 | cait2 | but that truly is the scariest bug we have right now |
15:07 | got to sign off work for now - bbl probably | |
15:10 | kohaputti | Other "pro" for the revision based approach was easy revertability of commits if needed, with follow-ups it seems really hard since you don't know which one of them you need to revert until you go trough all of them. |
15:10 | tcohen | I love the approach we took for the holds table |
15:11 | because it focuses on preserving reports | |
15:11 | and takes the opportunity to alter the column names following the currently agreed guidelines | |
15:13 | I agree with moving forward with 20271, I'm just mentioning this because it is one of those things that worry us, service providers. All the work that will need to be done after this changes | |
15:16 | inlibro joined #koha | |
15:16 | koha-jenkins | Project Koha_Master_U16 build #52: SUCCESS in 57 min: https://jenkins.koha-community[…]ha_Master_U16/52/ |
15:24 | reiveune | bye |
15:24 | reiveune left #koha | |
15:41 | koha-jenkins | Project Koha_Master_D9_MDB_Latest build #373: SUCCESS in 46 min: https://jenkins.koha-community[…]9_MDB_Latest/373/ |
15:43 | Project Koha_Master_D9_My8 build #397: STILL UNSTABLE in 51 min: https://jenkins.koha-community[…]aster_D9_My8/397/ | |
15:46 | wshealy joined #koha | |
15:57 | wshealy | Did anyone else loose hair and sleep the first time they installed Koha? |
16:01 | I have move on to challenges with library.conf in apache which koha create generated. It has no document root and I have no clue where it is finding the default apache page instead of koha intranet. I deleted apache default page from the default directory but it still finds it. | |
16:15 | khall_ joined #koha | |
16:16 | inlibro joined #koha | |
16:18 | mtj | hi wshealy, have you restarted apache after your apache changes? |
16:18 | wshealy: this install guide is good -> https://wiki.koha-community.or[…]ki/Koha_on_Debian | |
16:21 | $ sudo a2dissite 000-default | |
16:21 | $ apachectl restart | |
16:21 | oleonard | Hi again |
16:24 | mtj | wshealy: ive been hacking on koha for a while... have no hair and its 4am |
16:25 | caroline_test joined #koha | |
16:32 | wshealy | yes |
16:35 | caroline joined #koha | |
16:43 | wshealy | Progress, now I'm down for maintenance. Just got to figure out how to get to koha install and onboard |
16:49 | khall joined #koha | |
17:04 | cait joined #koha | |
17:16 | inlibro joined #koha | |
17:17 | wshealy | Help, instead off the install login I'm getting offline for maintenance on both the itra and cat sides |
17:21 | mjt, thanks...I think I have come to the end of Koha on Debian and now have to figure out why I'm getting offline and not install login | |
17:51 | oleonard | wshealy I'm not sure what to suggest but it sounds like an Apache config problem. You're getting served the OPAC at both addresses. |
17:52 | wshealy | Thanks |
17:52 | wish there was someone to talk too. doing this alone from home is no fun | |
18:16 | inlibro joined #koha | |
18:26 | wizzyrea | wshealy, sudo koha-enable <whatever you called your instance> |
18:26 | no brackets | |
18:27 | ^ it would be helpful to know if that's a "pretty" maintenance page, or a very texty looking maintenance page. | |
18:36 | wshealy | Pretty |
18:45 | Tired, taking a break | |
18:45 | cait | then it's probably the opac you are seeing |
18:45 | the staff side doesn't have a meintenance message | |
18:45 | caroline | anyone know what tax_rate_bak and tax_value_bak are for in aqorders? |
18:45 | wshealy | yep don't know why...Debug apache not my strong suit |
18:46 | caroline | in https://wiki.koha-community.or[…]ders_endpoint_RFC it says they were removed, but I still see them |
18:46 | tcohen | where do you see them? |
18:46 | we try to make the API expose what we belive is a good model for our data | |
18:47 | caroline | here and in my db http://schema.koha-community.o[…]les/aqorders.html |
18:47 | cait | caroline: _bak = backup |
18:47 | tcohen | but changing the underlying database structure requires much more efforts |
18:47 | cait | new databases shouldn't have those |
18:47 | wshealy | Thanks everybody, I'll be back --- far cry from loading wordpress - need a break |
18:47 | caroline | is it in update_database to remove them,? |
18:47 | cait | they were used to backup values when the tax calculation was changed |
18:47 | i think we kept them for existing | |
18:48 | that had the calculations remodel - safety measuere | |
18:48 | caroline | ah ok, then that's probably why |
18:48 | cait | but when you look at a newish database/installatoin they should not be there |
18:48 | caroline | I just update my database, I never create new ones |
18:48 | cait | all ours have these i think ormost |
18:48 | caroline | it's still in the schema though http://schema.koha-community.o[…]les/aqorders.html |
18:48 | cait | hm ineresting |
18:48 | Joubu would know i think | |
18:48 | hte was the tax rewriter | |
18:49 | caroline | acq are so complicated... (I'm trying to map existing data from another system to koha) just trying to figure out what all the different prices are and how they match, ugh |
18:50 | cait | i feel you... i need to continue with that tomorrow |
18:51 | but i don't think you need to use them | |
18:51 | if you look at new orders they shoudl not be filled | |
18:52 | caroline | yeah, I don't think they arefilled in my test db either. but I needed to know in case data from the old system should go in there |
18:54 | old system has 19 fields for prices and taxes and things, Koha has 18... that's so many! | |
18:55 | cait | ugh1 |
18:55 | ! | |
18:55 | can you share the system name? | |
18:55 | caroline | Portfolio |
18:55 | cait | hm never heard around here |
18:56 | hope it works all out! | |
18:56 | caroline | yeah, they're mostly here in Quebec and a little bit in france |
18:56 | we have a lot of small ils companies whose market is only quebec | |
18:59 | cait | there are a few german ones, but not that many i think |
19:13 | caroline | if you need help with your prices, you can look at bug 23981 to see what is used where |
19:13 | huginn | Bug http://bugs.koha-community.org[…]_bug.cgi?id=23981 normal, P5 - low, ---, koha-bugs, NEW , Inconsistencies with the different prices |
19:14 | caroline | I was looking at my things and going "grr" this doesn't make any sense!" then I go to bz and I see I reported the bug, hehe seems it's not the first time I've run into this |
19:16 | inlibro joined #koha | |
19:23 | cait | yeah |
19:23 | it#s not much better in the -german translation | |
19:29 | hm i am trying to run selenium tests on kohadevbox and not sure what is wrong but hte selenium server doesn't seem to start? | |
19:29 | start_selenium -v | |
19:29 | [1] 11832 | |
19:29 | [1]+ Exit 1 Xvfb :1 -screen 0 1024x768x24 > /dev/null 2>&1 | |
19:30 | tcohen: around maybe? | |
19:30 | tcohen | yes |
19:30 | cait | can you help maybe? |
19:30 | in user.yml selenium is already set to true | |
19:31 | when i run the test i get: Selenium server did not return proper status at /usr/local/share/perl/5.24.1/Selenium/Remote/Driver.pm line 543. | |
20:08 | did a complete provision and still no selenium | |
20:12 | tcohen | sorry, I cannot take a look now |
20:14 | cait | tcohen: i think i fixed it maybe |
20:14 | tcohen | glad to hear |
20:16 | cait | ... selenium server appears to be running now |
20:16 | kohadev-kohakohadevbox:/home/vagrant/kohaclone$ perl t/db_dependent/selenium/authentication.t | |
20:16 | 1..2 | |
20:16 | Could not create new session: Unable to connect to host 127.0.0.1 on port 7055 after 45000 ms. Firefox console output: | |
20:16 | but... no luck with the tests | |
20:16 | inlibro joined #koha | |
20:41 | rangi | @later tell joubu yes that is a good time |
20:41 | huginn | rangi: The operation succeeded. |
20:46 | * oleonard | waves to rangi |
20:46 | * cait | waves to rangi |
20:47 | rangi | hi oleonard and cait |
21:04 | aleisha | hello |
21:10 | kathryn joined #koha | |
21:10 | aleisha | @later tell Joubu sorry bout that, hopefully caught the branch up now! |
21:10 | huginn | aleisha: The operation succeeded. |
21:17 | inlibro joined #koha | |
21:39 | cait | kidclamp: still around? |
21:54 | * kidclamp | isn't far |
21:54 | kidclamp | Cait still around? |
21:55 | cait | here :) |
21:55 | i had a question about elastic but -r seemed to have the desired effect | |
21:55 | youare probably ont going to be happy withme for bringing up cyrillic | |
21:58 | kidclamp | https://stackoverflow.com/ques[…]acters-diacritics |
21:59 | You aren't wrong cait, but I will say incremental improvements are better than nothing ;-) | |
22:00 | cait | yeah i coudl see that coming |
22:00 | but i thought it was worth asking | |
22:01 | kidclamp: but i think we need to look for perl + regex in this case | |
22:02 | better support there | |
22:02 | "ou can match a single character belonging to the “letter” category with \p{L}." | |
22:03 | it doesn't read too hard here - maybe worth trying out | |
22:03 | https://www.regular-expressions.info/unicode.html | |
22:03 | \p{Lu} or \p{Uppercase_Letter}: an uppercase letter that has a lowercase variant. | |
22:04 | https://perldoc.perl.org/perlunicode.html | |
22:05 | For instance, \p{Uppercase} matches any single character with the Unicode "Uppercase" property, while \p{L} matches any character with a General_Category of "L" (letter) property | |
22:05 | kidclamp: that looks quite promising? | |
22:05 | kidclamp | That looks reasonable! Will try tomorrow, thank you! |
22:05 | Or you can offer a follow-up :-) | |
22:05 | cait | too late - almost midnight |
22:05 | i think better is tick with testing if that's ok | |
22:05 | ? | |
22:06 | actually i might have just turned into a pumpkin, 5 minutes past :) | |
22:07 | commenting with the link on the bug | |
22:09 | kidclamp | Hehe |
22:09 | cait | just finishing another go on the elastisearch tests fixes |
22:17 | inlibro joined #koha | |
22:34 | alexbuckley joined #koha | |
22:38 | cait | alexbuckley: great presentatoin :) I also had expected a percentage value when looking at the discount feature the first time |
22:39 | alexbuckley | hey cait , thank you! |
22:40 | good to hear others interpreted it the same way :) | |
22:40 | *the discount feature | |
23:00 | sophie_m left #koha | |
23:01 | sophie_m joined #koha | |
23:05 | dcook | @later tell Joubu I think 9 September 2020, 20 UTC sounds good for AU. That would be 8am for us. Not bad at all. |
23:05 | huginn | dcook: The operation succeeded. |
23:07 | oleonard_ joined #koha | |
23:08 | oleonar__ joined #koha | |
23:17 | inlibro joined #koha |
← Previous day | Today | Next day → | Search | Index