From 8e63763a77a0dd81791cb2c68fdd905b3e518bba Mon Sep 17 00:00:00 2001
From: Matěj Cepl <mcepl@redhat.com>
Date: Mon, 21 Jun 2010 15:41:22 +0200
Subject: Clean up logging

- remove unnecessary logging messages
- instead of dumping whole log store to stderr, give just count of items
---
 lib/bzpage.js | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

(limited to 'lib/bzpage.js')

diff --git a/lib/bzpage.js b/lib/bzpage.js
index 5344fbb..4906d13 100644
--- a/lib/bzpage.js
+++ b/lib/bzpage.js
@@ -50,7 +50,7 @@ var BZPage = function BZPage(doc, config) {
     if ("submitsLogging" in config.gJSONData.configData &&
         config.gJSONData.configData.submitsLogging) {
         this.log = config.logger;
-        console.log("this.log = " + this.log.toSource());
+        console.log("length of this.log.store = " + this.log.getLength());
         this.setUpLogging();
     }
 
@@ -69,7 +69,6 @@ var BZPage = function BZPage(doc, config) {
     var commentArea = this.doc.getElementById("comment_status_commit");
     if (commentArea) {
         var brElementPlacer = commentArea.getElementsByTagName("br");
-        console.log("brElementPlacer.length = " + brElementPlacer.length);
         brElementPlacer = brElementPlacer[0];
         if (brElementPlacer) {
             brElementPlacer.setAttribute("id","brElementPlacer_location");
@@ -316,7 +315,7 @@ BZPage.prototype.createNewButton = function createNewButton (location, after, pk
 
     // protection against double-firings
     if (this.doc.getElementById(newId)) {
-        console.error("Element with id " + newId + "already exists!");
+        console.log("Element with id " + newId + "already exists!");
         return ;
     }
 
@@ -550,7 +549,7 @@ BZPage.prototype.getOptionValue = function getOptionValue (id) {
     if (element) {
         return element.value;
     } else {
-        console.log("Failed to find element with id = " + id);
+        console.error("Failed to find element with id = " + id);
         return "#NA";
     }
 };
@@ -662,7 +661,6 @@ BZPage.prototype.setUpLogging = function setUpLogging () {
             console.log("Submit callback!");
 
             var resp = that.log.addLogRecord(that);
-            console.log("resp = " + resp);
             if (resp === null) {
                 console.log("Avoiding submitting!");
                 // FIXME doesn't work ... still submitting'
-- 
cgit