Also, please put the revision number in the Jira issue. That helps others doing research on changes.

Adrian Crum
Sandglass Software
www.sandglass-software.com

On 9/30/2014 5:43 PM, Jacques Le Roux wrote:
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