rus_bank_rails 0.6.0 → 0.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e10b086891d2677a9f774153765e346ebe27e5a4
-  data.tar.gz: 7eb8b625990512223419b082cd8a19265e9b9c8e
+  metadata.gz: 388c4b80c1232868cefade5b644967341efc0e19
+  data.tar.gz: b84a79de60765b872080fbf1d18e7007621e57c4
 SHA512:
-  metadata.gz: b1a01a165bbb74703bae4a993c74422071acd6f08e498091b9ea034407d461c7124923ff073ace7450166282d1fd2fdc0a0b049b48ec8ffd30d523ff718df5dd
-  data.tar.gz: 5a69498173c29ca765a23261e7dc6920834520023a0a3a8180afcac24213d77b272ddc8d9d5d0364715faf62a933a99e0bef07d7745e48517272ccc7d56e2f49
+  metadata.gz: 39b730fa5565e59b27552fd74bb33f8271b3f105bf5c8c68057b088b76bdd352baa1eba4f7fbf641fa1de374c889a52874792af599f473df1ae758fe48bd8252
+  data.tar.gz: 193717f212ce3f7bc20618fb16dee2f4b3200244619328ad45b7961d821672583e30ecc373af5f2ce9d9825e99174c318bf3855f2ec9da2aa90369e444de733e

data/README.md

@@ -36,7 +36,7 @@ Rails-обертка вокруг гема rus_bank - https://github.com/wildDAl
     @bank = Bank.new
     internal_code = @bank.BicToIntCode(some_bank_bic)
 
-Далее можно кастомизировать модель по своему усмотрению. Например, в модели можно переопределить поведение тех или иных методов гема. Например, логика определения, требуется ли возвращать банк из базы или тянуть из API ЦБ, построена на том факте, что справочник БИК вступает в действие в 0:00 по московскому времени на всей территории страны. Т.е. если дата обновления записи(по MSK) в бд не текущие сутки, то обновляем ее. Можно поменять эту логику, переопределив метод expire? в app/models/bank.rb своего приложения:
+Далее можно кастомизировать модель по своему усмотрению. Например, в модели можно переопределить поведение тех или иных методов гема. Например, логика определения, требуется ли возвращать банк из базы или тянуть из API ЦБ, построена на том факте, что справочник БИК вступает в действие в 0:00 по московскому времени на всей территории страны. Т.е. если дата обновления записи(по MSK) в бд не текущие сутки, то обновляем ее. Можно поменять эту логику, переопределив метод `expire?` в `app/models/bank.rb` своего приложения:
 
      def expire?
          self.updated_at < 1.minute.ago   # Например, обновляем все, что старше минуты
@@ -44,6 +44,7 @@ Rails-обертка вокруг гема rus_bank - https://github.com/wildDAl
 
 Подобным образом переопределям любое поведение гема.
 
+Кроме того, следует иметь в виду, что часть методов при каждом вызове "дергает" внешнее API(о чем упомянуто в описании соответствующего метода) и при этом сохраняет результат в базу, что может сказаться на отклике вашего приложения. Например, `SearchByName("")` вернет все банки из API и при этом добавит/обновит их в базе, что крайне накладно(на каждый банк, что требуется обновить, вызов внешнего сервиса). Потому стоит использовать подобные методы с осторожностью, как вариант выполнять их отложенно.
 
 Описание доступных методов тут - http://rubydoc.info/gems/rus_bank_rails/RusBankRails/ActsAsBank/LocalInstanceMethods
 

data/lib/rus_bank_rails/acts_as_bank.rb

@@ -16,6 +16,7 @@ module RusBankRails
     end
 
     module LocalInstanceMethods
+
       ##
       # Метод возвращает внутренний номер банка по БИК
 
@@ -36,6 +37,7 @@ module RusBankRails
       # Возвращает внутренний номер по регистрационному номеру
 
       def RegNumToIntCode(reg_number)
