]> git.phdru.name Git - m_lib.git/commitdiff
Raise Error(message) for Py3 compatibility
authorOleg Broytman <phd@phdru.name>
Mon, 25 Jul 2016 16:05:26 +0000 (19:05 +0300)
committerOleg Broytman <phd@phdru.name>
Mon, 25 Jul 2016 16:23:18 +0000 (19:23 +0300)
m_lib/flad/flad.py
m_lib/flad/fladc.py
m_lib/flad/fladm.py
m_lib/flad/fladw.py
m_lib/hash/MKhash.py
m_lib/m_shutil.py
m_lib/net/ftp/ftpscan.py
m_lib/net/sms.py
m_lib/opdate.py

index b2e91c87e96d9f7006cc0f6cb0669d3daf75bede..cfa6ff65deac1bca7aad5bc5e79c01fce7e4c4cd 100644 (file)
@@ -40,7 +40,7 @@ class Flad(UserList):
          if callable(self.check_record_func):
             return self.check_record_func(self, record)
          else:
-            raise TypeError, "non-callable restriction function"
+            raise TypeError("non-callable restriction function")
       else:
          return 1
 
@@ -105,7 +105,7 @@ class Flad(UserList):
                                   # so it is not ready to be checked :(
                                   # And, of course, two keys with the same name
                                   # cannot be added to dictionary
-            raise KeyError, "field key \"" + key + "\" already in record"
+            raise KeyError("field key \"" + key + "\" already in record")
 
          record[key] = value
 
index 6fd2f607cf19ebbe619b57764640887bee135406..d30aa445dca8b460b1c8b8a61fa2f29cd83fb647 100644 (file)
@@ -35,7 +35,7 @@ class Flad_Conf(dict):
       db.load_file(f)
 
       if len(db) != 1:
-         raise error, "incorrect number of records in config file `%s'; expected 1, got %d" % (str(f), len(db))
+         raise error("incorrect number of records in config file `%s'; expected 1, got %d" % (str(f), len(db)))
 
       self.data = db[0]
 
@@ -45,7 +45,7 @@ class Flad_Conf(dict):
       db.load_from_file(f)
 
       if len(db) != 1:
-         raise error, "incorrect number of records in config file `%s'; expected 1, got %d" % (str(f), len(db))
+         raise error("incorrect number of records in config file `%s'; expected 1, got %d" % (str(f), len(db)))
 
       self.data = db[0]
 
index c0e2886061121e064af58eea1d9fa47d7a001f90..3705a304e9cb20a3cd0626313c1597edb9859d6e 100644 (file)
@@ -71,11 +71,11 @@ def check_record(data, record):
       if must_keys and (key in must_keys):
          del copy_must[copy_must.index(key)] # Remove the key from copied list
       elif (must_keys and (key not in must_keys) and (other_keys and (key not in other_keys))) or (other_keys and (key not in other_keys)):
-         raise KeyError, "field key \"" + key + "\" is not in list of allowed keys"
+         raise KeyError("field key \"" + key + "\" is not in list of allowed keys")
 
    if copy_must: # If there is at least one key - it is an error:
                       # not all "must" keys are in record
-      raise KeyError, "not all \"must\" keys are in record; keys: " + str(copy_must)
+      raise KeyError("not all \"must\" keys are in record; keys: " + str(copy_must))
 
    return 1
 
index 0acc34de0c7530697841b442dd08e6d37d96ee12..781c472b0437ce4cfc3f0f2358fcc2941d677956 100644 (file)
@@ -34,7 +34,7 @@ class Flad_WIni(flad.Flad):
 
       if self.first_section:
          if string.strip(line) != '':
-            raise error, "non-empty line before 1st section"
+            raise error("non-empty line before 1st section")
 
       elif (string.strip(line) == '') or (string.lstrip(line)[0] == ';') : # Empty line or comment
          record[0].append(line)
@@ -46,7 +46,7 @@ class Flad_WIni(flad.Flad):
                                   # so it is not ready to be checked :(
                                   # And, of course, two keys with the same name
                                   # cannot be added to dictionary
-            raise KeyError, "field key \"" + key + "\" already in record"
+            raise KeyError("field key \"" + key + "\" already in record")
 
          record[0].append(key)
          record[1][key] = value
@@ -73,7 +73,7 @@ class Flad_WIni(flad.Flad):
 
          else:
             if self.first_section and (string.strip(line) != ''):
-               raise error, "non-empty line before 1st section"
+               raise error("non-empty line before 1st section")
             # else: line had been appended to section in __parse_line()
 
       else: # This called after last line of the source file
@@ -131,7 +131,7 @@ class Flad_WIni(flad.Flad):
    def add_section(self, section):
       rec_no = self.find_section(section)
       if rec_no >= 0:
-         raise section_error, "section [%s] already exists" % section
+         raise section_error("section [%s] already exists" % section)
 
       self.append((section, [], {}))
 
@@ -139,7 +139,7 @@ class Flad_WIni(flad.Flad):
    def del_section(self, section):
       rec_no = self.find_section(section)
       if rec_no < 0:
-         raise section_error, "section [%s] does not exists" % section
+         raise section_error("section [%s] does not exists" % section)
 
       del self[rec_no]
 
@@ -160,11 +160,11 @@ class Flad_WIni(flad.Flad):
    def get_keyvalue(self, section, key):
       rec_no = self.find_section(section)
       if rec_no < 0:
-         raise section_error, "section [%s] does not exists" % section
+         raise section_error("section [%s] does not exists" % section)
 
       record = self[rec_no]
       if key not in record[1]:
-         raise KeyError, "section [%s] does not has `%s' key" % (section, key)
+         raise KeyError("section [%s] does not has `%s' key" % (section, key))
 
       return record[2][key]
 
