Bug 43069 – property.nonempty_is_default is already boolean

Bug 43069 - property.nonempty_is_default is already boolean


Summary:	property.nonempty_is_default is already boolean

Status:	CLOSED FIXED

Product:	UCS
Classification:	Unclassified
Component:	UMC - Domain management (Generic)
Version:	UCS 4.1
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS 4.1-4-errata
Assigned To:	Florian Best
QA Contact:	Dirk Wiesenthal

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2016-11-28 18:30 CET by Florian Best
Modified:	2016-12-01 11:57 CET (History)
CC List:	0 users

See Also:	33329 42912
What kind of report is it?:	Development Internal
What type of bug is this?:	---
Who will be affected by this bug?:	---
How will those affected feel about the bug?:	---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):	API change
Max CVSS v3 score:

Flags:	best: Patch_Available+

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 Florian Best

2016-11-28 18:30:17 CET

management/univention-management-console-module-udm/umc/python/udm/udm_ldap.py
-»   »   »   »   'nonempty_is_default': ucr.is_true(value=str(prop.nonempty_is_default)),
+»   »   »   »   'nonempty_is_default': bool(prop.nonempty_is_default),

Comment 1 Florian Best

2016-11-28 18:39:26 CET

See Bug #42912 comment 11.

univention-management-console-module-udm (6.0.11-34):
r74783 | Bug #43069: property.nonempty_is_default is already boolean

univention-management-console-module-udm.yaml:
r74783 | Bug #43069: property.nonempty_is_default is already boolean

univention-management-console-module-udm (7.0.3-1):
r74784 | Bug #43069: property.nonempty_is_default is already boolean

Comment 2 Dirk Wiesenthal

2016-11-29 23:34:59 CET

Changeset: OK, works (e.g. mailHomeServer)
YAML: OK
4.2: OK

Strictly speaking, this is an API change. nonempty_is_default='no' will be evaluated as True now. I think it is unlikely that any 3rd party extension uses it this way.

Comment 3 Florian Best

2016-11-30 11:15:43 CET

(In reply to Dirk Wiesenthal from comment #2)
> Strictly speaking, this is an API change. nonempty_is_default='no' will be
> evaluated as True now. I think it is unlikely that any 3rd party extension
> uses it this way.
No, it's not an API change. Please have a look at svn r73803.

Comment 4 Janek Walkenhorst

2016-12-01 11:57:31 CET

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