FC019: Fix regression in var_ref handling.

features/019_check_for_consistent_node_access.feature

@@ -83,6 +83,11 @@ Feature: Check for consistency in node access
      When I check the cookbook
      Then the attribute consistency warning 019 should be not shown
 
+  Scenario: Interpolated access
+    Given a cookbook with a single recipe that mixes node access types in an interpolated value
+      When I check the cookbook
+     Then the attribute consistency warning 019 should be shown
+
   Scenario: Two cookbooks with differing approaches
     Given a cookbook with a single recipe that reads node attributes via strings only
       And another cookbook with a single recipe that reads node attributes via symbols only

features/step_definitions/cookbook_steps.rb

@@ -893,6 +893,14 @@
   end
 end
 
+Given 'a cookbook with a single recipe that mixes node access types in an interpolated value' do
+  write_recipe %q{
+    execute "interpolated-example" do
+      command "#{node['foo'][:bar]}'"
+    end
+  }
+end
+
 Given 'a cookbook with a single recipe that reads multiple node attributes via symbols,strings' do
   write_recipe %q{
     node[:foo] = 'bar'

lib/foodcritic/api.rb

@@ -443,11 +443,11 @@ def standard_attribute_access(ast, options)
       else
         type = options[:type] == :string ? 'tstring_content' : options[:type]
         expr = '//*[self::aref_field or self::aref]'
-        expr += '[count(descendant::aref/var_ref) = 0]'
+        expr += '[count(is_att_type(descendant::var_ref/ident/@value)) =
+          count(descendant::var_ref/ident/@value)]'
         expr += '[is_att_type(descendant::ident'
         expr += '[not(ancestor::aref/call)]' if options[:ignore_calls]
         expr += "/@value)]/descendant::#{type}"
-        expr += "[ident/@value != 'node']" if type == :symbol
         ast.xpath(expr, AttFilter.new).sort
       end
     end