RED-7185 - RM-46 - Error message when adjusting the Justification #165

Merged
corina.olariu.ext1 merged 2 commits from RED-7185-clone into master 2023-10-09 12:15:29 +02:00
3 changed files with 11 additions and 3 deletions

View File

@ -6,6 +6,7 @@ import java.util.ArrayList;
import java.util.List;
import java.util.UUID;
import org.apache.commons.lang3.StringUtils;
import org.springframework.beans.BeanUtils;
import org.springframework.stereotype.Service;
@ -16,6 +17,7 @@ import com.iqser.red.service.persistence.management.v1.processor.entity.dossier.
import com.iqser.red.service.persistence.management.v1.processor.entity.dossier.DossierTemplateEntity;
import com.iqser.red.service.persistence.management.v1.processor.entity.dossier.FileAttributeConfigEntity;
import com.iqser.red.service.persistence.management.v1.processor.entity.dossier.ReportTemplateEntity;
import com.iqser.red.service.persistence.management.v1.processor.exception.ConflictException;
import com.iqser.red.service.persistence.management.v1.processor.exception.NotFoundException;
import com.iqser.red.service.persistence.management.v1.processor.service.persistence.DictionaryPersistenceService;
import com.iqser.red.service.persistence.management.v1.processor.service.persistence.DossierAttributeConfigPersistenceService;
@ -66,6 +68,10 @@ public class DossierTemplateCloneService {
@Transactional
public DossierTemplateEntity cloneDossierTemplate(String dossierTemplateId, CloneDossierTemplateRequest cloneDossierTemplateRequest) {
if (StringUtils.isEmpty(cloneDossierTemplateRequest.getName())) {
throw new ConflictException("DossierTemplate name must be set");
}
dossierTemplatePersistenceService.validateDossierTemplate(cloneDossierTemplateRequest.getName(), cloneDossierTemplateRequest.getDescription());
dossierTemplatePersistenceService.validateDossierTemplateNameIsUnique(cloneDossierTemplateRequest.getName());
DossierTemplateEntity clonedDossierTemplate = new DossierTemplateEntity();

View File

@ -95,9 +95,9 @@ public class DossierTemplatePersistenceService {
}
}
private void validateDossierTemplate(String name, String description) {
public void validateDossierTemplate(String name, String description) {
if (name.length() > MAX_NAME_LENGTH) {
if (!StringUtils.isEmpty(name) && name.length() > MAX_NAME_LENGTH) {
throw new BadRequestException(String.format("The name is too long (%s), max length %s", name.length(), MAX_NAME_LENGTH));
}

View File

@ -73,7 +73,9 @@ public class EntityPerformanceTest extends AbstractPersistenceServerServiceTest
dictionaryClient.addEntry(type.getType(), type.getDossierTemplateId(), fiveKEntries, true, null, DictionaryEntryType.FALSE_POSITIVE);
t1 = System.currentTimeMillis();
var cloned = dossierTemplateClient.cloneDossierTemplate(template.getId(), new CloneDossierTemplateRequest());
CloneDossierTemplateRequest cloneRequest = new CloneDossierTemplateRequest();
cloneRequest.setName("cloneDossierTemplate");
var cloned = dossierTemplateClient.cloneDossierTemplate(template.getId(), cloneRequest);
t2 = System.currentTimeMillis();
log.info("Clone Time: {}", (t2 - t1));