Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature collection cleanup #161

Closed
wants to merge 6 commits into from
Closed

Conversation

jodygarnett
Copy link
Contributor

Remove references to deprecated feature collection methods, that are now removed from GeoTools.

  • Remove CollectionListener methods (addListener, removeListener)
  • Remove Iterator Methods (iterator, close, purge)
  • Remove Collection Methods (add,adAll,clear,remove,removeAll,retainAll)

For more information see geotools/geotools#44

The cleanup went smoothly for uDig ... except for TableView.

Had to migrate from FeatureCollection/CollectionListener methods (which are gone) to FeatureSource/FeatureListener methods. This may mean we need to ask TableView to latch onto a FeatureSource now

So testing is needed

@moovida moovida closed this Sep 22, 2013
@jodygarnett
Copy link
Contributor Author

Yeah we will need this when we update to a newer version of GeoTools.

Give the package name changes we will have to redo the work file by file, still it did not take too long the first time.

@eclipsewebmaster eclipsewebmaster deleted the uDigFeatureCollectionCleanup branch February 13, 2015 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants