Re: Unused attributes of ShoppingCart and ShoppingCartItem
Yes indeed, Deepak. Some additional infos: * With the definition of ShoppingCartItem.attributeNames you are able to get context entries auto added to the „attributes“ Map within ShoppingCartHelper.addToCart. Such attributes may than be used, as described by Deepak, to be persisted anywhere or to just influence the checkout process. Those attributes are not necessarily to be persistet, they may be temporarily attributes like tokens or else just for the current checkout logic. * The automatic adding of orderItemAttributes is controlled by the order.properties property „order.item.attr.prefix“. All context entries with this prefix (e.g. from an addToCart form) get added to the orderItemAttributes by ShoppingCartHelper.addToCart method. * It is also possible to distinguish cart items by different attribute (and orderItemAttribute) values because of their relevance within the ShoppingCartItem.equals method In my opinion the attribute maps differ fairly from the straightly domain model orderAttribute/orderItemAttribute ones and are not obsolete. Regards Martin Becker www.ecomify.de > Am 09.05.2017 um 09:02 schrieb Deepak Dixit : > > I think purpose of attributes is different, if you want to set some custom > attribute to implement custom flow so you can use attribute. > These attribute value can be store in any entity or can be used to perform > business logic implementation as well. > > Like in case of ShoppingCartItem, we set fromInventoryItemId as item > attribute and while creation order item we copy this field to > OrderItem.fromInventoryItem. > > And if you want to store attribute in order or item level then you use > orderAttributes or orderItemAttributes. > > Thanks & Regards > -- > Deepak Dixit > www.hotwaxsystems.com > > On Tue, May 9, 2017 at 10:31 AM, Suraj Khurana < > suraj.khur...@hotwaxsystems.com> wrote: > >> Hi Devs, >> >> I am curious about some protected data members of ShoppingCartItem and >> ShoppingCart class. >> ShoppingCartItem contains: >> >> >> >> *private Map orderItemAttributes = null;private Map> Object> attributes = null;* >> While ShoppingCart have: >> >> *private Map orderAttributes = FastMap.newInstance();* >> *private Map attributes = FastMap.newInstance();* >> >> I think orderAttributes and orderItemAttributes are pretty clear by the >> name itself, what is the actual use of *attributes* data member for each >> class. >> >> Can this be removed and replaced with orderAttribute or orderItemAttribute >> itself? >> >> -- >> Thanks and Regards, >> Suraj Khurana >> Sr. Enterprise Software Engineer >> HotWax Systems - The global leader in innovative enterprise commerce >> solutions powered by Apache OFBiz. >> smime.p7s Description: S/MIME cryptographic signature
Re: Unused attributes of ShoppingCart and ShoppingCartItem
I think purpose of attributes is different, if you want to set some custom attribute to implement custom flow so you can use attribute. These attribute value can be store in any entity or can be used to perform business logic implementation as well. Like in case of ShoppingCartItem, we set fromInventoryItemId as item attribute and while creation order item we copy this field to OrderItem.fromInventoryItem. And if you want to store attribute in order or item level then you use orderAttributes or orderItemAttributes. Thanks & Regards -- Deepak Dixit www.hotwaxsystems.com On Tue, May 9, 2017 at 10:31 AM, Suraj Khurana < suraj.khur...@hotwaxsystems.com> wrote: > Hi Devs, > > I am curious about some protected data members of ShoppingCartItem and > ShoppingCart class. > ShoppingCartItem contains: > > > > *private Map orderItemAttributes = null;private Map Object> attributes = null;* > While ShoppingCart have: > > *private Map orderAttributes = FastMap.newInstance();* > *private Map attributes = FastMap.newInstance();* > > I think orderAttributes and orderItemAttributes are pretty clear by the > name itself, what is the actual use of *attributes* data member for each > class. > > Can this be removed and replaced with orderAttribute or orderItemAttribute > itself? > > -- > Thanks and Regards, > Suraj Khurana > Sr. Enterprise Software Engineer > HotWax Systems - The global leader in innovative enterprise commerce > solutions powered by Apache OFBiz. >
Re: Unused attributes of ShoppingCart and ShoppingCartItem
+1 Suraj, If value of 'attributes' is not getting populated in DB then it should be replaced. -- Thanks & Regards --- Arun Patidar Manager, Enterprise Software Development HotWax Systems Pvt Ltd.www.hotwaxsystems.com On Tue, May 9, 2017 at 10:31 AM, Suraj Khurana < suraj.khur...@hotwaxsystems.com> wrote: > Hi Devs, > > I am curious about some protected data members of ShoppingCartItem and > ShoppingCart class. > ShoppingCartItem contains: > > > > *private Map orderItemAttributes = null;private Map Object> attributes = null;* > While ShoppingCart have: > > *private Map orderAttributes = FastMap.newInstance();* > *private Map attributes = FastMap.newInstance();* > > I think orderAttributes and orderItemAttributes are pretty clear by the > name itself, what is the actual use of *attributes* data member for each > class. > > Can this be removed and replaced with orderAttribute or orderItemAttribute > itself? > > -- > Thanks and Regards, > Suraj Khurana > Sr. Enterprise Software Engineer > HotWax Systems - The global leader in innovative enterprise commerce > solutions powered by Apache OFBiz. >
Unused attributes of ShoppingCart and ShoppingCartItem
Hi Devs, I am curious about some protected data members of ShoppingCartItem and ShoppingCart class. ShoppingCartItem contains: *private Map orderItemAttributes = null;private Map attributes = null;* While ShoppingCart have: *private Map orderAttributes = FastMap.newInstance();* *private Map attributes = FastMap.newInstance();* I think orderAttributes and orderItemAttributes are pretty clear by the name itself, what is the actual use of *attributes* data member for each class. Can this be removed and replaced with orderAttribute or orderItemAttribute itself? -- Thanks and Regards, Suraj Khurana Sr. Enterprise Software Engineer HotWax Systems - The global leader in innovative enterprise commerce solutions powered by Apache OFBiz.