5 files changed, 125 insertions, 25 deletions

diff --git a/kabc/addresseeview.cpp b/kabc/addresseeview.cpp
index aae923c..5c24acf 100644
--- a/kabc/addresseeview.cpp
+++ b/kabc/addresseeview.cpp
@@ -141,5 +141,7 @@ void AddresseeView::setAddressee( const KABC::Addressee& mAddressee )
 
   QString name =  mAddressee.realName();
-
+  QString assName = mAddressee.assembledName();
+  if ( assName.isEmpty() )
+      assName = name;
   QString dynamicPart;
 
@@ -155,5 +157,5 @@ void AddresseeView::setAddressee( const KABC::Addressee& mAddressee )
            "<td align=\"left\"><a href=\"mailto:%2 <%3> \">%4</a></td></tr>" )
           .arg( type )
-          .arg( name )
+          .arg( assName )
           .arg( *emailIt )
           .arg( *emailIt );
diff --git a/korganizer/koeventviewer.cpp b/korganizer/koeventviewer.cpp
index 976ee2c..02b54da 100644
--- a/korganizer/koeventviewer.cpp
+++ b/korganizer/koeventviewer.cpp
@@ -58,7 +58,7 @@
 #include <qpaintdevicemetrics.h>
 #else //DESKTOP_VERSION
-#include <externalapphandler.h>
 #include <qtopia/qcopenvelope_qws.h> 
 #endif //DESKTOP_VERSION
+#include <externalapphandler.h>
 
 KOEventViewer::KOEventViewer(QWidget *parent,const char *name)
@@ -174,5 +174,5 @@ void KOEventViewer::setSource(const QString& n)
     if ( n.left(6) == "mailto" ) {
         // qDebug("KOEventViewer::setSource %s ", n.mid(7).latin1());
-#ifndef DESKTOP_VERSION
+        //#ifndef DESKTOP_VERSION
         if ( n.mid(7,3) == "ALL" ) {
             mailToAttendees( true );
@@ -180,8 +180,9 @@ void KOEventViewer::setSource(const QString& n)
             mailToAttendees( false );
         } else {
-        QCopEnvelope e("QPE/Application/ompi", "newMail(QString)" );
-        e << n.mid(7);
+       ExternalAppHandler::instance()->mailToOneContact( n.mid(7) );
+       //QCopEnvelope e("QPE/Application/ompi", "newMail(QString)" );
+       //e << n.mid(7);
         }
-#endif
+        //#endif
 
     }
@@ -240,12 +241,19 @@ void KOEventViewer::mailToAttendees( bool all )
         if ( !all && !a->RSVP() ) continue;
         if (!a->email().isEmpty()) {
-            nameList.append (a->name() );
+#ifndef DESKTOP_VERSION
+            nameList.append (a->realName() );
             emailList.append (a->email() );
             uidList.append (a->uid() );
+#else
+            emailList.append(a->realName() +" <" + a->email() +">");
+#endif
         }
     }
-    QString uid = "ComposeMailUIpick2"+mMailSubject;
 #ifndef DESKTOP_VERSION
+    QString uid = "ComposeMailUIpick2"+mMailSubject;
     bool res = ExternalAppHandler::instance()->returnNameEmailUidListFromKAPI("QPE/Application/ompi", uid, nameList, emailList, uidList);
+
+#else
+    ExternalAppHandler::instance()->mailToMultipleContacts( emailList.join(","), mMailSubject );
 #endif
 
