From 74db7dd027ba87e4468875634d7fb6e7fbdf7c30 Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Tue, 4 Oct 2011 17:46:50 +0200
Subject: [PATCH] [FIX] correctly handle insertion in empty editable list

bzr revid: xmo@openerp.com-20111004154650-hx56g1lmefhzgf21
---
 addons/web/static/src/js/view_list_editable.js | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/addons/web/static/src/js/view_list_editable.js b/addons/web/static/src/js/view_list_editable.js
index 7daf2bd0685..41133c102d0 100644
--- a/addons/web/static/src/js/view_list_editable.js
+++ b/addons/web/static/src/js/view_list_editable.js
@@ -175,12 +175,16 @@ openerp.web.list_editable = function (openerp) {
                 if (row) {
                     $new_row.replaceAll(row);
                 } else if (self.options.editable) {
-                    if (self.options.editable === 'top') {
-                        $new_row.insertBefore(
-                            self.$current.find('> [data-id]:first'));
+                    if (self.records.length) {
+                        if (self.options.editable === 'top') {
+                            $new_row.insertBefore(
+                                self.$current.find('> [data-id]:first'));
+                        } else {
+                            $new_row.insertAfter(
+                                self.$current.find('> [data-id]:last'));
+                        }
                     } else {
-                        $new_row.insertAfter(
-                            self.$current.find('> [data-id]:last'));
+                        $new_row.prependTo(self.$current);
                     }
                     var $last_child = self.$current.find('> tr:last');
                     if ($last_child.is(':not([data-id])')) {