Skip to content

Commit

Permalink
Fixed list_remove_member bug when using OAuth
Browse files Browse the repository at this point in the history
  • Loading branch information
Wynn Netherland committed Nov 29, 2009
1 parent 9bf9418 commit b20b770
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions History
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
0.7.7 - November 29, 2009
* Fixed bug when removing list members using OAuth
0.7.6 - November 25, 2009
* Patch from coderifous for home_timeline support
0.7.5 - November 17, 2009
Expand Down
2 changes: 1 addition & 1 deletion lib/twitter/base.rb
Original file line number Diff line number Diff line change
Expand Up @@ -216,7 +216,7 @@ def list_add_member(list_owner_username, slug, new_id)
end

def list_remove_member(list_owner_username, slug, id)
perform_delete("/#{list_owner_username}/#{slug}/members.json", :body => {:id => id})
perform_delete("/#{list_owner_username}/#{slug}/members.json", :query => {:id => id})
end

def is_list_member?(list_owner_username, slug, id)
Expand Down
2 changes: 1 addition & 1 deletion test/twitter/base_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ class BaseTest < Test::Unit::TestCase
end

should "be able to remove a member from a list" do
stub_delete('/pengwynn/rubyists/members.json', 'user.json')
stub_delete('/pengwynn/rubyists/members.json?id=4243', 'user.json')
user = @twitter.list_remove_member('pengwynn', 'rubyists', 4243)
user.screen_name.should == 'jnunemaker'
end
Expand Down

0 comments on commit b20b770

Please sign in to comment.