[Mapguide-trac] [mapguide-trac] #2200: Stylization: GetRulesFilter does not return match for default rule

MapGuide Open Source trac_mapguide at osgeo.org
Tue Dec 11 07:13:30 PST 2012


#2200: Stylization: GetRulesFilter does not return match for default rule
---------------------------------+------------------------------------------
   Reporter:  gregboone          |       Owner:  gregboone               
       Type:  defect             |      Status:  new                     
   Priority:  medium             |   Milestone:  2.5                     
  Component:  Rendering Service  |     Version:  2.1.0                   
   Severity:  major              |    Keywords:  Stylization Rules Filter
External_id:  1484112            |  
---------------------------------+------------------------------------------
 Summary:

 There is a problem in the Autodesk windows client when using the code path
 to get the rules filter for the current layer based on the current map
 scale.  The problem is that the rules filter that is built does not
 account for the "default" rule which matches all features.  So now certain
 commands are filtering out some features because they don't match any of
 the other rules, but they do match the default rule.

 The fix is to modify the method DefaultStylizer::!GetRulesFilter so that
 it iterates backward (since default rule is at the end usually) and bail
 out returning NULL if it finds a default rule.

 Solution details:

 \\!MgDevBranch\2.5Knight\OS\Common\Stylization\!DefaultStylizer.cpp

 Reverse order of iteration so that we can find the default rule faster.
 Usually it is at the end of the collection.

 If we see the default rule in any of the rules, return NULL from
 !GetRulesFilter.  The default rule will match all features, so we don't
 need to return a rule filter if we encounter a default rule.

 Components Affected:

 !MgStylization.dll

-- 
Ticket URL: <http://trac.osgeo.org/mapguide/ticket/2200>
MapGuide Open Source <http://mapguide.osgeo.org/>
MapGuide Open Source Internals


More information about the mapguide-trac mailing list