Skip to content

Commit 90e38f6

Browse files
committed
Attempt to hide deleted uploads
1 parent d28ab2a commit 90e38f6

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

src/main/java/com/cronos/onlinereview/util/PhasesDetailsServices.java

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -465,21 +465,22 @@ private static Upload[][] getPreviousUploadsForSubmissions(HttpServletRequest re
465465
Upload[][] pastSubmissions = null;
466466
if (submissions.length > 0 && AuthorizationHelper.hasUserPermission(request, viewAllSubmissionsPermission)) {
467467

468-
// Find all deleted submissions for specified project
469-
Submission[] allDeletedSubmissions = ActionsHelper.getProjectSubmissions(project.getId(), null, "Deleted",
470-
true);
468+
// Find all undeleted submissions for specified project
469+
Submission[] allUnDeletedSubmissions = ActionsHelper.getProjectSubmissions(project.getId(), null, null,
470+
false);
471471

472472
pastSubmissions = new Upload[submissions.length][];
473473

474474
for (int j = 0; j < pastSubmissions.length; ++j) {
475475
List<Upload> temp = new ArrayList<Upload>();
476476
long currentUploadOwnerId = submissions[j].getUpload().getOwner();
477477

478-
for (Submission deletedSubmission : allDeletedSubmissions) {
479-
Upload deletedSubmissionUpload = deletedSubmission.getUpload();
480-
if (deletedSubmission.getSubmissionType().getId() == submissions[j].getSubmissionType().getId()
481-
&& deletedSubmissionUpload.getOwner() == currentUploadOwnerId) {
482-
temp.add(deletedSubmissionUpload);
478+
for (Submission unDeletedSubmission : allUnDeletedSubmissions) {
479+
Upload unDeletedSubmissionUpload = unDeletedSubmission.getUpload();
480+
if (unDeletedSubmission.getSubmissionType().getId() == submissions[j].getSubmissionType().getId()
481+
&& unDeletedSubmission.getId() != submissions[j].getId()
482+
&& unDeletedSubmissionUpload.getOwner() == currentUploadOwnerId) {
483+
temp.add(unDeletedSubmissionUpload);
483484
}
484485
}
485486

0 commit comments

Comments
 (0)