User:Victor Grousset - tuxayo/Community work/Reports/2024-02
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. Or questions to clarify something in the code.
- Partial QA is a way to contribute to a ticket that I can't QA alone but for which I can confidently review only part of the code or aren't sure enough about understanding the functional need and change.
Found issues: Bug 35357 - Item not removed from holds queue when checked out to a different patron
Bug 34234 - Item groups dropdown in detail page modal does not respect display order
Found minor issues: Bug 30324 - Parent and child itemtype checkout limits not enforced as expected
Round two: Bug 35357 - Item not removed from holds queue when checked out to a different patron
QA + minor code tidy up: Bug 31286 - Embed see-from headings into bibliographic records export
Note for the people funding my work and the rest of the community for transparency purposes: QA of an enhancement from BibLibre.
For QA of enhancements I'm asked to ideally focus on one monthly.
Found issues & added minor fixes: Bug 16117 - Log invalid patron cardnumbers and item barcodes
Note for the people funding my work and the rest of the community for transparency purposes: QA of an enhancement from ByWater Solutions.
For QA of enhancements I'm asked to ideally focus on one monthly.
Maybe found issues + minor follow-up: Bug 34793 - We should ship default 'CookieConsentPopup' data that describes our required cookies
Note for the people funding my work and the rest of the community for transparency purposes: QA of an enhancement from PTFS Europe. (as well as the next two very closely related tickets)
For QA of enhancements I'm asked to ideally focus on one monthly.
QA + minor follow-up Bug 35347 - 'More information' should always display in cookie consent bar
Bug 35346 - 'Accept essential cookies' should always appear if CookieConsent is enabled
Some security ticket
QA + minor follow-ups: Bug 35904 - C4::Auth::checkauth cannot be tested easily
Found issue: Bug 35610 - Missing FK on old_reserves.branchcode?
Found issue: Bug 32565 - Holds placed when all libraries are closed do not get added to holds queue if HoldsQueueSkipClosed and RealTimeHoldsQueue are enabled
Found issue: Bug 32702 - Item statuses that block holds should be checked in CanItemBeReserved
Found minor issue: Bug 35924 - The 'checkin slip' button should not be available for patrons whose privacy is set to never
Maybe found issue + move tests to separate commit Bug 35997 - Cancelling a hold should remove the hold from the queue
Found issue: Bug 27769 - Advanced editor shouldn't break copying selected text with Ctrl+C
Bug 35913 - Item order prices do not fall back to MarcFieldsToOrder if not set by MarcItemFieldsToOrder
Attempt to unblock: Bug 24172 - Locale is ignored when saving season name in the database
Alternative proposal for: Bug 35948 - cleanup_database.pl should remove not only finished background jobs
Some security bug
Bug 36010 - Items/AutomaticItemModificationByAge.t is failing
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.
Bug 22421 - accountlines.issue_id is missing a foreign key constraint
Attempt: Bug 29509 - GET /patrons* routes permissions excessive
Attempt: Bug 32707 - ElasticSearch should not auto truncate (even if QueryAutoTruncate = 1) for identifiers (and some other fields)
Quick assessment that it's not ready for testing: Bug 24172 - Locale is ignored when saving season name in the database
Bug 35714 - Clicking Print slips when no letter template selected causes error
Bug 35591 - bulkmarcimport.pl overwrites data if controlnumber eq authid
Test and fix test plan Bug 35959 - Inconsistent hierarchy during C3 merge of class 'Koha::AuthorisedValue' (and a few other modules)
Bug 28869 - Optionally restrict authorised values to tinyint
Can't reproduce but no regression found with patch: Bug 35927 - Selecting MARC framework again doesn't work when adding to basket from an external source
Signed off anyway to not have it blocked forever because it fixed the issue in the affected instance. And clarified test plan.