From 2a776ecffc08bd8658a3a0daad9a307c605e871d Mon Sep 17 00:00:00 2001
From: tarteo <dennissluijk@outlook.com>
Date: Thu, 19 Jan 2017 21:31:24 +0100
Subject: [PATCH 1/5] [ADD] web_listview_range_select

---
 web_listview_range_select/README.rst          | 58 ++++++++++++++++++
 web_listview_range_select/__init__.py         |  3 +
 web_listview_range_select/__manifest__.py     | 23 +++++++
 .../src/js/web_listview_multi_select.js       | 61 +++++++++++++++++++
 .../src/js/web_listview_range_select.js       | 61 +++++++++++++++++++
 .../templates/assets.xml                      | 12 ++++
 6 files changed, 218 insertions(+)
 create mode 100644 web_listview_range_select/README.rst
 create mode 100644 web_listview_range_select/__init__.py
 create mode 100644 web_listview_range_select/__manifest__.py
 create mode 100644 web_listview_range_select/static/src/js/web_listview_multi_select.js
 create mode 100644 web_listview_range_select/static/src/js/web_listview_range_select.js
 create mode 100644 web_listview_range_select/templates/assets.xml

diff --git a/web_listview_range_select/README.rst b/web_listview_range_select/README.rst
new file mode 100644
index 00000000..8dfe526a
--- /dev/null
+++ b/web_listview_range_select/README.rst
@@ -0,0 +1,58 @@
+.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg
+   :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
+   :alt: License: AGPL-3
+
+====================
+List Range Selection
+====================
+
+Enables selecting a range of records using the shift key.
+
+Configuration
+=============
+
+No configuration is needed.
+
+Usage
+=====
+
+To use this module, you need to:
+
+#. click a record;
+#. hold shift and click another record;
+#. you can repeat this operation as many times as you want.
+
+.. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
+   :alt: Try me on Runbot
+   :target: https://runbot.odoo-community.org/runbot/162/10.0
+
+Bug Tracker
+===========
+
+Bugs are tracked on `GitHub Issues
+<https://github.com/OCA/162/issues>`_. In case of trouble, please
+check there if your issue has already been reported. If you spotted it first,
+help us smashing it by providing a detailed and welcomed feedback.
+
+Credits
+=======
+
+Contributors
+------------
+
+* Dennis Sluijk <d.sluijk@onestein.nl>
+
+Maintainer
+----------
+
+.. image:: https://odoo-community.org/logo.png
+   :alt: Odoo Community Association
+   :target: https://odoo-community.org
+
+This module is maintained by the OCA.
+
+OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.
+
+To contribute to this module, please visit https://odoo-community.org.
diff --git a/web_listview_range_select/__init__.py b/web_listview_range_select/__init__.py
new file mode 100644
index 00000000..2fb6bd8c
--- /dev/null
+++ b/web_listview_range_select/__init__.py
@@ -0,0 +1,3 @@
+# -*- coding: utf-8 -*-
+# © 2017 Onestein (<http://www.onestein.eu>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
diff --git a/web_listview_range_select/__manifest__.py b/web_listview_range_select/__manifest__.py
new file mode 100644
index 00000000..7da8ae3e
--- /dev/null
+++ b/web_listview_range_select/__manifest__.py
@@ -0,0 +1,23 @@
+# -*- coding: utf-8 -*-
+# © 2017 Onestein (<http://www.onestein.eu>)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+
+{
+    'name': 'List Range Selection',
+    'summary': """
+        Enables selecting a range of records using the shift key
+    """,
+    'version': '10.0.1.0.0',
+    'category': 'Web',
+    'author': 'Onestein,Odoo Community Association (OCA)',
+    'website': 'http://www.onestein.eu',
+    'license': 'AGPL-3',
+    'depends': [
+        'web',
+    ],
+    'data': [
+        'templates/assets.xml'
+    ],
+    'installable': True,
+    'application': False,
+}
diff --git a/web_listview_range_select/static/src/js/web_listview_multi_select.js b/web_listview_range_select/static/src/js/web_listview_multi_select.js
new file mode 100644
index 00000000..d338273c
--- /dev/null
+++ b/web_listview_range_select/static/src/js/web_listview_multi_select.js
@@ -0,0 +1,61 @@
+/* Copyright 2017 Onestein
+* License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). */
+
+odoo.define('web_listview_multi_select', function (require) {
+"use strict";
+    var core = require('web.core'),
+        _t = core._t,
+        listview = require('web.ListView');
+
+    listview.List.include({
+        get_range_selection: function() {
+            var result = {ids: [], records: []};
+            if (!this.options.selectable) {
+                return result;
+            }
+            var records = this.records;
+            var start = null,
+                end = null;
+
+            //Get start and end;
+            this.$current.find('td.o_list_record_selector input').each(function (i, el) {
+                var checked = $(el).prop('checked');
+                if (checked) {
+                    if (start == null)
+                        start = i;
+                    else
+                        end = i;
+                }
+            });
+
+            this.$current.find('td.o_list_record_selector input').closest('tr').each(function (i, el) {
+                var record = records.get($(el).data('id'));
+                if (start != null && end != null && i >= start && i <= end) {
+                    result.ids.push(record.get('id'));
+                    result.records.push(record.attributes);
+                } else if(start != null && end == null && start == i) {
+                    result.ids.push(record.get('id'));
+                    result.records.push(record.attributes);
+                }
+            });
+            return result;
+        },
+        init: function() {
+            var self = this;
+            var res = this._super.apply(this, arguments);
+            this.$current.delegate('td.o_list_record_selector', 'click', function (e) {
+                if (e.shiftKey) {
+                    var selection = self.get_range_selection();
+                    self.$current.find('td.o_list_record_selector input').closest('tr').each(function () {
+                        var record = self.records.get($(this).data('id'));
+                        if (selection.ids.indexOf(record.get('id')) != -1)
+                            $(this).find('td.o_list_record_selector input').prop('checked', true);
+                    });
+                    $(self).trigger(
+                        'selected', [selection.ids, selection.records, true]);
+                }
+            });
+            return res;
+        }
+    });
+});
diff --git a/web_listview_range_select/static/src/js/web_listview_range_select.js b/web_listview_range_select/static/src/js/web_listview_range_select.js
new file mode 100644
index 00000000..e0b70616
--- /dev/null
+++ b/web_listview_range_select/static/src/js/web_listview_range_select.js
@@ -0,0 +1,61 @@
+/* Copyright 2017 Onestein
+* License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). */
+
+odoo.define('web_listview_range_select', function (require) {
+"use strict";
+    var core = require('web.core'),
+        _t = core._t,
+        listview = require('web.ListView');
+
+    listview.List.include({
+        get_range_selection: function() {
+            var result = {ids: [], records: []};
+            if (!this.options.selectable) {
+                return result;
+            }
+            var records = this.records;
+            var start = null,
+                end = null;
+
+            //Get start and end;
+            this.$current.find('td.o_list_record_selector input').each(function (i, el) {
+                var checked = $(el).prop('checked');
+                if (checked) {
+                    if (start == null)
+                        start = i;
+                    else
+                        end = i;
+                }
+            });
+
+            this.$current.find('td.o_list_record_selector input').closest('tr').each(function (i, el) {
+                var record = records.get($(el).data('id'));
+                if (start != null && end != null && i >= start && i <= end) {
+                    result.ids.push(record.get('id'));
+                    result.records.push(record.attributes);
+                } else if(start != null && end == null && start == i) {
+                    result.ids.push(record.get('id'));
+                    result.records.push(record.attributes);
+                }
+            });
+            return result;
+        },
+        init: function() {
+            var self = this;
+            var res = this._super.apply(this, arguments);
+            this.$current.delegate('td.o_list_record_selector', 'click', function (e) {
+                if (e.shiftKey) {
+                    var selection = self.get_range_selection();
+                    self.$current.find('td.o_list_record_selector input').closest('tr').each(function () {
+                        var record = self.records.get($(this).data('id'));
+                        if (selection.ids.indexOf(record.get('id')) != -1)
+                            $(this).find('td.o_list_record_selector input').prop('checked', true);
+                    });
+                    $(self).trigger(
+                        'selected', [selection.ids, selection.records, true]);
+                }
+            });
+            return res;
+        }
+    });
+});
diff --git a/web_listview_range_select/templates/assets.xml b/web_listview_range_select/templates/assets.xml
new file mode 100644
index 00000000..30c767f4
--- /dev/null
+++ b/web_listview_range_select/templates/assets.xml
@@ -0,0 +1,12 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!-- Copyright 2017 Onestein
+     License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). -->
+
+<odoo>
+    <template id="assets_backend" inherit_id="web.assets_backend">
+        <xpath expr=".">
+            <script type="text/javascript"
+                    src="/web_listview_range_select/static/src/js/web_listview_range_select.js"/>
+        </xpath>
+    </template>
+</odoo>

