Merge branch 'RED-7784-flag-default' into 'master'
RED-7784 - Include unprocessed redactions in reports by default Closes RED-7784 See merge request redactmanager/persistence-service!213
This commit is contained in:
commit
477113b6ae
@ -103,6 +103,10 @@ public class DownloadController implements DownloadResource {
|
|||||||
@PreAuthorize("hasAuthority('" + PROCESS_DOWNLOAD + "')")
|
@PreAuthorize("hasAuthority('" + PROCESS_DOWNLOAD + "')")
|
||||||
public DownloadResponse prepareDownload(@RequestBody PrepareDownloadWithOptionRequest request) {
|
public DownloadResponse prepareDownload(@RequestBody PrepareDownloadWithOptionRequest request) {
|
||||||
|
|
||||||
|
if (request.getIncludeUnprocessed() == null) {
|
||||||
|
request.setIncludeUnprocessed(true);
|
||||||
|
}
|
||||||
|
|
||||||
validateDossierId(request.getDossierId());
|
validateDossierId(request.getDossierId());
|
||||||
|
|
||||||
validateAndFilterFileIds(request);
|
validateAndFilterFileIds(request);
|
||||||
@ -174,7 +178,7 @@ public class DownloadController implements DownloadResource {
|
|||||||
.downloadFileTypes(request.getDownloadFileTypes())
|
.downloadFileTypes(request.getDownloadFileTypes())
|
||||||
.reportTemplateIds(request.getReportTemplateIds())
|
.reportTemplateIds(request.getReportTemplateIds())
|
||||||
.redactionPreviewColor(request.getRedactionPreviewColor())
|
.redactionPreviewColor(request.getRedactionPreviewColor())
|
||||||
.includeUnprocessed(request.isIncludeUnprocessed())
|
.includeUnprocessed(request.getIncludeUnprocessed())
|
||||||
.build();
|
.build();
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -303,7 +307,7 @@ public class DownloadController implements DownloadResource {
|
|||||||
.dossierId(request.getDossierId())
|
.dossierId(request.getDossierId())
|
||||||
.userId(KeycloakSecurity.getUserId())
|
.userId(KeycloakSecurity.getUserId())
|
||||||
.fileIds(request.getFileIds())
|
.fileIds(request.getFileIds())
|
||||||
.includeUnprocessed(request.isIncludeUnprocessed())
|
.includeUnprocessed(request.getIncludeUnprocessed())
|
||||||
.build();
|
.build();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@ -14,6 +14,6 @@ public class DownloadJob {
|
|||||||
|
|
||||||
private String userId;
|
private String userId;
|
||||||
private String storageId;
|
private String storageId;
|
||||||
private boolean includeUnprocessed;
|
private Boolean includeUnprocessed;
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|||||||
@ -61,7 +61,7 @@ public class DownloadService {
|
|||||||
.userId(request.getUserId())
|
.userId(request.getUserId())
|
||||||
.dossierId(request.getDossierId())
|
.dossierId(request.getDossierId())
|
||||||
.fileIds(request.getFileIds())
|
.fileIds(request.getFileIds())
|
||||||
.includeUnprocessed(request.isIncludeUnprocessed())
|
.includeUnprocessed(request.getIncludeUnprocessed())
|
||||||
.downloadFileTypes(dossier.getDownloadFileTypes())
|
.downloadFileTypes(dossier.getDownloadFileTypes())
|
||||||
.reportTemplateIds(dossier.getReportTemplates().stream().map(ReportTemplateEntity::getTemplateId).collect(Collectors.toList()))
|
.reportTemplateIds(dossier.getReportTemplates().stream().map(ReportTemplateEntity::getTemplateId).collect(Collectors.toList()))
|
||||||
.build();
|
.build();
|
||||||
@ -85,7 +85,7 @@ public class DownloadService {
|
|||||||
request.getDownloadFileTypes(),
|
request.getDownloadFileTypes(),
|
||||||
request.getReportTemplateIds(),
|
request.getReportTemplateIds(),
|
||||||
request.getRedactionPreviewColor());
|
request.getRedactionPreviewColor());
|
||||||
addToDownloadQueue(DownloadJob.builder().storageId(storageId).userId(request.getUserId()).includeUnprocessed(request.isIncludeUnprocessed()).build(), 1);
|
addToDownloadQueue(DownloadJob.builder().storageId(storageId).userId(request.getUserId()).includeUnprocessed(request.getIncludeUnprocessed()).build(), 1);
|
||||||
|
|
||||||
return new JSONPrimitive<>(storageId);
|
return new JSONPrimitive<>(storageId);
|
||||||
}
|
}
|
||||||
|
|||||||
@ -52,7 +52,7 @@ public class DownloadProcessorService {
|
|||||||
.downloadId(downloadJob.getStorageId())
|
.downloadId(downloadJob.getStorageId())
|
||||||
.dossierId(dossier.getId())
|
.dossierId(dossier.getId())
|
||||||
.dossierTemplateId(dossierService.getDossierById(dossier.getId()).getDossierTemplateId())
|
.dossierTemplateId(dossierService.getDossierById(dossier.getId()).getDossierTemplateId())
|
||||||
.includeUnprocessed(downloadJob.isIncludeUnprocessed())
|
.includeUnprocessed(downloadJob.getIncludeUnprocessed())
|
||||||
.fileIds(filenameSortedFileIds)
|
.fileIds(filenameSortedFileIds)
|
||||||
.templateIds(downloadStatus.getReports().stream().map(ReportTemplateEntity::getTemplateId).collect(Collectors.toSet()))
|
.templateIds(downloadStatus.getReports().stream().map(ReportTemplateEntity::getTemplateId).collect(Collectors.toSet()))
|
||||||
.build(), 1);
|
.build(), 1);
|
||||||
|
|||||||
@ -18,6 +18,6 @@ public class PrepareDownloadRequest {
|
|||||||
|
|
||||||
private String dossierId;
|
private String dossierId;
|
||||||
private List<String> fileIds;
|
private List<String> fileIds;
|
||||||
private boolean includeUnprocessed;
|
private Boolean includeUnprocessed;
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|||||||
@ -28,6 +28,6 @@ public class PrepareDownloadWithOptionRequest {
|
|||||||
private List<String> reportTemplateIds = new ArrayList<>();
|
private List<String> reportTemplateIds = new ArrayList<>();
|
||||||
private Set<DownloadFileType> downloadFileTypes = new HashSet<>();
|
private Set<DownloadFileType> downloadFileTypes = new HashSet<>();
|
||||||
private String redactionPreviewColor;
|
private String redactionPreviewColor;
|
||||||
private boolean includeUnprocessed ;
|
private Boolean includeUnprocessed;
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|||||||
@ -17,7 +17,7 @@ public class DownloadRequest {
|
|||||||
|
|
||||||
private String userId;
|
private String userId;
|
||||||
private String dossierId;
|
private String dossierId;
|
||||||
private boolean includeUnprocessed;
|
private Boolean includeUnprocessed;
|
||||||
|
|
||||||
@Builder.Default
|
@Builder.Default
|
||||||
private List<String> fileIds = new ArrayList<>();
|
private List<String> fileIds = new ArrayList<>();
|
||||||
|
|||||||
@ -21,7 +21,7 @@ public class DownloadWithOptionRequest {
|
|||||||
|
|
||||||
private String userId;
|
private String userId;
|
||||||
private String dossierId;
|
private String dossierId;
|
||||||
private boolean includeUnprocessed;
|
private Boolean includeUnprocessed;
|
||||||
|
|
||||||
@Builder.Default
|
@Builder.Default
|
||||||
private List<String> fileIds = new ArrayList<>();
|
private List<String> fileIds = new ArrayList<>();
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user