blizzard_api 0.5.3 → 0.6.1

data/lib/blizzard_api/starcraft/community/profile.rb

@@ -15,9 +15,9 @@ module BlizzardApi
       #
       # @!macro sc2_regions
       # @!macro request_options
-      def static(region_id, options = {})
+      def static(region_id, **options)
         reg = resolve_region(region_id)
-        api_request "#{base_url(:community)}/static/profile/#{reg}", { ttl: CACHE_DAY }.merge(options)
+        api_request "#{base_url(:community)}/static/profile/#{reg}", **{ ttl: CACHE_DAY }.merge(options)
       end
 
       ##
@@ -27,10 +27,10 @@ module BlizzardApi
       # @param [Integer] realm_id Realm ID
       # @param [Integer] profile_id Profile ID
       # @!macro request_options
-      def metadata(region_id, realm_id, profile_id, options = {})
+      def metadata(region_id, realm_id, profile_id, **options)
         reg = resolve_region(region_id)
         opts = { ttl: CACHE_DAY }.merge(options)
-        api_request "#{base_url(:community)}/metadata/profile/#{reg}/#{realm_id}/#{profile_id}", opts
+        api_request "#{base_url(:community)}/metadata/profile/#{reg}/#{realm_id}/#{profile_id}", **opts
       end
 
       ##
@@ -40,10 +40,10 @@ module BlizzardApi
       # @param [Integer] realm_id Realm ID
       # @param [Integer] profile_id Profile ID
       # @!macro request_options
-      def profile(region_id, realm_id, profile_id, options = {})
+      def profile(region_id, realm_id, profile_id, **options)
         reg = resolve_region(region_id)
         opts = { ttl: CACHE_DAY }.merge(options)
-        api_request "#{base_url(:community)}/profile/#{reg}/#{realm_id}/#{profile_id}", opts
+        api_request "#{base_url(:community)}/profile/#{reg}/#{realm_id}/#{profile_id}", **opts
       end
 
       ##
@@ -53,10 +53,10 @@ module BlizzardApi
       # @param [Integer] realm_id Realm ID
       # @param [Integer] profile_id Profile ID
       # @!macro request_options
-      def ladder_summary(region_id, realm_id, profile_id, options = {})
+      def ladder_summary(region_id, realm_id, profile_id, **options)
         reg = resolve_region(region_id)
         opts = { ttl: CACHE_DAY }.merge(options)
-        api_request "#{base_url(:community)}profile/#{reg}/#{realm_id}/#{profile_id}/ladder/summary ", opts
+        api_request "#{base_url(:community)}profile/#{reg}/#{realm_id}/#{profile_id}/ladder/summary ", **opts
       end
 
       ##
@@ -66,10 +66,10 @@ module BlizzardApi
       # @param [Integer] realm_id Realm ID
       # @param [Integer] profile_id Profile ID
       # @!macro request_options
-      def ladder(region_id, realm_id, profile_id, ladder_id, options = {})
+      def ladder(region_id, realm_id, profile_id, ladder_id, **options)
         reg = resolve_region(region_id)
         opts = { ttl: CACHE_DAY }.merge(options)
-        api_request "#{base_url(:community)}/profile/#{reg}/#{realm_id}/#{profile_id}/ladder/#{ladder_id}", opts
+        api_request "#{base_url(:community)}/profile/#{reg}/#{realm_id}/#{profile_id}/ladder/#{ladder_id}", **opts
       end
     end
   end

data/lib/blizzard_api/starcraft/game_data/league.rb

@@ -18,9 +18,9 @@ module BlizzardApi
       # @param [Integer] team_type Team type
       # @param [Integer] league_id League id
       # @!macro request_options
-      def get(season_id, queue_id, team_type, league_id, options = {})
+      def get(season_id, queue_id, team_type, league_id, **options)
         opts = { ttl: CACHE_DAY }.merge(options)
-        api_request "#{base_url(:game_data)}/league/#{season_id}/#{queue_id}/#{team_type}/#{league_id}", opts
+        api_request "#{base_url(:game_data)}/league/#{season_id}/#{queue_id}/#{team_type}/#{league_id}", **opts
       end
     end
   end

data/lib/blizzard_api/version.rb

@@ -2,5 +2,5 @@
 
 module BlizzardApi
   # Gem version
-  VERSION = '0.5.3'
+  VERSION = '0.6.1'
 end

data/lib/blizzard_api/wow.rb

@@ -275,13 +275,6 @@ module BlizzardApi
       BlizzardApi::Wow::WowToken.new(region)
     end
 
