Hi Nicolas,

When possible please put a least the number of the Jira issue in commit, it's 
easier to follow things (links are even easier ;)

Thanks

Jacques

Le 30/09/2014 13:59, nma...@apache.org a écrit :
Author: nmalin
Date: Tue Sep 30 11:59:29 2014
New Revision: 1628405

URL: http://svn.apache.org/r1628405
Log:
Convert the ProdCatalog CRUD simple service to entity-auto

Modified:
     
ofbiz/trunk/applications/product/script/org/ofbiz/product/catalog/CatalogServices.xml
     ofbiz/trunk/applications/product/servicedef/services.xml

Modified: 
ofbiz/trunk/applications/product/script/org/ofbiz/product/catalog/CatalogServices.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/catalog/CatalogServices.xml?rev=1628405&r1=1628404&r2=1628405&view=diff
==============================================================================
--- 
ofbiz/trunk/applications/product/script/org/ofbiz/product/catalog/CatalogServices.xml
 (original)
+++ 
ofbiz/trunk/applications/product/script/org/ofbiz/product/catalog/CatalogServices.xml
 Tue Sep 30 11:59:29 2014
@@ -20,92 +20,6 @@ under the License.
<simple-methods xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
          
xsi:noNamespaceSchemaLocation="http://ofbiz.apache.org/dtds/simple-methods-v2.xsd";>
-    <simple-method method-name="createProdCatalog" short-description="Create an 
ProdCatalog">
-        <make-value value-field="newEntity" entity-name="ProdCatalog"/>
-        <set-nonpk-fields map="parameters" value-field="newEntity"/>
-
-        <if-empty field="parameters.prodCatalogId">
-            <sequenced-id sequence-name="ProdCatalog" 
field="newEntity.prodCatalogId"/>
-        <else>
-            <set field="newEntity.prodCatalogId" 
from-field="parameters.prodCatalogId"/>
-            <check-id field="newEntity.prodCatalogId"/>
-            <check-errors/>
-        </else>
-        </if-empty>
-        <field-to-result field="newEntity.prodCatalogId" 
result-name="prodCatalogId"/>
-
-        <create-value value-field="newEntity"/>
-    </simple-method>
-    <simple-method method-name="updateProdCatalog" short-description="Update an 
ProdCatalog">
-        <entity-one entity-name="ProdCatalog"  value-field="prodCatalog"/>
-        <set-nonpk-fields map="parameters" value-field="prodCatalog"/>
-        <store-value value-field="prodCatalog"/>
-    </simple-method>
-    <simple-method method-name="deleteProdCatalog" short-description="Delete an 
ProdCatalog">
-        <entity-one entity-name="ProdCatalog"  value-field="prodCatalog"/>
-        <remove-value value-field="prodCatalog"/>
-    </simple-method>
-
-    <simple-method method-name="addProductCategoryToProdCatalog" 
short-description="Add Category To ProdCatalog">
-        <!-- Check that the ProductCategory exists -->
-        <entity-one entity-name="ProductCategory" 
value-field="productCategory"/>
-        <if-empty field="productCategory">
-            <add-error><fail-property resource="ProductUiLabels" 
property="ProductCategoryNotFoundForCategoryID"/></add-error>
-        </if-empty>
-        <check-errors/>
-
-        <make-value value-field="newEntity" entity-name="ProdCatalogCategory"/>
-        <set-pk-fields map="parameters" value-field="newEntity"/>
-        <set-nonpk-fields map="parameters" value-field="newEntity"/>
-
-        <if-empty field="newEntity.fromDate">
-            <now-timestamp field="nowTimestamp"/>
-            <set from-field="nowTimestamp" field="newEntity.fromDate"/>
-        </if-empty>
-
-        <create-value value-field="newEntity"/>
-    </simple-method>
-    <simple-method method-name="updateProductCategoryToProdCatalog" 
short-description="Remove ContactMech From ProdCatalog">
-
-        <make-value value-field="lookupPKMap" 
entity-name="ProdCatalogCategory"/>
-        <set-pk-fields map="parameters" value-field="lookupPKMap"/>
-        <find-by-primary-key entity-name="ProdCatalogCategory" map="lookupPKMap" 
value-field="prodCatalogCategory"/>
-        <set-nonpk-fields map="parameters" value-field="prodCatalogCategory"/>
-        <store-value value-field="prodCatalogCategory"/>
-    </simple-method>
-    <simple-method method-name="removeProductCategoryFromProdCatalog" 
short-description="Remove ContactMech From ProdCatalog">
-
-        <make-value value-field="lookupPKMap" 
entity-name="ProdCatalogCategory"/>
-        <set-pk-fields map="parameters" value-field="lookupPKMap"/>
-        <find-by-primary-key entity-name="ProdCatalogCategory" map="lookupPKMap" 
value-field="prodCatalogCategory"/>
-        <remove-value value-field="prodCatalogCategory"/>
-    </simple-method>
-
-    <!-- Party and Catalog services -->
-    <simple-method method-name="addProdCatalogToParty" short-description="Add 
ProdCatalog To Party">
-        <make-value value-field="newEntity" entity-name="ProdCatalogRole"/>
-        <set-pk-fields map="parameters" value-field="newEntity"/>
-        <set-nonpk-fields map="parameters" value-field="newEntity"/>
-
-        <if-empty field="newEntity.fromDate">
-            <now-timestamp field="newEntity.fromDate"/>
-        </if-empty>
-
-        <create-value value-field="newEntity"/>
-    </simple-method>
-    <simple-method method-name="updateProdCatalogToParty" short-description="Update 
ProdCatalog To Party">
-        <make-value value-field="lookupPKMap" entity-name="ProdCatalogRole"/>
-        <set-pk-fields map="parameters" value-field="lookupPKMap"/>
-        <find-by-primary-key entity-name="ProdCatalogRole" map="lookupPKMap" 
value-field="lookedUpValue"/>
-        <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
-        <store-value value-field="lookedUpValue"/>
-    </simple-method>
-    <simple-method method-name="removeProdCatalogFromParty" short-description="Remove 
ProdCatalog From Party">
-        <make-value value-field="lookupPKMap" entity-name="ProdCatalogRole"/>
-        <set-pk-fields map="parameters" value-field="lookupPKMap"/>
-        <find-by-primary-key entity-name="ProdCatalogRole" map="lookupPKMap" 
value-field="lookedUpValue"/>
-        <remove-value value-field="lookedUpValue"/>
-    </simple-method>
<simple-method method-name="getAllCategories" short-description="get All categories" login-required="false">
          <if-not-empty field="parameters.topCategory">
