<Swing Dev> [10] RFR JDK-8184272 : Jemmy JFileChooserOperator.enterSubDir does not navigate to sub directory

Vikrant Agarwal vikrant.v.agarwal at oracle.com
Fri Jul 14 03:39:03 UTC 2017


Hi Shura,

 

Yes, I will prepare a parallel fix to Jemmy code-tools repo once we send the OpenJDK fix for commit.

 

Best Regards,

Vikrant

 

From: Alexandre (Shura) Iline 
Sent: Thursday, July 13, 2017 8:51 PM
To: Vikrant Agarwal
Cc: swing-dev at openjdk.java.net; Sergey Bylokhov; Semyon Sadetsky; Srinivas Mandalika; Muneer Kolarkunnu
Subject: Re: <Swing Dev> [10] RFR JDK-8184272 : Jemmy JFileChooserOperator.enterSubDir does not navigate to sub directory

 

Will you be preparing a parallel fix to the Jemmy code-tools repo?

 

Shura

 

On Jul 13, 2017, at 2:50 AM, Vikrant Agarwal <HYPERLINK "mailto:vikrant.v.agarwal at oracle.com"vikrant.v.agarwal at oracle.com> wrote:

 

Hi All,

Please review the following:

 

Bug : HYPERLINK "https://bugs.openjdk.java.net/browse/JDK-8184272"JDK-8184272

 

JDK 10 Webrev: http://cr.openjdk.java.net/~vagarwal/8184272/webrev.00/

 

Summary: 

Jemmy JFileChooserOperator.enterSubDir does not navigate to sub directory and instead goes to user's default directory

 

The present code for JFileChooserOperator.enterSubDir(..) first selects the directory, then tries to navigate to the selected directory by getting the path of selected directory from getSelectedFile().

JFileChooserOperator.getSelectedFile() calls JFileChooser.getSelectedFile(), which returns the path for the selected file, but if the selection is a directory, then it returns Null if the fileSelectionMode is not set to select directory. 
This when passed to setCurrentDirectory() sets the diectory as user's default directory.

 

Doing setFileSelectionMode(JFileChooser.FILES_AND_DIRECTORIES) ensures that the getSelectedFile() returns the correct path even in case it is a directory.

Best Regards,

Vikrant

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20170713/45fb5891/attachment.html>


More information about the swing-dev mailing list