diff --git a/backends/dummy/pk-backend-dummy.c b/backends/dummy/pk-backend-dummy.c index 8ce7775..9ed8ab6 100644 --- a/backends/dummy/pk-backend-dummy.c +++ b/backends/dummy/pk-backend-dummy.c @@ -46,6 +46,7 @@ static gboolean _use_eula = FALSE; static gboolean _use_media = FALSE; static gboolean _use_gpg = FALSE; static gboolean _use_distro_upgrade = FALSE; +static PkBitfield _filters = 0; /** * backend_initialize: @@ -612,13 +613,17 @@ backend_resolve (PkBackend *backend, PkBitfield filters, gchar **packages) /* each one has a different detail for testing */ len = g_strv_length (packages); for (i=0; i& packages) +{ + RUN_TRANSACTION(SimulateInstallFiles(Util::packageListToPids(packages))) +} + +Transaction* Client::simulateInstallPackage(Package* package) +{ + return simulateInstallPackages(QList() << package); +} + +Transaction* Client::simulateRemovePackages(const QList& packages) +{ + RUN_TRANSACTION(SimulateRemovePackages(Util::packageListToPids(packages))) +} + +Transaction* Client::simulateRemovePackage(Package* package) +{ + return simulateRemovePackages(QList() << package); +} + +Transaction* Client::simulateUpdatePackages(const QList& packages) +{ + RUN_TRANSACTION(SimulateUpdatePackages(Util::packageListToPids(packages))) +} + +Transaction* Client::simulateUpdatePackage(Package* package) +{ + return simulateUpdatePackages(QList() << package); +} + Transaction* Client::updatePackages(bool only_trusted, const QList& packages) { RUN_TRANSACTION(UpdatePackages(only_trusted, Util::packageListToPids(packages))) diff --git a/lib/packagekit-qt/src/client.h b/lib/packagekit-qt/src/client.h index d46bb65..6243ad7 100644 --- a/lib/packagekit-qt/src/client.h +++ b/lib/packagekit-qt/src/client.h @@ -83,37 +83,41 @@ public: * \sa getActions */ typedef enum { - ActionCancel = 0x00000001, - ActionGetDepends = 0x00000002, - ActionGetDetails = 0x00000004, - ActionGetFiles = 0x00000008, - ActionGetPackages = 0x00000010, - ActionGetRepoList = 0x00000020, - ActionGetRequires = 0x00000040, - ActionGetUpdateDetail = 0x00000080, - ActionGetUpdates = 0x00000100, - ActionInstallFiles = 0x00000200, - ActionInstallPackages = 0x00000400, - ActionInstallSignature = 0x00000800, - ActionRefreshCache = 0x00001000, - ActionRemovePackages = 0x00002000, - ActionRepoEnable = 0x00004000, - ActionRepoSetData = 0x00008000, - ActionResolve = 0x00010000, - ActionRollback = 0x00020000, - ActionSearchDetails = 0x00040000, - ActionSearchFile = 0x00080000, - ActionSearchGroup = 0x00100000, - ActionSearchName = 0x00200000, - ActionUpdatePackages = 0x00400000, - ActionUpdateSystem = 0x00800000, - ActionWhatProvides = 0x01000000, - ActionAcceptEula = 0x02000000, - ActionDownloadPackages = 0x04000000, - ActionGetDistroUpgrades = 0x08000000, - ActionGetCategories = 0x10000000, - ActionGetOldTransactions = 0x20000000, - UnknownAction = 0x40000000 + ActionCancel = 0x000000001, + ActionGetDepends = 0x000000002, + ActionGetDetails = 0x000000004, + ActionGetFiles = 0x000000008, + ActionGetPackages = 0x000000010, + ActionGetRepoList = 0x000000020, + ActionGetRequires = 0x000000040, + ActionGetUpdateDetail = 0x000000080, + ActionGetUpdates = 0x000000100, + ActionInstallFiles = 0x000000200, + ActionInstallPackages = 0x000000400, + ActionInstallSignature = 0x000000800, + ActionRefreshCache = 0x000001000, + ActionRemovePackages = 0x000002000, + ActionRepoEnable = 0x000004000, + ActionRepoSetData = 0x000008000, + ActionResolve = 0x000010000, + ActionRollback = 0x000020000, + ActionSearchDetails = 0x000040000, + ActionSearchFile = 0x000080000, + ActionSearchGroup = 0x000100000, + ActionSearchName = 0x000200000, + ActionUpdatePackages = 0x000400000, + ActionUpdateSystem = 0x000800000, + ActionWhatProvides = 0x001000000, + ActionAcceptEula = 0x002000000, + ActionDownloadPackages = 0x004000000, + ActionGetDistroUpgrades = 0x008000000, + ActionGetCategories = 0x010000000, + ActionGetOldTransactions = 0x020000000, + ActionSimulateInstallFiles = 0x040000000, + ActionSimulateInstallPackages = 0x080000000, + ActionSimulateRemovePackages = 0x100000000, + ActionSimulateUpdatePackages = 0x200000000, + UnknownAction = 0x400000000 } Action; Q_DECLARE_FLAGS(Actions, Action); @@ -708,6 +712,46 @@ public: Package* searchFromDesktopFile(const QString& path); /** + * \brief Simulates an installation of \p files. + * + * You should call this method before installing \p files + * \note: This method might emit packages with INSTALLING, REMOVING, UPDATING, + * REINSTALLING or OBSOLETING status. + */ + Transaction* simulateInstallFiles(const QStringList& files); + Transaction* simulateInstallFile(const QString& file); + + /** + * \brief Simulates an installation of \p packages. + * + * You should call this method before installing \p packages + * \note: This method might emit packages with INSTALLING, REMOVING, UPDATING, + * REINSTALLING or OBSOLETING status. + */ + Transaction* simulateInstallPackages(const QList& packages); + Transaction* simulateInstallPackage(Package* package); + + /** + * \brief Simulates a removal of \p packages. + * + * You should call this method before removing \p packages + * \note: This method might emit packages with INSTALLING, REMOVING, UPDATING, + * REINSTALLING or OBSOLETING status. + */ + Transaction* simulateRemovePackages(const QList& packages); + Transaction* simulateRemovePackage(Package* package); + + /** + * \brief Simulates an update of \p packages. + * + * You should call this method before updating \p packages + * \note: This method might emit packages with INSTALLING, REMOVING, UPDATING, + * REINSTALLING or OBSOLETING status. + */ + Transaction* simulateUpdatePackages(const QList& packages); + Transaction* simulateUpdatePackage(Package* package); + + /** * Update the given \p packages */ Transaction* updatePackages(bool only_trusted, const QList& packages); diff --git a/lib/packagekit-qt/src/transactionproxy.h b/lib/packagekit-qt/src/transactionproxy.h index 68e9a33..015091a 100644 --- a/lib/packagekit-qt/src/transactionproxy.h +++ b/lib/packagekit-qt/src/transactionproxy.h @@ -277,6 +277,34 @@ public Q_SLOTS: // METHODS return callWithArgumentList(QDBus::Block, QLatin1String("SetLocale"), argumentList); } + inline QDBusReply SimulateInstallFiles(const QStringList &full_paths) + { + QList argumentList; + argumentList << qVariantFromValue(full_paths); + return callWithArgumentList(QDBus::Block, QLatin1String("SimulateInstallFiles"), argumentList); + } + + inline QDBusReply SimulateInstallPackages(const QStringList &package_ids) + { + QList argumentList; + argumentList << qVariantFromValue(package_ids); + return callWithArgumentList(QDBus::Block, QLatin1String("SimulateInstallPackages"), argumentList); + } + + inline QDBusReply SimulateRemovePackages(const QStringList &package_ids) + { + QList argumentList; + argumentList << qVariantFromValue(package_ids); + return callWithArgumentList(QDBus::Block, QLatin1String("SimulateRemovePackages"), argumentList); + } + + inline QDBusReply SimulateUpdatePackages(const QStringList &package_ids) + { + QList argumentList; + argumentList << qVariantFromValue(package_ids); + return callWithArgumentList(QDBus::Block, QLatin1String("SimulateUpdatePackages"), argumentList); + } + inline QDBusReply UpdatePackages(bool only_trusted, const QStringList &package_ids) { QList argumentList; @@ -287,7 +315,7 @@ public Q_SLOTS: // METHODS inline QDBusReply UpdateSystem(bool only_trusted) { QList argumentList; - argumentList << qVariantFromValue (only_trusted); + argumentList << qVariantFromValue (only_trusted); return callWithArgumentList(QDBus::Block, QLatin1String("UpdateSystem"), argumentList); } diff --git a/src/pk-transaction.c b/src/pk-transaction.c index 6a80e63..de1e2c6 100644 --- a/src/pk-transaction.c +++ b/src/pk-transaction.c @@ -3131,7 +3131,7 @@ pk_transaction_install_files (PkTransaction *transaction, gboolean only_trusted, GError *error; GError *error_local = NULL; PkServicePack *service_pack; - gchar *content_type = NULL; + gchar *content_type; guint length; guint i; @@ -3148,7 +3148,7 @@ pk_transaction_install_files (PkTransaction *transaction, gboolean only_trusted, "InstallFiles not yet supported by backend"); pk_transaction_release_tid (transaction); pk_transaction_dbus_return_error (context, error); - goto out; + return; } /* check if the sender is the same */ @@ -3156,7 +3156,7 @@ pk_transaction_install_files (PkTransaction *transaction, gboolean only_trusted, if (!ret) { /* don't release tid */ pk_transaction_dbus_return_error (context, error); - goto out; + return; } /* check all files exists and are valid */ @@ -3170,7 +3170,7 @@ pk_transaction_install_files (PkTransaction *transaction, gboolean only_trusted, "No such file %s", full_paths[i]); pk_transaction_release_tid (transaction); pk_transaction_dbus_return_error (context, error); - goto out; + return; } /* get content type */ @@ -3180,17 +3180,18 @@ pk_transaction_install_files (PkTransaction *transaction, gboolean only_trusted, "Failed to get content type for file %s", full_paths[i]); pk_transaction_release_tid (transaction); pk_transaction_dbus_return_error (context, error); - goto out; + return; } /* supported content type? */ ret = pk_transaction_is_supported_content_type (transaction, content_type); + g_free (content_type); if (!ret) { error = g_error_new (PK_TRANSACTION_ERROR, PK_TRANSACTION_ERROR_MIME_TYPE_NOT_SUPPORTED, "MIME type '%s' not supported %s", content_type, full_paths[i]); pk_transaction_release_tid (transaction); pk_transaction_dbus_return_error (context, error); - goto out; + return; } /* valid */ @@ -3204,7 +3205,7 @@ pk_transaction_install_files (PkTransaction *transaction, gboolean only_trusted, pk_transaction_release_tid (transaction); pk_transaction_dbus_return_error (context, error); g_error_free (error_local); - goto out; + return; } } } @@ -3214,7 +3215,7 @@ pk_transaction_install_files (PkTransaction *transaction, gboolean only_trusted, if (!ret) { pk_transaction_release_tid (transaction); pk_transaction_dbus_return_error (context, error); - goto out; + return; } /* save so we can run later */ @@ -3224,8 +3225,6 @@ pk_transaction_install_files (PkTransaction *transaction, gboolean only_trusted, /* return from async with success */ pk_transaction_dbus_return (context); -out: - g_free (content_type); return; }