Enhanced Decorator support about javaserverfaces-spec HOT 13 CLOSED

Reported by @edburns

from javaserverfaces-spec.

@edburns said:
up to p1 feature.

from javaserverfaces-spec.

@edburns said:
I have asked Ryan Lubke of the Sun team to produce a proposal for this, based on
Adam's content in this message.

from javaserverfaces-spec.

@edburns said:
Created an attachment (id=2)
source for proposal

from javaserverfaces-spec.

@edburns said:
From Ryan Lubke:

Motivation

AW> The faces-config.xml support for the decorator pattern is
AW> one of my favorite features in JSF, and I like pushing
AW> it at users whenever I get a chance. But it is a pain
AW> using it, especially on objects with large APIs (like ViewHandler).

AW> I'd recommend we follow the lead of the Servlet API and
AW> provide convenience classes for implementing the Decorator
AW> pattern. (They were a bit more than convenience classes
AW> for the Servlet API, but anyway...)

AW> This would make implementing the Decorator pattern much,
AW> much simpler. It also makes it much more robust - as things
AW> stand now, every time we add a method to any of these
AW> decoratable APIs in a new version of JSF, all existing
AW> Decorators will break!

Proposal

Create wrapper classes for the following:

• ViewHandlerWrapper

• StateManagerWrapper

• ResponseWriterWrapper

• PropertyResolverWrapper

• VariableResolverWrapper (this one was added more for balance)

• These classes have a single constructor which takes an instance
  of the class they are designed to wrap.

• The default behavior for the methods of each implementation call through
  to the wrapped instance

from javaserverfaces-spec.

@edburns said:
take ownership

from javaserverfaces-spec.

@edburns said:
I've resolved ViewHandler, StateManager, and ResponseWriter and left the EL bits
until we have the EL alignment done.

from javaserverfaces-spec.

@edburns said:
Prepare to delete "spec" subcomponent.

from javaserverfaces-spec.

@edburns said:
Move all to 1.2

from javaserverfaces-spec.

@manfredriem said:
Closing resolved issue out

from javaserverfaces-spec.

File: wrapper-src.zip
Attached By: @edburns

from javaserverfaces-spec.

This issue was imported from java.net JIRA JAVASERVERFACES_SPEC_PUBLIC-15

from javaserverfaces-spec.

Marked as won't fix on Thursday, March 4th 2010, 6:09:29 am

from javaserverfaces-spec.