[ICE-8840] icecore:refresh hitting multiple pages - ICEsoft JIRA Issue Tracker

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: EE-3.0.0.GA_P01, 3.2
Fix Version/s: EE-3.2.0.GA, 3.3
Component/s: ICE-Components
Labels:
None
Environment:
Tomcat 7.0.14, Firefox, Chrome, IE

Assignee Priority:
P1

Description

To reproduce:
1. uncomment icecore:refresh tag on start.xhtml page in attached source code
2. run sample application
3. Open for example start.xhtml in Chrome & start2.xhtml in IE or Firefox

ISSUE: After some time specified in the interval parameter of the tag you should be able to see page refreshes in both browsers due to the presence of icecore:refresh tag on start.xhtml page. (request scoped beans will be re-initialized in both browsers)

Setting interval="0" duration="0" for the tag does not stop this behavior

Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Download All

Attachments

Hide
Case11783.zip

10/Jan/13 11:19 PM

9.34 MB

Evgheni Sadovoi
Case11783.war 10.17 MB

Download Zip
Show

Case11783.zip

10/Jan/13 11:19 PM

9.34 MB

Evgheni Sadovoi
Hide
sampleSourceCode.zip

19/Dec/12 12:56 AM

56 kB

Evgheni Sadovoi
nbproject/ant-deploy.xml 3 kB

nbproject/build-impl.xml 58 kB

nbproject/genfiles.properties 0.5 kB

nbproject/private/private.properties 0.4 kB

nbproject/private/private.xml 0.2 kB

nbproject/project.properties 3 kB

nbproject/project.xml 1 kB

src/conf/MANIFEST.MF 0.0 kB

src/java/.../JobApplicantService.java 0.3 kB

src/java/.../JobApplicantServiceImpl.java 0.6 kB

src/java/.../ApplicantController.java 4 kB

src/.../WordCapitalizationConverter.java 2 kB

src/java/.../LoggingPhaseListener.java 0.7 kB

src/java/.../PagePhaseListener.java 2 kB

src/java/org/.../view/model/Applicants.java 0.9 kB

src/java/org/.../view/model/CityList.java 0.4 kB

src/java/org/.../view/model/CountryList.java 0.4 kB

src/java/org/.../model/FontSizeList.java 0.4 kB

src/java/org/.../model/JobApplicant.java 2 kB

src/java/org/.../view/model/RequestTest.java 0.4 kB

src/java/org/.../model/ViewProperties.java 2 kB

src/java/.../JSF-override_en.properties 0.0 kB

src/java/.../resources/Msgs_en.properties 0.2 kB

src/java/.../resources/Msgs_es.properties 0.2 kB

src/java/.../resources/Msgs_fr.properties 0.2 kB

src/java/org/.../view/util/FacesUtils.java 8 kB

src/java/.../validators/EmailValidator.java 1 kB

web/applicants.xhtml 1 kB

web/index.jsp 0.0 kB

web/job-applicant.xhtml 1 kB

Showing 30 of 43 items Download Zip
Show

sampleSourceCode.zip

19/Dec/12 12:56 AM

56 kB

Evgheni Sadovoi

Activity

Ascending order - Click to sort in descending order

Evgheni Sadovoi created issue - 19/Dec/12 12:37 AM

Evgheni Sadovoi made changes - 19/Dec/12 12:56 AM

Field	Original Value	New Value
Attachment		sampleSourceCode.zip [ 15187 ]

Ken Fyten made changes - 19/Dec/12 4:03 PM

Fix Version/s		EE-3.2.0.GA [ 10332 ]
Fix Version/s		3.3 [ 10370 ]

Ken Fyten made changes - 19/Dec/12 4:03 PM

Assignee

Ted Goddard [ ted.goddard ]

Ken Fyten made changes - 19/Dec/12 4:03 PM

Assignee Priority

P2 [ 10011 ]

Ted Goddard made changes - 19/Dec/12 4:08 PM

Assignee

Ted Goddard [ ted.goddard ]

Jack Van Ooststroom [ jack.van.ooststroom ]

Hide

Permalink

Ted Goddard added a comment - 19/Dec/12 4:09 PM

Assigning to Jack (to work on after the ICEpush backoff and cloud push implementations).

Show

Ted Goddard added a comment - 19/Dec/12 4:09 PM Assigning to Jack (to work on after the ICEpush backoff and cloud push implementations).

Ken Fyten made changes - 08/Jan/13 10:29 PM

Assignee	Jack Van Ooststroom [ jack.van.ooststroom ]	Ted Goddard [ ted.goddard ]
Assignee Priority	P2 [ 10011 ]	P1 [ 10010 ]

