You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Nicolas Malin (JIRA)" <ji...@apache.org> on 2014/09/03 09:18:51 UTC

[jira] [Commented] (OFBIZ-5750) Convert Party entites CRUD service from simple to entity-auto

    [ https://issues.apache.org/jira/browse/OFBIZ-5750?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14119491#comment-14119491 ] 

Nicolas Malin commented on OFBIZ-5750:
--------------------------------------

Ok, thanks for your review and comment Jacques.
I will continue on next entities ;)

> Convert Party entites CRUD service from simple to entity-auto
> -------------------------------------------------------------
>
>                 Key: OFBIZ-5750
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-5750
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: party
>    Affects Versions: Trunk
>            Reporter: Nicolas Malin
>            Assignee: Jacques Le Roux
>            Priority: Trivial
>              Labels: crud, entity-auto
>             Fix For: Upcoming Branch
>
>         Attachments: OFBIZ-5750.patch
>
>
> I converted CRUD service to entity-auto for :
>   PartyRole
>   PostalAddressBoundary
>   PartyClassification
>   PartyClassificationGroup
>   PartyAttribute
>   Vendor
>   PartyCarrierAccount
> Only entity with their services present on PartyServices.xml
> I change the PartyRole  attributes service like that
> {code}
> -        <attribute name="partyId" type="String" mode="IN" optional="true"/>
> -        <attribute name="roleTypeId" type="String" mode="IN" optional="false"/>
> +        <auto-attributes include="pk" mode="IN" optional="false"/>
> {code}
> The currently simple method haven't a specific resolution if the partyId isn't pass, so the service failed with database constraint error.
> I run manual test from Party Profile screen with success (except for PostalAddressBoundary that haven't standard screen, so call by runService) and I run ./ant clean-all load-demo run-tests without error



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)