Bug 36472 – python-notifier: timer handling is broken in nf_qt.py

Bug 36472 - python-notifier: timer handling is broken in nf_qt.py


Summary:	python-notifier: timer handling is broken in nf_qt.py

Status:	CLOSED FIXED

Product:	UCS
Classification:	Unclassified
Component:	General
Version:	UCS 3.2
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS 3.2-3-errata
Assigned To:	Sönke Schwardt-Krummrich
QA Contact:	Stefan Gohmann

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2014-11-06 10:40 CET by Sönke Schwardt-Krummrich
Modified:	2015-03-04 16:54 CET (History)
CC List:	3 users (show)

See Also:	37378 37937
What kind of report is it?:	---
What type of bug is this?:	---
Who will be affected by this bug?:	---
How will those affected feel about the bug?:	---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
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 Sönke Schwardt-Krummrich

2014-11-06 10:40:57 CET

The timers in the qt notifier backend are broken. The callbacks of the timers are never called.

Comment 1 Sönke Schwardt-Krummrich

2014-11-06 12:29:44 CET

The qt backend of python-notifier did not pass a parent object to the QTimer objects. Thus the qt main loop never processed any timer event. This broke the UCS@school computerroom module:
- it did not shut down after module timeout
- it did not shut down after EXIT command

Added 2014-11-06-python-notifier.yaml

The patch has also been ported to UCS 4.0.

Comment 2 Sönke Schwardt-Krummrich

2014-11-06 12:30:19 CET

The patch has been sent upstream today.

Comment 3 Stefan Gohmann

2014-11-06 14:48:24 CET

YAML: OK

Reproduction: OK

Code: OK

Tests: OK

4.0 merge: OK

UMC ucs-test: OK

Comment 4 Janek Walkenhorst

2014-11-07 15:39:27 CET

http://errata.univention.de/ucs/3.2/241.html