Context xml file should not be copied to domain1/conf directory when appserver = Glassfish.

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.1.2, 5.2.2
• Fix Version/s: 5.2.0, 5.2.3
• Component/s: None
• Labels: None
• Branch Version/s:
  5.1.x
• Backported to Branch:
  Committed
• Similar Issues: Show 5 results ; ;

  LPS-1217	Display name of an application is set to wrong value during Exploded Tomcat deployment through context xml file.
  LPS-1228	For Exploded Directory Deployment, war directory is not getting massaged properly when portlet.container.implementation=sun
  LPS-1783	Auto Deploy location for Glassfish doesn't match as for other appservers
  LPS-1917	Mail portlet doesn't build when appserver is glassfish
  LPS-1830	Need to provide some way to get the massaged war which can be deployed on appserver manually.

PortletExplodedTomcatListener is being used to support directory deployment through Portal Pack NetBeans Plug-in. By using this feature, instead of war file, the build directory of the project inside NetBeans gets deployed to the appserver. And any changes in the JSP/javascripts are automatically picked up by the glassfish/tomcat appserver without any redeployment.

PortletExplodedTomcatListener is currently doing the following things

1. Massage the exploded directory
2. Then copy context xml to tomcat's conf directory (Which is required for the deployment of portlet app on tomcat)

But the second step is not required when the appserver type is Glassfish. Currently the copyContextFile() copies the context xml file to Glassfish's domain1/conf directory. Because of that glassfish is throwing exception during every deployment (Though the exception is harmless).

Fix :

• In PortletExplodedTomcatListener, don't copy context xml file if the appserver type is glassfish.

[Brian Chan]

Modified the logic to be ServerDetector.isTomcat instead of !ServerDetector.isGlassfish

[satyaranjan]

The context xml is still getting copied incase of glassfish server. Beacause the server type is returned as "glassfish-tomcat" incase of glassfish container and the check ServerDetector.isTomcat() is true for glassfish also.

So the if condition should be changed to

if (ServerDetector.isTomcat() && !ServerDetector.isGlassFish()) {
copyContextFile(file);
}