Allow users to customize ActiveStorage controllers without having to override them

[santib]

Summary

Other Information

[pr-explainer-bot]

Hi there! Here's the Pull Request report for the commit history you provided:

Changes

1. In activestorage.rb file, added parent_controller attribute to ActiveStorage module.
2. In proxy_controller.rb, redirect_controller.rb, direct_uploads_controller.rb, representations/proxy_controller.rb, and representations/redirect_controller.rb files, changed the parent class of the controllers to the value of ActiveStorage.parent_controller.constantize.

Suggestions

No suggestions provided.

Bugs

No bugs found.

Improvements

No improvements suggested.

Refactoring

In activestorage.rb file, instead of using default: "ActiveStorage::BaseController", we can use default: -> { ActiveStorage::BaseController } to avoid string interpolation.

module ActiveStorage
  mattr_accessor :parent_controller, default: -> { ActiveStorage::BaseController }
end

Rating

I would rate this code a 8 out of 10. The code is well-organized and easy to read. The addition of the parent_controller attribute is a useful customization option. However, there is room for improvement in terms of performance optimization.