]> git.saurik.com Git - wxWidgets.git/blobdiff - src/common/ftp.cpp
Fixes for previous fixes
[wxWidgets.git] / src / common / ftp.cpp
index 23aad6bf511772eb643eefd457d6f736e14c234c..83f419d5462d81d73abcec7c31979ed786b5f74f 100644 (file)
 // Created:     07/07/1997
 // RCS-ID:      $Id$
 // Copyright:   (c) 1997, 1998 Guilhem Lavaux
-// Licence:     wxWindows license
+// Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // ============================================================================
 // declarations
 // ============================================================================
 
-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
   #pragma implementation "ftp.h"
 #endif
 
@@ -32,7 +32,7 @@
   #pragma hdrstop
 #endif
 
-#if wxUSE_SOCKETS && wxUSE_STREAMS
+#if wxUSE_PROTOCOL_FTP
 
 #ifndef WX_PRECOMP
     #include <stdlib.h>
@@ -50,7 +50,7 @@
 #include "wx/protocol/ftp.h"
 
 #if defined(__WXMAC__)
-    #include "/wx/mac/macsock.h"
+    #include "wx/mac/macsock.h"
 #endif
 
 #ifndef __MWERKS__
@@ -83,7 +83,7 @@ wxFTP::wxFTP()
 {
     m_lastError = wxPROTO_NOERR;
     m_streaming = FALSE;
-       m_currentTransfermode = NONE;
+    m_currentTransfermode = NONE;
 
     m_user = wxT("anonymous");
     m_passwd << wxGetUserId() << wxT('@') << wxGetFullHostName();
@@ -365,15 +365,15 @@ bool wxFTP::SetTransferMode(TransferMode transferMode)
 
     if ( !DoSimpleCommand(_T("TYPE"), mode) )
     {
-        wxLogError(_("Failed to set FTP transfer mode to %s."),
-                   transferMode == ASCII ? _("ASCII") : _("binary"));
+        wxLogError(_("Failed to set FTP transfer mode to %s."), (const wxChar*)
+                   (transferMode == ASCII ? _("ASCII") : _("binary")));
 
         return FALSE;
     }
 
-       // If we get here the operation has been succesfully completed
-       // Set the status-member
-       m_currentTransfermode = transferMode;
+    // If we get here the operation has been succesfully completed
+    // Set the status-member
+    m_currentTransfermode = transferMode;
 
     return TRUE;
 }
@@ -507,7 +507,7 @@ public:
     {
         delete m_i_socket;
 
-        if ( LastError() == wxStream_NOERROR )
+        if ( IsOk() )
         {
             // wait for "226 transfer completed"
             m_ftp->CheckResult('2');
@@ -524,6 +524,8 @@ public:
 
     wxFTP *m_ftp;
     size_t m_ftpsize;
+
+    DECLARE_NO_COPY_CLASS(wxInputFTPStream)
 };
 
 class wxOutputFTPStream : public wxSocketOutputStream
@@ -558,6 +560,8 @@ public:
     }
 
     wxFTP *m_ftp;
+
+    DECLARE_NO_COPY_CLASS(wxOutputFTPStream)
 };
 
 wxSocketClient *wxFTP::GetPort()
@@ -571,7 +575,7 @@ wxSocketClient *wxFTP::GetPort()
         return NULL;
     }
 
-    const char *addrStart = wxStrchr(m_lastResult, _T('('));
+    const wxChar *addrStart = wxStrchr(m_lastResult, _T('('));
     if ( !addrStart )
     {
         m_lastError = wxPROTO_PROTERR;
@@ -579,7 +583,7 @@ wxSocketClient *wxFTP::GetPort()
         return NULL;
     }
 
