Bug 708014 - Raw Unified patch view does not work -- leads to corrupted patches
Raw Unified patch view does not work -- leads to corrupted patches
Status: CLOSED DUPLICATE of bug 554878
Product: Bugzilla
Classification: Community
Component: User Interface (Show other bugs)
x86_64 Linux
unspecified Severity urgent (vote)
: ---
: ---
Assigned To: Simon Green
Depends On:
  Show dependency treegraph
Reported: 2011-05-26 10:45 EDT by Prarit Bhargava
Modified: 2014-10-12 18:46 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2012-06-20 20:43:13 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
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 ***

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