@@ -671,5 +679,5 @@ void KOEventViewer::formatAttendees(Incidence *event)
             if (!a->email().isEmpty()) {
                 if (iconPath) {
-                    mText += "<a href=\"mailto:" + a->name() +" <" + a->email() + ">:" + mMailSubject + "\">";
+                    mText += "<a href=\"mailto:" + a->realName() +" <" + a->email() + ">:" + mMailSubject + "\">";
                     if ( a->RSVP() ) {
                         ++a_count_nr;
diff --git a/libkcal/person.cpp b/libkcal/person.cpp
index aca28c2..858805d 100644
--- a/libkcal/person.cpp
+++ b/libkcal/person.cpp
@@ -50,5 +50,11 @@ bool KCal::operator==( const Person& p1, const Person& p2 )
 }
 
-
+QString Person::realName() const
+{
+    int ccc = mName.find (',');
+    if ( ccc < 0 )
+        return mName;
+    return mName.mid( ccc+1 ).stripWhiteSpace() + " " + mName.left( ccc ).stripWhiteSpace();
+}
 QString Person::fullName() const
 {
diff --git a/libkcal/person.h b/libkcal/person.h
index c46c5f0..3cec153 100644
--- a/libkcal/person.h
+++ b/libkcal/person.h
@@ -36,4 +36,5 @@ class Person
     void setName(const QString &);
     QString name() const { return mName; }
+    QString realName() const;
     
     void setEmail(const QString &);
diff --git a/libkdepim/externalapphandler.cpp b/libkdepim/externalapphandler.cpp
index 2ce6926..edefda4 100644
--- a/libkdepim/externalapphandler.cpp
+++ b/libkdepim/externalapphandler.cpp
@@ -40,4 +40,5 @@ $Id$
 #else
 #include <qapplication.h>
+#include <qprocess.h>
 #endif
 
@@ -435,4 +436,23 @@ void ExternalAppHandler::loadConfig()
   addDefaultAppItem(ExternalAppHandler::EMAIL, KPimGlobalPrefs::OTHER_EMC,  "Userdefined email client",       undefined,                undefined,        undefined, undefined, undefined);
 
+#ifdef DESKTOP_VERSION
+  QString appPath;
+#ifdef _WIN32_
+  appPath = "C:\\Programme\\Mozilla Thunderbird\\thunderbird.exe";
+#else
+  appPath = "/usr/bin/thunderbird";
+#endif
+  addDefaultAppItem(ExternalAppHandler::EMAIL, KPimGlobalPrefs::OMPI_EMC,   "Mozilla Thunderbird",   appPath,    "-compose", "to=%1 <%2>",       ",",   "subject=%1");
+
+#ifdef _WIN32_
+  appPath = "C:\\Programme\\Mozilla\\mozilla.exe";
+#else
+  appPath = "/usr/bin/mozilla";
+#endif
+  addDefaultAppItem(ExternalAppHandler::EMAIL, KPimGlobalPrefs::QTOPIA_EMC,   "Mozilla Suite",   appPath,    "-mail -compose", "to=%1 <%2>",       ",",   "subject=%1");
+
+
+
+#else
   if (( QFile::exists( qtopiapath + "/bin/ompi" )) ||
      ( QFile::exists( opiepath + "/bin/ompi" )) ||
@@ -449,5 +469,5 @@ void ExternalAppHandler::loadConfig()
   if ( QFile::exists( opiepath + "/bin/mailit" ))
     addDefaultAppItem(ExternalAppHandler::EMAIL, KPimGlobalPrefs::OPIE_MAILIT_EMC,   "Opie mailit email client",   "QPE/Application/mailit", mailmsg1, "%1;%2",       mailmsg2,    "TO=%1;ATTACHMENT=%2");
-
+#endif
 
 
@@ -529,5 +549,5 @@ DefaultAppItem* ExternalAppHandler::getDefaultItem(Types type, int clientid)
 bool ExternalAppHandler::isEmailAppAvailable()
 {
-#ifndef DESKTOP_VERSION
+
   if (mEmailAppAvailable == UNDEFINED)
   {
@@ -540,7 +560,4 @@ bool ExternalAppHandler::isEmailAppAvailable()
   return (mEmailAppAvailable == AVAILABLE);
 
-#else //DESKTOP_VERSION
-  return false;
-#endif //DESKTOP_VERSION
 }
 
@@ -689,5 +706,59 @@ bool ExternalAppHandler::mailToMultipleContacts( const QString& emails, const QS
 
 #else
-  KMessageBox::sorry( 0, i18n( "This version does not support the sending of emails." ) );
+  //qDebug("mtmc %s %s ", emails.latin1(), urls.latin1());
+
+  QString channel;
+  QString message2;
+  QString parameters2;
+  QString message;
+  QString parameters;
+
+
+  int client = KPimGlobalPrefs::instance()->mEmailClient;
+  if (client == KPimGlobalPrefs::OTHER_EMC)
+  {
+    channel = KPimGlobalPrefs::instance()->mEmailOtherChannel;
+    message2 = KPimGlobalPrefs::instance()->mEmailOtherMessage;
+    parameters2 = KPimGlobalPrefs::instance()->mEmailOtherMessageParameters;
+  }
+  else
+  {
+    DefaultAppItem* dai = getDefaultItem(EMAIL, client);
+    if (!dai)
+    {
+      qDebug("could not find configured email application.");
+          return false;
+    }
+    channel = dai->_channel;
+    message2 = dai->_message2;
+    parameters2 = dai->_parameters2;
+    message = dai->_message;
+    parameters = dai->_parameters;
+  }
+
+  //first check if one of the mailers need the emails right in the message.
+  message2 = translateMessage(message2, emails, urls);
+#ifdef DEBUG_EXT_APP_HANDLER
+  qDebug("4Using QCopEnvelope e(\"%s\",\"%s\")", channel.latin1(), message2.latin1());
+  qDebug("passing emailadresses(%s), attachmenturls(%s) as parameters in the form %s to QCopEnvelope", emails.latin1() , urls.latin1(), parameters2.latin1());
+#endif
+  qDebug("%s ---  %s   %s --- %s %s", channel.latin1(), message.latin1(),message2.latin1(), parameters.latin1(), parameters2.latin1() );
+  //KMessageBox::sorry( 0, message2 );
+  QProcess * proc = new QProcess( this );
+  QStringList list = QStringList::split( " ", message );
+  int i = 0;
+  proc->addArgument( channel );
+  while ( i < list.count ( ) ) {
+      //qDebug("add%sdd ",list[i].stripWhiteSpace().latin1() );
+      proc->addArgument( list[i].stripWhiteSpace() );
+      ++i;
+  }
+  parameters2 = translateMessage(parameters2, urls, "" ); 
+  QString arg = "to='%1'";
+  arg = arg.arg( emails ) + ","+parameters2;;
+ 
+  //qDebug("2add%sdd ",arg.latin1() );
+  proc->addArgument( arg);
+  proc->launch("");
 #endif
 
@@ -703,5 +774,5 @@ bool ExternalAppHandler::mailToMultipleContacts( const QString& emails, const QS
 bool ExternalAppHandler::mailToOneContact( const QString& name, const QString& emailadress )
 {
-#ifndef DESKTOP_VERSION
+
   QString channel;
   QString message;
@@ -729,9 +800,10 @@ bool ExternalAppHandler::mailToOneContact( const QString& name, const QString& e
   }
 
-
+#ifdef DESKTOP_VERSION
+  //message =  channel + " " +message + " \""+ parameters + "\"";
+#endif
   //first check if one of the mailers need the emails right in the message.
   message = translateMessage(message, name, emailadress);
 
-
 #ifdef DEBUG_EXT_APP_HANDLER
   qDebug("5Using QCopEnvelope e(\"%s\",\"%s\")", channel.latin1(), message.latin1());
@@ -739,15 +811,26 @@ bool ExternalAppHandler::mailToOneContact( const QString& name, const QString& e
 #endif
 
+#ifndef DESKTOP_VERSION
   QCopEnvelope e(channel.latin1(), message.latin1());
     //US we need no names in the To field. The emailadresses are enough
 
   passParameters(&e, parameters, name, emailadress);
-
-
-#else
-  KMessageBox::sorry( 0, i18n( "This version does not support the sending of emails." ) );
+#else // DESKTOP_VERSION
+
+  //KMessageBox::sorry( 0, message );
+  QProcess * proc = new QProcess( this );
+  QStringList list = QStringList::split( " ", message );
+  int i = 0;
+  proc->addArgument( channel );
+  while ( i < list.count ( ) ) {
+      //qDebug("add%sdd ",list[i].latin1() );
+      proc->addArgument( list[i] );
+      ++i;
+  }
+  parameters = translateMessage(parameters, name, emailadress);
+  proc->addArgument( parameters );
+  proc->launch("");
 #endif
 
-
   return true;
 }