singleton-client-test 0.7.0.36 → 0.7.0.39

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bf77c54658953532871e1efcaeefdd2eeafe82d79d45de57341bc1f36faef8c7
-  data.tar.gz: '031790359ed9af74295e1679aae41a486da96104ae3a02421494c1a6a3593fd4'
+  metadata.gz: e774b278cf5b49efb41f194eeb40fab549670a57fbc0197600c272e71f15a795
+  data.tar.gz: e921a5d1ee1c3535597fca5991ca7c02b210ee731a7b7434f2bcd47cc958d7ef
 SHA512:
-  metadata.gz: 48113ab44d513f63789e7821340fc2de03e8f0c2a73063354ea7fc35b378b8d1f48b22ac3fc4d444c95a810e95a80cee02d8a44866153c82fa39300c1cf142ca
-  data.tar.gz: 05ba491346a1079a8d4b793d3d7d727aaec7ced066d7410df3b11e8a602700ba46a2fee5504649877db567ee5adaa492ab0bc0702925bad230bd6771b4aeb002
+  metadata.gz: 3d505a75cea1047514cf6f6bbcc666b26f136307f8bcbe6d2952524c557fd4156bfe29125f9688eb7c9ac8d04237ef344901f9526a5cf8faae39b32439ae3223
+  data.tar.gz: 0e4ce99bcdcb6ec25d177a8aefd53e28a4beee4f1fe11c4c9be203c4e33e4f66e91d5f790e886a4f542c396acf5fc1355f5ae96e90c22d5387a478f11b1dedbb

data/lib/sgtn-client/core/config.rb

@@ -3,7 +3,7 @@
 
 require 'erb'
 require 'yaml'
-
+require 'observer'
 
 module SgtnClient
   #include Exceptions
@@ -40,6 +40,7 @@ module SgtnClient
   
 
   class Config
+    extend Observable
 
     attr_accessor :username, :password, :signature, :app_id, :cert_path,
     :token, :token_secret, :subject,
@@ -182,6 +183,8 @@ module SgtnClient
             def bundles.locales
               @locales ||= reduce(Set.new) { |locales, id| locales << id.locale }
             end
+            changed
+            notify_observers(:available_locales)
           end
           bundles.locales
         end

data/lib/sgtn-client/loader/loader_factory.rb

@@ -22,7 +22,7 @@ module SgtnClient
         raise SgtnClient::SingletonError, 'no translation is available!' if loaders.empty?
 
         chain_loader = Class.new(Chain)
-        chain_loader.include SourceComparer if config['source_bundle'] || config['vip_server']
+        chain_loader.include SourceComparer
         chain_loader.include SingleLoader
         chain_loader.include Cache
 

data/lib/sgtn-client/loader/local_translation.rb

@@ -21,8 +21,6 @@ module SgtnClient
       end
 
       def load_bundle(component, locale)
-        return if locale == CONSTS::REAL_SOURCE_LOCALE # return when querying source
-
         SgtnClient.logger.debug "[#{method(__callee__).owner}.#{__callee__}] component=#{component}, locale=#{locale}"
 
         file_name = BUNDLE_PREFIX + locale + BUNDLE_SUFFIX

data/lib/sgtn-client/loader/server.rb

@@ -23,7 +23,7 @@ module SgtnClient
       def initialize(config)
         @server_url = config['vip_server']
 
-        product_root = format('/i18n/api/v2/translation/products/%s/versions/%s', config['product_name'], config['version'])
+        product_root = format('i18n/api/v2/translation/products/%s/versions/%s', config['product_name'], config['version'])
 
         @bundle_url = "#{product_root}/locales/%s/components/%s"
         @locales_url = "#{product_root}/localelist"

data/lib/sgtn-client/loader/single_loader.rb

@@ -12,12 +12,9 @@ module SgtnClient
       def load_bundle(component, locale)
         SgtnClient.logger.debug "[#{__FILE__}][#{__callee__}] component=#{component}, locale=#{locale}"
 
-        @single_bundle_loader ||= single_loader { |c,l| super(c,l) }
+        @single_bundle_loader ||= single_loader { |c, l| super(c, l) }
         id = CacheUtil.get_cachekey(component, locale)
         @single_bundle_loader.operate(id, component, locale)&.value
-      ensure
-        # delete thread from hash after finish
-        @single_bundle_loader.remove_object(id)
       end
 
       def available_bundles
@@ -25,21 +22,26 @@ module SgtnClient
 
         @single_available_bundles_loader ||= single_loader { super }
         @single_available_bundles_loader.operate(CONSTS::AVAILABLE_BUNDLES_KEY)&.value
-      ensure
-        @single_available_bundles_loader.remove_object(CONSTS::AVAILABLE_BUNDLES_KEY)
       end
 
       private
+
       def single_loader(&block)
