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

Bug 324192

Summary:

Cannot open a renamed file

Product:

[Tools] Target Management

Reporter:

Samuel Wu <samuelwu>

Component:

RSE

Assignee:

David McKnight <dmcknigh>

Status:

RESOLVED FIXED

QA Contact:

Martin Oberhuber <mober.at+eclipse>

Severity:

major

Priority:

CC:

dmcknigh, kjdoyle

Version:

unspecified

Flags:

mober.at+eclipse: pmc_approved+
kjdoyle: review+

Target Milestone:

3.2.2

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
patch to update the editable on rename even if it's not open	none

Description Samuel Wu

2010-09-01 09:38:25 EDT

Build Identifier: org.eclipse.rse.core_3.2.1.v201008191400-7a7FFMVFC7sRd_S_c_dcsY

This is a regression problem we didn't see with RSE 3.0.x


Reproducible: Always

Steps to Reproduce:
1) Double click on a local file D:\temp\test.txt to open it in the editor.
2) Close the file.
3) Hit F2 and rename the file to D:\temp\test1.txt.
4) Double click on the renamed file.
RSEF1002
Operation failed. File system input or output error
Message reported from file system: D:\temp\test.txt (The system cannot find the file specified.)

When the above steps were tried on linux host, the file open action kept running and never ended.

Comment 1 David McKnight

2010-09-01 10:02:45 EDT

Created attachment 177954 [details]
patch to update the editable on rename even if it's not open

Comment 2 David McKnight

2010-09-01 10:03:49 EDT

We don't hit the problem if the editor isn't closed but, in the case where the editor is closed, we aren't updating the corresponding SystemEditableRemoteFile during the rename operation.  Can you try with this patch?

Comment 3 Samuel Wu

2010-09-01 10:56:27 EDT

Thank you, Dave, for the prompt response. The patch fixed the problem.

Comment 4 David McKnight

2010-09-01 11:01:38 EDT

Kevin, could you review this one?  Thanks.

Comment 5 Kevin Doyle

2010-09-01 11:52:14 EDT

Review +.

Comment 6 David McKnight

2010-09-01 12:12:28 EDT

Thanks for the review.  I've committed the change to cvs.

Comment 7 Martin Oberhuber

2010-09-08 04:21:29 EDT

Released for 3.2.1rc4.

Comment 8 Martin Oberhuber

2010-11-09 08:16:06 EST

Verified that this has been released to 3.3m3 as well.