sports_data_api 0.11.6 → 0.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 3511d9beaa61092bb9e781e948e51229d09daeb5
-  data.tar.gz: e6c9aafd2aacffa71397b865c151cb71a5bfa21c
+SHA256:
+  metadata.gz: 8a2556b1cd7f55fd366400025462306b46fb1dc9a45449e4d038b15e3c2dc344
+  data.tar.gz: b3a2452b53e1aa99f4ea928bc64ec12f60fc18ae2f3140a8c8379a4d0f2742c2
 SHA512:
-  metadata.gz: 9dc906dedbf8503dbe1aed7dc3bedbf78f8f9fdb444fc9d2344bb322c94059224c7335f5558a972511aa24831bcda20e9ebe86d7350511107d90c0e26ab7d9dd
-  data.tar.gz: 6e7aaebc3155e47b2975c7faf65d55fdd22eb1adfa6217f70af9979676306ae5d468e146c0bb91bd3288ef96c4fd6d96f0a4bc7ee55c9b13ab86ed422533bba2
+  metadata.gz: d63d5635e7b7097506310fd81257ad21cd0f209bf42cbc8fa30667ade1ab11feadef2811f4d675da5dc6ffcc8cf2b2e2d03d6b19c0ee7327e3244ae5db58d588
+  data.tar.gz: 47adca1c31f93dd5cfadffd80dd475c036cae42ef1cf371d409a92f1c2eca7f8ae21d0bca8d4cfddbb44e97518a25eeec4e82e592fe48aa7e4988e449abdaa1b

data/.travis.yml

@@ -5,6 +5,7 @@ rvm:
   - 2.2
   - 2.3
   - 2.4.0
+  - 2.5.0
   - ruby-head
 addons:
   code_climate:

data/lib/sports_data_api/nba/broadcast.rb

@@ -1,14 +1,6 @@
 module SportsDataApi
   module Nba
-    class Broadcast
-      attr_reader :network, :satellite
-      def initialize(xml)
-        xml = xml.first if xml.is_a? Nokogiri::XML::NodeSet
-        if xml.is_a? Nokogiri::XML::Element
-          @network = xml['network']
-          @satellite = xml['satellite']
-        end
-      end
+    class Broadcast < SportsDataApi::JsonData
     end
   end
 end

data/lib/sports_data_api/nba/game.rb

@@ -1,49 +1,56 @@
 module SportsDataApi
   module Nba
     class Game
-      attr_reader :id, :scheduled, :home, :home_team, :away,
-        :away_team, :status, :venue, :broadcast, :year, :season,
-        :date, :quarter, :clock, :home_team_id, :away_team_id
-
-      def initialize(args={})
-        xml = args.fetch(:xml)
-        @year = args[:year] ? args[:year].to_i : nil
-        @season = args[:season] ? args[:season].to_sym : nil
-        @date = args[:date]
-
-        xml = xml.first if xml.is_a? Nokogiri::XML::NodeSet
-        if xml.is_a? Nokogiri::XML::Element
-          @id = xml['id']
-          @scheduled = Time.parse xml['scheduled']
-          @home = xml['home_team']
-          @away = xml['away_team']
-          @home_team_id = xml['home_team']
-          @away_team_id = xml['away_team']
-          @status = xml['status']
-          @clock = xml['clock']
-          @quarter = xml['quarter'] ? xml['quarter'].to_i : nil
-
-          team_xml = xml.xpath('team')
-          if team_xml.empty?
-            # we are coming from the schedule API
-            @home_team = Team.new(xml.xpath('home'))
-            @away_team = Team.new(xml.xpath('away'))
-          else
-            # we are coming from the Game Summary API
-            @home_team = Team.new(team_xml.first)
-            @away_team = Team.new(team_xml.last)
-          end
-
-          @venue = Venue.new(xml.xpath('venue'))
-          @broadcast = Broadcast.new(xml.xpath('broadcast'))
-        end
+      attr_reader :id, :status, :year, :season, :clock
+
+      def initialize(json:, year: nil, season: nil)
+        @json = json
+        @year = year
+        @season = season
+        @id = json['id']
+        @status = json['status']
+        @clock = json['clock']
+      end
+
+      def scheduled
+        @scheduled ||= Time.iso8601 json['scheduled']
+      end
+
+      def home_team_id
+        json['home']['id']
+      end
+
+      def away_team_id
+        json['away']['id']
+      end
+
+      def home_team
+        @home_team ||= Team.new(json['home'])
+      end
+
+      def away_team
+        @away_team ||= Team.new(json['away'])
+      end
+
+      def quarter
+        return unless json['quarter']
+        json['quarter'].to_i
+      end
+
+      def venue
+        @venue ||= Venue.new(json['venue'])
+      end
+
+      def broadcast
+        return nil if json['broadcast'].nil? || json['broadcast'].empty?
+        @broadcast ||= Broadcast.new(json['broadcast'])
       end
 
       ##
       # Wrapper for Nba.game_summary
       # TODO
       def summary
