GeoAPI
  1. GeoAPI
  2. GEO-37

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

        Hide
        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
        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
        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.
        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
        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
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

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