Author: jleroux
Date: Wed May 22 07:39:08 2019
New Revision: 1859691

URL: http://svn.apache.org/viewvc?rev=1859691&view=rev
Log:
Fixed: It's impossible to create more than 1 ProductManufacturingRule
(OFBIZ-11047)

In a previous commit for OFBIZ-9997 I wrote:
>DeleteProductManufacturingRule seems weird to me because I don't understand how
>to create several Manufacturing Rules. I'll check Sharan's book :)

I checked Sharan's book and there is nothing about it

It also shows ManufacturingMachineGroupIdAlreadyExist ("Fixed Asset Group ID 
doesn't exist") when you try to create one using manually the 
AddProductManufacturingRule service which makes no sense.

This fixes it by adding the necessary in BomForms.xml and fixing 
addProductManufacturingRule which was wrong from start (pre Apache era)

Modified:
    
ofbiz/ofbiz-framework/trunk/applications/manufacturing/minilang/bom/BomSimpleMethods.xml
    
ofbiz/ofbiz-framework/trunk/applications/manufacturing/widget/manufacturing/BomForms.xml

Modified: 
ofbiz/ofbiz-framework/trunk/applications/manufacturing/minilang/bom/BomSimpleMethods.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/manufacturing/minilang/bom/BomSimpleMethods.xml?rev=1859691&r1=1859690&r2=1859691&view=diff
==============================================================================
--- 
ofbiz/ofbiz-framework/trunk/applications/manufacturing/minilang/bom/BomSimpleMethods.xml
 (original)
+++ 
ofbiz/ofbiz-framework/trunk/applications/manufacturing/minilang/bom/BomSimpleMethods.xml
 Wed May 22 07:39:08 2019
@@ -117,24 +117,17 @@ under the License.
             <fail-property resource="ManufacturingUiLabels" 
property="ManufacturingMachineGroupPermissionError"/>
         </check-permission>
         <check-errors/>
-        <make-value entity-name="ProductManufacturingRule" 
value-field="lookupPKMap"/>
-        <set-pk-fields map="parameters" value-field="lookupPKMap"/>
-        <find-by-primary-key entity-name="ProductManufacturingRule" 
map="lookupPKMap" value-field="newEntity"/>
-        <if-not-empty field="newEntity.ruleId">
-            <add-error>
-                <fail-property resource="ManufacturingUiLabels" 
property="ManufacturingMachineGroupIdAlreadyExist"/>
-            </add-error>
-            <check-errors/>
-        <else>
-            <make-value entity-name="ProductManufacturingRule" 
value-field="newEntity"/>
-            <if-empty field="parameters.ruleId">
-                <sequenced-id sequence-name="ProductManufacturingRule" 
field="parameters.ruleId"/>
-            </if-empty>
-            <set-pk-fields map="parameters" value-field="newEntity"/>
-            <set-nonpk-fields map="parameters" value-field="newEntity"/>
-            <create-value value-field="newEntity"/>
-        </else>
-        </if-not-empty>
+
+        <make-value entity-name="ProductManufacturingRule" 
value-field="newEntity"/>
+        <set-pk-fields map="parameters" value-field="newEntity"/>
+        <set-nonpk-fields map="parameters" value-field="newEntity"/>
+        
+        <if-empty field="newEntity.ruleId">
+            <sequenced-id sequence-name="FinAccount" field="ruleId"/>
+            <to-string field="ruleId"/>
+            <set from-field="ruleId" field="newEntity.ruleId"/>
+        </if-empty>
+        <create-value value-field="newEntity"/>
     </simple-method>
 
     <simple-method method-name="deleteProductManufacturingRule" 
short-description="Remove a Product Manufacturing Rule">

Modified: 
ofbiz/ofbiz-framework/trunk/applications/manufacturing/widget/manufacturing/BomForms.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/manufacturing/widget/manufacturing/BomForms.xml?rev=1859691&r1=1859690&r2=1859691&view=diff
==============================================================================
--- 
ofbiz/ofbiz-framework/trunk/applications/manufacturing/widget/manufacturing/BomForms.xml
 (original)
+++ 
ofbiz/ofbiz-framework/trunk/applications/manufacturing/widget/manufacturing/BomForms.xml
 Wed May 22 07:39:08 2019
@@ -125,7 +125,10 @@ under the License.
                 <option key="AND" description="${uiLabelMap.CommonAnd}"/>
             </drop-down>
         </field>
-        <field name="submitButton" title="${uiLabelMap.CommonUpdate}">
+        <field use-when="ruleId==null" name="submitButton" 
title="${uiLabelMap.CommonCreate}">
+            <submit button-type="button"/>
+        </field>
+        <field use-when="ruleId!=null" name="submitButton" 
title="${uiLabelMap.CommonUpdate}">
             <submit button-type="button"/>
         </field>
     </form>


Reply via email to