[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdeextragear-libs-1/kipi-plugins/sendimages
From:       Renchi Raju <renchi () pooh ! tam ! uiuc ! edu>
Date:       2004-12-22 2:30:43
Message-ID: 20041222023043.3C6971CF31 () office ! kde ! org
[Download RAW message or body]

CVS commit by pahlibar: 


fix issue with sending images with non-latin characters in filepath. Use
QFile::encodeName() if you want to deal with a non qt/kde tool
this code is a nightmare to read without proper indentation


  M +72 -72    sendimages.cpp   1.18


--- kdeextragear-libs-1/kipi-plugins/sendimages/sendimages.cpp  #1.17:1.18
@@ -347,5 +347,5 @@ void SendImages::invokeMailAgent(void)
 
        for ( KURL::List::Iterator it = m_filesSendList.begin() ; it != m_filesSendList.end() ; ++it )
-           *m_mailAgentProc << (*it).path();
+            *m_mailAgentProc << QFile::encodeName((*it).path());
 
        if ( m_mailAgentProc->start() == false )
@@ -365,5 +365,5 @@ void SendImages::invokeMailAgent(void)
            {
            *m_mailAgentProc << "-a";
-           *m_mailAgentProc << (*it).path();
+            *m_mailAgentProc << QFile::encodeName((*it).path());
            }
 
@@ -384,5 +384,5 @@ void SendImages::invokeMailAgent(void)
            {
            Temp.append("&attach=");
-           Temp.append( (*it).path() );
+            Temp.append( QFile::encodeName((*it).path()) );
            }
 
@@ -418,5 +418,5 @@ void SendImages::invokeMailAgent(void)
            {
            Temp.append( "file://" );
-           Temp.append( (*it).path() );
+            Temp.append( QFile::encodeName((*it).path()) );
            Temp.append( "," );
            }
@@ -673,5 +673,5 @@ void SendImages::slotMozillaTimeout(void
         {
         Temp.append( "file://" );
-        Temp.append( (*it).path() );
+        Temp.append( QFile::encodeName((*it).path()) );
         Temp.append( "," );
         }


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic