Bug 137227 - Patch diffs using PatchReader not displaying properly
Summary: Patch diffs using PatchReader not displaying properly
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Bugzilla
Classification: Community
Component: Bugzilla General
Version: 2.18
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mike MacKenzie
QA Contact: Mike MacKenzie
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2004-10-26 19:43 UTC by David Lawrence
Modified: 2007-04-18 17:14 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2004-10-27 16:16:55 UTC
Embargoed:


Attachments (Terms of Use)
Command line perl script to test PatchReader (1.29 KB, text/plain)
2004-10-26 19:43 UTC, David Lawrence
no flags Details
Sample diff file for use with testpatchreader.pl (1.15 KB, text/plain)
2004-10-26 19:45 UTC, David Lawrence
no flags Details

Description David Lawrence 2004-10-26 19:43:49 UTC
Description of problem:
When viewing a patch attachment as a diff, nothing is displaying properly 
between the header and footer. Other navigation links are displaying so the
proper actiion is being taken in attachment.cgi. It will work from the command
line if using a test perl script (attached).

Version-Release number of selected component (if applicable):
2.18 beta2

Comment 1 David Lawrence 2004-10-26 19:43:49 UTC
Created attachment 105811 [details]
Command line perl script to test PatchReader

Comment 2 David Lawrence 2004-10-26 19:45:07 UTC
Created attachment 105812 [details]
Sample diff file for use with testpatchreader.pl

Comment 3 David Lawrence 2004-10-26 19:45:43 UTC
URL to Main Bugzilla database showing what something like this should look like.

https://bugzilla.mozilla.org/attachment.cgi?id=163228&action=diff

Comment 4 Mike MacKenzie 2004-10-27 16:16:55 UTC
Okay this is fixed.  The problem was that the data being sent to the patch
reader was base64 encoded.

Comment 5 Hotmail Account 2004-10-27 16:20:37 UTC
Ah, right! now i feel stupid. Mucho Thanks.


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