← Back to team overview

openerp-dev-web team mailing list archive

[Merge] lp:~openerp-dev/openobject-client-web/6.0-opw-5138-sma into lp:openobject-client-web/6.0

 

Sananaz (Open ERP) has proposed merging lp:~openerp-dev/openobject-client-web/6.0-opw-5138-sma into lp:openobject-client-web/6.0.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-5138-sma/+merge/57477

Hello,

Check with the following steps:
1> Go to Accounting > Journal Entries > Journal Entries (Create New)
2> Add Journal Items lines fill all fields.
3> When save Journal Items line it create new record (Its blank record)

Here problem occur::
1>  When create new record it contains all defaults of first one records.
   See when you directly click on `new` button after `save` one record it behave fine(you can see record with default values) but when `save` record it create new with blank raw. then problem is faced.

2> For editable listview float values are not contain default `0.0`

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-5138-sma/+merge/57477
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client-web/6.0-opw-5138-sma.
=== modified file 'addons/openerp/static/javascript/listgrid.js'
--- addons/openerp/static/javascript/listgrid.js	2011-03-29 15:51:28 +0000
+++ addons/openerp/static/javascript/listgrid.js	2011-04-13 12:18:30 +0000
@@ -613,11 +613,20 @@
             } else {
                 openobject.dom.get(prefix + '_terp_id').value = obj.id;
                 openobject.dom.get(prefix + '_terp_ids').value = obj.ids;
+                if (jQuery(idSelector(prefix + '_terp_default_get_ctx'))) {
+                	this.default_get_ctx = jQuery(idSelector(prefix + '_terp_default_get_ctx')).value;
+                }
 
                 if(prev_id != undefined) {
                     self.reload(prev_id , prefix ? 1 : 0);
                 } else {
-                    self.reload(id > 0 ? null : -1, prefix ? 1 : 0);
+                    // Need to remove existing editable row otherwise it create problem when eval context
+                    // Problem due to getFormData it takes all editable fields values
+                    jQuery('tr.editors').remove()
+                    var req = eval_domain_context_request({source: self.name, context: this.default_get_ctx});
+                    req.addCallback(function(res) {
+                    	self.reload(id > 0 ? null : -1, prefix ? 1 : 0, res.context);
+                    });
                 }
             }
         });

=== modified file 'addons/openerp/widgets/listgrid.py'
--- addons/openerp/widgets/listgrid.py	2011-03-29 15:51:28 +0000
+++ addons/openerp/widgets/listgrid.py	2011-04-13 12:18:30 +0000
@@ -230,6 +230,10 @@
                         self.edit_inline = False
                         self.editors = {}
                         break
+
+                    if fa.get('type') == 'float':
+                        fa['default'] = 0.0
+
                     Widget = get_widget(fa.get('type', 'char')) or get_widget('char')
                     self.editors[f] = Widget(**fa)
 


Follow ups