[packagekit] [PATCH 4/4] Get detail info for dependency packages

[Richard Hughes]

On 26 September 2010 04:04, Zhang Qiang <qiang.z.zhang at intel.com> wrote:
> +                       package_ids = pk_package_ids_from_string (package_id);
> +                       /* get the details of dependency packages */
> +                       pk_client_get_update_detail_async (PK_CLIENT(priv->task), package_ids, priv->cancellable,
> +                                                  (PkProgressCallback) gpk_update_viewer_progress_cb, update_viewer,
> +                                                  (GAsyncReadyCallback) gpk_update_viewer_get_update_detail_cb, update_viewer);
> +                       /* get the size of dependency packages */
> +                       pk_client_get_details_async (PK_CLIENT(priv->task), package_ids, priv->cancellable,
> +                                                    (PkProgressCallback) gpk_update_viewer_progress_cb, update_viewer,
> +                                                    (GAsyncReadyCallback) gpk_update_viewer_get_details_cb, update_viewer);
> +                       g_strfreev(package_ids);

I think this needs to be in the:

		/* update icon */
		path = gpk_update_viewer_model_get_path (model, package_id);
		if (path == NULL) {

...block. This will ensure that only packages no previously in the
list have the details requested.

Richard.