yurloc / iypt-planner

Helps organizers of IYPT tournament schedule juries
4 stars 0 forks source link

Cannot submit selected file in bias computation file browser #56

Closed yurloc closed 8 years ago

yurloc commented 8 years ago

last_dir_bug_crop

yurloc commented 8 years ago

This was caused by using . as the initial value of MRU (last used) directory and a faulty validation in the file browser sheet that checks selected file's parent == root directory of the sheet:

java.lang.IllegalArgumentException
    at org.apache.pivot.wtk.FileBrowser.setSelectedFiles(FileBrowser.java:290)
    at org.apache.pivot.wtk.skin.terra.TerraFileBrowserSkin$9.selectedRangesChanged(TerraFileBrowserSkin.java:835)
    at org.apache.pivot.wtk.TableView$TableViewSelectionListenerList.selectedRangesChanged(TableView.java:988)
    at org.apache.pivot.wtk.TableView.setSelectedRanges(TableView.java:1496)
    at org.apache.pivot.wtk.TableView.setSelectedRange(TableView.java:1434)
    at org.apache.pivot.wtk.TableView.setSelectedIndex(TableView.java:1419)
    at org.apache.pivot.wtk.skin.terra.TerraTableViewSkin.mouseDown(TerraTableViewSkin.java:1246)
    at org.apache.pivot.wtk.Component$ComponentMouseButtonListenerList.mouseDown(Component.java:481)

This piece of code probably is not ready to handle relative root directory (such as .):

289:            if (!file.getParentFile().equals(rootDirectory)) {
290:                throw new IllegalArgumentException();
291:            }