Bug 42401 - App Center sets ucr variables (with no default) from apps ucr file to empty string
Summary: App Center sets ucr variables (with no default) from apps ucr file to empty s...
Status: CLOSED FIXED
Alias: None
Product: UCS
Classification: Unclassified
Component: App Center
Version: UCS 4.1
Hardware: Other Linux
: P5 normal
Target Milestone: UCS 4.1-3-errata
Assignee: Dirk Wiesenthal
QA Contact: Felix Botner
URL:
Keywords:
Depends on:
Blocks:
 
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):
Customer ID:
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 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>