sports_data_api 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4d8be1761997f03cc4d7aa1b666c29982fbbb2e2
-  data.tar.gz: ae0d4e0bee91e4549235ad79c196fccead66a429
+  metadata.gz: 62aab7aee23b0a730d1134d17519b149c0094327
+  data.tar.gz: f95759783b3b4d34a87f79db7e6e5be0d0ed414c
 SHA512:
-  metadata.gz: 1bfc851b3033f99db197871765983dc185aeddba7636e284efb150c3eb3af6eeb83b5000b08a57e3d4bbde38abeec0b3195029646ceced18fd3d307226240d04
-  data.tar.gz: 255550917ad90c562706c64d36d9a01344ce37657db4afeb41e6e30ca045947bd620ef12e625872dcd70512ca0465b573b83f13a25fb9a7cddab31444d2433af
+  metadata.gz: 9152043f7a7d85ef9245dd0466f83d7f970d23c60a3a0ecb2007edda75623f01b3b7ec4781d2e16fd38aa010dbd90f1b7d6ba898d3148d8bd393ad8d018512b5
+  data.tar.gz: a2f99223b3092b2639a04ad843fd12e60f55af1388a3d937e550922e49bc6ae24f7457c9396c5d72ab35359815bc6452b4d73da1e57e59f742623a84793c3939

data/.ruby-version

@@ -1 +1 @@
-ruby-2.1.2
+ruby-2.1

data/.travis.yml

@@ -3,3 +3,4 @@ rvm:
   - 1.9.3
   - 2.0.0
   - 2.1.0
+  - ruby-head

data/README.md

