GeoAPI

createDirectPosition is in wrong factory interface

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.0, 2.0
  • Fix Version/s: 2.1-M7
  • Component/s: geometry
  • Labels:
    None
  • Number of attachments :
    0

Description

Currently, the createDirectPosition method is in the PrimitiveFactory interface. DirectPosition is not a Primitive, so this method should be in GeometryFactory.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Martin Desruisseaux added a comment -

Copied the method from PrimitiveFactory to GeometryFactory. Deprecated the old method. The deprecated method will need to be removed later.

Show
Martin Desruisseaux added a comment - Copied the method from PrimitiveFactory to GeometryFactory. Deprecated the old method. The deprecated method will need to be removed later.
Hide
Permalink
Martin Desruisseaux added a comment -

Removed deprecated methods on trunk as part of post-GeoAPI 2.0 release.

Show
Martin Desruisseaux added a comment - Removed deprecated methods on trunk as part of post-GeoAPI 2.0 release.
Hide
Permalink
Jody Garnett added a comment -

DirectPosition is not a geometry - it should have its own Factory, possibly with List<DirectPosition> as well.

Show
Jody Garnett added a comment - DirectPosition is not a geometry - it should have its own Factory, possibly with List<DirectPosition> as well.
Hide
Permalink
Jody Garnett added a comment -

Sample has been provided on trunk:

Show
Jody Garnett added a comment - Sample has been provided on trunk: http://geoapi.svn.sourceforge.net/viewvc/*checkout*/geoapi/trunk/geoapi/src/main/java/org/opengis/spatialschema/geometry/PositionFactory.java
Hide
Permalink
Jody Garnett added a comment -

PositionFactory now holds this method.

Show
Jody Garnett added a comment - PositionFactory now holds this method.

People

  • Assignee:
    Martin Desruisseaux
    Reporter:
    Christopher Dillard
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:

Time Tracking

Estimated:
2m
Original Estimate - 2 minutes Original Estimate - 2 minutes
Remaining:
Not Specified
Remaining Estimate - Not Specified
Logged:
5m
Time Spent - 5 minutes