+1 from me on the revert as well (making it clear as it is only
implied in my initial reply).
-David
On Apr 12, 2008, at 11:04 PM, Jacopo Cappellato wrote:
No feedback on this?
However, a committer should always be around to explain and provide
further details if there are concerns or questions on his commits.
Should we revert this commit?
Jacopo
On Apr 9, 2008, at 4:29 AM, David E Jones wrote:
Actually, transactions ARE important for reading as well as
writing, and should pretty much always be used.
What was the problem you had that this change is supposed to solve?
-David
On Apr 8, 2008, at 5:21 PM, [EMAIL PROTECTED] wrote:
Author: sichen
Date: Tue Apr 8 16:21:14 2008
New Revision: 646132
URL: http://svn.apache.org/viewvc?rev=646132&view=rev
Log:
turning off transactions for loading of cart from quote, since
there does not seem to be any data being stored, and having a
transaction consumes resources unnecessarily and could cause it to
fail if there are many items on a quote
Modified:
ofbiz/trunk/applications/order/servicedef/services_cart.xml
Modified: ofbiz/trunk/applications/order/servicedef/
services_cart.xml
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_cart.xml?rev=646132&r1=646131&r2=646132&view=diff
=
=
=
=
=
=
=
=
=
=
====================================================================
--- ofbiz/trunk/applications/order/servicedef/services_cart.xml
(original)
+++ ofbiz/trunk/applications/order/servicedef/services_cart.xml
Tue Apr 8 16:21:14 2008
@@ -103,7 +103,7 @@
</service>
<service name="loadCartFromQuote" engine="java" auth="true"
-
location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
invoke="loadCartFromQuote">
+
location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
invoke="loadCartFromQuote" use-transaction="false">
<description>Create a ShoppingCart Object based on an
existing quote. If applyQuoteAdjustments is set to false then
standard cart adjustments are generated.</description>
<attribute name="quoteId" type="String" mode="IN"
optional="false"/>
<attribute name="applyQuoteAdjustments" type="String"
mode="IN" optional="true"/>