GeoTools
  1. GeoTools
  2. GEOT-2869

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. GEOT-2869.patch
        3 kB
        Justin Deoliveira
      2. GEOT-2869-michael.patch
        10 kB
        Michael Bedward

        Activity

        Hide
        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
        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
        Justin Deoliveira added a comment -
        Awesome, thanks Michael, you rock!
        Show
        Justin Deoliveira added a comment - Awesome, thanks Michael, you rock!
        Hide
        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
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: