Attachment 73284 Details for Bug 195709 – Only enQuote the source string once

[patch] Only enQuote the source string once

patch.txt (text/plain), 1.62 KB, created by Kevin Doyle

on 2007-07-08 21:04:16 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Kevin Doyle

Created: 2007-07-08 21:04:16 EDT

Size: 1.62 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.rse.services.dstore
>Index: miners/org/eclipse/rse/dstore/universal/miners/UniversalFileSystemMiner.java
>===================================================================
>RCS file: /cvsroot/dsdp/org.eclipse.tm.rse/plugins/org.eclipse.rse.services.dstore/miners/org/eclipse/rse/dstore/universal/miners/UniversalFileSystemMiner.java,v
>retrieving revision 1.11
>diff -u -r1.11 UniversalFileSystemMiner.java
>--- miners/org/eclipse/rse/dstore/universal/miners/UniversalFileSystemMiner.java	14 Jun 2007 12:56:43 -0000	1.11
>+++ miners/org/eclipse/rse/dstore/universal/miners/UniversalFileSystemMiner.java	7 Jul 2007 16:55:18 -0000
>@@ -17,6 +17,7 @@
>  * Xuan Chen (IBM) - Fix 189487 - copy and paste a folder did not work - workbench hang
>  * Xuan Chen (IBM)        - [189681] [dstore][linux] Refresh Folder in My Home messes up Refresh in Root
>  * Xuan Chen (IBM)        - [191280] [dstore] Expand fails for folder "/folk" with 3361 children
>+ * Kevin Doyle (IBM) - [195709] Windows Copying doesn't work when path contains space
>  *******************************************************************************/
> 
> package org.eclipse.rse.dstore.universal.miners;
>@@ -337,7 +338,8 @@
> 					if (_isWindows)
> 					{
> 						tgt = tgtFolder.getAbsolutePath() + File.separatorChar + srcFile.getName();
>-						doCopyCommand(enQuote(src), enQuote(tgt), folderCopy, status);
>+						// Both unix and windows need src quoted, so it's already done
>+						doCopyCommand(src, enQuote(tgt), folderCopy, status);
> 						if (status.getAttribute(DE.A_SOURCE) == IServiceConstants.FAILED)
> 						{
> 							break;

Actions: View | Diff

Attachments on bug 195709: 73284 | 73360