Commit 56b195e4 by lepdou

Bugfix: check application namespace existed

parent d010aa59
...@@ -46,18 +46,19 @@ public class AppNamespaceService { ...@@ -46,18 +46,19 @@ public class AppNamespaceService {
@Transactional @Transactional
public void createDefaultAppNamespace(String appId) { public void createDefaultAppNamespace(String appId) {
if (!isAppNamespaceNameUnique(appId, appId)) { if (!isAppNamespaceNameUnique(appId, ConfigConsts.NAMESPACE_APPLICATION)) {
throw new ServiceException("appnamespace not unique"); throw new BadRequestException(String.format("App already has application namespace. AppId = %s", appId));
} }
AppNamespace appNs = new AppNamespace(); AppNamespace appNs = new AppNamespace();
appNs.setAppId(appId); appNs.setAppId(appId);
appNs.setName(ConfigConsts.NAMESPACE_APPLICATION); appNs.setName(ConfigConsts.NAMESPACE_APPLICATION);
appNs.setComment("default app namespace"); appNs.setComment("default app namespace");
appNs.setFormat(ConfigFileFormat.Properties.getValue()); appNs.setFormat(ConfigFileFormat.Properties.getValue());
String userId = userInfoHolder.getUser().getUserId(); String userId = userInfoHolder.getUser().getUserId();
appNs.setDataChangeCreatedBy(userId); appNs.setDataChangeCreatedBy(userId);
appNs.setDataChangeLastModifiedBy(userId); appNs.setDataChangeLastModifiedBy(userId);
appNamespaceRepository.save(appNs); appNamespaceRepository.save(appNs);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment