BasixXYNode implementaion does not correspond to documentation
--------------------------------------------------------------

                 Key: GEOT-1446
                 URL: http://jira.codehaus.org/browse/GEOT-1446
             Project: GeoTools
          Issue Type: Bug
    Affects Versions: 2.3.4
         Environment: Linux 2.6
Java 1.6
            Reporter: Jan Torben Heuer


BasicXYNode#getObject and setObject should according to the Interface Graphable 
store the represented entity.

But the get/set Coordinate overwrites the Object:

  /**
   * @see XYNode#getCoordinate()
   */
  public Coordinate getCoordinate() {
    return((Coordinate)getObject());
  }

  /**
   * @see XYNode#setCoordinate(Coordinate)
   */
  public void setCoordinate(Coordinate c) {
    setObject(c);
  }

This is neither documented nor does the BasicXYNode fullfill the sematic of 
Graphable anymore. Please save the Coordinate either in a different variable.

In this form, the BasicXYNode is unusable, because a backreference to its 
entity is deleted.

Thanks,

Jan

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to