-        none_alive = proc { |_, thread| thread.nil? || thread.alive? == false }
+        loader = nil
+        none_alive = proc { |_, thread| thread.nil? }
         creator = proc do |id, _, *args|
           Thread.new do
             SgtnClient.logger.debug "start single loading #{id}"
-            block.call(*args)
+            begin
+              block.call(*args)
+            ensure
+              # delete thread from hash after finish
+              loader.remove_object(id)
+            end
           end
         end
 
-        SgtnClient::SingleOperation.new(none_alive, &creator)
+        loader = SgtnClient::SingleOperation.new(none_alive, &creator)
       end
     end
   end

data/lib/sgtn-client/loader/source.rb

@@ -2,6 +2,7 @@
 # SPDX-License-Identifier: EPL-2.0
 
 require 'pathname'
+require 'yaml'
 
 module SgtnClient
   module Common
@@ -24,7 +25,7 @@ module SgtnClient
         total_messages = {}
 
         (@source_bundle_path + component).glob('**/*.{yml, yaml}') do |f|
-          bundle = YAML.safe_load(File.read(f))
+          bundle = YAML.load(File.read(f))
           messages = bundle&.first&.last # TODO: Warn about inconsistent source locale
           if messages.is_a?(Hash)
             total_messages.merge!(messages)

data/lib/sgtn-client/sgtn-client.rb

@@ -4,6 +4,9 @@
 module SgtnClient
       LOGFILE_SHIFT_AGE = 4
 
+      module Common
+        autoload :BundleID,        "sgtn-client/common/data"
+      end
       module Core
             autoload :Cache,        "sgtn-client/core/cache"
       end
@@ -18,6 +21,7 @@ module SgtnClient
       autoload :LocaleUtil,         "sgtn-client/util/locale-util"
       autoload :FileUtil,           "sgtn-client/util/file-util"
       autoload :CacheUtil,          "sgtn-client/util/cache-util"
+      autoload :StringUtil,          "sgtn-client/util/string-util"
 
       module Formatters
             autoload :PluralFormatter,    "sgtn-client/formatters/plurals/plural_formatter"

data/lib/sgtn-client/util/locale-util.rb

@@ -1,23 +1,29 @@
+# frozen_string_literal: true
+
 # Copyright 2022 VMware, Inc.
 # SPDX-License-Identifier: EPL-2.0
+
 require 'set'
 
 module SgtnClient
   class LocaleUtil
     MAP_LOCALES = {
-      'zh-CN' => 'zh-Hans',
-      'zh-TW' => 'zh-Hant',
-      'zh-Hans-CN' => 'zh-Hans',
-      'zh-Hant-TW' => 'zh-Hant'
+      'zh-cn' => 'zh-hans',
+      'zh-tw' => 'zh-hant',
+      'zh-hans-cn' => 'zh-hans',
+      'zh-hant-tw' => 'zh-hant'
     }.freeze
+    LOCALE_SEPARATOR = '-'
 
     def self.get_best_locale(locale)
+      return locale if Config.available_locales.include?(locale)
+
       return get_fallback_locale if locale.nil?
 
       locale = locale.to_s
       return get_fallback_locale if locale.empty?
 
-      get_best_match(locale.gsub('_', '-'))
+      get_best_match(locale.gsub('_', LOCALE_SEPARATOR).downcase)
     end
 
     def self.is_source_locale(locale = nil)
@@ -26,12 +32,12 @@ module SgtnClient
 
     def self.get_best_match(locale)
       locale = MAP_LOCALES[locale] || locale
-      return locale if Config.available_locales.include?(locale)
-
-      index = locale.rindex('-')
-      return get_fallback_locale if index.nil?
+      lowercase_locales_map[locale] or begin
+        index = locale.rindex(LOCALE_SEPARATOR)
+        return get_fallback_locale if index.nil?
 
-      get_best_match(locale[0...index])
+        get_best_match(locale[0...index])
+      end
     end
 
     def self.get_source_locale
@@ -47,6 +53,18 @@ module SgtnClient
       @fallback_locale ||= get_default_locale || get_source_locale || 'en'
     end
 
-    private_class_method :get_best_match
+    def self.lowercase_locales_map
+      @lowercase_locales_map ||= Config.available_locales.each_with_object({}) do |locale, memo|
+        memo[locale.to_s.downcase] = locale
+      end
+    end
+
+    def self.reset_available_locales(type)
+      @lowercase_locales_map = nil if type == :available_locales
+    end
+
+    SgtnClient::Config.add_observer(self, :reset_available_locales)
+
+    private_class_method :get_best_match, :lowercase_locales_map, :reset_available_locales
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: singleton-client-test
 version: !ruby/object:Gem::Version
-  version: 0.7.0.36
+  version: 0.7.0.39
 platform: ruby
 authors:
 - VMware G11n Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-05-11 00:00:00.000000000 Z
+date: 2022-06-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rdoc