Skip to content

Commit fbaee03

Browse files
Prefer literals over constructor for dicts (#34)
1 parent 81a84bc commit fbaee03

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

stream_chat/channel.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ def create(self, user_id):
7171
:param user_id: the ID of the user creating this channel
7272
:return:
7373
"""
74-
self.custom_data["created_by"] = dict(id=user_id)
74+
self.custom_data["created_by"] = {'id': user_id}
7575
return self.query(watch=False, state=False, presence=False)
7676

7777
def query(self, **options):
@@ -262,5 +262,5 @@ def show(self, user_id):
262262

263263
def add_user_id(payload, user_id):
264264
payload = payload.copy()
265-
payload.update(dict(user=dict(id=user_id)))
265+
payload.update({'user': {'id': user_id}})
266266
return payload

stream_chat/client.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ def __init__(self, api_key, api_secret, timeout=6.0, **options):
4040
self.session = requests.Session()
4141

4242
def get_default_params(self):
43-
return dict(api_key=self.api_key)
43+
return {'api_key': self.api_key}
4444

4545
def _parse_response(self, response):
4646
try:
@@ -127,32 +127,32 @@ def export_user(self, user_id, **options):
127127
return self.get("users/{}/export".format(user_id), options)
128128

129129
def ban_user(self, target_id, **options):
130-
data = dict(target_user_id=target_id)
130+
data = {'target_user_id': target_id}
131131
data.update(options)
132132
return self.post("moderation/ban", data=data)
133133

134134
def unban_user(self, target_id, **options):
135-
params = dict(target_user_id=target_id)
135+
params = {'target_user_id': target_id}
136136
params.update(options)
137137
return self.delete("moderation/ban", params)
138138

139139
def flag_message(self, target_id, **options):
140-
data = dict(target_message_id=target_id)
140+
data = {'target_message_id': target_id}
141141
data.update(options)
142142
return self.post("moderation/flag", data=data)
143143

144144
def unflag_message(self, target_id, **options):
145-
data = dict(target_message_id=target_id)
145+
data = {'target_message_id': target_id}
146146
data.update(options)
147147
return self.post("moderation/unflag", data=data)
148148

149149
def flag_user(self, target_id, **options):
150-
data = dict(target_user_id=target_id)
150+
data = {'target_user_id': target_id}
151151
data.update(options)
152152
return self.post("moderation/flag", data=data)
153153

154154
def unflag_user(self, target_id, **options):
155-
data = dict(target_user_id=target_id)
155+
data = {'target_user_id': target_id}
156156
data.update(options)
157157
return self.post("moderation/unflag", data=data)
158158

@@ -177,7 +177,7 @@ def unmute_user(self, target_id, user_id):
177177
:return:
178178
"""
179179

180-
data = dict(target_id=target_id, user_id=user_id)
180+
data = {'target_id': target_id, 'user_id': user_id}
181181
return self.post("moderation/unmute", data=data)
182182

183183
def mark_all_read(self, user_id):

0 commit comments

Comments
 (0)