Skip to content

UnencodedStatementVisitor resets value to previously#477

Merged
oformaniuk merged 2 commits intoHandlebars-Net:masterfrom
tommysor:UnencodedStatementVisitor_PreserveNoEscape
Dec 18, 2021
Merged

UnencodedStatementVisitor resets value to previously#477
oformaniuk merged 2 commits intoHandlebars-Net:masterfrom
tommysor:UnencodedStatementVisitor_PreserveNoEscape

Conversation

@tommysor
Copy link
Contributor

existing value when done.

Fix #468 HandlebarsDotNet.Handlebars.Configuration.NoEscape Applied Inconsistently

Ref PR #473 [WIP] Configuration.NoEscape inconsistent fix

This PR does not deal with the question of general rules for encoding that was brought up in #473.

Co-authored-by: Oleh Formaniuk <zjklee@gmail.com>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@oformaniuk oformaniuk merged commit b32b3c0 into Handlebars-Net:master Dec 18, 2021
@tommysor tommysor deleted the UnencodedStatementVisitor_PreserveNoEscape branch December 18, 2021 11:52
@tonysneed
Copy link

Thank you @zjklee for merging this fix. Will you release the patch sometime soon? 😄

@oformaniuk
Copy link
Member

@tonysneed , yes, I do plan to release a new version in a couple of days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HandlebarsDotNet.Handlebars.Configuration.NoEscape Applied Inconsistently

3 participants