You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by xu...@apache.org on 2010/11/09 07:19:52 UTC
svn commit: r1032859 -
/geronimo/server/trunk/plugins/myfaces/geronimo-myfaces-builder/src/main/java/org/apache/geronimo/myfaces/deployment/MyFacesModuleBuilderExtension.java
Author: xuhaihong
Date: Tue Nov 9 06:19:52 2010
New Revision: 1032859
URL: http://svn.apache.org/viewvc?rev=1032859&view=rev
Log:
Explicitly import a MyFaces package, so that MyFaces-core package could wired to the application bundle at the first time, not a final solution.
Modified:
geronimo/server/trunk/plugins/myfaces/geronimo-myfaces-builder/src/main/java/org/apache/geronimo/myfaces/deployment/MyFacesModuleBuilderExtension.java
Modified: geronimo/server/trunk/plugins/myfaces/geronimo-myfaces-builder/src/main/java/org/apache/geronimo/myfaces/deployment/MyFacesModuleBuilderExtension.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/myfaces/geronimo-myfaces-builder/src/main/java/org/apache/geronimo/myfaces/deployment/MyFacesModuleBuilderExtension.java?rev=1032859&r1=1032858&r2=1032859&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/myfaces/geronimo-myfaces-builder/src/main/java/org/apache/geronimo/myfaces/deployment/MyFacesModuleBuilderExtension.java (original)
+++ geronimo/server/trunk/plugins/myfaces/geronimo-myfaces-builder/src/main/java/org/apache/geronimo/myfaces/deployment/MyFacesModuleBuilderExtension.java Tue Nov 9 06:19:52 2010
@@ -173,6 +173,9 @@ public class MyFacesModuleBuilderExtensi
if (!hasFacesServlet(webApp)) {
return;
}
+ //This is a workaround for wired bundle cache in the BundleClassloader,
+ //add the import package explicitly to make sure the myface-core-impl is wired immediately the application bundle is resolved
+ module.getEnvironment().addImportPackage("org.apache.myfaces.ee6");
EARContext moduleContext = module.getEarContext();
Map sharedContext = module.getSharedContext();
Re: svn commit: r1032859 - /geronimo/server/trunk/plugins/myfaces/geronimo-myfaces-builder/src/main/java/org/apache/geronimo/myfaces/deployment/MyFacesModuleBuilderExtension.java
Posted by Jarek Gawor <jg...@gmail.com>.
Can you add that to the plan.xml instead of the code?
Jarek
On Tue, Nov 9, 2010 at 1:19 AM, <xu...@apache.org> wrote:
> Author: xuhaihong
> Date: Tue Nov 9 06:19:52 2010
> New Revision: 1032859
>
> URL: http://svn.apache.org/viewvc?rev=1032859&view=rev
> Log:
> Explicitly import a MyFaces package, so that MyFaces-core package could wired to the application bundle at the first time, not a final solution.
>
> Modified:
> geronimo/server/trunk/plugins/myfaces/geronimo-myfaces-builder/src/main/java/org/apache/geronimo/myfaces/deployment/MyFacesModuleBuilderExtension.java
>
> Modified: geronimo/server/trunk/plugins/myfaces/geronimo-myfaces-builder/src/main/java/org/apache/geronimo/myfaces/deployment/MyFacesModuleBuilderExtension.java
> URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/myfaces/geronimo-myfaces-builder/src/main/java/org/apache/geronimo/myfaces/deployment/MyFacesModuleBuilderExtension.java?rev=1032859&r1=1032858&r2=1032859&view=diff
> ==============================================================================
> --- geronimo/server/trunk/plugins/myfaces/geronimo-myfaces-builder/src/main/java/org/apache/geronimo/myfaces/deployment/MyFacesModuleBuilderExtension.java (original)
> +++ geronimo/server/trunk/plugins/myfaces/geronimo-myfaces-builder/src/main/java/org/apache/geronimo/myfaces/deployment/MyFacesModuleBuilderExtension.java Tue Nov 9 06:19:52 2010
> @@ -173,6 +173,9 @@ public class MyFacesModuleBuilderExtensi
> if (!hasFacesServlet(webApp)) {
> return;
> }
> + //This is a workaround for wired bundle cache in the BundleClassloader,
> + //add the import package explicitly to make sure the myface-core-impl is wired immediately the application bundle is resolved
> + module.getEnvironment().addImportPackage("org.apache.myfaces.ee6");
>
> EARContext moduleContext = module.getEarContext();
> Map sharedContext = module.getSharedContext();
>
>
>