Issue 12558

Dataset page: remove georeferenced occurrences map for checklists

12558
Reporter: kbraak
Assignee: kbraak
Type: Bug
Summary: Dataset page: remove georeferenced occurrences map for checklists
Priority: Minor
Resolution: Fixed
Status: Closed
Created: 2012-12-20 11:42:51.955
Updated: 2013-08-29 14:44:53.118
Resolved: 2012-12-21 15:01:59.283
        
Description: Looking at GBIF Backbone Taxonomy checklist:

http://staging.gbif.org:8080/portal-web-dynamic/dataset/d7dddbf4-2cf0-4f39-9b2a-bb099caae36c

The Georeferenced occurrences map is empty. It isn't clear where content would come from. Furthermore, it confuses the user leading them to believe that they are actually looking at an occurrence dataset.

Until we actually have some content for maps on checklist dataset pages, can we remove them?]]>
    


Author: kbraak@gbif.org
Comment: [~trobertson@gbif.org] - Do you remember what was intended to be shown on a checklist's map? Do you agree with the proposal to remove them until content is available? 
Created: 2012-12-20 11:44:06.835
Updated: 2012-12-20 11:44:06.835


Author: trobertson@gbif.org
Created: 2012-12-21 14:01:19.389
Updated: 2012-12-21 14:01:19.389
        
If there is a non global taxonomic coverage then it should show a bounding box.  Otherwise it should not show.

see the DetailAction which looks like it must operate incorrectly for the checklists:

{code}
    // the map article is rendered only if the cube has indicated georeferenced records, or if there are
    // coverages that are not global (they don't really warrant visualizing).
    renderMaps = renderMaps || getNumGeoreferencedOccurrences() != null && getNumGeoreferencedOccurrences() > 0;
    if (!renderMaps) {
      for (GeospatialCoverage gc : dataset.getGeographicCoverages()) {
        renderMaps = renderMaps ||
          (gc.getBoundingBox() != null && !gc.getBoundingBox().isGlobalCoverage());
        if (renderMaps) {
          break; // for speed
        }
      }
    }
{code}

    


Author: kbraak@gbif.org
Created: 2012-12-21 15:01:59.31
Updated: 2012-12-21 15:01:59.31
        
For checklists augmented by their eml.xml, it makes sense to display a geographic coverage that isn't global.

I just added the eml.xml for this checklist, try it out and see: http://staging.gbif.org:8080/portal-web-dynamic/dataset/3f8a1297-3259-4700-91fc-acc4170b27ce

Change made in http://code.google.com/p/gbif-portal/source/detail?r=1279 fixes issue. Closing.