sportdb-config 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 55fee2eb8f0f1a3d97024defcabe6d749f592172
-  data.tar.gz: 4ceed1be167b9688f9e2f9c7b6e0fe02ab351078
+  metadata.gz: 6e1c949ddbf113fbfc92ac6b49c642efea51cf1e
+  data.tar.gz: 9bbcf4c6f00c08453267110fd79937999eabaed1
 SHA512:
-  metadata.gz: d495289d9ba9f1cfe4ac5a9cc19d8d916067be3a3101e07657a80640e9ec8aa21c46c5f28bad661dd3be9846d1b91548e74b94105384eded3cbdfb95093f3917
-  data.tar.gz: 44d015f48b75cfd062c94797405eaa064e3265c5dd155402b946aa8ab6c4a2eba13e20ea0b287ca69a6b06864fad89e91145e9e02d2bed11083dee59d2a9a1a7
+  metadata.gz: 76b550ec4e4b67425403284eb4cbb52888df3b9408ea56c6514f255509e03ba323610c38ae6caa41582b10279e50f8889fce69a855484ef070f58e79d20478fe
+  data.tar.gz: 5d2c9c90d4111df3e8d59381801e4db47fc6f3a5369f636fbf2248511debdfe0ac8eb5d47fe2bd2fbbd7397e9ac6af201ef6d03de408b7a16a6639c4a0c39f98

data/lib/sportdb/config/config.rb

@@ -78,6 +78,7 @@ class Configuration
     ## unify team names; team (builtin/known/shared) name mappings
     ## cleanup team names - use local ("native") name with umlaut etc.
     recs = []
+    errors = []
 
     ## todo/fix: pass along / use country code too
     CLUBS_DATAFILES.each do |country, path|
@@ -91,8 +92,14 @@ class Configuration
 
     recs.each do |rec|
        rec.alt_names.each do |alt_name|
-         ## todo/fix: warn about duplicates (if key exits) ???????
-         @team_mappings[ alt_name ] = rec.name
+         name = @team_mappings[ alt_name ]
+         if name  ## todo/fix: add better warn about duplicates (if key exits) ???????
+            msg = "** !!! WARN !!! - alt name conflict/duplicate - >#{alt_name}< will overwrite >#{name}< with >#{rec.name}<"
+            puts msg
+            errors << msg
+         else
+           @team_mappings[ alt_name ] = rec.name
+         end
        end
     end
 
@@ -109,6 +116,14 @@ class Configuration
       @teams[ rec.name ] = rec
     end
 
+    if errors.size > 0
+      puts ""
+      puts "!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!"
+      puts " #{errors.size} errors:"
+      pp errors
+      ## exit 1
+    end
+
     self  ## return self for chaining
 end
 

data/lib/sportdb/config/version.rb

@@ -8,7 +8,7 @@ module Boot      ## note: use a different module than Config to avoid confusion
 
   MAJOR = 0    ## todo: namespace inside version or something - why? why not??
   MINOR = 2
-  PATCH = 0
+  PATCH = 1
   VERSION = [MAJOR,MINOR,PATCH].join('.')
 
   def self.version

data/test/test_config.rb

@@ -10,7 +10,6 @@ require 'helper'
 class TestConfig < MiniTest::Test
 
   def test_teams
-
     SportDb::Import.config.clubs_dir = '../../../openfootball/clubs'
 
     pp SportDb::Import.config.teams

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sportdb-config
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Gerald Bauer