Bug 757425 - [abrt] picard-0.16-1.fc16: webservice.py:76:__getattr__:AttributeError: response
Summary: [abrt] picard-0.16-1.fc16: webservice.py:76:__getattr__:AttributeError: response
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: picard
Version: 16
Hardware: x86_64
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Alex Lancaster
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard: abrt_hash:a5a8302374c92c0682ba83daeea...
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-11-27 00:16 UTC by Jeffrey C. Ollie
Modified: 2012-06-20 19:30 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-06-20 19:28:52 UTC
Type: ---


Attachments (Terms of Use)

Description Jeffrey C. Ollie 2011-11-27 00:16:46 UTC
libreport version: 2.0.7
abrt_version:   2.0.6
cmdline:        /usr/bin/python /usr/bin/picard
comment:        Scanning files using chromaprint
executable:     /usr/bin/picard
kernel:         3.1.0-7.fc16.x86_64
reason:         webservice.py:76:__getattr__:AttributeError: response
time:           Sat 26 Nov 2011 06:15:54 PM CST
uid:            500
username:       jcollie

backtrace:
:webservice.py:76:__getattr__:AttributeError: response
:
:Traceback (most recent call last):
:  File "/usr/lib64/python2.7/site-packages/picard/webservice.py", line 206, in _process_reply
:    handler(xml_handler.document, reply, error)
:  File "/usr/lib64/python2.7/site-packages/picard/acoustid.py", line 59, in _on_lookup_finished
:    for result in document.response[0].results[0].children.get('result', []):
:  File "/usr/lib64/python2.7/site-packages/picard/webservice.py", line 76, in __getattr__
:    raise AttributeError, name
:AttributeError: response
:
:Local variables in innermost frame:
:self: {'attribs': {}, 'text': u'', 'children': {}}
:name: 'response'

Comment 1 Rudd-O DragonFear 2012-02-21 19:36:55 UTC
scanning a file with acoustid fingerprinting

Package: picard-0.16-1.fc16
OS Release: Fedora release 16 (Verne)

Comment 2 Rudd-O DragonFear 2012-02-21 19:47:48 UTC
self.tagger.xmlws.query_acoustid(partial(self._on_lookup_finished, next, file), **params) in acoustid.py calls _on_lookup_finished with a weird document object that does not have a response attribute.

xmlws is a XmlWebService() assigned in the Tagger class in tagger.py

XmlWebService comes from picard's webservice.py and is a Qt object.  query_acoustid there creates a body and self.post() that body.  they implement an HTTP client that does post in that class (Bad form, as python already has one).  the post gets added to self.active_requests.

When a reply comes back, self._process_reply pops the requests from active_Requests  and then it creates an XmlHandler with a .document, which then gets dispatched to the handler (_on_lookup_finished partial call).

This .document has no response attribute.

And that's why Picard barfs.

End of story.

Comment 3 Rudd-O DragonFear 2012-02-21 19:48:07 UTC
(That was the missing traceback from async style programming code.  It sucks because of that.)

Comment 4 Alex Lancaster 2012-02-21 21:33:28 UTC
Thanks for the explanation. As I haven't been able to run into this error myself,  can you check if this already posted upstream: http://tickets.musicbrainz.org/browse/PICARD   If you can't let me know, I'll try to post a bug.

Comment 5 Rudd-O DragonFear 2012-02-22 00:55:30 UTC
Sorry, I have no fucking idea how to post a bug in that crap Web site.  :-)

Comment 6 Fedora Update System 2012-06-04 19:23:56 UTC
picard-1.0-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/picard-1.0-1.fc17

Comment 7 Fedora Update System 2012-06-04 19:53:01 UTC
picard-1.0-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/picard-1.0-1.fc16

Comment 8 Fedora Update System 2012-06-07 02:38:10 UTC
Package picard-1.0-1.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing picard-1.0-1.fc17'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-8916/picard-1.0-1.fc17
then log in and leave karma (feedback).

Comment 9 Fedora Update System 2012-06-20 19:28:52 UTC
picard-1.0-1.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 10 Fedora Update System 2012-06-20 19:30:25 UTC
picard-1.0-1.fc16 has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.


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