-    ##
-    # @param region [String] API Region
-    # @return {Character}
-    def self.character(region = BlizzardApi.region)
-      BlizzardApi::Wow::Character.new(region)
-    end
-
     require_relative 'wow/profile/profile'
     require_relative 'wow/profile/guild'
     require_relative 'wow/profile/character_profile'

data/lib/blizzard_api/wow/game_data/achievement.rb

@@ -24,8 +24,8 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def categories(options = {})
-        api_request "#{endpoint_uri('category')}/index", default_options.merge(options)
+      def categories(**options)
+        api_request "#{endpoint_uri('category')}/index", **default_options.merge(options)
       end
 
       ##
@@ -34,8 +34,8 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def category(id, options = {})
-        api_request "#{endpoint_uri('category')}/#{id}", default_options.merge(options)
+      def category(id, **options)
+        api_request "#{endpoint_uri('category')}/#{id}", **default_options.merge(options)
       end
 
       ##
@@ -44,8 +44,8 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def media(id, options = {})
-        api_request "#{base_url(:media)}/achievement/#{id}", default_options.merge(options)
+      def media(id, **options)
+        api_request "#{base_url(:media)}/achievement/#{id}", **default_options.merge(options)
       end
 
       protected

data/lib/blizzard_api/wow/game_data/auction.rb

@@ -10,16 +10,34 @@ module BlizzardApi
     # You can get an instance of this class using the default region as follows:
     #   api_instance = BlizzardApi::Wow.auction
     class Auction < Wow::Request
+      ##
+      # Return all auction houses for the specified connected realm. Classic only.
+      #
+      # @param connected_realm_id [Integer] A valid connected realm id
+      # @!macro request_options
+      #
+      # @!macro response
+      def index(connected_realm_id, **options)
+        opts = { ttl: CACHE_HOUR, namespace: :dynamic }.merge(options)
+        api_request "#{base_url(:game_data)}/connected-realm/#{connected_realm_id}/auctions/index", **opts
+      end
+
       ##
       # Return all active auctions for the specified connected realm
       #
       # @param connected_realm_id [Integer] A valid connected realm id
       # @!macro request_options
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro response
-      def get(connected_realm_id, options = {})
+      def get(connected_realm_id, auction_house_id = nil, **options)
         opts = { ttl: CACHE_HOUR, namespace: :dynamic }.merge(options)
-        api_request "#{base_url(:game_data)}/connected-realm/#{connected_realm_id}/auctions", opts
+
+        unless auction_house_id.nil?
+          return api_request "#{base_url(:game_data)}/connected-realm/#{connected_realm_id}/auctions/#{auction_house_id}", **opts
+        end
+
+        api_request "#{base_url(:game_data)}/connected-realm/#{connected_realm_id}/auctions", **opts
       end
     end
   end

data/lib/blizzard_api/wow/game_data/azerite_essence.rb

@@ -20,8 +20,8 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def media(id, options = {})
-        api_request "#{base_url(:media)}/azerite-essence/#{id}", default_options.merge(options)
+      def media(id, **options)
+        api_request "#{base_url(:media)}/azerite-essence/#{id}", **default_options.merge(options)
       end
 
       protected

data/lib/blizzard_api/wow/game_data/covenant.rb

@@ -18,8 +18,8 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def media(id, options = {})
-        api_request "#{base_url(:media)}/covenant/#{id}", default_options.merge(options)
+      def media(id, **options)
+        api_request "#{base_url(:media)}/covenant/#{id}", **default_options.merge(options)
       end
 
       ##
@@ -28,8 +28,8 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def soulbinds(options = {})
-        api_request "#{base_url(:game_data)}/covenant/soulbind/index", default_options.merge(options)
+      def soulbinds(**options)
+        api_request "#{base_url(:game_data)}/covenant/soulbind/index", **default_options.merge(options)
       end
 
       ##
@@ -40,8 +40,8 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def soulbind(id, options = {})
-        api_request "#{base_url(:game_data)}/covenant/soulbind/#{id}", default_options.merge(options)
+      def soulbind(id, **options)
+        api_request "#{base_url(:game_data)}/covenant/soulbind/#{id}", **default_options.merge(options)
       end
 
       ##
@@ -50,8 +50,8 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def conduits(options = {})
-        api_request "#{base_url(:game_data)}/covenant/conduit/index", default_options.merge(options)
+      def conduits(**options)
+        api_request "#{base_url(:game_data)}/covenant/conduit/index", **default_options.merge(options)
       end
 
       ##
