Merge pull request #26 from cbus-sea-lions-2015/fix-usersfetcher-nil-…

…libraryid

fix libraries not being made on seed

app/controllers/libraries_controller.rb

@@ -23,7 +23,7 @@ def create
     response = BggApi.new.collection({ username: bgg_user })
     if response.keys[0] != "error"
       library = Library.find_or_create_by(bgg_username: bgg_user)
-      UsersFetcher.perform_async(bgg_user,library.id)
+      UsersFetcher.perform_async(bgg_user)
       user.library_id = library.id
       user.save
       message = 1

app/workers/libraries_fetcher.rb

@@ -1,7 +1,7 @@
 class LibrariesFetcher
   include Sidekiq::Worker
 
-  def perform(game_ids, library_id = nil)
+  def perform(game_ids, library_id)
     game_ids.each_slice(100) do |slice|
       GamesFetcher.perform_async(slice, library_id)
     end

app/workers/users_fetcher.rb

@@ -1,13 +1,12 @@
 class UsersFetcher
   include Sidekiq::Worker
 
-  def perform(bgg_username,library_id = nil)
-    library = Library.find_by(id: library_id) if library_id
+  def perform(bgg_username)
+    library = Library.find_or_create_by(bgg_username: bgg_username)
     bgg_user_library = BggApi.new.collection({ username: bgg_username })
-    existing_games_array = []
-    existing_games_array += library.games.map { |game| game.bgg_id } if library
+    existing_games_array = library.games.map { |game| game.bgg_id }
     collection_array = bgg_user_library["item"].map { |game| game["objectid"].to_i }
     collection_array -= existing_games_array
-    LibrariesFetcher.perform_async(collection_array, library_id)
+    LibrariesFetcher.perform_async(collection_array, library.id)
   end
 end