Class: RuboCop::Cop::Captive::Rails::NoHasManyAttached

Inherits:

Base

• Object
• Base
• RuboCop::Cop::Captive::Rails::NoHasManyAttached

Defined in:

lib/rubocop/cop/captive/rails/no_has_many_attached.rb

Overview

Interdit l’utilisation de ‘has_many_attached` (Active Storage) dans les modèles Rails.

À chaque utilisation de ‘has_many_attached`, on finit par avoir besoin d’informations supplémentaires sur les fichiers (image de couverture, position, ordre, etc.), ce qu’on ne peut pas faire avec des blobs seuls. Par exemple, un produit avec ‘has_many_attached :images` permet d’afficher la première image en couverture, mais si cette image s’avère être une grille de tailles et qu’on veut en choisir une autre, c’est impossible : les pièces jointes sont de simples blobs sans attributs métier.

La solution recommandée est d’utiliser un modèle associé (ex. ‘Photo`) avec `has_one_attached :file` et des attributs (position, type, etc.). Cela évite une migration coûteuse plus tard pour faire passer des millions d’images de blobs à un vrai modèle.

See Also:

• https://discuss.rubyonrails.org/t/active-storage-in-production-lessons-learned-and-in-depth-look-at-how-it-works/83289#p-254385-bonus-1-if-someone-were-to-write-active-storage-the-good-parts-then-has_many_attached-would-not-be-in-it-31

Constant Summary

MSG =

"N'utilisez pas `has_many_attached`. Préférez un modèle associé avec " \
"`has_one_attached` pour pouvoir ajouter des attributs " \
"(position, image de couverture, etc.)."

Instance Method Summary

• #on_send(node) ⇒ Object

Instance Method Details

#on_send(node) ⇒ Object

# File 'lib/rubocop/cop/captive/rails/no_has_many_attached.rb', line 31

def on_send(node)
  return unless has_many_attached?(node)

  add_offense(node, message: MSG)
end