Bug 33383 – Wrong protocol shown when opening printer shares

Bug 33383 - Wrong protocol shown when opening printer shares


Summary:	Wrong protocol shown when opening printer shares

Status:	CLOSED FIXED

Product:	UCS
Classification:	Unclassified
Component:	UMC - Printers
Version:	UNSTABLE
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS 3.2-0-errata
Assigned To:	Florian Best
QA Contact:	Dirk Wiesenthal

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2013-11-14 15:12 CET by Lukas Walter
Modified:	2014-01-29 11:21 CET (History)
CC List:	3 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
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 Lukas Walter

2013-11-14 15:12:29 CET

When opening an existing printer share in the UMC, protocol is allways set to "cups-pdf" instead of the correct value as stored in LDAP.

Comment 1 Lukas Walter

2013-12-11 17:00:24 CET

Might be a problem with how UMC handles the PrinterURI complex yntax, which consists of a PrinterProcol and a string syntax.

Seems to be working in other modules with other syntaxes.

Comment 2 Florian Best

2013-12-12 14:48:18 CET

The printer URI used the ComplexInput widget which did not have a setInitialValue method. The printer protocol had a ComboBox widget. When the form values were set, the ComboBox widget did not yet have the possible values loaded from the server (udm/syntax/choices request) when the value was set, so the ComboBox declared the value as invalid. It falled back to the first value (which was cupd-pdf://) when the values were finished loaded.

Fixed by adding setInitialValue to ComplexInput which calls setInitialValue for every subtype.

YAML: 2013-12-12-univention-management-console-frontend.yaml

Comment 3 Dirk Wiesenthal

2014-01-23 11:37:50 CET

OK, works.

YAML OK

Comment 4 Moritz Muehlenhoff

2014-01-29 11:21:16 CET

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