Ulrich Loup ( 
https://osgeo-org.atlassian.net/secure/ViewProfile.jspa?accountId=60101270c8c36c00695f6343
 ) *created* an issue

GeoServer ( 
https://osgeo-org.atlassian.net/browse/GEOS?atlOrigin=eyJpIjoiY2I0NzhlZmIwMmRlNGE0YzhiYjc5YTZkMzVhOTg4NjYiLCJwIjoiaiJ9
 ) / Improvement ( 
https://osgeo-org.atlassian.net/browse/GEOS-9876?atlOrigin=eyJpIjoiY2I0NzhlZmIwMmRlNGE0YzhiYjc5YTZkMzVhOTg4NjYiLCJwIjoiaiJ9
 ) GEOS-9876 ( 
https://osgeo-org.atlassian.net/browse/GEOS-9876?atlOrigin=eyJpIjoiY2I0NzhlZmIwMmRlNGE0YzhiYjc5YTZkMzVhOTg4NjYiLCJwIjoiaiJ9
 ) Custom coverage dimension has no effect ( 
https://osgeo-org.atlassian.net/browse/GEOS-9876?atlOrigin=eyJpIjoiY2I0NzhlZmIwMmRlNGE0YzhiYjc5YTZkMzVhOTg4NjYiLCJwIjoiaiJ9
 )

Issue Type: Improvement Affects Versions: 2.18.1, 2.18.2 Assignee: Unassigned 
Created: 26/Jan/21 3:12 PM Priority: Medium Reporter: Ulrich Loup ( 
https://osgeo-org.atlassian.net/secure/ViewProfile.jspa?accountId=60101270c8c36c00695f6343
 )

The class

org.geoserver.catalog.CoverageDimensionCustomizerReader

is a wrapper for GridCoverage2DReader to use custom GridSampleDimension , e.g., 
for custom band names as the attribute description. These custom dimensions can 
be specified as parameters to the read method of GridCoverage2DReader.

However, any custom description is also checked against the predefined 
CoverageDimensionInfo name and ignored in case the name and the custom 
description do not match (see
CoverageDimensionCustomizerReader.java#L586 ( 
https://github.com/geoserver/geoserver/blob/2.18.x/src/main/src/main/java/org/geoserver/catalog/CoverageDimensionCustomizerReader.java#L586
 ) )

To enable customized GridSampleDimension descriptions to take effect I propose 
the patch (Diff patch):

586,587c586
<                     (sampleDimDescription == null
<                                     || 
!sampleDimDescription.toString().equalsIgnoreCase(name))
---
>                     sampleDimDescription == null

( 
https://osgeo-org.atlassian.net/browse/GEOS-9876#add-comment?atlOrigin=eyJpIjoiY2I0NzhlZmIwMmRlNGE0YzhiYjc5YTZkMzVhOTg4NjYiLCJwIjoiaiJ9
 ) Add Comment ( 
https://osgeo-org.atlassian.net/browse/GEOS-9876#add-comment?atlOrigin=eyJpIjoiY2I0NzhlZmIwMmRlNGE0YzhiYjc5YTZkMzVhOTg4NjYiLCJwIjoiaiJ9
 )

Get Jira notifications on your phone! Download the Jira Cloud app for Android ( 
https://play.google.com/store/apps/details?id=com.atlassian.android.jira.core&referrer=utm_source%3DNotificationLink%26utm_medium%3DEmail
 ) or iOS ( 
https://itunes.apple.com/app/apple-store/id1006972087?pt=696495&ct=EmailNotificationLink&mt=8
 ) This message was sent by Atlassian Jira (v1001.0.0-SNAPSHOT#100154- 
sha1:4dba4bb )
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to