[jira] [Commented] (MYFACES-4130) CDI @ManagedProperty does not work with all types
[
https://issues.apache.org/jira/browse/MYFACES-4130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16125590#comment-16125590
]
Thomas Andraschko commented on MYFACES-4130:
JFY: Your commit breaks my applications as you included a EL-Impl without the
right scope. I changed the scope and looks better now.
> CDI @ManagedProperty does not work with all types
> -
>
> Key: MYFACES-4130
> URL: https://issues.apache.org/jira/browse/MYFACES-4130
> Project: MyFaces Core
> Issue Type: Bug
> Components: JSR-372
>Affects Versions: 2.3.0-beta
>Reporter: Paul Nicolucci
>Assignee: Eduardo Breijo
> Fix For: 2.3.0
>
> Attachments: MYFACES-4130.patch, MYFACES-4130-with-test.patch
>
>
> CDI Replacement for @ManagedProperty does not work with primitives,
> ParameterizedType or arrays.
> For example:
> @Inject
> @ManagedProperty("#{testBean.list}")
> private List listManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.util.List
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.number}")
> private int numberManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException: int
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.stringArray}")
> private String[] stringArrayManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.lang.String[]
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> I've attached a patch. If no objections by Wednesday close of business I'll
> commit it.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
[jira] [Commented] (MYFACES-4130) CDI @ManagedProperty does not work with all types
[
https://issues.apache.org/jira/browse/MYFACES-4130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16116734#comment-16116734
]
Paul Nicolucci commented on MYFACES-4130:
-
Thanks for the help committing Eduardo!
> CDI @ManagedProperty does not work with all types
> -
>
> Key: MYFACES-4130
> URL: https://issues.apache.org/jira/browse/MYFACES-4130
> Project: MyFaces Core
> Issue Type: Bug
> Components: JSR-372
>Affects Versions: 2.3.0-beta
>Reporter: Paul Nicolucci
>Assignee: Eduardo Breijo
> Fix For: 2.3.0
>
> Attachments: MYFACES-4130.patch, MYFACES-4130-with-test.patch
>
>
> CDI Replacement for @ManagedProperty does not work with primitives,
> ParameterizedType or arrays.
> For example:
> @Inject
> @ManagedProperty("#{testBean.list}")
> private List listManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.util.List
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.number}")
> private int numberManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException: int
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.stringArray}")
> private String[] stringArrayManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.lang.String[]
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> I've attached a patch. If no objections by Wednesday close of business I'll
> commit it.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
[jira] [Commented] (MYFACES-4130) CDI @ManagedProperty does not work with all types
[
https://issues.apache.org/jira/browse/MYFACES-4130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16116610#comment-16116610
]
Bernd Bohmann commented on MYFACES-4130:
ok
> CDI @ManagedProperty does not work with all types
> -
>
> Key: MYFACES-4130
> URL: https://issues.apache.org/jira/browse/MYFACES-4130
> Project: MyFaces Core
> Issue Type: Bug
> Components: JSR-372
>Affects Versions: 2.3.0-beta
>Reporter: Paul Nicolucci
>Assignee: Paul Nicolucci
> Attachments: MYFACES-4130.patch, MYFACES-4130-with-test.patch
>
>
> CDI Replacement for @ManagedProperty does not work with primitives,
> ParameterizedType or arrays.
> For example:
> @Inject
> @ManagedProperty("#{testBean.list}")
> private List listManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.util.List
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.number}")
> private int numberManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException: int
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.stringArray}")
> private String[] stringArrayManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.lang.String[]
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> I've attached a patch. If no objections by Wednesday close of business I'll
> commit it.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
[jira] [Commented] (MYFACES-4130) CDI @ManagedProperty does not work with all types
[
https://issues.apache.org/jira/browse/MYFACES-4130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16116606#comment-16116606
]
Eduardo Breijo commented on MYFACES-4130:
-
I talked with Paul and I'm going to commit his patch.
> CDI @ManagedProperty does not work with all types
> -
>
> Key: MYFACES-4130
> URL: https://issues.apache.org/jira/browse/MYFACES-4130
> Project: MyFaces Core
> Issue Type: Bug
> Components: JSR-372
>Affects Versions: 2.3.0-beta
>Reporter: Paul Nicolucci
>Assignee: Paul Nicolucci
> Attachments: MYFACES-4130.patch, MYFACES-4130-with-test.patch
>
>
> CDI Replacement for @ManagedProperty does not work with primitives,
> ParameterizedType or arrays.
> For example:
> @Inject
> @ManagedProperty("#{testBean.list}")
> private List listManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.util.List
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.number}")
> private int numberManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException: int
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.stringArray}")
> private String[] stringArrayManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.lang.String[]
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> I've attached a patch. If no objections by Wednesday close of business I'll
> commit it.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
[jira] [Commented] (MYFACES-4130) CDI @ManagedProperty does not work with all types
[
https://issues.apache.org/jira/browse/MYFACES-4130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16111646#comment-16111646
]
Paul Nicolucci commented on MYFACES-4130:
-
I've been able to modify a test I had to fit in the myfaces unit test
framework. I had to make some updates to the el dependency used for testing.
I'll leave the patch here until tomorrow and then will work to commit it if no
comments.
> CDI @ManagedProperty does not work with all types
> -
>
> Key: MYFACES-4130
> URL: https://issues.apache.org/jira/browse/MYFACES-4130
> Project: MyFaces Core
> Issue Type: Bug
> Components: JSR-372
>Affects Versions: 2.3.0-beta
>Reporter: Paul Nicolucci
>Assignee: Paul Nicolucci
> Attachments: MYFACES-4130.patch, MYFACES-4130-with-test.patch
>
>
> CDI Replacement for @ManagedProperty does not work with primitives,
> ParameterizedType or arrays.
> For example:
> @Inject
> @ManagedProperty("#{testBean.list}")
> private List listManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.util.List
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.number}")
> private int numberManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException: int
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.stringArray}")
> private String[] stringArrayManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.lang.String[]
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> I've attached a patch. If no objections by Wednesday close of business I'll
> commit it.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
[jira] [Commented] (MYFACES-4130) CDI @ManagedProperty does not work with all types
[
https://issues.apache.org/jira/browse/MYFACES-4130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16109352#comment-16109352
]
Paul Nicolucci commented on MYFACES-4130:
-
Fixed some check style issues in the patch and attached to the issue again.
> CDI @ManagedProperty does not work with all types
> -
>
> Key: MYFACES-4130
> URL: https://issues.apache.org/jira/browse/MYFACES-4130
> Project: MyFaces Core
> Issue Type: Bug
> Components: JSR-372
>Affects Versions: 2.3.0-beta
>Reporter: Paul Nicolucci
>Assignee: Paul Nicolucci
> Attachments: MYFACES-4130.patch
>
>
> CDI Replacement for @ManagedProperty does not work with primitives,
> ParameterizedType or arrays.
> For example:
> @Inject
> @ManagedProperty("#{testBean.list}")
> private List listManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.util.List
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.number}")
> private int numberManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException: int
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.stringArray}")
> private String[] stringArrayManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.lang.String[]
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> I've attached a patch. If no objections by Wednesday close of business I'll
> commit it.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
[jira] [Commented] (MYFACES-4130) CDI @ManagedProperty does not work with all types
[
https://issues.apache.org/jira/browse/MYFACES-4130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16108687#comment-16108687
]
Bernd Bohmann commented on MYFACES-4130:
A unit test would be nice as well. But the patch looks good :-)
> CDI @ManagedProperty does not work with all types
> -
>
> Key: MYFACES-4130
> URL: https://issues.apache.org/jira/browse/MYFACES-4130
> Project: MyFaces Core
> Issue Type: Bug
> Components: JSR-372
>Affects Versions: 2.3.0-beta
>Reporter: Paul Nicolucci
>Assignee: Paul Nicolucci
> Attachments: MYFACES-4130.patch
>
>
> CDI Replacement for @ManagedProperty does not work with primitives,
> ParameterizedType or arrays.
> For example:
> @Inject
> @ManagedProperty("#{testBean.list}")
> private List listManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.util.List
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.number}")
> private int numberManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException: int
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> @Inject
> @ManagedProperty("#{testBean.stringArray}")
> private String[] stringArrayManagedProperty;
> javax.faces.FacesException: java.lang.ClassNotFoundException:
> java.lang.String[]
> at
> org.apache.myfaces.shared.util.ClassUtils.simpleClassForName(ClassUtils.java:218)
> at
> org.apache.myfaces.cdi.bean.DynamicManagedPropertyProducer.(DynamicManagedPropertyProducer.java:58)
> at
> org.apache.myfaces.cdi.bean.ManagedPropertyExtension.afterBean(ManagedPropertyExtension.java:62)
> I've attached a patch. If no objections by Wednesday close of business I'll
> commit it.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
