Launchpad has imported 20 comments from the remote bug at
https://bugzilla.redhat.com/show_bug.cgi?id=625502.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2010-08-19T16:24:02+00:00 Ben wrote:

abrt version: 1.1.13
architecture: i686
cmdline: python /usr/share/system-config-printer/system-config-printer.py
component: system-config-printer
executable: /usr/share/system-config-printer/system-config-printer.py
kernel: 2.6.33.6-147.2.4.fc13.i686
package: system-config-printer-1.2.3-3.fc13
reason: system-config-printer.py:3151:rename_printer:AttributeError: 'NoneType' 
object has no attribute 'setAccepting'
release: Fedora release 13 (Goddard)
time: 1282235009
uid: 500

backtrace
-----
system-config-printer.py:3151:rename_printer:AttributeError: 'NoneType' object 
has no attribute 'setAccepting'

Traceback (most recent call last):
  File "/usr/share/system-config-printer/system-config-printer.py", line 3085, 
in printer_name_edited
    self.rename_printer (name, newname)
  File "/usr/share/system-config-printer/system-config-printer.py", line 3151, 
in rename_printer
    self.printer.setAccepting (True)
AttributeError: 'NoneType' object has no attribute 'setAccepting'

Local variables in innermost frame:
new_name: 'c6'
self: <__main__.GUI instance at 0x8867bcc>
rejecting: False
old_name: u'C6'

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/2

------------------------------------------------------------------------
On 2010-08-19T16:24:04+00:00 Ben wrote:

Created an attachment (id=439729)
File: backtrace

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/3

------------------------------------------------------------------------
On 2010-09-20T10:37:54+00:00 Jiri wrote:

*** Bug 612315 has been marked as a duplicate of this bug. ***

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/4

------------------------------------------------------------------------
On 2010-09-20T10:38:03+00:00 Jiri wrote:

*** Bug 635538 has been marked as a duplicate of this bug. ***

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/5

------------------------------------------------------------------------
On 2010-09-21T14:31:30+00:00 Jiri wrote:

Steps to reproduce:
1) rename printer queue where the new name differs
    only in size (lower/upper case) of some characters.
    E.g. C6 -> c6 (comment #0)

When s-c-p renames queue, it first makes a new one with the new name,
copies attributes and deletes the old one.
Problem is that CUPS-Add-Modify-Printer operation is case-insensitive in sense
that when it discovers that adding printer already exists (ignoring case) it 
modifies the existing one.
For looking up/comparing strcasecmp() is used, see. 
scheduler/printers.c:compare_printers().

In our case it means that when s-c-p tries to add a new printer in 
save_printer(), there's actually no adding
happening because there's already one with the same name (ignoring case) 
existing.
And at the end of save_printer() we discover that the printer with new name 
(case sensitive) doesn't exist
and populateList() does self.printer=None.

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/6

------------------------------------------------------------------------
On 2010-09-21T14:56:03+00:00 Jiri wrote:

Created attachment 448714
double-renaming

Possibilities
A) Do not rename when the new name differs only in size
   (lower/upper case) of some characters.
    def rename_printer (self, old_name, new_name):
-        if old_name == new_name:
+        if old_name.lower() == new_name.lower():
            return

B) Double-rename (old_name->random_name, random_name->new_name)
   (see attached patch)

C) Some dialog with warning

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/7

------------------------------------------------------------------------
On 2010-09-22T10:16:13+00:00 Tim wrote:

I think A is best.  Let's follow the example of how cupsd behaves.

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/8

------------------------------------------------------------------------
On 2010-09-22T13:10:37+00:00 Jiri wrote:

http://git.fedorahosted.org/git/?p=system-config-
printer.git;a=commitdiff;h=332fdc86be253a3b91720f64685ed2507b308184

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/9

------------------------------------------------------------------------
On 2010-09-22T14:22:50+00:00 Tim wrote:

