ymdp 0.5.1 → 0.6.0

data/VERSION

@@ -1 +1 @@
-0.5.1
+0.6.0

data/lib/ymdp/javascripts/ab_testing.js

@@ -83,7 +83,7 @@ ABTesting = {
   error: function(data) {
     Debug.log("applyError", data);
     if (data.error !== 3002) {
-      YAHOO.logger.error("Received body contents fetch error on page " + View.name + ": " + data.error + ' - ' + data.errorMsg);
+      Debug.error("Received body contents fetch error on page " + View.name + ": " + data.error + ' - ' + data.errorMsg);
     }
     YAHOO.init.showAndFinish();
   },
@@ -117,7 +117,7 @@ ABTesting = {
         try {
           $(content_id).update(response.html);
         } catch(omg) {
-          YAHOO.logger.error(omg);
+          Debug.error(omg);
         }
       }
       YAHOO.init.showAndFinish();

data/lib/ymdp/javascripts/ajax.js

@@ -33,7 +33,6 @@ OIB = {
         if (error_function) {
           error_function(response);
         } else {
-          Debug.error("No error_function", response);
           YAHOO.oib.showError(
             {
               "method": "OIB.call",
@@ -45,7 +44,6 @@ OIB = {
         if (success_function) {
           success_function(response); 
         } else {
-          Debug.error("no success function", response);
           YAHOO.oib.showError(
             {
               "method": "OIB.call",
@@ -61,13 +59,19 @@ OIB = {
 	ajax_response: false,
 	
 	ajax: function(url, method, params, success_function, error_function) {
+	  var debug;
+	  
 	  params = params || {};
 	  params["application"] = View.application;
 	  
-    Debug.log("OIB.ajax: About to call openmail.Application.callWebService: ", {
-      "method": method,
-      "url": url + "?" + Object.toQueryString(params)
-    });
+	  debug = !params["_hide_debug"];
+	  
+	  if (debug) {
+      Debug.log("OIB.ajax: About to call openmail.Application.callWebService: ", {
+        "method": method,
+        "url": url + "?" + Object.toQueryString(params)
+      });
+    }
     
     openmail.Application.callWebService(
     {
@@ -78,25 +82,30 @@ OIB = {
     function(response) {
       // response from Ajax call was a 200 response
       //
-      Debug.log("inside response from openMail.Application.callWebService", response);
+      if (debug) {
+        Debug.log("inside response from openMail.Application.callWebService", response);
+      }
       if (response.error) {
         // response has a parameter called "error"
         //
         if (error_function) {
     		  error_function(response);
     		} else {
-          OIB.error(url, params, response);
+    		  if (debug) {
+            OIB.error(url, params, response);
+          }
         }
       } else {
         // SUCCESSFUL RESPONSE
         //
         // response doesn't have a parameter called "error"
         // 
-        Debug.log("success response inside openMail.Application.callWebService", response);
     		try {
         	success_function(response);
     		} catch(e) {
-    		  Debug.log("Error in OIB.request success function", e);
+    		  if (debug) {
+      		  Debug.log("Error in OIB.request success function", e);
+    		  }
     		  YAHOO.oib.showError({
     		    "method": "OIB.request",
     		    "description": "exception caught in OIB.request success callback",
@@ -108,11 +117,16 @@ OIB = {
 	},
 	
 	request: function(oib_path, params, success_function, error_function, base_url) {
-    Debug.log("inside OIB.request: ", {
-      "oib_path": oib_path,
-      "params": Object.toJSON(params)
-    });
-    var oib_url, method;
+    var oib_url, method, debug;
+	  
+	  debug = !params["_hide_debug"];
+	  
+	  if (debug) {
+      Debug.log("inside OIB.request: ", {
+        "oib_path": oib_path,
+        "params": Object.toJSON(params)
+      });
+    }
   
     oib_url = base_url ? base_url : YAHOO.constants.base_url;
   
@@ -134,17 +148,23 @@ OIB = {
     }
     params.version = params.version || <%= @version %>;
   
-    Debug.log("about to call OIB.ajax");
+    if (debug) {
+      Debug.log("about to call OIB.ajax");
+    }
     
     OIB.ajax(oib_url, method, params, success_function, error_function);
   },
 	
   // overwrite this function locally if you need to
 	error: function(url, params, response) {
-	  var message;
+	  var message, debug;
+	  
+	  debug = !params["_hide_debug"];
 	  
-	  message = "OIB.error: " + Object.toJSON(response) + " calling url: " + url + "?" + Object.toQueryString(params);
-	  Debug.error(message);
+	  if (debug) {
+	    message = "OIB.error: " + Object.toJSON(response) + " calling url: " + url + "?" + Object.toQueryString(params);
+  	  Debug.error(message);
+	  }
 	},
 	
 	// advance the user to the next state in the signup process

data/lib/ymdp/javascripts/application.js

@@ -52,8 +52,6 @@ function formatUnixDate(unixtime) {
 YAHOO.oib.showError = function(options) {
   var params;
   
-  Debug.log("YAHOO.oib.showError", options);
-  
   options = options || {};
   
   if (options["heading"]) {

data/lib/ymdp/javascripts/debug.js

@@ -35,6 +35,7 @@ Debug = {
     if (this.consoleOn()) {
       console[level](message);
     }
+    Logger.observe(message);
   },
   
   log: function() {
@@ -89,16 +90,17 @@ Debug = {
   },
   
   timestamp: function() {
-    var time, hour, minute, second, timestamp, checktime;
+    var months, day, year, time, hour, minute, second, timestamp, checktime;
     
     time = new Date();
+    month = this.checktime(time.getMonth());
+    date = this.checktime(time.getDate());
+    year = this.checktime(time.getFullYear());
     hour = this.checktime(time.getHours());
     minute = this.checktime(time.getMinutes());
     second = this.checktime(time.getSeconds());
     
-    // timestamp = month + "/" + date + "/" + year + " " + 
-    
-    timestamp = hour + ":" + minute + ":" + second;
+    timestamp = month + "/" + date + "/" + year + " " + hour + ":" + minute + ":" + second;
     
     return timestamp;
   },
@@ -106,4 +108,4 @@ Debug = {
   generalInfo: function() {
     return "[<%= @domain %>/<%= @server %>] [<%= @version %> <%= @sprint_name %>]";
   }
-};
+};

data/lib/ymdp/javascripts/flash.js

@@ -61,7 +61,6 @@ Flash.setFlashClass = function(flash_id, type) {
 
 
 Flash.error = function(message) {
-  // YAHOO.logger.error(message);
   Flash.write("error", message, true);
 };
 

data/lib/ymdp/javascripts/init.js

@@ -92,6 +92,7 @@ YAHOO.init.after = function() {
 YAHOO.oib.init = function() {
   Debug.log("OIB.init for view " + View.name, "<%= @message %>");
   try {
+    Logger.init();
     YAHOO.init.browser();
     YAHOO.init.resources();
     I18n.addLanguageToBody();

data/lib/ymdp/javascripts/logger.js

@@ -1,64 +1,27 @@
-/*
-	  YMDP LOGGER
-
-		Send logging messages to OIB. 
-		
-		Messages are saved in log/ymdp.log
-
-
-  // DO NOT USE the @view instance variable in any files in /app/javascripts/base.
-  // The way they are cached makes it not safe to do so.
-
-*/
-
-
-/* set asset version */
-
-var LOGGER, Logger;
-
-LOGGER = {
-  VERSION: "<%= @hash %>",
-  MESSAGE: "<%= @message %>",
-  DEPLOYED: <%= Time.now.to_i %>,
-  DEPLOYED_STRING: "<%= Time.now.to_s %>"
-};
-
-	YAHOO.namespace("logger");
-
-	YAHOO.logger.write = function(level, message) {
-		if (YAHOO.oib.login) {
-			message = "[login: " + YAHOO.oib.login + "] " + message;
-		}
-		if (YAHOO.oib.guid) {
-			message = "[guid: " + YAHOO.oib.guid + "] " + message;
-		}
-	  OIB.post("ymdp/log",
-	    {
-	      level: level,
-	      message: message
-	    },
-	    function(response) {
-	      // log message written successfully
-	  });
-	};
-	
-	YAHOO.logger.debug = function(message) {
-	  YAHOO.logger.write("debug", message);
-	};
-	YAHOO.logger.info = function(message) {
-	  YAHOO.logger.write("info", message);
-	};
-	YAHOO.logger.warn = function(message) {
-	  YAHOO.logger.write("warn", message);
-	};
-	YAHOO.logger.error = function(message) {
-	  YAHOO.logger.write("warn", message);
-	};
-	YAHOO.logger.fatal = function(message) {
-	  YAHOO.logger.write("fatal", message);
-	};
-	
-
-Logger = YAHOO.logger;	
-
-// END YMDP LOGGER
+var Logger;
+
+Logger = {
+  on: false,
+  
+  init: function() {
+    OIB.get("ymdp/state", {}, function(response) {
+      Logger.on = response.observe;
+    });
+  },
+  
+  observe: function(message) {
+    if (this.on) {
+      this.log(message);
+    }
+  },
+  
+  log: function(message) {
+    // console.log("LOGGING " + message);
+    OIB.post("ymdp/logs", {
+      "_hide_debug": true,
+      "log": message
+    }, function() {
+      
+    });
+  }
+};

data/lib/ymdp/javascripts/reporter.js

@@ -14,7 +14,7 @@ var Reporter;
 //
 Reporter = {
   error: function(guid, params) {
-    Debug.log("Reporter.error", params);
+    // Debug.log("Reporter.error", params);
     Reporter.report(guid, "error", params);
   },
   
@@ -30,7 +30,7 @@ Reporter = {
     params["ymail_guid"] = guid;
     params["view"] = view;
     
-    Debug.log("Reporting guid " + guid + ", view " + view);
+    // Debug.log("Reporting guid " + guid + ", view " + view);
     Reporter.post(params);
   },
   
@@ -39,9 +39,9 @@ Reporter = {
   post: function(params) {
     params = params || {};
     OIB.post("ymdp/report", params, function(response) {
-      Debug.log("Reported page view", params);
+      // Debug.log("Reported page view", params);
     }, function(response) {
-      Debug.error("Error reporting page view with params", response);
+      // Debug.error("Error reporting page view with params", response);
     });
   },
 };

data/lib/ymdp/javascripts/user.js

@@ -13,7 +13,6 @@ YAHOO.oib.getUserState = function(success_function, error_function) {
 	  }
 	},
 	function() {
-    // YAHOO.logger.error("Failed to get user's state");
     Debug.log("Failed to get user's state");
     if (error_function) {
       error_function();
@@ -204,7 +203,6 @@ YAHOO.oib.deactivateUser = function() {
 			"ymail_wssid": ymail_wssid
 		}, 
 		function(response) {
-      // YAHOO.logger.info("Finished deactivating user");
 		  if (View.name !== "deactivate") {
 			  YAHOO.launcher.launchGoodbye();
 		  }

data/ymdp.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{ymdp}
-  s.version = "0.5.1"
+  s.version = "0.6.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Isaac Priestley"]
-  s.date = %q{2011-04-22}
+  s.date = %q{2011-05-06}
   s.description = %q{Framework for developing applications in the Yahoo! Mail Development Platform.}
   s.email = %q{progressions@gmail.com}
   s.extra_rdoc_files = [

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: ymdp
 version: !ruby/object:Gem::Version 
-  hash: 9
+  hash: 7
   prerelease: 
   segments: 
   - 0
-  - 5
-  - 1
-  version: 0.5.1
+  - 6
+  - 0
+  version: 0.6.0
 platform: ruby
 authors: 
 - Isaac Priestley
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-04-22 00:00:00 -05:00
+date: 2011-05-06 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency