Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 401326

Summary: No refresh of folders after "Link to Server"
Product: [ECD] Orion Reporter: Simon Kaegi <simon_kaegi>
Component: ServerAssignee: Susan McCourt <susan>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: ken_walker, mamacdon, simon_kaegi
Version: 2.0Flags: mamacdon: review+
simon_kaegi: review+
Target Milestone: 2.0 RC3   
Hardware: PC   
OS: Windows 7   
Whiteboard:

Description Simon Kaegi CLA 2013-02-20 10:56:20 EST
I just installed RC3 for my local server setup. 

I was using an orion.conf with ...
orion.file.allowedPaths=C:\\workspaces

and in the Link to Server dialog entered:
workspace
C:\workspaces

which gave no feedback and I had to refresh to show the new folder.
Comment 1 Susan McCourt CLA 2013-02-20 13:07:31 EST
This was most likely introduced when removing a dojo.hitch.
The UI refresh code was bound to the wrong object.

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=af2777ca4f1d2416173290c852ac3eb44dc719c0
Branch is bug401326
Comment 2 Mark Macdonald CLA 2013-02-20 13:19:41 EST
this
> function() {
>     ex.loadResourceList.bind(ex)(ex.treeRoot.Path, true);
> }, // refresh the root

is equivalent to just

> function() {
>     ex.loadResourceList(ex.treeRoot.Path, true);
> }, // refresh the root

But either way it's fine, +1
Comment 3 Simon Kaegi CLA 2013-02-20 14:27:21 EST
What Mark said... +1

Alternately  -- ex.loadResourceList.bind(ex, ex.treeRoot.Path, true);

But for now having ...

function() {
  ex.loadResourceList(ex.treeRoot.Path, true);
}

... is probably the simplest and safest
Comment 4 Simon Kaegi CLA 2013-02-21 00:43:31 EST
Fixed as per comment 2
Comment 5 Susan McCourt CLA 2013-02-21 10:49:09 EST
thanks, Simon.