[Bug][Fix]FIleList can't write to remote server
Doug Way
dway at riskmetrics.com
Thu Dec 11 05:47:10 UTC 2003
Actually, it looks like Michael Rueger's recent serverFixes-mir also
happens to fix the same problem along with fixing some other things.
(I just approved his changes but they haven't been incorporated yet.)
So we won't need both fixes, since they both fix
RemoteFileStream>>close. But I notice Ted's implementation of that
method is quite a bit more concise, so if they are both correct, maybe
we should include Ted's version of that method. :)
- Doug
close
remoteFile writable ifTrue: [
remoteFile putFile: (self as: RWBinaryOrTextStream) reset named:
remoteFile fileName]
close
| keepRemoteFile |
remoteFile
ifNil: [^self].
[keepRemoteFile := remoteFile.
remoteFile := nil.
keepRemoteFile writable ifTrue: [
keepRemoteFile reset.
keepRemoteFile putFile: self named: keepRemoteFile fileName]]
ensure: [remoteFile := keepRemoteFile]
On Thursday, December 4, 2003, at 12:33 AM, Ted Kaehler wrote:
> Folks,
>
> The FileList supports viewing files on remote servers via FTP. For
> years now, editing a file and writing it back to the server has been
> broken. (Why didn't anyone complain?) Dan finally did complain, and
> the problem was that the close message in RemoteFileStream was
> calling itself recursively. With this fix, files on remote servers
> can be edited in a FileList, and 'accept' will save them to the
> server.
>
> Change Set: FileListFTPWrite-tk
> Date: 3 December 2003
> Author: Ted Kaehler
>
>
> --Ted.
>
> --
> Ted Kaehler http://www.squeakland.org/~ted/
> (home) 3261 Montecito Drive, Las Vegas, NV 89120. voice (702) 456-7930
> In 1747 Ben Franklin proposed that electricity is a single fluid
> instead of two, and called the source of current "positive". In 1897
> J.J. Thomson showed that it is really the "negative" electrons that
> move. Luckily, all the equations are the same either way.
>
>
>
> <FileListFTPWrite-tk.1.cs><ATT03902.txt>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: text/enriched
Size: 2132 bytes
Desc: not available
Url : http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20031211/c52987d8/attachment.bin
More information about the Squeak-dev
mailing list
|