Bug 708014

Summary: Raw Unified patch view does not work -- leads to corrupted patches
Product: [Community] Bugzilla Reporter: Prarit Bhargava <prarit>
Component: User InterfaceAssignee: Simon Green <sgreen>
Severity: urgent Docs Contact:
Priority: unspecified    
Version: 4.2CC: ebaak, jpokorny, jsafrane, sgreen
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-06-20 20:43:13 EDT Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Description Prarit Bhargava 2011-05-26 10:45:46 EDT
Description of problem: After adding a patch to a Bugzilla and (correctly) using the "Patch" button, the next viewer of the Bugzilla will be able to see the patch I've added by clicking on the "attachment" link.

When viewing the patch, the filenames are stripped from the patch rendering the viewer useless.  In addition, and much more critically, *the Raw Unified view is also munging the patch so that the filenames are missing*.

Version-Release number of selected component (if applicable): current bugzilla.redhat.com

How reproducible: 100%

Steps to Reproduce:
1. Add patch

For example, https://bugzilla.redhat.com/show_bug.cgi?id=699451#c3

2. Click attachment to view patch -- filenames are missing.


3. Click "Raw Unified" to view patch -- filenames are missing.


Compare that to the actual patch that was attached:

Actual results:  Corrupted patches due to Bugzilla.

Expected results:  Patches should have filenames intact o/w the viewer and "Raw Unified" views are effectively useless.

Additional info:  IMO, this is _critical_.  We (Engineering) use BZ to pass test patches back and forth to customers.  We use it to store patches on committed bugzillas.
Comment 1 Jan Pokorný 2011-10-19 16:17:04 EDT
I have a similar issue.

Original patch:

snippet from the top (for illustration, not 1:1):
> --- a/ricci/modules/cluster/clumon/src/daemon/main.cpp
> +++ b/ricci/modules/cluster/clumon/src/daemon/main.cpp

Raw unified (navigation: [1] -> "Raw Unified" link):

snippet from the top (for illustration, not 1:1):
> --- a/ricci/modules/cluster/clumon/src/daemon/main.cpp
> +++ a/ricci/modules/cluster/clumon/src/daemon/main.cpp

The question is, why it is turned the second "b" to "a".
Is it a correct behavior?

[1] https://bugzilla.redhat.com/attachment.cgi?id=529083&action=diff
Comment 2 Jan Safranek 2012-05-29 04:52:41 EDT
Reproducible also with Bugzilla 4.2

Original patch:

Raw unified patch:

-> file gtk/column_prefs.c is missing!
Comment 3 Simon Green 2012-06-20 20:43:13 EDT

*** This bug has been marked as a duplicate of bug 554878 ***