Bug 635779

Created attachment 448520 [details]
Screenshot (Fedora 13).png

Exception report dialog in F13 - doesn't have a click-able link, but also doesn't display the link twice

Created attachment 448522 [details]
Screenshot (Fedora 14 Alpha).png

Exception report dialog in Fedora 14 Alpha - Has a click-able link to the exception report.  This is what I was expecting when testing Fedora 14 Beta

Anaconda uses report to handle this.

the report library was recently changed to try to determine if a browser was a callable browser for use by the clickable link before creating it (so that it didn't create clickable links that did nothing).  If there does seem to be a callable browser, it still creates a clickable link.  If there doesn't seem to a callable browser, the report lib just shows the original label and URL that would have made up the clickable link (and admittadly the formatting of this label and URL are not the best).

I didn't think to test the live installer when I implemented this, and so the method I use to determine if there is a usable browser may be faulty in this case.

So for the cases snapshotted above that don't have a clickable link, were these done in live installer, or the normal installer?

(In reply to comment #4)
> So for the cases snapshotted above that don't have a clickable link, were these
> done in live installer, or the normal installer?

This bug was filed while testing exception reporting in the normal installer.

I just tested the Live installer and it properly shows a clickable link (as one would expect).

Hey Gavin!  Any updates?  This issue remains with the current installer and not much time remains to include fixes.

Proposing as a nice-to-have fix for F-14.  This issue is clearly not a blocker, but a cosmetic nice-to-have for any users reporting installer crashes through bugzilla.

Created attachment 452768 [details]
fixes this bz by changing the description of the bug to _not_ look like a URL

The problem is not that the formatting is off, the problem is that the text in the dialog box that is supposed to be a title or name for the bug is output as a URL.  This proposed patch fixes this bz by changing the 'title/name' text to not be a URL.

(In reply to comment #6)
> Hey Gavin!  Any updates?  This issue remains with the current installer and not
> much time remains to include fixes.
> 
> Proposing as a nice-to-have fix for F-14.  This issue is clearly not a blocker,
> but a cosmetic nice-to-have for any users reporting installer crashes through
> bugzilla.

James,
I've posted a proposed patch to this bz.  I didn't think that the fedora-powers-that-be would consider this important enough to break freeze, but you would know better than I.  The patch is very safe, but it just fixes a cosmetic thing.  If you want me to, I'll push this to bodhi and ask for an exception.  Just let me know.

(In reply to comment #8)
> (In reply to comment #6)
> > Hey Gavin!  Any updates?  This issue remains with the current installer and not
> > much time remains to include fixes.
> > 
> > Proposing as a nice-to-have fix for F-14.  This issue is clearly not a blocker,
> > but a cosmetic nice-to-have for any users reporting installer crashes through
> > bugzilla.
> 
> James,
> I've posted a proposed patch to this bz.  I didn't think that the
> fedora-powers-that-be would consider this important enough to break freeze, but
> you would know better than I.  The patch is very safe, but it just fixes a
> cosmetic thing.  If you want me to, I'll push this to bodhi and ask for an
> exception.  Just let me know.

I'm planning to test the fix now, and pending testing, we can propose this for the nice-to-have list.  This means it would be accepted and pulled into F-14 if the fix was tested and available.  However, the F-14 release wouldn't block on this issue if it weren't tested and available.

Created attachment 452789 [details]
Screenshot demonstrating behavior using proposed patch

(In reply to comment #9)
> I'm planning to test the fix now, and pending testing, we can propose this for
> the nice-to-have list. 

http://jlaska.fedorapeople.org/updates/635779.img

Using the proposed patch (attachment#452768 [details]), I created an installer updates.img image that 1) contains the patch, 2) another unrelated installer fix, 3) some code to force a traceback at the welcome screen.

Using the provided updates.img, I'm not able to confirm that the proposed fix resolves the issue.  I expected to see a hyper-link (underlined) link to the bug after filing the report.  See attached screenshot.

jlaska asked in irc:
<jlaska> prior to F-14, it was a link whether in Live or normal install
<jlaska> if you clicked on the link during a normal install, nothing happened
<jlaska> so the intended change here was to not make that link clickable for the normal installer?
<jlaska> gavin: My expectation was that the behavior observed in F-14-Alpha was the correct intended behavior (see https://bugzilla.redhat.com/attachment.cgi?id=448522)
<jlaska> can you confirm?

prior to F-14, and in F-14 alpha, it was a link whether in Live or normal install.  if you clicked on the link during a normal install, nothing happened.  This behavior was intentionally changed between F14 alpha and F14 beta, such that if there was no browser installed (which is true in the case of the normal installer), then don't display a non-functional hyper link.

Thanks Gavin, so I understand that this bug involves rewording the F-14-Beta report dialog to not list the URL twice, and to continue with the intended F-14 behavior change noted in comment#11.  I can confirm that the dialog no longer displays the bugzilla bug URL twice using your proposed patch (see attachment#452789 [details]). 

Without this change, the bugzilla report dialog will show the bugURL twice when reporting an exception to bugzilla from a normal (non-live) install?

(In reply to comment #12)
 
> Without this change, the bugzilla report dialog will show the bugURL twice when
> reporting an exception to bugzilla from a normal (non-live) install?

Yes.

Would you like me to rebuild (in bodhi) the F14 report with this patch?

(In reply to comment #13)
> Would you like me to rebuild (in bodhi) the F14 report with this patch?

Please do, thanks!  

This issue will be reviewed at the Friday blocker bug meeting.  Also note, while I think it is a low risk change for the release, and a good fit for the nice-to-have process, there is a possibility it would not qualify as a 'nice-to-have' bug.  As a result, the proposed update would still be tested and available in Fedora 14, but just not included in the official installation media.

report-0.20-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/report-0.20-1.fc14

I've built and pushed the fix for this into testing (using bodhi). 

Do I need to do anything else for the nice-to-have process?  (I searched for this in Fedoraproject but couldn't find it.)

(In reply to comment #16)
> I've built and pushed the fix for this into testing (using bodhi). 
> 
> Do I need to do anything else for the nice-to-have process?  

No sir, I believe you are good to go.  This bug has been added to the F14-accepted (aka nice-to-have) Tracking bug.  Release Engineering will use that Tracking bug to pull in any bugs which have tested fixes and are available in bodhi.  

To supply positive bodhi karma, I'll need to figure out how to test the proposed 'report' update.  I gather I can build a live image and test using the bodhi update in updates-testing.  I'll follow-up on that.

> (I searched for
> this in Fedoraproject but couldn't find it.)

The 'nice-to-have' process has just recently been discussed+drafted and you can find it linked off of the QA SOP documents at https://fedoraproject.org/wiki/User:Adamwill/QA:SOP_nth_process_nth_draft

The document will be renamed soon, but will REDIRECT correctly.

report-0.20-1.fc14 has been pushed to the Fedora 14 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 report'.  You can provide feedback for this update here: https://admin.fedoraproject.org/updates/report-0.20-1.fc14

Discussed at 2010-10-15 review meeting, accepted as NTH.



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

report-0.20-1.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.