Bug 42068

Summary: univention-app-appliance sets notifyvender info wrong
Product: UCS Reporter: Erik Damrose <damrose>
Component: GeneralAssignee: Erik Damrose <damrose>
Status: CLOSED FIXED QA Contact: Alexander Kläser <klaeser>
Severity: normal    
Priority: P5 CC: gohmann
Version: UCS 4.1   
Target Milestone: UCS 4.1-3-errata   
Hardware: Other   
OS: Linux   
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?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.034 Enterprise Customer affected?:
School Customer affected?: ISV affected?: Yes
Waiting Support: Flags outvoted (downgraded) after PO Review:
Ticket number: Bug group (optional):
Max CVSS v3 score:

Description Erik Damrose univentionstaff 2016-08-19 16:04:42 CEST
univention-app-appliance sets the UCRv appliance/id/<appid>/notifyVendor wrong, because it tries to parse the ini value ApplianceNotifyVendor, when the correct ini value is NotifyVendor.

This causes the activation page to never send an installation notification, as only apps with NotifyVendor=True are reported
Comment 1 Erik Damrose univentionstaff 2016-08-19 16:13:58 CEST
Updated the value that is read from the INI to NotifyVendor

r71757 univention-app-appliance 0.2.2-36
r71758 merge to 4.2
r71759 yaml
Comment 2 Alexander Kläser univentionstaff 2016-08-23 17:10:47 CEST
Changes: OK, previous version always returned None for notifyVendor, after patch 
    this works fine now.
Merge to 4.2: OK
YAML file: OK, I renamed to package from univention-app-activation to 
    univention-app-appliance ;) [r71851]
Comment 3 Alexander Kläser univentionstaff 2016-08-23 17:34:46 CEST
BTW, is a release for 4.1-2 and 4.1-3 correct? If not, please adjust the YAML file.
Comment 4 Janek Walkenhorst univentionstaff 2016-08-24 12:06:46 CEST
<http://errata.software-univention.de/ucs/4.1/239.html>