@@ -62,8 +62,8 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def conduit(id, options = {})
-        api_request "#{base_url(:game_data)}/covenant/conduit/#{id}", default_options.merge(options)
+      def conduit(id, **options)
+        api_request "#{base_url(:game_data)}/covenant/conduit/#{id}", **default_options.merge(options)
       end
 
       protected

data/lib/blizzard_api/wow/game_data/creature.rb

@@ -13,7 +13,7 @@ module BlizzardApi
       include BlizzardApi::Wow::Searchable
 
       def index
-        raise BlizzardApi::ApiException, 'Creatures endpoint doesn\'t have a index method'
+        raise BlizzardApi::ApiException, 'Creatures endpoint doesn\'t have an index method'
       end
 
       def complete
@@ -25,10 +25,11 @@ module BlizzardApi
       #
       # @!macro request_options
       # @option options [Boolean] :classic If set to true, this method will call the classic version
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro response
-      def families(options = {})
-        api_request "#{endpoint_uri('family')}/index", default_options.merge(options)
+      def families(**options)
+        api_request "#{endpoint_uri('family')}/index", **default_options.merge(options)
       end
 
       ##
@@ -38,10 +39,11 @@ module BlizzardApi
       #
       # @!macro request_options
       # @option options [Boolean] :classic If set to true, this method will call the classic version
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro response
-      def family(id, options = {})
-        api_request "#{endpoint_uri('family')}/#{id}", default_options.merge(options)
+      def family(id, **options)
+        api_request "#{endpoint_uri('family')}/#{id}", **default_options.merge(options)
       end
 
       ##
@@ -51,10 +53,11 @@ module BlizzardApi
       #
       # @!macro request_options
       # @option options [Boolean] :classic If set to true, this method will call the classic version
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro response
-      def family_media(id, options = {})
-        api_request "#{base_url(:media)}/creature-family/#{id}", default_options.merge(options)
+      def family_media(id, **options)
+        api_request "#{base_url(:media)}/creature-family/#{id}", **default_options.merge(options)
       end
 
       ##
@@ -62,10 +65,11 @@ module BlizzardApi
       #
       # @!macro request_options
       # @option options [Boolean] :classic If set to true, this method will call the classic version
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro response
-      def types(options = {})
-        api_request "#{endpoint_uri('type')}/index", default_options.merge(options)
+      def types(**options)
+        api_request "#{endpoint_uri('type')}/index", **default_options.merge(options)
       end
 
       ##
@@ -75,10 +79,11 @@ module BlizzardApi
       #
       # @!macro request_options
       # @option options [Boolean] :classic If set to true, this method will call the classic version
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro response
-      def type(id, options = {})
-        api_request "#{endpoint_uri('type')}/#{id}", default_options.merge(options)
+      def type(id, **options)
+        api_request "#{endpoint_uri('type')}/#{id}", **default_options.merge(options)
       end
 
       ##
@@ -88,10 +93,11 @@ module BlizzardApi
       #
       # @!macro request_options
       # @option options [Boolean] :classic If set to true, this method will call the classic version
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro response
-      def display_media(id, options = {})
-        api_request "#{base_url(:media)}/creature-display/#{id}", default_options.merge(options)
+      def display_media(id, **options)
+        api_request "#{base_url(:media)}/creature-display/#{id}", **default_options.merge(options)
       end
 
       protected

data/lib/blizzard_api/wow/game_data/generic_data_endpoint.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'thwait'
+
 module BlizzardApi
   module Wow
     ##
@@ -18,8 +20,8 @@ module BlizzardApi
       #
       # @!macro request_options
       # @!macro response
-      def index(options = {})
-        api_request "#{endpoint_uri}/index", default_options.merge(options)
+      def index(**options)
+        api_request "#{endpoint_uri}/index", **default_options.merge(options)
       end
 
       ##
@@ -29,21 +31,31 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def get(id, options = {})
-        api_request "#{endpoint_uri}/#{id}", default_options.merge(options)
+      def get(id, **options)
+        api_request "#{endpoint_uri}/#{id}", **default_options.merge(options)
       end
 
       ##
       # @!macro complete
