kag-gather 1.4.7 → 1.4.8

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ZjFhNmIyZDJiNzUwNzczMzFhZTQxYWY3NzBjYmVjZDAwZWNjY2I4MQ==
+    N2E4YWM4ODA4OGM2NGZjZGU3ZjAwNWY1ZGNhOWJiNWFlYjI3OWZlNw==
   data.tar.gz: !binary |-
-    NTEyMWNmODk5NjgwNDcyNDU4YjI1ZDMyM2Q4YzUxYWI3N2JjZmMzNw==
+    ZjA5M2QzZmY0OWNjZjA0N2YzYTUyNDNiODg5MGZlMzdjOTJhYTRhYw==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    Mjc4Nzg5OGQ0MDAyZTY2YWZiYTQ4ZjRjNjEyYTQwMjcxM2Y4ZjE0N2RmMWI4
-    ODU0Zjk1N2U2MWM5NDhkMDA4ZjVjNDg3OTg1N2VjMjg0ZGY1NzQ0NjZmNDc1
-    NTg3YzljMjBhMDViYTVhN2I5ZTJhY2QxM2NkNWFlMTU2ZjhmMzQ=
+    NjExZGRjMzY2ZTFjN2M2YjUxMjdiMTg2Y2JlODg5ZGMxY2M2OWZiMjQwZmIy
+    NjEyMTgwYWViNmU3NWExNzFlYTMxZjdhOWYzNWVhMjBmNTAxNGUxNGIzYTEz
+    ODM0Yzk3N2IzYzc0MjUzNDg3MWFjMDA2OTIxYTdmZDdkNTZmZGQ=
   data.tar.gz: !binary |-
-    MWI1MDdlYWM3ZjM4MjNmOGYwNTNkOGEzMmZmN2E2NDM5MzllNDBmM2E3YTJj
-    NTY2MzBiZTUwNDk3NjNmNzUxM2M4NmE2ZWIyOTU2Mzg3YWQ1NjFkMzAxZmQ0
-    Yzk5YjBlOWRkM2Y5Y2Q3ZDAxNGJmM2QyNDkxNmY0NWFlMWQ2MDk=
+    YWNmMDk1ZDMxN2M0MWU5MjNlYjllN2ViOGZhY2I2MDJhNGU2OTg1ZmY5MDVl
+    ZjA5YmNhYjE0NGNlNzQ5MmI0MWQ3NjJjMzE3NTIzODlkNjVmOWZhZDEzYzM1
+    NzdiN2E4MDZkOWYxZDEyZDVmN2M4ZTk3OGVhNTQ0ZDUwNWYwMzY=

data/lib/kag/gather/plugin.rb

@@ -142,20 +142,29 @@ module KAG
       def add_user_to_queue(m,user,send_msg = true)
         if @queue.has_player?(user)
           reply m,"#{user.authname} is already in the queue!"
+          false
         elsif get_match_in(user)
           reply m,"#{user.authname} is already in a match!"
+          false
         else
-          @queue.add(user)
-          send_channels_msg "Added #{user.authname} to queue (#{KAG::Gather::Match.type_as_string}) [#{@queue.length}]" if send_msg
-          check_for_new_match
+          if @queue.add(user)
+            send_channels_msg "Added #{user.authname} to queue (#{KAG::Gather::Match.type_as_string}) [#{@queue.length}]" if send_msg
+            check_for_new_match
+            true
+          else
+            false
+          end
         end
       end
 
       def remove_user_from_queue(user,send_msg = true)
         if @queue.has_player?(user)
-          @queue.remove(user)
-          send_channels_msg "Removed #{user.authname} from queue (#{KAG::Gather::Match.type_as_string}) [#{@queue.length}]" if send_msg
-          true
+          if @queue.remove(user)
+            send_channels_msg "Removed #{user.authname} from queue (#{KAG::Gather::Match.type_as_string}) [#{@queue.length}]" if send_msg
+            true
+          else
+            false
+          end
         else
           false
         end

data/lib/kag/gather/queue.rb

@@ -9,12 +9,18 @@ module KAG
       end
 
       def add(user)
+        return false unless user.authname and !user.authname.to_s.empty?
         self.players[user.authname.to_sym] = user
+        true
       end
 
       def remove(user)
+        return false unless user.authname and !user.authname.to_s.empty?
         if has_player?(user)
           self.players.delete(user.authname.to_sym)
+          true
+        else
+          false
         end
       end
 

data/lib/kag/version.rb

@@ -1,5 +1,5 @@
 module KAG
-  VERSION = '1.4.7'
+  VERSION = '1.4.8'
   def self.version
     VERSION
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: kag-gather
 version: !ruby/object:Gem::Version
-  version: 1.4.7
+  version: 1.4.8
 platform: ruby
 authors:
 - Shaun McCormick