This project is archived and is in readonly mode.
[PATCH] ActionDispatch MemCacheStore violates encapsulation principle
#4842
When pass :cache param to
ActionDispatch::Session::MemCacheStore, it should be
removed from options, otherwise it remains "published". This
violates the OOP encapsulation principle.
I attached a patch for this issue.
-
Michael Koziarski- State changed from new to wontfix
We don't generally modify argument hashes otherwise it breaks code which uses a single hash passed to multiple constructors. e.g. 5.times { Whatever.new(@options) }