svn commit: r1726544 - in /ofbiz/branches/release15.12: ./ applications/marketing/widget/sfa/forms/OpportunityForms.xml applications/order/script/org/ofbiz/order/opportunity/OpportunityServices.xml applications/order/servicedef/secas.xml

Author: jleroux
Date: Sun Jan 24 19:09:20 2016
New Revision: 1726544

URL: http://svn.apache.org/viewvc?rev=1726544&view=rev
Log:
"Applied fix from trunk for revision: 1723248 "
------------------------------------------------------------------------
r1723248 | hansbak | 2016-01-06 08:53:26 +0100 (mer. 06 janv. 2016) | 1 ligne

various fixes to the salesopportunity list and create functions
------------------------------------------------------------------------


Modified:
    ofbiz/branches/release15.12/   (props changed)
    ofbiz/branches/release15.12/applications/marketing/widget/sfa/forms/OpportunityForms.xml
    ofbiz/branches/release15.12/applications/order/script/org/ofbiz/order/opportunity/OpportunityServices.xml
    ofbiz/branches/release15.12/applications/order/servicedef/secas.xml

Propchange: ofbiz/branches/release15.12/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Sun Jan 24 19:09:20 2016
@@ -9,4 +9,4 @@
 /ofbiz/branches/json-integration-refactoring:1634077-1635900
 /ofbiz/branches/multitenant20100310:921280-927264
 /ofbiz/branches/release13.07:1547657
-/ofbiz/trunk:1722712,1723007,1724402,1724411,1724566,1724689,1724763,1724916,1724918,1724925,1724930,1724940,1724943,1724946,1724951,1724954,1724957,1724975,1724978,1725006,1725217,1725257,1725561,1726388
+/ofbiz/trunk:1722712,1723007,1723248,1724402,1724411,1724566,1724689,1724763,1724916,1724918,1724925,1724930,1724940,1724943,1724946,1724951,1724954,1724957,1724975,1724978,1725006,1725217,1725257,1725561,1726388

Modified: ofbiz/branches/release15.12/applications/marketing/widget/sfa/forms/OpportunityForms.xml
URL: http://svn.apache.org/viewvc/ofbiz/branches/release15.12/applications/marketing/widget/sfa/forms/OpportunityForms.xml?rev=1726544&r1=1726543&r2=1726544&view=diff
==============================================================================
--- ofbiz/branches/release15.12/applications/marketing/widget/sfa/forms/OpportunityForms.xml (original)
+++ ofbiz/branches/release15.12/applications/marketing/widget/sfa/forms/OpportunityForms.xml Sun Jan 24 19:09:20 2016
@@ -64,7 +64,6 @@ under the License.
             </service>
         </actions>
         <row-actions>
-            <set field="isDateAfterNow" value="${groovy:org.ofbiz.base.util.UtilValidate.isDateAfterNow(estimatedCloseDate)}" type="Boolean"/>
             <entity-one entity-name="SalesOpportunity" value-field="salesOpportunity"/>
         </row-actions>
         <field name="salesOpportunityId"><hidden/></field>
@@ -85,7 +84,7 @@ under the License.
         <field name="estimatedAmount" title="${uiLabelMap.SfaEstimatedAmount}"><display/></field>
         <field name="nextStepDate" position="1" title="${uiLabelMap.SfaNextStepDate}" sort-field="true"><display/></field>
         <field name="estimatedCloseDate" title="${uiLabelMap.SfaCloseDate}"><display/></field>
-        <field name="editButton" title="${uiLabelMap.CommonClose}" use-when="${groovy: estimatedCloseDate == null || isDateAfterNow == true || opportunityStageId != &quot;SOSTG_CLOSED&quot;}" widget-style="buttontext">
+        <field name="editButton" title="${uiLabelMap.CommonClose}" use-when="estimatedCloseDate == void || estimatedCloseDate == null || org.ofbiz.base.util.UtilValidate.isDateAfterNow(estimatedCloseDate) || opportunityStageId != &quot;SOSTG_CLOSED&quot;" widget-style="buttontext">
             <hyperlink target="closeSalesOpportunity" description="${uiLabelMap.CommonClose}">
                 <parameter param-name="salesOpportunityId"/>
                 <parameter param-name="opportunityStageId" value="SOSTG_CLOSED"/>
@@ -151,10 +150,10 @@ under the License.
                 </entity-options>
             </drop-down>
         </field>
