From 56aedb8f50898a772b0b8d08be91aa69bbc011e5 Mon Sep 17 00:00:00 2001
From: mattab <matthieu.aubry@gmail.com>
Date: Tue, 13 May 2014 17:43:47 +1200
Subject: Refs #4877 Tweak the feedback@piwik email subject ftw

---
 plugins/Feedback/API.php | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

(limited to 'plugins/Feedback/API.php')

diff --git a/plugins/Feedback/API.php b/plugins/Feedback/API.php
index 73b3dbaf02..b5d601cb66 100644
--- a/plugins/Feedback/API.php
+++ b/plugins/Feedback/API.php
@@ -44,13 +44,20 @@ class API extends \Piwik\Plugin\API
         }
 
         $body = sprintf("Feature: %s\nLike: %s\n", $featureName, $likeText, $message);
-        if (!empty($message)) {
-            $body .= sprintf("Feedback:\n%s\n", $message);
-        } else {
-            $body .= "No feedback\n";
+
+        $feedbackMessage = '';
+        if (!empty($message) && $message != 'undefined') {
+            $feedbackMessage = sprintf("Feedback:\n%s\n", trim($message));
         }
+        $body .= $feedbackMessage;
+
+        $subject = sprintf("%s for %s %s",
+            empty($like) ? "-1" : "+1",
+            $featureName,
+            empty($feedbackMessage) ? "" : "(w/ feedback)"
+        );
 
-        $this->sendMail($featureName, $body);
+        $this->sendMail($subject, $body);
     }
 
     private function sendMail($subject, $body)
-- 
cgit v1.2.3