[Rt-commit] rt branch, 4.4/asset-search-cf-fix, created. rt-4.4.2-133-g1e74d6f13

? sunnavy sunnavy at bestpractical.com
Mon Apr 9 08:48:37 EDT 2018


The branch, 4.4/asset-search-cf-fix has been created
        at  1e74d6f13736ad25035a574252ffb172a7cbab82 (commit)

- Log -----------------------------------------------------------------
commit 1e74d6f13736ad25035a574252ffb172a7cbab82
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Mon Apr 9 20:11:42 2018 +0800

    Fix CF name escape for asset search's spreadsheet download
    
    Apparently the capture parenthesis was somehow missing.

diff --git a/share/html/Asset/Search/Results.tsv b/share/html/Asset/Search/Results.tsv
index 72026a1dd..de12d524d 100644
--- a/share/html/Asset/Search/Results.tsv
+++ b/share/html/Asset/Search/Results.tsv
@@ -61,7 +61,7 @@ $Format .= "$_, " for RT::Asset->Roles;
 my $CFs = RT::CustomFields->new( $session{CurrentUser} );
 $CFs->LimitToCatalog( $catalog_obj->Id );
 $CFs->LimitToObjectId( 0 ); # LimitToGlobal but no LookupType restriction
-$Format .= "'__CF.{$_}__/TITLE:$_', " for map {$_ = $_->Name; s/['\\]/\\$1/g; $_} @{$CFs->ItemsArrayRef};
+$Format .= "'__CF.{$_}__/TITLE:$_', " for map {$_ = $_->Name; s/(['\\])/\\$1/g; $_} @{$CFs->ItemsArrayRef};
 
 $m->callback(CallbackName => "ModifyFormat", Format => \$Format );
 

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


More information about the rt-commit mailing list