fixed, thank you Jacques.

Jacopo

On Jul 16, 2014, at 9:55 AM, Jacopo Cappellato <jacopo.cappell...@gmail.com> 
wrote:

> 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