[mapserver-commits] [MapServer/MapServer] 12dfa9: SLD: fix having a TextSymbolizer rule after a Poly...

[Even Rouault]

  Branch: refs/heads/main
  Home:   https://github.com/MapServer/MapServer
  Commit: 12dfa91c8250592aed4951280d7cc06c44921d25
      https://github.com/MapServer/MapServer/commit/12dfa91c8250592aed4951280d7cc06c44921d25
  Author: Even Rouault <even.rouault at spatialys.com>
  Date:   2024-10-17 (Thu, 17 Oct 2024)

  Changed paths:
    M msautotest/sld/label.map
    M src/mapogcsld.cpp

  Log Message:
  -----------
  SLD: fix having a TextSymbolizer rule after a PolygonSymbolizer one

Currently if we have a SLD like:
```xml
<NamedLayer>
        <Name>...</Name>
        <UserStyle>
            <Title>default</Title>
            <FeatureTypeStyle>
                <Rule>
                    <PolygonSymbolizer>...</PolygonSymbolizer>
                </Rule>
                <Rule>
                    <TextSymbolizer>...</TextSymbolizer>
                </Rule>
            </FeatureTypeStyle>
        </UserStyle>
    </NamedLayer>
</StyledLayerDescriptor>
```

the PolygonSymbolizer rule is no applied, only the TextSymolizer one.

This is due to the logic in msSLDParseRule() that inappropriately
overrides the layerObj::type member.


  Commit: 8537ff4bc9a3fa2e70e3b9c01603ae440766c449
      https://github.com/MapServer/MapServer/commit/8537ff4bc9a3fa2e70e3b9c01603ae440766c449
  Author: Even Rouault <even.rouault at spatialys.com>
  Date:   2024-10-18 (Fri, 18 Oct 2024)

  Changed paths:
    M msautotest/sld/label.map
    M src/mapogcsld.cpp

  Log Message:
  -----------
  Merge pull request #7175 from rouault/RulePolygonSymbolizeRuleTextSymbolizer

SLD: fix having a TextSymbolizer rule after a PolygonSymbolizer one


Compare: https://github.com/MapServer/MapServer/compare/e6a3c9e7067f...8537ff4bc9a3

To unsubscribe from these emails, change your notification settings at https://github.com/MapServer/MapServer/settings/notifications