--- a/VirtualMailManager/VirtualMailManager.py Tue Jan 08 21:35:40 2008 +0000
+++ b/VirtualMailManager/VirtualMailManager.py Wed Jan 09 14:47:30 2008 +0000
@@ -42,8 +42,8 @@
Throws a VMMNotRootException if your uid is greater 0.
"""
self.__cfgFileName = '/usr/local/etc/vmm.cfg'
- self.__permWarnMsg = "fix permissions for '"+self.__cfgFileName \
- +"'.\n`chmod 0600 "+self.__cfgFileName+"` would be great.\n"
+ self.__permWarnMsg = "fix permissions for '%s'\n`chmod 0600 %s` would\
+ be great." % (self.__cfgFileName, self.__cfgFileName)
self.__warnings = []
self.__Cfg = None
self.__dbh = None
@@ -114,17 +114,17 @@
raise VMMException((str(e), ERR.DATABASE_ERROR))
def __chkLocalpart(self, localpart):
- """Validates the local part of an email address.
+ """Validates the local part of an e-mail address.
Keyword arguments:
- localpart -- the email address that should be validated (str)
+ localpart -- the e-mail address that should be validated (str)
"""
if len(localpart) > 64:
raise VMMException(('The local part is too long',
ERR.LOCALPART_TOO_LONG))
if re.compile(RE_LOCALPART).search(localpart):
raise VMMException((
- 'The local part «%s» contains invalid characters.' % localpart,
+ 'The local part »%s« contains invalid characters.' % localpart,
ERR.LOCALPART_INVALID))
return localpart
@@ -155,7 +155,7 @@
return '.'.join(tmp)
def __chkDomainname(self, domainname):
- """Validates the domain name of an email address.
+ """Validates the domain name of an e-mail address.
Keyword arguments:
domainname -- the domain name that should be validated
@@ -170,21 +170,21 @@
ERR.DOMAIN_INVALID))
return domainname
- def __chkEmailadress(self, address):
+ def __chkEmailAddress(self, address):
try:
localpart, domain = address.split('@')
except ValueError:
- raise VMMException(("Missing '@' sign in emailaddress «%s»." %
+ raise VMMException(("Missing '@' sign in e-mail address »%s«." %
address, ERR.INVALID_ADDRESS))
except AttributeError:
- raise VMMException(("'%s' looks not like an email address." %
+ raise VMMException(("»%s« looks not like an e-mail address." %
address, ERR.INVALID_ADDRESS))
domain = self.__chkDomainname(domain)
localpart = self.__chkLocalpart(localpart)
return '%s@%s' % (localpart, domain)
def __getAccount(self, address, password=None):
- address = self.__chkEmailadress(address)
+ address = self.__chkEmailAddress(address)
self.__dbConnect()
if not password is None:
password = self.__pwhash(password)
@@ -192,10 +192,10 @@
password)
def __getAlias(self, address, destination=None):
- address = self.__chkEmailadress(address)
+ address = self.__chkEmailAddress(address)
if not destination is None:
if destination.count('@'):
- destination = self.__chkEmailadress(destination)
+ destination = self.__chkEmailAddress(destination)
else:
destination = self.__chkLocalpart(destination)
self.__dbConnect()
@@ -339,7 +339,7 @@
if not section:
self.__Cfg.configure(self.__cfgSections)
elif section not in self.__cfgSections:
- raise VMMException(("Invalid section: «%s»" % section,
+ raise VMMException(("Invalid section: »%s«" % section,
ERR.INVALID_SECTION))
else:
self.__Cfg.configure([section])
@@ -357,7 +357,7 @@
def domain_delete(self, domainname, force=None):
if not force is None and force not in ['deluser','delalias','delall']:
- raise VMMDomainException(('Invalid option: «%s»' % force,
+ raise VMMDomainException(('Invalid argument: «%s»' % force,
ERR.INVALID_OPTION))
dom = self.__getDomain(domainname)
gid = dom.getID()
@@ -386,7 +386,7 @@
elif detailed == 'detailed':
return dominfo, dom.getAccounts(), dom.getAliases()
else:
- raise VMMDomainException(('Invalid option: «%s»' % detailed,
+ raise VMMDomainException(('Invalid argument: »%s«' % detailed,
ERR.INVALID_OPTION))
def user_add(self, emailaddress, password):