Commits

Anonymous committed 6c23bbd

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.

  • Participants
  • Parent commits e719397

Comments (0)

Files changed (1)

File tweepy/binder.py

 
                 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)