openminted / omtd-registry-api

Other
0 stars 0 forks source link

Unable to set several fields of PersonInfo #5

Closed reckart closed 7 years ago

reckart commented 7 years ago

PersonInfo has only getters but no setters for the following fields:

@pennyl67 might be interesting for you as well.

courado commented 7 years ago

From the docs :

  /**
   * Gets the value of the names property.
   * 
   * <p>
   * This accessor method returns a reference to the live list,
   * not a snapshot. Therefore any modification you make to the
   * returned list will be present inside the JAXB object.
   * This is why there is not a <CODE>set</CODE> method for the names property.
   * 
   * <p>
   * For example, to add a new item, do as follows:
   * <pre>
   *    getNames().add(newItem);
   * </pre>
   * 
   * 
   * <p>
   * Objects of the following type(s) are allowed in the list
   * {@link Names }
   * 
   * 
   */

Click here to see the jenkins build

reckart commented 7 years ago

Uh ok... that seems as bit ugly and not matching the Java bean conventions... but ok.

reckart commented 7 years ago

Why does IdentificationInfo have set methods for list-type properties then? E.g.

    public void setResourceShortNames(List<ResourceShortName> resourceShortNames) {
        this.resourceShortNames = resourceShortNames;
    }