There's also the very real possibility of the queue actually being
deleted (by another user) in the mean time, so we'll have to deal with
that too by checking whether self.printer == None after the call to
self.duplicate_printer().

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/10

------------------------------------------------------------------------
On 2010-09-23T09:05:14+00:00 Jiri wrote:

http://git.fedorahosted.org/git/?p=system-config-
printer.git;a=commitdiff;h=01d61dcfc52ef002accd9d16f840ef4ffd3b4d4c

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/11

------------------------------------------------------------------------
On 2010-10-13T14:47:00+00:00 Fedora wrote:

system-config-printer-1.2.5-1.fc14 has been submitted as an update for Fedora 
14.
https://admin.fedoraproject.org/updates/system-config-printer-1.2.5-1.fc14

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/12

------------------------------------------------------------------------
On 2010-10-13T14:54:44+00:00 Fedora wrote:

system-config-printer-1.2.5-1.fc13 has been submitted as an update for Fedora 
13.
https://admin.fedoraproject.org/updates/system-config-printer-1.2.5-1.fc13

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/13

------------------------------------------------------------------------
On 2010-10-14T06:23:59+00:00 Fedora wrote:

system-config-printer-1.2.5-1.fc13 has been pushed to the Fedora 13 testing 
repository.  If problems still persist, please make note of it in this bug 
report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update system-config-printer'.  You 
can provide feedback for this update here: 
https://admin.fedoraproject.org/updates/system-config-printer-1.2.5-1.fc13

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/14

------------------------------------------------------------------------
On 2010-10-28T12:08:18+00:00 Tim wrote:

*** Bug 647340 has been marked as a duplicate of this bug. ***

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/15

------------------------------------------------------------------------
On 2010-10-31T07:52:21+00:00 Simon wrote:

Package: system-config-printer-1.2.4-2.fc14
Architecture: x86_64
OS Release: Fedora release 14 (Laughlin)


How to reproduce
-----
1. 
2.
3.


Comment
-----
Attempted to rename a printer putting parenthesis in the name

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/16

------------------------------------------------------------------------
On 2010-11-05T15:10:30+00:00 Tim wrote:

*** Bug 650226 has been marked as a duplicate of this bug. ***

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/17

------------------------------------------------------------------------
On 2010-11-06T21:12:43+00:00 Theophanis wrote:

Package: system-config-printer-1.2.4-2.fc14
Architecture: x86_64
OS Release: Fedora release 14 (Laughlin)


Comment
-----
Changed the last part of the printer name from _fax to _FAX...

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/18

------------------------------------------------------------------------
On 2010-11-11T03:09:52+00:00 Hamidou wrote:

Package: system-config-printer-1.2.4-2.fc14
Architecture: i686
OS Release: Fedora release 14 (Laughlin)


How to reproduce
-----
It happened when playing around with the rename button trying to reproduce in 
the bugzilla ticket that
you Tim forwarded to the Compiz maintainer Adel.

Regards.

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/19

------------------------------------------------------------------------
On 2010-11-16T23:23:07+00:00 Fedora wrote:

system-config-printer-1.2.5-6.fc14 has been pushed to the Fedora 14
stable repository.  If problems still persist, please make note of it in
this bug report.

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/20

------------------------------------------------------------------------
On 2010-11-22T22:12:48+00:00 Fedora wrote:

system-config-printer-1.2.5-6.fc13 has been pushed to the Fedora 13
stable repository.  If problems still persist, please make note of it in
this bug report.

Reply at: https://bugs.launchpad.net/ubuntu/+source/system-config-
printer/+bug/526623/comments/21


** Changed in: system-config-printer (Fedora)
       Status: Unknown => Fix Released

** Changed in: system-config-printer (Fedora)
   Importance: Unknown => Medium

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/526623

Title:
  system-config-printer.py crashed with AttributeError in
  rename_printer()

To manage notifications about this bug go to:
https://bugs.launchpad.net/system-config-printer/+bug/526623/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to