Opened 12 years ago
Closed 9 years ago
#368 closed defect (fixed)
expand/format parameters should not be allowed in CCRS
Reported by: | Dimitar Misev | Owned by: | Bang Pham Huu |
---|---|---|---|
Priority: | minor | Milestone: | 9.2 |
Component: | secore | Version: | 8.4 |
Keywords: | Cc: | Piero Campalani | |
Complexity: | Easy |
Description
Using expand or format parameters in the inner CRS URLs of a compound CRS URL should throw an exception.
Change History (12)
comment:1 by , 11 years ago
Milestone: | 8.5 → 9.0-alpha1 |
---|
comment:2 by , 11 years ago
Milestone: | 9.0-beta1 → 9.0 |
---|
comment:3 by , 9 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:4 by , 9 years ago
Milestone: | 9.0.x → 9.2 |
---|
comment:5 by , 9 years ago
I think there is a exclusive problem if "expand" parameter is not allowed in compound CRS as you also want to add "expand=full" to get the full resolved CRS in here http://rasdaman.org/ticket/1106.
follow-up: 7 comment:6 by , 9 years ago
But these are not expanded by default anyway, check http://localhost:8080/def/crs-compound?1=http://localhost:8080/def/crs/EPSG/0/4326&2=http://localhost:8080/def/crs/EPSG/0/4327 for example.
If I add an expand parameter to some single CRS, e.g. http://localhost:8080/def/crs-compound?1=http://localhost:8080/def/crs/EPSG/0/4326?expand=full&2=http://localhost:8080/def/crs/EPSG/0/4327 it has no effect and is simply put in the componentReferenceSystem:
<?xml version="1.0" encoding="ISO-8859-1"?> <CompoundCRS xmlns:gml='http://www.opengis.net/gml/3.2' xmlns:epsg='urn:x-ogp:spec:schema-xsd:EPSG:0.1:dataset' xmlns:xlink='http://www.w3.org/1999/xlink' gml:id='crs' xmlns:gco="http://www.isotc211.org/2005/gco" xmlns:gmd="http://www.isotc211.org/2005/gmd"> <metaDataProperty> <epsg:CommonMetaData> <epsg:type>compound</epsg:type> </epsg:CommonMetaData> </metaDataProperty> <scope>not known</scope> <identifier codeSpace='OGC'>http://localhost:8080/def/crs-compound?1=http://localhost:8080/def/crs/EPSG/0/4326?expand=full%262=http://localhost:8080/def/crs/EPSG/0/4327</identifier> <name>geographic3D to geographic2D / WGS 84 (geographic 3D)</name> <componentReferenceSystem xlink:href='http://localhost:8080/def/crs/EPSG/0/4326?expand=full'/> <componentReferenceSystem xlink:href='http://localhost:8080/def/crs/EPSG/0/4327/'/> </CompoundCRS>
Please check if this is a valid definition btw.
follow-up: 8 comment:7 by , 9 years ago
Replying to dmisev:
Please check if this is a valid definition btw.
This is a low priority ticket and actually I'm not sure if it's necessary at all. But the validity of the compound CRS definition should be checked.
comment:8 by , 9 years ago
Replying to dmisev:
Replying to dmisev:
Please check if this is a valid definition btw.
This is a low priority ticket and actually I'm not sure if it's necessary at all. But the validity of the compound CRS definition should be checked.
high to me and I think one can not use Rasdaman but when SECORE was deployed publicly then it is another problem and I'm interested in to check valid.
comment:9 by , 9 years ago
I mean it's low prio regarding the original intention of the ticket, it's not so critical if no exception is thrown. The priority is suitably minor, there are more important bugs.
Using expand or format parameters in the inner CRS URLs of a compound CRS URL should throw an exception.
comment:10 by , 9 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Patch was accepted, I will close this ticket.
comment:11 by , 9 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
comment:12 by , 9 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Milestone 9.0-beta1 deleted