[JBoss JIRA] Created: (WELD-913) SeamApplicationWrapper doesn't implement setApplication() correctly
12:29 a.m.
SeamApplicationWrapper doesn't implement setApplication() correctly
-------------------------------------------------------------------
Key: WELD-913
URL: https://issues.jboss.org/browse/WELD-913
Project: Weld
Issue Type: Bug
Components: Web Tier integration (JSF, JSP, EL and Servlet)
Affects Versions: 1.1.0.Final
Environment: Tomcat 7.0.8, Seam Faces 3.0.1, MyFaces 2.0.5
Reporter: Christian Kaltepoth
While debugging SEAMFACES-165 I discovered that {{SeamApplicationWrapper}} doesn't
implement {{setApplication()}} correctly. The class should recreate the locally cached
{{Application}} so that future calls to {{getApplication()}} return a correctly wrapped
version of the new {{Application}} instance.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
9:42 a.m.
New subject: [JBoss JIRA] Assigned: (WELD-913) SeamApplicationWrapper doesn't implement setApplication() correctly
[
https://issues.jboss.org/browse/WELD-913?page=com.atlassian.jira.plugin.s...
]
Ales Justin reassigned WELD-913:
--------------------------------
Assignee: Shane Bryzak
This looks like Seam issue? Shane?
SeamApplicationWrapper doesn't implement setApplication()
correctly
-------------------------------------------------------------------
Key: WELD-913
URL: https://issues.jboss.org/browse/WELD-913
Project: Weld
Issue Type: Bug
Components: Web Tier integration (JSF, JSP, EL and Servlet)
Affects Versions: 1.1.0.Final
Environment: Tomcat 7.0.8, Seam Faces 3.0.1, MyFaces 2.0.5
Reporter: Christian Kaltepoth
Assignee: Shane Bryzak
While debugging SEAMFACES-165 I discovered that {{SeamApplicationWrapper}} doesn't
implement {{setApplication()}} correctly. The class should recreate the locally cached
{{Application}} so that future calls to {{getApplication()}} return a correctly wrapped
version of the new {{Application}} instance.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
9:54 a.m.
New subject: [JBoss JIRA] Updated: (WELD-913) SeamApplicationWrapper doesn't implement setApplication() correctly
[
https://issues.jboss.org/browse/WELD-913?page=com.atlassian.jira.plugin.s...
]
Christian Kaltepoth updated WELD-913:
-------------------------------------
Description: While debugging SEAMFACES-165 I discovered that
{{WeldApplicationFactory}} doesn't implement {{setApplication()}} correctly. The class
should recreate the locally cached {{Application}} so that future calls to
{{getApplication()}} return a correctly wrapped version of the new {{Application}}
instance. (was: While debugging SEAMFACES-165 I discovered that
{{SeamApplicationWrapper}} doesn't implement {{setApplication()}} correctly. The class
should recreate the locally cached {{Application}} so that future calls to
{{getApplication()}} return a correctly wrapped version of the new {{Application}}
instance.)
Oh, sorry! I just saw there was a typo in my bug description. It is
{{WeldApplicationFactory}} and not {{SeamApplicationWrapper}} that doesn't implement
{{setApplication()}} correctly. I fixed the description. So it is definitively a Weld
issue.
Sorry! Seems like I got a bit confused by all the wrappers! :-)
SeamApplicationWrapper doesn't implement setApplication()
correctly
-------------------------------------------------------------------
Key: WELD-913
URL: https://issues.jboss.org/browse/WELD-913
Project: Weld
Issue Type: Bug
Components: Web Tier integration (JSF, JSP, EL and Servlet)
Affects Versions: 1.1.0.Final
Environment: Tomcat 7.0.8, Seam Faces 3.0.1, MyFaces 2.0.5
Reporter: Christian Kaltepoth
Assignee: Shane Bryzak
While debugging SEAMFACES-165 I discovered that {{WeldApplicationFactory}} doesn't
implement {{setApplication()}} correctly. The class should recreate the locally cached
{{Application}} so that future calls to {{getApplication()}} return a correctly wrapped
version of the new {{Application}} instance.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
10:20 a.m.
New subject: [JBoss JIRA] Commented: (WELD-913) SeamApplicationWrapper doesn't implement setApplication() correctly
[
https://issues.jboss.org/browse/WELD-913?page=com.atlassian.jira.plugin.s...
]
Ales Justin commented on WELD-913:
----------------------------------
It's ForwardingApplicationFactory that implements setApplication().
You're saying we should invalidate the current application instance in WeldAF if
someone re-set the application via setApplication?
SeamApplicationWrapper doesn't implement setApplication()
correctly
-------------------------------------------------------------------
Key: WELD-913
URL: https://issues.jboss.org/browse/WELD-913
Project: Weld
Issue Type: Bug
Components: Web Tier integration (JSF, JSP, EL and Servlet)
Affects Versions: 1.1.0.Final
Environment: Tomcat 7.0.8, Seam Faces 3.0.1, MyFaces 2.0.5
Reporter: Christian Kaltepoth
Assignee: Shane Bryzak
While debugging SEAMFACES-165 I discovered that {{WeldApplicationFactory}} doesn't
implement {{setApplication()}} correctly. The class should recreate the locally cached
{{Application}} so that future calls to {{getApplication()}} return a correctly wrapped
version of the new {{Application}} instance.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
10:31 a.m.
New subject: [JBoss JIRA] Updated: (WELD-913) SeamApplicationWrapper doesn't implement setApplication() correctly
[
https://issues.jboss.org/browse/WELD-913?page=com.atlassian.jira.plugin.s...
]
Ales Justin updated WELD-913:
-----------------------------
Fix Version/s: 1.1.2.Final
SeamApplicationWrapper doesn't implement setApplication()
correctly
-------------------------------------------------------------------
Key: WELD-913
URL: https://issues.jboss.org/browse/WELD-913
Project: Weld
Issue Type: Bug
Components: Web Tier integration (JSF, JSP, EL and Servlet)
Affects Versions: 1.1.0.Final
Environment: Tomcat 7.0.8, Seam Faces 3.0.1, MyFaces 2.0.5
Reporter: Christian Kaltepoth
Assignee: Shane Bryzak
Fix For: 1.1.2.Final
While debugging SEAMFACES-165 I discovered that {{WeldApplicationFactory}} doesn't
implement {{setApplication()}} correctly. The class should recreate the locally cached
{{Application}} so that future calls to {{getApplication()}} return a correctly wrapped
version of the new {{Application}} instance.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
10:32 a.m.
New subject: [JBoss JIRA] Commented: (WELD-913) SeamApplicationWrapper doesn't implement setApplication() correctly
[
https://issues.jboss.org/browse/WELD-913?page=com.atlassian.jira.plugin.s...
]
Christian Kaltepoth commented on WELD-913:
------------------------------------------
Exactly!
Currently {{WeldApplicationFactory}} lazily creates a wrapped {{Application}} in
{{getApplication()}} and save a reference to this wrapped instance for further calls to
{{getApplication()}. Calls to {{setApplication()}} are simply forwarded to the underlying
factory (by {{ForwardingApplicationFactory}}). So further calls to {{getApplication()}}
will return the old {{Application}} instances created in the first call which is not
correct according to the spec.
You can find a few more comment on this in SEAMFACES-165.
SeamApplicationWrapper doesn't implement setApplication()
correctly
-------------------------------------------------------------------
Key: WELD-913
URL: https://issues.jboss.org/browse/WELD-913
Project: Weld
Issue Type: Bug
Components: Web Tier integration (JSF, JSP, EL and Servlet)
Affects Versions: 1.1.0.Final
Environment: Tomcat 7.0.8, Seam Faces 3.0.1, MyFaces 2.0.5
Reporter: Christian Kaltepoth
Assignee: Shane Bryzak
Fix For: 1.1.2.Final
While debugging SEAMFACES-165 I discovered that {{WeldApplicationFactory}} doesn't
implement {{setApplication()}} correctly. The class should recreate the locally cached
{{Application}} so that future calls to {{getApplication()}} return a correctly wrapped
version of the new {{Application}} instance.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
10:32 a.m.
New subject: [JBoss JIRA] Issue Comment Edited: (WELD-913) SeamApplicationWrapper doesn't implement setApplication() correctly
[
https://issues.jboss.org/browse/WELD-913?page=com.atlassian.jira.plugin.s...
]
Christian Kaltepoth edited comment on WELD-913 at 6/8/11 11:31 AM:
-------------------------------------------------------------------
Exactly!
Currently {{WeldApplicationFactory}} lazily creates a wrapped {{Application}} in
{{getApplication()}} and save a reference to this wrapped instance for further calls to
{{getApplication()}. Calls to {{setApplication()}} are simply forwarded to the underlying
factory (by {{ForwardingApplicationFactory}}). So further calls to {{getApplication()}}
will return the old {{Application}} instance created in the first call which is not
correct according to the spec.
You can find a few more comment on this in SEAMFACES-165.
was (Author: chkal):
Exactly!
Currently {{WeldApplicationFactory}} lazily creates a wrapped {{Application}} in
{{getApplication()}} and save a reference to this wrapped instance for further calls to
{{getApplication()}. Calls to {{setApplication()}} are simply forwarded to the underlying
factory (by {{ForwardingApplicationFactory}}). So further calls to {{getApplication()}}
will return the old {{Application}} instances created in the first call which is not
correct according to the spec.
You can find a few more comment on this in SEAMFACES-165.
SeamApplicationWrapper doesn't implement setApplication()
correctly
-------------------------------------------------------------------
Key: WELD-913
URL: https://issues.jboss.org/browse/WELD-913
Project: Weld
Issue Type: Bug
Components: Web Tier integration (JSF, JSP, EL and Servlet)
Affects Versions: 1.1.0.Final
Environment: Tomcat 7.0.8, Seam Faces 3.0.1, MyFaces 2.0.5
Reporter: Christian Kaltepoth
Assignee: Shane Bryzak
Fix For: 1.1.2.Final
While debugging SEAMFACES-165 I discovered that {{WeldApplicationFactory}} doesn't
implement {{setApplication()}} correctly. The class should recreate the locally cached
{{Application}} so that future calls to {{getApplication()}} return a correctly wrapped
version of the new {{Application}} instance.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
10:52 a.m.
New subject: [JBoss JIRA] Issue Comment Edited: (WELD-913) SeamApplicationWrapper doesn't implement setApplication() correctly
[
https://issues.jboss.org/browse/WELD-913?page=com.atlassian.jira.plugin.s...
]
Christian Kaltepoth edited comment on WELD-913 at 6/8/11 11:52 AM:
-------------------------------------------------------------------
Exactly!
Currently {{WeldApplicationFactory}} lazily creates a wrapped {{Application}} in
{{getApplication()}} and saves a reference to this instance for further use. Invocations
of {{setApplication()}} are simply forwarded to the underlying factory (by
{{ForwardingApplicationFactory}}). So further calls to {{getApplication()}} will return
the old {{Application}} instance and not the new one that was set with
{{setApplication()}} which is not correct according to the spec.
You can find a few more comment on this in SEAMFACES-165.
was (Author: chkal):
Exactly!
Currently {{WeldApplicationFactory}} lazily creates a wrapped {{Application}} in
{{getApplication()}} and save a reference to this wrapped instance for further calls to
{{getApplication()}. Calls to {{setApplication()}} are simply forwarded to the underlying
factory (by {{ForwardingApplicationFactory}}). So further calls to {{getApplication()}}
will return the old {{Application}} instance created in the first call which is not
correct according to the spec.
You can find a few more comment on this in SEAMFACES-165.
SeamApplicationWrapper doesn't implement setApplication()
correctly
-------------------------------------------------------------------
Key: WELD-913
URL: https://issues.jboss.org/browse/WELD-913
Project: Weld
Issue Type: Bug
Components: Web Tier integration (JSF, JSP, EL and Servlet)
Affects Versions: 1.1.0.Final
Environment: Tomcat 7.0.8, Seam Faces 3.0.1, MyFaces 2.0.5
Reporter: Christian Kaltepoth
Assignee: Shane Bryzak
Fix For: 1.1.2.Final
While debugging SEAMFACES-165 I discovered that {{WeldApplicationFactory}} doesn't
implement {{setApplication()}} correctly. The class should recreate the locally cached
{{Application}} so that future calls to {{getApplication()}} return a correctly wrapped
version of the new {{Application}} instance.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
6:32 p.m.
New subject: [JBoss JIRA] Closed: (WELD-913) SeamApplicationWrapper doesn't implement setApplication() correctly
[
https://issues.jboss.org/browse/WELD-913?page=com.atlassian.jira.plugin.s...
]
Stuart Douglas closed WELD-913.
-------------------------------
Assignee: Ales Justin (was: Shane Bryzak)
Resolution: Done
Fixed in upstream
SeamApplicationWrapper doesn't implement setApplication()
correctly
-------------------------------------------------------------------
Key: WELD-913
URL: https://issues.jboss.org/browse/WELD-913
Project: Weld
Issue Type: Bug
Components: Web Tier integration (JSF, JSP, EL and Servlet)
Affects Versions: 1.1.0.Final
Environment: Tomcat 7.0.8, Seam Faces 3.0.1, MyFaces 2.0.5
Reporter: Christian Kaltepoth
Assignee: Ales Justin
Fix For: 1.1.2.Final
While debugging SEAMFACES-165 I discovered that {{WeldApplicationFactory}} doesn't
implement {{setApplication()}} correctly. The class should recreate the locally cached
{{Application}} so that future calls to {{getApplication()}} return a correctly wrapped
version of the new {{Application}} instance.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
4686
days inactive
4734
days old
8 comments
3 participants
participants (3)
-
Ales Justin (JIRA) -
Christian Kaltepoth (JIRA)
-
Stuart Douglas (JIRA)