[ICE-11004] showcase & poll - mobi:cloudPush demo update required for URI cloudPush API changes - ICEsoft JIRA Issue Tracker

Details

Type: Task
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: None
Fix Version/s: EE-4.1.0.GA, 4.2.BETA, 4.2
Component/s: Sample Apps
Labels:
None
Environment:
ICEfaces4 trunk r48623
Browsers: all
Server: Tomcat7/8

Assignee Priority:
P1

Description

showcase > mobi:cloudPush:
Demo requires updates, as server errors occur when enabling cloudPush (desktop and mobile).

13-Apr-2016 10:29:52.130 SEVERE [http-apr-8080-exec-5] com.sun.faces.context.AjaxExceptionHandlerImpl.handlePartialResponseError java.lang.IllegalArgumentException: Illegal argument targetURI: 'null'. Argument cannot be null.
         at org.icesoft.util.PreCondition.checkArgument(PreCondition.java:44)
         at org.icefaces.application.PushMessage.setTargetURI(PushMessage.java:79)
         at org.icefaces.application.PushMessage.<init>(PushMessage.java:38)
         at org.icefaces.samples.showcase.example.mobi.cloudpush.CloudPushBean.sendPriorityPushMessage(CloudPushBean.java:115)
         at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
         at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
         at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
         at java.lang.reflect.Method.invoke(Method.java:606)
         at org.apache.el.parser.AstValue.invoke(AstValue.java:245)
....
13-Apr-2016 10:30:48.203 WARNING [http-apr-8080-exec-9] com.sun.faces.lifecycle.
InvokeApplicationPhase.execute /resources/examples/mobi/cloudpush/cloudpush-example.xhtml @106,100 actionListener="#{cloudPushBean.sendPriorityPushMessage}": java.lang.IllegalArgumentException: Illegal argument targetURI: 'null'. Argumentcannot be null.
  javax.el.ELException: /resources/examples/mobi/cloudpush/cloudpush-example.xhtml @106,100 actionListener="#{cloudPushBean.sendPriorityPushMessage}":java.lang.IllegalArgumentException: Illegal argument targetURI: 'null'. Argument cannot be null.
         at com.sun.faces.facelets.el.TagMethodExpression.invoke(TagMethodExpression.java:111)
         at javax.faces.event.MethodExpressionActionListener.processAction(MethodExpressionActionListener.java:147)
         at javax.faces.event.ActionEvent.processListener(ActionEvent.java:88)
         at javax.faces.component.UIComponentBase.broadcast(UIComponentBase.java:814)
         at javax.faces.component.UICommand.broadcast(UICommand.java:300)
         at javax.faces.component.UIViewRoot.broadcastEvents(UIViewRoot.java:790)
         at javax.faces.component.UIViewRoot.processApplication(UIViewRoot.java:1282)
         ...........
Caused by: java.lang.IllegalArgumentException: Illegal argument targetURI: 'null'. Argument cannot be null.
         at org.icesoft.util.PreCondition.checkArgument(PreCondition.java:44)
         at org.icefaces.application.PushMessage.setTargetURI(PushMessage.java:79)
         at org.icefaces.application.PushMessage.<init>(PushMessage.java:38)
         at org.icefaces.samples.showcase.example.mobi.cloudpush.CloudPushBean.sendPriorityPushMessage(CloudPushBean.java:115)
         at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
         at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
         ......

Issue Links

depends on

ICE-10969 Runtime ICEpush dependencies need to be included in samples

Closed

PUSH-388 Include the uri of a cloud push notification in the public API

Resolved

Activity

Ascending order - Click to sort in descending order

Carmen Cristurean created issue - 13/Apr/16 3:05 PM

Carmen Cristurean made changes - 13/Apr/16 3:05 PM

Field	Original Value	New Value
Fix Version/s		EE-4.1.0.GA [ 12171 ]

Carmen Cristurean made changes - 13/Apr/16 3:06 PM

Link

This issue depends on ~~PUSH-388~~ [ ~~PUSH-388~~ ]

