yeah, sorry for this, I am fixing it now.

Jacopo

On Jul 16, 2014, at 9:39 AM, Jacques Le Roux <jacques.le.r...@les7arts.com> 
wrote:

> Hi Jacopo,
> 
> Are you aware this does not compile?
> 
> Jacques
> 
> Le 16/07/2014 09:16, jaco...@apache.org a écrit :
>> Author: jacopoc
>> Date: Wed Jul 16 07:16:15 2014
>> New Revision: 1610918
>> 
>> URL: http://svn.apache.org/r1610918
>> Log:
>> Fixed miscellaneous errors spotted while reviewing the logs generated by the 
>> ArtifactInfo tool.
>> 
>> Modified:
>>     
>> ofbiz/trunk/applications/accounting/script/org/ofbiz/accounting/agreement/AgreementServices.xml
>>     ofbiz/trunk/applications/accounting/widget/AgreementScreens.xml
>>     
>> ofbiz/trunk/applications/manufacturing/script/org/ofbiz/manufacturing/bom/BomSimpleMethods.xml
>>     
>> ofbiz/trunk/applications/marketing/script/org/ofbiz/sfa/lead/LeadServices.xml
>>     
>> ofbiz/trunk/applications/product/script/org/ofbiz/shipment/picklist/PicklistServices.xml
>>     ofbiz/trunk/framework/common/webcommon/WEB-INF/portal-controller.xml
>>     
>> ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ServiceArtifactInfo.java
>> 
>> Modified: 
>> ofbiz/trunk/applications/accounting/script/org/ofbiz/accounting/agreement/AgreementServices.xml
>> URL: 
>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/script/org/ofbiz/accounting/agreement/AgreementServices.xml?rev=1610918&r1=1610917&r2=1610918&view=diff
>> ==============================================================================
>> --- 
>> ofbiz/trunk/applications/accounting/script/org/ofbiz/accounting/agreement/AgreementServices.xml
>>  (original)
>> +++ 
>> ofbiz/trunk/applications/accounting/script/org/ofbiz/accounting/agreement/AgreementServices.xml
>>  Wed Jul 16 07:16:15 2014
>> @@ -100,12 +100,12 @@ under the License.
>>              </iterate>
>>          </if-compare>
>>          <if-compare field="parameters.copyAgreementFacilities" value="Y" 
>> operator="equals">
>> -            <get-related value-field="agreement" 
>> relation-name="AgreementFaclityAppl" list="agreementFaclityAppls"/>
>> -            <iterate list="agreementFaclityAppls" 
>> entry="agreementFaclityAppl">
>> -                <clear-field field="createAgreementFaclityApplInMap"/>
>> -                <set-service-fields 
>> service-name="createAgreementFaclityAppl" 
>> to-map="createAgreementFaclityApplInMap" map="agreementFaclityAppl"/>
>> -                <set from-field="agreementIdTo" 
>> field="createAgreementFaclityApplInMap.agreementId"/>
>> -                <call-service service-name="createAgreementFaclityAppl" 
>> in-map-name="createAgreementFaclityApplInMap"/>
>> +            <get-related value-field="agreement" 
>> relation-name="AgreementFacilityAppl" list="agreementFacilityAppls"/>
>> +            <iterate list="agreementFacilityAppls" 
>> entry="agreementFacilityAppl">
>> +                <clear-field field="createAgreementFacilityApplInMap"/>
>> +                <set-service-fields 
>> service-name="createAgreementFacilityAppl" 
>> to-map="createAgreementFacilityApplInMap" map="agreementFacilityAppl"/>
>> +                <set from-field="agreementIdTo" 
>> field="createAgreementFacilityApplInMap.agreementId"/>
>> +                <call-service service-name="createAgreementFacilityAppl" 
>> in-map-name="createAgreementFacilityApplInMap"/>
>>              </iterate>
>>          </if-compare>
>>          <if-compare field="parameters.copyAgreementParties" value="Y" 
>> operator="equals">
>> 
>> Modified: ofbiz/trunk/applications/accounting/widget/AgreementScreens.xml
>> URL: 
>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/widget/AgreementScreens.xml?rev=1610918&r1=1610917&r2=1610918&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/applications/accounting/widget/AgreementScreens.xml 
>> (original)
>> +++ ofbiz/trunk/applications/accounting/widget/AgreementScreens.xml Wed Jul 
>> 16 07:16:15 2014
>> @@ -485,9 +485,6 @@ under the License.
>>                          <screenlet 
>> title="${uiLabelMap.AccountingAgreementItem}">
>>                              <include-form 
>> name="ViewAgreementItemInfoForReport" 
>> location="component://accounting/widget/AgreementForms.xml"/>
>>                          </screenlet>
>> -                        <screenlet 
>> title="${uiLabelMap.AccountingAgreementItemFacilitiesForReport}">
>> -                            <include-form 
>> name="ListAgreementItemFacilitiesForReport" 
>> location="component://accounting/widget/AgreementForms.xml"/>
>> -                        </screenlet>
>>                      </decorator-section>
>>                  </decorator-screen>
>>              </widgets>
>> 
>> Modified: 
>> ofbiz/trunk/applications/manufacturing/script/org/ofbiz/manufacturing/bom/BomSimpleMethods.xml
>> URL: 
>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/manufacturing/script/org/ofbiz/manufacturing/bom/BomSimpleMethods.xml?rev=1610918&r1=1610917&r2=1610918&view=diff
>> ==============================================================================
>> --- 
>> ofbiz/trunk/applications/manufacturing/script/org/ofbiz/manufacturing/bom/BomSimpleMethods.xml
>>  (original)
>> +++ 
>> ofbiz/trunk/applications/manufacturing/script/org/ofbiz/manufacturing/bom/BomSimpleMethods.xml
>>  Wed Jul 16 07:16:15 2014
>> @@ -25,7 +25,6 @@ under the License.
>>          <if-empty field="parameters.fromDate">
>>              <now-timestamp field="parameters.fromDate"/>
>>          </if-empty>
>> -        <set field="" from-field=""/>
>>            <set field="searchDuplicatedAncestorContext.productId" 
>> from-field="parameters.productId"/>
>>          <set field="searchDuplicatedAncestorContext.productIdTo" 
>> from-field="parameters.productIdTo"/>
>> 
>> Modified: 
>> ofbiz/trunk/applications/marketing/script/org/ofbiz/sfa/lead/LeadServices.xml
>> URL: 
>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/marketing/script/org/ofbiz/sfa/lead/LeadServices.xml?rev=1610918&r1=1610917&r2=1610918&view=diff
>> ==============================================================================
>> --- 
>> ofbiz/trunk/applications/marketing/script/org/ofbiz/sfa/lead/LeadServices.xml
>>  (original)
>> +++ 
>> ofbiz/trunk/applications/marketing/script/org/ofbiz/sfa/lead/LeadServices.xml
>>  Wed Jul 16 07:16:15 2014
>> @@ -139,7 +139,7 @@
>>              <set-service-fields service-name="updatePartyRelationship" 
>> map="partyRelationship" to-map="deletePartyRelationship"/>
>>              <set from-field="nowTimestamp" 
>> field="deletePartyRelationship.thruDate"/>
>>              <call-service service-name="updatePartyRelationship" 
>> in-map-name="deletePartyRelationship"/>
>> -            <log level="info" message="Expiring relationship  
>> ${deletePartyRelationship}"> </log>
>> +            <log level="info" message="Expiring relationship  
>> ${deletePartyRelationship}"/>
>>              <clear-field field="deletePartyRelationship"/>
>>              <clear-field field="partyRelationship"/>
>>          </if-not-empty>
>> 
>> Modified: 
>> ofbiz/trunk/applications/product/script/org/ofbiz/shipment/picklist/PicklistServices.xml
>> URL: 
>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/shipment/picklist/PicklistServices.xml?rev=1610918&r1=1610917&r2=1610918&view=diff
>> ==============================================================================
>> --- 
>> ofbiz/trunk/applications/product/script/org/ofbiz/shipment/picklist/PicklistServices.xml
>>  (original)
>> +++ 
>> ofbiz/trunk/applications/product/script/org/ofbiz/shipment/picklist/PicklistServices.xml
>>  Wed Jul 16 07:16:15 2014
>> @@ -947,8 +947,7 @@ under the License.
>>              - editPicklistItem
>>      -->
>>      <simple-method method-name="editPicklistItem" short-description="Edit a 
>> Picklist Item">
>> -        <log level="info"><string>Edit a Picklist Item</string></log>
>> -
>> +
>>          <make-value value-field="lookupPKMap" entity-name="PicklistItem"/>
>>          <set-pk-fields map="parameters" value-field="lookupPKMap"/>
>>          <find-by-primary-key map="lookupPKMap" value-field="picklistItem"/>
>> 
>> Modified: 
>> ofbiz/trunk/framework/common/webcommon/WEB-INF/portal-controller.xml
>> URL: 
>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/webcommon/WEB-INF/portal-controller.xml?rev=1610918&r1=1610917&r2=1610918&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/framework/common/webcommon/WEB-INF/portal-controller.xml 
>> (original)
>> +++ ofbiz/trunk/framework/common/webcommon/WEB-INF/portal-controller.xml Wed 
>> Jul 16 07:16:15 2014
>> @@ -21,7 +21,10 @@ under the License.
>>  <site-conf xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
>>          
>> xsi:noNamespaceSchemaLocation="http://ofbiz.apache.org/dtds/site-conf.xsd";>
>>      <description>Portal ControlServlet Configuration File</description>
>> -
>> +
>> +    <request-map uri="main">
>> +        <response name="success" type="view" value="showPortalPage"/>
>> +    </request-map>
>>      <!-- Portlet show requests -->
>>      <request-map uri="showPortlet">
>>          <security https="true" auth="true"/>
>> 
>> Modified: 
>> ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ServiceArtifactInfo.java
>> URL: 
>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ServiceArtifactInfo.java?rev=1610918&r1=1610917&r2=1610918&view=diff
>> ==============================================================================
>> --- 
>> ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ServiceArtifactInfo.java
>>  (original)
>> +++ 
>> ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ServiceArtifactInfo.java
>>  Wed Jul 16 07:16:15 2014
>> @@ -121,7 +121,7 @@ public class ServiceArtifactInfo extends
>>      }
>>      protected void populateEntitiesFromNameSet(Set<String> 
>> allEntityNameSet) throws GeneralException {
>>          for (String entityName: allEntityNameSet) {
>> -            if (entityName.contains("${")) {
>> +            if (UtilValidate.isEmail(entityName) || 
>> entityName.contains("${")) {
>>                  continue;
>>              }
>>              // attempt to convert relation names to entity names
>> 
>> 
>> 
> 
> -- 

Reply via email to