From f24199fcc3f88c91a3af3827ec01118778d26960 Mon Sep 17 00:00:00 2001
From: Yoshinori Okuji <yo@nexedi.com>
Date: Mon, 3 Apr 2006 15:54:31 +0000
Subject: [PATCH] Disable generateNewId and explain why.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@6436 20353a03-c40f-0410-a6d1-a30d3c3de9de
---
 product/ERP5/Document/Item.py | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/product/ERP5/Document/Item.py b/product/ERP5/Document/Item.py
index eb78552dd0..10381279e5 100644
--- a/product/ERP5/Document/Item.py
+++ b/product/ERP5/Document/Item.py
@@ -77,10 +77,15 @@ class Item(XMLObject, Amount, ImmobilisableItem):
                       , PropertySheet.Reference
                       )
 
-    security.declareProtected(Permissions.ModifyPortalContent,'generateNewId')
-    def generateNewId(self, id_group='item_id_group', default=None, method=None):
-      """
-      We want a different id for all Item
-      """
-      return XMLObject.generateNewId(self, id_group=id_group, default=default, method=method)
+    if 0:
+      # The following code is disabled. The original intention was to generate
+      # an unique reference for each item. We now use reference instead of id,
+      # so this is not applicable any longer. We need something different for
+      # reference.
+      security.declareProtected(Permissions.ModifyPortalContent,'generateNewId')
+      def generateNewId(self, id_group='item_id_group', default=None, method=None):
+        """
+        We want a different id for all Item
+        """
+        return XMLObject.generateNewId(self, id_group=id_group, default=default, method=method)
 
-- 
2.30.9