-        Nba.game_summary(@id)
+        Nba.game_summary(id)
       end
 
       ##
@@ -59,6 +66,18 @@ module SportsDataApi
       def boxscore
         raise NotImplementedError
       end
+
+      private
+
+      attr_reader :json
+
+      def game_summary?
+        !team_json.nil?
+      end
+
+      def team_json
+        @team_json ||= json['team']
+      end
     end
   end
 end

data/lib/sports_data_api/nba/games.rb

@@ -2,25 +2,28 @@ module SportsDataApi
   module Nba
     class Games
       include Enumerable
-      attr_reader :games, :date
 
-      def initialize(xml)
-        @date = xml.first['date']
+      attr_reader :date
 
-        @games = xml.xpath("games/game").map do |game_xml|
-          Game.new(date: @date, xml: game_xml)
-        end
+      def initialize(json)
+        @json = json
+        @date = json['date']
       end
 
-      def each &block
-        @games.each do |game|
-          if block_given?
-            block.call game
-          else
-            yield game
-          end
+      def games
+        @games ||= json['games'].map do |game_json|
+          Game.new(json: game_json)
         end
       end
+
+      def each
+        return games.each unless block_given?
+        games.each { |game| yield game }
+      end
+
+      private
+
+      attr_reader :json
     end
   end
 end

data/lib/sports_data_api/nba/player.rb

@@ -1,21 +1,9 @@
 module SportsDataApi
   module Nba
-    class Player
-      attr_reader :stats
-
-      def initialize(xml)
-        if xml.is_a? Nokogiri::XML::Element
-          player_ivar = self.instance_variable_set("@#{xml.name}", {})
-          self.class.class_eval { attr_reader :"#{xml.name}" }
-          xml.attributes.each do | attr_name, attr_value|
-            player_ivar[attr_name.to_sym] = attr_value.value
-          end
-
-          stats_xml = xml.xpath('statistics')
-          if stats_xml.is_a? Nokogiri::XML::NodeSet and stats_xml.count > 0
-            @stats = SportsDataApi::Stats.new(stats_xml.first)
-          end
-        end
+    class Player < SportsDataApi::JsonData
+      def stats
+        return if player[:statistics].nil? || player[:statistics].empty?
+        @stats ||= SportsDataApi::Nba::Stats.new(player[:statistics])
       end
     end
   end

data/lib/sports_data_api/nba/season.rb

@@ -1,17 +1,18 @@
 module SportsDataApi
   module Nba
     class Season
-      attr_reader :id, :year, :type, :games
+      attr_reader :id, :year, :type
 
-      def initialize(xml)
-        if xml.is_a? Nokogiri::XML::NodeSet
-          @id = xml.first["id"]
-          @year = xml.first["year"].to_i
-          @type = xml.first["type"].to_sym
+      def initialize(json)
+        @json = json
+        @id = json['season']['id']
+        @year = json['season']['year']
+        @type = json['season']['type'].to_sym
+      end
 
-          @games = xml.first.xpath("games/game").map do |game_xml|
-            Game.new(year: @year, season: @type, xml: game_xml)
-          end
+      def games
+        @games ||= json['games'].map do |game_json|
+          Game.new(year: year, season: type, json: game_json)
         end
       end
 
@@ -21,8 +22,12 @@ module SportsDataApi
       #
       # The only valid types are: :pre, :reg, :pst
       def self.valid?(season)
-        [:PRE, :REG, :PST].include?(season)
+        %i[PRE REG PST].include?(season)
       end
