Author: jleroux
Date: Wed May 22 07:39:54 2019
New Revision: 1859693
URL: http://svn.apache.org/viewvc?rev=1859693&view=rev
Log:
"Applied fix from trunk for revision: 1859691"
------------------------------------------------------------------------
r1859691 | jleroux | 2019-05-22 09:39:08 +0200 (mer. 22 mai 2019) | 15 lignes
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/branches/release17.12/ (props changed)
ofbiz/ofbiz-framework/branches/release17.12/applications/manufacturing/minilang/bom/BomSimpleMethods.xml
ofbiz/ofbiz-framework/branches/release17.12/applications/manufacturing/widget/manufacturing/BomForms.xml
Propchange: ofbiz/ofbiz-framework/branches/release17.12/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Wed May 22 07:39:54 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,1849165,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,1857088,1857099,1857173,1857180,1857213,1857392,1857617,1857692,1857813,1858035,1858250,1858256,1858275,1858319,1858432,1858444,1858523,1858539,1858965,1858980,1859033,1859055,1859087,1859255,1859263,1859543,1859571,1859576
+/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,1849165,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,1857088,1857099,1857173,1857180,1857213,1857392,1857617,1857692,1857813,1858035,1858250,1858256,1858275,1858319,1858432,1858444,1858523,1858539,1858965,1858980,1859033,1859055,1859087,1859255,1859263,1859543,1859571,1859576,1859691
Modified:
ofbiz/ofbiz-framework/branches/release17.12/applications/manufacturing/minilang/bom/BomSimpleMethods.xml
URL:
http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/branches/release17.12/applications/manufacturing/minilang/bom/BomSimpleMethods.xml?rev=1859693&r1=1859692&r2=1859693&view=diff
==============================================================================
---
ofbiz/ofbiz-framework/branches/release17.12/applications/manufacturing/minilang/bom/BomSimpleMethods.xml
(original)
+++
ofbiz/ofbiz-framework/branches/release17.12/applications/manufacturing/minilang/bom/BomSimpleMethods.xml
Wed May 22 07:39:54 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/branches/release17.12/applications/manufacturing/widget/manufacturing/BomForms.xml
URL:
http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/branches/release17.12/applications/manufacturing/widget/manufacturing/BomForms.xml?rev=1859693&r1=1859692&r2=1859693&view=diff
==============================================================================
---
ofbiz/ofbiz-framework/branches/release17.12/applications/manufacturing/widget/manufacturing/BomForms.xml
(original)
+++
ofbiz/ofbiz-framework/branches/release17.12/applications/manufacturing/widget/manufacturing/BomForms.xml
Wed May 22 07:39:54 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>
