Project

General

Profile

Actions

Defect #8712

closed

False 'Description updated' journal details messages

Added by Alex Shulgin over 13 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Issues
Target version:
Start date:
2011-06-29
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

1. Report an issue by sending new mail to redmine@.
2. Open up the issue in web interface and update (comment on it.)

The journal details for the update will contain 'Description updated (diff)' line, even though the description was intentionally not updated.

The reason to this seems to be that the email contained Unix (LF) newlines, while the update from web interface sends it with HTTP (CR-LF) newlines.

A proper fix would be to normalize newlines before processing.

Didn't happen with redmine-0.9 or 1.1, only started to happen with 1.2.

We're currently tracking 1.2-stable cloned from https://github.com/edavis10/redmine

$ ./script/about
Ruby version              1.8.7 (i686-linux)
RubyGems version          1.3.5
Rack version              1.1.1
Rails version             2.3.11
Active Record version     2.3.11
Active Resource version   2.3.11
Action Mailer version     2.3.11
Active Support version    2.3.11
Application root          ...
Environment               development
Database adapter          postgresql
Database schema version   20101114115359
Actions #1

Updated by Etienne Massip over 13 years ago

  • Target version set to Candidate for next minor release
Actions #2

Updated by Jean-Philippe Lang over 13 years ago

  • Status changed from New to Resolved
  • Target version changed from Candidate for next minor release to 1.2.1
  • Resolution set to Fixed

Fixed in #6147, #6149.

Actions #3

Updated by Alex Shulgin over 13 years ago

Jean-Philippe Lang wrote:

Fixed in #6147, #6149.

Thank you.

I've just tested before and after applying the above revisions: r6147 and r6149. This is fixed indeed.

--
Alex

Actions #4

Updated by Jean-Philippe Lang over 13 years ago

  • Status changed from Resolved to Closed

Merged in 1.2-stable. Thanks for the feedback.

Actions

Also available in: Atom PDF