Project

General

Profile

Actions

Defect #9440

closed

Escaping issue with inline diff-highlighting in revision diff view

Added by Mischa The Evil about 13 years ago. Updated about 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
SCM
Target version:
-
Start date:
2011-10-20
Due date:
% Done:

0%

Estimated time:
Resolution:
Duplicate
Affected version:

Description

While doing some manual tests after the recent upgrade to CodeRay 1.0 I first noticed this defect.
At first I though it was a regression caused by the CodeRay update. So, I reverted back to source:/trunk@7617, deleted the repo and reimported it, but that didn't solve the issue. So, I dug a little deeper and noticed it happened inside inline diff-highlighting section.
I haven't been able to pinpoint it to a specific revision which introduced the error, thus it could be included already since the initial implementation of the inline diff-highlighting feature with r5094.

The issue can be best described using some screenshots. First comes the screenshot which displays the faulty escaping:

Following a screenshot of the plain patch file of the change, displayed through Redmines own repository file view, which is rendered correctly:

Environement:
  • Redmine source:/trunk@7630
  • Rails 2.3.14
  • Passenger 3.0.9
  • Ruby Enterprise Edition 1.8.7-2011.03

Please leave a note if more information is required...


Files

escaping-issue.jpg (85.7 KB) escaping-issue.jpg Mischa The Evil, 2011-10-20 03:25
escaping-issue-ok.jpg (90.3 KB) escaping-issue-ok.jpg Mischa The Evil, 2011-10-20 03:25

Related issues

Is duplicate of Redmine - Defect #9143: Partial diff comparison should be done on actual code, not on htmlClosedJean-Philippe Lang2011-08-29

Actions
Actions #1

Updated by Etienne Massip about 13 years ago

Seems like a duplicate of #9143 (with much more details)?

Actions #2

Updated by Etienne Massip about 13 years ago

  • Status changed from New to Closed
  • Resolution set to Duplicate

Closed as duplicate of #9143.

Actions

Also available in: Atom PDF