[Geotools-devel] [JIRA] (GEOT-5704) Color literal encoding for CQL
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5704 Color literal encoding for CQL Issue Type: Bug Affects Versions: 17.0 Assignee: Jody Garnett [Administrator] Components: cql Created: 18/Apr/17 6:46 AM Priority: Medium Reporter: Jody Garnett [Administrator] CQL and ECQL can parse color values of the form "#FF" into a literal that can be used as a Color. However if you make a literal programatically using ff.literal( Color.RED ) it will not be coverted to a valid CQL representations. This causes problems for YSLD when encoding CQL expressions that list a Color. Add Comment
[Geotools-devel] [JIRA] (GEOT-5689) Setup and build instructions for IntelliJ
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5689 Setup and build instructions for IntelliJ Issue Type: Improvement Assignee: Nuno Oliveira Components: docs Created: 31/Mar/17 11:14 AM Priority: Medium Reporter: Jody Garnett [Administrator] We have an outstanding request to update the codebase formatting (to get rid of style differences). Before we can this we need to ask IntelliJ users and Eclipse users to configure their IDE in a consist manner for code formatting. We have the following information: http://docs.geotools.org/latest/developer/conventions/code/style.html http://docs.geotools.org/latest/userguide/build/maven/eclipse.html We need a setup guide for IntelliJ, and some more instructions for code style for IntelliJ. Add Comment
[Geotools-devel] [JIRA] (GEOT-5683) MemoryDataStore preservation of FeatureIDs during addFeatures
Title: Message Title
Jody Garnett [Administrator] created an issue
GeoTools / GEOT-5683
MemoryDataStore preservation of FeatureIDs during addFeatures
Issue Type:
Bug
Affects Versions:
2.11
Assignee:
Alex Goudine
Components:
main
Created:
21/Mar/17 7:41 PM
Priority:
Medium
Reporter:
Jody Garnett [Administrator]
MemoryDataStore used to preserve feature ids, as part of migration to ContentDataStore this functionality was lost! The fix is illustrated in the following example subclass:
public class FIdMemoryDataStore extends MemoryDataStore {
public FIdMemoryDataStore(SimpleFeatureType featureType) {
super(featureType);
}
// advertise this supports setting FID for features
@Override
protected ContentFeatureSource createFeatureSource(ContentEntry entry, Query query) {
return new MemoryFeatureStore(entry, query) {
@Override
protected QueryCapabilities buildQueryCapabilities() {
return new QueryCapabilities() {
@Override
public boolean isUseProvidedFIDSupported() {
return true;
}
};
}
//need to actually use the provided FID
@Override
protected FeatureWriter getWriterInternal(
Query query, int flags) throws IOException {
return new MemoryFeatureWriter(getState(), query) {
@Override
public void write() throws IOException {
if (Boolean.TRUE.equals(current.getUserData().get(H
[Geotools-devel] [JIRA] (GEOT-5682) Double FeatureCollection handling of RefernecedEnvelope3d
Title: Message Title
Jody Garnett [Administrator] created an issue
GeoTools / GEOT-5682
Double FeatureCollection handling of RefernecedEnvelope3d
Issue Type:
Bug
Assignee:
Unassigned
Created:
21/Mar/17 7:30 PM
Priority:
Medium
Reporter:
Jody Garnett [Administrator]
ReferencedEnvelope constructors should not be called directly, because the factory method needs to choose the correct implementation to match the CoordinateReferenceSystem.
if(this.bounds == null) {
this.bounds = new ReferencedEnvelope();
Iterator i = this.contents.values().iterator();
while(i.hasNext()) {
BoundingBox geomBounds = ((SimpleFeature)i.next()).getBounds();
if(!geomBounds.isEmpty()) {
this.bounds.include(geomBounds);
}
}
}
Add Comment
[Geotools-devel] [JIRA] (GEOT-5673) Windows service memory settings incorrect
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5673 Windows service memory settings incorrect Issue Type: Bug Assignee: Unassigned Created: 13/Mar/17 9:20 PM Environment: Windows 7 Priority: Medium Reporter: Jody Garnett [Administrator] With all the changes to jetty it appears as if the memory settings for windows service are incorrect: Memory Usage: 72 MB / 123 MB This memory setting is enough to run and render the default dataset, but not what we expected. Add Comment
[Geotools-devel] [JIRA] (GEOT-5665) gt-wfs-ng cannot set feature ids
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5665 gt-wfs-ng cannot set feature ids Issue Type: Improvement Assignee: Unassigned Components: wfs-ng Created: 07/Mar/17 12:34 AM Priority: Medium Reporter: Jody Garnett [Administrator] This is a feature of ContentDataStore that could be respected; it would be good to pass the provided featureIds through during Transaction Insert; so they could be used for datastores that can preserve the provided feature ids. Add Comment
[Geotools-devel] [JIRA] (GEOT-5664) Migrate NestedAttributeExpressionImpl to PropertyAccessor
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5664 Migrate NestedAttributeExpressionImpl to PropertyAccessor Issue Type: Improvement Assignee: Unassigned Created: 06/Mar/17 8:16 PM Priority: Medium Reporter: Jody Garnett [Administrator] As noted in GEOT-4981the class NestedAttributeExpresionImpl is incompatible with the design of GeoTools; specifically the query language is fixed (because we have a fixed grammar of Filter 2.0, CQL and ECQL) defining what filters and expressions classes are available. There are two main ways to extend: Functions - adds additional processing options PropertyAccessors - adds additional property access options NestedAttributeExpressionImpl is an attempt to override the default functionality of PropertyName; however this flexibility is already available (by implementing a PropertyAccessor above). The GeoTools codebase assume that PropertyName expressions can be copied as a data structure, NestedAttributeExpressionImpl breaks this assumption (resulting in patches like GEOT-4981 Open ). We need to depend on the design of GeoTools as we will not be able to cover all the corner cases required by a custom implementation of PropertyName.
[Geotools-devel] [JIRA] (GEOT-5649) Remove AbstractDataStore
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5649 Remove AbstractDataStore Issue Type: Improvement Assignee: Jody Garnett [Administrator] Components: main Created: 17/Feb/17 12:39 AM Priority: Medium Reporter: Jody Garnett [Administrator] ContentDataStore has now taken over from AbstractDataStore. Remove AbstractDataStore (and its support classes from the codebase). AbstractAttributeIO AbstractDataStore AbstractFeatrueSource AbstractFeatrueStore AbstractFeatrueLocking TransactionStateDiff Add Comment
[Geotools-devel] [JIRA] (GEOT-5638) Style Editor Preview Legend assume SLD
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5638 Style Editor Preview Legend assume SLD Issue Type: Improvement Assignee: Alex Goudine Components: Configuration Created: 09/Feb/17 12:30 AM Priority: Medium Reporter: Jody Garnett [Administrator] Steve Ikeoka reported to user list: Using the Preview Legend feature on YSLD styles causes an XML parsing error "Content is not allowed in prolog" since org.geoserver.wms.web.data.StyleAdminPanel currently only handles SLD styles: StyleInfo si = new StyleInfoImpl(stylePage.getCatalog()); String styleName = "tmp" + UUID.randomUUID().toString(); String styleFileName = styleName + ".sld"; si.setFilename(styleFileName); si.setName(styleName); si.setWorkspace(stylePage.styleModel.getObject().getWorkspace()); Also, if a proxy base URL is set that is missing a slash at the end (e.g., "http://geoserver.org/geoserver" instead of "http://geoserver.org/geoserver/"), then the Preview Legend would fail for external legend images because it would build a URL like "http://geoserver.org/geoserverstyles" instead of "http://geoserver.org/geoserver/styles".
[Geotools-devel] [JIRA] (GEOT-5628) SLD Parser Label parsing trips up over CDATA handling with whitespace
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5628 SLD Parser Label parsing trips up over CDATA handling with whitespace Issue Type: Improvement Affects Versions: 16.1 Assignee: Unassigned Components: xml Created: 01/Feb/17 11:10 PM Priority: Medium Reporter: Jody Garnett [Administrator] Follow on from GEOS-7375 Open debugging: foo <[CDATA[ ]]>bar Produces the following _expression_: strConcat(strConcat(strConcat(foo,' '),'\n'),bar) We expected: strConcat(strConcat(foo,'\n'),bar) Whitespace next to cdata white space needs to be skipped.
[Geotools-devel] [JIRA] (GEOT-5625) Add config level for log4jlogger
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5625 Add config level for log4jlogger Issue Type: Improvement Assignee: Jody Garnett [Administrator] Created: 31/Jan/17 4:21 PM Priority: Medium Reporter: Jody Garnett [Administrator] This will allow config to be used, it ends up one step below INFO level Add Comment This message was sent by Atlassian JIRA (v1000.718.4#100026-sha1:69a2752)
[Geotools-devel] [JIRA] (GEOT-5579) DataUtilities.urlToFile fails with query parameters (blocking SVG parameter use)
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5579 DataUtilities.urlToFile fails with query parameters (blocking SVG parameter use) Issue Type: Improvement Assignee: Unassigned Components: main Created: 18/Nov/16 6:32 PM Priority: Medium Reporter: Jody Garnett [Administrator] The DataUtilities.urlToFile method was not written with query parameters in mind, this is causing problems with the new SVG with parameter references supported by StyleParser. Add Comment
[Geotools-devel] [JIRA] (GEOT-5533) ServiceException constructor for initCause
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5533 ServiceException constructor for initCause Issue Type: New Feature Assignee: Unassigned Created: 01/Oct/16 1:43 AM Environment: The use of throw new ServiceException( "message").initCause( e ) by ows client code during wms getcapabilities parsing is in effective. The superclass SaxException( message, e ) must be called instead for the exception to be available. Priority: Medium Reporter: Jody Garnett [Administrator] Add Comment This message was sent by Atlassian JIRA (v1000.383.2#100014-sha1:783d425)
[Geotools-devel] [JIRA] (GEOT-5358) repo.boundlessgeo.com not updating pom.xml snapshot list correctly
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5358 repo.boundlessgeo.com not updating pom.xml snapshot list correctly Issue Type: Bug Assignee: Unassigned Components: admin Created: 09/Feb/16 10:24 PM Priority: Medium Reporter: Jody Garnett [Administrator] As reported on mailing list: Hi, all builds of the 2.8.x series built with geootols jars off the repository (as opposed to freshly built locally) are failing in the WMS module with several failures. This for example breaks all Travis checks on 2.8.x (and of course the build of anyone that is not accustomed to also build GeoTools locally). Looking into the issue I've found that the metadata.xml in the repository is reporting the wrong snapshot version, see here: https://repo.boundlessgeo.com/main/org/geotools/gt-render/14-SNAPSHOT/maven-metadata.xml Here is the current content for reference in case it gets fixed later: "1.0" encoding="UTF-8"?> org.geotools gt-render 14-SNAPSHOT 20151006.130203 251 20160129113257 sources jar 14-20151006.130203-251 20151006130203 jar 14-20151006.130203-251 20151006130203 tests jar 14-20151006.130203-251 20151006130203 pom 14-20151006.130203-251 20151006130203
[Geotools-devel] [JIRA] (GEOT-5326) PostGISDialect is using ST_Simplify instead of ST_SimplifyPreserveTopology
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5326 PostGISDialect is using ST_Simplify instead of ST_SimplifyPreserveTopology Issue Type: Bug Affects Versions: 14.1 Assignee: Unassigned Components: jdbc-postgis plugin Created: 17/Dec/15 12:47 AM Priority: Medium Reporter: Jody Garnett [Administrator] Originally reported as GEOT-532: The class org.geotools.data.postgis.PostGISDialect uses ST_Simplify() instead of ST_SimplifyPreserveTopology() - see lines 333 and 344. ST_Simplify will often create extremely small polygons for polygons that have a portion that is quite thin (when the result should be a large polygon). ST_SimplifyPreserveTopology() is much better at avoiding this (however, its a bit slower). Add Comment
[Geotools-devel] [JIRA] (GEOT-5323) Remove load library calls preventing java.lang.UnsatisfiedLinkError
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5323 Remove load library calls preventing java.lang.UnsatisfiedLinkError Issue Type: Bug Assignee: Unassigned Components: ogr Created: 15/Dec/15 10:33 PM Priority: Medium Reporter: Jody Garnett [Administrator] The following pull request has stalled out: https://github.com/geotools/geotools/pull/1055 The general issue seems to be: java.lang.UnsatisfiedLinkError: library already loaded in classloader"). Add Comment
[Geotools-devel] [JIRA] (GEOT-5190) General Oblique Transform for Rotated Pole Coordinates
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5190 General Oblique Transform for Rotated Pole Coordinates Issue Type: Bug Assignee: Jody Garnett [Administrator] Components: referencing Created: 10/Aug/15 11:18 PM Fix Versions: 14-beta Priority: Medium Reporter: Jody Garnett [Administrator] As per pull request #870: Add new transformation used quite often by Numerical Weather Forecasting community to support "Rotated Pole" coordinates. That transformation is also available in proj.4 as "+proj=ob_tran". For initial idea of this pull request, see: http://sourceforge.net/p/geoserver/mailman/message/34095134/
[Geotools-devel] [JIRA] (GEOT-5183) Text Symbolizer Multi font support
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5183 Text Symbolizer Multi font support Issue Type: Bug Affects Versions: 14-M1 Assignee: Unassigned Components: api Created: 07/Aug/15 12:29 AM Priority: Medium Reporter: Jody Garnett [Administrator] As per change proposal I would like to add TextSymbolizer.fonts() for direct access to the font array previously required for SLD 1.0 functionality. This change is consistent with our style api which has previously moved over to use direct collection access (for a reduction in the number of methods used). Add Comment
[Geotools-devel] [JIRA] (GEOT-5155) LGPL implementation of GridCoverage2DRIA.computeRect method
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5155 LGPL implementation of GridCoverage2DRIA.computeRect method Issue Type: Bug Affects Versions: 14-M0 Assignee: Unassigned Components: coverage Created: 25/Jun/15 1:29 AM Priority: Highest Reporter: Jody Garnett [Administrator] From IP review of 14-M0: GridCoverage2DRIA.java - Indicates it was "Copied and adapted from WarpGeneralOpImage". Can you help us understand the origin of the original content and applicable license? Email list: Look for WarpGeneralOpImage shows up as from jai_core - so the code would be java research license and not open source right? Emanuele do you have any details on this one? You may need to rewrite your own copy of computeRect(PlanarImage[] sources, WritableRaster dest, Rectangle destRect). Perhaps you could put it in a utility class if it is gene
[Geotools-devel] [JIRA] (GEOT-5129) Make Transaction AutoClosable
Title: Message Title
Jody Garnett [Administrator] created an issue
GeoTools / GEOT-5129
Make Transaction AutoClosable
Issue Type:
Bug
Assignee:
Unassigned
Components:
main
Created:
25/May/15 9:41 PM
Priority:
Medium
Reporter:
Jody Garnett [Administrator]
Changing to:
interface Transaction extrends Closable {
...
void close() throws IOException;
}
Does not result in an API change as our close() method already throws an IOException.
This change allows the use of transaction in a try-with-resources block:
try( Transaction t = new DefaultTransaction() ){
store.setTransaction(t);
...
}
This change makes GeoTools easier to use, and allows us to quickly fix test cases such as wfs-ng GeoServerIntegrationTests.
[Geotools-devel] [JIRA] (GEOT-5128) replace gt-wfs module with gt-wfs-ng
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5128 replace gt-wfs module with gt-wfs-ng Issue Type: Bug Affects Versions: 14-beta Assignee: Jody Garnett [Administrator] Components: WFS, wfs plugin Created: 25/May/15 7:34 PM Priority: Medium Reporter: Jody Garnett [Administrator] Remove the gt-wfs module based on AbstractDataStore now that gt-wfs-ng is available as a "drop-in" replacement. Add Comment
[Geotools-devel] [JIRA] (GEOT-5116) upgrade dependencies for 14-M0 release
Title: Message Title Jody Garnett [Administrator] created an issue GeoTools / GEOT-5116 upgrade dependencies for 14-M0 release Issue Type: Improvement Assignee: Unassigned Created: 13/May/15 8:55 PM Priority: Medium Reporter: Jody Garnett [Administrator] We are cutting a 14-M0 milestone release in response to LocationTech IP review process: The following dependencies have been short-listed: vecmath - GEOT-22 hsql 2.2.8 --> 2.3.0 Add Comment
