ofbiz-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Paul Foxworthy (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OFBIZ-7950) Improve consistency of service createEmployee
Date Thu, 27 Oct 2016 06:13:58 GMT

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

Paul Foxworthy commented on OFBIZ-7950:
---------------------------------------

Hi Montalbano,

What do you mean by "Same way for the address"? Addresses have more than one component, sure,
but the components that are mandatory for a valid address vary from one place to another.
I wonder if address is too hard, and we're better leaving it alone.

> Improve consistency of service createEmployee
> ---------------------------------------------
>
>                 Key: OFBIZ-7950
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-7950
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: humanres
>    Affects Versions: Trunk
>            Reporter: Montalbano Florian
>            Assignee: Nicolas Malin
>            Priority: Minor
>              Labels: consistency, create, employee, service
>         Attachments: OFBIZ-7950_proposition.patch
>
>
> In the humanres component, we can create an employee through the form https://localhost:8443/humanres/control/NewEmployee
.
> This form has required fields that are not the same requirement than the service called
when submitting the form.
> The service called is createEmployee.
> In the service, everything is declared optional but the postalAddContactMechPurpTypeId
(which is set in the form as an hidden field).
> This means we could create an Employee without forcing a telephone number or a primary
address or even a first name.
> But then, within the service, a check is done on the firstName and lastName parameters
and if missing, an error shows up.
> We could harmonize things a little.



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

Mime
View raw message