aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/lib/controller
diff options
context:
space:
mode:
authorMaurizio De Santis <desantis.maurizio@gmail.com>2019-01-16 14:16:46 +0100
committerMaurizio De Santis <desantis.maurizio@gmail.com>2019-01-16 14:16:46 +0100
commit1a098c542dde88e02e810b379251af2aaca0afe3 (patch)
tree7162108ff03d867e772e43f45f47aa111c9f6720 /actionpack/test/lib/controller
parent41ffddbc8b7faec66a26bd48dfd36a9def6cc23f (diff)
downloadrails-1a098c542dde88e02e810b379251af2aaca0afe3.tar.gz
rails-1a098c542dde88e02e810b379251af2aaca0afe3.tar.bz2
rails-1a098c542dde88e02e810b379251af2aaca0afe3.zip
Remove Decorator pattern in Overriding existing classes examples
Engines guide in the Overriding Models and Controllers section references the Decorator pattern, which isn't appropriate, since Decorator pattern is not about reopening existing classes, is about adding functionality to existing object instances; something that in Ruby is commonly implemented using Delegators. Moreover, the suggested naming convention for overrides, `app/decorators/**/*_decorator*.rb`, conflicts with a naming convention commonly used for View Model / Presentation Model decorators, adopted by popular gems such as `draper`, as well as by custom implementations.
Diffstat (limited to 'actionpack/test/lib/controller')
0 files changed, 0 insertions, 0 deletions