Skip to content

Commit 8531eb4

Browse files
authored
Merge pull request #251 from topcoder-platform/reskin-or
Reskin OR to DEV
2 parents fcde8bd + 1c9d6a2 commit 8531eb4

File tree

3 files changed

+12
-22
lines changed

3 files changed

+12
-22
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

web/includes/inc_header_reskin.jsp

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -25,17 +25,6 @@
2525
o=t.getElementsByTagName(e)[0];i.async=1;i.type="module";i.src=a;o.parentNode.insertBefore(i,o)
2626
}(window,document,"script",scriptURL,"tcUniNav");
2727
28-
var userId = `${userId}`;
29-
if (userId !== '-1') {
30-
var handle = "${userHandle}";
31-
}
32-
var initials = handle ? handle.substr(0, 2).toUpperCase() : '';
33-
34-
var user = {
35-
userId,
36-
initials,
37-
handle
38-
};
3928
4029
var signInUrl = "https://accounts-auth0.topcoder-dev.com/?retUrl=https://software.topcoder-dev.com/review"
4130
@@ -53,7 +42,7 @@
5342
type: 'tool',
5443
toolName: 'Review',
5544
toolRoot: '/',
56-
user: handle ? user : null,
45+
user: 'auto',
5746
signOut() {
5847
window.location.replace(signOutUrl)
5948
},

web/includes/project/project_phase_reskin.jsp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1180,9 +1180,9 @@
11801180
<c:choose>
11811181
<c:when test="${review.committed}">
11821182
<c:choose>
1183-
<c:when test="${isSubmitter and not group.checkpointReviewFinished}">
1183+
<%-- <c:when test="${isSubmitter and not group.checkpointReviewFinished}">
11841184
<or:text key="Pending"/>
1185-
</c:when>
1185+
</c:when> --%>
11861186
<c:when test="${isAllowedToViewCheckpointReview}">
11871187
<a href="<or:url value='/actions/ViewCheckpointReview?rid=${review.id}' />">
11881188
${orfn:displayScore(pageContext.request, review.score)}</a>

0 commit comments

Comments
 (0)