[gnome-control-center] printers: Make error messages clearer
- From: Marek KaÅÃk <mkasik src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-control-center] printers: Make error messages clearer
- Date: Mon, 10 Dec 2012 16:06:32 +0000 (UTC)
commit 2bd01a70d347da0ea31e9bc13e6963158df99272
Author: Marek Kasik <mkasik redhat com>
Date: Mon Dec 10 16:50:58 2012 +0100
printers: Make error messages clearer
Point to cups-pk-helper for errors originating from cups-pk-helper.
https://bugzilla.gnome.org/show_bug.cgi?id=689983
panels/printers/pp-new-printer.c | 9 +++------
panels/printers/pp-utils.c | 32 ++++++++++++++++----------------
2 files changed, 19 insertions(+), 22 deletions(-)
---
diff --git a/panels/printers/pp-new-printer.c b/panels/printers/pp-new-printer.c
index 953f547..3da5117 100644
--- a/panels/printers/pp-new-printer.c
+++ b/panels/printers/pp-new-printer.c
@@ -409,8 +409,8 @@ printer_add_real_async_dbus_cb (GObject *source_object,
GAsyncResult *res,
gpointer user_data)
{
- PpNewPrinter *printer;
- PpNewPrinterPrivate *priv;
+ PpNewPrinter *printer = (PpNewPrinter *) user_data;
+ PpNewPrinterPrivate *priv = printer->priv;
GVariant *output;
GError *error = NULL;
@@ -426,7 +426,7 @@ printer_add_real_async_dbus_cb (GObject *source_object,
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
{
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: addition of printer %s failed: %s", priv->name, ret_error);
}
g_variant_unref (output);
@@ -442,9 +442,6 @@ printer_add_real_async_dbus_cb (GObject *source_object,
error->domain != G_IO_ERROR ||
error->code != G_IO_ERROR_CANCELLED)
{
- printer = (PpNewPrinter *) user_data;
- priv = printer->priv;
-
get_named_dest_async (priv->name,
printer_add_real_async_cb,
printer);
diff --git a/panels/printers/pp-utils.c b/panels/printers/pp-utils.c
index b34ee5e..54cbeda 100644
--- a/panels/printers/pp-utils.c
+++ b/panels/printers/pp-utils.c
@@ -579,7 +579,7 @@ printer_rename (const gchar *old_name,
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: rename of printer %s to %s failed: %s", old_name, new_name, ret_error);
g_variant_unref (output);
}
@@ -672,7 +672,7 @@ printer_set_location (const gchar *printer_name,
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: setting of location for printer %s failed: %s", printer_name, ret_error);
else
result = TRUE;
@@ -730,7 +730,7 @@ printer_set_accepting_jobs (const gchar *printer_name,
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: setting of acceptance of jobs for printer %s failed: %s", printer_name, ret_error);
else
result = TRUE;
g_variant_unref (output);
@@ -783,7 +783,7 @@ printer_set_enabled (const gchar *printer_name,
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: setting of enablement of printer %s failed: %s", printer_name, ret_error);
else
result = TRUE;
@@ -836,7 +836,7 @@ printer_delete (const gchar *printer_name)
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: removing of printer %s failed: %s", printer_name, ret_error);
else
result = TRUE;
@@ -901,7 +901,7 @@ printer_set_default (const gchar *printer_name)
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: setting default printer to %s failed: %s", printer_name, ret_error);
else
result = TRUE;
@@ -964,7 +964,7 @@ printer_set_shared (const gchar *printer_name,
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: setting of sharing of printer %s failed: %s", printer_name, ret_error);
else
result = TRUE;
@@ -1019,7 +1019,7 @@ printer_set_job_sheets (const gchar *printer_name,
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: setting of job sheets for printer %s failed: %s", printer_name, ret_error);
else
result = TRUE;
@@ -1087,7 +1087,7 @@ printer_set_policy (const gchar *printer_name,
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: setting of a policy for printer %s failed: %s", printer_name, ret_error);
else
result = TRUE;
@@ -1161,7 +1161,7 @@ printer_set_users (const gchar *printer_name,
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: setting of access list for printer %s failed: %s", printer_name, ret_error);
else
result = TRUE;
@@ -1215,7 +1215,7 @@ class_add_printer (const gchar *class_name,
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: adding of printer %s to class %s failed: %s", printer_name, class_name, ret_error);
else
result = TRUE;
@@ -1382,7 +1382,7 @@ printer_set_default_media_size (const gchar *printer_name)
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: setting of media size for printer %s failed: %s", printer_name, ret_error);
g_variant_unref (output);
}
@@ -1666,7 +1666,7 @@ printer_set_ppd_async_dbus_cb (GObject *source_object,
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: setting of driver for printer %s failed: %s", data->printer_name, ret_error);
else
result = TRUE;
@@ -2303,7 +2303,7 @@ get_device_attributes_async_dbus_cb (GObject *source_object,
if (ret_error[0] != '\0')
{
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: getting of attributes for printer %s failed: %s", data->printer_name, ret_error);
}
if (devices_variant)
@@ -3334,7 +3334,7 @@ printer_add_option_async_dbus_cb (GObject *source_object,
g_variant_get (output, "(&s)", &ret_error);
if (ret_error[0] != '\0')
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: setting of an option failed: %s", ret_error);
else
success = TRUE;
@@ -3463,7 +3463,7 @@ get_cups_devices_async_dbus_cb (GObject *source_object,
if (ret_error[0] != '\0')
{
- g_warning ("%s", ret_error);
+ g_warning ("cups-pk-helper: getting of CUPS devices failed: %s", ret_error);
}
if (devices_variant)
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]