Commit 2d4bb892 authored by Joshua Roesslein's avatar Joshua Roesslein
Browse files

Fix an issue where 'user' parameter value was getting overridden.

This caused issues with lists_memberships() when trying to get memberships
of an user besides the authenticated one. We now only set 'user' parameter
to the authenticated user when no 'user' value is provided when calling
the API method.
parent 8e680477
......@@ -81,18 +81,20 @@ def bind_api(**config):
self.parameters[k] = convert_to_utf8_str(arg)
# Set 'user' paramter to the authenticated user's name
# if no 'user' parameter value provided
if(self.api.auth and 'user' not in self.parameters):
self.parameters['user'] = self.api.auth.get_username()
def build_path(self):
for variable in re_path_template.findall(self.path):
name = variable.strip('{}')
if name == 'user' and self.api.auth:
value = self.api.auth.get_username()
else:
try:
value = urllib.quote(self.parameters[name])
except KeyError:
raise TweepError('No parameter value found for path variable: %s' % name)
del self.parameters[name]
try:
value = urllib.quote(self.parameters[name])
except KeyError:
raise TweepError('No parameter value found for path variable: %s' % name)
del self.parameters[name]
self.path = self.path.replace(variable, value)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment