Update OlStyleParser.ts fixing issue 795

[xevxx]

Bug fix for scope declared variables in geoStylerFilterToOlParserFilter method

Thank you 💞

Thank you for considering giving code and/or documentation back to this project, you're awesome and we appreciate your work.

Some guidelines for the proposed change

Please review the guidelines for contributing and the code of conduct of this repository. This makes it easy for you to give back and for us to accept your changes.

Please use the template below the line for the pull request description, and make sure to tick off the boxes in the checklists. In your final pull request everything above the next horizontal line can be removed to not distract reviewers or other readers.

Again: thank you for your contribution.

---

Description

geoStylerFilterToOlParserFilter Filter or rule only breaking bug with suggested fix issue #795 geoStylerFilterToOlParserFilter in the switch statement, intermediate and restFilter are declared in the && case but used in the || case, this breaks if you only have an or filter.

Related issues or pull requests

795

Pull request type

Please check the type of change your PR introduces:

• [x] Bugfix
• [ ] Feature
• [ ] Dependency updates
• [ ] Code style update (formatting, renaming)
• [ ] Refactoring (no functional changes, no api changes)
• [ ] Build related changes
• [ ] Documentation content changes
• [ ] Other (please describe)
• [ ] I am unsure (we'll look into it together)

Do you introduce a breaking change?

• [ ] Yes
• [x] No
• [ ] I am unsure (no worries, we'll find out)

Checklist

• [ ] I understand and agree that the changes in this PR will be licensed under the BSD 2-Clause License
• [ ] I have followed the guidelines for contributing
• [ ] The proposed change fits to the content of the code of conduct
• [ ] I have added or updated tests and documentation, and the test suite passes (run npm test locally)
• [ ] I'm lost; why do I have to check so many boxes? Please help!

[KaiVolland]

Thanks a lot.

I'll had to squash merge and modifiy the commit message it was not matching the conventional commit syntax this repository relies on.

[xevxx]

Thanks, ill follow that process if i do anything else.

Thank you

[jansule]

:tada: This PR is included in version 5.0.0-next.1 :tada:

The release is available on:

• npm package (@next dist-tag)
• GitHub release

Your semantic-release bot :package::rocket:

[jansule]

:tada: This PR is included in version 5.0.0 :tada:

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot :package::rocket: