Make use of the new varadic template feature + cosmetic changes.

Change constructor to take the a package manager argument. Add a
package manager member to the operation. Remove setter function.

Change-Id: Iffb860e24089beb6496ac4ee479d547a29463728
Reviewed-by: Kai Koehne <kai.koehne@theqtcompany.com>

1 files changed, 3 insertions, 10 deletions

diff --git a/src/libs/kdtools/kdupdaterupdateoperationfactory.cpp b/src/libs/kdtools/kdupdaterupdateoperationfactory.cpp
index 3268facbe..a74658cc2 100644
--- a/src/libs/kdtools/kdupdaterupdateoperationfactory.cpp
+++ b/src/libs/kdtools/kdupdaterupdateoperationfactory.cpp
@@ -35,10 +35,8 @@
 #include "kdupdaterupdateoperationfactory.h"
 
 #include "kdupdaterupdateoperations.h"
-#include "packagemanagercore.h"
 
 using namespace KDUpdater;
-using namespace QInstaller;
 
 /*!
     \inmodule kdupdater
@@ -64,10 +62,13 @@ using namespace QInstaller;
 */
 
 /*!
+    \obsolete
     \fn void KDUpdater::UpdateOperationFactory::registerUpdateOperation(const QString &name)
 
     Registers a new update operation with the factory based on \a name. When create() is called
     with that \a name, the update operation is constructed using its default constructor.
+
+    Deprecated. Use registerProduct() instead.
 */
 
 /*!
@@ -93,11 +94,3 @@ UpdateOperationFactory::UpdateOperationFactory()
     registerUpdateOperation<AppendFileOperation>(QLatin1String("AppendFile"));
     registerUpdateOperation<PrependFileOperation>(QLatin1String("PrependFile"));
 }
-
-UpdateOperation *UpdateOperationFactory::create(const QString &name, PackageManagerCore *core) const
-{
-    UpdateOperation *operation = KDGenericFactory<UpdateOperation>::create(name);
-    if (operation)
-        operation->setPackageManager(core);
-    return operation;
-}