-      def complete(options = {})
-        [].tap do |complete_data|
-          index_data = index options
-          index_data[@collection.to_sym].each do |item|
-            link = item.key?(:key) ? item[:key][:href] : item[:href]
-            item_data = request link
-            complete_data.push item_data
+      def complete(**options)
+        payload = [].tap do |complete_data|
+          index_data = index(**options)
+          threads = []
+
+          concurrency = options.key?(:concurrency) ? options[:concurrency].to_i : BlizzardApi.concurrency
+          concurrency.times do
+            threads << Thread.new do
+              while index_data[@collection.to_sym].size.positive?
+                item = index_data[@collection.to_sym].pop
+                link = item.key?(:key) ? item[:key][:href] : item[:href]
+                item_data = request link
+                complete_data.push item_data
+              end
+            end
           end
+          ThreadsWait.all_waits threads
         end
+        payload.sort { |a, b| a[:id] <=> b[:id] }
       end
 
       protected

data/lib/blizzard_api/wow/game_data/guild_crest.rb

@@ -21,10 +21,11 @@ module BlizzardApi
       #
       # @!macro request_options
       # @option options [Boolean] :classic If set to true, this method will call the classic version
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro response
-      def border_media(id, options = {})
-        api_request "#{base_url(:media)}/#{@endpoint}/border/#{id}", default_options.merge(options)
+      def border_media(id, **options)
+        api_request "#{base_url(:media)}/#{@endpoint}/border/#{id}", **default_options.merge(options)
       end
 
       ##
@@ -32,12 +33,13 @@ module BlizzardApi
       #
       # @param id [Integer] Emblem id
       # @option options [Boolean] :classic If set to true, this method will call the classic version
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro request_options
       #
       # @!macro response
-      def emblem_media(id, options = {})
-        api_request "#{base_url(:media)}/#{@endpoint}/emblem/#{id}", default_options.merge(options)
+      def emblem_media(id, **options)
+        api_request "#{base_url(:media)}/#{@endpoint}/emblem/#{id}", **default_options.merge(options)
       end
 
       protected

data/lib/blizzard_api/wow/game_data/item.rb

@@ -33,10 +33,11 @@ module BlizzardApi
       #
       # @!macro request_options
       # @option options [Boolean] :classic If set to true, this method will call the classic version
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro response
-      def classes(options = {})
-        api_request "#{endpoint_uri('class')}/index", default_options.merge(options)
+      def classes(**options)
+        api_request "#{endpoint_uri('class')}/index", **default_options.merge(options)
       end
 
       ##
@@ -45,10 +46,11 @@ module BlizzardApi
       # @param id [Integer] Item class id
       # @!macro request_options
       # @option options [Boolean] :classic If set to true, this method will call the classic version
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro response
-      def class(id, options = {})
-        api_request "#{endpoint_uri('class')}/#{id}", default_options.merge(options)
+      def class(id, **options)
+        api_request "#{endpoint_uri('class')}/#{id}", **default_options.merge(options)
       end
 
       ##
@@ -58,10 +60,11 @@ module BlizzardApi
       # @param subclass_id [Integer] Item subclass id
       # @!macro request_options
       # @option options [Boolean] :classic If set to true, this method will call the classic version
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro response
-      def subclass(id, subclass_id, options = {})
-        api_request "#{endpoint_uri('class')}/#{id}/item-subclass/#{subclass_id}", default_options.merge(options)
+      def subclass(id, subclass_id, **options)
+        api_request "#{endpoint_uri('class')}/#{id}/item-subclass/#{subclass_id}", **default_options.merge(options)
       end
 
       ##
@@ -71,10 +74,11 @@ module BlizzardApi
       #
       # @!macro request_options
       # @option options [Boolean] :classic If set to true, this method will call the classic version
+      # @option options [Boolean] :classic1x If set to true, this method will call the classic era version
       #
       # @!macro response
-      def media(id, options = {})
-        api_request "#{base_url(:media)}/item/#{id}", default_options.merge(options)
+      def media(id, **options)
+        api_request "#{base_url(:media)}/item/#{id}", **default_options.merge(options)
       end
 
       ##
@@ -83,8 +87,8 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def sets(options = {})
-        api_request "#{endpoint_uri('set')}/index", default_options.merge(options)
+      def sets(**options)
+        api_request "#{endpoint_uri('set')}/index", **default_options.merge(options)
       end
 
       ##
@@ -94,8 +98,8 @@ module BlizzardApi
       # @!macro request_options
       #
       # @!macro response
-      def set(id, options = {})
-        api_request "#{endpoint_uri('set')}/#{id}", default_options.merge(options)
+      def set(id, **options)
+        api_request "#{endpoint_uri('set')}/#{id}", **default_options.merge(options)
       end
 
       protected