[Rt-commit] r10507 - in rt/branches/3.7-EXPERIMENTAL: . docs

[ruz at bestpractical.com]

Author: ruz
Date: Sat Jan 26 20:01:11 2008
New Revision: 10507

Modified:
   rt/branches/3.7-EXPERIMENTAL/   (props changed)
   rt/branches/3.7-EXPERIMENTAL/docs/extending_clickable_links.pod
   rt/branches/3.7-EXPERIMENTAL/html/Elements/MakeClicky

Log:
 r10491 at cubic-pc (orig r10457):  ruz | 2008-01-24 01:38:00 +0300
 * improve make clicky actions, update its docs


Modified: rt/branches/3.7-EXPERIMENTAL/docs/extending_clickable_links.pod
==============================================================================
--- rt/branches/3.7-EXPERIMENTAL/docs/extending_clickable_links.pod	(original)
+++ rt/branches/3.7-EXPERIMENTAL/docs/extending_clickable_links.pod	Sat Jan 26 20:01:11 2008
@@ -92,9 +92,28 @@
 
 =back
 
+=head2 Actions' arguments
+
+A hash is passed to action with two keys that always exist:
+
+=over 4
+
+=item value - full match of the regular expression, this block of text will be
+replaced with action's result.
+
+=item all_matches - array with all matches including groups defined in the
+regular expression, for example if your regexp is C<qr{ticket\s+#(\d+)}> then
+the first element will be full match ("ticket #XXX") the same as in 'value' key,
+but the second one element of the array will be id of a ticket (XXX), so you
+can avoid parsing value in the action. Only eight groups of your regexps are
+passed to actions.
+
+=back
+
 =head2 Custom MakeClicky action example
 
-Create a new file F</opt/rt3/local/html/MyCallbacks/Elements/MakeClicky/Default> with the content:
+Create a new file F</opt/rt3/local/html/MyCallbacks/Elements/MakeClicky/Default>
+with the content:
 
   <%ARGS>
   $types   => []
@@ -106,7 +125,7 @@
   # action that takes ticket ID as argument and returns link to the ticket
   $actions->{'link_ticket'} = sub {
       my %args = @_;
-      my ($id) = ($args{'value'} =~ /(\d+)/);
+      my $id = $args{'all_matches'}[1];
       return qq{<a href="$web_path/Ticket/Display.html?id=$id">$args{value}</a>};
   };
   
@@ -115,7 +134,7 @@
       # name, that should be used in config to activate action
       name   => 'short_ticket_link',
       # regular expression that matches text 'ticket #xxx'
-      regex  => qr{ticket\s+#\d+}i,
+      regex  => qr{ticket\s+#(\d+)}i,
       # name of the action that should be applied
       action => 'link_ticket',
   };
@@ -124,7 +143,7 @@
 That's all. Add C<short_ticket_link> to C<@Active_MakeClicky> option in your C<RT_SiteConfig.pm>.
 Restart your server and create test ticket with 'ticket #1' text.
 
-=head3 Notes for custom clicky actions writers
+=head2 Notes for custom clicky actions writers
 
 Note that an action B<must escape> illegal HTML characters with entities and/or
 arguments in URLs.

Modified: rt/branches/3.7-EXPERIMENTAL/html/Elements/MakeClicky
==============================================================================
--- rt/branches/3.7-EXPERIMENTAL/html/Elements/MakeClicky	(original)
+++ rt/branches/3.7-EXPERIMENTAL/html/Elements/MakeClicky	Sat Jan 26 20:01:11 2008
@@ -142,8 +142,11 @@
         substr( $$content, $pos, $skipped_len ) = $plain;
         $pos += length($plain);
     }
-
-    my $plain = $handle->( %ARGS, value => $match );
+    my $plain = $handle->(
+        %ARGS, 
+        value => $match,
+        all_matches => [ $1, $2, $3, $4, $5, $6, $7, $8, $9 ],
+    );
     substr( $$content, $pos, length($match) ) = $plain;
     pos($$content) = ( $pos += length($plain) );
 
@@ -167,7 +170,7 @@
    then 30 chars. This allow browser to wrap long URLs and avoid
    horizontal scrolling.
 
-To extend this with your own types od data, use the callback.
+To extend this with your own types of data, use the callback.
 It will be provided with:
 
  * 'types': An array reference of hash references.  Modify this array
@@ -187,4 +190,6 @@
     have to. This can be used to add pre- or post-processing around
     all actions.
 
+Read more about writing new actions in docs/extending_clickable_links.pod
+
 </%doc>