mirror of
https://github.com/eclipse-cdt/cdt
synced 2025-07-25 18:05:33 +02:00
[199854] Fix regressions in unit tests
This commit is contained in:
parent
ad14cc2ddf
commit
128bc5d5f8
1 changed files with 16 additions and 14 deletions
|
@ -1207,6 +1207,7 @@ public class LocalFileService extends AbstractFileService implements ILocalServi
|
||||||
{
|
{
|
||||||
File sourceFolderOrFile = new File(srcParent, srcName);
|
File sourceFolderOrFile = new File(srcParent, srcName);
|
||||||
File targetFolder = new File(tgtParent, tgtName);
|
File targetFolder = new File(tgtParent, tgtName);
|
||||||
|
boolean movedOk = false;
|
||||||
boolean sourceIsVirtual = ArchiveHandlerManager.isVirtual(sourceFolderOrFile.getAbsolutePath());
|
boolean sourceIsVirtual = ArchiveHandlerManager.isVirtual(sourceFolderOrFile.getAbsolutePath());
|
||||||
boolean targetIsVirtual = ArchiveHandlerManager.isVirtual(targetFolder.getAbsolutePath());
|
boolean targetIsVirtual = ArchiveHandlerManager.isVirtual(targetFolder.getAbsolutePath());
|
||||||
boolean targetIsArchive = ArchiveHandlerManager.getInstance().isArchive(targetFolder);
|
boolean targetIsArchive = ArchiveHandlerManager.getInstance().isArchive(targetFolder);
|
||||||
|
@ -1219,25 +1220,26 @@ public class LocalFileService extends AbstractFileService implements ILocalServi
|
||||||
{
|
{
|
||||||
File fileToMove = new File(srcParent, srcName);
|
File fileToMove = new File(srcParent, srcName);
|
||||||
File newFile = new File(tgtParent, tgtName);
|
File newFile = new File(tgtParent, tgtName);
|
||||||
fileToMove.renameTo(newFile);
|
//Try plain Java Filesystem move first
|
||||||
return;
|
movedOk = fileToMove.renameTo(newFile);
|
||||||
}
|
}
|
||||||
|
|
||||||
copy(srcParent, srcName, tgtParent, tgtName, monitor);
|
if (!movedOk)
|
||||||
try
|
|
||||||
{
|
{
|
||||||
delete(srcParent, srcName, monitor);
|
copy(srcParent, srcName, tgtParent, tgtName, monitor);
|
||||||
}
|
try {
|
||||||
catch (SystemMessageException exc)
|
delete(srcParent, srcName, monitor);
|
||||||
{
|
} catch (SystemMessageException exc)
|
||||||
if (monitor.isCanceled())
|
|
||||||
{
|
{
|
||||||
//This mean the copy operation is ok, but delete operation has been cancelled by user.
|
if (monitor.isCanceled())
|
||||||
//The delete() call will take care of recovered from the cancel operation.
|
{
|
||||||
//So we need to make sure to remove the already copied file/folder.
|
//This mean the copy operation is ok, but delete operation has been cancelled by user.
|
||||||
delete(tgtParent, tgtName, null);
|
//The delete() call will take care of recovered from the cancel operation.
|
||||||
|
//So we need to make sure to remove the already copied file/folder.
|
||||||
|
delete(tgtParent, tgtName, null);
|
||||||
|
}
|
||||||
|
throw exc;
|
||||||
}
|
}
|
||||||
throw exc;
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Add table
Reference in a new issue