User:Victor Grousset - tuxayo/Community work/Reports/2021-02

Release maintenance of 19.11.x branch

Backport list: 41

• Bug 27308 - Advanced editor should skip blank lines when inserting new fields
• Bug 24863 - QueryFuzzy syspref says it requires Zebra but Elasticsearch has some support
• Bug 27196 - Waiting title level hold checked in at wrong location via SIP leaves hold in a broken state and drops connection
• Bug 26713 - Add pagination to bottom of saved SQL reports table
• Bug 24254 - Add Koha::Items->filter_by_visible_in_opac
• Bug 27299 - Zebra phrase register is incorrectly tokenized when using ICU
  • Reverted it, see the ticket discussion.
• Bug 15448 - Placing hold on specific items doesn't enforce OpacHiddenItems
• Bug 17429 - Document the --plack option for koha-list
• Bug 27416 - String 'Modify tag' in breadcrumb is untranslatable
• Bug 24272 - Add a command line script to compare the syspref cache to the database
• Bug 27192 - Set focus for cursor to item type input box when creating new item types
• Bug 27457 - Set focus for cursor to Debit type code field when creating new debit type
• Bug 7143 - Bug for tracking changes to the about page
• Bug 24447 - POD of C4::Members::Messaging::GetMessagingPreferences() is misleading
• Bug 20527 - <label> linked to the wrong <input> (wrong "for" attribute) in paycollect.tt
• Bug 24810 - French SQL files for "news" contain "Welcome into Koha 3!"
• Bug 24811 - French SQL files for "news" contain broken link to the wiki
• Bug 27210 - Typo in patron-attr-types.tt
• Bug 27179 - Misspelling of Method in REST API files
• Bug 27327 - Indirect object notation in Koha::Club::Hold
• Bug 27333 - Wrong exception thrown in Koha::Club::Hold::add
• Bug 11344 - Perldoc issues in misc/cronjobs/advance_notices.pl
• Bug 27531 - Remove type attribute from script tags: Cataloging plugins
• Bug 27547 - Typo in parcel.tt
• Bug 14004 - Add ability to temporarily disable added CSS and Javascript in OPAC and interface
• Bug 20238 - Show description of ITEMTYPECAT instead of code in itemtypes summary table
• Bug 27580 - NULL values not correctly handled in Koha::Items->filter_by_visible_in_opac
• Bug 27554 - Clarify and add tests for Koha::Patrons->update_category_to child to adult
• Bug 27450 - Making password required for patron registration breaks patron modification
• Bug 26298 - If MaxItemsToProcessForBatchMod is set to 1000, the max is actually 999
• Bug 27576 - Don't show import records table when cleaning a batch
• Bug 27548 - Warnings "use of uninitialized value" on branchoverdues.pl
• Bug 27549 - Warning "use of uninitialized value" on renew.pl
• Bug 27395 - Add warning to PatronSelfRegistrationDefaultCategory to avoid accidental patron deletion
• Some security bug
• Bug 27586 - Import patrons script has a confirm switch that doesn't do anything
• Bug 27676 - finesMode=off not correctly handled
• Bug 27422 - HTML5Media is http by default but Youtube is https only
• Some security bug
• Bug 26958 - Move Elasticsearch mapping template JS to the footer
• Bug 27707 - Renewing doesn't work when renewal notices are enabled

Minor stuff

Patchsets analyzed but not backported: 33

This plus the backported patches gives and idea of the stream of patches to process.

Not backported due to one of the following:

• severity too low considering this is the oldoldstable branch of Koha
• missing dependencies
• not enough benefits compared to the risks of regression

Backport attempts: 3

Some time needed to determine whether it impacts 19.11 or the feasibility of the backport. And also attempts to backport not possible due to code conflicts.

• Bug 25808 - Renewal via the SIP 'checkout' message gives incorrect message
• Bug 26634 - Hold rules applied incorrectly when All Libraries rules are more specific than branch rules
• Bug 27530 - Sample patron data should be updated and/or use relative dates
• Some security patch

QA

Notes:

• An attempt results in feedback provided on the bug. To ask if I correctly followed the test plan due to results that I couldn't interpret. Or if there is indeed an issue.
• Partial QA is a way to contribute to a ticket that I can't QA alone but for which I can apply the test plan and run the QA script. Leaving the functional or code review for someone knowing that area.

Partial QA: Bug 18017 - Use index_heading and index_match_heading in UNIMARC authorities zebra configuration

Tested the patch and reached out for help to review the UNIMARC part.

Began analysis of 9 patchsets

Often one must start reviewing a patch and test plan to really know if it's possible or not to QA it. This is still a way to get to know more Koha and the code though. And eventually find issues.

Signoffs (testing submitted patches so they can move on to be integrated)

Note: an attempt usually results in feedback provided on the bug. To ask if I correctly followed the test plan due to results that I couldn't interpret. Or if there is indeed an issue.

Misc tasks and work on tickets: Analysis or implementation

Bug 25285 - Wrong message when plugin minimum version isn't met

Found bug origin and what should be written instead.

Bug 27619 - Remove fr-FR installer data

Began analysis to try to list the subtasks

Research and discussions about eventual changes of the testing stack

https://lists.koha-community.org/pipermail/koha-devel/2021-February/046278.html

Suggest adding a new tip in the french translation documentation

Based on https://wiki.koha-community.org/wiki/Translating_Koha#Translating_system_preferences

Chaired development meeting of February the 24st

https://wiki.koha-community.org/wiki/Development_IRC_meeting_24_February_2021

Documentation

Release process: Change the way the system requirements and recommendations are communicated in the release notes

https://wiki.koha-community.org/wiki/Release_maintenance#System_requirements_and_recommendations

New tickets opened

i.e. Found something wrong and documented how to reproduce it. Or suggest an enhancement (usually for dev tools)

koha-testing-docker: YAML::XS not installed, fail to start master

https://gitlab.com/koha-community/koha-testing-docker/-/issues/243

koha-testing-docker: Debian 11 doesn't have the `ip` command

https://gitlab.com/koha-community/koha-testing-docker/-/issues/244

Debian docker image: debian:bullseye (11) doesn't have iproute2 installed (providing the ip command)

https://github.com/debuerreotype/docker-debian-artifacts/issues/117