Skip to content

Commit 8c9355d

Browse files
committed
Use str.format() for string formatting
Migrate the package’s usage of the older style ‘%’ string formatting for newer str.format() method.
1 parent ffad830 commit 8c9355d

File tree

4 files changed

+13
-13
lines changed

4 files changed

+13
-13
lines changed

ciscosparkapi/exceptions.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,9 @@ def __init__(self, response_code, request=None, response=None):
3939
response_text = SPARK_RESPONSE_CODES.get(response_code)
4040
if response_text:
4141
self.response_text = response_text
42-
error_message = "Response Code [%s] - %s" % \
43-
(response_code, response_text)
42+
error_message = "Response Code [{!s}] - {}".format(response_code,
43+
response_text)
4444
else:
45-
error_message = "Response Code [%s] - Unknown Response Code" % \
46-
response_code
45+
error_message = "Response Code [{!s}] - " \
46+
"Unknown Response Code".format(response_code)
4747
super(SparkApiError, self).__init__(error_message)

ciscosparkapi/helper.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ def validate_base_url(base_url):
3535
def raise_if_extra_kwargs(kwargs):
3636
"""Raise a TypeError if kwargs is not empty."""
3737
if kwargs:
38-
raise TypeError("Unexpected **kwargs: %r" % kwargs)
38+
raise TypeError("Unexpected **kwargs: {!r}".format(kwargs))
3939

4040

4141
def check_response_code(response, erc):

ciscosparkapi/restsession.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -148,8 +148,8 @@ def get_items(self, url, params=None, **kwargs):
148148
for item in items:
149149
yield item
150150
else:
151-
error_message = "'items' object not found in JSON data: %r" \
152-
% json_page
151+
error_message = "'items' object not found in JSON data: " \
152+
"{!r}".format(json_page)
153153
raise ciscosparkapiException(error_message)
154154

155155
def post(self, url, json, **kwargs):

ciscosparkapi/sparkdata.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,8 @@ def _json_dict(json):
4848
elif isinstance(json, string_types):
4949
return json_pkg.loads(json)
5050
else:
51-
error = "'json' must be a dictionary or string; " \
52-
"received: %r" % json
51+
error = "'json' must be a dictionary or JSON string; " \
52+
"received: {!r}".format(json)
5353
raise TypeError(error)
5454

5555

@@ -96,19 +96,19 @@ def __getattr__(self, item):
9696
else:
9797
return item_data
9898
else:
99-
error = "'%s' object has no attribute '%s'" % \
100-
(self.__class__.__name__, item)
99+
error = "'{}' object has no attribute " \
100+
"'{}'".format(self.__class__.__name__, item)
101101
raise AttributeError(error)
102102

103103
def __str__(self):
104104
"""Return a human-readable string representation of this object."""
105105
class_str = self.__class__.__name__
106106
json_str = json_pkg.dumps(self._json, indent=2)
107-
return "%s:\n%s" % (class_str, json_str)
107+
return "{}:\n{}".format(class_str, json_str)
108108

109109
def __repr__(self):
110110
"""Return a string representing this object as valid Python expression.
111111
"""
112112
class_str = self.__class__.__name__
113113
json_str = json_pkg.dumps(self._json, ensure_ascii=False)
114-
return "%s(%r)" % (class_str, json_str)
114+
return "{}({})".format(class_str, json_str)

0 commit comments

Comments
 (0)