Delay the execution of start(), so ServiceJob implementors don't have to worry about it.

svn path=/trunk/KDE/kdebase/workspace/libs/plasma/; revision=841266

service.cpp

@@ -21,6 +21,7 @@
 #include "private/service_p.h"
 
 #include <QFile>
+#include <QTimer>
 
 #include <KDebug>
 #include <KService>
@@ -148,7 +149,7 @@ ServiceJob* Service::startOperationCall(const KConfigGroup &description)
 
     ServiceJob *job = createJob(description.name(), params);
     connect(job, SIGNAL(finished(KJob*)), this, SLOT(jobFinished(KJob*)));
-    job->start();
+    QTimer::singleShot(0, job, SLOT(slotStart()));
     return job;
 }
 

servicejob.cpp

@@ -25,22 +25,32 @@ namespace Plasma
 class ServiceJobPrivate
 {
 public:
-    ServiceJobPrivate(const QString &dest, const QString &op, const QMap<QString, QVariant> &params)
-        : destination(dest),
+    ServiceJobPrivate(ServiceJob *owner,
+                      const QString &dest,
+                      const QString &op,
+                      const QMap<QString, QVariant> &params)
+        : q(owner),
+          destination(dest),
           operation(op),
           parameters(params)
     {
     }
+    ServiceJob* q;
     QString destination;
     QString operation;
     QMap<QString, QVariant> parameters;
     QVariant result;
+
+    void slotStart()
+    {
+        q->start();
+    }
 };
 
 ServiceJob::ServiceJob(const QString &destination, const QString &operation,
                        const QMap<QString, QVariant> &parameters, QObject *parent)
     : KJob(parent),
-      d(new ServiceJobPrivate(destination, operation, parameters))
+      d(new ServiceJobPrivate(this, destination, operation, parameters))
 {
 }
 

servicejob.h

@@ -32,6 +32,19 @@ class ServiceJobPrivate;
 
 /**
  * @brief This class provides jobs for use with Plasma::Service
+ *
+ * Unlike KJob, you can do the work in start(), since Plasma::Service already
+ * delays the call to start() until the event loop is reached.
+ *
+ * If the job is quick enough that it is not worth reporting the progress,
+ * you just need to implement start() to do the task, then call emitResult()
+ * at the end of it.  If the task does not complete successfully, you should
+ * set a non-zero error code with setError(int) and an error message with
+ * setErrorText(QString).
+ *
+ * If the job is longer (involving network access, for instance), you should
+ * report the progress at regular intervals.  See the KJob documentation for
+ * information on how to do this.
  */
 class PLASMA_EXPORT ServiceJob : public KJob
 {
@@ -41,6 +54,9 @@ public:
     /**
      * Default constructor
      *
+     * @arg destination the subject that the job is acting on
+     * @arg operation   the action that the job is performing on the @p destination
+     * @arg parameters  the parameters of the @p action
      * @arg parent      the parent object for this service
      */
     ServiceJob(const QString &destination, const QString &operation,
@@ -52,22 +68,30 @@ public:
     ~ServiceJob();
 
     /**
-     * @return the destination, if any, that this service is associated with
+     * @return the subject that the job is acting on
      */
     QString destination() const;
 
     /**
-     * @return the operation this job should perform
+     * @return the operation the job is performing on the destination
      */
     QString operationName() const;
 
     /**
-     * @return the parameters for the job
+     * @return the parameters for the operation
      */
     QMap<QString, QVariant> parameters() const;
 
     /**
-     * @return the resulting value from the operation
+     * Returns the result of the operation
+     *
+     * The result will be invalid if the job has not completed yet, or
+     * if the job does not have a meaningful result.
+     *
+     * Note that this should not be used to find out whether the operation
+     * was successful.  Instead, you should check the value of error().
+     *
+     * @return the result of the operation
      */
     QVariant result() const;
 
@@ -78,6 +102,8 @@ protected:
     void setResult(const QVariant &result);
 
 private:
+    Q_PRIVATE_SLOT(d, void slotStart())
+
     ServiceJobPrivate * const d;
 };