harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sergey Krivenko (JIRA)" <j...@apache.org>
Subject [jira] Updated: (HARMONY-2597) [classlib][swing] javax.swing.JFileChooser.getFileFilter() returns non-null value after calling setFileFilter(null)
Date Mon, 05 Feb 2007 09:32:06 GMT

     [ https://issues.apache.org/jira/browse/HARMONY-2597?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Sergey Krivenko updated HARMONY-2597:
-------------------------------------

    Attachment: Harmony-2597-FileChooser.patch

The attached is a fix to the bug. 

I have commented some code in MetalFileChooserUI class. It looks at least strange when propertyChange()
method invoked on registered listeners calls another fireXXXChange() method. It leads to a
loop and several changes of the property itself. 

As a result of the fix JFileChooserRTest failed but was fixed by changes in JFileChooser.setSelectedFile()
method.

> [classlib][swing] javax.swing.JFileChooser.getFileFilter() returns non-null value after
calling setFileFilter(null)
> -------------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-2597
>                 URL: https://issues.apache.org/jira/browse/HARMONY-2597
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Pavel Dolgov
>            Priority: Minor
>         Attachments: Harmony-2597-FileChooser.patch
>
>
> Test case:
> --------------
> import javax.swing.JFileChooser;
> import junit.framework.TestCase;
> public class TheTest extends TestCase {
>     public static void main(String args[]) {
>         junit.textui.TestRunner.run(TheTest.class);
>     }
>     public void testcase1() {
>         JFileChooser fc = new JFileChooser();
>         fc.setFileFilter(null);
>         assertNull(fc.getFileFilter());
>     }
> }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message