rhack 1.3.3 → 1.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 40e4929e52203b0121c9ca2eac2f289f656ff543
-  data.tar.gz: abb0fe53f2f8e43836473fdc8d7ae517fcd63602
+  metadata.gz: cd5c5d3b1ed427ecf3c7db8b0763325661c1c58d
+  data.tar.gz: 4c443acc77cdbf2259cee3b257875a709b4ab9bf
 SHA512:
-  metadata.gz: 23e4c2b43ee95212d0c8069ed7e2c731dc1878ec4bd4f7b8984bb0dfea55f1cd496ad161c098d2a0551a49d995a6cf4faca8b0041fc891d9f3fc3b42bfce5243
-  data.tar.gz: b6e3fce288abf2aaa51cf7910b1b99c90e0c673f13bf6541e8a047b496d83fe6d90b5d11379f8d8e6610a9ad94c6d8834d58c9b0c8cfa2fd171d43852791d987
+  metadata.gz: 3a0bba0ca3414970de52d94bb2418010d689f0910226acca375c4b7be1bd2dc495fcb29808cc96dc2039a0319b5c66675b86cd6bb941d266dced153157433a91
+  data.tar.gz: 8a7cd7e1b7c71fbaaab65b79891479553c8e2b5a7f3b3fc54f88ce50a30b95db8dc8c832d3f9200b87d45a9a229b0c8f49df00cf24ff3e6391d24683283dc10a

data/lib/rhack/curl/response.rb

@@ -8,7 +8,7 @@ module Curl
     def to_s
       str = '<#'
       if @error
-        str << "#{@error.name}: #{@error.message}"
+        str << "#{@error.class.name}: #{@error.message}"
       else
         str << (@header[/\d{3}/] == @code.to_s ? @header : "#{@header[/\S+/]} #{@code}") if @header
         if @hash.location

data/lib/rhack/frame.rb

@@ -90,7 +90,7 @@ module RHACK
     
     def inspect
       sssize = @ss.size
-      "<#Frame @ #{@ss.untargeted ? 'no target' : @loc.root}: #{sssize} #{sssize == 1 ? 'scout' : 'scouts'}#{', static'+(' => '+@static.protocol if @static.is(Hash)) if @static}, cookies #{@ss[0].cookie_enabled ? 'on' : 'off'}>"
+      "<#Frame @ #{@ss.untargeted ? 'no target' : @loc.root}: #{sssize} #{sssize == 1 ? 'scout' : 'scouts'}#{', static'+(' => '+@static.protocol if @static.is(Hash)) if @static}, cookies #{@ss[0].cookies_enabled ? 'on' : 'off'}>"
     end
     
     # All opts going in one hash.
@@ -103,23 +103,45 @@ module RHACK
     # Opts passed to Scout:
     #   :headers, :redir, :relvl
     #
-    # @ :result : враппер результата исполнения; по умолчанию Page, для Client — если определён — Result; при асинхронном вызове будет возвращён незамедлительно
-    # @ &callback : в него будет передан инстанс result, а его результат будет записан в result#res (по умолчанию это ссылка на себя)
-    # @ :thread_safe : не использовать луп исполнения Curl::Multi#perform, а вызывать #perform прямо в этом треде; если установлен, то невозможно прерывание исполнения клавиатурой (продолжит работать, выполняя колбеки, в фоне), и невозможно задавать больше параллельных реквестов, чем разрешено параллельных соединений (просто застрянет)
-    # @ :sync : остановить (Thread#kill) perform-loop после исполнения всех запросов; подразумевает wait=true; при вызове одиночного реквеста подразумевает thread_safe=true
-    # @ :wait : ждать исполнения всех реквестов
-    # @ :save_result: возвращает #res для каждого инстанса result вместо самого инстанса; если не задан :proc_result, то подразумевает wait=true
-    # @ :proc_result: Proc, в который будет передан result#res, если задан также &callback; служит для создания вложенных блоков для клиентов; если =nil, то подразумевает wait=true
-    # @ :raw : сохраняем *только* тело ответа, без хедеров, без отладочной инфы в #res
-    # @ :raw + :sync : подразумевает save_result=true
-    # @ :xhr, :content_type, :auth : формируют хедеры X-Requested-With, Content-Type, Authorization для передачи в Scout
+    # Формирование хедеров запроса X-Requested-With, Content-Type, Authorization для передачи в Scout:
     # @ :xhr : boolean
     # @ :content_type : symbol<extension>  |  raw string
     # @ :auth : "<username>:<password>"
