Fix warnings

This commit is contained in:
Andreas Svanberg 2024-10-24 10:15:15 +02:00
parent 6bcf7dce3f
commit 1756c9829e
4 changed files with 3 additions and 5 deletions

@ -66,7 +66,7 @@ public class AdminGradingTemplateEditPage extends AbstractAdminProjectPage imple
add(form);
}
public static GradingReportTemplateUpdate toUpdate(EditingGradingTemplate editingGradingTemplate) {
static GradingReportTemplateUpdate toUpdate(EditingGradingTemplate editingGradingTemplate) {
List<GradingReportTemplateUpdate.GradeLimit> gradeLimits = editingGradingTemplate
.getGradeLimits()
.getGradeLimits()

@ -87,8 +87,7 @@ public class AdminGradingTemplatePage extends AbstractAdminProjectPage implement
}
private List<GradingCriterionTemplate> getCriteria(GradingReportTemplate gradingReportTemplate) {
ArrayList<GradingCriterionTemplate> criteria = new ArrayList<>();
gradingReportTemplate.getCriteria().forEach(criteria::add);
ArrayList<GradingCriterionTemplate> criteria = new ArrayList<>(gradingReportTemplate.getCriteria());
criteria.sort(Comparator.comparing(GradingCriterionTemplate::getSortOrder));
return Collections.unmodifiableList(criteria);
}

@ -8,7 +8,6 @@ import org.apache.wicket.markup.html.list.ListItem;
import org.apache.wicket.markup.html.list.ListView;
import org.apache.wicket.model.IModel;
import org.apache.wicket.model.LoadableDetachableModel;
import org.apache.wicket.model.Model;
import se.su.dsv.scipro.admin.pages.AbstractAdminProjectPage;
import se.su.dsv.scipro.components.NonEmptyLabel;
import se.su.dsv.scipro.grading.GradingReportTemplateService;

@ -86,7 +86,7 @@ class EditingGradingTemplate implements Serializable {
this.titleEn = criteria.getTitleEn();
this.pointsRequiredToPass = criteria.getPointsRequiredToPass();
for (var point : criteria.getGradingCriterionPointTemplates()) {
if (point.getPoint() == 0) continue;;
if (point.getPoint() == 0) continue;
Point editingPoint = new Point(point);
this.points.add(editingPoint);
}