Fix issue with displayName being empty

Author: tharikaGitHub

components/apimgt/org.wso2.carbon.apimgt.rest.api.publisher.v1.common/src/main/java/org/wso2/carbon/apimgt/rest/api/publisher/v1/common/mappings/APIMappingUtil.java

@@ -226,7 +226,8 @@ public static API fromDTOtoAPI(APIDTO dto, String provider) throws APIManagement
         context = updateContextWithVersion(dto.getVersion(), originalContext, context);
         model.setContext(context);
         model.setDescription(dto.getDescription());
-        model.setDisplayName(dto.getDisplayName());
+        model.setDisplayName((dto.getDisplayName() != null && !dto.getDisplayName().trim().isEmpty())
+                ? dto.getDisplayName() : dto.getName());
 
         Object endpointConfig = dto.getEndpointConfig();
         if (endpointConfig != null) {
@@ -819,8 +820,10 @@ public static API fromMCPServerDTOtoAPI(MCPServerDTO dto, String provider) throw
             model.getMetadata().put(APIConstants.MCP.PROTOCOL_VERSION_KEY, protocolVersion);
         }
         String displayName = dto.getDisplayName();
-        if (displayName != null && !displayName.isEmpty()) {
+        if (displayName != null && !displayName.trim().isEmpty()) {
             model.setDisplayName(displayName);
+        } else {
+            model.setDisplayName(dto.getName());
         }
 
         try {
@@ -1617,7 +1620,8 @@ public static APIDTO fromAPItoDTO(API model, boolean preserveCredentials,
         }
         APIDTO dto = new APIDTO();
         dto.setName(model.getId().getApiName());
-        dto.setDisplayName(model.getDisplayName() != null ? model.getDisplayName() : model.getId().getApiName());
+        dto.setDisplayName((model.getDisplayName() != null && !model.getDisplayName().trim().isEmpty())
+                ? model.getDisplayName() : model.getId().getApiName());
         dto.setVersion(model.getId().getVersion());
         String providerName = model.getId().getProviderName();
         dto.setProvider(APIUtil.replaceEmailDomainBack(providerName));
@@ -3914,6 +3918,8 @@ public static APIProductDTO fromAPIProducttoDTO(APIProduct product, boolean pres
         productDto.setName(product.getId().getName());
         productDto.setDisplayName(
                 product.getDisplayName() != null ? product.getDisplayName() : product.getId().getName());
+        productDto.setDisplayName((product.getDisplayName() != null && !product.getDisplayName().trim().isEmpty())
+                ? product.getDisplayName() : product.getId().getName());
         productDto.setProvider(APIUtil.replaceEmailDomainBack(product.getId().getProviderName()));
         productDto.setId(product.getUuid());
         productDto.setVersion(product.getId().getVersion());
@@ -4179,7 +4185,8 @@ public static APIProduct fromDTOtoAPIProduct(APIProductDTO dto, String provider)
         product.setID(id);
         product.setUuid(dto.getId());
         product.setDescription(dto.getDescription());
-        product.setDisplayName(dto.getDisplayName() != null ? dto.getDisplayName() : dto.getName());
+        product.setDisplayName((dto.getDisplayName() != null && !dto.getDisplayName().trim().isEmpty())
+                ? dto.getDisplayName() : dto.getName());
 
         String context = dto.getContext();
         final String originalContext = context;

components/apimgt/org.wso2.carbon.apimgt.rest.api.publisher.v1.common/src/main/java/org/wso2/carbon/apimgt/rest/api/publisher/v1/common/mappings/PublisherCommonUtils.java

@@ -909,7 +909,9 @@ private static API prepareForUpdateApi(API originalAPI, APIDTO apiDtoToUpdate, A
         }
         apiToUpdate.setWsdlUrl(apiDtoToUpdate.getWsdlUrl());
         apiToUpdate.setGatewayType(apiDtoToUpdate.getGatewayType());
-        apiToUpdate.setDisplayName(apiDtoToUpdate.getDisplayName());
+        apiToUpdate.setDisplayName((apiDtoToUpdate.getDisplayName() != null
+                && !apiDtoToUpdate.getDisplayName().trim().isEmpty()) ? apiDtoToUpdate.getDisplayName()
+                : apiDtoToUpdate.getName());
 
         //validate API categories
         List<APICategory> apiCategories = apiToUpdate.getApiCategories();
@@ -1108,7 +1110,9 @@ public static API prepareForUpdateApi(API originalAPI, MCPServerDTO apiDtoToUpda
         SwaggerData swaggerData = new SwaggerData(apiToUpdate);
         String definitionToAdd = new OAS3Parser().generateAPIDefinition(swaggerData);
         apiToUpdate.setSwaggerDefinition(definitionToAdd);
-        apiToUpdate.setDisplayName(apiDtoToUpdate.getDisplayName());
+        apiToUpdate.setDisplayName((apiDtoToUpdate.getDisplayName() != null
+                && !apiDtoToUpdate.getDisplayName().trim().isEmpty()) ? apiDtoToUpdate.getDisplayName()
+                : apiDtoToUpdate.getName());
 
         apiToUpdate.setOrganization(originalAPI.getOrganization());
         apiToUpdate.setSubtype(originalAPI.getSubtype());
@@ -2935,7 +2939,6 @@ public static API prepareToCreateAPIByDTO(APIDTOTypeWrapper apiDtoTypeWrapper, A
         api.setSubtype(apiDtoTypeWrapper.getResolvedApiSubtype());
         api.setAiConfiguration(apiDtoTypeWrapper.getAiConfiguration());
         api.setInitiatedFromGateway(apiDtoTypeWrapper.getInitiatedFromGateway());
-        api.setDisplayName(apiDtoTypeWrapper.getDisplayName());
         if (apiDtoTypeWrapper.isMCPServerDTO()) {
             String protocolVersion = apiDtoTypeWrapper.getProtocolVersion();
             api.getMetadata().put(APIConstants.MCP.PROTOCOL_VERSION_KEY,