Another in the series "I've asked for the strict checking, and I've got

one" ;)

1 files changed, 6 insertions, 4 deletions

diff --git a/lib/rhbzpage.js b/lib/rhbzpage.js
index f19d13b..e858145 100644
--- a/lib/rhbzpage.js
+++ b/lib/rhbzpage.js
@@ -99,7 +99,7 @@ var RHBugzillaPage = function RHBugzillaPage(win, config) {
         this.setDefaultAssignee();
     }
 
-    // Dig out backtrace
+    // Dig out backtrace protection against double-firing?
     this.btSnippet = "";
 
     var parseAbrtBacktraces = config.gJSONData.configData.parseAbrtBacktraces;
@@ -287,8 +287,10 @@ RHBugzillaPage.prototype.pasteBacktraceInComments = function() {
         // just the first one will do for now, we would need to do async
         // parsing
         btAttachments.forEach(function(x) {
-            attURL = "https://bugzilla.redhat.com/attachment.cgi?id="
+            var attURL = "https://bugzilla.redhat.com/attachment.cgi?id="
                     + x[1];
+            console.log("attURL = " + attURL);
+            console.log("this.btSnippet = " + this.btSnippet);
             if (!this.btSnippet) {
                 var that = this;
                 Request({
@@ -667,12 +669,12 @@ RHBugzillaPage.prototype.analyzeXorgLog = function analyzeXorgLog(attachID) {
     // doc.body.style.cursor = "wait";
     var preElem = doc.getElementById("textPre");
 
-    attURL = "https://bugzilla.redhat.com/attachment.cgi?id=" + attachID;
+    var attURL = "https://bugzilla.redhat.com/attachment.cgi?id=" + attachID;
     var that = this;
     Request({
         url: attURL,
         onComplete: function() {
-           if (this.response.status == 200) {
+            if (this.response.status == 200) {
                 var results = this.response.text.split("\n").
                     filter(function(line) {
                         return (that.RE.soughtLines.test(line));