-        <field name="accountPartyId" use-when="communicationEvent==null" title="${uiLabelMap.SfaInitialAccount}" ><lookup target-form-name="LookupPartyGroup" default-value="${accountPartyId}"/></field>
-        <field name="accountPartyId" use-when="communicationEvent!=null" title="${uiLabelMap.SfaInitialAccount}" ><lookup target-form-name="LookupPartyGroup" default-value="${accountPartyId}"/></field>
-        <field name="leadPartyId" use-when="communicationEvent==null" title="${uiLabelMap.SfaLead}"><lookup target-form-name="LookupPerson" default-value="${leadPartyId}"/></field>
-        <field name="leadPartyId" use-when="communicationEvent!=null" title="${uiLabelMap.SfaLead}"><lookup target-form-name="LookupPerson" default-value="${communicationEvent.partyIdFrom}"/></field>
+        <field name="accountPartyId" use-when="communicationEvent==null" title="${uiLabelMap.SfaInitialAccount}" ><lookup target-form-name="LookupAccount" default-value="${accountPartyId}"/></field>
+        <field name="accountPartyId" use-when="communicationEvent!=null" title="${uiLabelMap.SfaInitialAccount}" ><lookup target-form-name="LookupAccount" default-value="${accountPartyId}"/></field>
+        <field name="leadPartyId" use-when="communicationEvent==null" title="${uiLabelMap.SfaLead}"><lookup target-form-name="LookupLead" default-value="${leadPartyId}"/></field>
+        <field name="leadPartyId" use-when="communicationEvent!=null" title="${uiLabelMap.SfaLead}"><lookup target-form-name="LookupLead" default-value="${communicationEvent.partyIdFrom}"/></field>
         <field name="submitButton" title="${uiLabelMap.CommonSave}" use-when="salesOpportunity==null"><submit button-type="button"/></field>
         <field name="submitButton" title="${uiLabelMap.CommonUpdate}" use-when="salesOpportunity!=null"><submit button-type="button"/></field>
     </form>

Modified: ofbiz/branches/release15.12/applications/order/script/org/ofbiz/order/opportunity/OpportunityServices.xml
URL: http://svn.apache.org/viewvc/ofbiz/branches/release15.12/applications/order/script/org/ofbiz/order/opportunity/OpportunityServices.xml?rev=1726544&r1=1726543&r2=1726544&view=diff
==============================================================================
--- ofbiz/branches/release15.12/applications/order/script/org/ofbiz/order/opportunity/OpportunityServices.xml (original)
+++ ofbiz/branches/release15.12/applications/order/script/org/ofbiz/order/opportunity/OpportunityServices.xml Sun Jan 24 19:09:20 2016
@@ -143,13 +143,11 @@ This file contains basic services for Sa
             </if-not-empty>
             <!-- Check party role for create Account Role -->
             <if-not-empty field="parameters.accountPartyId">
-                <set field="roleMap.roleTypeId" value="ACCOUNT"/>
-                <set field="roleMap.partyId" from-field="parameters.accountPartyId"/>
-                <call-service service-name="ensurePartyRole" in-map-name="roleMap"/>
-            </if-not-empty>
             <set field="salesOpportunityRole.roleTypeId" value="ACCOUNT"/>
-            <set field="salesOpportunityRole.salesOpportunityId" from-field="parameters.salesOpportunityId"/>
             <set field="salesOpportunityRole.partyId" from-field="parameters.accountPartyId"/>
+                <call-service service-name="ensurePartyRole" in-map-name="salesOpportunityRole"/>
+            </if-not-empty>
+            <set field="salesOpportunityRole.salesOpportunityId" from-field="parameters.salesOpportunityId"/>
             <call-service service-name="createSalesOpportunityRole" in-map-name="salesOpportunityRole"/>
         </if-compare-field>
     </simple-method>
@@ -164,12 +162,10 @@ This file contains basic services for Sa
                 <remove-value value-field="lookedUpValue"/>
             </if-not-empty>
             <!-- Check party role for create Lead Role -->
-            <if-not-empty field="parameters.leadPartyId">
-                <call-service service-name="ensurePartyRole" in-map-name="roleMap"/>
-            </if-not-empty>
             <set field="salesOpportunityRole.roleTypeId" value="LEAD"/>
-            <set field="salesOpportunityRole.salesOpportunityId" from-field="parameters.salesOpportunityId"/>
             <set field="salesOpportunityRole.partyId" from-field="parameters.leadPartyId"/>
+            <call-service service-name="ensurePartyRole" in-map-name="salesOpportunityRole"/>
+            <set field="salesOpportunityRole.salesOpportunityId" from-field="parameters.salesOpportunityId"/>
             <call-service service-name="createSalesOpportunityRole" in-map-name="salesOpportunityRole"/>
         </if-compare-field>
     </simple-method>

Modified: ofbiz/branches/release15.12/applications/order/servicedef/secas.xml
URL: http://svn.apache.org/viewvc/ofbiz/branches/release15.12/applications/order/servicedef/secas.xml?rev=1726544&r1=1726543&r2=1726544&view=diff
==============================================================================
--- ofbiz/branches/release15.12/applications/order/servicedef/secas.xml (original)
+++ ofbiz/branches/release15.12/applications/order/servicedef/secas.xml Sun Jan 24 19:09:20 2016
@@ -421,7 +421,6 @@ under the License.
         <action service="createSalesOpportunityAccountRole" mode="sync"/>
     </eca>
     <eca service="createSalesOpportunity" event="commit">
-        <condition field-name="leadPartyId" operator="is-not-empty"/>
         <action service="createSalesOpportunityLeadRole" mode="sync"/>
     </eca>
     <eca service="updateSalesOpportunity" event="commit">