ArrayIndexOutOfBoundsException in WebFileChooser

[moizhb]

While instantiating WebFileChooser I get the following error. Interestingly, I do not see this error every time. but it does happen 1 in 20 times.

Usage: fileChooser = new WebFileChooser();

Error:
java.lang.ArrayIndexOutOfBoundsException: 3
at com.alee.utils.FileUtils.getDiskRoots(FileUtils.java:375)
at com.alee.extended.filechooser.WebPathField.(WebPathField.java:101)
at com.alee.laf.filechooser.WebFileChooserPanel.createNorthContent(WebFileChooserPanel.java:432)
at com.alee.laf.filechooser.WebFileChooserPanel.(WebFileChooserPanel.java:297)
at com.alee.laf.filechooser.WebFileChooserUI.installUI(WebFileChooserUI.java:108)
at javax.swing.JComponent.setUI(JComponent.java:664)
at com.alee.laf.filechooser.WebFileChooser.updateUI(WebFileChooser.java:308)
at javax.swing.JFileChooser.setup(JFileChooser.java:370)
at javax.swing.JFileChooser.(JFileChooser.java:352)
at javax.swing.JFileChooser.(JFileChooser.java:311)
at com.alee.laf.filechooser.WebFileChooser.(WebFileChooser.java:58)

[mgarin]

This happends in this method:

    public static File[] getDiskRoots ()
    {
        final File[] roots = File.listRoots ();
        int proper = 0;
        for ( final File root : roots )
        {
            if ( root.isDirectory () )
            {
                proper++;
            }
        }
        final File[] properRoots = new File[ proper ];
        int currentIndex = 0;
        for ( final File root : roots )
        {
            if ( root.isDirectory () )
            {
                properRoots[ currentIndex ] = root;
                currentIndex++;
            }
        }
        return properRoots;
    }

My guess is that sometimes your file system (well yeh, I have never seens this exception on any of my machines with different OS) returns different value for isDirectory () call. This might happen due to some new JDK bugs (or old ones) or due to your specific system.

In any case - I'll just reduce this method usage to one call and it will be fixed. Thanks for pointing out this possible issue :)

I will include this fix into v1.28 update.

[moizhb]

Thanks alot quick response.!

[mgarin]

All changes are available in sources so you can build temporary library version using provided ANT script if you want to test it out right away or you can wait for v1.28 release which will include it.

[mgarin]

Fix is now available in v1.28 update:
https://github.com/mgarin/weblaf/releases/tag/v1.28