[Rt-commit] rt branch, 4.0/loose-cf-name-in-rest, updated. rt-4.0.5-62-gc6970dd

? sunnavy sunnavy at bestpractical.com
Mon Feb 13 13:50:49 EST 2012


The branch, 4.0/loose-cf-name-in-rest has been updated
       via  c6970ddd7f22a3113091b65c9c970ef733894630 (commit)
      from  5243430d16282e96afcd7284fc0941575cf8973a (commit)

Summary of changes:
 bin/rt.in |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

- Log -----------------------------------------------------------------
commit c6970ddd7f22a3113091b65c9c970ef733894630
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Tue Feb 14 02:37:27 2012 +0800

    non-greedy regex is more right in splitting cf's name and value
    
    e.g. in "CF.{foobar}=ab{cd}=e", splitting from the first "=" is more resonable
    as cf names usually don't contain "}=", but values do.

diff --git a/bin/rt.in b/bin/rt.in
index 640c7b4..2a8a411 100755
--- a/bin/rt.in
+++ b/bin/rt.in
@@ -119,7 +119,7 @@ sub DEBUG { warn @_ if $config{debug} >= shift }
 # (XXX: Ask Autrijus how i18n changes these definitions.)
 
 my $name    = '[\w.-]+';
-my $CF_name = '[^,]+';
+my $CF_name = '[^,]+?';
 my $field   = '(?i:[a-z][a-z0-9_-]*|C(?:ustom)?F(?:ield)?-'.$CF_name.'|CF\.\{'.$CF_name.'\})';
 my $label   = '[a-zA-Z0-9 at _.+-]+';
 my $labels  = "(?:$label,)*$label";

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


More information about the Rt-commit mailing list