@@ -127,6 +41,7 @@ under the License.
              <log level="info" 
message="=========${category.productCategoryId}"/>
          </iterate -->
      </simple-method>
+
      <simple-method method-name="getRelatedCategories" short-description="get All Related 
categories" login-required="false">
          <set value="sequenceNum" field="orderByString"/>
          <field-to-list list="orderByStringList" field="orderByString"/>
@@ -182,6 +97,7 @@ under the License.
          </iterate -->
          <field-to-result result-name="categories" field="categories"/>
      </simple-method>
+
      <simple-method method-name="checkImageUrlForAllCategories" short-description="Check for 
image url exists or not for All categories" login-required="false">
          <!-- Get all the categories from top category -->
          <set-service-fields to-map="categoryFindContext" service-name="getAllCategories" 
map="parameters"/>
@@ -401,6 +317,7 @@ under the License.
              <field-to-result field="filesImageMap" 
result-name="filesImageMap"/>
          </if-not-empty>
      </simple-method>
+
      <simple-method method-name="checkImageUrlForProduct" short-description="Check for image url 
exists or not for product" login-required="false">
          <if-not-empty field="parameters.productId">
              <set field="productFindContext.productId" 
from-field="parameters.productId"/>
@@ -466,8 +383,8 @@ under the License.
              <!-- set-service-fields service-name="updateProduct" map-name="product" 
to-map-name="updateProductContext"/>
              <call-service service-name="updateProduct" 
in-map-name="updateProductContext" / -->
              </if-not-empty>
-
      </simple-method>
+
      <!-- get file image url and return Y or N -->
      <simple-method method-name="checkImageUrl" short-description="Check for image url exists or 
not" login-required="false">
          <set field="imageUrl" from-field="parameters.imageUrl"/>
@@ -530,7 +447,7 @@ under the License.
          <entity-and list="prodCatalogCategoryList" entity-name="ProdCatalogCategory" 
filter-by-date="false">
              <field-map field-name="prodCatalogId" 
from-field="parameters.prodCatalogId"/>
          </entity-and>
-
+
          <!-- Get all categories -->
          <call-class-method method-name="newInstance" class-name="javolution.util.FastList" 
ret-field="parameters.productCategories"/>
          <iterate entry="prodCatalogCategory" list="prodCatalogCategoryList">