+
+      private
+
+      attr_reader :json
     end
   end
 end

data/lib/sports_data_api/nba/stats.rb

@@ -0,0 +1,6 @@
+module SportsDataApi
+  module Nba
+    class Stats < SportsDataApi::JsonData
+    end
+  end
+end

data/lib/sports_data_api/nba/team.rb

@@ -1,23 +1,31 @@
 module SportsDataApi
   module Nba
     class Team
-      attr_reader :id, :name, :market, :alias, :conference, :division,
-                  :stats, :players, :points
-
-      def initialize(xml, conference = nil, division = nil)
-        xml = xml.first if xml.is_a? Nokogiri::XML::NodeSet
-        if xml.is_a? Nokogiri::XML::Element
-          @id = xml['id']
-          @name = xml['name']
-          @market = xml['market']
-          @alias = xml['alias']
-          @points = xml['points'] ? xml['points'].to_i : nil
-          @conference = conference
-          @division = division
-          @players = xml.xpath("players/player").map do |player_xml|
-            Player.new(player_xml)
-          end
-        end
+      attr_reader :id, :name, :market, :alias, :conference, :division, :stats
+
+      def initialize(json, conference = nil, division = nil)
+        @conference = conference
+        @division = division
+        @id = json['id']
+        @name = json['name']
+        @market = json['market']
+        @alias = json['alias']
+        @points = json['points']
+        @alias = json['alias']
+        @json = json
+      end
+
+      def points
+        @points ||= json['points'] ? json['points'].to_i : nil
+      end
+
+      def players
+        return [] if json['players'].nil? || json['players'].empty?
+        @players ||= json['players'].map { |x| Player.new(x) }
+      end
+
+      def venue
+        @venue ||= Venue.new(json['venue']) if json['venue']
       end
 
       ##
@@ -34,6 +42,10 @@ module SportsDataApi
           super(other)
         end
       end
+
+      private
+
+      attr_reader :json
     end
   end
 end

data/lib/sports_data_api/nba/teams.rb

@@ -3,81 +3,30 @@ module SportsDataApi
     class Teams
       include Enumerable
 
-      attr_reader :conferences, :divisions
-
-      ##
-      # Initialize by passing the raw XML returned from the API
-      def initialize(xml)
-        @teams = []
-        xml = xml.first if xml.is_a? Nokogiri::XML::NodeSet
-        if xml.is_a? Nokogiri::XML::Element
-          xml.xpath('conference').each do |conference|
-            # Conference ID, e.g., EASTERN or WESTERN
-            cname = conference['alias']
-
-            conference.xpath('division').each do |division|
-              # Division ID, e.g., ATLANTIC, PACIFIC
-              dname = division['alias']
+      def initialize(json)
+        @json = json
+      end
 
-              # Create a new team
-              @teams << division.xpath('team').map { |team| Team.new(team, cname, dname) }
+      def teams
+        @teams ||= json['conferences'].flat_map do |conference|
+          conference['divisions'].flat_map do |division|
+            division['teams'].map do |json|
+              Team.new(json, conference['alias'], division['alias'])
             end
           end
         end
-
-        @teams.flatten!
-
-        uniq_conferences = @teams.map { |team| team.conference.upcase }.uniq
-        @allowed_conferences = uniq_conferences.map { |str| str.to_sym }.concat(uniq_conferences.map { |str| str.downcase.to_sym })
-        @conferences = uniq_conferences.map { |conf| conf.to_sym }
-
-        uniq_divisions = @teams.map { |team| team.division.upcase }.uniq
-        @divisions =  @teams.map { |team| team.division.to_sym }.uniq
-        @allowed_divisions = uniq_divisions.map { |str| str.to_sym }.concat(uniq_divisions.map { |str| str.downcase.to_sym })
-      end
-
-      def [](search_index)
-        found_index = @teams.index(search_index)
-        unless found_index.nil?
-          @teams[found_index]
-        end
       end
 
       ##
-      #
-      def respond_to?(method)
-        @allowed_conferences.include?(method) || @allowed_divisions.include?(method)
+      # Make the class Enumerable
+      def each
+        return teams.each unless block_given?
+        teams.each { |team| yield team }
       end
 