-    const char *addrEnd = wxStrchr(addrStart, _T(')'));
+    const wxChar *addrEnd = wxStrchr(addrStart, _T(')'));
     if ( !addrEnd )
     {
         m_lastError = wxPROTO_PROTERR;
@@ -631,7 +635,7 @@ wxInputStream *wxFTP::GetInputStream(const wxString& path)
     int pos_size;
     wxInputFTPStream *in_stream;
 
-       if ( ( m_currentTransfermode == NONE ) && !SetTransferMode(BINARY) )
+    if ( ( m_currentTransfermode == NONE ) && !SetTransferMode(BINARY) )
         return NULL;
 
     wxSocketClient *sock = GetPort();
@@ -665,14 +669,14 @@ wxInputStream *wxFTP::GetInputStream(const wxString& path)
 
 wxOutputStream *wxFTP::GetOutputStream(const wxString& path)
 {
-       if ( ( m_currentTransfermode == NONE ) && !SetTransferMode(BINARY) )
+    if ( ( m_currentTransfermode == NONE ) && !SetTransferMode(BINARY) )
         return NULL;
 
     wxSocketClient *sock = GetPort();
 
     wxString tmp_str = wxT("STOR ") + path;
     if ( !CheckCommand(tmp_str, '1') )
-        return FALSE;
+        return NULL;
 
     m_streaming = TRUE;
 
@@ -722,27 +726,27 @@ bool wxFTP::GetList(wxArrayString& files,
 
 bool wxFTP::FileExists(const wxString& fileName)
 {
-       // This function checks if the file specified in fileName exists in the 
-       // current dir. It does so by simply doing an NLST (via GetList).
-       // If this succeeds (and the list is not empty) the file exists.
-       
-       bool retval = FALSE;
-       wxArrayString fileList;
-
-       if ( GetList(fileList, fileName, FALSE) )
-       {
-               // Some ftp-servers (Ipswitch WS_FTP Server 1.0.5 does this)
-               // displays this behaviour when queried on a non-existing file:
-               // NLST this_file_does_not_exist
-               // 150 Opening ASCII data connection for directory listing
-               // (no data transferred)
-               // 226 Transfer complete
+    // This function checks if the file specified in fileName exists in the
+    // current dir. It does so by simply doing an NLST (via GetList).
+    // If this succeeds (and the list is not empty) the file exists.
+
+    bool retval = FALSE;
+    wxArrayString fileList;
+
+    if ( GetList(fileList, fileName, FALSE) )
+    {
+        // Some ftp-servers (Ipswitch WS_FTP Server 1.0.5 does this)
+        // displays this behaviour when queried on a non-existing file:
+        // NLST this_file_does_not_exist
+        // 150 Opening ASCII data connection for directory listing
+        // (no data transferred)
+        // 226 Transfer complete
         // Here wxFTP::GetList(...) will succeed but it will return an empty
         // list.
         retval = !fileList.IsEmpty();
-       }
+    }
 
-       return retval;
+    return retval;
 }
 
 // ----------------------------------------------------------------------------
@@ -751,13 +755,13 @@ bool wxFTP::FileExists(const wxString& fileName)
 
 int wxFTP::GetFileSize(const wxString& fileName)
 {
-       // return the filesize of the given file if possible
-       // return -1 otherwise (predominantly if file doesn't exist
-       // in current dir)
+    // return the filesize of the given file if possible
+    // return -1 otherwise (predominantly if file doesn't exist
+    // in current dir)
+
+    int filesize = -1;
 
-       int filesize = -1;
-       
-       // Check for existance of file via wxFTP::FileExists(...)
+    // Check for existance of file via wxFTP::FileExists(...)
     if ( FileExists(fileName) )
     {
         wxString command;
@@ -767,7 +771,7 @@ int wxFTP::GetFileSize(const wxString& fileName)
         // transfermodes and reports different filesizes accordingly.
         // The BINARY size is the interesting one: How much memory
         // will we need to hold this file?
-           TransferMode oldTransfermode = m_currentTransfermode;
+        TransferMode oldTransfermode = m_currentTransfermode;
         SetTransferMode(BINARY);
         command << _T("SIZE ") << fileName;
 
@@ -777,55 +781,57 @@ int wxFTP::GetFileSize(const wxString& fileName)
         {
             // The answer should be one line: "213 <filesize>\n"
             // 213 is File Status (STD9)
-                       // "SIZE" is not described anywhere..? It works on most servers
-                       int statuscode;
-                       if ( wxSscanf(GetLastResult().c_str(), _T("%i %i"),
+            // "SIZE" is not described anywhere..? It works on most servers
+            int statuscode;
+            if ( wxSscanf(GetLastResult().c_str(), _T("%i %i"),
                           &statuscode, &filesize) == 2 )
-                       {
-                               // We've gotten a good reply.
-                               ok = TRUE; 
-                       }
-                       else
-                       {
-                               // Something bad happened.. A "2yz" reply with no size
-                               // Fallback
-                               ok = FALSE;
-                       }
+            {
+                // We've gotten a good reply.
+                ok = TRUE;
+            }
+            else
+            {
+                // Something bad happened.. A "2yz" reply with no size
+                // Fallback
+                ok = FALSE;
+            }
         }
-               
-               // Set transfermode back to the original. Only the "SIZE"-command
-               // is dependant on transfermode
+
+        // Set transfermode back to the original. Only the "SIZE"-command
+        // is dependant on transfermode
         if ( oldTransfermode != NONE )
         {
             SetTransferMode(oldTransfermode);
         }
-               
-               if ( !ok ) // this is not a direct else clause.. The size command might return an invalid "2yz" reply
+
+        // this is not a direct else clause.. The size command might return an
+        // invalid "2yz" reply
+        if ( !ok )
         {
-                       // The server didn't understand the "SIZE"-command or it
-                       // returned an invalid reply.
-                       // We now try to get details for the file with a "LIST"-command
-                       // and then parse the output from there..
-                       wxArrayString fileList;
-                       if ( GetList(fileList, fileName, TRUE) )
-                       {
-                               if ( !fileList.IsEmpty() )
-                               {
+            // The server didn't understand the "SIZE"-command or it
+            // returned an invalid reply.
+            // We now try to get details for the file with a "LIST"-command
+            // and then parse the output from there..
+            wxArrayString fileList;
+            if ( GetList(fileList, fileName, TRUE) )
+            {
+                if ( !fileList.IsEmpty() )
+                {
                     // We _should_ only get one line in return, but just to be
                     // safe we run through the line(s) returned and look for a
                     // substring containing the name we are looking for. We
                     // stop the iteration at the first occurrence of the
                     // filename. The search is not case-sensitive.
-                                       bool foundIt = FALSE;
+                    bool foundIt = FALSE;
 
                     size_t i;
-                                       for ( i = 0; !foundIt && i < fileList.Count(); i++ )
-                                       {
-                                               foundIt = fileList[i].Upper().Contains(fileName.Upper());
-                                       }
+                    for ( i = 0; !foundIt && i < fileList.Count(); i++ )
+                    {
+                        foundIt = fileList[i].Upper().Contains(fileName.Upper());
+                    }
 
-                                       if ( foundIt )
-                                       {
+                    if ( foundIt )
+                    {
                         // The index i points to the first occurrence of
                         // fileName in the array Now we have to find out what
                         // format the LIST has returned. There are two
@@ -841,89 +847,37 @@ int wxFTP::GetFileSize(const wxString& fileName)
                         // indicate Unix-style (this also limits this function
                         // to searching for files, not directories)
                         if ( fileList[i].Mid(0, 1) == _T("-") )
-                                               {
-
-                                                       if ( wxSscanf(fileList[i].c_str(),
-                                          _("%*s %*s %*s %*s %i %*s %*s %*s %*s"), 
-                                          &filesize) == 9 )
-                                                       {
-                                                               // We've gotten a good response
-                                                               ok = TRUE;
-                                                       }
-                                                       else
-                                                       {
-                                                               // Hmm... Invalid response
-                                                               wxLogTrace(FTP_TRACE_MASK,
+                        {
+
+                            if ( wxSscanf(fileList[i].c_str(),
+                                          _T("%*s %*s %*s %*s %i %*s %*s %*s %*s"),
+                                          &filesize) != 9 )
+                            {
+                                // Hmm... Invalid response
+                                wxLogTrace(FTP_TRACE_MASK,
                                            _T("Invalid LIST response"));
-                                                       }
-                                               }
-                                               else // Windows-style response (?)
-                                               {
-                                                       if ( wxSscanf(fileList[i].c_str(),
+                            }
+                        }
+                        else // Windows-style response (?)
+                        {
+                            if ( wxSscanf(fileList[i].c_str(),
                                           _T("%*s %*s %i %*s"),
-                                          &filesize) == 4 )
-                                                       {
-                                                               // valid response
-                                                               ok = TRUE;
-                                                       }
-                                                       else
-                                                       {
-                                                               // something bad happened..?
-                                                               wxLogTrace(FTP_TRACE_MASK,
+                                          &filesize) != 4 )
+                            {
+                                // something bad happened..?
+                                wxLogTrace(FTP_TRACE_MASK,
                                            _T("Invalid or unknown LIST response"));
-                                                       }
-                                               }
-                                       }
-                               }
-                       }
+                            }
+                        }
+                    }
+                }
+            }
         }
     }
 
-       // filesize might still be -1 when exiting
-       return filesize;
+    // filesize might still be -1 when exiting
+    return filesize;
 }
 
+#endif // wxUSE_PROTOCOL_FTP
 
-#ifdef WXWIN_COMPATIBILITY_2
-// deprecated
-wxList *wxFTP::GetList(const wxString& wildcard, bool details)
-{
- wxSocketBase *sock = GetPort();
- if (!sock)
-  return FALSE;
- wxList *file_list = new wxList;
- wxString line;
- // NLST : List of Filenames (including Directory's !)
- // LIST : depending on BS of FTP-Server
- //        - Unix    : result like "ls" command
- //        - Windows : like "dir" command
- //        - others  : ?
- if (!details)
-  line = _T("NLST");   // Default
- else
-  line = _T("LIST");
- if (!wildcard.IsNull())
-  line += wildcard;
- if (!CheckCommand(line, '1'))
- {
-  delete sock;
-  delete file_list;
-  return NULL;
- }
- while (GetLine(sock, line) == wxPROTO_NOERR)
- {
-  file_list->Append((wxObject *)(new wxString(line)));
- }
- if (!CheckResult('2'))
- {
-  delete sock;
-  file_list->DeleteContents(TRUE);
-  delete file_list;
-  return NULL;
- }
- return file_list;
-}
-#endif // WXWIN_COMPATIBILITY_2
-
-#endif
-  // wxUSE_SOCKETS