order_cop 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eec6b83744b3873a01b1e5bbc3babf32c5d7a0ba76608b0aeacdd91094263eb2
-  data.tar.gz: 3c2a4a792e2670544a4cd327ee094aab10f8212a3017cb937464ee3e3cea0dec
+  metadata.gz: 9844113f7fde924bc0804ffd7a8a3e81b034e1009e2b8e399538e791756f1ba6
+  data.tar.gz: 71231522bb6415cb282945f68efcf58acc66d8f5400a70cbc039a076c49877ce
 SHA512:
-  metadata.gz: f3e2281df0bbbc4ed5e17a9b2fb671b91af40c016fde050a6bb23d2969feee877b62c720255f49de9544756472692fb9df4fa8d957ff97540821263a31693d4b
-  data.tar.gz: 3247b3dc2bd877226cd03437acb5cac3bf66712d7a7bfdc5d17357a41460dcf5783e265667394c1df0055958be2c7a3bcfa18cc534127761eb20dedf59a308d3
+  metadata.gz: 874cf90c0aefe66aa65a936bf2f48ee2fb2b383c41d5b137af448a5fafbabd5c138940254481ae27a36ea59c004692ca5d75cdc6e61104a6cebd81dfa676226f
+  data.tar.gz: ca59878e2544b486200256d224b5fc99ec0433ee134ea2617f46db658880f3860c42e316391b77afd332b7077fe073d51cd95db2758aaccaf2e5ba813c944cf7

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    order_cop (0.1.0)
+    order_cop (0.1.2)
       binding_of_caller (~> 1.0)
       rails (~> 7.0)
 

data/lib/order_cop/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module OrderCop
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

data/lib/order_cop.rb

@@ -54,15 +54,29 @@ module OrderCop
 
     private
 
-    def detect_missing_order(method)
-      return if OrderCop.disabled?
-      level = 1.upto(binding.frame_count).find do |i|
+    def stack_is_whitelisted?
+      1.upto(binding.frame_count).each do |i|
         lbinding = binding.of_caller(i)
         lmethod = lbinding.eval("__method__")
         next if lmethod.nil?
+        puts "lmethod: #{lmethod}" if OrderCop.debug?
         if OrderCop::WHITELIST.include?(lmethod)
-          return nil
+          puts "#{lmethod} is whitelisted, ignoring" if OrderCop.debug?
+          return true
         end
+      end
+      false
+    end
+
+    def detect_missing_order(method)
+      return if OrderCop.disabled?
+      puts "missing order, detect if allowed" if OrderCop.debug?
+      puts "stack size: #{binding.frame_count}" if OrderCop.debug?
+
+      return if stack_is_whitelisted?
+
+      level = 1.upto(binding.frame_count).find do |i|
+        lbinding = binding.of_caller(i)
         location = lbinding.source_location[0]
         location.include?(Rails.root.to_s) && !location.include?(Rails.root.join("config").to_s)
       end
@@ -134,8 +148,12 @@ module OrderCop
     config.raise
   end
 
+  def self.debug?
+    config.debug
+  end
+
   def self.config(**options)
-    @config ||= OpenStruct.new(rails_logger: false, raise: true, enabled: true)
+    @config ||= OpenStruct.new(rails_logger: false, raise: true, enabled: true, debug: false)
     options.each do |k, v|
       @config[k] = v
     end
@@ -169,4 +187,3 @@ end
 ActiveSupport::Reloader.to_prepare do
   OrderCop.apply(Rails.application)
 end
-# OrderCop.setup(rails_logger: true) unless Rails.env.production?

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: order_cop
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Gauthier Monserand