sportdb-sync 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 415893e89d2c9d1a8dfa9b493ecff8452095f94f
-  data.tar.gz: 9e55c19b40ee247d6b3b9e286b9dc799fe95c5d7
+  metadata.gz: 7c47d454d68dbeb13a19af73d5ea67e82093a7f7
+  data.tar.gz: b6de1499660943410dd9f8c293d5ac3a1d99c0dd
 SHA512:
-  metadata.gz: 4e4487b6d40e296e9de680f99cb331693ccff0aac5dc3d73a55cae418f6ba9410966da1542bfa090fefa3e60123ab5822745041ab132d0876e7d89fa44ae7da7
-  data.tar.gz: b59e3712eb51503aee9cc7c56c604fd8c5b0513a75995b7ceefe383f67820cd25ab17f97bd32f1ca6e0dfdacab49f4333eaa5b6af82102198714f7429d0b9ea9
+  metadata.gz: cb7827d0b7fe557ae9988ce6145db9f28ad0a62ec4773c0c8e55558b5b26a007f3d487fee966252ad819182eb2caa228897b41b198eecf57eae424a86e11e4e4
+  data.tar.gz: a3821524471837e9f7d6fc4a4016d39abd733aaa0f0c9036932d2edfc27f810b8fb55d3a1364e070808c05ddc929cfe052d1a1bba9fd899799f8cea4a6adaafb

data/Rakefile

@@ -21,7 +21,7 @@ Hoe.spec 'sportdb-sync' do
 
   self.extra_deps = [
     ['sportdb-config',  '>= 1.0.4'],
-    ['sportdb-models',  '>= 1.19.0'],
+    ['sportdb-models',  '>= 2.0.0'],
   ]
 
   self.spec_extras = {

data/lib/sportdb/sync/club.rb

@@ -50,8 +50,8 @@ module SportDb
       #  finders
 
       def self.find_or_create( club )
-        ## note: assume "canonical uniquie" names/titles for now for clubs
-        rec = Model::Team.find_by( title: club.name )
+        ## note: assume "canonical uniquie" names for now for clubs
+        rec = Model::Team.find_by( name: club.name )
         if rec.nil?
 
           ## todo/fix:  move auto-key gen to structs for re(use)!!!!!!
@@ -61,7 +61,7 @@ module SportDb
 
           attribs = {
               key:        key,
-              title:      club.name,
+              name:       club.name,
               country_id: Sync::Country.find_or_create( club.country ).id,
               club:       true,
               national:   false  ## check -is default anyway - use - why? why not?
@@ -71,7 +71,7 @@ module SportDb
           attribs[:code] = club.code   if club.code   ## add code (abbreviation) if present
 
           if club.alt_names.empty? == false
-            attribs[:synonyms] = club.alt_names.join('|')
+            attribs[:alt_names] = club.alt_names.join('|')
           end
 
           rec = Model::Team.create!( attribs )

data/lib/sportdb/sync/event.rb

@@ -79,8 +79,8 @@ module SportDb
           ##                               otherwise use 2017, 7, 1
           ##  start_at use year and 7,1 e.g. Date.new( 2017, 7, 1 )
           ## hack:  fix/todo1!!
-          ##   add "fake" start_at date for now
-          attribs[:start_at]  = if season.year?  ## e.g. assume 2018 etc.
+          ##   add "fake" start_date for now
+          attribs[:start_date]  = if season.year?  ## e.g. assume 2018 etc.
                                    Date.new( season.start_year, 1, 1 )
                                 else  ## assume 2014/15 etc.
                                    Date.new( season.start_year, 7, 1 )

data/lib/sportdb/sync/league.rb

@@ -40,11 +40,8 @@ module SportDb
     def self.find_or_create( league )
       rec = find( league )
       if rec.nil?
-         ## use title and not name - why? why not?
-         ##  quick fix:  change name to title
-
          attribs = { key:   league.key,
-                     title: league.name }
+                     name:  league.name }
 
         if league.country
            attribs[ :country_id ] = Sync::Country.find_or_create( league.country ).id

data/lib/sportdb/sync/season.rb

@@ -43,7 +43,7 @@ module SportDb
         rec = find( season )
         if rec.nil?
            attribs = { key:   season.key,
-                       title: season.name  }
+                       name:  season.name  }
            rec = Model::Season.create!( attribs )
         end
         rec

data/lib/sportdb/sync/sync.rb

@@ -6,14 +6,14 @@ module SportDb
 
   class NationalTeam
     def self.find_or_create( team )
-      rec = Model::Team.find_by( title: team.name )
+      rec = Model::Team.find_by( name: team.name )
       if rec.nil?
         puts "add national team: #{team.key}, #{team.name}, #{team.country.name} (#{team.country.key})"
 
         ### note: key expected three or more lowercase letters a-z /\A[a-z]{3,}\z/
         attribs = {
           key:        team.key,   ## note: always use downcase fifa code for now!!!
-          title:      team.name,
+          name:       team.name,
           code:       team.code,
           country_id: Sync::Country.find_or_create( team.country ).id,
           club:       false,
@@ -21,7 +21,7 @@ module SportDb
         }
 
         if team.alt_names.empty? == false
-          attribs[:synonyms] = team.alt_names.join('|')
+          attribs[:alt_names] = team.alt_names.join('|')
         end
 
         rec = Model::Team.create!( attribs )
@@ -64,19 +64,19 @@ module SportDb
 
   class Round
     def self.find_or_create( round, event: )
-       rec = Model::Round.find_by( title: round.title, event_id: event.id )
+       rec = Model::Round.find_by( name: round.name, event_id: event.id )
        if rec.nil?
          ## find last pos - check if it can be nil?
          max_pos = Model::Round.where( event_id: event.id ).maximum( 'pos' )
          max_pos = max_pos ? max_pos+1 : 1
 
          attribs = { event_id: event.id,
-                     title:    round.title,
+                     name:     round.name,
                      pos:      max_pos
                    }
 
          ## todo/fix:  check if round has (optional) start or end date and add!!!
-         ## attribs[ :start_at] = round.start_at.to_date
+         ## attribs[ :start_date] = round.start_date.to_date
 
          rec = Model::Round.create!( attribs )
        end
@@ -87,14 +87,14 @@ module SportDb
 
   class Group
     def self.find_or_create( group, event: )
-       rec = Model::Group.find_by( title: group.title, event_id: event.id )
+       rec = Model::Group.find_by( name: group.name, event_id: event.id )
        if rec.nil?
          ## find last pos - check if it can be nil?
          max_pos = Model::Group.where( event_id: event.id ).maximum( 'pos' )
          max_pos = max_pos ? max_pos+1 : 1
 
          attribs = { event_id: event.id,
-                     title:    group.title,
+                     name:     group.name,
                      pos:      max_pos
                    }
 
@@ -109,7 +109,7 @@ module SportDb
 
   class Stage
     def self.find( name, event: )
-      Model::Stage.find_by( title: name, event_id: event.id  )
+      Model::Stage.find_by( name: name, event_id: event.id  )
     end
     def self.find!( name, event: )
       rec = find( name, event: event  )
@@ -125,10 +125,8 @@ module SportDb
     def self.find_or_create( name, event: )
        rec = find( name, event: event )
        if rec.nil?
-         ## use title and not name - why? why not?
-         ##  quick fix:  change name to title
          attribs = { event_id: event.id,
-                     title:    name,
+                     name:     name,
                    }
          rec = Model::Stage.create!( attribs )
        end
@@ -138,21 +136,21 @@ module SportDb
 
 
 
-  class Match   ## todo/check:  add alias for Game class - why? why not?
+  class Match
     def self.create_or_update( match, event: )
        ## note: MUST find round, thus, use bang (!)
 
        ## todo/check: allow strings too - why? why not?
 
        ## query for round - allow string or round rec
-       round_title = match.round.is_a?( String ) ? match.round : match.round.title
+       round_name  = match.round.is_a?( String ) ? match.round : match.round.name
        round_rec   = Model::Round.find_by!( event_id: event.id,
-                                            title:    round_title )
+                                            name:     round_name )
 
        ## todo/check: allow fallback with db lookup if NOT found in cache - why? why not?
        ##  or better use Sync::Team.find_or_create( team )  !!!!!!! to auto-create on first hit!
        ##    || Team.find_or_create( team1 )  -- note: does NOT work for string (only recs) - what to do?
-       ##    || Model::Team.find_by!( title: team1_name )
+       ##    || Model::Team.find_by!( name: team1_name )
        team1_name   = match.team1.is_a?( String ) ? match.team1 : match.team1.name
        team1_rec    = Team.cache[ team1_name ]
        team2_name   = match.team2.is_a?( String ) ? match.team2 : match.team2.name
@@ -160,20 +158,20 @@ module SportDb
 
        ## check optional group (e.g. Group A, etc.)
        group_rec = if match.group
