From 049964817d771dbf1ece031ae3ac810eba5c545d Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Fri, 25 Mar 2011 11:34:25 +0100
Subject: [PATCH] [FIX] handle default values setting on search view

bzr revid: xmo@openerp.com-20110325103425-ag08qufuavok6kz0
---
 addons/base/controllers/main.py             |  1 -
 addons/base/static/openerp/base.xml         | 13 +++--
 addons/base/static/openerp/js/base_views.js | 60 ++++++++++++++++++---
 3 files changed, 64 insertions(+), 10 deletions(-)

diff --git a/addons/base/controllers/main.py b/addons/base/controllers/main.py
index 60076d930e9..79ff0cce632 100644
--- a/addons/base/controllers/main.py
+++ b/addons/base/controllers/main.py
@@ -161,7 +161,6 @@ class Menu(openerpweb.Controller):
         actions = Values.get('action', 'tree_but_open', [('ir.ui.menu', menu_id)], False, {})
 
         for _, _, action in actions:
-            print action
             action['context'] = req.session.eval_context(
                 action['context']) or {}
             action['domain'] = req.session.eval_domain(
diff --git a/addons/base/static/openerp/base.xml b/addons/base/static/openerp/base.xml
index a42a8a7287f..9ffe67b45da 100644
--- a/addons/base/static/openerp/base.xml
+++ b/addons/base/static/openerp/base.xml
@@ -300,7 +300,7 @@
 <button t-name="SearchView.filter" type="button"
         t-att-id="element_id"
         t-att-title="attrs.help"
-        t-att-class="attrs.string ? 'filter_label' : 'filter_icon'">
+        t-att-class="classes.join(' ')">
     <img t-if="attrs.icon" t-att-src="'/base/static/openerp/img/icons/' + attrs.icon + '.png'" width="16" height="16"/>
     <br t-if="attrs.icon and attrs.string"/>
     <t t-esc="attrs.string"/>
@@ -316,7 +316,9 @@
         <span t-if="attrs.help">(?)</span>
     </label>
     <div style="white-space: nowrap;">
-        <input type="text" t-att-name="attrs.name" t-att-id="element_id"/>
+        <input type="text" t-att-name="attrs.name"
+               t-att-id="element_id"
+               t-att-value="defaults[attrs.name] || ''"/>
         <t t-if="filters.length" t-raw="filters.render(defaults)"/>
     </div>
 </t>
@@ -330,7 +332,12 @@
         <select t-att-name="attrs.name" t-att-id="element_id">
             <option/>
             <t t-foreach="attrs.selection" t-as="option">
-                <option t-att-value="option[0]">
+                <t t-set="selected" t-value="defaults[attrs.name] === option[0]"/>
+                <option t-if="selected"
+                        t-att-value="option[0]" selected="selected">
+                    <t t-esc="option[1]"/>
+                </option>
+                <option t-if="not selected" t-att-value="option[0]">
                     <t t-esc="option[1]"/>
                 </option>
             </t>
diff --git a/addons/base/static/openerp/js/base_views.js b/addons/base/static/openerp/js/base_views.js
index f5dee2ab3a2..80d325322ae 100644
--- a/addons/base/static/openerp/js/base_views.js
+++ b/addons/base/static/openerp/js/base_views.js
@@ -45,6 +45,23 @@ openerp.base.ViewManager =  openerp.base.Controller.extend({
            this.views[i].controller.$element.toggle(i === view_type);
         }
     },
+    /**
+     * Extract search view defaults from the current action's context.
+     *
+     * These defaults are of the form {search_default_*: value}
+     *
+     * @returns {Object} a clean defaults mapping of {field_name: value}
+     */
+    search_defaults: function () {
+        var defaults = {};
+        _.each(this.action.context, function (value, key) {
+            var match = /^search_default_(.*)$/.exec(key);
+            if (match) {
+                defaults[match[1]] = value;
+            }
+        });
+        return defaults;
+    },
     do_action_window: function(action) {
         var self = this;
         var prefix_id = "#" + this.element_id;
@@ -57,7 +74,10 @@ openerp.base.ViewManager =  openerp.base.Controller.extend({
         this.searchview_id = false;
         if(this.search_visible && action.search_view_id) {
             this.searchview_id = action.search_view_id[0];
-            this.searchview = new openerp.base.SearchView(this.session, this.element_id + "_search", this.dataset, this.searchview_id);
+            this.searchview = new openerp.base.SearchView(
+                    this.session, this.element_id + "_search",
+                    this.dataset, this.searchview_id,
+                    this.search_defaults());
             this.searchview.on_search.add(this.do_search);
             this.searchview.start();
 
@@ -363,12 +383,14 @@ openerp.base.DataRecord =  openerp.base.Controller.extend({
 });
 
 openerp.base.SearchView = openerp.base.Controller.extend({
-    init: function(session, element_id, dataset, view_id) {
+    init: function(session, element_id, dataset, view_id, defaults) {
         this._super(session, element_id);
         this.dataset = dataset;
         this.model = dataset.model;
         this.view_id = view_id;
 
+        this.defaults = defaults || {};
+
         this.inputs = [];
     },
     start: function() {
@@ -480,13 +502,10 @@ openerp.base.SearchView = openerp.base.Controller.extend({
             data.fields_view['arch'].children,
             data.fields_view.fields);
 
-        // TODO: get default values
-        var default_values = {};
-
         var render = QWeb.render("SearchView", {
             'view': data.fields_view['arch'],
             'lines': lines,
-            'defaults': default_values
+            'defaults': this.defaults
         });
         this.$element.html(render);
 
@@ -655,6 +674,7 @@ openerp.base.search.Filter = openerp.base.search.Input.extend({
     init: function (node, view) {
         this._super(view);
         this.attrs = node.attrs;
+        this.classes = [this.attrs.string ? 'filter_label' : 'filter_icon'];
         this.make_id('filter', this.attrs.name);
     },
     start: function () {
@@ -665,6 +685,18 @@ openerp.base.search.Filter = openerp.base.search.Input.extend({
             $view_form.submit();
         });
     },
+    /**
+     * If the filter is present in the defaults (and has a truthy value),
+     * enable the filter.
+     *
+     * @param {Object} defaults the search view's default values
+     */
+    render: function (defaults) {
+        if (this.attrs.name && defaults[this.attrs.name]) {
+            this.classes.push('enabled');
+        }
+        return this._super(defaults);
+    },
     get_context: function () {
         if (!this.$element.hasClass('enabled')) {
             return;
@@ -741,6 +773,22 @@ openerp.base.search.BooleanField = openerp.base.search.Field.extend({
             ['false', 'No']
         ];
     },
+    /**
+     * Search defaults likely to be boolean values (for a boolean field).
+     *
+     * In the HTML, we only get strings, and our strings here are
+     * <code>'true'</code> and <code>'false'</code>, so ensure we get only
+     * those by truth-testing the default value.
+     *
+     * @param {Object} defaults default values for this search view
+     */
+    render: function (defaults) {
+        var name = this.attrs.name;
+        if (name in defaults) {
+            defaults[name] = defaults[name] ? "true" : "false";
+        }
+        return this._super(defaults);
+    },
     get_value: function () {
         switch (this.$element.val()) {
             case 'false': return false;