From 1774b91d91558e53a9ec635e4001f805c4a897a8 Mon Sep 17 00:00:00 2001
From: Dave Lawrence <dlawrence@mozilla.com>
Date: Tue, 14 May 2013 12:39:48 -0400
Subject: Bug 872022 - don't link a review flag to splinter unless the
 attachment is a patch

---
 extensions/MyDashboard/web/js/flags.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'extensions/MyDashboard/web/js')

diff --git a/extensions/MyDashboard/web/js/flags.js b/extensions/MyDashboard/web/js/flags.js
index 9c4da50dc..2498ba8ae 100644
--- a/extensions/MyDashboard/web/js/flags.js
+++ b/extensions/MyDashboard/web/js/flags.js
@@ -93,7 +93,10 @@ YUI({
     };
 
     var flagNameFormatter = function (o) {
-        if (o.data.attach_id && o.data.is_patch && MyDashboard.splinter_base) {
+        if (parseInt(o.data.attach_id)
+            && parseInt(o.data.is_patch)
+            && MyDashboard.splinter_base)
+        {
             return '<a href="' + MyDashboard.splinter_base +
                    (MyDashboard.splinter_base.indexOf('?') == -1 ? '?' : '&') +
                    'bug=' + encodeURIComponent(o.data.bug_id) +
-- 
cgit v1.2.3-24-g4f1b