[Bps-public-commit] RT-Extension-rt_cpan_org branch, master, updated. e737a08ea306b89f035458879025c01eeba2b858

Thomas Sibley trs at bestpractical.com
Fri May 17 00:22:57 EDT 2013


The branch, master has been updated
       via  e737a08ea306b89f035458879025c01eeba2b858 (commit)
      from  36fdeb0b4b2bf1f4fed72f031fddf30a1124379e (commit)

Summary of changes:
 patches/4.0.14-conservative-quote-folding.patch | 28 +++++++++++++++++++++++++
 1 file changed, 28 insertions(+)
 create mode 100644 patches/4.0.14-conservative-quote-folding.patch

- Log -----------------------------------------------------------------
commit e737a08ea306b89f035458879025c01eeba2b858
Author: Thomas Sibley <trs at bestpractical.com>
Date:   Thu May 16 21:22:24 2013 -0700

    Patch to only use ">" as a quote character when finding folds

diff --git a/patches/4.0.14-conservative-quote-folding.patch b/patches/4.0.14-conservative-quote-folding.patch
new file mode 100644
index 0000000..52425d7
--- /dev/null
+++ b/patches/4.0.14-conservative-quote-folding.patch
@@ -0,0 +1,28 @@
+From 9299787baa49b7d824d82bd584b628a6fc253c0b Mon Sep 17 00:00:00 2001
+From: Thomas Sibley <trs at bestpractical.com>
+Date: Thu, 16 May 2013 14:50:09 -0700
+Subject: [PATCH] Only treat ">" as an email quoting character
+
+All of the other defaults (!, #, %, =, |, :) cause too many false
+positives and hardly any true use as a quote character.
+---
+ share/html/Ticket/Elements/ShowTransactionAttachments | 3 +++
+ 1 file changed, 3 insertions(+)
+
+diff --git a/share/html/Ticket/Elements/ShowTransactionAttachments b/share/html/Ticket/Elements/ShowTransactionAttachments
+index f70f279..b219205 100644
+--- a/share/html/Ticket/Elements/ShowTransactionAttachments
++++ b/share/html/Ticket/Elements/ShowTransactionAttachments
+@@ -246,6 +246,9 @@ my $render_attachment = sub {
+                 unless ( length $name ) {
+                     eval {
+                         require Text::Quoted;
++                        # XXX: Deprecate ->can check in 4.2 and simply bump version requirement.
++                        Text::Quoted::set_quote_characters(undef) # only use >
++                            if Text::Quoted->can("set_quote_characters");
+                         $content = Text::Quoted::extract($content);
+                     };
+                     if ($@) {
+-- 
+1.8.2.2
+

-----------------------------------------------------------------------



More information about the Bps-public-commit mailing list