Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 366776 - The RSE implementation of IRemoteUIFileManager doesn't implement the showConnections method
Summary: The RSE implementation of IRemoteUIFileManager doesn't implement the showConn...
Status: RESOLVED FIXED
Alias: None
Product: PTP
Classification: Tools
Component: RDT (show other bugs)
Version: 5.0.2   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 5.0.5   Edit
Assignee: Greg Watson CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-14 21:03 EST by Corey Ashford CLA
Modified: 2012-01-11 00:50 EST (History)
0 users

See Also:

Attachments
Patch which implements the showConnections() method for RSE (2.08 KB, patch)
2011-12-14 21:05 EST, Corey Ashford CLA 		g.watson: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Corey Ashford CLA 2011-12-14 21:03:28 EST 
Build Identifier: M20110909-1335

the method showConnections is supposed to allow the file and directory browser methods to display a combo box of available connections.

This is implemented in the remotetools implementation, but for some reason, not in the RSE implementation.


Reproducible: Always

Steps to Reproduce:
Create a IRemoteUIFileManager object using a URI containing the "rse" scheme and then call showConnections(true) on that object.  Next, call browserFile(...) and notice that you do not get a connection chooser.
Comment 1 Corey Ashford CLA 2011-12-14 21:05:21 EST 
Created attachment 208418 [details]
Patch which implements the showConnections() method for RSE
Comment 2 Corey Ashford CLA 2012-01-10 13:45:30 EST 
Can we get this patch committed for 5.0.5?
Comment 3 Greg Watson CLA 2012-01-10 14:59:17 EST 
Committed to ptp_5_0 and master, with the exception that onlyConnection is initialized to true to preserve the current behavior.
Comment 4 Corey Ashford CLA 2012-01-11 00:50:44 EST 
Thanks for dealing with this so quickly, and also thanks for catching that error.

We need these changes for our remote tool launcher.