Merge lp:~camptocamp/ocb-addons/ocb-7.0-fix-1231856-afe into lp:ocb-addons

[Alexandre Fayolle - camptocamp]

port of https://code.launchpad.net/~camptocamp/openobject-addons/7.0-fix-1231856-afe/+merge/188012 to OCB

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

this is used in production

[Pedro Manuel Baeza (pedro.baeza)]

LGTM

review: Approve (code review, no test)

[Stefan Rijnhart (Opener) (stefan-opener)]

I must admit that I am forever confused about whether to use the context date or the utc date, but I have the feeling that as this is a wizard run from the interface, you should use context_today. What do you think?

review: Needs Information

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

I'm rejecting this MP, because I think Amit is right and it is the other wizard which needs fixing.

As for which date is to be used I have absolutely no effing idea and I'm totally confused on that topic :-(

review: Disapprove

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

review: Needs Fixing

Unmerged revisions

9540. By Alexandre Fayolle - camptocamp on 2013-09-27

[FIX] add a default value for invoice_date in stock.invoice.onshipping (lp:1231856)

=== modified file 'stock/wizard/stock_invoice_onshipping.py'
--- stock/wizard/stock_invoice_onshipping.py	2012-10-23 16:05:04 +0000
+++ stock/wizard/stock_invoice_onshipping.py	2013-09-27 10:47:04 +0000
@@ -80,6 +80,7 @@
 
     _defaults = {
         'journal_id' : _get_journal,
+        'invoice_date': fields.date.today,
     }
 
     def view_init(self, cr, uid, fields_list, context=None):