Bug 178077 - Check problems with Ubuntu bugzilla
Summary: Check problems with Ubuntu bugzilla
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: eclipse-bugzilla
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Igor Foox
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 177734
TreeView+ depends on / blocked
 
Reported: 2006-01-17 18:38 UTC by Igor Foox
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-01-18 22:04:37 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Patch to fix non-existant icon problem (1.52 KB, patch)
2006-01-17 22:13 UTC, Igor Foox
no flags Details | Diff

Description Igor Foox 2006-01-17 18:38:04 UTC
Apparently the Ubuntu and Kernel bugzillas aren't functioning properly. Find out
what the problem is.

Comment 1 Igor Foox 2006-01-17 21:47:22 UTC
Kernel bugzilla seems to work fine.

Comment 2 Igor Foox 2006-01-17 21:52:18 UTC
The problem with the ubuntu bugzilla seems to be that it doesn't provide a
favicon.ico at all. When the search moves from
http://bugzilla.ubuntu.com/favicon.ico to http://ubuntu.com/favicon.ico it gets
a non-standard "file not found" page from the server, that tricks it to think
that it's an image and it creates an image descriptor from this data. When it
tries to get an ImageData from the descriptor later on (to check if it's not
null) an exception gets thrown because the ImageDescriptor doesn't have valid
image data.

We need to add a check for this.

Comment 3 Igor Foox 2006-01-17 22:11:19 UTC
Here's a patch that checks to see if the ImageData is valid or not by attempting
to get it from the ImageDescriptor and looking for the appropriate exceptions.
This makes the Ubuntu bugzilla default to the regular icon.

Andrew, could you take a look and let me know if this looks alright?

Comment 4 Igor Foox 2006-01-17 22:13:30 UTC
Created attachment 123338 [details]
Patch to fix non-existant icon problem

This patch is against CVS HEAD (with your Andrew's previous patch already
submitted).

Comment 5 Andrew Overholt 2006-01-18 14:53:12 UTC
(In reply to comment #4)
> Created an attachment (id=123338) [edit]
> Patch to fix non-existant icon problem

This patch looks good and I think it should be applied.  However, Ubuntu
recently moved away from bugzilla so that entry should probably be removed.  If
launchpad/malone works as a standard bugzilla instance, the let's just change
the URL.  Otherwise, this could be a good first test of the extensibility of
org.eclipse.team.bugs ;)

Comment 6 Igor Foox 2006-01-18 22:04:37 UTC
Patch committed. I also took out the Ubuntu bugzilla for now.


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