[mapguide][MG101][New] MgLayer::IsVisibleAtScale has incorrect logic

[Walt Welton-Lair]

You can view the artifact detail at the following URL:

    https://mapguide.osgeo.org/servlets/Scarab/id/MG101

Type
 Defect

Artifact ID
 MG101 (MgLayer::IsVisibleAtScale has incorrect logic)

Reported by
 Walt Welton-Lair
 waltweltonlair (walt.welton-lair at autodesk.com)

New artifact details:
---------------------------------------------------------

- Priority set to new value
  P2
- Artifact Status set to new value
  New
- Component set to new value
  Server
- Subcomponent set to new value
  Core
- Description set to new value
  The MgLayer::IsVisibleAtScale() method has the following block of code:

    if(group == NULL)
        return true;
    else
        return group->IsVisible();

This code always returns - the code which does the scale range check further down is unreachable.

The fix is to replace the above block of code with:

    if(group && !group->IsVisible())
        return false;

- Summary set to new value
  MgLayer::IsVisibleAtScale has incorrect logic
- Version set to new value
  1.0.0
- Operating system set to new value
  All
- Artifact created
- Defect Severity set to new value
  High
- Platform set to new value
  All
- Steps to Reproduce set to new value
  See description above

---------------------------------------------------------
This message was automatically generated by Project Tracker.