Stability: employ more sanity checks when parsing author details from history.js

Author: andreberg

html/views/history/history.js

@@ -32,15 +32,25 @@ var Commit = function(obj) {
 		}
 		this.header = this.raw.substring(0, messageStart);
 
-		var match = this.header.match(/\nauthor (.*) <(.*@.*|.*)> ([0-9].*)/);
-		if (!(match[2].match(/@[a-f0-9]{8}-[a-f0-9]{4}-[a-f0-9]{4}-[a-f0-9]{4}-[a-f0-9]{12}/)))
-			this.author_email = match[2];
+        if (typeof this.header !== 'undefined') {
+            var match = this.header.match(/\nauthor (.*) <(.*@.*|.*)> ([0-9].*)/);
+            if (typeof match !== 'undefined' && typeof match[2] !== 'undefined') {
+                if (!(match[2].match(/@[a-f0-9]{8}-[a-f0-9]{4}-[a-f0-9]{4}-[a-f0-9]{4}-[a-f0-9]{12}/)))
+                    this.author_email = match[2];
+                
+                this.author_date = new Date(parseInt(match[3]) * 1000);
+                
+                match = this.header.match(/\ncommitter (.*) <(.*@.*|.*)> ([0-9].*)/);
+                if (typeof match !== 'undefined') {
+                    this.committer_name = match[1];
+                    this.committer_email = match[2];
+                } else {
+                    this.committer_name = "undefined";
+                    this.committer_email = "undefined";
+                }
+            } 
+        }
 
-		this.author_date = new Date(parseInt(match[3]) * 1000);
-
-		match = this.header.match(/\ncommitter (.*) <(.*@.*|.*)> ([0-9].*)/);
-		this.committer_name = match[1];
-		this.committer_email = match[2];
 		this.committer_date = new Date(parseInt(match[3]) * 1000);		
 	}