Opened 12 years ago

Closed 8 years ago

#150 closed enhancement (fixed)

SECORE: disallow and flag nested composition

Reported by: Peter Baumann Owned by: Bang Pham Huu
Priority: minor Milestone: Future
Component: secore Version: 8.3
Keywords: Cc: Dimitar Misev
Complexity: Medium

Description

as per OGC Abstract Topic 2, compound CRSs shall not contain compounds again - hence, no recursion in CRS URL composition.

SECORE should have a check on this in the resolver, it should return a suitable exception.

Change History (9)

comment:1 by Peter Baumann, 11 years ago

Milestone: Future

comment:2 by Peter Baumann, 11 years ago

Version: 8.3

comment:3 by Peter Baumann, 11 years ago

Component: applicationsSECORE
Owner: changed from Dimitar Misev to mrusu

adjusted component.

comment:4 by Peter Baumann, 11 years ago

Owner: changed from mrusu to Dimitar Misev
Status: newassigned

oops, seems I inadvertently changed the owner. Undoing that.

comment:5 by Bang Pham Huu, 8 years ago

Complexity: Very Hard
Owner: changed from Dimitar Misev to Bang Pham Huu

As Dimitar has suggested before, so I will take this ticket and try to understand SECORE.

comment:6 by Bang Pham Huu, 8 years ago

Complexity: Very HardMedium

comment:7 by Bang Pham Huu, 8 years ago

Hi Prof. Peter,

I'd like to ask for your clarification about "nested composition" as I and Dimiar has different perspectives:

+ Dimitar thinks an exception should be thrown in this case (nested "crs-compound" flag in URL).

http://localhost:8088/def/crs-compound?
1=http://localhost:8088/def/crs-compound?1=...

+ I think an exception should be thrown in this case (nested "crs type: GeodeticCRS, ProjectedCRS, TemporalCRS,…" as from Compound CRS definition of OGC). So 2 or more CRS types can be combined but not "duplicate CRS types" as (GeodeticCRS + GeodeticCRS) as below:

http://localhost:8088/def/crs-compound?
1=http://localhost:8088/def/crs/EPSG/0/4326&
2=http://localhost:8088/def/crs/EPSG/0/4326

Please help us with your consideration.

comment:8 by Dimitar Misev, 8 years ago

I think it's clear from this in the description:

compound CRSs shall not contain compounds again - hence, no recursion in CRS URL composition

+ I think an exception should be thrown in this case (nested "crs type: GeodeticCRS, ProjectedCRS, TemporalCRS,…" as from Compound CRS definition of OGC). So 2 or more CRS types can be combined but not "duplicate CRS types" as (GeodeticCRS + GeodeticCRS) as below:

I think we discussed this in another ticket as well. It's an unnecessary restriction, and I'm not sure what's the reason for it. This ticket is not suitable for this however, we should open a new ticket.

comment:9 by Dimitar Misev, 8 years ago

Resolution: fixed
Status: assignedclosed

This ticket can be closed actually, as secore now throws an exception when a compound CRS contains further compound CRS.

Note: See TracTickets for help on using tickets.