Bug 54811 – [Kelvin] Format hooks are not executed

Bug 54811 - [Kelvin] Format hooks are not executed


Summary:	[Kelvin] Format hooks are not executed

Status:	NEW

Product:	UCS@school
Classification:	Unclassified
Component:	HTTP-API (Kelvin)
Version:	UCS@school 5.0
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	UCS@school maintainers
QA Contact:

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2022-05-31 15:25 CEST by Jan-Luca Kiok
Modified:	2022-06-01 07:26 CEST (History)
CC List:	2 users (show)

See Also:
What kind of report is it?:	Bug Report
What type of bug is this?:	4: Minor Usability: Impairs usability in secondary scenarios
Who will be affected by this bug?:	3: Will affect average number of installed domains
How will those affected feel about the bug?:	2: A Pain – users won’t like this once they notice it
User Pain:	0.137
Enterprise Customer affected?:
School Customer affected?:	Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Jan-Luca Kiok

2022-05-31 15:25:23 CEST

When using a format hook in Kelvin it is loaded, but never seems to run (no debug statements and even no exceptions seem to be raised).

To test this I copied to FormatPyHook from the Kelvin Unittests: https://git.knut.univention.de/univention/ucsschool/-/blob/feature/kelvin/kelvin-api/tests/test_pyhook.py#L67

The hook is loaded ("Loaded hooks: {'patch_fields_staff': ['CamelCaseLastnameFormatPyHook.patch_fields_staff'], ..."), but nothing changes and my guess is that the "patch_" methods are not evaluated in any way.