null checks
This commit is contained in:
parent
0cd97ca9a4
commit
d851e94dd6
@ -21,6 +21,11 @@ import org.apache.commons.lang3.StringUtils;
|
|||||||
import org.springframework.beans.BeanUtils;
|
import org.springframework.beans.BeanUtils;
|
||||||
import org.springframework.stereotype.Service;
|
import org.springframework.stereotype.Service;
|
||||||
|
|
||||||
|
import com.iqser.red.service.persistence.management.v1.processor.dataexchange.models.ComponentMappingImportModel;
|
||||||
|
import com.iqser.red.service.persistence.management.v1.processor.dataexchange.models.ImportTemplateResult;
|
||||||
|
import com.iqser.red.service.persistence.management.v1.processor.dataexchange.models.TemplateImportInfo;
|
||||||
|
import com.iqser.red.service.persistence.management.v1.processor.dataexchange.zipreaders.DossierTemplateArchiveReader;
|
||||||
|
import com.iqser.red.service.persistence.management.v1.processor.dataexchange.zipreaders.ZipEntryIterator;
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.entity.configuration.ColorsEntity;
|
import com.iqser.red.service.persistence.management.v1.processor.entity.configuration.ColorsEntity;
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.entity.configuration.FileAttributesGeneralConfigurationEntity;
|
import com.iqser.red.service.persistence.management.v1.processor.entity.configuration.FileAttributesGeneralConfigurationEntity;
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.entity.configuration.WatermarkEntity;
|
import com.iqser.red.service.persistence.management.v1.processor.entity.configuration.WatermarkEntity;
|
||||||
@ -32,11 +37,6 @@ import com.iqser.red.service.persistence.management.v1.processor.entity.dossier.
|
|||||||
import com.iqser.red.service.persistence.management.v1.processor.entity.dossier.ReportTemplateEntity;
|
import com.iqser.red.service.persistence.management.v1.processor.entity.dossier.ReportTemplateEntity;
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.exception.BadRequestException;
|
import com.iqser.red.service.persistence.management.v1.processor.exception.BadRequestException;
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.exception.ConflictException;
|
import com.iqser.red.service.persistence.management.v1.processor.exception.ConflictException;
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.dataexchange.models.ComponentMappingImportModel;
|
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.dataexchange.models.ImportTemplateResult;
|
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.dataexchange.models.TemplateImportInfo;
|
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.dataexchange.zipreaders.DossierTemplateArchiveReader;
|
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.dataexchange.zipreaders.ZipEntryIterator;
|
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.service.ColorsService;
|
import com.iqser.red.service.persistence.management.v1.processor.service.ColorsService;
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.service.ComponentMappingService;
|
import com.iqser.red.service.persistence.management.v1.processor.service.ComponentMappingService;
|
||||||
import com.iqser.red.service.persistence.management.v1.processor.service.DossierTemplateManagementService;
|
import com.iqser.red.service.persistence.management.v1.processor.service.DossierTemplateManagementService;
|
||||||
@ -160,7 +160,9 @@ public class DossierTemplateImportService {
|
|||||||
|
|
||||||
} catch (Exception e) {
|
} catch (Exception e) {
|
||||||
log.error("Error with message: {} occurred during import, undo import", e.getMessage(), e);
|
log.error("Error with message: {} occurred during import, undo import", e.getMessage(), e);
|
||||||
dossierTemplateManagementService.deleteDossierTemplateCompletely(templateImportInfo.getDossierTemplateId());
|
if (templateImportInfo != null && templateImportInfo.getDossierTemplateId() != null) {
|
||||||
|
dossierTemplateManagementService.deleteDossierTemplateCompletely(templateImportInfo.getDossierTemplateId());
|
||||||
|
}
|
||||||
throw e;
|
throw e;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@ -76,7 +76,10 @@ public class FileExchangeImportService {
|
|||||||
}
|
}
|
||||||
return templateImportInfo.getDossierTemplateId();
|
return templateImportInfo.getDossierTemplateId();
|
||||||
} catch (Exception e) {
|
} catch (Exception e) {
|
||||||
dossierTemplateManagementService.deleteDossierTemplateCompletely(templateImportInfo.getDossierTemplateId());
|
log.error("Error with message {} ocurred, deleting imported dossier template again.", e.getMessage());
|
||||||
|
if (templateImportInfo.getDossierTemplateId() != null) {
|
||||||
|
dossierTemplateManagementService.deleteDossierTemplateCompletely(templateImportInfo.getDossierTemplateId());
|
||||||
|
}
|
||||||
throw e;
|
throw e;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@ -43,7 +43,6 @@ public class DossierTemplatePersistenceService {
|
|||||||
final DossierTemplateRepository dossierTemplateRepository;
|
final DossierTemplateRepository dossierTemplateRepository;
|
||||||
final LegalBasisMappingPersistenceService legalBasisMappingPersistenceService;
|
final LegalBasisMappingPersistenceService legalBasisMappingPersistenceService;
|
||||||
final RulesPersistenceService rulesPersistenceService;
|
final RulesPersistenceService rulesPersistenceService;
|
||||||
final DictionaryPersistenceService dictionaryPersistenceService;
|
|
||||||
final TypeRepository typeRepository;
|
final TypeRepository typeRepository;
|
||||||
|
|
||||||
static int MAX_NAME_LENGTH = 255;
|
static int MAX_NAME_LENGTH = 255;
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user