Bug 42165 – univention-app-appliance: No welcome screen theme if INI contains no theme info

Bug 42165 - univention-app-appliance: No welcome screen theme if INI contains no theme info


Summary:	univention-app-appliance: No welcome screen theme if INI contains no theme info

Status:	CLOSED FIXED

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

Importance:	P5 normal (vote)
Target Milestone:	UCS 4.1-3-errata
Assigned To:	Erik Damrose
QA Contact:	Alexander Kläser

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2016-08-29 10:51 CEST by Erik Damrose
Modified:	2016-09-21 21:04 CEST (History)
CC List:	1 user (show)

See Also:
What kind of report is it?:	Bug Report
What type of bug is this?:	3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?:	2: Will only affect a few installed domains
How will those affected feel about the bug?:	2: A Pain – users won’t like this once they notice it
User Pain:	0.069
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

2016-08-29 10:51:13 CEST

An App ini does not contain branding settings if no app branding has been defined. In that case, the default welcome screen should be shown.

univention-app-appliance does however dpkg-divert the default ucs.plymouth theme, but then does nothing to set a new theme if it is not defined in the app ini.

Comment 1 Erik Damrose

2016-08-29 11:17:26 CEST

The package diverted the default ucs.plymouth theme unnecessarily. If an app branding is to be done, another theme is created anyway (ucs-appliance-[light|dark]).

r71982 univention-app-appliance 0.2.2-37
r71984 yaml

Comment 2 Erik Damrose

2016-08-31 11:28:12 CEST

r72102 merge to ucs 4.2

Comment 3 Alexander Kläser

2016-08-31 12:43:48 CEST

Changes: OK, looks good in my tests; update logic not necessary
Merges to 4.2-0: OK
YAML file: OK, however, I changed the entry to be less technical [r72110]

Comment 4 Janek Walkenhorst

2016-09-07 18:42:01 CEST

<http://errata.software-univention.de/ucs/4.1/258.html>