Bug 42673 – MultiInput and ComboBox.ready() is broken

Bug 42673 - MultiInput and ComboBox.ready() is broken


Summary:	MultiInput and ComboBox.ready() is broken

Status:	REOPENED

Product:	UCS
Classification:	Unclassified
Component:	UMC (Generic)
Version:	UCS 4.4
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	UMC maintainers
QA Contact:

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2016-10-13 17:44 CEST by Florian Best
Modified:	2019-03-13 14:36 CET (History)
CC List:	1 user (show)

See Also:	42177
What kind of report is it?:	Development Internal
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
patch (1.39 KB, patch) 2016-10-13 17:44 CEST, Florian Best	Details \| Diff
View All 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 Florian Best

2016-10-13 17:44:27 CEST

Created attachment 8099 [details]
patch

There are cases in which the deferred never resolves!

Affects the _searchForm.ready() calls in all opened UDM DHCP/DNS modules.

Patch attached: In these cases the ready call is immediately resolved. Should be fine, I guess.

Comment 1 Florian Best

2016-10-13 17:48:33 CEST

Hmm, not really fine. @Alex: Do you have an idea how to improve the patch?

This affects the widgets objectType and objectProperty of UDM. With the patch the intialvalue isn't set and the initial query is done too early so that a error message is shown:
curl 'http://10.200.27.117/univention-management-console/command/udm/query' -H 'Content-Type: application/json' -H 'X-Requested-With: XMLHttpRequest' http://10.200.27.117/univention-management-console/?lang=de' --data-binary '{"options":{"objectType":"","objectProperty":"","objectPropertyValue":"","hidden":false,"fields":["name","","path"]},"flavor":"dns/dns"}'

I will try to implement some workaround in UDM.

Comment 2 Florian Best

2016-10-13 18:43:11 CEST

Please revert the 3 blocks in svn r73180 after this is fixed.

Comment 3 Florian Best

2016-10-18 18:58:16 CEST

I found the error. HiddenInput doesn't support dependencies. I added some other workarounds in UDM so that ready() works again. svn r73349.