Bug 37840 - Traceback in computer room UMC module when deactivating printing
Traceback in computer room UMC module when deactivating printing
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - Computer room
UCS@school 4.2
Other Linux
: P5 normal (vote)
: UCS@school 4.2 v2
Assigned To: Florian Best
Daniel Tröder
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-02-18 19:33 CET by Arvid Requate
Modified: 2017-06-23 13:32 CEST (History)
2 users (show)

See Also:
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
vunset.patch (605 bytes, patch)
2015-02-18 19:33 CET, Arvid Requate
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Arvid Requate univentionstaff 2015-02-18 19:33:44 CET
Created attachment 6698 [details]
vunset.patch

The following traceback occurred in the computerroom UMC module when changed the print mode for a room from default (global settings) to deactivated and clicked on the "set" (save) button:

============================================================================
Die Ausführung des Kommandos computerroom/settings/set ist fehlgeschlagen:

Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.7/univention/management/console/modules/__init__.py", line 176, in _decorated
    return function(self, request, *args, **kwargs)
  File "/usr/lib/pymodules/python2.7/univention/management/console/modules/computerroom/__init__.py", line 192, in _decorated
    return func(self, request)
  File "/usr/lib/pymodules/python2.7/univention/management/console/modules/computerroom/__init__.py", line 781, in settings_set
    del vunset[varname]
TypeError: list indices must be integers, not str
============================================================================

It's true, vunset is an array, not a dictionary. The attached patch fixed the problem for me.
Comment 1 Florian Best univentionstaff 2016-11-07 06:33:13 CET
Still unfixed.
Comment 2 Florian Best univentionstaff 2017-06-01 13:02:25 CEST
Occurred again.
Comment 3 Florian Best univentionstaff 2017-06-01 13:11:33 CEST
Applied that patch.

ucs-school-umc-computerroom.yaml:
r79978 | YAML Bug #37840

ucs-school-umc-computerroom (9.0.8-1):
r79977 | Bug #37840: fix syntax error
Comment 4 Daniel Tröder univentionstaff 2017-06-01 15:19:43 CEST
OK: code
OK: advisory
Comment 5 Sönke Schwardt-Krummrich univentionstaff 2017-06-23 13:32:27 CEST
UCS@school 4.2 v2 has been released.

http://docs.software-univention.de/changelog-ucsschool-4.2v2-de.html

If this error occurs again, please clone this bug.