GeoTools

Allow Geometries to lookup by subclass

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 2.6.1
  • Component/s: main
  • Labels:
    None

Description

In my application the classes I send into Geometries.getForBinding() are actually subclasses of jts geometries so the current lookup fails.

  1. Text File
    GEOT-2869.patch
    30/Nov/09 11:20 AM
    3 kB
    Justin Deoliveira
  2. Text File
    GEOT-2869-michael.patch
    30/Nov/09 6:24 PM
    10 kB
    Michael Bedward

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Justin Deoliveira added a comment -
Hi Michael, here is a patch for this one which changes the exact match by class to a "looser" match allowing for subclasses to be passed in. Let me know if you have any issue with it.
Show
Justin Deoliveira added a comment - Hi Michael, here is a patch for this one which changes the exact match by class to a "looser" match allowing for subclasses to be passed in. Let me know if you have any issue with it.
Hide
Permalink
Michael Bedward added a comment -
Hi Justin - that's much better, thanks !

I've just tweaked the sub-class matching because it wasn't working for a class derived from Geometry.
Show
Michael Bedward added a comment - Hi Justin - that's much better, thanks ! I've just tweaked the sub-class matching because it wasn't working for a class derived from Geometry.
Hide
Permalink
Justin Deoliveira added a comment -
Awesome, thanks Michael, you rock!
Show
Justin Deoliveira added a comment - Awesome, thanks Michael, you rock!
Hide
Permalink
Andrea Aime added a comment -
Mass closing all issues that have been in "resolved" state for 2 months or more without any feedback or update
Show
Andrea Aime added a comment - Mass closing all issues that have been in "resolved" state for 2 months or more without any feedback or update

People

  • Assignee:
    Justin Deoliveira
    Reporter:
    Justin Deoliveira
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: