Melloware created MYFACES-4536:
----------------------------------
Summary: PartialResponseWriter: Do no wrap the writer
Key: MYFACES-4536
URL: https://issues.apache.org/jira/browse/MYFACES-4536
Project: MyFaces Core
Issue Type: Improvement
Components: General
Affects Versions: 4.0.0-RC2, 2.3-next-M7, 2.3.10, 2.2.14
Reporter: Melloware
Per BalusC:
Since JSF 2.3 the default constructor of {{FacesWrapper}} subclasses has been
deprecated in order to force implementors to instead use the constructor taking
the wrapped instance (and to raise their awareness), so that logically the
inherited {{getWrapped()}} method will be used throughout the implementation
instead of the local {{wrapped}} variable. This will ensure that the correct
implementation is returned and correct behavior is performed might the
{{FacesWrapper}} implementation itself being wrapped by yet another
{{FacesWrapper}} implementation further down the chain. Because, when the
{{FacesWrapper}} implementation incorrectly/accidentally uses the local
{{wrapped}} variable instead of the {{getWrapped()}} method, then that other
{{FacesWrapper}} implementation will basically be completely ignored, hereby
breaking the decorator pattern.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
