Refine config method in ViewComponent::Base (#2278)

* Let's be a bit gentler with trying the module parents on config.
Also, avoid naming variables inside after the method name itself.

Refactor config method in ViewComponent to improve readability and safety. Use explicit checks for module parent configuration instead of chaining try methods.

* fix allocations

* more allocations

---------

Co-authored-by: Joel Hawksley <[email protected]>
Co-authored-by: Joel Hawksley <[email protected]>

Author: tomasc

lib/view_component/base.rb

@@ -26,9 +26,10 @@ class << self
       #
       # @return [ActiveSupport::OrderedOptions]
       def config
-        module_parents.each do |m|
-          config = m.try(:config).try(:view_component)
-          return config if config
+        module_parents.each do |module_parent|
+          next unless module_parent.respond_to?(:config)
+          module_parent_config = module_parent.config.try(:view_component)
+          return module_parent_config if module_parent_config
         end
         ViewComponent::Config.current
       end

test/sandbox/test/rendering_test.rb

@@ -17,11 +17,11 @@ def test_render_inline_allocations
 
     allocations =
       if Rails.version.to_f < 8.0
-        {"3.3.8" => 128, "3.3.0" => 140, "3.2.8" => 126, "3.1.7" => 126, "3.0.7" => 135}
+        {"3.3.8" => 124, "3.3.0" => 140, "3.2.8" => 122, "3.1.7" => 122, "3.0.7" => 131}
       elsif Rails.version.split(".").first(2).map(&:to_i) == [8, 0]
-        {"3.5.0" => 121, "3.4.3" => 125, "3.3.8" => 137}
+        {"3.5.0" => 117, "3.4.4" => 121, "3.3.8" => 133}
       else
-        {"3.4.3" => 123}
+        {"3.4.4" => 119}
       end
 
     assert_allocations(**allocations) do