Lint adapter query strategies (#781)

CanCanCommunity · Mar 30, 2022 · be14fda · be14fda
1 parent 9c75a78
commit be14fda
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/cancan/model_adapters/strategies/joined_alias_each_rule_as_exists_subquery.rb b/lib/cancan/model_adapters/strategies/joined_alias_each_rule_as_exists_subquery.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: false
+
 module CanCan
   module ModelAdapters
     class Strategies
@@ -15,7 +17,7 @@ def double_exists_sql
           double_exists_sql = ''
 
           compressed_rules.each_with_index do |rule, index|
-            double_exists_sql << ' OR ' if index > 0
+            double_exists_sql << ' OR ' if index.positive?
             double_exists_sql << "EXISTS (#{sub_query_for_rule(rule).to_sql})"
           end
 

diff --git a/lib/cancan/model_adapters/strategies/joined_alias_exists_subquery.rb b/lib/cancan/model_adapters/strategies/joined_alias_exists_subquery.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module CanCan
   module ModelAdapters
     class Strategies