Re: svn commit: r1526276 - /ofbiz/trunk/framework/common/servicedef/services.xml

Posted by Jacques Le Roux on
URL: http://ofbiz.116.s1.nabble.com/Re-svn-commit-r1526276-ofbiz-trunk-framework-common-servicedef-services-xml-tp4644171.html

Is this rather not hidding the warning?
Is a FIXME (moreover in xml  code) better?
Do you intent to fix it soon?

Jacques

[hidden email] wrote:

> Author: adrianc
> Date: Wed Sep 25 19:57:13 2013
> New Revision: 1526276
>
> URL: http://svn.apache.org/r1526276
> Log:
> Fixed a warning caused by a framework dependency on the Party component.
>
> Modified:
>    ofbiz/trunk/framework/common/servicedef/services.xml
>
> Modified: ofbiz/trunk/framework/common/servicedef/services.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/servicedef/services.xml?rev=1526276&r1=1526275&r2=1526276&view=diff
> ==============================================================================
> --- ofbiz/trunk/framework/common/servicedef/services.xml (original)
> +++ ofbiz/trunk/framework/common/servicedef/services.xml Wed Sep 25 19:57:13 2013
> @@ -181,9 +181,9 @@ under the License.
>         <description>Create a CustomTimePeriod record</description>
>         <auto-attributes mode="OUT" include="pk" optional="false"/>
>         <auto-attributes mode="IN" include="nonpk" optional="true"/>
> +        <attribute name="organizationPartyId" type="String" mode="IN" optional="false"/><!-- FIXME: Framework dependency on
>         Party component --> <override name="fromDate" optional="false"/>
>         <override name="thruDate" optional="false"/>
> -        <override name="organizationPartyId" optional="false"/>
>         <override name="periodTypeId" optional="false"/>
>     </service>
>     <service name="updateCustomTimePeriod" default-entity-name="CustomTimePeriod" engine="simple"