Excerpts from Nicolas's message on Nov 4, 2008 about 2 PM:
> However there is already a WRAP_LEN constant in this file, please provide
> a patch that reuse the same constant.
Find it below. I have moved the constant out of the Attachment and
Text classes, and it is now in the context of the Chunk module.
Cheers,
Israel
---
lib/sup/message-chunks.rb | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/lib/sup/message-chunks.rb b/lib/sup/message-chunks.rb
index cc895f3..5b5c314 100644
--- a/lib/sup/message-chunks.rb
+++ b/lib/sup/message-chunks.rb
@@ -41,6 +41,8 @@ end
module Redwood
module Chunk
+ WRAP_LEN = 80 # wrap messages and text attachments at this width
+
class Attachment
HookManager.register "mime-decode", <<EOS
Executes when decoding a MIME attachment.
@@ -95,6 +97,7 @@ EOS
@lines = nil
if text
@lines = text.gsub("\r\n", "\n").gsub(/\t/, " ").gsub(/\r/,
"").split("\n")
+ @lines = lines.map {|l| l.chomp.wrap WRAP_LEN}.flatten
@quotable = true
end
end
@@ -143,7 +146,6 @@ EOS
end
class Text
- WRAP_LEN = 80 # wrap at this width
attr_reader :lines
def initialize lines
--
1.5.6.5
_______________________________________________
sup-talk mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/sup-talk
