Attachment 3040 Details for Bug 17915

[patch] Weitere Änderungswünsche 3 (ohne Whitespace)

17915_udm_example3w.diff (text/plain), 21.10 KB, created by Philipp Hahn on 2011-02-16 08:01 CET

(hide)

Description:

Filename:

MIME Type:

Creator: Philipp Hahn

Created: 2011-02-16 08:01 CET

Size: 21.10 KB

patch

obsolete

>Index: schema/ip-phone.schema
>===================================================================
>--- schema/ip-phone.schema	(Revision 22601)
>+++ schema/ip-phone.schema	(Arbeitskopie)
>@@ -1,4 +1,9 @@
> # univention registered the namespace 10176.
>+#objectIdentifier univention 1.3.6.1.4.1.10176
>+#objectIdentifier univentionCustomers univention:99999
>+#objectIdentifier univentionExampleAttributeType univentionCustomers:1
>+#objectIdentifier univentionExampleObjectClass univentionCustomers:2
>+
> attributetype ( 1.3.6.1.4.1.10176.9999.1.1 NAME 'testPhoneActive'
> 	DESC 'state of the IP phone'
> 	EQUALITY caseIgnoreIA5Match
>@@ -42,4 +47,3 @@
> 	MUST ( cn )
> 	MAY ( testPhoneRedirectUser )
> 	)
>-
>Index: debian/univention-directory-manager-module-example-schema.postrm
>===================================================================
>--- debian/univention-directory-manager-module-example-schema.postrm	(Revision 22601)
>+++ debian/univention-directory-manager-module-example-schema.postrm	(Arbeitskopie)
>@@ -36,8 +36,12 @@
> 
> case "$1" in
> remove)
>-	/usr/sbin/univention-config-registry commit /etc/ldap/slapd.conf
>-	invoke-rc.d slapd restart
>+	case "$(/usr/sbin/univention-config-registry get server/role)" in
>+	domaincontroller_master)
>+		/usr/sbin/univention-config-registry commit /etc/ldap/slapd.conf
>+		invoke-rc.d slapd restart
>+		;;
>+	esac
> 	;;
> 
> purge|upgrade|failed-upgrade|abort-install|abort-upgrade|disappear)
>Index: debian/changelog
>===================================================================
>--- debian/changelog	(Revision 22601)
>+++ debian/changelog	(Arbeitskopie)
>@@ -1,3 +1,14 @@
>+univention-directory-manager-module-example (2.0.5-1) unstable; urgency=low
>+
>+  * Prepare for translation (Bug #17915)
>+  * Restart slapd only on domaincontroller_master.
>+  * Fix unicode strings.
>+  * Add missing DocString to exists() and _remove_attr().
>+  * Document advanced=True.
>+  * Use True and False instead of 1 and 0.
>+
>+ -- Philipp Hahn <hahn@univention.de>  Wed, 16 Feb 2011 07:59:08 +0100
>+
> univention-directory-manager-module-example (2.0.4-1) unstable; urgency=low
> 
>   * suggestions from QA feedback (Bug #17915)
>Index: modules/ip_phone.py
>===================================================================
>--- modules/ip_phone.py	(Revision 22601)
>+++ modules/ip_phone.py	(Arbeitskopie)
>@@ -28,39 +28,41 @@
> # /usr/share/common-licenses/AGPL-3; if not, see
> # <http://www.gnu.org/licenses/>.
> 
>-import re, sys, string, copy
>+import re
>+import copy
> import univention.admin.filter		# Definiert Filterausdruck-Objekt fÃ¼r 'lookup' Funktion unten
> import univention.admin.handlers	# EnthÃ¤lt simpleLdap, die Basisklasse fÃ¼r 'object' unten
> import univention.admin.syntax		# Liefert standard Syntax-Definitionen fÃ¼r die UDM 'property_descriptions' unten
> 
>+# FÃ¼r das Einbinden von Ãbersetzungskatalogen fÃ¼r verschiedene Sprachen
> translation=univention.admin.localization.translation('univention.admin.handlers.test')
>-_=translation.translate
>+_ = translation.translate
> 
> ############################ <Syntax definitions> #############################
> 
> class SynVoIP_Protocols(univention.admin.syntax.select):
> 	"""Diese Klasse definiert einen neue Syntax fÃ¼r eine Auswahlliste von VoIP-Protokollen"""
> 	## der Text, der in dem Web-Frontend vor die Auswahlliste geschrieben werden soll
>-	name=u'VoIP_Protocol'
>+	name = u'VoIP_Protocol'
> 
> 	## die Liste der AuswahlmÃ¶glichkeiten: Jedes der Elemente enthÃ¤lt einen eindeutigen SchlÃ¼ssel
> 	## und den anzuzeigenden Text
>-	choices=[ ( 'sip', u'SIP'), ( 'h323', u'H.323' ), ('skype', u'Skype' ) ]
>+	choices = [('sip', _(u'SIP')), ('h323', _(u'H.323')), ('skype', _(u'Skype'))]
> 
> class SynVoIP_Address(univention.admin.syntax.simple):
> 	"""Diese Klasse dient als Syntax fÃ¼r VoIP Adresse. Der Aufbau ist einer E-Mail Adresse Ã¤hnlich,
> 	kann aber als PrÃ¤fix noch ein Schema gefolgt von einem ':' enthalten.
> 	Valide Schemta sind: sip, h323 und skype"""
> 
>-	name='VoIP_Address'
>-	min_length=4
>-	max_length=256
>+	name = 'VoIP_Address'
>+	min_length = 4
>+	max_length = 256
> 	_re = re.compile('((^(sip|h323|skype):)?([a-zA-Z])[a-zA-Z0-9._-]+)@[a-zA-Z0-9._-]+$')
> 
> 	def parse(self, text):
> 		if self._re.match(text) != None:
> 			return text
>-		raise univention.admin.uexceptions.valueError, u'Not a valid VoIP Address'
>+		raise univention.admin.uexceptions.valueError(_(u'Not a valid VoIP Address'))
> 
> 
> ############################ </Syntax definitions> ############################
>@@ -72,29 +74,29 @@
> ## dieses Objekt kann keine Unterobjekte enthalten
> childs = 0
> ## ein sprechender Name fÃ¼r das Web-Frontend
>-short_description = u'IP-Phone'
>+short_description = _(u'IP-Phone')
> ## eine ausfÃ¼hrliche Beschreibung
>-long_description = u'An example module for the Univention Directory Manager'
>+long_description = _(u'An example module for the Univention Directory Manager')
> ## die LDAP Operationen, die auf diesem Objekt ausgefÃ¼hrt werden kÃ¶nnen
>-operations=['add','edit','remove','search','move']
>+operations = ['add', 'edit', 'remove', 'search', 'move']
> 
> 
> ##### Um einen eigenen Wizard zu erstellen, der im UDM-Web links in der Navigationsleiste erscheint:
> ## usewizard = 1
> ## wizardmenustring = "VoIP"
>-## wizarddescription =  "Hinzufuegen, Loeschen und Suchen von VoIP Objekten"
>-## wizardoperations = { 'add' : [ "Hinzufuegen", "Fuegt ein VoIP Objekt hinzu" ],
>-##                    'find' : [ "Suchen", "Sucht VoIP Objekte" ] }
>+## wizarddescription =  _(u"Add, delete and search VoIP objects"
>+## wizardoperations = { 'add' : [_(u"add"), _(u"Add an VoIP object")],
>+##                    'find' : [_(u"Search"), _(u"Search VoIP objects"]) }
> ## wizardpath="univentionUsersObject"
> 
> ############################ <UDM module options> #############################
> ## Liste der Optionen fÃ¼r dieses Modul, bzw. fÃ¼r den behandelten Objekttyp
>-options={
>+options = {
> 	# durch 'options' werden optionale Eigenschaften eines Objekts definiert
> 	'redirection': univention.admin.option(
> 			short_description=_('Call redirect option'),
>-			default=1,
>-			editable=1,
>+			default=True,
>+			editable=True,
> 			objectClasses = ['testPhoneCallRedirect'],
> 		)
> }
>@@ -102,80 +104,80 @@
> ########################### <UDM module properties> ###########################
> 
> ## Liste der Eigenschaften fÃ¼r dieses Modul
>-property_descriptions={
>+property_descriptions = {
> 	# der eindeutige Name eines IP-Telefons
> 	'name': univention.admin.property(
>-			short_description= u'Name',
>-			long_description= u'ID of the IP-phone',
>+			short_description=_(u'Name'),
>+			long_description=_(u'ID of the IP-phone'),
> 			syntax=univention.admin.syntax.hostName,	# Eigenschaft muss der Syntax eines Rechnernamens entsprechen, Def. in syntax.py
>-			multivalue=0,
>+			multivalue=False,
> 			options=[],
>-			required=1,								# Eigenschaft muss angegeben werden
>-			may_change=0,								# Eigenschaft darf nach Erstellung nicht verÃ¤ndert werden
>-			identifies=1								# Eigenschaft muss eindeutig sein
>+			required=True,								# Eigenschaft muss angegeben werden
>+			may_change=False,							# Eigenschaft darf nach Erstellung nicht verÃ¤ndert werden
>+			identifies=True								# Eigenschaft muss eindeutig sein
> 		),
> 	#
> 	'active': univention.admin.property(
>-			short_description= u'active',
>-			long_description= u'The IP-phone can be deactivated',
>+			short_description=_(u'active'),
>+			long_description=_(u'The IP-phone can be deactivated'),
> 			syntax=univention.admin.syntax.boolean,		# kann nur die Werte '1' oder '0' annehmen, Definition in syntax.py
>-			multivalue=0,
>+			multivalue=False,
> 			options=[],
>-			required=0,								# Eigenschaft muss nicht zwingend angegeben werden
>+			required=False,								# Eigenschaft muss nicht zwingend angegeben werden
> 			default='1',								# Eigenschaft ist standardmÃ¤Ãig aktiviert
>-			may_change=1,								# Eigenschaft darf modifiziert werden
>-			identifies=0
>+			may_change=True,							# Eigenschaft darf modifiziert werden
>+			identifies=False
> 		),
> 	'protocol': univention.admin.property(
>-			short_description= u'Protocol',
>-			long_description= u'Supported VoIP protocols',
>+			short_description=_(u'Protocol'),
>+			long_description=_(u'Supported VoIP protocols'),
> 			syntax=SynVoIP_Protocols,					# nutzt die selbst definierte Auswahlliste als Syntax
>-			multivalue=0,
>+			multivalue=False,
> 			options=[],
>-			required=0,
>+			required=False,
> 			default='sip',							# der Eintrag 'sip' ist vorausgewÃ¤hlt
>-			may_change=1,
>-			identifies=0
>+			may_change=True,
>+			identifies=False
> 		),
> 	'ip': univention.admin.property(
>-			short_description = u'IP-Address',
>-			long_description = u'IP-Address of the IP-phone',
>+			short_description=_(u'IP-Address'),
>+			long_description=_(u'IP-Address of the IP-phone'),
> 			syntax=univention.admin.syntax.ipAddress,	# muss der Syntax einer IP (Version 4) Adresse entsprechen
>-			multivalue=0,
>+			multivalue=False,
> 			options=[],
>-			required=1,
>-			may_change=1,
>-			identifies=0
>+			required=True,
>+			may_change=True,
>+			identifies=False
> 		),
> 	'priuser': univention.admin.property(
>-			short_description = u'Primary User',
>-			long_description = u'The primary user of this IP-phone',
>+			short_description=_(u'Primary User'),
>+			long_description=_(u'The primary user of this IP-phone'),
> 			syntax=SynVoIP_Address,						# muss der Syntax einer VoIP Adresse entsprechen
>-			multivalue=0,
>+			multivalue=False,
> 			options=[],
>-			required=1,
>-			may_change=1,
>-			identifies=0
>+			required=True,
>+			may_change=True,
>+			identifies=False
> 		),
> 	'users': univention.admin.property(
>-			short_description = u'Additional Users',
>-			long_description = u'Users, that may register with this phone',
>+			short_description=_(u'Additional Users'),
>+			long_description=_(u'Users, that may register with this phone'),
> 			syntax=SynVoIP_Address,						# jeder Eintrag muss der Syntax einer VoIP Adresse entsprechen
>-			multivalue=1,							# Dies ist eine Liste von Adressen
>+			multivalue=True,						# Dies ist eine Liste von Adressen
> 			options=[],
>-			required=0,
>-			may_change=1,
>-			identifies=0
>+			required=False,
>+			may_change=True,
>+			identifies=False
> 		),
> 	'redirect_user': univention.admin.property(
>-			short_description = u'Redirection User',
>-			long_description = u'Address for call redirection',
>+			short_description=_(u'Redirection User'),
>+			long_description=_(u'Address for call redirection'),
> 			syntax=SynVoIP_Address,
>-			multivalue=0,
>+			multivalue=False,
> 			options=['redirection'],					# Ist nur da, wenn die Option gesetzt ist
>-			required=0,
>-			may_change=1,
>-			identifies=0
>+			required=False,
>+			may_change=True,
>+			identifies=False
> 		),
> }
> 
>@@ -186,30 +188,35 @@
> ##   * Der erste Parameter ist der Name des Reiters und der zweite Parameter
> ##     ist eine Beschreibung der EinstellungsmÃ¶glich fÃ¼r diesen Reiter
> ##   * Die folgende Liste definiert die Anordnung der einzelnen Eigenschaftsfelder.
>-layout=[
>-	univention.admin.tab( u'Gerneral', u'Basic Settings',
>-			[ [ univention.admin.field( "name" ), univention.admin.field( "active" ) ],
>-			[ univention.admin.field( "ip" ), univention.admin.field( "protocol" ) ],
>-			[ univention.admin.field( "priuser" ) ] ] ),
>-	univention.admin.tab( u'Advanced', u'Advanced Settings', [
>-				[ univention.admin.field( "users" ) ],
>-			], advanced = True ),
>-	univention.admin.tab( u'Redirect', u'Redirect Option', [
>-				[ univention.admin.field( "redirect_user" ) ],
>-			], advanced = True ),
>+##   * Per advanced=True wird der Reiter nur angezeigt, wenn das Anzeigen der
>+##     erweiterten Einstellungen aktiviert ist.
>+layout = [
>+	univention.admin.tab((u'Gerneral'), _(u'Basic Settings'), [
>+			[ univention.admin.field("name"), univention.admin.field("active") ],
>+			[ univention.admin.field("ip"), univention.admin.field("protocol") ],
>+			[ univention.admin.field("priuser") ],
>+			]),
>+	univention.admin.tab(_(u'Advanced'), _(u'Advanced Settings'), [
>+				[ univention.admin.field("users") ],
>+			], advanced=True),
>+	univention.admin.tab(_(u'Redirect'), _(u'Redirect Option'), [
>+				[ univention.admin.field("redirect_user") ],
>+			], advanced=True),
> 	]
> 
> ################ <Mapping of UDM properties to LDAP attribute> ################
> 
> ## Die folgenden beiden Hilfsfunktionen dienen zur Abbildung von bool'schen Werten '0' und '1' auf 'no' und 'yes' (siehe Mapping)
> def boolToString(value):
>+	u"""Wandelt den Wert des LDAP-Attributs in den Wert fÃ¼r die Anzeige der UDM Property."""
> 	if value == '1':
>-		return 'yes'
>+		return _('yes')
> 	else:
>-		return 'no'
>+		return _('no')
> 
> def stringToBool(value):
>-	if value[0].lower() == 'yes':
>+	u"""Wandelt den Wert der UDM Property zurÃ¼ck in den Wert des LDAP-Attributs."""
>+	if value[0].lower() == _('yes'):
> 		return '1'
> 	else:
> 		return '0'
>@@ -230,8 +237,9 @@
> mapping.register('users', 'testPhoneUsers')
> mapping.register('redirect_user', 'testPhoneRedirectUser', None, univention.admin.mapping.ListToString)
> 
>+
> class object(univention.admin.handlers.simpleLdap):
>-	"""Dieses Objekt unterstÃ¼tzt den Univention Directory Manager bei LDAP-Operationen,
>+	u"""Dieses Objekt unterstÃ¼tzt den Univention Directory Manager bei LDAP-Operationen,
> 	die sich auf dieses Modul beziehen.
> 	Die Basisklasse univention.admin.handlers.simpleLdap implementiert die komplette Kommunikation Ã¼ber LDAP,
> 	so dass hier nur die Anpassungen fÃ¼r dieses spezielle LDAP-Objekt implementiert werden mÃ¼ssen.
>@@ -240,22 +248,22 @@
> 	um einen Ãberblick der MÃ¶glichkeiten zu geben."""
> 
> 	## den Modulnamen als Attribute der Klasse Ã¼bernehmen (oben definiert als module='test/ip-phone')
>-	module=module
>+	module = module
> 
> 	def __init__(self, co, lo, position, dn='', superordinate=None, arg=None):
>-		"""Initialisierung des Objektes. Hier mÃ¼ssen die oben definierten globalen Variablen 'mapping'
>+		u"""Initialisierung des Objektes. Hier mÃ¼ssen die oben definierten globalen Variablen 'mapping'
> 		und 'property_descriptions' Ã¼bernommen werden"""
> 		global options
> 		global mapping
> 		global property_descriptions
> 
>-		self.co=co
>-		self.lo=lo
>-		self.dn=dn
>-		self.position=position
>-		self._exists=0
>-		self.mapping=mapping
>-		self.descriptions=property_descriptions
>+		self.co = co
>+		self.lo = lo
>+		self.dn = dn
>+		self.position = position
>+		self._exists = 0
>+		self.mapping = mapping
>+		self.descriptions = property_descriptions
> 
> 		## Initialisierungsfunktion der Basisobjektklasse simpleLdap in univention.admin.handlers
> 		## * liest die Objekt Attribute fÃ¼r den 'dn' aus dem LDAP ud stellt sie als self.oldattr bereit.
>@@ -273,7 +281,7 @@
> 		## 'options' lÃ¤sst sich ermitteln, welche Optionen an dem Objekt aktiviert sind. Die so ermittelte
> 		## Auswahl sollte als # self.options an diesem Objekt notiert werden, damit darauf basierend
> 		## weitere Entscheidungen getroffen werden kÃ¶nnen.
>-		self.options=[]
>+		self.options = []
> 		if self.oldattr.has_key('objectClass'):
> 			ocs = set(self.oldattr['objectClass'])
> 			for opt in ('redirection', ):
>@@ -281,15 +289,17 @@
> 					self.options.append(opt)
> 		else:
> 			univention.debug.debug(univention.debug.ADMIN, univention.debug.INFO, '%s: reset options to default by _define_options' % module)
>-			self._define_options( options )
>+			self._define_options(options)
> 
>-		self.old_options= copy.deepcopy( self.options )
>+		self.old_options= copy.deepcopy(self.options)
> 
> 	def exists(self):
>+		u"""Von SimpleLdap intern verwendete Methode, um zu entscheiden, ob ein
>+		Objekt neu angelegt werden muÃ oder ein vorhandenes editiert wird."""
> 		return self._exists
> 
> 	def open(self):
>-		"""Ãffnen des LDAP-Objekts."""
>+		u"""Ãffnen des LDAP-Objekts."""
> 
> 		univention.admin.handlers.simpleLdap.open(self)
> 		## In dieser Methode kÃ¶nnen die Eigenschaften des Objekts in self.info dynamisch Vor-initialisiert werden. 
>@@ -303,45 +313,47 @@
> 		## in self.oldinfo und self.oldpolicies gespeichert. Diese dienen spÃ¤ter zum Vergleich mit dem
> 		## aktualisierten Eigenschaften in self.info.
> 		self.save()
>-		self.old_options= copy.deepcopy( self.options ) # Optionen zum spÃ¤teren Vergleich speichern.
>+		self.old_options = copy.deepcopy(self.options) # Optionen zum spÃ¤teren Vergleich speichern.
> 
> 	def _ldap_pre_create(self):
>-		"""Wird vor dem Anlegen des LDAP Objektes aufgerufen."""
>-		self.dn='%s=%s,%s' % (mapping.mapName('name'), mapping.mapValue('name', self.info['name']), self.position.getDn())
>+		u"""Wird vor dem Anlegen des LDAP Objektes aufgerufen."""
>+		self.dn = '%s=%s,%s' % (mapping.mapName('name'), mapping.mapValue('name', self.info['name']), self.position.getDn())
> 
> 	def _ldap_post_create(self):
>-		"""Wird nach dem Anlegen des Objektes aufgerufen."""
>+		u"""Wird nach dem Anlegen des Objektes aufgerufen."""
> 		pass
> 
> 	def _ldap_pre_modify(self):
>-		"""Wird vor dem Modifizieren des Objektes aufgerufen."""
>+		u"""Wird vor dem Modifizieren des Objektes aufgerufen."""
> 		pass
> 
> 	def _ldap_post_modify(self):
>-		"""Wird nach dem Modifizieren des Objektes aufgerufen."""
>+		u"""Wird nach dem Modifizieren des Objektes aufgerufen."""
> 		pass
> 
> 	def _ldap_pre_remove(self):
>-		"""Wird vor dem LÃ¶schen des Objektes aufgerufen."""
>+		u"""Wird vor dem LÃ¶schen des Objektes aufgerufen."""
> 		pass
> 
> 	def _ldap_post_remove(self):
>-		"""Wird nach dem LÃ¶schen des Objektes aufgerufen."""
>+		u"""Wird nach dem LÃ¶schen des Objektes aufgerufen."""
> 		pass
> 
> 	def _update_policies(self):
> 		pass
> 
> 	def _ldap_addlist(self):
>-		"""Diese Funktion muss definiert werden, weil sie von 'create' verwendet wird.
>+		u"""Diese Funktion muss definiert werden, weil sie von 'create' verwendet wird.
> 		Sie sollte die nur zum Anlegen notwendigen LDAP-Attribute zurÃ¼ckgeben, d.h. mindestens die
> 		'objectClass' Definition. Nach dieser Methode ruft 'create' _ldap_modlist auf, um weitere
> 		Modifikationen an Eigenschaften festzustellen."""
> 
>-		al = [ ('objectClass', ['top', 'testPhone' ] ) ]
>+		al = [('objectClass', ['top', 'testPhone' ])]
> 		return al
> 
> 	def _remove_attr(self, ml, attr):
>+		u"""Hilfmethode zum Entfernen das Attribut 'attr' aus der Liste der zu
>+		modifizierenden Attribute 'ml'."""
> 		for m in ml:
> 			if m[0] == attr:
> 				ml.remove(m)
>@@ -350,7 +362,7 @@
> 		return ml
> 
> 	def _ldap_modlist(self):
>-		"""Diese Funktion kann definiert werden. Die gleichnamige ererbte Methode von 'simpleLdap'
>+		u"""Diese Funktion kann definiert werden. Die gleichnamige ererbte Methode von 'simpleLdap'
> 		erstellt eine LDAP-modlist aus der Differenz zwischen self.oldinfo und self.info."""
> 
> 		ml = univention.admin.handlers.simpleLdap._ldap_modlist(self)
>@@ -361,39 +373,39 @@
> 			univention.debug.debug(univention.debug.ADMIN, univention.debug.INFO, 'old_options: %s' % self.old_options)
> 			if 'redirection' in self.options and not 'redirection' in self.old_options:
> 				univention.debug.debug(univention.debug.ADMIN, univention.debug.INFO, 'added redirection option')
>-				ocs=self.oldattr.get('objectClass', [])
>+				ocs = self.oldattr.get('objectClass', [])
> 				if not 'testPhoneCallRedirect' in ocs:
> 					ml.insert(0, ('objectClass', '', 'testPhoneCallRedirect'))
> 			if not 'redirection' in self.options and 'redirection' in self.old_options:
> 				univention.debug.debug(univention.debug.ADMIN, univention.debug.INFO, 'remove redirection option')
>-				ocs=self.oldattr.get('objectClass', [])
>+				ocs = self.oldattr.get('objectClass', [])
> 				if 'testPhoneCallRedirect' in ocs:
> 					ml.insert(0, ('objectClass', 'testPhoneCallRedirect', ''))
> 
> 				for key in [ 'testPhoneRedirectUser', ]:
>-					ml=self._remove_attr(ml,key)
>+					ml = self._remove_attr(ml, key)
> 		return ml
>  
> 
> def lookup(co, lo, filter_s, base='', superordinate=None, scope='sub', unique=0, required=0, timeout=-1, sizelimit=0):
>-	"""Diese Function sucht nach Objekten, die dem in diesem Modul verwalteten Typ (objectClass)
>+	u"""Diese Function sucht nach Objekten, die dem in diesem Modul verwalteten Typ (objectClass)
> 	die den angegebenen Suchkriterien entsprechen. Der RÃ¼ckgabewert dieser Funktion ist ein Liste
> 	der gefunden Objekte."""
> 
>-	filter=univention.admin.filter.conjunction('&', [
>+	filter = univention.admin.filter.conjunction('&', [
> 				univention.admin.filter.expression('objectClass', 'testPhone'),
> 				])
> 
> 	if filter_s:
> 		## Vom Benutzer Ã¼bergebene Zeichenkette in ein Filterausdruck-Objekt Ã¼bersetzten:
>-		filter_p=univention.admin.filter.parse(filter_s)
>+		filter_p = univention.admin.filter.parse(filter_s)
> 		## Ãbersetzung der UDM Objekt-Variablennamen ('property_descriptions') und -Werte im Filterausdruck
> 		## auf LDAP-Attributnamen und -Werte, wie durch 'mapping' oben definiert:
> 		univention.admin.filter.walk(filter_p, univention.admin.mapping.mapRewrite, arg=mapping)
> 		## Oben definierten Objektklassenfilter ergÃ¤nzen um den vom Benutzer Ã¼bergebenen Filterausdruck:
> 		filter.expressions.append(filter_p)
> 
>-	res=[]
>+	res = []
> 	## LDAP-Suche Ã¶ber das LDAP-Connection-Objekt 'lo' unter Verwendung des unicode-Encodings,
> 	## das Python intern verwendet:
> 	for dn in lo.searchDn(unicode(filter), base, scope, unique, required, timeout, sizelimit):
>@@ -402,9 +414,8 @@
> 	return res
> 
> 
>-def identify(dn, attr, canonical=0):
>-	"""PrÃ¼ft ob die verwaltete Objektklasse diese Moduls in der Ã¼bergebenen Liste enthalten ist,
>-	d.h. ob dieses Modul fÃ¼r die Handhabung des Objekts zustÃ¤ndig ist"""
>+def identify(dn, attr, canonical=False):
>+	u"""PrÃ¼ft ob die verwaltete Objektklasse diese Moduls in der Ã¼bergebenen Liste enthalten ist,
>+	d.h. ob dieses Modul fÃ¼r die Handhabung des Objekts zustÃ¤ndig ist."""
> 
> 	return 'testPhone' in attr.get('objectClass', [])
>-

Actions: View | Diff

Attachments on bug 17915: 3016 | 3038 | 3039 | 3040 | 3046 | 3050