Bug 219283 - Displays IEEE 1284 IDs instead of foomatic model names
Displays IEEE 1284 IDs instead of foomatic model names
Product: Fedora
Classification: Fedora
Component: desktop-printing (Show other bugs)
All Linux
high Severity high
: ---
: ---
Assigned To: Tim Waugh
Depends On:
  Show dependency treegraph
Reported: 2006-12-12 08:49 EST by Tim Waugh
Modified: 2008-02-27 07:00 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-02-27 07:00:25 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
eggcups-xml-parsing.patch (1.22 KB, patch)
2006-12-12 12:09 EST, Tim Waugh
no flags Details | Diff

  None (edit)
Description Tim Waugh 2006-12-12 08:49:09 EST
Description of problem:
Whoever wrote the foomatic XML parsing code in eggcups did so with very little
thought.  It just reads nodes and acts on them without any regard to their context.

So for example, whichever 'driver' is listed last in the 'drivers' section (i.e.
printer/drivers/driver) might take precendence over the actual recommended
driver (i.e. printer/driver).

But the most important mistake is that any old 'model' tag will be used as the
for-display model name.  Even this:

      <description>Hewlett-Packard DeskJet 990C</description>
      <model>DESKJET 990C</model>

which should *only* ever be used for matching devices to database entries, and
*never* be shown to the user.  These device IDs are really rough-and-ready
things that the manufacturers put in their devices.  There is no consistency in
them -- unlike the foomatic names which *are* consistent.

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1.Provoke the eggcups driver prompt dialog
Comment 1 Tim Waugh 2006-12-12 12:09:21 EST
Created attachment 143415 [details]

Here's a minimal fix (tested).	Now it can actually create queues based on the
user selection.

Note You need to log in before you can comment on or make changes to this bug.