diff options
author | shugo <shugo@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-10-28 04:09:20 +0000 |
---|---|---|
committer | shugo <shugo@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-10-28 04:09:20 +0000 |
commit | 91c84cda9b4c12be09424202f0ea30b7d4a91e0b (patch) | |
tree | 477b8a4398e5458a3341d7210188e2fe47d98a8c | |
parent | 249569fd8e6ba2c10082ab3eaaff6d1230998f01 (diff) | |
download | ruby-91c84cda9b4c12be09424202f0ea30b7d4a91e0b.tar.gz |
* lib/net/ftp.rb (Net::FTP#initialize): sets @binary to true.
* lib/net/ftp.rb (Net::FTP#binary=): sends a TYPE command only when
logged in. [ruby-dev:39548]
* lib/net/ftp.rb (Net::FTP#send_type_command): new private method
which sends an appropriate TYPE command according to the value of
@binary.
* lib/net/ftp.rb (Net::FTP#login): calls send_type_command instead
of binary=.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@25519 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 14 | ||||
-rw-r--r-- | lib/net/ftp.rb | 17 |
2 files changed, 28 insertions, 3 deletions
@@ -1,3 +1,17 @@ +Wed Oct 28 13:02:10 2009 Shugo Maeda <shugo@ruby-lang.org> + + * lib/net/ftp.rb (Net::FTP#initialize): sets @binary to true. + + * lib/net/ftp.rb (Net::FTP#binary=): sends a TYPE command only when + logged in. [ruby-dev:39548] + + * lib/net/ftp.rb (Net::FTP#send_type_command): new private method + which sends an appropriate TYPE command according to the value of + @binary. + + * lib/net/ftp.rb (Net::FTP#login): calls send_type_command instead + of binary=. + Tue Oct 27 22:46:44 2009 NARUSE, Yui <naruse@ruby-lang.org> * lib/net/ftp.rb (Net::FTP#initialize): @sock = nil. diff --git a/lib/net/ftp.rb b/lib/net/ftp.rb index 239bd3cbc6..c35ee368af 100644 --- a/lib/net/ftp.rb +++ b/lib/net/ftp.rb @@ -128,11 +128,12 @@ module Net # def initialize(host = nil, user = nil, passwd = nil, acct = nil) super() - @binary = false + @binary = true @passive = false @debug_mode = false @resume = false @sock = nil + @logged_in = false if host connect(host) if user @@ -144,10 +145,19 @@ module Net def binary=(newmode) if newmode != @binary @binary = newmode - @binary ? voidcmd("TYPE I") : voidcmd("TYPE A") unless closed? + send_type_command if @logged_in end end + def send_type_command + if @binary + voidcmd("TYPE I") + else + voidcmd("TYPE A") + end + end + private :send_type_command + def with_binary(newmode) oldmode = binary self.binary = newmode @@ -393,7 +403,8 @@ module Net raise FTPReplyError, resp end @welcome = resp - self.binary = true + send_type_command + @logged_in = true end # |