Enable Active Record strict loading by default about suspenders HOT 4 OPEN

Interesting! I didn't realize logging was an option.

I'd say the developer could disable it on a model/association basis.

I was thinking the same thing, although, there could be cases in which one would want to lazy-load a query in a specific context?

It looks like you can call strict_loading on a specific query to have the feature enabled. It looks like we could pass false to that method.

from suspenders.

@crackofdusk and I are working on this in https://github.com/thoughtbot/suspenders/tree/1119-enable-active-record-strict-loading-by-default

from suspenders.

@MatheusRich overall, I like this idea. If I'm understanding this correctly, enabling this feature will result in ActiveRecord::StrictLoadingViolationError being raised anytime an attempt to lazily load records was made.

However, what do we do in the case where we desire lazy loading, as mentioned in your example?

This article mentions a few approaches to enabling/disabling it on a more granular level.

My concern is that a developer my not know where the error is coming from, and decide to simply disable strict_loading_by_default as a whole.

from suspenders.

@stevepolitodesign In those cases, I'd say the developer could disable it on a model/association basis. That being said, we could enable make it just log errors, instead of raising them

# development.rb:

config.active_record.action_on_strict_loading_violation = :log

from suspenders.