okuribito 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d7493f6fcde494c7876664a9e9f08bd7ead6f424
-  data.tar.gz: 457a6555c87a74574bcd5e9876522116af76659c
+  metadata.gz: 546ff74e85424ffdd61c223346d05db2df28232c
+  data.tar.gz: 3972e942e590bc37402de3ffcdc7ead2bcaad654
 SHA512:
-  metadata.gz: 7a2b5d71abdf9659e15a197e4a2dcb4f3eef06609ef5b1a823c2f04cf7d5fc9e7fd0f86fb1e05a7c63d7114fbbd1886b8f59b01a4c3d7c84722d043e38f6e966
-  data.tar.gz: 36048f58088c47ee0598c0cdb1cbd7737de7e119a62ca4c72a78f012db7a7a4caadf9da8158509d81928f96fd27df43d4ab1ca72a1e219ff6c663243f5f3045d
+  metadata.gz: 1a52c32a171b6a7f6fddfe0f4c181d7b6e6bd7bdba80c1597a63bf3e06c1d5ebc7d76c127ced27aa4b65bdf0b6299a48a36dc0e46ef05cc09bea9b08e1795f2a
+  data.tar.gz: def1c5cba98bfbd1d0a6549712e7497f91da83a162e61a8a506923def5460b2aed7796f8abe505a65cd5d0b822ba31482812f33956d2df31fdbd9c92e6e27aee

data/.rubocop.yml

@@ -20,7 +20,7 @@ Metrics/CyclomaticComplexity:
   Max: 10
 
 Metrics/PerceivedComplexity:
-  Max: 10
+  Max: 9
 
 Metrics/MethodLength:
   Max: 20

data/README.md

@@ -2,10 +2,14 @@
 
 # Okuribito
 
+https://rubygems.org/gems/okuribito
+
 ![okuribito](okuribito_logo.png)
 
 Okuribito is a gem to judge whether methods should be sent to the heaven :innocent:.
+
 In other words, it can be used in order to extract the obsolete method.
+
 Okuribito was named after a japanese movie.
 
 ## Installation
@@ -122,33 +126,6 @@ okuribito = Okuribito::OkuribitoPatch.new do |method_name, obj_name, caller_info
 end
 ```
 
-### Send to slack
-
-```ruby
-okuribito = Okuribito::OkuribitoPatch.new do |method_name, obj_name, caller_info|
-  uri = URI.parse("https://hooks.slack.com/services/xxx...")
-  params = {
-      text: "OKURIBITO detect a method call.",
-      username: "OKURIBITO",
-      icon_emoji: ":innocent:",
-      attachments: [{
-                        fields: [{
-                                     title: "#{obj_name}::#{method_name}",
-                                     value: "#{caller_info[0]}",
-                                     short: false
-                                 }]
-                    }]
-  }
-  http = Net::HTTP.new(uri.host, uri.port)
-  http.use_ssl = true
-  http.start do
-    request = Net::HTTP::Post.new(uri.path)
-    request.set_form_data(payload: params.to_json)
-    http.request(request)
-  end
-end
-```
-
 ### Other ideas
 - Send to Fluentd, TreasureData, Slack...
 

data/lib/okuribito.rb

@@ -15,7 +15,7 @@ module Okuribito
     end
 
     module SimplePatchModule
-      def define_okuribito_patch(method_name, _opt = {})
+      def define_patch(method_name, _patch, _id, _opt = {})
         define_method(method_name) do |*args|
           yield(to_s, caller) if block_given?
           super(*args)
@@ -24,12 +24,12 @@ module Okuribito
     end
 
     module FunctionalPatchModule
-      def define_okuribito_patch(method_name, opt = {})
-        instance_variable_set("@#{method_name}_called", false)
+      def define_patch(method_name, patch, id, opt = {})
+        patch.instance_variable_set("@#{method_name}_#{id}_called", false)
         define_method(method_name) do |*args|
-          if block_given? && !instance_variable_get("@#{method_name}_called")
+          if block_given? && !patch.instance_variable_get("@#{method_name}_#{id}_called")
             yield(to_s, caller)
-            instance_variable_set("@#{method_name}_called", true) if opt[:once_detect]
+            patch.instance_variable_set("@#{method_name}_#{id}_called", true) if opt[:once_detect]
           end
           super(*args)
         end
@@ -50,14 +50,26 @@ module Okuribito
 
       klass.class_eval do
         if opt.present?
-          instance_method_patch = Module.new.extend(FunctionalPatchModule)
-          class_method_patch    = Module.new.extend(FunctionalPatchModule)
+          i_patch_name = "#{class_name}InstancePatch"
+          c_patch_name = "#{class_name}ClassPatch"
+          if FunctionalPatchModule.const_defined?(i_patch_name.to_sym)
+            i_method_patch = Module.new.extend(FunctionalPatchModule)
+          else
+            i_method_patch = FunctionalPatchModule
+                             .const_set(i_patch_name, Module.new.extend(FunctionalPatchModule))
+          end
+          if FunctionalPatchModule.const_defined?(c_patch_name.to_sym)
+            c_method_patch = Module.new.extend(FunctionalPatchModule)
+          else
+            c_method_patch = FunctionalPatchModule
+                             .const_set(c_patch_name, Module.new.extend(FunctionalPatchModule))
+          end
         else
-          instance_method_patch = Module.new.extend(SimplePatchModule)
-          class_method_patch    = Module.new.extend(SimplePatchModule)
+          i_method_patch = Module.new.extend(SimplePatchModule)
+          c_method_patch = Module.new.extend(SimplePatchModule)
         end
-        instance_method_patched = 0
-        class_method_patched    = 0
+        i_method_patched = 0
+        c_method_patched = 0
 
         observe_methods.each do |observe_method|
           next unless md = PATTERN.match(observe_method)
@@ -67,24 +79,24 @@ module Okuribito
           case symbol
           when INSTANCE_METHOD_SYMBOL
             next unless klass.instance_methods.include?(method_name)
-            instance_method_patch.module_eval do
-              define_okuribito_patch(method_name, opt) do |obj_name, caller_info|
+            i_method_patch.module_eval do
+              define_patch(method_name, i_method_patch, "i", opt) do |obj_name, caller_info|
                 callback.call(method_name, obj_name, caller_info)
               end
             end
-            instance_method_patched += 1
+            i_method_patched += 1
           when CLASS_METHOD_SYMBOL
             next unless klass.respond_to?(method_name)
-            class_method_patch.module_eval do
-              define_okuribito_patch(method_name, opt) do |obj_name, caller_info|
+            c_method_patch.module_eval do
+              define_patch(method_name, c_method_patch, "c", opt) do |obj_name, caller_info|
                 callback.call(method_name, obj_name, caller_info)
               end
             end
-            class_method_patched += 1
+            c_method_patched += 1
           end
         end
-        prepend instance_method_patch if instance_method_patched > 0
-        singleton_class.send(:prepend, class_method_patch) if class_method_patched > 0
+        prepend i_method_patch if i_method_patched > 0
+        singleton_class.send(:prepend, c_method_patch) if c_method_patched > 0
       end
     end
   end

data/lib/okuribito/version.rb

@@ -1,3 +1,3 @@
 module Okuribito
-  VERSION = "0.1.3".freeze
+  VERSION = "0.1.4".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: okuribito
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - muramurasan
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-10-08 00:00:00.000000000 Z
+date: 2016-10-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport