From 980898da89bd08768dadd4671c4095df316beb2d Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Wed, 14 Sep 2011 14:25:25 +0200
Subject: [PATCH] [ADD] correctly link log entries to the corresponding form
 view object

bzr revid: xmo@openerp.com-20110914122525-mo3oirufdgks83ug
---
 addons/web/static/src/js/views.js | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/addons/web/static/src/js/views.js b/addons/web/static/src/js/views.js
index b113c60afd3..0e4016f4250 100644
--- a/addons/web/static/src/js/views.js
+++ b/addons/web/static/src/js/views.js
@@ -462,10 +462,22 @@ openerp.web.ViewManagerAction = openerp.web.ViewManager.extend(/** @lends oepner
      * @param {Array<Object>} log_records
      */
     do_display_log: function (log_records) {
-        var logs = this.$element.find('ul.oe-view-manager-logs:first').empty();
+        var self = this,
+            $logs = this.$element.find('ul.oe-view-manager-logs:first').empty();
         _(log_records).each(function (record) {
-            // TODO: fix link: bind action?
-            logs.append(_.sprintf('<li><a href="#">%s</a></li>', record.name));
+            $(_.sprintf('<li><a href="#">%s</a></li>', record.name))
+                .appendTo($logs)
+                .delegate('a', 'click', function (e) {
+                    self.do_action({
+                        type: 'ir.actions.act_window',
+                        res_model: record.res_model,
+                        res_id: record.res_id,
+                        // TODO: need to have an evaluated context here somehow
+                        //context: record.context,
+                        views: [[false, 'form']]
+                    });
+                    return false;
+                });
         });
     }
 });