Bug 45836 – Login dialog not prefilled on demo system / hooks not loaded

Bug 45836 - Login dialog not prefilled on demo system / hooks not loaded


Summary:	Login dialog not prefilled on demo system / hooks not loaded

Status:	CLOSED FIXED

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

Importance:	P5 normal (vote)
Target Milestone:	UCS 4.2-3-errata
Assigned To:	Florian Best
QA Contact:	Erik Damrose

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2017-12-07 17:20 CET by Erik Damrose
Modified:	2017-12-08 10:44 CET (History)
CC List:	2 users (show)

See Also:
What kind of report is it?:	Bug Report
What type of bug is this?:	5: Major Usability: Impairs usability in key scenarios
Who will be affected by this bug?:	5: Will affect all installed domains
How will those affected feel about the bug?:	3: A User would likely not purchase the product
User Pain:	0.429
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 Erik Damrose

2017-12-07 17:20:18 CET

univention-demo-configuration has a hook that prefills the login dialog with username and password. Currently, the hook is not called. This is a usability issue because testers do not know the login information.

Comment 1 Erik Damrose

2017-12-07 17:44:58 CET

It is a general UCS problem: On the Portal page and in the login window the side menu has no entries

Comment 2 Johannes Keiser

2017-12-07 20:04:49 CET

univention-web (1.0.42-63):

cb2b191 Bug #45836: fix import error
f12c067 Bug #45836: Add debian changelog
185c906 Bug #45836: Merge branch 'jkeiser/bug_45836__failed__import' into 4.2-3
02e86c9 Bug #45836: Added YAML

Comment 3 Florian Best

2017-12-08 06:57:52 CET

The problem was introduced by Bug #45809. The dynamic loading of the javascript module "umc/app" only works inside of UMC but not on portal/etc because the umc/app alias is not defined there.

(In reply to Johannes Keiser from comment #2)
> univention-web (1.0.42-63):
> 
> cb2b191 Bug #45836: fix import error
This fix introduced a typo, which I fixed:

univention-web (1.0.42-64)
99a9387503a7 | Bug #45836: fix typo

I build the package and adjusted the YAML file.

Comment 4 Erik Damrose

2017-12-08 08:55:31 CET

Impact of the bug:
* Side menu empty on portal, so no language change, certificate download, ...
* Side effect of hooks not loading: App appliances will not show the first steps

The change looks good, the menu works. I tested the package together with univention-demo-configuration, and username/password are prefilled again.

Reopen: I think two typos were introduced, and only one was fixed. Please also fix app.getModule('stop'). It should read 'top'

Comment 5 Florian Best

2017-12-08 10:14:22 CET

Thanks, done

Comment 6 Erik Damrose

2017-12-08 10:25:17 CET

OK: menu on the portal works again
OK: Default login is prefilled with the demo-configuration package installed
OK~: I fixed the yaml
Verified

Comment 7 Arvid Requate

2017-12-08 10:44:54 CET

<http://errata.software-univention.de/ucs/4.2/246.html>