diff --git a/application-home/conf/datasources/ds-sites.xml b/application-home/conf/datasources/ds-sites.xml index 8145efd..070523e 100644 --- a/application-home/conf/datasources/ds-sites.xml +++ b/application-home/conf/datasources/ds-sites.xml @@ -100,7 +100,7 @@ - + reload diff --git a/application-home/conf/events/ev-sites.xml b/application-home/conf/events/ev-sites.xml index 36d069b..e2660bf 100644 --- a/application-home/conf/events/ev-sites.xml +++ b/application-home/conf/events/ev-sites.xml @@ -87,12 +87,12 @@ - + - diff --git a/application-home/conf/pages/pg-sites.xml b/application-home/conf/pages/pg-sites.xml index 514e665..593e86b 100644 --- a/application-home/conf/pages/pg-sites.xml +++ b/application-home/conf/pages/pg-sites.xml @@ -380,7 +380,7 @@ ${act} - ${siteid} + ${sitename} diff --git a/src/main/java/org/appng/application/manager/business/Sites.java b/src/main/java/org/appng/application/manager/business/Sites.java index 4ad9406..cebfb49 100644 --- a/src/main/java/org/appng/application/manager/business/Sites.java +++ b/src/main/java/org/appng/application/manager/business/Sites.java @@ -24,11 +24,8 @@ import org.appng.api.FieldProcessor; import org.appng.api.InvalidConfigurationException; import org.appng.api.Options; -import org.appng.api.Platform; import org.appng.api.Request; -import org.appng.api.Scope; import org.appng.api.model.Application; -import org.appng.api.model.Properties; import org.appng.api.model.Site; import org.appng.application.manager.MessageConstants; import org.appng.application.manager.form.SiteForm; @@ -85,9 +82,7 @@ public void perform(Site site, Application application, Environment environment, service.reloadSite(request, application, siteId, fp); okMessage = MessageConstants.SITE_RELOADED; } else if (ACTION_RELOAD_TEMPLATE.equals(action)) { - Properties platformProps = environment.getAttribute(Scope.PLATFORM, - Platform.Environment.PLATFORM_CONFIG); - service.reloadTemplate(siteId, platformProps); + service.reloadTemplate(environment, options.getString(SITE, "sitename")); okMessage = MessageConstants.SITE_TEMPLATE_RELOADED; } else if (ACTION_RELOAD_PLATFORM.equals(action)) { errorMessage = MessageConstants.PLATFORM_RELOAD_ERROR; diff --git a/src/main/java/org/appng/application/manager/service/ManagerService.java b/src/main/java/org/appng/application/manager/service/ManagerService.java index 85b44a5..e44e522 100644 --- a/src/main/java/org/appng/application/manager/service/ManagerService.java +++ b/src/main/java/org/appng/application/manager/service/ManagerService.java @@ -63,7 +63,6 @@ import org.appng.api.model.NameProvider; import org.appng.api.model.Nameable; import org.appng.api.model.Permission; -import org.appng.api.model.Properties; import org.appng.api.model.Property; import org.appng.api.model.Resource; import org.appng.api.model.ResourceType; @@ -113,8 +112,8 @@ import org.appng.core.model.RepositoryUtils; import org.appng.core.service.CoreService; import org.appng.core.service.InitializerService; -import org.appng.core.service.PlatformProperties; import org.appng.core.service.MigrationService.MigrationStatus; +import org.appng.core.service.PlatformProperties; import org.appng.core.service.PropertySupport; import org.appng.core.xml.repository.PackageVersions; import org.appng.core.xml.repository.Packages; @@ -1613,8 +1612,7 @@ public String startSite(Request request, Application application, Integer siteId throws BusinessException { SiteImpl site = getSite(siteId); Environment env = request.getEnvironment(); - Map siteMap = env.getAttribute(Scope.PLATFORM, Platform.Environment.SITES); - Site activeSite = siteMap.get(site.getName()); + Site activeSite = RequestUtil.getSiteByName(env, site.getName()); if (null == activeSite || SiteState.STOPPED.equals(activeSite.getState())) { if (site.isActive()) { try { @@ -1638,8 +1636,7 @@ public String stopSite(Request request, Application application, Integer siteId, throws BusinessException { SiteImpl site = getSite(siteId); Environment env = request.getEnvironment(); - Map siteMap = env.getAttribute(Scope.PLATFORM, Platform.Environment.SITES); - Site activeSite = siteMap.get(site.getName()); + Site activeSite = RequestUtil.getSiteByName(env, site.getName()); if (null != activeSite) { if (SiteState.STARTED.equals(activeSite.getState())) { getInitializerService(application).shutDownSite(env, site, false); @@ -1847,10 +1844,8 @@ public void resetConnection(Integer conId) { } @Override - public void reloadTemplate(Integer siteId, Properties platformConfig) { - SiteImpl site = siteRepository.findOne(siteId); - initSiteProperties(site); - refreshTemplate(site, PlatformProperties.get(platformConfig)); + public void reloadTemplate(Environment env, String siteName) { + refreshTemplate(RequestUtil.getSiteByName(env, siteName), PlatformProperties.get(env)); } public SiteApplication getSiteApplication(Integer siteId, Integer appId) { diff --git a/src/main/java/org/appng/application/manager/service/Service.java b/src/main/java/org/appng/application/manager/service/Service.java index d951830..b6ba2cc 100644 --- a/src/main/java/org/appng/application/manager/service/Service.java +++ b/src/main/java/org/appng/application/manager/service/Service.java @@ -30,7 +30,6 @@ import org.appng.api.model.Application; import org.appng.api.model.Identifier; import org.appng.api.model.Permission; -import org.appng.api.model.Properties; import org.appng.api.model.ResourceType; import org.appng.api.model.Site; import org.appng.api.model.Subject; @@ -251,6 +250,6 @@ String addArchiveToRepository(Request request, Integer repositoryId, FormUpload void createEvent(Type type, String message); - void reloadTemplate(Integer siteId, Properties platformProps); + void reloadTemplate(Environment env, String siteName); } diff --git a/src/test/resources/xml/SitesTest-testShowSites.xml b/src/test/resources/xml/SitesTest-testShowSites.xml index ce6cb7b..c0c71c2 100644 --- a/src/test/resources/xml/SitesTest-testShowSites.xml +++ b/src/test/resources/xml/SitesTest-testShowSites.xml @@ -1,218 +1,218 @@ - - Sites - - - - - - - - - - - - - - - - - - - - - - - - - - - - led_green - led_red - - - - led_green - led_red - - - - - - - - - - - - - - - new - - - - - - - - - - activate - - - - - - - - deactivate - Stop site #{name}? - - - - - - - edit - - - - - - - - delete - Do you really want to delete the site "#{name}"? - - - - - - - - reload - Reload site #{name}? - - - - - - - - reload - - - - - - - Name - - - Domain - - - Active - - - - - - localhost - - - localhost - - - localhost - - - - - - true - led_green - - - true - led_green - - - - - - - - deactivate - Stop site localhost? - - - - edit - - - - reload - Reload site localhost? - - - - reload - - - - - - site2 - - - example.com - - - example.com - - - a description - - - true - led_green - - - false - led_red - - - - - - - - activate - - - - edit - - - - delete - Do you really want to delete the site "site2"? - - - - reload - - - - - + + Sites + + + + + + + + + + + + + + + + + + + + + + + + + + + + led_green + led_red + + + + led_green + led_red + + + + + + + + + + + + + + + new + + + + + + + + + + activate + + + + + + + + deactivate + Stop site #{name}? + + + + + + + edit + + + + + + + + delete + Do you really want to delete the site "#{name}"? + + + + + + + + reload + Reload site #{name}? + + + + + + + + reload + + + + + + + Name + + + Domain + + + Active + + + + + + localhost + + + localhost + + + localhost + + + + + + true + led_green + + + true + led_green + + + + + + + + deactivate + Stop site localhost? + + + + edit + + + + reload + Reload site localhost? + + + + reload + + + + + + site2 + + + example.com + + + example.com + + + a description + + + true + led_green + + + false + led_red + + + + + + + + activate + + + + edit + + + + delete + Do you really want to delete the site "site2"? + + + + reload + + + + + diff --git a/src/test/resources/xml/SitesTest-testShowSitesFiltered.xml b/src/test/resources/xml/SitesTest-testShowSitesFiltered.xml index e6b182e..8c5a45c 100644 --- a/src/test/resources/xml/SitesTest-testShowSitesFiltered.xml +++ b/src/test/resources/xml/SitesTest-testShowSitesFiltered.xml @@ -1,173 +1,173 @@ - - Sites - - - - - site - example - true - - - - - - - - - - - - - - - - - - - - - led_green - led_red - - - - led_green - led_red - - - - - - - - - - - - - - - new - - - - - - - - - - activate - - - - - - - - deactivate - Stop site #{name}? - - - - - - - edit - - - - - - - - delete - Do you really want to delete the site "#{name}"? - - - - - - - - reload - Reload site #{name}? - - - - - - - - reload - - - - - - - Name - - - Domain - - + + Sites + + + + + site + example + true + + + + + + + + + + + + + + + + + + + + + led_green + led_red + + + + led_green + led_red + + + + + + + + + + + + + + + new + + + + + + + + + + activate + + + + + + + + deactivate + Stop site #{name}? + + + + + + + edit + + + + + + + + delete + Do you really want to delete the site "#{name}"? + + + + + + + + reload + Reload site #{name}? + + + + + + + + reload + + + + + + + Name + + + Domain + + Active - - - - - site2 - - - example.com - - - example.com - - - a description - - - true - led_green - - - false - led_red - - - - - - - - activate - - - - edit - - - - delete - Do you really want to delete the site "site2"? - - - - reload - - - - - + + + + + site2 + + + example.com + + + example.com + + + a description + + + true + led_green + + + false + led_red + + + + + + + + activate + + + + edit + + + + delete + Do you really want to delete the site "site2"? + + + + reload + + + + +