From 890a5a8cb33213eb010021db0bdb72c822271f1a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Sat, 1 Sep 2018 01:02:03 +0200
Subject: [PATCH] Improve ModelStorage.copy

* Allow callable in copy default
* Support dotted notation in copy default
* Use setdefault in copy
* Copy default when needed
* Do not set default state in copy
* Use callable and dotted notation in copy default

issue5576
issue7648
review72291002
---
 asset.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/asset.py b/asset.py
index 7f82bb1dc9..7677548052 100644
--- a/asset.py
+++ b/asset.py
@@ -619,9 +619,9 @@ class Asset(Workflow, ModelSQL, ModelView):
     def copy(cls, assets, default=None):
         if default is None:
             default = {}
-        default = default.copy()
+        else:
+            default = default.copy()
         default.setdefault('lines', [])
-        default.setdefault('state', 'draft')
         default.setdefault('number', None)
         default.setdefault('supplier_invoice_line', None)
         return super(Asset, cls).copy(assets, default=default)