diff options
| author | http://jneen.net/ <jneen@jneen.net> | 2016-11-22 15:25:09 +0900 |
|---|---|---|
| committer | http://jneen.net/ <jneen@jneen.net> | 2016-12-21 09:51:41 -0800 |
| commit | 602c3198626ae2a23f8f2962963c18f3b3bf1664 (patch) | |
| tree | 8b1a5e1d3ad3fb58ac24244c436b56360d6aa1b2 /lib/gitlab/email | |
| parent | 017579fe149e976fb0b2b21b234ea923913cadb2 (diff) | |
| download | gitlab-ce-602c3198626ae2a23f8f2962963c18f3b3bf1664.tar.gz | |
allow empty-quotes
Diffstat (limited to 'lib/gitlab/email')
| -rw-r--r-- | lib/gitlab/email/reply_parser.rb | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/lib/gitlab/email/reply_parser.rb b/lib/gitlab/email/reply_parser.rb index e714923c4d8..5957cbbad7e 100644 --- a/lib/gitlab/email/reply_parser.rb +++ b/lib/gitlab/email/reply_parser.rb @@ -18,10 +18,6 @@ module Gitlab # [jneen] not using /\s+$/ here because that deletes empty lines body = body.gsub(/[ \t]$/, '') - # TODO [jneen]: do we want to allow empty-quoting? (replies only containing a blockquote) - # EmailReplyTrimmer allows this as a special case, so we detect it manually here. - return "" if body.lines.all? { |l| l.strip.empty? || l.start_with?('>') } - body.force_encoding(encoding).encode("UTF-8") end |