mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-07-16 13:35:22 +02:00
Code Issues Wiki Activity

[225573] [dstore] client not falling back to single operation when missing batch descriptors (due to old server)

Browse source
This commit is contained in:
David McKnight 2008-04-04 14:19:08 +00:00
parent c5d2502f09
commit d164641fe6
1 changed files with 22 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
rse/plugins/org.eclipse.rse.services.dstore/src/org/eclipse/rse/internal/services/dstore/files/DStoreFileService.java
View file

@ -39,7 +39,7 @@
* David McKnight (IBM) - [220547] [api][breaking] SimpleSystemMessage needs to specify a message id and some messages should be shared * David McKnight (IBM) - [220547] [api][breaking] SimpleSystemMessage needs to specify a message id and some messages should be shared
* Radoslav Gerganov (ProSyst) - [216195] [dstore] Saving empty file fails * Radoslav Gerganov (ProSyst) - [216195] [dstore] Saving empty file fails
* David McKnight (IBM) - [220379] [api] Provide a means for contributing custom BIDI encodings * David McKnight (IBM) - [220379] [api] Provide a means for contributing custom BIDI encodings
* David McKnight (IBM) - [225573] dstore] client not falling back to single operation when missing batch descriptors (due to old server) * David McKnight (IBM) - [225573] [dstore] client not falling back to single operation when missing batch descriptors (due to old server)
*******************************************************************************/ *******************************************************************************/
package org.eclipse.rse.internal.services.dstore.files; package org.eclipse.rse.internal.services.dstore.files;
@ -1449,10 +1449,31 @@ public class DStoreFileService extends AbstractDStoreService implements IFileSer
} }
DataElement de = getElementFor(oldPath); DataElement de = getElementFor(oldPath);
// new servers use the full path
de.setAttribute(DE.A_SOURCE, newPath); de.setAttribute(DE.A_SOURCE, newPath);
DataElement status = dsStatusCommand(de, IUniversalDataStoreConstants.C_RENAME, monitor); DataElement status = dsStatusCommand(de, IUniversalDataStoreConstants.C_RENAME, monitor);
if (status != null && status.getAttribute(DE.A_SOURCE).equals("failed"))
{
// in the patch for bug 196211, a change was made to the UniversalFileSystemMiner that
// had the rename expecting the full path in the A_SOURCE attribute itself
// prior to that, we constructed the target file name from the value and the source
// I must have missed that while reviewing
// this is our attempt at recovering, but doing the operation with the old
// format. Old servers only used the name for the source attribute on C_RENAME
de.setAttribute(DE.A_SOURCE, newName);
// trying again
status = dsStatusCommand(de, IUniversalDataStoreConstants.C_RENAME, monitor);
}
if (status == null) return false; if (status == null) return false;
if (null != monitor && monitor.isCanceled()) if (null != monitor && monitor.isCanceled())
{ {