Add a git log view.

plietar · plietar · commit accb9ce509bf · 2014-03-22T10:56:36.000+01:00
diff --git a/coder-apps/common/vcs/app/gitapp.js b/coder-apps/common/vcs/app/gitapp.js
@@ -5,7 +5,7 @@ var Git = require("./git");
 var App = coderlib.App;
 var LocalApp = coderlib.LocalApp;
 
-var GitApp = exports = function(name, rev) {
+var GitApp = module.exports = function(name, rev) {
     GitApp.super_.call(this, name);
 
     this.revision = rev;
@@ -97,19 +97,26 @@ GitApp.history = function(name, callback) {
 
             (function loadRev(rev) {
                 repo.parseCommit(rev, function(err, commit) {
-                    if (err)
+                    if (err) {
                         callback(err);
-                    else {
-                        commits.push(commit);
-                        if (commit.parents.length == 0)
-                            callback(null, commits);
-                        else
-                            loadRev(commit.parents[0]);
+                        return;
                     }
+                    repo.rev_parse(["--short", rev], function(err, short_rev) {
+                        commit.revision = rev;
+                        commit.short = short_rev;
+                        if (err)
+                            callback(err);
+                        else {
+                            commits.push(commit);
+                            if (commit.parents.length == 0)
+                                callback(null, commits);
+                            else
+                                loadRev(commit.parents[0]);
+                        }
+                    });
                 });
             })(revision);
         }
     ], callback);
 };
 
-
diff --git a/coder-apps/common/vcs/app/index.js b/coder-apps/common/vcs/app/index.js
@@ -12,7 +12,8 @@ var isVersionned = function(app, callback) {
 
 exports.get_routes = [
     { path:'/', handler:'index_handler' },
-    { path:/^\/view\/(\w+)\/([0-9a-f]+)\/static\/(.+)$/, handler: 'static_handler' }
+    { path:/^\/view\/(\w+)\/([0-9a-f]+)\/static\/(.+)$/, handler: 'static_handler' },
+    { path:/^\/log\/(\w+)\/?$/, handler:'log_handler' }
 ];
 
 exports.post_routes = [
@@ -55,6 +56,16 @@ exports.static_handler = function( app, req, res, match ) {
     });
 };
 
+exports.log_handler = function( app, req, res, match ) {
+    var appname = match[1];
+
+    GitApp.history(appname, function(err, results) {
+        console.log(err);
+        res.render( app.view("log"), {commits: results} );
+    });
+};
+
+
 exports.on_destroy = function() {
 };
 
diff --git a/coder-apps/common/vcs/views/log.html b/coder-apps/common/vcs/views/log.html
@@ -0,0 +1,34 @@
+<!DOCTYPE html>
+<html>
+<head> 
+    <title>Coder</title>
+    <meta charset="utf-8">
+    <!-- Standard Coder Includes -->
+    <script>
+        var appname = "{{app_name}}"; //app name (id) of this app
+        var appurl = "{{&app_url}}";
+        var staticurl = "{{&static_url}}"; //base path to your static files /static/apps/yourapp
+    </script>
+    <link href="/static/apps/coderlib/css/index.css" media="screen" rel="stylesheet" type="text/css"/>
+    <script src="/static/common/js/jquery.min.js"></script>
+    <script src="/static/common/ace-min/ace.js" type="text/javascript" charset="utf-8"></script>
+    <script src="/static/apps/coderlib/js/index.js"></script>
+    <!-- End Coder Includes -->
+
+    <!-- This app's includes -->
+    <link href="{{&static_url}}/css/index.css" media="screen" rel="stylesheet" type="text/css"/>
+    <script src="{{&static_url}}/js/index.js"></script>
+    <!-- End apps includes -->
+</head>
+<body class="">
+    <div class="pagecontent">
+        <h1>Log</h1>
+        <ul>
+            {{#commits}}
+            <li>{{message}} - {{short}}</li>
+            {{/commits}}
+        </ul>
+    </div>
+</body>
+</html>
+
