Error message shows markup

Hi,

Would you forward this change upstream please?

Thanks,

James

Patch looks wrong to me.

Tim, can you elaborate on that?
This is the patch:
http://bazaar.launchpad.net/~simono/ubuntu/lucid/system-config-printer/escape-message/annotate/head%3A/debian/patches/78_escape_message_in_error_dialog.patch

I saw the patch. It's wrong.

Take a look at the surrounding code, and all the other dialogs, to see why. We call set_use_markup() so pango should mark up the text correctly. If it isn't doing that for you, you need to investigate why that is. (Works for me...)

It marks up the text correctly in most cases.
But if the text contains markup that cannot be parsed (like <email address hidden> in my screenshot) the whole label is displayed as plain text. So I would suggest to escape all markup that may be in the document title.

Does it work for you, if you have a document with a title that contains markup?

The following example demonstrates this:
  label = gtk.Label ("<b>Valid</b> -- <b_>invalid</b_>")
  label.set_use_markup (True)

Which leads to the following error:
  __main__:1: GtkWarning: Failed to set text from markup due to error parsing markup: Unknown tag 'b_' on line 1 char 29

Ah, I understand now, I apologise.

Tim, sorry if I was unclear. Can you apply the patch upstream? Or how do we proceed?

Btw, here is package:
https://edge.launchpad.net/~simono/+archive/ppa

Applied. I wanted to investigate whether xml.sax is an additional dependency outside of the main python package, and it looks like it's provided as part of python so that's fine.

Applied upstream:
http://git.fedorahosted.org/git/?p=system-config-printer.git;a=commitdiff;h=d850270b019b3977a12115c79a31b3f45512e88e
Thanks Tim.

This bug was fixed in the package system-config-printer - 1.2.3+20100713-0ubuntu1

---------------
system-config-printer (1.2.3+20100713-0ubuntu1) maverick; urgency=low

  * New upstream release
     o GIT 1.2.x snapshot from 13 July 2010
     o Version 1.2.3 plus some additional fixes and translations
     o Plug'n'Print also for Bluetooth printers now.
     o Do not automatically set up a print queue if a driver executable is
       missing.
     o Improvements on device ID handling: Discovery via SNMP, non-UTF-8
       environments, ignore the deprecated hal backend.
     o Avoid Yes/No buttons (Upstream bug #204).
     o Re-introduced and added keyboard accelerators for system-config-printer
       itself and for the job viewer.
     o New HIG-compliant job viewer UI. Thanks to Sergey Nizovtsev for the
       patch (LP: #555213, upstream bug #205).
     o Attempt to translate backend device-info strings and other fixes
       on the translation mechanism (LP: #557199).
     o Fixed markup in error pop-ups (LP: #567324).
     o Made width of "Verify" button in New Printer Wizard adapt to the length
       of the actual translation of "Verify" (LP: #575048).
     o Make Kyocera detected if it reports only the model name (Upstreamized
       LP: #564633).
     o Removed doubled-up 'translatable' attribute in UI file (LP: #571662).
     o Handle IPP failure fetching printer attrs for completed jobs
       (LP: #561082).
     o Many more bug fixes and UI improvements.
     o Translation updates.
  * debian/patches/57_backend-integration-dnssd-bluetooth.patch,
    debian/patches/67_udev_configure_printer_match_usblp_libusb_and_hplip_uris.patch,
    debian/patches/75_use-dnssd-uris-for-hplip.patch,
    debian/patches/94_cache-and-timeout-for-avahi-browse.patch
    debian/patches/96_new-printer-setup-check-for-local-drivers-first.patch:
    Regenerated to work with new upstream code.
  * debian/patches/97_add-missing-make-name-kyocera-mita.patch: Removed
    patches for upstream fixes.
  * debian/patches/57_backend-integration-dnssd-bluetooth.patch: To check for
    HP fax devices do not do an exact match of "Fax" with the model name but
    check whether the model name begins with "Fax".
  * debian/patches/44_no-runtime-error-on-missing-firewall.patch: Let the
    firewall functions exit silently if there is no firewall service on the
    D-Bus.
  * debian/patches/50_give-priority-to-hpcups.patch: Prioritize hpcups against
    HPIJS again. It should have matured enough now.
  * debian/system-config-printer-common.install: Added new dnssdresolve.py
    file.
  * debian/rules: Added directory settings for UDEV rules and scripts to the
    "make install" command line.
  * debian/control: Added libglib2.0-dev as build dependency, needed for the
    Bluetooth Plug'n'Print support.
 -- Till Kamppeter <email address hidden> Tue, 13 Jul 2010 13:21:33 +0200