You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2019/04/09 12:00:02 UTC
svn commit: r1857182 - in /ofbiz/ofbiz-framework/branches/release17.12: ./
applications/order/minilang/shoppinglist/ applications/order/minilang/test/
applications/order/src/main/java/org/apache/ofbiz/order/shoppinglist/
Author: jleroux
Date: Tue Apr 9 12:00:01 2019
New Revision: 1857182
URL: http://svn.apache.org/viewvc?rev=1857182&view=rev
Log:
"Applied fix from trunk for revision: 1857180"
------------------------------------------------------------------------
r1857180 | jleroux | 2019-04-09 13:58:22 +0200 (mar. 09 avr. 2019) | 19 lignes
Fixed: Quick Add button for shopping list on Orderentry screen is not working
(OFBIZ-9908)
Steps to generate
1. Navigate to https://demo-trunk.ofbiz.apache.org/ordermgr/control/orderentry
2. Press continue on sales order screen.
3. Add any finished good product to the sales order
4. In 'Add Order Items to Shopping List' section, select 'new shopping list'
from the drop down and click on 'Add to shopping list' button
5. you navigate to 'addBulkToShoppingList' screen, click on 'Quick Add' button.
Issue: The quick add button is not working.
jleroux: as suggested by Ankush: "One of possible workaround is calling
createShoppingList with new transaction to resolve this deadlock issue."
This just does that for every calls.
Thanks: Garima jain for report, Rohit Hukkeri for discussion and Ankush Upadhyay
for fix suggestion
------------------------------------------------------------------------
Modified:
ofbiz/ofbiz-framework/branches/release17.12/ (props changed)
ofbiz/ofbiz-framework/branches/release17.12/applications/order/minilang/shoppinglist/ShoppingListServices.xml
ofbiz/ofbiz-framework/branches/release17.12/applications/order/minilang/test/ShoppingListTests.xml
ofbiz/ofbiz-framework/branches/release17.12/applications/order/src/main/java/org/apache/ofbiz/order/shoppinglist/ShoppingListEvents.java
ofbiz/ofbiz-framework/branches/release17.12/applications/order/src/main/java/org/apache/ofbiz/order/shoppinglist/ShoppingListServices.java
Propchange: ofbiz/ofbiz-framework/branches/release17.12/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Tue Apr 9 12:00:01 2019
@@ -10,4 +10,4 @@
/ofbiz/branches/json-integration-refactoring:1634077-1635900
/ofbiz/branches/multitenant20100310:921280-927264
/ofbiz/branches/release13.07:1547657
-/ofbiz/ofbiz-framework/trunk:1819499,1819598,1819800,1819805,1819811,1819947,1820038,1820262,1820374-1820375,1820441,1820457,1820644,1820658,1820790,1820823,1820949,1820966,1821012,1821036,1821112,1821115,1821144,1821186,1821219,1821226,1821230,1821386,1821613,1821628,1821965,1822125,1822310,1822377,1822383,1822393,1823467,1823562,1823876,1824314,1824316,1824732,1824803,1824847,1824855,1825192,1825211,1825216,1825233,1825450,1826374,1826502,1826592,1826671,1826674,1826780,1826805,1826938,1826997,1827439,1828255,1828316,1828346,1828424,1828512,1828514,1829690,1830936,1831074,1831078,1831234,1831608,1831831,1832577,1832662,1832756,1832800,1832944,1833173,1833211,1834181,1834191,1834736,1835235,1835887,1835891,1835953,1835964,1836144,1836871,1837857,1838032,1838256,1838381,1840189,1840199,1840828,1841657,1841662,1842372,1842921,1843225,1843893,1844943,1845418,1845420,1845466,1845544,1845552,1845558,1845933,1845995,1846097,1846107,1846214,1846594,1846632,1847398,1847478,1847670,1847715,
1847890,1848263,1848336,1848386,1848398,1848441,1848444,1848447,1848449,1848467,1848469,1848745,1848849-1848850,1849021,1849191,1849193,1849275,1849467,1849528,1849540,1849567,1849693,1850015,1850023,1850530,1850647,1850685,1850694,1850914,1850918,1850948,1850953,1851006,1851068,1851074,1851130,1851158,1851163,1851200,1851247,1851319,1851350,1851805,1851998,1852587,1852818,1853070,1853109,1853691,1853745,1853750,1854306,1854457,1855078,1855287,1855371,1855403,1855488,1855492,1855497,1855501,1855898,1856405,1856455,1856459-1856460,1856484,1856598,1856617,1856667,1857173
+/ofbiz/ofbiz-framework/trunk:1819499,1819598,1819800,1819805,1819811,1819947,1820038,1820262,1820374-1820375,1820441,1820457,1820644,1820658,1820790,1820823,1820949,1820966,1821012,1821036,1821112,1821115,1821144,1821186,1821219,1821226,1821230,1821386,1821613,1821628,1821965,1822125,1822310,1822377,1822383,1822393,1823467,1823562,1823876,1824314,1824316,1824732,1824803,1824847,1824855,1825192,1825211,1825216,1825233,1825450,1826374,1826502,1826592,1826671,1826674,1826780,1826805,1826938,1826997,1827439,1828255,1828316,1828346,1828424,1828512,1828514,1829690,1830936,1831074,1831078,1831234,1831608,1831831,1832577,1832662,1832756,1832800,1832944,1833173,1833211,1834181,1834191,1834736,1835235,1835887,1835891,1835953,1835964,1836144,1836871,1837857,1838032,1838256,1838381,1840189,1840199,1840828,1841657,1841662,1842372,1842921,1843225,1843893,1844943,1845418,1845420,1845466,1845544,1845552,1845558,1845933,1845995,1846097,1846107,1846214,1846594,1846632,1847398,1847478,1847670,1847715,
1847890,1848263,1848336,1848386,1848398,1848441,1848444,1848447,1848449,1848467,1848469,1848745,1848849-1848850,1849021,1849191,1849193,1849275,1849467,1849528,1849540,1849567,1849693,1850015,1850023,1850530,1850647,1850685,1850694,1850914,1850918,1850948,1850953,1851006,1851068,1851074,1851130,1851158,1851163,1851200,1851247,1851319,1851350,1851805,1851998,1852587,1852818,1853070,1853109,1853691,1853745,1853750,1854306,1854457,1855078,1855287,1855371,1855403,1855488,1855492,1855497,1855501,1855898,1856405,1856455,1856459-1856460,1856484,1856598,1856617,1856667,1857173,1857180
Modified: ofbiz/ofbiz-framework/branches/release17.12/applications/order/minilang/shoppinglist/ShoppingListServices.xml
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/branches/release17.12/applications/order/minilang/shoppinglist/ShoppingListServices.xml?rev=1857182&r1=1857181&r2=1857182&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/branches/release17.12/applications/order/minilang/shoppinglist/ShoppingListServices.xml (original)
+++ ofbiz/ofbiz-framework/branches/release17.12/applications/order/minilang/shoppinglist/ShoppingListServices.xml Tue Apr 9 12:00:01 2019
@@ -307,7 +307,7 @@ under the License.
<set field="createShoppingListInMap.listName" value="Auto Suggestions"/>
<set field="createShoppingListInMap.shoppingListTypeId" value="SLT_WISH_LIST"/>
<set field="createShoppingListInMap.productStoreId" from-field="parameters.productStoreId"/>
- <call-service service-name="createShoppingList" in-map-name="createShoppingListInMap">
+ <call-service service-name="createShoppingList" in-map-name="createShoppingListInMap" require-new-transaction="true">
<result-to-field result-name="shoppingListId"/>
</call-service>
<else>
Modified: ofbiz/ofbiz-framework/branches/release17.12/applications/order/minilang/test/ShoppingListTests.xml
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/branches/release17.12/applications/order/minilang/test/ShoppingListTests.xml?rev=1857182&r1=1857181&r2=1857182&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/branches/release17.12/applications/order/minilang/test/ShoppingListTests.xml (original)
+++ ofbiz/ofbiz-framework/branches/release17.12/applications/order/minilang/test/ShoppingListTests.xml Tue Apr 9 12:00:01 2019
@@ -33,7 +33,7 @@ under the License.
<field-map field-name="userLoginId" value="DemoCustomer"/>
</entity-one>
<set field="serviceCtx.userLogin" from-field="userLogin"/>
- <call-service service-name="createShoppingList" in-map-name="serviceCtx">
+ <call-service service-name="createShoppingList" in-map-name="serviceCtx" require-new-transaction="true">
<result-to-field result-name="shoppingListId"/>
</call-service>
<entity-one entity-name="ShoppingList" value-field="shoppingList"/>
Modified: ofbiz/ofbiz-framework/branches/release17.12/applications/order/src/main/java/org/apache/ofbiz/order/shoppinglist/ShoppingListEvents.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/branches/release17.12/applications/order/src/main/java/org/apache/ofbiz/order/shoppinglist/ShoppingListEvents.java?rev=1857182&r1=1857181&r2=1857182&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/branches/release17.12/applications/order/src/main/java/org/apache/ofbiz/order/shoppinglist/ShoppingListEvents.java (original)
+++ ofbiz/ofbiz-framework/branches/release17.12/applications/order/src/main/java/org/apache/ofbiz/order/shoppinglist/ShoppingListEvents.java Tue Apr 9 12:00:01 2019
@@ -104,7 +104,10 @@ public class ShoppingListEvents {
// create a new shopping list
Map<String, Object> newListResult = null;
try {
- newListResult = dispatcher.runSync("createShoppingList", UtilMisc.<String, Object>toMap("userLogin", userLogin, "productStoreId", cart.getProductStoreId(), "partyId", cart.getOrderPartyId(), "shoppingListTypeId", shoppingListTypeId, "currencyUom", cart.getCurrency()));
+ newListResult = dispatcher.runSync("createShoppingList", UtilMisc.<String, Object>toMap("userLogin", userLogin,
+ "productStoreId", cart.getProductStoreId(), "partyId", cart.getOrderPartyId(),
+ "shoppingListTypeId", shoppingListTypeId, "currencyUom", cart.getCurrency()),
+ 90, true);
} catch (GenericServiceException e) {
Debug.logError(e, "Problems creating new ShoppingList", module);
errMsg = UtilProperties.getMessage(resource_error,"shoppinglistevents.cannot_create_new_shopping_list", cart.getLocale());
@@ -373,7 +376,7 @@ public class ShoppingListEvents {
}
if (list == null && dispatcher != null) {
Map<String, Object> listFields = UtilMisc.<String, Object>toMap("userLogin", userLogin, "productStoreId", productStoreId, "shoppingListTypeId", "SLT_SPEC_PURP", "listName", PERSISTANT_LIST_NAME);
- Map<String, Object> newListResult = dispatcher.runSync("createShoppingList", listFields);
+ Map<String, Object> newListResult = dispatcher.runSync("createShoppingList", listFields, 90, true););
if (newListResult != null) {
autoSaveListId = (String) newListResult.get("shoppingListId");
@@ -649,7 +652,7 @@ public class ShoppingListEvents {
if (UtilValidate.isEmpty(autoSaveListId)) {
try {
Map<String, Object> listFields = UtilMisc.<String, Object>toMap("userLogin", userLogin, "productStoreId", productStoreId, "shoppingListTypeId", "SLT_SPEC_PURP", "listName", PERSISTANT_LIST_NAME);
- Map<String, Object> newListResult = dispatcher.runSync("createShoppingList", listFields);
+ Map<String, Object> newListResult = dispatcher.runSync("createShoppingList", listFields, 90, true););
if (newListResult != null) {
autoSaveListId = (String) newListResult.get("shoppingListId");
}
Modified: ofbiz/ofbiz-framework/branches/release17.12/applications/order/src/main/java/org/apache/ofbiz/order/shoppinglist/ShoppingListServices.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/branches/release17.12/applications/order/src/main/java/org/apache/ofbiz/order/shoppinglist/ShoppingListServices.java?rev=1857182&r1=1857181&r2=1857182&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/branches/release17.12/applications/order/src/main/java/org/apache/ofbiz/order/shoppinglist/ShoppingListServices.java (original)
+++ ofbiz/ofbiz-framework/branches/release17.12/applications/order/src/main/java/org/apache/ofbiz/order/shoppinglist/ShoppingListServices.java Tue Apr 9 12:00:01 2019
@@ -291,7 +291,7 @@ public class ShoppingListServices {
Map<String, Object> newListResult = null;
try {
- newListResult = dispatcher.runSync("createShoppingList", serviceCtx);
+ newListResult = dispatcher.runSync("createShoppingList", serviceCtx, 90, true););
} catch (GenericServiceException e) {
Debug.logError(e, "Problems creating new ShoppingList", module);
return ServiceUtil.returnError(UtilProperties.getMessage(resource_error,"OrderUnableToCreateNewShoppingList",locale));