Bug 445086 - viewing update detail unbound local error yumbackend
Summary: viewing update detail unbound local error yumbackend
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: PackageKit
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Robin Norwood
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: F9Blocker
TreeView+ depends on / blocked
 
Reported: 2008-05-03 12:10 UTC by David Timms
Modified: 2008-05-06 17:11 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-05-06 17:11:44 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
error dialog. (25.96 KB, image/png)
2008-05-03 12:10 UTC, David Timms
no flags Details
An untested patch that may fix the problem. (1.57 KB, patch)
2008-05-04 01:58 UTC, Luke Macken
no flags Details | Diff
A tested patch that fixes this issue (1.47 KB, patch)
2008-05-05 14:54 UTC, Luke Macken
no flags Details | Diff

Description David Timms 2008-05-03 12:10:56 UTC
Description of problem:
During use of the view updates dialog, an error was produced

Version-Release number of selected component (if applicable):
PackageKit-0.1.12-8.20080425.fc9.i386
PackageKit-libs-0.1.12-8.20080425.fc9.i386

Reproduced with the python-sqlalchemy package listed.

Steps to Reproduce:
1. update icon says one update pending
2. click, show updates
3. review
  
Actual results:
The dialog expands, and opens up the lower part for description, and then
disappears, showing error dialog:
Update System
An internal system error has occurred
A problem that we were not expecting has occurred. Please report this bug with
the error description.
More details:
Error Type: <type 'exception.UnboundLocalError'>
...
line 1325, in _get_update_extras
  href = ref['href']

Expected results:
details of that package update.
This has worked for other updates.

Comment 1 David Timms 2008-05-03 12:10:56 UTC
Created attachment 304449 [details]
error dialog.

Comment 2 Jeremy Katz 2008-05-03 19:55:48 UTC
This looks like a real problem from a quick look at the code -- if there aren't
references, then ref is never getting set before its used

Comment 3 Luke Macken 2008-05-04 01:58:29 UTC
This bug was caused by improper tab->space conversion.  This problem does not
exist in the yum2 backend.

Comment 4 Luke Macken 2008-05-04 01:58:57 UTC
Created attachment 304472 [details]
An untested patch that may fix the problem.

Comment 5 Jeremy Katz 2008-05-04 17:25:04 UTC
You lost the 'if refs' check -- which may or may not matter.  Otherwise, it
looks sane from not knowing the code ;)

Comment 6 Luke Macken 2008-05-04 23:47:53 UTC
Yeah, the refs check shouldn't matter -- the
yum.update_md.UpdateNotice.references defaults to an empty list.

Comment 7 Matthias Clasen 2008-05-05 13:45:58 UTC
Robin, Richard, can we get this fix built ?

Comment 8 Luke Macken 2008-05-05 14:54:05 UTC
Ok, so apparently the refs check *is* necessary.  Attaching a tested patch that
seems to do the trick.

Comment 9 Luke Macken 2008-05-05 14:54:34 UTC
Created attachment 304527 [details]
A tested patch that fixes this issue

Comment 10 Luke Macken 2008-05-05 14:59:27 UTC
Fixed in 34368f7fa13b3ac7d2a16d570d450a027e66dd75


Comment 11 Jeremy Katz 2008-05-05 15:45:31 UTC
Need a build in dist-f9-updates-candidate and mail to rel-eng
with the nvr for tagging please :)

Comment 12 Richard Hughes 2008-05-05 16:02:41 UTC
Is this okay to mark as a release blocker? Cheers for fixing this guys. I'll do
a snapshot release tmw morning and get this into F9.

Comment 14 Bill Nottingham 2008-05-05 17:24:09 UTC
Tagged for the tree.


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