+        # TODO: Подумать, стоит ли учет смены регистрационного номера столь лишнего кода. Данный подход основывается на данных с API ЦБ, что более точно. Но можно все упростить, если допустить, что регистрационный код никогда не меняется.
         bank = self.class.find_by_reg_number(reg_number.to_i)
         get_int_code_by_reg_number = lambda {
             cbr = RusBank.new
@@ -60,25 +62,11 @@ module RusBankRails
       # Возвращает регистрационный номер по внутреннему номеру
 
       def IntCodeToRegNum(internal_code)
-        bank = self.class.find_by_internal_code(internal_code.to_i)
-        get_reg_number = lambda {
-            cbr = RusBank.new
-            info = cbr.CreditInfoByIntCode(internal_code)
-            if info.nil?
-              return nil
-            else
-              return check_and_update(bic: info[:co][:bic]).reg_number
-            end
-        }
-        if bank.nil?
-          get_reg_number.call
+        bank = check_and_update(internal_code: internal_code)
+        if bank
+          bank.reg_number
         else
-          resp = check_and_update(bic: bank.bic)
-          if resp.internal_code == internal_code.to_i
-            resp.reg_number
-          else
-            get_reg_number.call
-          end
+          nil
         end
       end
 
@@ -87,17 +75,7 @@ module RusBankRails
 
       def SearchByName(bank_name)
         cbr = RusBank.new
-        result = cbr.SearchByName(bank_name)
-        if result
-          banks = []
-          result.each do |b|
-            bank = check_and_update(internal_code: b[:int_code])
-            banks << bank unless bank.nil?
-          end
-          banks
-        else
-          nil
-        end
+        get_updated_array( cbr.SearchByName(bank_name) )
       end
 
       ##
@@ -105,7 +83,7 @@ module RusBankRails
 
       def SearchByRegionCode(region_code)
         cbr = RusBank.new
-        cbr.SearchByRegionCode(region_code)
+        get_updated_array( cbr.SearchByRegionCode(region_code) )
       end
 
       ##
@@ -161,7 +139,7 @@ module RusBankRails
       end
 
       ##
-      # Возвращает true, если банк действующий
+      # Проверяет, действующий ли банк
 
       def is_active?
         (self.org_status != "лицензия отозвана") && (self.org_status != "ликвидирована") && !(self.get_licences_as_array_of_hashes.empty?)
@@ -223,6 +201,22 @@ module RusBankRails
         end
       end
 
+      ##
+      # Получает массив хэшей банков и возвращает массив соответствующих записей в базе данных
+
+      def get_updated_array(array_of_banks)
+        if array_of_banks
+          banks = []
+          array_of_banks.each do |b|
+            bank = check_and_update(internal_code: b[:int_code])
+            banks << bank unless bank.nil?
+          end
+          banks
+        else
+          nil
+        end
+      end
+
       ##
       # Метод возвращает актуальную информацию по банку с сайта ЦБР по БИК банка
 
@@ -232,15 +226,6 @@ module RusBankRails
           get_info_by_internal_code(internal_code) if internal_code
       end
 
-      ##
-      # Метод возвращает актуальную информацию по банку с сайта ЦБР по регистрационному номеру
-
-      def get_info_by_reg_number(reg_number)
-        cbr = RusBank.new
-        internal_code = cbr.RegNumToIntCode(reg_number)
-        get_info_by_internal_code(internal_code) if internal_code
-      end
-
       ##
       # Метод возвращает актуальную информацию по банку с сайта ЦБР по внутреннему коду
 
@@ -267,12 +252,6 @@ module RusBankRails
         end
       end
 
-      #def handle_exception(e)
-      #  puts "==========  ==========  =========="
-      #  puts e.inspect
-      #  puts "==========  ==========  =========="
-      #end
-
     end
 
   end

data/lib/rus_bank_rails/version.rb

@@ -1,3 +1,3 @@
 module RusBankRails
-  VERSION = "0.6.0"
+  VERSION = "0.7.1"
 end

data/test/dummy/log/development.log

@@ -6710,3 +6710,10 @@ Migrating to DeleteOldLicFromBanks (20140208123404)
    (0.1ms)  begin transaction
   SQL (0.3ms)  INSERT INTO "banks" ("created_at", "date_kgr_registration", "fact_adr", "internal_code", "licences", "org_full_name", "org_name", "org_status", "reg_code", "reg_number", "updated_at", "ustav_adr") VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)  [["created_at", Wed, 12 Feb 2014 13:07:07 UTC +00:00], ["date_kgr_registration", Mon, 17 Aug 1992], ["fact_adr", "Нет данных"], ["internal_code", 460001027], ["licences", "--- []\n"], ["org_full_name", "Акционерный коммерческий банк \"Московский купеческий банк\""], ["org_name", "МОСКОВСКИЙ КУПЕЧЕСКИЙ БАНК"], ["org_status", "ликвидирована"], ["reg_code", 17], ["reg_number", 2019], ["updated_at", Wed, 12 Feb 2014 13:07:07 UTC +00:00], ["ustav_adr", "103012, г.Москва, ул.Варварка, 8"]]
    (0.4ms)  commit transaction