+    #
+    ### Обработка результата:
+    #   преобразование к понятному для клиента формату производится в result#process
+    #   использование данных из result производится
+    #     либо в &callback (functional),
+    #     либо использованием результата #run (imperative)
+    #   в первом случае в целях сборки мусора будет возвращён 
+    # @ :result : враппер результата исполнения; по умолчанию Page, для Client — если определён — <Class>::Result; при асинхронном вызове будет возвращён незамедлительно
+    # @ &callback : в него будет передан инстанс result, а его результат будет записан в <result>.res (по умолчанию это ссылка на <result>); в целях сборки мусора, если &callback задан, #run возвращает #res для каждого инстанса result вместо самого инстанса
+    # @ :complete : при вызове нескольких реквестов, в него будет передан [ <result>.res, ... ] от каждого из них, при вызове единичного — <result>.res от него
+    # @ :complete : в него будет передан [ <result>.res, ... ] относительно всех реквестов, когда все 
+    # @ :raw : сохраняем в #res *только* тело ответа — без хедеров, без отладочной инфы
+    # 
+    # @ [deprecated] :save_result : подразумевает callback=Proc::SELF; если не задан :proc_result, то подразумевает wait=true 
+    # @ [deprecated] :proc_result : Proc, в который будет передан result#res, если задан также &callback; если =nil, то подразумевает wait=true
+    #
+    ### Управление потоками:
+    # @ :thread_safe : не использовать луп исполнения Curl::Multi#perform, а вызывать #perform прямо в этом треде;
+    # если установлен, то невозможно прерывание исполнения клавиатурой (продолжит работать, выполняя колбеки, в фоне), и невозможно задавать больше параллельных реквестов, чем разрешено параллельных соединений (просто застрянет)
+    # @ :sync : остановить (Thread#kill) perform-loop после исполнения всех запросов; подразумевает wait=true; при вызове одиночного реквеста подразумевает thread_safe=true
+    # @ :wait : ждать исполнения всех реквестов
+    # @ :raw + :sync : подразумевает save_result=true
     # 
-    # @ :zip, :stream и все опции для result : deprecated
+    # @ [deprecated] :zip, :stream и все опции для result
     #
     # TODO: Семантически разделить синхронное и асинхронное выполнение запросов (не важно, серии или отдельных), с учётом, что асинхронность по сути своей перегружена и требуется, например, в очередях сообщений, но не в синхронных контроллерах Rails
+    #
+    # Пример использования коллбеков в общих/common методах клиента:
+    # def api(requests, **params, &callback)
+    #   @f.run(requests, **params.slice(:complete, :sync)) {|data|
+    #     params[:before].(data)
+    #     process_result = common_process(data)
+    #     custom_result = (callback || params[:after]).(data, process_result)
+    #   } # => [ custom_result, ... ]
+    # Весь процессинг 
     def exec *args, &callback
       many, order, orders, with_opts = interpret_request *args
       L.log({:many => many, :order => order, :orders => orders, :with_opts => with_opts})
@@ -132,6 +154,9 @@ module RHACK
       # if we aren't said explicitly about the opposite
       Johnson::Runtime.set_browser_for_curl with_opts
       
+      if with_opts[:save_result]
+        callback ||= Proc::SELF
+      end
       if many
         result = exec_many orders, with_opts, &callback
       else
@@ -340,11 +365,15 @@ module RHACK
         yres = callback.call page
         # if we don't want callback to affect page.res 
         # then we should not set :save_result
-        if yres != :skip
+        if yres == :skip
+          return # DEPRECATED
+        else
+          page.res = yres
           if opts[:proc_result].is Proc
             # yres is intermediate result that we should proc
             page.res = opts[:proc_result].call yres
-          elsif opts[:save_result] or :proc_result.in opts
+          #elsif opts[:save_result] or :proc_result.in opts
+          else
             # yres is total result that we should save
             page.res = yres
           end
@@ -352,13 +381,16 @@ module RHACK
           # so we can return result from any depth as @res attribute of what we have on top
         end
       end
+      if opts[:complete]
+        page.res = opts[:complete].call page.res
+      end
     end
     
     # TODO: found why/how IO on callbacks breaks +curl.res.body+ content and how to fix or how to avoid it
     def exec_one(order, opts, &callback)
       if @use_cache and order[0] == :loadGet and page = @@cache[order[1]]
         run_callbacks! page, opts, &callback
-        res = opts[:wait] && (opts[:save_result] or :proc_result.in opts) ? page.res : page
+        res = opts[:wait] && (callback or :proc_result.in opts) ? page.res : page
         return res
       end
       # must result in Page (default) or it's subclass
@@ -399,7 +431,7 @@ module RHACK
       end
       if opts[:wait]
         opts[:thread_safe] ? Curl.carier.perform : Curl.wait
-        (opts[:save_result] or :proc_result.in opts) ? page.res : page
+        (callback or :proc_result.in opts) ? page.res : page
       else page
       end
     end
@@ -416,9 +448,22 @@ module RHACK
           exec_one order, with_opts.merge(:exec => false), &callback
         }
       else
+        if oncomplete = with_opts.delete(:complete)
+          pages = nil
+          completed_count = 0
+          original_callback = callback
+          callback = lambda {|page|
+            page.res = original_callback ? original_callback.call(page) : page
+            completed_count += 1
+            if orders.size == completed_count
+              oncomplete.call pages.ress
+            end
+            page.res
+          }
+        end
         # если ss.next будет не хватать скаутов, то он сам запустит курл
         # правда, это с :thread_safe никак не вяжется
-        pages = orders.send(iterator) {|order| exec_one order, with_opts, &callback }
+        pages = orders.send(iterator) {|order| exec_one order, with_opts, &callback}
       end
       unless w and with_opts[:thread_safe] or opts[:exec] == false
         Curl.execute :unless_already

data/lib/rhack/version.rb

@@ -1,3 +1,3 @@
 module RHACK
-  VERSION = '1.3.3'
+  VERSION = '1.3.4'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rhack
 version: !ruby/object:Gem::Version
-  version: 1.3.3
+  version: 1.3.4
 platform: ruby
 authors:
 - Sergey Baev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-14 00:00:00.000000000 Z
+date: 2014-12-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rmtools