From 34818b7d7bd6bc5e6659555acbde6c3f5bdecbd3 Mon Sep 17 00:00:00 2001
From: metys <metys@9a5d940c-df78-4338-8860-37cbe7942ffe>
Date: Wed, 4 May 2011 07:12:41 +0000
Subject: [PATCH] If Explain creation fails:  + message "Failed to create..."
 above form  + log ("info") message  + send mail with invalid plan

---
 lib/Explain/Controller.pm          | 30 ++++++++++++++++++++++++++----
 templates/controller/index.html.ep |  4 ++++
 2 files changed, 30 insertions(+), 4 deletions(-)

diff --git a/lib/Explain/Controller.pm b/lib/Explain/Controller.pm
index 83a3461..fbd5c88 100755
--- a/lib/Explain/Controller.pm
+++ b/lib/Explain/Controller.pm
@@ -20,14 +20,36 @@ sub index {
     return $self->render( message => 'Your plan is too long.', status  => 413 )
         if 10_000_000 < length $plan;
 
-    # validate plan
+    # try
     eval {
+
+        # make "explain"
         my $explain = Pg::Explain->new( source => $plan );
-        $explain->top_node;
+
+        # something goes wrong...
+        die q|Can't create explain! Explain "top_node" is undef!|
+            unless defined $explain->top_node;
     };
 
-    # something goes wrong
-    return $self->render( message => q|Failed to parse your plan.| ) if $EVAL_ERROR;
+    # catch
+    if ( $EVAL_ERROR ) {
+
+        # log message
+        $self->app->log->info( $EVAL_ERROR );
+
+        # try
+        eval {
+
+            # send mail
+            $self->send_mail( {
+                subject => q|Can't create explain from...|,
+                msg     => $plan
+            } );
+        };
+
+        # leave...
+        return $self->render( message => q|Failed to parse your plan| );
+    }
 
     # public
     my $is_public = $self->req->param( 'is_public' ) ? 1 : 0;
diff --git a/templates/controller/index.html.ep b/templates/controller/index.html.ep
index 1c37075..bde6553 100755
--- a/templates/controller/index.html.ep
+++ b/templates/controller/index.html.ep
@@ -5,6 +5,10 @@
 
 <h1><%= $title =%></h1>
 
+% if ( stash( 'message' ) ) {
+    <p class="message"><%= stash( 'message' ) =%></p>
+% }
+
 <form id="new-explain" method="post" action="<%= url_for 'current' %>">
 
     <div class="fe fe-first fe_plan">