From 9b3f8246e1cdbee87fd0f5efe7c33a16239a7d69 Mon Sep 17 00:00:00 2001
From: qsm-odoo <qsm@odoo.com>
Date: Mon, 14 Sep 2015 14:50:42 +0200
Subject: [PATCH] [REF] account_asset: widgetonbutton widget (LESS/JS/XML)

The widget was still using gtk icons

* Delete useless template (use JS+LESS)
* JS Refactoring
---
 .../static/src/js/account_asset.js            | 34 +++++++------------
 .../static/src/less/account_asset.less        |  6 ++++
 .../static/src/xml/account_asset_template.xml |  8 -----
 addons/account_asset/views/account_asset.xml  |  1 +
 4 files changed, 20 insertions(+), 29 deletions(-)
 create mode 100644 addons/account_asset/static/src/less/account_asset.less
 delete mode 100644 addons/account_asset/static/src/xml/account_asset_template.xml

diff --git a/addons/account_asset/static/src/js/account_asset.js b/addons/account_asset/static/src/js/account_asset.js
index dd79e4e5e93..69c4fc1f523 100644
--- a/addons/account_asset/static/src/js/account_asset.js
+++ b/addons/account_asset/static/src/js/account_asset.js
@@ -7,34 +7,26 @@ Details : called in list view with "<button name="create_move" type="object" wid
 odoo.define('account_asset.widget', function(require) {
 "use strict";
 
-// openerp.account_asset = function (instance) {
-    // var _t = instance.web._t,
-    //     _lt = instance.web._lt;
-    // var QWeb = instance.web.qweb;
-    // instance.web.account_asset = instance.web.account_asset || {};
-
 var core = require('web.core');
 var QWeb = core.qweb;
 var session = require('web.session');
-var list_widget_registry = core.list_widget_registry;
-var Column = list_widget_registry.get('field');
 
-var WidgetOnButton = Column.extend({
-    format: function (row_data, options) {
-        this._super(row_data, options);
+var _t = core._t;
+
+var WidgetOnButton = core.list_widget_registry.get('field').extend({
+    format: function(row_data, options) {
+        this._super.apply(this, arguments);
         this.has_value = !!row_data.move_check.value;
         this.parent_state = row_data.parent_state.value;
-        this.icon = this.has_value ? 'gtk-yes' : 'gtk-no'; // or STOCK_YES and STOCK_NO
-        this.string = this.has_value ? 'Posted' : 'Unposted';
-        var template = this.icon && 'ListView.row.buttonwidget';
-        return QWeb.render(template, {
-            widget: this,
-            prefix: session.prefix,
-            disabled: this.has_value,
-            invisible : true ? this.parent_state !== 'open' : false
-        });
+
+        return $('<div/>').append((this.parent_state === 'open')? $('<button/>', {
+            type: 'button',
+            title: (this.has_value)? _t('Posted') : _t('Unposted'),
+            disabled: !!this.has_value,
+            'class': 'btn btn-sm btn-link fa fa-circle o_widgetonbutton' + ((this.has_value)? ' o_has_value' : ''),
+        }) : '').html();
     },
 });
 
-list_widget_registry.add("button.widgetonbutton", WidgetOnButton);
+core.list_widget_registry.add("button.widgetonbutton", WidgetOnButton);
 });
diff --git a/addons/account_asset/static/src/less/account_asset.less b/addons/account_asset/static/src/less/account_asset.less
new file mode 100644
index 00000000000..f1b9b3e7cd4
--- /dev/null
+++ b/addons/account_asset/static/src/less/account_asset.less
@@ -0,0 +1,6 @@
+.o_web_client .o_widgetonbutton {
+    color: @brand-danger;
+    &.o_has_value {
+        color: @brand-success;
+    }
+}
diff --git a/addons/account_asset/static/src/xml/account_asset_template.xml b/addons/account_asset/static/src/xml/account_asset_template.xml
deleted file mode 100644
index 49990fcb6ca..00000000000
--- a/addons/account_asset/static/src/xml/account_asset_template.xml
+++ /dev/null
@@ -1,8 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<templates id="template">
-    <button t-name="ListView.row.buttonwidget" t-if="!invisible" type="button"
-            t-att-title="widget.string" t-att-disabled="disabled || undefined"
-            t-att-class="disabled ? 'oe_list_button_disabled' : ''"
-            ><img t-attf-src="#{prefix}/web/static/src/img/icons/#{widget.icon}.png"
-            t-att-alt="widget.string"/></button>
-</templates>
\ No newline at end of file
diff --git a/addons/account_asset/views/account_asset.xml b/addons/account_asset/views/account_asset.xml
index 198e95172d4..53b12fe1daf 100644
--- a/addons/account_asset/views/account_asset.xml
+++ b/addons/account_asset/views/account_asset.xml
@@ -2,6 +2,7 @@
 <odoo>
     <template id="assets_backend" name="account assets" inherit_id="web.assets_backend">
         <xpath expr="." position="inside">
+            <link rel="stylesheet" href="/account_asset/static/src/less/account_asset.less"/>
             <script type="text/javascript" src="/account_asset/static/src/js/account_asset.js"></script>
         </xpath>
     </template>