A (slightly modified) patch from  Ritu Raj Lakhera for "Order return is not refunding the money" https://issues.apache.org/jira/browse/OFBIZ-5528

When we placed an order with two items and generate a return towards one item then code is refunding the money correctly.
But if we again do return for second item then code is not refunding the money because first return-refund process has changed the status of OrderPaymentPreference from 'PAYMENT_SETTLED' to ‘PAYMENT_REFUNDED’.
I think We should create the new OrderPaymentPreference with PAYMENT_REFUNDED status against old OrderPaymentPreference of 'PAYMENT_SETTLED' status at the time of return-refund process. So second return can also refund the money.

In other words we should create the new OrderPaymentPreference for each refund amount as we are doing with Payment entity(processRefundResult PaymentGatewayServices.java).

jleroux: I 
* replaced the deprecated findByPrimaryKey by a finByOne
* replaced Exception GenericServiceException in catch (as it was before)
* reformatted some parts 

git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1564111 13f79535-47bb-0310-9956-ffa450edef68
When I replaced the deprecated findByPrimaryKey by a finByOne I forgot to put a catch around, this fixes it.
Also removed an useless logError, ServiceUtil.returnError will take care of it

git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1564113 13f79535-47bb-0310-9956-ffa450edef68
Update commons-fileupload to 1.3

git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1564463 13f79535-47bb-0310-9956-ffa450edef68
Updates geronimo-connector & geronimo-transaction to 3.1.1

git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1564473 13f79535-47bb-0310-9956-ffa450edef68
Updates geronimo-saaj to _1.3_spec-1.1

git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1564493 13f79535-47bb-0310-9956-ffa450edef68
Updates axis2-kernel, axis2-transport-http & axis2-transport-local to 1.6.2


git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1564543 13f79535-47bb-0310-9956-ffa450edef68
Reverts 1564543, SOAP tests fail with axis2 1.6.2, keep 1.5.2

git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1564661 13f79535-47bb-0310-9956-ffa450edef68
Updates checkout-sdk to 2.5.1 


git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1564674 13f79535-47bb-0310-9956-ffa450edef68
Updates geronimo-activation to 1.1_spec-1.1

git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1564680 13f79535-47bb-0310-9956-ffa450edef68
Reverts r1564680, keeps geronimo-activation_1.0.2_spec-1.0.jar

git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1564686 13f79535-47bb-0310-9956-ffa450edef68
Reverts r1564674, last checkout-sdk fails to compile keeps 0.8.8

git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1564691 13f79535-47bb-0310-9956-ffa450edef68
Adds a link to the documentation in wiki

git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1564775 13f79535-47bb-0310-9956-ffa450edef68
Closes "Upgrade BIRT Runtime to 4.3.1" https://issues.apache.org/jira/browse/OFBIZ-5530

jleroux: what I did:
Replaced all (20) old birt libs by new one provided in the zip Chatree provided
I did not touch any other libs, I checked they were all the same, but the names of Axis 1 libs which were not versionned in the zip. So I removed the related Axis 1 changes in LICENSE and .classpath files


git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1565375 13f79535-47bb-0310-9956-ffa450edef68
Update Commons File Upload to 1.3.1.


git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1565760 13f79535-47bb-0310-9956-ffa450edef68
Some bug fixes in Mini-language:

1. Updated schema to indicate required attributes.
2. Fixed inconsistent implementation of the alternate delegator feature: https://issues.apache.org/jira/browse/OFBIZ-5531


git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1566096 13f79535-47bb-0310-9956-ffa450edef68
A patch from  Chris Lombardi  for "Void item, quantity update, and discounts don't work for configurable items" https://issues.apache.org/jira/browse/OFBIZ-5535

Void item, quantity update and discounts work off of the sku in the pos.  I fixed it to use the cart index so these operations will work when there are multiple configurations per sku in the cart.



git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1566273 13f79535-47bb-0310-9956-ffa450edef68
A series of cleanups, simplifications and improvements (to leverage some of the features of the recent Tomcat API) to the way that the CatalinaContainer sets up the embedded Tomcat instance. 


git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1568269 13f79535-47bb-0310-9956-ffa450edef68
Prevents a possible misconfiguration spotted on trunk demo:
Expression blog.contentName is undefined on line 31, column 116 in component://ecommerce/webapp/ecommerce/blog/blogs.ftl. The problematic instruction: ---------- ==> ${blog.contentName} [on line 31, column 114 in 

git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1569078 13f79535-47bb-0310-9956-ffa450edef68
Fixed MissingPropertyException happening when success method was called in the context of Groovy services: thanks to Paolo Piersant for the bug report and fix.



git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1569743 13f79535-47bb-0310-9956-ffa450edef68
Fixed path: the appserver component has been moved from the framework to the specialpurpose folder.


git-svn-id: https://svn.apache.org/repos/asf/ofbiz/trunk@1570595 13f79535-47bb-0310-9956-ffa450edef68