@@ -541,9 +458,8 @@ under the License.
              <set field="parameters.parentProductCategoryId" 
from-field="rootProductCategoryId"/>
              <call-simple-method 
method-name="createMissingCategoryAltUrlInline"/>
          </iterate>
-
+
          <iterate entry="productCategoryList" 
list="parameters.productCategories">
-
              <!-- Create Product Category Alternative URLs -->
              <if-empty field="category">
                  <entity-condition list="productCategoryContentAndInfoList" 
entity-name="ProductCategoryContentAndInfo" filter-by-date="true" use-cache="true">
@@ -598,7 +514,7 @@ under the License.
                  </else>
                  </if-empty>
              </if-empty>
-
+
              <!-- Create Product Alternative URLs -->
              <if-empty field="product">
                  <entity-condition list="productCategoryMemberList" 
entity-name="ProductCategoryMember" filter-by-date="true" use-cache="true">
@@ -659,6 +575,7 @@ under the License.
          <field-to-result field="categoriesUpdated" 
result-name="categoriesUpdated"/>
          <field-to-result field="productsUpdated" 
result-name="productsUpdated"/>
      </simple-method>
+
      <simple-method method-name="createMissingCategoryAltUrlInline" 
short-description="create missing category alternative inline">
          <entity-and list="productCategoryRollups" entity-name="ProductCategoryRollup" 
filter-by-date="true">
              <field-map field-name="parentProductCategoryId" 
from-field="parameters.parentProductCategoryId"/>
@@ -669,7 +586,6 @@ under the License.
                  <field-map field-name="productCategoryId" 
from-field="productCategoryRollup.productCategoryId"/>
              </entity-one>
              <field-to-list list="parameters.productCategories" 
field="productCategory"/>
-
              <!-- find rollup product categories -->
              <set field="parameters.parentProductCategoryId" 
from-field="productCategoryRollup.productCategoryId"/>
              <call-simple-method 
method-name="createMissingCategoryAltUrlInline"/>

Modified: ofbiz/trunk/applications/product/servicedef/services.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/servicedef/services.xml?rev=1628405&r1=1628404&r2=1628405&view=diff
==============================================================================
--- ofbiz/trunk/applications/product/servicedef/services.xml (original)
+++ ofbiz/trunk/applications/product/servicedef/services.xml Tue Sep 30 
11:59:29 2014
@@ -624,69 +624,60 @@ under the License.
      <!-- ======================================== -->
      <!-- Product Catalog Services -->
      <!-- ======================================== -->
-    <service name="createProdCatalog" default-entity-name="ProdCatalog" 
engine="simple"
-                location="component://product/script/org/ofbiz/product/catalog/CatalogServices.xml" 
invoke="createProdCatalog" auth="true">
+    <service name="createProdCatalog" default-entity-name="ProdCatalog" engine="entity-auto" 
invoke="create" auth="true">
          <description>Create a ProdCatalog</description>
          <permission-service service-name="catalogPermissionCheck" 
main-action="CREATE"/>
          <auto-attributes include="pk" mode="INOUT" optional="true"/>
          <auto-attributes include="nonpk" mode="IN" optional="true"/>
          <override name="catalogName" optional="false"/>
      </service>
-    <service name="updateProdCatalog" default-entity-name="ProdCatalog" 
engine="simple"
-                location="component://product/script/org/ofbiz/product/catalog/CatalogServices.xml" 
invoke="updateProdCatalog" auth="true">
+    <service name="updateProdCatalog" default-entity-name="ProdCatalog" engine="entity-auto" 
invoke="update" auth="true">
          <description>Update an ProdCatalog</description>
          <permission-service service-name="catalogPermissionCheck" 
main-action="UPDATE"/>
          <auto-attributes include="pk" mode="IN" optional="false"/>
          <auto-attributes include="nonpk" mode="IN" optional="true"/>
          <override name="catalogName" optional="false"/>
      </service>
-    <service name="deleteProdCatalog" default-entity-name="ProdCatalog" 
engine="simple"
-                location="component://product/script/org/ofbiz/product/catalog/CatalogServices.xml" 
invoke="deleteProdCatalog" auth="true">
+    <service name="deleteProdCatalog" default-entity-name="ProdCatalog" engine="entity-auto" 
invoke="delete" auth="true">
          <description>Delete an ProdCatalog</description>
          <permission-service service-name="catalogPermissionCheck" 
