Skip to content
Snippets Groups Projects
Commit 01468337 authored by srosse's avatar srosse
Browse files

OO-4260: check boundaries of rubric list (or reload container)

parent 602725d5
No related branches found
No related tags found
No related merge requests found
...@@ -647,13 +647,15 @@ public class RubricEditorController extends FormBasicController implements PageE ...@@ -647,13 +647,15 @@ public class RubricEditorController extends FormBasicController implements PageE
private void swapSliders(int i, int j) { private void swapSliders(int i, int j) {
List<Slider> rubricSliders = rubric.getSliders(); List<Slider> rubricSliders = rubric.getSliders();
Slider tempRubricSlider = rubricSliders.get(i); if(i >= 0 && j >= 0 && i < rubricSliders.size() && j < rubricSliders.size()) {
rubricSliders.set(i, rubricSliders.get(j)); Slider tempRubricSlider = rubricSliders.get(i);
rubricSliders.set(j, tempRubricSlider); rubricSliders.set(i, rubricSliders.get(j));
rubricSliders.set(j, tempRubricSlider);
SliderRow tempSlider = sliders.get(i);
sliders.set(i, sliders.get(j)); SliderRow tempSlider = sliders.get(i);
sliders.set(j, tempSlider); sliders.set(i, sliders.get(j));
sliders.set(j, tempSlider);
}
} }
private void doRemoveSlider(SliderRow row) { private void doRemoveSlider(SliderRow row) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment