mq: replace hasattr() with util.safehasattr(), update check-code.py stable
authorPatrick Mezard <patrick@mezard.eu>
Fri, 13 Apr 2012 15:07:13 +0200
branchstable
changeset 16416 c3aedd526d53
parent 16396 03e408a122c4
child 16428 9b26d541e972
child 16450 c9c8c9053119
mq: replace hasattr() with util.safehasattr(), update check-code.py
contrib/check-code.py
hgext/mq.py
--- a/contrib/check-code.py	Sun Apr 08 11:14:56 2012 +0200
+++ b/contrib/check-code.py	Fri Apr 13 15:07:13 2012 +0200
@@ -177,7 +177,7 @@
      "comparison with singleton, use 'is' or 'is not' instead"),
     (r'^\s*(while|if) [01]:',
      "use True/False for constant Boolean expression"),
-    (r'(?<!def)\s+hasattr',
+    (r'(?:(?<!def)\s+|\()hasattr',
      'hasattr(foo, bar) is broken, use util.safehasattr(foo, bar) instead'),
     (r'opener\([^)]*\).read\(',
      "use opener.read() instead"),
--- a/hgext/mq.py	Sun Apr 08 11:14:56 2012 +0200
+++ b/hgext/mq.py	Fri Apr 13 15:07:13 2012 +0200
@@ -3308,7 +3308,7 @@
         repo._phasedefaults.append(mqphasedefaults)
 
 def mqimport(orig, ui, repo, *args, **kwargs):
-    if (hasattr(repo, 'abortifwdirpatched')
+    if (util.safehasattr(repo, 'abortifwdirpatched')
         and not kwargs.get('no_commit', False)):
         repo.abortifwdirpatched(_('cannot import over an applied patch'),
                                    kwargs.get('force'))