Judy Guglielmin made changes - 13/Apr/16 3:39 PM

Assignee

Jack Van Ooststroom [ jack.van.ooststroom ]

Hide

Permalink

Carmen Cristurean added a comment - 14/Apr/16 4:07 PM

The poll application from ICEfaces EE4-trunk also requires updates for URI cloudpush API changes.

Show

Carmen Cristurean added a comment - 14/Apr/16 4:07 PM The poll application from ICEfaces EE4-trunk also requires updates for URI cloudpush API changes.

Hide

Permalink

Jack Van Ooststroom added a comment - 15/Apr/16 9:13 AM

Sending src/main/java/org/icefaces/samples/showcase/example/mobi/cloudpush/CloudPushBean.java
Transmitting file data .
Committed revision 48632.

Show

Jack Van Ooststroom added a comment - 15/Apr/16 9:13 AM Sending src/main/java/org/icefaces/samples/showcase/example/mobi/cloudpush/CloudPushBean.java Transmitting file data . Committed revision 48632.

Hide

Permalink

Jack Van Ooststroom added a comment - 15/Apr/16 9:13 AM

Fix for Showcase has been successfully tested and committed.

Show

Jack Van Ooststroom added a comment - 15/Apr/16 9:13 AM Fix for Showcase has been successfully tested and committed.

Repository	Revision	Date	User	Message
ICEsoft Public SVN Repository	#48632	Fri Apr 15 09:20:43 MDT 2016	jack.van.ooststroom	Fixed JIRA ~~ICE-11004~~ : showcase - mobi:cloudPush demo update required for URI cloudPush API changes
				Files Changed
				MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/mobi/cloudpush/CloudPushBean.java

Hide

Permalink

Carmen Cristurean added a comment - 15/Apr/16 9:36 AM - edited

Verified showcase > cloudPush with IF4 trunk r48632/ EE4-icefaces-trunk r44945, on Chrome50.
The cloudPush eMail notification works only on ICEfaces4 trunk, that is presumably a known issue.

Show

Carmen Cristurean added a comment - 15/Apr/16 9:36 AM - edited Verified showcase > cloudPush with IF4 trunk r48632/ EE4-icefaces-trunk r44945, on Chrome50. The cloudPush eMail notification works only on ICEfaces4 trunk, that is presumably a known issue.

Hide

Permalink

Jack Van Ooststroom added a comment - 18/Apr/16 9:16 AM

Sending src/main/java/com/icesoft/faces/demo/poll/ParticipantView.java
Sending src/main/java/com/icesoft/faces/demo/poll/Poll.java
Sending src/main/java/com/icesoft/faces/demo/poll/PollCollection.java
Sending src/main/java/com/icesoft/faces/demo/poll/PollMessageService.java
Transmitting file data ....
Committed revision 44955.

Show

Jack Van Ooststroom added a comment - 18/Apr/16 9:16 AM Sending src/main/java/com/icesoft/faces/demo/poll/ParticipantView.java Sending src/main/java/com/icesoft/faces/demo/poll/Poll.java Sending src/main/java/com/icesoft/faces/demo/poll/PollCollection.java Sending src/main/java/com/icesoft/faces/demo/poll/PollMessageService.java Transmitting file data .... Committed revision 44955.

Hide

Permalink

Jack Van Ooststroom added a comment - 18/Apr/16 9:16 AM

Fix for Poll has been successfully tested and committed.

Show

Jack Van Ooststroom added a comment - 18/Apr/16 9:16 AM Fix for Poll has been successfully tested and committed.

Jack Van Ooststroom made changes - 18/Apr/16 9:16 AM

Summary

showcase - mobi:cloudPush demo update required for URI cloudPush API changes

showcase & poll - mobi:cloudPush demo update required for URI cloudPush API changes

Hide

Permalink

Jack Van Ooststroom added a comment - 18/Apr/16 9:17 AM

Changed the JIRA's title to include Poll as well. Both fixes have a similar base.