-      ##
-      #
-      def method_missing(method)
-        if @allowed_conferences.include?(method)
-          @teams.select do |team|
-            up = team.conference.upcase.to_sym
-            dw = team.conference.downcase.to_sym
-            up === method || dw === method
-          end
-        elsif @allowed_divisions.include?(method)
-          @teams.select do |team|
-            up = team.division.upcase.to_sym
-            dw = team.division.downcase.to_sym
-            up === method || dw === method
-          end
-        end
-      end
+      private
 
-      ##
-      # Make the class Enumerable
-      def each(&block)
-        @teams.each do |team|
-          if block_given?
-            block.call team
-          else
-            yield team
-          end
-        end
-      end
+      attr_reader :json
     end
   end
 end

data/lib/sports_data_api/nba/venue.rb

@@ -2,18 +2,16 @@ module SportsDataApi
   module Nba
     class Venue
       attr_reader :id, :name, :address, :city, :state, :zip, :country, :capacity
-      def initialize(xml)
-        xml = xml.first if xml.is_a? Nokogiri::XML::NodeSet
-        if xml.is_a? Nokogiri::XML::Element
-          @id = xml['id']
-          @name = xml['name']
-          @address = xml['address']
-          @city = xml['city']
-          @state = xml['state']
-          @zip = xml['zip']
-          @country = xml['country']
-          @capacity = xml['capacity']
-        end
+
+      def initialize(json)
+        @id = json['id']
+        @name = json['name']
+        @address = json['address']
+        @city = json['city']
+        @state = json['state']
+        @zip = json['zip']
+        @country = json['country']
+        @capacity = json['capacity']
       end
     end
   end

data/lib/sports_data_api/nba.rb

@@ -5,8 +5,8 @@ module SportsDataApi
     class Exception < ::Exception
     end
 
-    API_VERSION = 3
-    BASE_URL = 'https://api.sportsdatallc.org/nba-%{access_level}%{version}'
+    API_VERSION = 4
+    BASE_URL = 'https://api.sportradar.us/nba/%{access_level}/v%{version}/en'
     DIR = File.join(File.dirname(__FILE__), 'nba')
     SPORT = :nba
 
@@ -16,6 +16,7 @@ module SportsDataApi
     autoload :Game, File.join(DIR, 'game')
     autoload :Games, File.join(DIR, 'games')
     autoload :Season, File.join(DIR, 'season')
+    autoload :Stats, File.join(DIR, 'stats')
     autoload :Venue, File.join(DIR, 'venue')
     autoload :Broadcast, File.join(DIR, 'broadcast')
 
@@ -26,31 +27,31 @@ module SportsDataApi
         season = season.to_s.upcase.to_sym
         raise Exception.new("#{season} is not a valid season") unless Season.valid?(season)
 
-        Season.new(response_xml_xpath("/games/#{year}/#{season}/schedule.xml", "/league/season-schedule"))
+        Season.new(response_json("/games/#{year}/#{season}/schedule.json"))
       end
 
       ##
       # Fetches NBA team roster
-      def team_roster(team)
-        Team.new(response_xml_xpath("/teams/#{team}/profile.xml", 'team'))
+      def team_roster(team_id)
+        Team.new(response_json("/teams/#{team_id}/profile.json"))
       end
 
       ##
       # Fetches NBA game summary for a given game
-      def game_summary(game)
-        Game.new(xml: response_xml_xpath("/games/#{game}/summary.xml", '/game'))
+      def game_summary(game_id)
+        Game.new(json: response_json("/games/#{game_id}/summary.json"))
       end
 
       ##
       # Fetches all NBA teams
       def teams
-        Teams.new(response_xml_xpath('/league/hierarchy.xml', '/league'))
+        Teams.new(response_json('/league/hierarchy.json'))
       end
 
       ##
       # Fetches NBA daily schedule for a given date
       def daily(year, month, day)
-        Games.new(response_xml_xpath("/games/#{year}/#{month}/#{day}/schedule.xml", 'league/daily-schedule'))
+        Games.new(response_json("/games/#{year}/#{month}/#{day}/schedule.json"))
       end
     end
   end

data/lib/sports_data_api/version.rb

@@ -1,3 +1,3 @@
 module SportsDataApi
-  VERSION = '0.11.6'
+  VERSION = '0.12.0'
 end