[ 
https://issues.jboss.org/browse/SEAMWICKET-40?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ondrej Zizka updated SEAMWICKET-40:
-----------------------------------

    Description: 
Sometimes, you may need a standalone (Java SE) application, with an optional 
web module.
For project which are (for any reason, e.g. historically) not based on an app 
server, you would need a weld-se-core for the core.
weld-se-core can't co-exist with weld-servlet-core because they are different 
impl of the same API.
However, seam-wicket silently relies on having weld-servlet-core, at least to 
support RequestContext.

I suggest to make this need 
1) explicitly stated in POM,
2) optional.

To make it optional, it's enough to make this block
{code}
  if( conversation.isActive() ){ ... }
{code}
in SeamRequestCycleProcessor's response() optional, because without active 
RequestContext, it fails.


Also, and that would be for another jira, I suggest to modify weld-servlet-core 
to detect presence of weld-se-core, and use it's WeldContainer instead of 
bootstraping a new one.


  was:
Sometimes, you may need a standalone (Java SE) application, with an optional 
web module.
For project which are (for any reason, e.g. historically) not based on an app 
server, you would need a weld-se-core for the core.
weld-se-core can't co-exist with weld-servlet-core because they are different 
impl of the same API.
However, seam-wicket silently relies on having weld-servlet-core, at least to 
support RequestContext.

I suggest to make this need 
1) explicitly stated in POM,
2) optional.

To make it optional, it's enough to make this block
{code}
  if( conversation.isActive() ){ ... }
{code}
in SeamRequestCycleProcessor's response() optional, because without active 
RequestContext, it fails.





> Make dependency on weld-servlet-core truly optional
> ---------------------------------------------------
>
>                 Key: SEAMWICKET-40
>                 URL: https://issues.jboss.org/browse/SEAMWICKET-40
>             Project: Seam for Apache Wicket
>          Issue Type: Enhancement
>    Affects Versions: 3.0.0.Final
>         Environment: An app with weld-se-core and embedded container (like 
> embedded Jetty) running servlet/filter based web app.
>            Reporter: Ondrej Zizka
>
> Sometimes, you may need a standalone (Java SE) application, with an optional 
> web module.
> For project which are (for any reason, e.g. historically) not based on an app 
> server, you would need a weld-se-core for the core.
> weld-se-core can't co-exist with weld-servlet-core because they are different 
> impl of the same API.
> However, seam-wicket silently relies on having weld-servlet-core, at least to 
> support RequestContext.
> I suggest to make this need 
> 1) explicitly stated in POM,
> 2) optional.
> To make it optional, it's enough to make this block
> {code}
>   if( conversation.isActive() ){ ... }
> {code}
> in SeamRequestCycleProcessor's response() optional, because without active 
> RequestContext, it fails.
> Also, and that would be for another jira, I suggest to modify 
> weld-servlet-core to detect presence of weld-se-core, and use it's 
> WeldContainer instead of bootstraping a new one.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        
_______________________________________________
seam-issues mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/seam-issues

Reply via email to