Bug 42401 - App Center sets ucr variables (with no default) from apps ucr file to empty string
App Center sets ucr variables (with no default) from apps ucr file to empty s...
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: App Center
UCS 4.1
Other Linux
: P5 normal (vote)
: UCS 4.1-3-errata
Assigned To: Dirk Wiesenthal
Felix Botner
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2016-09-15 11:32 CEST by Felix Botner
Modified: 2016-09-22 07:16 CEST (History)
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?: 2: Will only affect a few installed domains
How will those affected feel about the bug?: 3: A User would likely not purchase the product
User Pain: 0.171
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

Note You need to log in before you can comment on or make changes to this bug.
Description Felix Botner univentionstaff 2016-09-15 11:32:34 CEST
see horde app

[horde/mailer/params/port]
Description[de]=SMTP Port
Description[en]=SMTP Port
Type=str
Categories=service-mail

during the app installation, before the actual packages are installed, the appcenter sets horde/mailer/params/port to ''

2016-09-15 10:58:52: set horde/mailer/params/port='' old:

now, the postinst of the horde integration package can't set its default (horde/mailer/params/port?25) and i end up with a broken config.

=> App Center should not set (if not set) or unset (if set) variables which have no value in the App Settings
Comment 1 Dirk Wiesenthal univentionstaff 2016-09-20 01:05:19 CEST
Fixed in
  univention-appcenter 5.0.22-17.223.201609200055

TextBox widgets (Type=str) with an empty value ('') instead emit None. This should remove the variable. Also fixed a bug in the Advanced attribute of a variable.
Comment 2 Felix Botner univentionstaff 2016-09-21 10:47:36 CEST
OK - variables without default are longer set to an empty string
OK - merged to 4.2-0
OK - YAML
Comment 3 Janek Walkenhorst univentionstaff 2016-09-21 14:30:28 CEST
<http://errata.software-univention.de/ucs/4.1/272.html>