leaderboard 3.0.0 → 3.0.1

data/CHANGELOG.markdown

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## leaderboard 3.0.1 (2012-12-19)
+
+* Fixed a bug in `remove_member` that would remove all of the optional member data.
+
 ## leaderboard 3.0.0 (2012-12-03)
 
 * Added `rank_member_if` and `rank_member_if_in` methods that allow you to rank a member in the leaderboard based on execution of a lambda. 

data/lib/leaderboard.rb

@@ -255,7 +255,7 @@ class Leaderboard
   def remove_member_from(leaderboard_name, member)
     @redis_connection.multi do |transaction|
       transaction.zrem(leaderboard_name, member)
-      transaction.del(member_data_key(leaderboard_name), member)
+      transaction.hdel(member_data_key(leaderboard_name), member)
     end
   end
 

data/lib/leaderboard/version.rb

@@ -1,4 +1,4 @@
 class Leaderboard
   # Leaderboard version
-  VERSION = '3.0.0'.freeze
+  VERSION = '3.0.1'.freeze
 end

data/spec/leaderboard_spec.rb

@@ -659,4 +659,16 @@ describe 'Leaderboard' do
     @leaderboard.rank_member_if(highscore_check, 'david', 1338)
     @leaderboard.score_for('david').should eql(1338.0)
   end
+
+  it 'should not delete all the member data when calling remove_member' do
+    rank_members_in_leaderboard
+    
+    @redis_connection.exists("name:member_data").should be_true
+    @redis_connection.hgetall("name:member_data").size.should be(5)
+    @leaderboard.total_members.should be(5)
+    @leaderboard.remove_member('member_1')
+    @redis_connection.exists("name:member_data").should be_true
+    @redis_connection.hgetall("name:member_data").size.should be(4)
+    @leaderboard.total_members.should be(4)
+  end
 end

data/spec/version_spec.rb

@@ -2,6 +2,6 @@ require 'spec_helper'
 
 describe 'Leaderboard::VERSION' do
   it 'should be the correct version' do
-    Leaderboard::VERSION.should == '3.0.0'
+    Leaderboard::VERSION.should == '3.0.1'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: leaderboard
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.0.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-03 00:00:00.000000000 Z
+date: 2012-12-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis