GeoTools

Delete unneeded classes from the legacy module

Details

  • Type: Task Task
  • Status: Closed Closed
  • Priority: Trivial Trivial
  • Resolution: Fixed
  • Affects Version/s: 2.2.M0, 2.3.0, 2.4-M0
  • Fix Version/s: None
  • Component/s: referencing
  • Labels:
    None

Description

The CRS framework is fully ported to GeoAPI interfaces, and we should start to delete as much classes from the legacy module as we can without breaking the migrate module. A quick search suggest that we can safely delete the following classes:

  • CoordinateSystemOracleEPSGFactory
  • CoordinateSystemModifiedEPSGFactory
  • CoordinateSystemEPSGFactory
  • CoordinateSystemAUTOFactory
  • NoSuchAuthorityCodeException (the one in legacy module, not the geoapi's one).

More classes to be deleted later.

Issue Links

depends upon

Task - A task that needs to be done. GEOT-26 Refactor all classes for implementing the new interfaces from GeoAPI

  • Critical - Crashes, loss of data, severe memory leak.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Task - A task that needs to be done. GEOT-413 Port GridCoverage to the new CRS API and implement GeoAPI interfaces.

  • Critical - Crashes, loss of data, severe memory leak.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Task - A task that needs to be done. GEOT-334 Update geotools code to use new geoapi referencing implementation

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
relates to

Task - A task that needs to be done. GEOT-491 demos module depends on legacy

  • Blocker - Blocks development and/or testing work, production could not run
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Martin Desruisseaux added a comment -
Deleted the following files as of revision 14661:

gt\module\legacy\src\org\geotools\cs\CoordinateSystemAUTOFactory.java
gt\module\legacy\src\org\geotools\cs\CoordinateSystemEPSGFactory.java
gt\module\legacy\src\org\geotools\cs\CoordinateSystemModifiedEPSGFactory.java
gt\module\legacy\src\org\geotools\cs\CoordinateSystemOracleEPSGFactory.java
gt\module\legacy\src\org\geotools\cs\GeodeticCalculator.java
gt\module\legacy\src\org\geotools\gc\LocalizationGrid.java
gt\module\legacy\src\org\geotools\gc\LocalizationGridTransform2D.java
Show
Martin Desruisseaux added a comment - Deleted the following files as of revision 14661: gt\module\legacy\src\org\geotools\cs\CoordinateSystemAUTOFactory.java gt\module\legacy\src\org\geotools\cs\CoordinateSystemEPSGFactory.java gt\module\legacy\src\org\geotools\cs\CoordinateSystemModifiedEPSGFactory.java gt\module\legacy\src\org\geotools\cs\CoordinateSystemOracleEPSGFactory.java gt\module\legacy\src\org\geotools\cs\GeodeticCalculator.java gt\module\legacy\src\org\geotools\gc\LocalizationGrid.java gt\module\legacy\src\org\geotools\gc\LocalizationGridTransform2D.java
Hide
Permalink
Martin Desruisseaux added a comment -
The "legacy" and "migrates" module have been removed from trunk and 2.2 branch. A branch were created there, in order to help to manage the port to the GO-1 renderer:

    http://svn.geotools.org/geotools/branches/legacy/

Classes removal will continue on this branch as a way to keep trace of what has been ported, and what is remaining.
Show
Martin Desruisseaux added a comment - The "legacy" and "migrates" module have been removed from trunk and 2.2 branch. A branch were created there, in order to help to manage the port to the GO-1 renderer:      http://svn.geotools.org/geotools/branches/legacy/ Classes removal will continue on this branch as a way to keep trace of what has been ported, and what is remaining.
Hide
Permalink
Martin Desruisseaux added a comment -
Changed fix version from "2.4" to "unknown" since this is a long term project to be finish at an unknown date (depending available resources) and will cross many GeoTools versions.
Show
Martin Desruisseaux added a comment - Changed fix version from "2.4" to "unknown" since this is a long term project to be finish at an unknown date (depending available resources) and will cross many GeoTools versions.
Hide
Permalink
Andrea Aime added a comment -
Legacy modules have been gone for a while
Show
Andrea Aime added a comment - Legacy modules have been gone for a while

People

  • Assignee:
    Unassigned
    Reporter:
    Martin Desruisseaux
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: