openstack/python-novaclient
Code Issues Proposed changes

PEP8 python-novaclient cleanup

Browse Source 
Fixes bug #911552

The None, True, and False values are singletons.

All variable *comparisons* to singletons should use 'is' or 'is not'.
All variable *evaluations* to boolean should use 'if' or 'if not'.
All Object type comparisons should use isinstance()
instead of comparing types directly

Change-Id: Ia5571e58e2662c652f0e996d8c1a1acb4531623d
This commit is contained in:
lzyeval 2012-01-04 09:54:39 +08:00
parent 138dad3992
commit 4ff27703d0
5 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
AUTHORS
View File

@ -37,3 +37,4 @@ Robie Basak <robie.basak@canonical.com>
Sandy Walsh <sandy@darksecretsoftware.com>
Unmesh Gurjar <unmesh.gurjar@vertex.co.in>
William Wolf <throughnothing@gmail.com>
Zhongyue Luo <lzyeval@gmail.com>

2
novaclient/base.py
View File

@ -74,7 +74,7 @@ class Manager(utils.HookableMixin):
data = body[response_key]
# NOTE(ja): keystone returns values as list as {'values': [ ... ]}
# unlike other services which just return the list...
if type(data) is dict:
if isinstance(data, dict):
try:
data = data['values']
except KeyError:

2
novaclient/client.py
View File

@ -192,7 +192,7 @@ class HTTPClient(httplib2.Http):
magic_tuple = urlparse.urlsplit(self.auth_url)
scheme, netloc, path, query, frag = magic_tuple
port = magic_tuple.port
if port == None:
if port is None:
port = 80
path_parts = path.split('/')
for part in path_parts:

2
novaclient/v1_1/quotas.py
View File

@ -51,7 +51,7 @@ class QuotaSetManager(base.ManagerWithFind):
'cores': cores}}
for key in body['quota_set'].keys():
if body['quota_set'][key] == None:
if body['quota_set'][key] is None:
body['quota_set'].pop(key)
self._update('/os-quota-sets/%s' % (tenant_id), body)

4
novaclient/v1_1/shell.py
View File

@ -503,7 +503,7 @@ def do_rebuild(cs, args):
server = _find_server(cs, args.server)
image = _find_image(cs, args.image)
if args.rebuild_password != False:
if args.rebuild_password is not False:
_password = args.rebuild_password
else:
_password = None
@ -1012,7 +1012,7 @@ def _print_secgroup_rules(rules):
elif k == 'group':
k = 'source_group'
v = v.get('name')
if v == None:
if v is None:
v = ''
setattr(self, k, v)