main-action="DELETE"/>
          <auto-attributes include="pk" mode="IN" optional="false"/>
      </service>
- <service name="addProductCategoryToProdCatalog" default-entity-name="ProdCatalogCategory" engine="simple"
-                location="component://product/script/org/ofbiz/product/catalog/CatalogServices.xml" 
invoke="addProductCategoryToProdCatalog" auth="true">
+    <service name="addProductCategoryToProdCatalog" default-entity-name="ProdCatalogCategory" 
engine="entity-auto" invoke="create" auth="true">
          <description>Add ProductCategory To ProdCatalog</description>
          <permission-service service-name="catalogPermissionCheck" 
main-action="CREATE"/>
          <auto-attributes include="pk" mode="IN" optional="false"/>
          <auto-attributes include="nonpk" mode="IN" optional="true"/>
          <override name="fromDate" optional="true"/>
      </service>
-    <service name="updateProductCategoryToProdCatalog" 
default-entity-name="ProdCatalogCategory" engine="simple"
-                location="component://product/script/org/ofbiz/product/catalog/CatalogServices.xml" 
invoke="updateProductCategoryToProdCatalog" auth="true">
+    <service name="updateProductCategoryToProdCatalog" default-entity-name="ProdCatalogCategory" 
engine="entity-auto" invoke="update" auth="true">
          <description>Add ProductCategory To ProdCatalog</description>
          <permission-service service-name="catalogPermissionCheck" 
main-action="UPDATE"/>
          <auto-attributes include="pk" mode="IN" optional="false"/>
          <auto-attributes include="nonpk" mode="IN" optional="true"/>
          <override name="prodCatalogCategoryTypeId" optional="false"/>
      </service>
-    <service name="removeProductCategoryFromProdCatalog" 
default-entity-name="ProdCatalogCategory" engine="simple"
-                location="component://product/script/org/ofbiz/product/catalog/CatalogServices.xml" 
invoke="removeProductCategoryFromProdCatalog" auth="true">
+    <service name="removeProductCategoryFromProdCatalog" default-entity-name="ProdCatalogCategory" 
engine="entity-auto" invoke="delete" auth="true">
          <description>Remove ProductCategory From ProdCatalog</description>
          <permission-service service-name="catalogPermissionCheck" 
main-action="DELETE"/>
          <auto-attributes include="pk" mode="IN" optional="false"/>
      </service>
- <service name="addProdCatalogToParty" default-entity-name="ProdCatalogRole" engine="simple"
-                location="component://product/script/org/ofbiz/product/catalog/CatalogServices.xml" 
invoke="addProdCatalogToParty" auth="true">
+    <service name="addProdCatalogToParty" default-entity-name="ProdCatalogRole" engine="entity-auto" 
invoke="create" auth="true">
          <description>Add ProdCatalog To Party</description>
          <permission-service service-name="prodCatalogToPartyPermissionCheck" 
main-action="CREATE"/>
          <auto-attributes include="pk" mode="IN" optional="false"/>
          <auto-attributes include="nonpk" mode="IN" optional="true"/>
          <override name="fromDate" optional="true"/>
      </service>
-    <service name="updateProdCatalogToParty" default-entity-name="ProdCatalogRole" 
engine="simple"
-                location="component://product/script/org/ofbiz/product/catalog/CatalogServices.xml" 
invoke="updateProdCatalogToParty" auth="true">
+    <service name="updateProdCatalogToParty" default-entity-name="ProdCatalogRole" 
engine="entity-auto" invoke="update" auth="true">
          <description>Add ProdCatalog To Party</description>
          <permission-service service-name="prodCatalogToPartyPermissionCheck" 
main-action="UPDATE"/>
          <auto-attributes include="pk" mode="IN" optional="false"/>
          <auto-attributes include="nonpk" mode="IN" optional="true"/>
      </service>
-    <service name="removeProdCatalogFromParty" default-entity-name="ProdCatalogRole" 
engine="simple"
-                location="component://product/script/org/ofbiz/product/catalog/CatalogServices.xml" 
invoke="removeProdCatalogFromParty" auth="true">
+    <service name="removeProdCatalogFromParty" default-entity-name="ProdCatalogRole" 
engine="entity-auto" invoke="delete" auth="true">
          <description>Remove ProdCatalog From Party</description>
          <permission-service service-name="prodCatalogToPartyPermissionCheck" 
main-action="DELETE"/>
          <auto-attributes include="pk" mode="IN" optional="false"/>



Reply via email to