Show

Jack Van Ooststroom added a comment - 18/Apr/16 9:17 AM Changed the JIRA's title to include Poll as well. Both fixes have a similar base.

Ken Fyten made changes - 28/Apr/16 11:07 AM

Assignee Priority

P1 [ 10010 ]

Ken Fyten made changes - 29/Apr/16 10:41 AM

Fix Version/s

EE-4.1.0.BETA [ 13072 ]

Ken Fyten made changes - 29/Apr/16 3:00 PM

Issue Type

Bug [ 1 ]

Task [ 3 ]

Ken Fyten made changes - 29/Apr/16 3:13 PM

Security

Private [ 10001 ]

Ken Fyten made changes - 29/Apr/16 3:13 PM

Affects Version/s

EE-4.1.0.GA [ 12171 ]

Jack Van Ooststroom made changes - 06/May/16 9:16 AM

Link

This issue blocks ~~ICE-10969~~ [ ~~ICE-10969~~ ]

Jack Van Ooststroom made changes - 06/May/16 9:17 AM

Link

This issue blocks ~~ICE-10969~~ [ ~~ICE-10969~~ ]

Jack Van Ooststroom made changes - 06/May/16 9:17 AM

Link

This issue depends on ~~ICE-10969~~ [ ~~ICE-10969~~ ]

Repository	Revision	Date	User	Message
ICEsoft Public SVN Repository	#48703	Fri May 06 09:28:29 MDT 2016	jack.van.ooststroom	Fixed JIRA ~~ICE-11004~~ : Runtime ICEpush dependencies need to be included in samples; First part fix.
				Files Changed
				MODIFY /icefaces4/trunk/icefaces/build.xml MODIFY /icefaces4/trunk/icefaces/samples/build/common.xml MODIFY /icefaces4/trunk/icefaces/build.properties

Hide

Permalink

Jack Van Ooststroom added a comment - 06/May/16 10:53 AM

As ~~ICE-10969~~ has been resolved, marking this one as FIXED as well.

Show

Jack Van Ooststroom added a comment - 06/May/16 10:53 AM As ICE-10969 has been resolved, marking this one as FIXED as well.

Jack Van Ooststroom made changes - 06/May/16 10:53 AM

Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

Hide

Permalink

Carmen Cristurean added a comment - 06/May/16 11:57 AM

EE4 trunk r48703: verified samples built .war files using Ant.

Show

Carmen Cristurean added a comment - 06/May/16 11:57 AM EE4 trunk r48703: verified samples built .war files using Ant.

Ken Fyten made changes - 11/May/16 11:34 AM

Fix Version/s		EE-4.1.1.BETA [ 13079 ]
Fix Version/s	EE-4.1.0.GA [ 12171 ]
Fix Version/s	EE-4.1.0.RC1 [ 13072 ]

Ken Fyten made changes - 19/Dec/16 10:35 AM

Fix Version/s		EE-4.1.0.GA [ 12171 ]
Fix Version/s	EE-4.1.1.BETA [ 13079 ]

Ken Fyten made changes - 22/Dec/16 10:46 AM

Fix Version/s		4.2.BETA [ 13091 ]
Fix Version/s		4.2 [ 12870 ]

Repository	Revision	Date	User	Message
ICEsoft Public SVN Repository	#50744	Mon Feb 27 09:34:56 MST 2017	jack.van.ooststroom	Reverted ~~ICE-11004~~ fixes to ICEfaces-layer
				Files Changed
				MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/mobi/cloudpush/CloudPushBean.java

Ken Fyten made changes - 13/Mar/17 1:33 PM

Status

Resolved [ 5 ]

Closed [ 6 ]

People

Assignee:

Jack Van Ooststroom

Reporter:

Carmen Cristurean

Votes:

0 Vote for this issue

Watchers:

2 Start watching this issue

Dates

Created:

13/Apr/16 3:05 PM

Updated:

13/Mar/17 1:33 PM

Resolved:

06/May/16 10:53 AM