+  Bank Load (0.2ms)  SELECT "banks".* FROM "banks" WHERE "banks"."internal_code" = 450000650 LIMIT 1
+   (0.1ms)  begin transaction
+  SQL (24.4ms)  INSERT INTO "banks" ("bic", "created_at", "date_kgr_registration", "fact_adr", "internal_code", "licences", "main_date_reg", "main_reg_number", "org_full_name", "org_name", "org_status", "phones", "reg_code", "reg_number", "ssv_date", "updated_at", "ust_money", "ustav_adr") VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)  [["bic", "044585216"], ["created_at", Thu, 13 Feb 2014 15:52:31 UTC +00:00], ["date_kgr_registration", Tue, 12 May 1992], ["fact_adr", "125040, г. Москва, ул. Правды, д. 8, корпус 1"], ["internal_code", 450000650], ["licences", "---\n- :l_code: !ruby/string:Nori::StringWithAttributes\n    str: !binary |-\n      Nw==\n    attributes: {}\n  :lt: !ruby/string:Nori::StringWithAttributes\n    str: !binary |-\n      0JPQtdC90LXRgNCw0LvRjNC90LDRjyDQu9C40YbQtdC90LfQuNGPINC90LAg\n      0L7RgdGD0YnQtdGB0YLQstC70LXQvdC40LUg0LHQsNC90LrQvtCy0YHQutC4\n      0YUg0L7Qv9C10YDQsNGG0LjQuQ==\n    attributes: {}\n  :l_date: !ruby/object:DateTime 2012-03-15 00:00:00.000000000 +04:00\n"], ["main_date_reg", Fri, 04 Oct 2002], ["main_reg_number", "1027700280937"], ["org_full_name", "Общество с ограниченной ответственностью \"Хоум Кредит энд Финанс Банк\""], ["org_name", "ХКФ БАНК"], ["org_status", "норм."], ["phones", "(495)785-82-25"], ["reg_code", 16], ["reg_number", 316], ["ssv_date", Wed, 10 Nov 2004], ["updated_at", Thu, 13 Feb 2014 15:52:31 UTC +00:00], ["ust_money", 4173000000], ["ustav_adr", "125040, г. Москва, ул. Правды, д. 8, корпус 1"]]
+   (0.4ms)  commit transaction
+  Bank Load (0.2ms)  SELECT "banks".* FROM "banks" WHERE "banks"."internal_code" = 450000650 LIMIT 1
+  Bank Load (0.2ms)  SELECT "banks".* FROM "banks" WHERE "banks"."internal_code" = 45000099650 LIMIT 1
+  Bank Load (0.2ms)  SELECT "banks".* FROM "banks" WHERE "banks"."internal_code" = 45000099 LIMIT 1