Ted Goddard made changes - 08/Jan/13 10:35 PM

Assignee

Ted Goddard [ ted.goddard ]

Evgheni Sadovoi [ evgheni.sadovoi ]

Hide

Permalink

Ted Goddard added a comment - 08/Jan/13 10:35 PM

Please attach a .war file that runs on tomcat and assign back to me.

Show

Ted Goddard added a comment - 08/Jan/13 10:35 PM Please attach a .war file that runs on tomcat and assign back to me.

Hide

Permalink

Evgheni Sadovoi added a comment - 10/Jan/13 11:19 PM

Unzip case11783 file to get deployable .war file

Show

Evgheni Sadovoi added a comment - 10/Jan/13 11:19 PM Unzip case11783 file to get deployable .war file

Evgheni Sadovoi made changes - 10/Jan/13 11:19 PM

Attachment

Case11783.zip [ 15333 ]

Evgheni Sadovoi made changes - 10/Jan/13 11:19 PM

Assignee

Evgheni Sadovoi [ evgheni.sadovoi ]

Ted Goddard [ ted.goddard ]

Hide

Permalink

Ted Goddard added a comment - 11/Jan/13 12:24 AM

Both start.jsf and start2.jsf are being refreshed: this appears to be working as expected.

Show

Ted Goddard added a comment - 11/Jan/13 12:24 AM Both start.jsf and start2.jsf are being refreshed: this appears to be working as expected.

Ted Goddard made changes - 11/Jan/13 12:24 AM

Assignee

Ted Goddard [ ted.goddard ]

Evgheni Sadovoi [ evgheni.sadovoi ]

Repository	Revision	Date	User	Message
ICEsoft Public SVN Repository	#33039	Fri Jan 11 14:49:41 MST 2013	ted.goddard	icecore:refresh taghandler now adding component directly to viewroot (~~ICE-8840~~)
				Files Changed
				MODIFY /icefaces3/trunk/icefaces/core/src/main/java/org/icefaces/impl/facelets/tag/icefaces/core/RefreshHandler.java

Hide

Permalink

Ted Goddard added a comment - 11/Jan/13 4:51 PM

The pages are as follows:

start.jsf contains an icecore:refresh tag
start2.jsf does not contain an icecore:refresh tag

When start2.jsf is loaded after start.jsf (even in a different browser) start2.jsf will be automatically refreshed.

Show

Ted Goddard added a comment - 11/Jan/13 4:51 PM The pages are as follows: start.jsf contains an icecore:refresh tag start2.jsf does not contain an icecore:refresh tag When start2.jsf is loaded after start.jsf (even in a different browser) start2.jsf will be automatically refreshed.

Hide

Permalink

Ted Goddard added a comment - 11/Jan/13 4:56 PM

RefreshHandler is implemented as an Application Scope SystemEvent listener, causing it to be incorrectly applied to all pages once activated.

Show

Ted Goddard added a comment - 11/Jan/13 4:56 PM RefreshHandler is implemented as an Application Scope SystemEvent listener, causing it to be incorrectly applied to all pages once activated.

Hide

Permalink

Ted Goddard added a comment - 11/Jan/13 9:51 PM

The following fix is checked in: rather than add a JavaScript output component from a global SystemEventListener, the TagHandler for this feature is now adding a transient component directly to the ViewRoot. The TagHandler has been modified so that different instances should pick up different configuration parameters, but this suspected bug in the original implementation has not been confirmed or tested. The fixed implementation has been verified to work as desired with the provided test case.

Show

Ted Goddard added a comment - 11/Jan/13 9:51 PM The following fix is checked in: rather than add a JavaScript output component from a global SystemEventListener, the TagHandler for this feature is now adding a transient component directly to the ViewRoot. The TagHandler has been modified so that different instances should pick up different configuration parameters, but this suspected bug in the original implementation has not been confirmed or tested. The fixed implementation has been verified to work as desired with the provided test case.

Ted Goddard made changes - 11/Jan/13 9:51 PM

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

Ken Fyten made changes - 14/Nov/14 5:39 PM

Status

Resolved [ 5 ]

Closed [ 6 ]

People

Assignee:

Evgheni Sadovoi

Reporter:

Evgheni Sadovoi

Votes:

0 Vote for this issue

Watchers:

2 Start watching this issue

Dates

Created:

19/Dec/12 12:37 AM

Updated:

14/Nov/14 5:39 PM

Resolved:

11/Jan/13 9:51 PM