magda-io / magda-minion-format

Magda Format Minion
Other
0 stars 0 forks source link

Should detect WMS, WFS, WCS formats correctly. #18

Closed mwu2018 closed 2 years ago

mwu2018 commented 3 years ago

Describe the bug If a dataset has url of https://portal.spatial.nsw.gov.au/server/services/Cadastre_History/MapServer/WMSServer, its format is incorrectly detected as "ESRI MAPSERVER". It should be "WMS".

Technical Note:

  1. If a dataset has url of https://portal.spatial.nsw.gov.au/server/services/Cadastre_History/MapServer, its format is correctly detected as "ESRI MAPSERVER".
  2. A dataset with url of https://portal.spatial.nsw.gov.au/server/services/Cadastre_History/MapServer/WFSServer should be detected as "WFS" format.
  3. A dataset with url of https://portal.spatial.nsw.gov.au/server/services/Cadastre_History/MapServer/WCSServer should be detected as "WCS" format.

To Reproduce

  1. Go to https://dev.nsw.digitaltwin.terria.io/#share=s-tF1QNV8Vw58HmVoZf4qRylwghYS
  2. An error will occur if clicking on the item "Cadastre History". (The format is detected as "ESRI MAPSERVER").
  3. No error will occur if clicking on the item "Cadastre History (WMS)". (The format is manually set as "WMS").

image

t83714 commented 3 years ago

@mwu2018 For a map service, there might be served in the following ways formats (also see here):

It's also for matching the logic we had in MagdaReference --- in terriajs, they needs to be handled by differently (e.g. WebMapServiceCatalogItem vs ArcGisMapServerCatalogItem)

Similar applies to WFS & ESRI FeatureServer.

Regarding ESRI WCSServer, can terriajs support it now?

mwu2018 commented 3 years ago

Version v0.0.58-terria.0 is being used for NSW DT where it correctly identifies https://portal.spatial.nsw.gov.au/server/services/Cadastre_History/MapServer as ESRI MAPSERVER because of https://github.com/magda-io/magda-minion-format/blob/v0.0.58-terria.0/synonyms.json that has

    "esri mapserver": [
        "esri rest",
        "ESRI ArcGIS",
        "map service",
        "esri map service"
    ],

v0.0.58-terria.0 incorrectly identifies https://portal.spatial.nsw.gov.au/server/services/Cadastre_History/MapServer/WMSServer as ESRI MAPSERVER but v1.0.1 will correctly identify it as WMS. ~However v1.0.1 may not identify https://portal.spatial.nsw.gov.au/server/services/Cadastre_History/MapServer as ESRI MAPSERVER as v0.0.58-terria.0 does.~

It seems that v1.0.1 will correctly detect both formats. Need to test it in NSW DT though.

t83714 commented 2 years ago

@mwu2018 I think the test has been done and we can close this ticket, right?