Univention Bugzilla – Attachment 7662 Details for
Bug 38110
UDM does not correctly check object existence when modifying objects
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Requests
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
patch
38110.patch (text/plain), 1.32 KB, created by
Florian Best
on 2016-05-13 12:45:22 CEST
(
hide
)
Description:
patch
Filename:
MIME Type:
Creator:
Florian Best
Created:
2016-05-13 12:45:22 CEST
Size:
1.32 KB
patch
obsolete
>diff --git a/management/univention-directory-manager-modules/modules/univention/admin/handlers/__init__.py b/management/univention-directory-manager-modules/modules/univention/admin/handlers/__init__.py >index 9a2ed9e..8d52201 100644 >--- a/management/univention-directory-manager-modules/modules/univention/admin/handlers/__init__.py >+++ b/management/univention-directory-manager-modules/modules/univention/admin/handlers/__init__.py >@@ -553,7 +553,7 @@ def _not_implemented_error(self, *args, **kwargs): > > class simpleLdap(base): > >- def __init__(self, co, lo, position, dn='', superordinate=None, attributes = [] ): >+ def __init__(self, co, lo, position, dn='', superordinate=None, attributes=None): > global s4connector_present > global s4connector_search > >@@ -575,10 +575,14 @@ def __init__(self, co, lo, position, dn='', superordinate=None, attributes = [] > s4connector_present = False > self.s4connector_present = s4connector_present > >+ self.oldattr = {} > if attributes: > self.oldattr = attributes >- else: >- self.oldattr=self.lo.get(self.dn) >+ elif self.dn: >+ try: >+ self.oldattr = self.lo.get(self.dn, required=True) >+ except ldap.NO_SUCH_OBJECT: >+ raise univention.admin.uexceptions.noObject(self.dn) > if self.oldattr: > self._exists = True > oldinfo=univention.admin.mapping.mapDict(self.mapping, self.oldattr)
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
Attachments on
bug 38110
:
7662
|
7767