From e86f7fc09deb7064a93f8d1d2834a17ed2720830 Mon Sep 17 00:00:00 2001
From: tarteo <dennissluijk@outlook.com>
Date: Mon, 23 Jan 2017 20:55:37 +0100
Subject: [PATCH 2/5] [ADD] Selecting multiple ranges

---
 .../src/js/web_listview_multi_select.js       | 61 -------------------
 .../src/js/web_listview_range_select.js       | 43 +++++++++++--
 2 files changed, 38 insertions(+), 66 deletions(-)
 delete mode 100644 web_listview_range_select/static/src/js/web_listview_multi_select.js

diff --git a/web_listview_range_select/static/src/js/web_listview_multi_select.js b/web_listview_range_select/static/src/js/web_listview_multi_select.js
deleted file mode 100644
index d338273c..00000000
--- a/web_listview_range_select/static/src/js/web_listview_multi_select.js
+++ /dev/null
@@ -1,61 +0,0 @@
-/* Copyright 2017 Onestein
-* License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). */
-
-odoo.define('web_listview_multi_select', function (require) {
-"use strict";
-    var core = require('web.core'),
-        _t = core._t,
-        listview = require('web.ListView');
-
-    listview.List.include({
-        get_range_selection: function() {
-            var result = {ids: [], records: []};
-            if (!this.options.selectable) {
-                return result;
-            }
-            var records = this.records;
-            var start = null,
-                end = null;
-
-            //Get start and end;
-            this.$current.find('td.o_list_record_selector input').each(function (i, el) {
-                var checked = $(el).prop('checked');
-                if (checked) {
-                    if (start == null)
-                        start = i;
-                    else
-                        end = i;
-                }
-            });
-
-            this.$current.find('td.o_list_record_selector input').closest('tr').each(function (i, el) {
-                var record = records.get($(el).data('id'));
-                if (start != null && end != null && i >= start && i <= end) {
-                    result.ids.push(record.get('id'));
-                    result.records.push(record.attributes);
-                } else if(start != null && end == null && start == i) {
-                    result.ids.push(record.get('id'));
-                    result.records.push(record.attributes);
-                }
-            });
-            return result;
-        },
-        init: function() {
-            var self = this;
-            var res = this._super.apply(this, arguments);
-            this.$current.delegate('td.o_list_record_selector', 'click', function (e) {
-                if (e.shiftKey) {
-                    var selection = self.get_range_selection();
-                    self.$current.find('td.o_list_record_selector input').closest('tr').each(function () {
-                        var record = self.records.get($(this).data('id'));
-                        if (selection.ids.indexOf(record.get('id')) != -1)
-                            $(this).find('td.o_list_record_selector input').prop('checked', true);
-                    });
-                    $(self).trigger(
-                        'selected', [selection.ids, selection.records, true]);
-                }
-            });
-            return res;
-        }
-    });
-});
diff --git a/web_listview_range_select/static/src/js/web_listview_range_select.js b/web_listview_range_select/static/src/js/web_listview_range_select.js
index e0b70616..622323bf 100644
--- a/web_listview_range_select/static/src/js/web_listview_range_select.js
+++ b/web_listview_range_select/static/src/js/web_listview_range_select.js
@@ -8,19 +8,23 @@ odoo.define('web_listview_range_select', function (require) {
         listview = require('web.ListView');
 
     listview.List.include({
-        get_range_selection: function() {
+        _range_history: [],
+        _get_range_selection: function() {
+            var self = this;
             var result = {ids: [], records: []};
             if (!this.options.selectable) {
                 return result;
             }
             var records = this.records;
+
+            //Get start and end
             var start = null,
                 end = null;
 
-            //Get start and end;
             this.$current.find('td.o_list_record_selector input').each(function (i, el) {
-                var checked = $(el).prop('checked');
-                if (checked) {
+                var id = $(el).closest('tr').data('id');
+                var checked = self._range_history.indexOf(id) != -1;
+                if (checked && $(el).prop('checked')) {
                     if (start == null)
                         start = i;
                     else
@@ -28,6 +32,23 @@ odoo.define('web_listview_range_select', function (require) {
                 }
             });
 
+            //Preserve already checked
+            this.$current.find('td.o_list_record_selector input:checked').closest('tr').each(function (i, el) {
+                if (i == start || i == end) return;
+                var record = records.get($(el).data('id'));
+                result.ids.push(record.get('id'));
+                result.records.push(record.attributes);
+            });
+
+            var new_range = this.get_range_selection(start, end);
+            result.records = result.records.concat(new_range.records);
+            result.ids = result.ids.concat(new_range.ids);
+
+            return result;
+        },
+        get_range_selection: function(start, end) {
+            var records = this.records;
+            var result = {ids: [], records: []};
             this.$current.find('td.o_list_record_selector input').closest('tr').each(function (i, el) {
                 var record = records.get($(el).data('id'));
                 if (start != null && end != null && i >= start && i <= end) {
@@ -40,17 +61,29 @@ odoo.define('web_listview_range_select', function (require) {
             });
             return result;
         },
+        push_range_history: function(id) {
+            this._range_history.push(id);
+            if (this._range_history.length == 3)
+                this._range_history.shift();
+        },
         init: function() {
             var self = this;
             var res = this._super.apply(this, arguments);
             this.$current.delegate('td.o_list_record_selector', 'click', function (e) {
+                //Update history
+                if ($(this).find('input').prop('checked'))
+                    self.push_range_history($(this).closest('tr').data('id'));
+
                 if (e.shiftKey) {
-                    var selection = self.get_range_selection();
+                    //Get selection
+                    var selection = self._get_range_selection();
                     self.$current.find('td.o_list_record_selector input').closest('tr').each(function () {
+                        //Check input visual
                         var record = self.records.get($(this).data('id'));
                         if (selection.ids.indexOf(record.get('id')) != -1)
                             $(this).find('td.o_list_record_selector input').prop('checked', true);
                     });
+                    //Check input internal
                     $(self).trigger(
                         'selected', [selection.ids, selection.records, true]);
                 }

From f4bcf9c0f283f9532dd46f4819df8b3498aab364 Mon Sep 17 00:00:00 2001
From: Aldo_Soares <soares_aldo@hotmail.com>
Date: Wed, 11 Apr 2018 16:50:08 -0300
Subject: [PATCH 3/5] [MIG] web_listview_range_select: Migration to 11.0

---
 web_listview_range_select/README.rst          |  7 +-
 web_listview_range_select/__init__.py         |  1 -
 web_listview_range_select/__manifest__.py     |  5 +-
 .../src/js/web_listview_range_select.js       | 76 +++++++++----------
 4 files changed, 42 insertions(+), 47 deletions(-)

diff --git a/web_listview_range_select/README.rst b/web_listview_range_select/README.rst
index 8dfe526a..bade96dd 100644
--- a/web_listview_range_select/README.rst
+++ b/web_listview_range_select/README.rst
@@ -1,5 +1,5 @@
-.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg
-   :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
+.. image:: https://img.shields.io/badge/license-AGPL--3-blue.png
+   :target: https://www.gnu.org/licenses/agpl
    :alt: License: AGPL-3
 
 ====================
@@ -24,7 +24,7 @@ To use this module, you need to:
 
 .. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
    :alt: Try me on Runbot
-   :target: https://runbot.odoo-community.org/runbot/162/10.0
+   :target: https://runbot.odoo-community.org/runbot/162/11.0
 
 Bug Tracker
 ===========
@@ -41,6 +41,7 @@ Contributors
 ------------
 
 * Dennis Sluijk <d.sluijk@onestein.nl>
+* Aldo Soares <soares_aldo@hotmail.com>
 
 Maintainer
 ----------
diff --git a/web_listview_range_select/__init__.py b/web_listview_range_select/__init__.py
index 2fb6bd8c..9b1a1095 100644
--- a/web_listview_range_select/__init__.py
+++ b/web_listview_range_select/__init__.py
@@ -1,3 +1,2 @@
-# -*- coding: utf-8 -*-
 # © 2017 Onestein (<http://www.onestein.eu>)
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
diff --git a/web_listview_range_select/__manifest__.py b/web_listview_range_select/__manifest__.py
index 7da8ae3e..019a2e27 100644
--- a/web_listview_range_select/__manifest__.py
+++ b/web_listview_range_select/__manifest__.py
@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 # © 2017 Onestein (<http://www.onestein.eu>)
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
@@ -7,10 +6,10 @@
     'summary': """
         Enables selecting a range of records using the shift key
     """,
-    'version': '10.0.1.0.0',
+    'version': '11.0.1.0.0',
     'category': 'Web',
     'author': 'Onestein,Odoo Community Association (OCA)',
-    'website': 'http://www.onestein.eu',
+    'website': 'https://github.com/oca/web',
     'license': 'AGPL-3',
     'depends': [
         'web',
diff --git a/web_listview_range_select/static/src/js/web_listview_range_select.js b/web_listview_range_select/static/src/js/web_listview_range_select.js
index 622323bf..74b29701 100644
--- a/web_listview_range_select/static/src/js/web_listview_range_select.js
+++ b/web_listview_range_select/static/src/js/web_listview_range_select.js
@@ -3,25 +3,21 @@
 
 odoo.define('web_listview_range_select', function (require) {
 "use strict";
-    var core = require('web.core'),
-        _t = core._t,
-        listview = require('web.ListView');
 
-    listview.List.include({
+    var ListRenderer = require('web.ListRenderer');
+
+    ListRenderer.include({
         _range_history: [],
+
         _get_range_selection: function() {
             var self = this;
             var result = {ids: [], records: []};
-            if (!this.options.selectable) {
-                return result;
-            }
-            var records = this.records;
 
             //Get start and end
             var start = null,
                 end = null;
 
-            this.$current.find('td.o_list_record_selector input').each(function (i, el) {
+            this.$el.find('td.o_list_record_selector input').each(function (i, el) {
                 var id = $(el).closest('tr').data('id');
                 var checked = self._range_history.indexOf(id) != -1;
                 if (checked && $(el).prop('checked')) {
@@ -33,11 +29,11 @@ odoo.define('web_listview_range_select', function (require) {
             });
 
             //Preserve already checked
-            this.$current.find('td.o_list_record_selector input:checked').closest('tr').each(function (i, el) {
+            this.$el.find('td.o_list_record_selector input:checked').closest('tr').each(function (i, el) {
                 if (i == start || i == end) return;
-                var record = records.get($(el).data('id'));
-                result.ids.push(record.get('id'));
-                result.records.push(record.attributes);
+                var record_id = $(el).data('id')
+                result.ids.push(record_id);
+                result.records.push(el.attributes);
             });
 
             var new_range = this.get_range_selection(start, end);
@@ -46,48 +42,48 @@ odoo.define('web_listview_range_select', function (require) {
 
             return result;
         },
+
         get_range_selection: function(start, end) {
-            var records = this.records;
             var result = {ids: [], records: []};
-            this.$current.find('td.o_list_record_selector input').closest('tr').each(function (i, el) {
-                var record = records.get($(el).data('id'));
+            this.$el.find('td.o_list_record_selector input').closest('tr').each(function (i, el) {
+                var record_id = $(el).data('id');
                 if (start != null && end != null && i >= start && i <= end) {
-                    result.ids.push(record.get('id'));
-                    result.records.push(record.attributes);
+                    result.ids.push(record_id);
+                    result.records.push(el.attributes);
                 } else if(start != null && end == null && start == i) {
-                    result.ids.push(record.get('id'));
-                    result.records.push(record.attributes);
+                    result.ids.push(record_id);
+                    result.records.push(el.attributes);
                 }
             });
             return result;
         },
+
         push_range_history: function(id) {
             this._range_history.push(id);
             if (this._range_history.length == 3)
                 this._range_history.shift();
         },
-        init: function() {
+
+        _onSelectRecord: function(event) {
+            var res = this._super(event);
             var self = this;
-            var res = this._super.apply(this, arguments);
-            this.$current.delegate('td.o_list_record_selector', 'click', function (e) {
-                //Update history
-                if ($(this).find('input').prop('checked'))
-                    self.push_range_history($(this).closest('tr').data('id'));
+            var el = $(event.currentTarget);
+            if (el.find('input').prop('checked'))
+                self.push_range_history(el.closest('tr').data('id'));
 
-                if (e.shiftKey) {
-                    //Get selection
-                    var selection = self._get_range_selection();
-                    self.$current.find('td.o_list_record_selector input').closest('tr').each(function () {
-                        //Check input visual
-                        var record = self.records.get($(this).data('id'));
-                        if (selection.ids.indexOf(record.get('id')) != -1)
-                            $(this).find('td.o_list_record_selector input').prop('checked', true);
-                    });
-                    //Check input internal
-                    $(self).trigger(
-                        'selected', [selection.ids, selection.records, true]);
-                }
-            });
+            if (event.shiftKey) {
+                //Get selection
+                var selection = self._get_range_selection();
+                this.$el.find('td.o_list_record_selector input').closest('tr').each(function () {
+                    //Check input visual
+                    var record_id = $(this).data('id');
+                    if (selection.ids.indexOf(record_id) != -1)
+                        $(this).find('td.o_list_record_selector input').prop('checked', true);
+                });
+                //Check input internal
+                $(self).trigger(
+                    'selected', [selection.ids, selection.records, true]);
+            }
             return res;
         }
     });

From 086c2a8b8dc8b21bce673b6ded049864671c4f7d Mon Sep 17 00:00:00 2001
From: oca-travis <oca+oca-travis@odoo-community.org>
Date: Sun, 24 Jun 2018 10:28:13 +0000
Subject: [PATCH 4/5] [UPD] Update web_listview_range_select.pot

---
 .../i18n/web_listview_range_select.pot             | 14 ++++++++++++++
 1 file changed, 14 insertions(+)
 create mode 100644 web_listview_range_select/i18n/web_listview_range_select.pot

diff --git a/web_listview_range_select/i18n/web_listview_range_select.pot b/web_listview_range_select/i18n/web_listview_range_select.pot
new file mode 100644
index 00000000..447d3bb3
--- /dev/null
+++ b/web_listview_range_select/i18n/web_listview_range_select.pot
@@ -0,0 +1,14 @@
+# Translation of Odoo Server.
+# This file contains the translation of the following modules:
+#
+msgid ""
+msgstr ""
+"Project-Id-Version: Odoo Server 11.0\n"
+"Report-Msgid-Bugs-To: \n"
+"Last-Translator: <>\n"
+"Language-Team: \n"
+"MIME-Version: 1.0\n"
+"Content-Type: text/plain; charset=UTF-8\n"
+"Content-Transfer-Encoding: \n"
+"Plural-Forms: \n"
+

From 6ec6e6aebaa1d221d759b37e92bfff01ec3317ff Mon Sep 17 00:00:00 2001
From: tarteo <d.sluijk@onestein.nl>
Date: Mon, 1 Oct 2018 15:34:15 +0200
Subject: [PATCH 5/5] [MIG] web_listview_range_select: Migration to 12.0

[FIX] Firefox shift click

[REM] Removed unnecessary configure section

[ADD] Roadmap with new features
---
 web_listview_range_select/README.rst          | 71 +++++++++++----
 web_listview_range_select/__manifest__.py     |  2 +-
 .../readme/CONTRIBUTORS.rst                   |  2 +
 .../readme/DESCRIPTION.rst                    |  1 +
 web_listview_range_select/readme/ROADMAP.rst  |  3 +
 web_listview_range_select/readme/USAGE.rst    |  5 +
 .../src/js/web_listview_range_select.js       | 91 ++++++++++---------
 7 files changed, 113 insertions(+), 62 deletions(-)
 create mode 100644 web_listview_range_select/readme/CONTRIBUTORS.rst
 create mode 100644 web_listview_range_select/readme/DESCRIPTION.rst
 create mode 100644 web_listview_range_select/readme/ROADMAP.rst
 create mode 100644 web_listview_range_select/readme/USAGE.rst

diff --git a/web_listview_range_select/README.rst b/web_listview_range_select/README.rst
index bade96dd..15458608 100644
--- a/web_listview_range_select/README.rst
+++ b/web_listview_range_select/README.rst
@@ -1,17 +1,36 @@
-.. image:: https://img.shields.io/badge/license-AGPL--3-blue.png
-   :target: https://www.gnu.org/licenses/agpl
-   :alt: License: AGPL-3
-
 ====================
 List Range Selection
 ====================
 
+.. !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! This file is generated by oca-gen-addon-readme !!
+   !! changes will be overwritten.                   !!
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+
+.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
+    :target: https://odoo-community.org/page/development-status
+    :alt: Beta
+.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png
+    :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
+    :alt: License: AGPL-3
+.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fweb-lightgray.png?logo=github
+    :target: https://github.com/OCA/web/tree/12.0/web_listview_range_select
+    :alt: OCA/web
+.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
+    :target: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_listview_range_select
+    :alt: Translate me on Weblate
+.. |badge5| image:: https://img.shields.io/badge/runbot-Try%20me-875A7B.png
+    :target: https://runbot.odoo-community.org/runbot/162/12.0
+    :alt: Try me on Runbot
+
+|badge1| |badge2| |badge3| |badge4| |badge5| 
+
 Enables selecting a range of records using the shift key.
 
-Configuration
-=============
+**Table of contents**
 
-No configuration is needed.
+.. contents::
+   :local:
 
 Usage
 =====
@@ -22,38 +41,50 @@ To use this module, you need to:
 #. hold shift and click another record;
 #. you can repeat this operation as many times as you want.
 
-.. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
-   :alt: Try me on Runbot
-   :target: https://runbot.odoo-community.org/runbot/162/11.0
+Known issues / Roadmap
+======================
+
+* Allow to click on the whole line (not just the checkbox) to select a row.
+* Enable the same behaviour with Ctrl button.
+* Shift and drag to select rows.
 
 Bug Tracker
 ===========
 
-Bugs are tracked on `GitHub Issues
-<https://github.com/OCA/162/issues>`_. In case of trouble, please
-check there if your issue has already been reported. If you spotted it first,
-help us smashing it by providing a detailed and welcomed feedback.
+Bugs are tracked on `GitHub Issues <https://github.com/OCA/web/issues>`_.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us smashing it by providing a detailed and welcomed
+`feedback <https://github.com/OCA/web/issues/new?body=module:%20web_listview_range_select%0Aversion:%2012.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+
+Do not contact contributors directly about support or help with technical issues.
 
 Credits
 =======
 
+Authors
+~~~~~~~
+
+* Onestein
+
 Contributors
-------------
+~~~~~~~~~~~~
 
 * Dennis Sluijk <d.sluijk@onestein.nl>
 * Aldo Soares <soares_aldo@hotmail.com>
 
-Maintainer
-----------
+Maintainers
+~~~~~~~~~~~
+
+This module is maintained by the OCA.
 
 .. image:: https://odoo-community.org/logo.png
    :alt: Odoo Community Association
    :target: https://odoo-community.org
 
-This module is maintained by the OCA.
-
 OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.
 
-To contribute to this module, please visit https://odoo-community.org.
+This module is part of the `OCA/web <https://github.com/OCA/web/tree/12.0/web_listview_range_select>`_ project on GitHub.
+
+You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.
diff --git a/web_listview_range_select/__manifest__.py b/web_listview_range_select/__manifest__.py
index 019a2e27..e2e0ee02 100644
--- a/web_listview_range_select/__manifest__.py
+++ b/web_listview_range_select/__manifest__.py
@@ -6,7 +6,7 @@
     'summary': """
         Enables selecting a range of records using the shift key
     """,
-    'version': '11.0.1.0.0',
+    'version': '12.0.1.0.0',
     'category': 'Web',
     'author': 'Onestein,Odoo Community Association (OCA)',
     'website': 'https://github.com/oca/web',
diff --git a/web_listview_range_select/readme/CONTRIBUTORS.rst b/web_listview_range_select/readme/CONTRIBUTORS.rst
new file mode 100644
index 00000000..761e89a4
--- /dev/null
+++ b/web_listview_range_select/readme/CONTRIBUTORS.rst
@@ -0,0 +1,2 @@
+* Dennis Sluijk <d.sluijk@onestein.nl>
+* Aldo Soares <soares_aldo@hotmail.com>
diff --git a/web_listview_range_select/readme/DESCRIPTION.rst b/web_listview_range_select/readme/DESCRIPTION.rst
new file mode 100644
index 00000000..a2e7858d
--- /dev/null
+++ b/web_listview_range_select/readme/DESCRIPTION.rst
@@ -0,0 +1 @@
+Enables selecting a range of records using the shift key.
diff --git a/web_listview_range_select/readme/ROADMAP.rst b/web_listview_range_select/readme/ROADMAP.rst
new file mode 100644
index 00000000..4efed838
--- /dev/null
+++ b/web_listview_range_select/readme/ROADMAP.rst
@@ -0,0 +1,3 @@
+* Allow to click on the whole line (not just the checkbox) to select a row.
+* Enable the same behaviour with Ctrl button.
+* Shift and drag to select rows.
diff --git a/web_listview_range_select/readme/USAGE.rst b/web_listview_range_select/readme/USAGE.rst
new file mode 100644
index 00000000..12e9db1e
--- /dev/null
+++ b/web_listview_range_select/readme/USAGE.rst
@@ -0,0 +1,5 @@
+To use this module, you need to:
+
+#. click a record;
+#. hold shift and click another record;
+#. you can repeat this operation as many times as you want.
diff --git a/web_listview_range_select/static/src/js/web_listview_range_select.js b/web_listview_range_select/static/src/js/web_listview_range_select.js
index 74b29701..edbe2824 100644
--- a/web_listview_range_select/static/src/js/web_listview_range_select.js
+++ b/web_listview_range_select/static/src/js/web_listview_range_select.js
@@ -9,80 +9,89 @@ odoo.define('web_listview_range_select', function (require) {
     ListRenderer.include({
         _range_history: [],
 
-        _get_range_selection: function() {
+        _render: function() {
+            var res = this._super.apply(this, arguments);
+            this.$table = this.$el.find('.o_list_view');
+            return res;
+        },
+
+        _getRangeSelection: function() {
             var self = this;
-            var result = {ids: [], records: []};
 
-            //Get start and end
+            // Get start and end
             var start = null,
                 end = null;
 
             this.$el.find('td.o_list_record_selector input').each(function (i, el) {
                 var id = $(el).closest('tr').data('id');
-                var checked = self._range_history.indexOf(id) != -1;
-                if (checked && $(el).prop('checked')) {
-                    if (start == null)
+                var checked = self._range_history.indexOf(id) !== -1;
+                if (checked && $(el).is(':checked')) {
+                    if (start == null) {
                         start = i;
-                    else
+                    } else {
                         end = i;
+                    }
                 }
             });
+            var new_range = this._getSelectionByRange(start, end);
 
-            //Preserve already checked
-            this.$el.find('td.o_list_record_selector input:checked').closest('tr').each(function (i, el) {
-                if (i == start || i == end) return;
-                var record_id = $(el).data('id')
-                result.ids.push(record_id);
-                result.records.push(el.attributes);
-            });
-
-            var new_range = this.get_range_selection(start, end);
-            result.records = result.records.concat(new_range.records);
-            result.ids = result.ids.concat(new_range.ids);
-
-            return result;
+            var current_selection = this.selection;
+            current_selection = _.uniq(current_selection.concat(new_range));
+            return current_selection;
         },
 
-        get_range_selection: function(start, end) {
-            var result = {ids: [], records: []};
+        _getSelectionByRange: function(start, end) {
+            var result = [];
             this.$el.find('td.o_list_record_selector input').closest('tr').each(function (i, el) {
                 var record_id = $(el).data('id');
                 if (start != null && end != null && i >= start && i <= end) {
-                    result.ids.push(record_id);
-                    result.records.push(el.attributes);
+                    result.push(record_id);
                 } else if(start != null && end == null && start == i) {
-                    result.ids.push(record_id);
-                    result.records.push(el.attributes);
+                    result.push(record_id);
                 }
             });
             return result;
         },
 
-        push_range_history: function(id) {
+        _pushRangeHistory: function(id) {
+            if (this._range_history.length === 2) {
+                this._range_history = [];
+            }
             this._range_history.push(id);
-            if (this._range_history.length == 3)
-                this._range_history.shift();
+        },
+
+        _deselectTable: function() {
+            // This is needed because the checkboxes are not real checkboxes.
+            window.getSelection().removeAllRanges();
         },
 
         _onSelectRecord: function(event) {
-            var res = this._super(event);
-            var self = this;
             var el = $(event.currentTarget);
-            if (el.find('input').prop('checked'))
-                self.push_range_history(el.closest('tr').data('id'));
+            var res = this._super(event);
+
+            // Firefox shift click fix
+            if (/firefox/i.test(navigator.userAgent) && event.shiftKey) {
+                el.find('input').prop('checked', !el.find('input').prop('checked'));
+            }
+
+            if (el.find('input').prop('checked')) {
+                this._pushRangeHistory(el.closest('tr').data('id'));
+            }
 
             if (event.shiftKey) {
-                //Get selection
-                var selection = self._get_range_selection();
-                this.$el.find('td.o_list_record_selector input').closest('tr').each(function () {
-                    //Check input visual
+                // Get selection
+                var selection = this._getRangeSelection();
+                var $rows = this.$el.find('td.o_list_record_selector input').closest('tr');
+                $rows.each(function () {
+                    // Check input visual
                     var record_id = $(this).data('id');
-                    if (selection.ids.indexOf(record_id) != -1)
+                    if (selection.indexOf(record_id) !== -1) {
                         $(this).find('td.o_list_record_selector input').prop('checked', true);
+                    }
                 });
-                //Check input internal
-                $(self).trigger(
-                    'selected', [selection.ids, selection.records, true]);
+                // Update selection internally
+                this._updateSelection();
+                this._deselectTable();
             }
             return res;
         }