Dev - [Fwd: [OFBiz] SVN: r6461 - trunk/applications/party/servicedef]

[Si Chen-2]

Jacopo,

This actually broke the service, because statusId is not a field of
Person but of Party.  We were passing it in so that after creating
person it can be set on party, but using override caused the parameter
to get lost.

Should we revert it?  Is there a different validation error issue?

Si

Author: jacopo
Date: 2006-01-03 05:43:10 -0600 (Tue, 03 Jan 2006)
New Revision: 6461

Modified:
   trunk/applications/party/servicedef/services.xml
Log:
Fixed wrong service definition that was causing a validation error.


Modified: trunk/applications/party/servicedef/services.xml
===================================================================
--- trunk/applications/party/servicedef/services.xml 2006-01-03 08:02:10 UTC (rev 6460)
+++ trunk/applications/party/servicedef/services.xml 2006-01-03 11:43:10 UTC (rev 6461)
@@ -50,7 +50,7 @@
         <auto-attributes mode="IN" include="nonpk" optional="true"/>
         <override name="firstName" optional="false"/>
         <override name="lastName" optional="false"/>
-        <attribute name="statusId" type="String" mode="IN" optional="true"/>
+        <override name="statusId" type="String" mode="IN" optional="true"/>
     </service>
     <service name="createPersonAndUserLogin" engine="simple" require-new-transaction="true"
             location="org/ofbiz/party/party/PartySimpleMethods.xml" invoke="createPersonAndUserLogin" auth="false">

 
_______________________________________________
Svn mailing list
[hidden email]
http://lists.ofbiz.org/mailman/listinfo/svn


 
_______________________________________________
Dev mailing list
[hidden email]
http://lists.ofbiz.org/mailman/listinfo/dev

[Jacopo Cappellato]

Si,

yes my patch actually fixed the XML validation error but I've broken the
service... sorry.
Now, with revision 6474, both the problems should be fixed: I've changed
back the "statusId" from <override> to <attribute> but I've moved it
before the existing <override> elements.

Thanks,

Jacopo

Si Chen wrote:

> Jacopo,
>
> This actually broke the service, because statusId is not a field of
> Person but of Party.  We were passing it in so that after creating
> person it can be set on party, but using override caused the parameter
> to get lost.
>
> Should we revert it?  Is there a different validation error issue?
>
> Si
>
> ------------------------------------------------------------------------
>
> Subject:
> [OFBiz] SVN: r6461 - trunk/applications/party/servicedef
> From:
> [hidden email]
> Date:
> 3 Jan 2006 11:43:16 -0000
> To:
> [hidden email]
>
> To:
> [hidden email]
>
>
> Author: jacopo
> Date: 2006-01-03 05:43:10 -0600 (Tue, 03 Jan 2006)
> New Revision: 6461
>
> Modified:
>    trunk/applications/party/servicedef/services.xml
> Log:
> Fixed wrong service definition that was causing a validation error.
>
>
> Modified: trunk/applications/party/servicedef/services.xml
> ===================================================================
> --- trunk/applications/party/servicedef/services.xml 2006-01-03 08:02:10 UTC (rev 6460)
> +++ trunk/applications/party/servicedef/services.xml 2006-01-03 11:43:10 UTC (rev 6461)
> @@ -50,7 +50,7 @@
>          <auto-attributes mode="IN" include="nonpk" optional="true"/>
>          <override name="firstName" optional="false"/>
>          <override name="lastName" optional="false"/>
> -        <attribute name="statusId" type="String" mode="IN" optional="true"/>
> +        <override name="statusId" type="String" mode="IN" optional="true"/>
>      </service>
>      <service name="createPersonAndUserLogin" engine="simple" require-new-transaction="true"
>              location="org/ofbiz/party/party/PartySimpleMethods.xml" invoke="createPersonAndUserLogin" auth="false">
>
>  
> _______________________________________________
> Svn mailing list
> [hidden email]
> http://lists.ofbiz.org/mailman/listinfo/svn
>
>
>
> ------------------------------------------------------------------------
>
>  
> _______________________________________________
> Dev mailing list
> [hidden email]
> http://lists.ofbiz.org/mailman/listinfo/dev

 
_______________________________________________
Dev mailing list
[hidden email]
http://lists.ofbiz.org/mailman/listinfo/dev