Merge pull request #52 from geoscript/GEOSCRIPT-PY-GT22

Upgrade to GeoTools 22

Author: jericks

.gitignore

@@ -14,3 +14,7 @@ env
 tests/dbexts.ini
 scripts
 site
+.idea/
+*.iml
+build/
+*.jython_cache

bin/geoscript

@@ -0,0 +1,16 @@
+#!/bin/bash
+
+JYTHON=""
+if [ "$JYTHON_HOME_DIR" != "" ] && [ -e $JYTHON_HOME_DIR/jython ]; then
+  JYTHON=$JYTHON_HOME/bin/jython
+else
+  which jython > /dev/null
+  if [ "$?" == "1" ]; then
+    echo "JYTHON_HOME_DIR not defined and jython not on the current PATH"
+    exit -1
+  fi
+  JYTHON=`which jython`
+fi
+
+export CLASSPATH=`geoscript-py-classpath`
+$JYTHON

bin/geoscript-py

@@ -0,0 +1,32 @@
+@ECHO OFF
+
+REM SETLOCAL ENABLEDELAYEDEXPANSION
+
+REM ensure JYTHON_HOME_DIR set
+IF NOT DEFINED JYTHON_HOME_DIR (
+  GOTO NO_JYTHON_HOME
+)
+
+REM ensure jython.bat can be found
+SET JYTHON_EXE="%JYTHON_HOME_DIR%\jython.bat"
+IF NOT EXIST %JYTHON_EXE% (
+  GOTO NO_JYTHON
+)
+
+REM run geoscript-classpath script
+FOR /f "tokens=*" %%X IN ('%JYTHON_EXE% %~dp0geoscript-py-classpath') DO (
+    SET CLASSPATH=%%X
+)
+
+%JYTHON_EXE%
+SET JYTHON_EXE=
+
+:NO_JYTHON_HOME
+ECHO Error, JYTHON_HOME_DIR is unset. Please set JYTHON_HOME_DIR to root of Jython installation.
+GOTO DONE
+
+:NO_JYTHON
+ECHO Error, JYTHON_HOME_DIR was set but could not find 'jython' exe.
+GOTO DONE
+
+:DONE

bin/geoscript-classpath bin/geoscript-py-classpathbin/geoscript-classpath renamed to bin/geoscript-py-classpath

@@ -8,19 +8,19 @@ This section describes how to set up a GeoScript development environment.
 Install Jython
 --------------
 
-Jython version 2.5.1 or greater is required for GeoScript. 
+Jython version 2.7.2 or greater is required for GeoScript.
 
 * Install `Jython <http://www.jython.org/downloads.html>`_
 
-  For the remainder of the document the Jython installation root will be referred to as ``JYTHON_HOME``.
+  For the remainder of the document the Jython installation root will be referred to as ``JYTHON_HOME_DIR``.
 
 Install setuptools 
 ------------------
 
 #. Downlaod `ez_setup.py <http://peak.telecommunity.com/dist/ez_setup.py>`_
 #. Execute :file:`ez_setup.py` with the :command:`jython` command::
 
-     $ <JYTHON_HOME>/bin/jython ez_setup.py
+     $ <JYTHON_HOME_DIR>/bin/jython ez_setup.py
 
   Upon success :file:`<JYTYON_HOME>/bin/easy_install` will be created.
 
@@ -33,11 +33,11 @@ Install virtualenv
 
 #. Using :file:`easy_install` installed in the previous section easy install the virtualenv library::
 
-    $ <JYTHON_HOME>/bin/easy_install virtualenv
+    $ <JYTHON_HOME_DIR>/bin/easy_install virtualenv
 
 #. Create a new virtualenv named ``geoscript``::
 
-    $ <JYTHON_HOME>/bin/virtualenv geoscript
+    $ <JYTHON_HOME_DIR>/bin/virtualenv geoscript
 
 #. Activate the ``geoscript`` virtualenv::
 
@@ -102,7 +102,7 @@ GeoScript also requires a few Python dependencies.
 
 * Easy install py-dom-xpath::
 
-     (geoscript)$ easy_install py-dom-xpath
+     (geoscript)$ easy_install py-dom-xpath-redux
 
 Install nose
 ------------

bin/geoscript-py.bat

@@ -2,7 +2,6 @@
 from java.lang import System
 
 try:
-   import org.geotools
    from org.geotools.util.factory import Hints
 
    # by default the geotools referenceing Systemtem assumes yx or lat/lon 

bin/geoscript.bat

@@ -31,7 +31,7 @@ def __init__(self, west=None, south=None, east=None, north=None, prj=None, env=N
         ReferencedEnvelope.__init__(self)
 
   def getwest(self):
-    return self.minX()
+    return self.getMinX()
   west = property(getwest,None,None,'The leftmost/westmost oordinate of the bounds.')
 
 
@@ -41,7 +41,7 @@ def get_l(self):
   l = property(get_l, None, None, "Use west.")
 
   def getsouth(self):
-    return self.minY()
+    return self.getMinY()
   south = property(getsouth,None,None,'The bottomtmost/southmost oordinate of the bounds.')
 
   @deprecated
@@ -50,7 +50,7 @@ def get_b(self):
   b = property(get_b, None, None, "Use south.")
 
   def geteast(self):
-    return self.maxX()
+    return self.getMaxX()
   east = property(geteast,None,None,'The rightmost/eastmost oordinate of the bounds.')
 
   @deprecated
@@ -59,7 +59,7 @@ def get_r(self):
   r = property(get_r, None, None, 'Use east.')
 
   def getnorth(self):
-    return self.maxY()
+    return self.getMaxY()
   north = property(getnorth,None,None,'The topmost/northmost oordinate of the bounds.')
 
   @deprecated

doc/devel.rst

@@ -14,7 +14,7 @@
 from org.geotools.data import Query, Transaction
 from org.geotools.factory import CommonFactoryFinder
 from org.geotools.util.factory import Hints
-from org.geotools.feature import FeatureCollection, FeatureCollections
+from org.geotools.feature import FeatureCollection, DefaultFeatureCollection
 from org.opengis.filter.sort import SortOrder
 
 _filterFactory = CommonFactoryFinder.getFilterFactory(None)
@@ -162,7 +162,7 @@ def bounds(self, filter=None):
       fc = self._source.getFeatures(q)
       e = fc.getBounds()
     if e:
-      if e.crs():
+      if e.getCoordinateReferenceSystem():
         return geom.Bounds(env=e)
       else:
         return geom.Bounds(env=e, prj=self.proj)
@@ -367,7 +367,7 @@ def add(self, o):
     elif isinstance(o, (dict,list)):
       f = self.schema.feature(o)
 
-    fc = FeatureCollections.newCollection() 
+    fc = DefaultFeatureCollection()
     fc.add(f._feature)
     self._source.addFeatures(fc)