fix: don't apply ignorecase to matchToken

#873

internal/check/substitution.go

@@ -119,7 +119,7 @@ func (s Substitution) Run(blk nlp.Block, _ *core.File, cfg *core.Config) ([]core
 					return alerts, msgErr
 				}
 
-				same := matchToken(expected, observed, s.Ignorecase)
+				same := matchToken(expected, observed, false)
 				if !same && !isMatch(s.exceptRe, observed) {
 					action := s.Fields().Action
 					if action.Name == "replace" && len(action.Params) == 0 {

testdata/features/checks.feature

@@ -95,6 +95,8 @@ Feature: Checks
             test.md:21:1:Bugs.TermCase:Use 'JavaScript' rather than Javascript.
             test.md:21:53:Bugs.TermCase:Use 'JavaScript' rather than javascript.
             test.md:23:1:Bugs.TermCase:Use 'iOS' rather than IOS.
+            test.md:25:1:Bugs.SameCase:Use 'MPL 2.0' instead of 'mpl 2.0'
+            test.md:27:1:Bugs.SameCase:Use 'MPL 2.0' instead of 'MPL2.0'
             test2.md:3:1:demo.CapSub:Use 'Change to the `/etc` directory' instead of 'Change into the `/etc` directory'.
             test2.md:7:1:demo.CapSub:Use 'Change to the `/home/user` directory' instead of 'Change into the `/home/user` directory'.
             test2.md:9:1:demo.CapSub:Use 'Change to the `/etc/X11` directory' instead of 'Change into the `/etc/X11` directory'.

testdata/fixtures/checks/Substitution/.vale.ini

@@ -8,6 +8,7 @@ Bugs.URLCtx = YES
 Bugs.MatchCase = YES
 Bugs.KeepCase = YES
 Bugs.TermCase = YES
+Bugs.SameCase = YES
 
 [test2.md]
 demo.CapSub = YES

testdata/fixtures/checks/Substitution/test.md

@@ -21,3 +21,9 @@ The Mutual TLS is a good practice. VNIC is another word.
 Javascript is a programming language. I developed a javascript application.
 
 IOS is a mobile operating system.
+
+mpl 2.0
+
+MPL2.0
+
+MPL 2.0

testdata/styles/Bugs/SameCase.yml

@@ -0,0 +1,9 @@
+extends: substitution
+ignorecase: true
+level: warning
+message: Use '%s' instead of '%s'
+action:
+  name: replace
+swap:
+  MPL: MPL
+  MPL[ -]?2(?:\.0)?: MPL 2.0