Define 'self.force' in the constructor and remove the hack in '__getattr__()'
to account for it not being defined in the constructor.
This commit is contained in:
parent
5f8b12f27e
commit
d197a3a12c
|
@ -59,6 +59,12 @@ class Command:
|
||||||
self._verbose = None
|
self._verbose = None
|
||||||
self._dry_run = None
|
self._dry_run = None
|
||||||
|
|
||||||
|
# Some commands define a 'self.force' option to ignore file
|
||||||
|
# timestamps, but methods defined *here* assume that
|
||||||
|
# 'self.force' exists for all commands. So define it here
|
||||||
|
# just to be safe.
|
||||||
|
self.force = None
|
||||||
|
|
||||||
# The 'help' flag is just used for command-line parsing, so
|
# The 'help' flag is just used for command-line parsing, so
|
||||||
# none of that complicated bureaucracy is needed.
|
# none of that complicated bureaucracy is needed.
|
||||||
self.help = 0
|
self.help = 0
|
||||||
|
@ -79,11 +85,6 @@ class Command:
|
||||||
return getattr (self.distribution, attr)
|
return getattr (self.distribution, attr)
|
||||||
else:
|
else:
|
||||||
return myval
|
return myval
|
||||||
|
|
||||||
# Needed because some Command methods assume 'self.force' exists,
|
|
||||||
# but not all commands define 'self.force'. Ugh.
|
|
||||||
elif attr == 'force':
|
|
||||||
return None
|
|
||||||
else:
|
else:
|
||||||
raise AttributeError, attr
|
raise AttributeError, attr
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue