[ 
https://issues.apache.org/jira/browse/MYFACES-3797?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13788581#comment-13788581
 ] 

Gerhard Petracek commented on MYFACES-3797:
-------------------------------------------

@custom state-saving:
esp. for the upcoming version of the spec. there are ways to support it as 
well, however, i didn't disagree for the current version of the spec.
i just said that it isn't mandatory to support it (for a jsf 2.2 impl.).

@"only in those cases where CDI injection is required":
if you mean that you would like to check the classes in detail: you can't do 
that easily. you would lose many cdi features like @Alternative, Specializes, 
@Veto (v1.1),...

@libs which do such checks:
i know at least one proprietary component lib and at least 2-3 applications, 
but i agree for sure that it's easy to fix them.

@PartialStateHolder
agreed, however, i was talking about full state-saving for sure. if we are 
using a PartialStateHolder, we have to document this restriction.

@spec:
i haven't said that an adapter isn't easier for the spec., however, only if it 
is added to the spec. and all libs are compatible with it, we can be sure that 
it works (with partial-state-saving only).

however, since we also need a patch for the spec.-discussion and it's just a 
small change for the current patch, i'll do it in any case and we can use it 
for now. (if jsf 2.3 will specify a different approach, we can still change it.)

> cdi support for converters and validators
> -----------------------------------------
>
>                 Key: MYFACES-3797
>                 URL: https://issues.apache.org/jira/browse/MYFACES-3797
>             Project: MyFaces Core
>          Issue Type: New Feature
>          Components: JSR-344
>            Reporter: Gerhard Petracek
>            Assignee: Gerhard Petracek
>         Attachments: MYFACES-3797_2.patch, MYFACES-3797.patch
>
>
> with
>     <context-param>
>         
> <param-name>org.apache.myfaces.CONVERTER_INJECTION_ENABLED</param-name>
>         <param-value>true</param-value>
>     </context-param>
> and
>     <context-param>
>         
> <param-name>org.apache.myfaces.VALIDATOR_INJECTION_ENABLED</param-name>
>         <param-value>true</param-value>
>     </context-param>
> it should be possible to enable cdi support for converters/validators.
> we need the config, because it was postponed for the spec.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to