[Rt-commit] rt branch, 4.2/cf-grouping-link, created. rt-4.1.6-344-g4a01441

[Alex Vandiver]

The branch, 4.2/cf-grouping-link has been created
        at  4a01441f2f1f7f1153bb5ae7d2e54bd6af10be10 (commit)

- Log -----------------------------------------------------------------
commit 4a01441f2f1f7f1153bb5ae7d2e54bd6af10be10
Author: Alex Vandiver <alexmv at bestpractical.com>
Date:   Tue Mar 12 17:02:21 2013 -0400

    Add grouping information to links on CF grouping headers
    
    This is useful for other objct types which may split CF editing into
    separate pages for each CF grouping.

diff --git a/share/html/Elements/ShowCustomFieldCustomGroupings b/share/html/Elements/ShowCustomFieldCustomGroupings
index d3f65fc..2a6ed96 100644
--- a/share/html/Elements/ShowCustomFieldCustomGroupings
+++ b/share/html/Elements/ShowCustomFieldCustomGroupings
@@ -50,6 +50,7 @@
     title => $group? loc($group) : loc('Custom Fields'),
     class => $css_class .' '. ($group? CSSClass("$css_class-$group") : ''),
     hide_empty => 1,
+    title_href => $title_href ? "$title_href?id=".$Object->id.($group?";Grouping=".$m->interp->apply_escapes($group, 'u'):"") : undef,
     %$TitleBoxARGS,
 &>
 <& ShowCustomFields, %ARGS, Object => $Object, Grouping => $group &>
@@ -57,6 +58,7 @@
 % }
 <%ARGS>
 $Object
+$title_href => ""
 </%ARGS>
 <%INIT>
 my $css_class = lc(ref($Object)||$Object);
diff --git a/share/html/Ticket/Elements/ShowSummary b/share/html/Ticket/Elements/ShowSummary
index 35ce442..98dde90 100644
--- a/share/html/Ticket/Elements/ShowSummary
+++ b/share/html/Ticket/Elements/ShowSummary
@@ -56,11 +56,7 @@
 
     <& /Elements/ShowCustomFieldCustomGroupings,
         Object       => $Ticket,
-        TitleBoxARGS => {
-            (($can_modify || $can_modify_cf)
-                ? (title_href => RT->Config->Get('WebPath')."/Ticket/Modify.html?id=".$Ticket->Id)
-                : ()),
-        },
+        title_href   => ($can_modify || $can_modify_cf) ? RT->Config->Get('WebPath')."/Ticket/Modify.html" : "",
         &>
 
     <&| /Widgets/TitleBox, title => loc('People'),

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