@@ -184,11 +184,11 @@ class Flad_WIni(flad.Flad):
    def del_key(self, section, key):
       rec_no = self.find_section(section)
       if rec_no < 0:
-         raise section_error, "section [%s] does not exists" % section
+         raise section_error("section [%s] does not exists" % section)
 
       record = self[rec_no]
       if key not in record[1]:
-         raise KeyError, "section [%s] does not has `%s' key" % (section, key)
+         raise KeyError("section [%s] does not has `%s' key" % (section, key))
 
       klist = record[1]
       del klist[klist.index(key)]
index 812a14a9eba5661389cac80a741fd404ac88d0c1..d11a701e316005d291e0cc9dc25eef454238a0cd 100644 (file)
@@ -57,7 +57,7 @@ class MKhash:
         vw = self.vw
         ix = vw.find(key=key)
         if ix == -1:
-            raise KeyError, key
+            raise KeyError(key)
         return vw[ix].value
 
     def __setitem__(self, key, value):
@@ -73,7 +73,7 @@ class MKhash:
         vw = self.vw
         ix = vw.find(key=key)
         if ix == -1:
-            raise KeyError, key
+            raise KeyError(key)
         vw.delete(ix)
         self._add_tran()
 
index 7546b0d3c17cf61cae3e185ce45743c1366b6e2c..f1034e0f65ec90f9a5f1e543276c71e380aa21d0 100755 (executable)
@@ -15,7 +15,7 @@ def mkhier(path): # Python implementation of UNIX' mkdir -p /path/to/dir
       return # It's Ok to have the directory already created
 
    if os.path.exists(path):
-      raise mkhier_error, "`%s' is file" % path
+      raise mkhier_error("`%s' is file" % path)
 
    list_dirs = string.split(path, os.sep)
    #print(list_dirs)
index 469e44be58882796908c8546927cb156da6c1596..c3a4b5e23a19fcdc0138187fe381d91c87486acb 100755 (executable)
@@ -165,7 +165,7 @@ def ftpscanrecursive(ftp_server, ftp_port=None, login=None, password=None,
       # The server does not implement LIST -R and
       # treats -R as a name of a directory (-:
       ftp.quit()
-      raise FtpScanError, "the server does not implement recursive listing"
+      raise FtpScanError("the server does not implement recursive listing")
    ftp.quit()
 
    tree = []
@@ -193,7 +193,7 @@ def ftpscanrecursive(ftp_server, ftp_port=None, login=None, password=None,
    tree.append((current_dir, files))
 
    if len(tree) == 1:
-      raise FtpScanError, "the server ignores -R in LIST"
+      raise FtpScanError("the server ignores -R in LIST")
 
    return tree
 
index 9b7339001f1801fc0340b4adc3f6c008a160830a..c3df7373518a28f2a2e314fb2fc683670bab4955 100644 (file)
@@ -249,7 +249,7 @@ class SendmailTransport(EMailTransport):
 
       sendmail = self.find_sendmail()
       if not sendmail:
-         raise ValueError, "cannot find sendmail binary"
+         raise ValueError("cannot find sendmail binary")
       self.sendmail = sendmail
 
    def find_sendmail(self):
@@ -359,7 +359,7 @@ class BeeOnLine(CP1251HTTPTransport):
          reply_to=''):      # send reply to this e-mail
       Transport.__init__(self, phone, message)
       if mode not in ('GSM', 'DAMPS'):
-         raise ValueError, "mode (%s) must be either 'GSM' or 'DAMPS'" % mode
+         raise ValueError("mode (%s) must be either 'GSM' or 'DAMPS'" % mode)
       self.mode = mode
       self.transliterate = transliterate
       self.reply_to = reply_to
@@ -382,7 +382,7 @@ class BeeOnLine(CP1251HTTPTransport):
       elif prf == '7901':
          dict['network_code'] = '1'
       else:
-         raise RuntimeError, "incorrect combination of mode (%s) and prefix (%s)" % (self.mode, prf)
+         raise RuntimeError("incorrect combination of mode (%s) and prefix (%s)" % (self.mode, prf))
 
       dict['phone'] = self.phone[4:]
       dict['message'] = text
@@ -561,9 +561,9 @@ def Phone2Provider(phone):
    prefix = phone[1:4]
 
    if prefix in ("095", "901"): # 901 is being used by Beeline and SkyLink
-      raise SMSError, "unknown provider for phone %s" % phone
+      raise SMSError("unknown provider for phone %s" % phone)
 
    if Prefix2Provider.has_key(prefix):
       return Prefix2Provider[prefix]
 
-   raise SMSError, "bad prefix for phone %s" % phone
+   raise SMSError("bad prefix for phone %s" % phone)
index 46b9916b401556e5d2e1bec5c7a2b8128c4036d0..6d230860dcaa3c92d9be8897e04133e801260fd2 100755 (executable)
@@ -75,7 +75,7 @@ def DaysInMonth(Month, Year):
       return 28+IsLeapYear(_setYear(Year))
 
    else:
-      raise opdate_error, "bad month `%s'" % str(Month)
+      raise opdate_error("bad month `%s'" % str(Month))
 
 
 def ValidDate(Day, Month, Year):
@@ -235,7 +235,7 @@ def DateDiff(Date1, Date2):
 def DayOfWeek(Julian):
    """ Return the day of the week for the date. Returns DayType(7) if Julian == BadDate. """
    if Julian == BadDate:
-      raise opdate_error, "bad date `%s'" % str(Julian)
+      raise opdate_error("bad date `%s'" % str(Julian))
    else:
       return (Julian+FirstDayOfWeek) % 7