@@ -1,4 +1,4 @@
-# SportsDataApi [![Build Status](https://travis-ci.org/RLovelett/sports_data_api.png?branch=master)](https://travis-ci.org/RLovelett/sports_data_api) [![Coverage Status](https://coveralls.io/repos/RLovelett/sports_data_api/badge.png?branch=master)](https://coveralls.io/r/RLovelett/sports_data_api?branch=master)
+# SportsDataApi [![Build Status](https://travis-ci.org/RLovelett/sports_data_api.png?branch=master)](https://travis-ci.org/RLovelett/sports_data_api) [![Coverage Status](https://coveralls.io/repos/RLovelett/sports_data_api/badge.png?branch=master)](https://coveralls.io/r/RLovelett/sports_data_api?branch=master) [![Gem Version](https://badge.fury.io/rb/sports_data_api.svg)](http://badge.fury.io/rb/sports_data_api) [![Code Climate](https://codeclimate.com/github/RLovelett/sports_data_api/badges/gpa.svg)](https://codeclimate.com/github/RLovelett/sports_data_api)
 
 SportsDataApi is an attempt to make a Ruby interface to the
 [SportsData](http://www.sportsdatallc.com/) API. The goal is to
@@ -22,11 +22,11 @@ log](https://github.com/rlovelett/sports_data_api/issues).
 
   * [NFL](http://developer.sportsdatallc.com/docs/NFL_API)
   * [NBA](http://developer.sportsdatallc.com/docs/NBA_API)
+  * [MLB](http://developer.sportsdatallc.com/docs/MLB_API)
+  * [NHL](http://developer.sportsdatallc.com/docs/NHL_API)
   * TODO
-    1. [MLB](http://developer.sportsdatallc.com/docs/MLB_API)
-    2. [NHL](http://developer.sportsdatallc.com/docs/NHL_API)
-    3. [NCAA Basketball](http://developer.sportsdatallc.com/docs/NCAA_Mens_Basketball)
-    4. [NCAA Football](http://developer.sportsdatallc.com/docs/NCAA_Football_API)
+    2. [NCAA Basketball](http://developer.sportsdatallc.com/docs/NCAA_Mens_Basketball)
+    3. [NCAA Football](http://developer.sportsdatallc.com/docs/NCAA_Football_API)
 
 ## Installation
 
@@ -61,7 +61,7 @@ However, if you want to refresh the actual server API responses you will need to
 This can be achieved simply by performing the following two steps:
 
 1. Delete all the cassettes (`rm spec/cassettes/*.yml`)
-2. Run specs passing the API key as environment variable (`SPORTS_DATA_<NBA|NFL>_API_KEY=realapikey bundle exec rake spec`)
+2. Run specs passing the API key as environment variable (`SPORTS_DATA_<NFL|NBA|MLB|NHL>_API_KEY=realapikey bundle exec rake spec`)
 
 ## Contributing
 

data/lib/sports_data_api.rb

@@ -2,6 +2,7 @@ require "sports_data_api/version"
 require "nokogiri"
 require "rest_client"
 require "time"
+require "multi_json"
 
 module SportsDataApi
   def self.key(sport)
@@ -49,5 +50,6 @@ module SportsDataApi
   autoload :Nfl,         File.join(LIBRARY_PATH, 'nfl')
   autoload :Nba,         File.join(LIBRARY_PATH, 'nba')
   autoload :Mlb,         File.join(LIBRARY_PATH, 'mlb')
+  autoload :Nhl,         File.join(LIBRARY_PATH, 'nhl')
   autoload :Exception,   File.join(LIBRARY_PATH, 'exception')
 end

data/lib/sports_data_api/nfl.rb

@@ -29,33 +29,33 @@ module SportsDataApi
       season = season.to_s.upcase.to_sym
       raise SportsDataApi::Nfl::Exception.new("#{season} is not a valid season") unless Season.valid?(season)
 
-      response = self.response_xml(version, "/#{year}/#{season}/schedule.xml")
+      response = self.response_json(version, "/#{year}/#{season}/schedule.json")
 
-      return Season.new(response.xpath("/season"))
+      return Season.new(response)
     end
 
     ##
     # Fetch NFL team roster
     def self.team_roster(team, version = DEFAULT_VERSION)
-      response = self.response_xml(version, "/teams/#{team}/roster.xml")
+      response = self.response_json(version, "/teams/#{team}/roster.json")
 
-      return TeamRoster.new(response.xpath("team"))
+      return TeamRoster.new(response)
     end
 
     ##
     # Fetch NFL team seaon stats for a given team, season and season type
     def self.team_season_stats(team, season, season_type, version = DEFAULT_VERSION)
-      response = self.response_xml(version, "/teams/#{team}/#{season}/#{season_type}/statistics.xml")
+      response = self.response_json(version, "/teams/#{team}/#{season}/#{season_type}/statistics.json")
 
-      return TeamSeasonStats.new(response.xpath("/season").xpath("team"))
+      return TeamSeasonStats.new(response)
     end
 
     ##
     # Fetch NFL player seaon stats for a given team, season and season type
     def self.player_season_stats(team, season, season_type, version = DEFAULT_VERSION)
-      response = self.response_xml(version, "/teams/#{team}/#{season}/#{season_type}/statistics.xml")
+      response = self.response_json(version, "/teams/#{team}/#{season}/#{season_type}/statistics.json")
 
-      return PlayerSeasonStats.new(response.xpath("/season").xpath("team").xpath("players"))
+      return PlayerSeasonStats.new(response)
     end
 
     ##
@@ -64,17 +64,28 @@ module SportsDataApi
       season = season.to_s.upcase.to_sym
       raise SportsDataApi::Nfl::Exception.new("#{season} is not a valid season") unless Season.valid?(season)
 
-      response = self.response_xml(version, "/#{year}/#{season}/#{week}/#{away}/#{home}/boxscore.xml")
+      response = self.response_json(version, "/#{year}/#{season}/#{week}/#{away}/#{home}/boxscore.json")
 
-      return Game.new(year, season, week, response.xpath("/game"))
+      return Game.new(year, season, week, response)
+    end
+
+    ##
+    # Fetches statistics for a given NFL game
+    def self.game_statistics(year, season, week, home, away, version = DEFAULT_VERSION)
+      season = season.to_s.upcase.to_sym
+      raise SportsDataApi::Nfl::Exception.new("#{season} is not a valid season") unless Season.valid?(season)
+
+      response = self.response_json(version, "/#{year}/#{season}/#{week}/#{away}/#{home}/statistics.json")
+
+      return Game.new(year, season, week, response)
     end
 
     ##
     # Fetches all NFL teams
     def self.teams(version = DEFAULT_VERSION)
-      response = self.response_xml(version, "/teams/hierarchy.xml")
+      response = self.response_json(version, "/teams/hierarchy.json")
 
-      return Teams.new(response.xpath('/league'))
+      return Teams.new(response)
     end
 
     ##
@@ -83,16 +94,17 @@ module SportsDataApi
       season = season.to_s.upcase.to_sym
       raise SportsDataApi::Nfl::Exception.new("#{season} is not a valid season") unless Season.valid?(season)
 
-      response = self.response_xml(version, "/#{year}/#{season}/#{week}/schedule.xml")
+      response = self.response_json(version, "/#{year}/#{season}/#{week}/schedule.json")
 
-      return Games.new(response.xpath('/games'))
+      return Games.new(year, season, week, response)
     end
 
     private
-    def self.response_xml(version, url)
+
+    def self.response_json(version, url)
       base_url = BASE_URL % { access_level: SportsDataApi.access_level(SPORT), version: version }
       response = SportsDataApi.generic_request("#{base_url}#{url}", SPORT)
-      Nokogiri::XML(response.to_s).remove_namespaces!
+      MultiJson.load(response.to_s)
     end
   end
 end

data/lib/sports_data_api/nfl/broadcast.rb

@@ -2,14 +2,13 @@ module SportsDataApi
   module Nfl
     class Broadcast
       attr_reader :network, :satellite, :internet
-      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']
-          @internet = xml['internet']
+      def initialize(broadcast_hash)
+        if broadcast_hash
+          @network = broadcast_hash['network']
+          @satellite = broadcast_hash['satellite'] || ''
+          @internet = broadcast_hash['internet']
         end
       end
     end
   end
-end
+end

data/lib/sports_data_api/nfl/game.rb

@@ -5,35 +5,30 @@ module SportsDataApi
         :away_team, :status, :quarter, :clock, :venue, :broadcast, :weather,
         :year, :season, :week
 
-      def initialize(year, season, week, xml)
-        @year = year.to_i
-        @season = season.to_sym
-        @week = week.to_i
+      def initialize(year, season, week, game_hash)
+        @year = year
+        @season = season
+        @week = week
 
-        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']
-          @away = xml['away']
-          @status = xml['status']
-          @quarter = xml['quarter'].to_i
-          @clock = xml['clock']
+        @id = game_hash['id']
+        @scheduled = Time.parse game_hash['scheduled']
+        @home = game_hash['home'] || game_hash['home_team']['id']
+        @away = game_hash['away'] || game_hash['away_team']['id']
+        @status = game_hash['status']
+        @quarter = game_hash['quarter'].to_i
+        @clock = game_hash['clock']
 
-          team_xml = xml.xpath('team')
-          @home_team = Team.new(team_xml.first)
-          @away_team = Team.new(team_xml.last)
-          @venue = Venue.new(xml.xpath('venue'))
-          @broadcast = Broadcast.new(xml.xpath('broadcast'))
-          @weather = Weather.new(xml.xpath('weather'))
-        end
+        @home_team = Team.new(game_hash['home_team'])
+        @away_team = Team.new(game_hash['away_team'])
+        @venue = Venue.new(game_hash['venue'])
+        @broadcast = Broadcast.new(game_hash['broadcast'])
+        @weather = Weather.new(game_hash['weather'])
       end
 
       ##
       # Wrapper for Nfl.statistics
-      # TODO
       def statistics
-        raise NotImplementedError
+        Nfl.game_statistics(year, season, week, home, away, 1)
       end
 
       ##

data/lib/sports_data_api/nfl/games.rb

@@ -4,13 +4,13 @@ module SportsDataApi
       include Enumerable
       attr_reader :games, :year, :season, :week
 
-      def initialize(xml)
-        @year = xml.first['season'].to_i
-        @season = xml.first['type'].to_sym
-        @week = xml.first['week'].to_i
+      def initialize(year, season, week, games_hash)
+        @year = year
+        @season = season
+        @week = week
 
-        @games = xml.xpath("game").map do |game_xml|
-          Game.new(@year, @season, @week, game_xml)
+        @games = games_hash['games'].map do |game_hash|
+          Game.new(@year, @season, @week, game_hash)
         end
       end
 
@@ -23,7 +23,6 @@ module SportsDataApi
           end
         end
       end
-
     end
   end
-end
+end

data/lib/sports_data_api/nfl/player.rb

@@ -1,13 +1,11 @@
 module SportsDataApi
   module Nfl
     class Player
-      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
+      def initialize(json)
+        player_ivar = self.instance_variable_set("@player", {})
+        self.class.class_eval { attr_reader :"player" }
+        json.each_pair do | attr_name, attr_value|
+          player_ivar[attr_name.to_sym] = attr_value
         end
       end
     end

data/lib/sports_data_api/nfl/player_season_stats.rb

@@ -2,22 +2,12 @@ module SportsDataApi
   module Nfl
     class PlayerSeasonStats
       attr_reader :stats, :id, :players
-      def initialize(xml)
-        xml = xml.first if xml.is_a? Nokogiri::XML::NodeSet
+      def initialize(json)
         @players = []
-        xml.xpath("player").each do | p |
+        json['players'].each do | p |
           player = {}
-          stats = []
-
-          p.children.each do | stat |
-            if stat.name == "text"
-              next
-            end
-            stats << SportsDataApi::Stats.new(stat)
-          end
-
           player[:id] = p['id']
-          player[:stats] = stats
+          player[:stats] = p['statistics']
           @players << player
         end
       end

data/lib/sports_data_api/nfl/season.rb

@@ -3,14 +3,12 @@ module SportsDataApi
     class Season
       attr_reader :year, :type, :weeks
 
-      def initialize(xml)
+      def initialize(season_hash)
         @weeks = []
-        if xml.is_a? Nokogiri::XML::NodeSet
-          @year = xml.first["season"].to_i
-          @type = xml.first["type"].to_sym
-          @weeks = xml.first.xpath("week").map do |week_xml|
-            Week.new(@year, @type, week_xml)
-          end
+        @year = season_hash['season']
+        @type = season_hash['type'].to_sym
+        @weeks = season_hash['weeks'].map do |week_hash|
+          Week.new(@year, @type, week_hash)
         end
       end
 

data/lib/sports_data_api/nfl/team.rb

@@ -1,22 +1,36 @@
 module SportsDataApi
   module Nfl
     class Team
-      attr_reader :id, :name, :conference, :division, :market, :remaining_challenges, :remaining_timeouts, :score, :quarters
+      attr_reader :id, :name, :conference, :division, :market, :remaining_challenges,
+                  :remaining_timeouts, :score, :quarters, :venue, :points, :statistics, :players
 
-      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']
+      def initialize(team_hash, conference = nil, division = nil)
+        if team_hash
+          @id = team_hash['id']
+          @name = team_hash['name']
           @conference = conference
           @division = division
-          @market = xml['market']
-          @remaining_challenges = xml['remaining_challenges'].to_i
-          @remaining_timeouts = xml['remaining_timeouts'].to_i
-          @quarters = xml.xpath('scoring/quarter').map do |quarter|
-            quarter['points'].to_i
+          @market = team_hash['market']
+          @remaining_challenges = team_hash['remaining_challenges']
+          @remaining_timeouts = team_hash['remaining_timeouts']
+          @quarters = []
+          if team_hash['scoring']
+            team_hash['scoring'].each do |scoring_hash|
+              @quarters[scoring_hash['quarter']-1] = scoring_hash['points']
+            end
           end
           @quarters = @quarters.fill(0, @quarters.size, 4 - @quarters.size)
+
+          # Parse the Venue data if it exists
+          if team_hash.key?('venue')
+            @venue = Venue.new(team_hash['venue'])
+          end
+
+          if team_hash['statistics']
+            @statistics = parse_team_statistics(team_hash['statistics'])
+            @players = parse_player_statistics(team_hash['statistics'])
+          end
+          @points = team_hash['points'] || score
         end
       end
 
@@ -41,6 +55,31 @@ module SportsDataApi
         end
       end
 
+      private
+
+      def parse_team_statistics(stats_hash)
+        stats = {}
+        stats_hash.keys.each do |key|
+          stats[key] = stats_hash[key]['team']
+        end
+        stats
+      end
+
+      def parse_player_statistics(stats_hash)
+        players = []
+        stats_hash.keys.each do |key|
+          next if !stats_hash[key]['players']
+          stats_hash[key]['players'].each do |p|
+            player = players.select{|a| a['id'] == p['id']}.first || {}
+            players << player if !players.select{|a| a['id'] == p['id']}.first
+            ['id', 'name', 'jersey', 'position'].each do |k|
+              player[k] = p.delete(k)
+            end
+            player[key] = p
+          end
+        end
+        players
+      end
     end
   end
 end

data/lib/sports_data_api/nfl/team_roster.rb

@@ -3,12 +3,9 @@ module SportsDataApi
     class TeamRoster
       attr_reader :players
 
-      def initialize(xml)
+      def initialize(json)
         @players = []
-        xml.children.each do | player |
-          if player.name == "text"
-            next
-          end
+        json['players'].each do | player |
           @players << Player.new(player)
         end
       end

data/lib/sports_data_api/nfl/team_season_stats.rb

@@ -2,16 +2,11 @@ module SportsDataApi
   module Nfl
     class TeamSeasonStats
       attr_reader :id, :stats
-      def initialize(xml)
-        xml = xml.first if xml.is_a? Nokogiri::XML::NodeSet
-        @id = xml['id']
+      def initialize(json)
+        @id = json['id']
         @stats = []
-        xml.xpath("players").remove
-        xml.children.each do | stat |
-          if stat.name == "text"
-            next
-          end
-          @stats << SportsDataApi::Stats.new(stat)
+        json['statistics'].each_pair do | key, val |
+          @stats << { key: val }
         end
       end
     end