Abort processing on error

In relation to https://rt.cpan.org/Ticket/Display.html?id=111355 and gh #1, errors should be a stopping condition.
RexOps · Apr 13, 2021 · cf937eb · cf937eb
1 parent 8ef8d4f
commit cf937eb
Showing 1 changed file with 18 additions and 5 deletions.
diff --git a/lib/Rex/Template/TT.pm b/lib/Rex/Template/TT.pm
@@ -46,9 +46,16 @@ sub template_toolkit {
 
   # process template
   my $output = '';
-  my $template = Template->new( { ABSOLUTE => 1 } );
+  my $template = Template->new( { ABSOLUTE => 1 } )
+    or do {
+      Rex::Logger::info( $Template::ERROR, 'error' );
+      die $Template::ERROR;
+    };
   $template->process( $template_path, $vars, \$output )
-    || Rex::Logger::info( $template->error(), 'error' );
+    or do {
+      Rex::Logger::info( $template->error(), 'error' );
+      die $template->error();
+    };
 
   return $output;
 }
@@ -65,11 +72,17 @@ sub import {
 
       validate_vars( $vars );
 
-      my $template = Template->new;
-
+      my $template = Template->new()
+        or do {
+          Rex::Logger::info( $Template::ERROR, 'error' );
+          die $Template::ERROR;
+        };
       my $output;
       $template->process( \$content, $vars, \$output )
-        || Rex::Logger::info( $template->error(), 'error' );
+        or do {
+          Rex::Logger::info( $template->error(), 'error' );
+          die $template->error();
+        };
 
       return $output;
     };