GeoTools
  1. GeoTools
  2. GEOT-4121

DWithin filter does not work from GeoServer when passing units parameter

    Details

    • Type: Bug Bug
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.7.5, 8.0-RC1
    • Component/s: jdbc-oracle plugin
    • Labels:
      None
    • Environment:
      Oracle 11.2.0.1
    • Testcase included:
      yes

      Description

      For a description of the problem, please see http://jira.codehaus.org/browse/GEOS-4384. Creating it here as the fix is purely GeoTools. Patch includes fix for most common distance units and a test case.

      1. GEOT-4021-moreunits.patch
        12 kB
        Andrea Aime
      2. GEOT-4021-moreunits.patch
        6 kB
        Andrea Aime
      3. GEOT-4121.patch
        5 kB
        Juan Marin Otero

        Issue Links

          Activity

          Hide
          Andrea Aime added a comment -
          Here is a revised version, more units, more tests. For some reason I don't understand the test for millimeters does not work, but all the others do (against my Oracle XE)
          Show
          Andrea Aime added a comment - Here is a revised version, more units, more tests. For some reason I don't understand the test for millimeters does not work, but all the others do (against my Oracle XE)
          Hide
          Andrea Aime added a comment -
          And here is a variation of the patch that makes dwithin filters work also for postgis geography columns. Adding support for geometry is trickier as the whole thing would have to be reprojected.
          Show
          Andrea Aime added a comment - And here is a variation of the patch that makes dwithin filters work also for postgis geography columns. Adding support for geometry is trickier as the whole thing would have to be reprojected.
          Hide
          Juan Marin Otero added a comment -
          Andrea, thanks for reviewing this. Your extended patch looks good, and you are going to love this. On my Oracle setup (11.2.0.1, Enterprise Edition) the test with mm works just fine, I just uncommented that line and run it, no issues. There are quite a bit of differences between the XE version and the current enterprise one, but it comes to a surprise that this might be one of them.
          Show
          Juan Marin Otero added a comment - Andrea, thanks for reviewing this. Your extended patch looks good, and you are going to love this. On my Oracle setup (11.2.0.1, Enterprise Edition) the test with mm works just fine, I just uncommented that line and run it, no issues. There are quite a bit of differences between the XE version and the current enterprise one, but it comes to a surprise that this might be one of them.
          Hide
          Andrea Aime added a comment -
          Ah Oracle, always being the difficult one to deal with... All right, will commit the patch as is.
          Show
          Andrea Aime added a comment - Ah Oracle, always being the difficult one to deal with... All right, will commit the patch as is.
          Hide
          Andrea Aime added a comment -
          Patch applied on 2.7.x and trunk
          Show
          Andrea Aime added a comment - Patch applied on 2.7.x and trunk

            People

            • Assignee:
              Andrea Aime
              Reporter:
              Juan Marin Otero
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: