[jira] [Commented] (MYFACES-4130) CDI @ManagedProperty does not work with all types

2017-08-07 Thread Paul Nicolucci (JIRA)

[ 
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] [Resolved] (MYFACES-4130) CDI @ManagedProperty does not work with all types

2017-08-07 Thread Eduardo Breijo (JIRA)

 [ 
https://issues.apache.org/jira/browse/MYFACES-4130?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eduardo Breijo resolved MYFACES-4130.
-
   Resolution: Fixed
 Assignee: Eduardo Breijo  (was: Paul Nicolucci)
Fix Version/s: 2.3.0

> 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

2017-08-07 Thread Bernd Bohmann (JIRA)

[ 
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-4131) begin and end do not look to be implemented / working

2017-08-07 Thread Bernd Bohmann (JIRA)

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

Bernd Bohmann commented on MYFACES-4131:


ok

>  begin and end do not look to be implemented / working
> --
>
> Key: MYFACES-4131
> URL: https://issues.apache.org/jira/browse/MYFACES-4131
> Project: MyFaces Core
>  Issue Type: Bug
>  Components: JSR-372
>Affects Versions: 2.3.0-beta
>Reporter: Paul Nicolucci
> Attachments: MYFACES-4131.patch, MYFACES-4131-WITH-TEST.patch
>
>
> I started to test the  constraint feature of JSF 2.3 and it does 
> not look to function on MyFaces. 
> The changes required are for the following JSF 2.3 spec issue : 
> https://github.com/javaee/javaserverfaces-spec/issues/1102
> According to the spec the  tag will now have begin and end 
> attributes. For instance:
> 
> #{x}
> 
> In the above example if testList had 10 items in it each entry containing a 
> number 0-9 then we would expect the following output:
> 0123456789
> If we changed it to:
> 
> #{x}
> 
> We would expect the following output:
> 56789



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MYFACES-4130) CDI @ManagedProperty does not work with all types

2017-08-07 Thread Eduardo Breijo (JIRA)

[ 
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-4131) begin and end do not look to be implemented / working

2017-08-07 Thread Dora Rajappan (JIRA)

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

Dora Rajappan commented on MYFACES-4131:


If no one has objection I like to commit this patch.

>  begin and end do not look to be implemented / working
> --
>
> Key: MYFACES-4131
> URL: https://issues.apache.org/jira/browse/MYFACES-4131
> Project: MyFaces Core
>  Issue Type: Bug
>  Components: JSR-372
>Affects Versions: 2.3.0-beta
>Reporter: Paul Nicolucci
> Attachments: MYFACES-4131.patch, MYFACES-4131-WITH-TEST.patch
>
>
> I started to test the  constraint feature of JSF 2.3 and it does 
> not look to function on MyFaces. 
> The changes required are for the following JSF 2.3 spec issue : 
> https://github.com/javaee/javaserverfaces-spec/issues/1102
> According to the spec the  tag will now have begin and end 
> attributes. For instance:
> 
> #{x}
> 
> In the above example if testList had 10 items in it each entry containing a 
> number 0-9 then we would expect the following output:
> 0123456789
> If we changed it to:
> 
> #{x}
> 
> We would expect the following output:
> 56789



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MYFACES-4131) begin and end do not look to be implemented / working

2017-08-07 Thread Dora Rajappan (JIRA)

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

Dora Rajappan commented on MYFACES-4131:


One difference I notice while implemeting this is if offset + size is used with 
UI repeat it.(offset 0, size 10). It will print 0..9. While begin + end is used 
with UIRepeat (begin 0, end 9 )It will print 0..9. (According to Paul). So 
begin end prints including begin and  end. 

>  begin and end do not look to be implemented / working
> --
>
> Key: MYFACES-4131
> URL: https://issues.apache.org/jira/browse/MYFACES-4131
> Project: MyFaces Core
>  Issue Type: Bug
>  Components: JSR-372
>Affects Versions: 2.3.0-beta
>Reporter: Paul Nicolucci
> Attachments: MYFACES-4131.patch
>
>
> I started to test the  constraint feature of JSF 2.3 and it does 
> not look to function on MyFaces. 
> The changes required are for the following JSF 2.3 spec issue : 
> https://github.com/javaee/javaserverfaces-spec/issues/1102
> According to the spec the  tag will now have begin and end 
> attributes. For instance:
> 
> #{x}
> 
> In the above example if testList had 10 items in it each entry containing a 
> number 0-9 then we would expect the following output:
> 0123456789
> If we changed it to:
> 
> #{x}
> 
> We would expect the following output:
> 56789



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)