-                     group_title = match.group.is_a?( String ) ? match.group : match.group.title
+                     group_name = match.group.is_a?( String ) ? match.group : match.group.name
                      Model::Group.find_by!( event_id: event.id,
-                                            title:    group_title )
+                                            name:     group_name )
                    else
                      nil
                    end
 
 
-       rec = Model::Game.find_by( round_id: round_rec.id,
+       rec = Model::Match.find_by( round_id: round_rec.id,
                                   team1_id: team1_rec.id,
                                   team2_id: team2_rec.id )
        if rec.nil?
         ## find last pos - check if it can be nil?
-         max_pos = Model::Game.where( event_id: event.id ).maximum( 'pos' )
+         max_pos = Model::Match.where( event_id: event.id ).maximum( 'pos' )
          max_pos = max_pos ? max_pos+1 : 1
 
          attribs = { event_id: event.id,          ## todo/fix: change to data struct too?
@@ -181,14 +179,14 @@ module SportDb
                      team1_id: team1_rec.id,
                      team2_id: team2_rec.id,
                      pos:      max_pos,
-                     play_at:  match.date.to_date,
+                     date:     match.date.to_date,  ## todo/fix: split and add date & time!!!!
                      score1:   match.score1,
                      score2:   match.score2,
                      score1i:  match.score1i,
                      score2i:  match.score2i }
          attribs[ :group_id ] = group_rec.id   if group_rec
 
-         rec = Model::Game.create!( attribs )
+         rec = Model::Match.create!( attribs )
        else
          # update - todo
        end

data/lib/sportdb/sync/version.rb

@@ -7,7 +7,7 @@ module Sync
 
   MAJOR = 1    ## todo: namespace inside version or something - why? why not??
   MINOR = 0
-  PATCH = 2
+  PATCH = 3
   VERSION = [MAJOR,MINOR,PATCH].join('.')
 
   def self.version

data/test/test_league.rb

@@ -14,25 +14,25 @@ class TestLeague < MiniTest::Test
 
   def test_search
     rec = League.search_or_create!( 'eng' )
-    assert_equal 'Premier League',  rec.title    ## fix: add name alias (rename title to name!!)
+    assert_equal 'Premier League',  rec.name
     assert_equal 'eng.1',           rec.key
     assert_equal 'eng',             rec.country.key
     assert_equal 'England',         rec.country.name
-    # assert_equal true,              rec.club
+    # assert_equal true,              rec.clubs
 
     rec = League.search!( 'eng' )
-    assert_equal 'Premier League',  rec.title    ## fix: add name alias (rename title to name!!)
+    assert_equal 'Premier League',  rec.name
     assert_equal 'eng.1',           rec.key
     assert_equal 'eng',             rec.country.key
     assert_equal 'England',         rec.country.name
-    ## assert_equal true,              rec.club
+    ## assert_equal true,              rec.clubs
 
     ## try 2nd call (just lookup)
     rec = League.search_or_create!( 'eng' )
-    assert_equal 'Premier League', rec.title    ## fix: add name alias (rename title to name!!)
+    assert_equal 'Premier League', rec.name
     assert_equal 'eng.1',          rec.key
     assert_equal 'eng',            rec.country.key
     assert_equal 'England',        rec.country.name
-    ## assert_equal true,                 rec.club
+    ## assert_equal true,                 rec.clubs
   end
 end # class TestLeague

data/test/test_misc.rb

@@ -14,15 +14,15 @@ class TestMisc < MiniTest::Test
 
   def test_season
     rec = Season.search_or_create( '2017-18' )
-    assert_equal '2017/18', rec.title    ## fix: add name alias (rename title to name!!)
+    assert_equal '2017/18', rec.name
     assert_equal '2017/18', rec.key
 
     rec = Season.search_or_create( '2017/2018' )
-    assert_equal '2017/18', rec.title    ## fix: add name alias (rename title to name!!)
+    assert_equal '2017/18', rec.name
     assert_equal '2017/18', rec.key
 
     rec = Season.search_or_create( '2017/8' )
-    assert_equal '2017/18', rec.title    ## fix: add name alias (rename title to name!!)
+    assert_equal '2017/18', rec.name
     assert_equal '2017/18', rec.key
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sportdb-sync
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Gerald Bauer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-23 00:00:00.000000000 Z
+date: 2020-05-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sportdb-config
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.19.0
+        version: 2.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.19.0
+        version: 2.0.0
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement