Univention Bugzilla – Attachment 3039 Details for
Bug 17915
Aktualisierung des UDM-Beispiel-Moduls
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Requests
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Weitere Änderungswünsche 3 (mit Whitespace)
17915_udm_example3.diff (text/plain), 15.90 KB, created by
Philipp Hahn
on 2011-02-16 08:00 CET
(
hide
)
Description:
Weitere Änderungswünsche 3 (mit Whitespace)
Filename:
MIME Type:
Creator:
Philipp Hahn
Created:
2011-02-16 08:00 CET
Size:
15.90 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,9 +36,13 @@ > > case "$1" in > remove) >+ 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,11 +28,13 @@ > # /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 > >@@ -45,7 +47,7 @@ > > ## 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, >@@ -60,7 +62,7 @@ > 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,9 +74,9 @@ > ## 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'] > >@@ -82,9 +84,9 @@ > ##### 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> ############################# >@@ -93,8 +95,8 @@ > # 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'], > ) > } >@@ -105,77 +107,77 @@ > 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,15 +188,18 @@ > ## * 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. >+## * 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.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("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.tab(_(u'Redirect'), _(u'Redirect Option'), [ > [ univention.admin.field( "redirect_user" ) ], > ], advanced = True ), > ] >@@ -203,13 +208,15 @@ > > ## 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. >@@ -243,7 +251,7 @@ > 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 >@@ -286,10 +294,12 @@ > 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. >@@ -306,34 +316,34 @@ > 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.""" >+ 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.""" >@@ -342,6 +352,8 @@ > 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) >@@ -376,7 +388,7 @@ > > > 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.""" > >@@ -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', []) >-
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
Actions:
View
|
Diff
Attachments on
bug 17915
:
3016
|
3038
| 3039 |
3040
|
3046
|
3050