[Bugfix release] Allow class-based helpers in strict-mode

[NullVoxPopuli]

#19877

Related: #19878

[mixonic]

Looks great! I would love a gut check from @chancancode @wycats or someone else close to the template/vm. This seems correct to me.

[chancancode]

Seems fine to me

[NullVoxPopuli]

🥳 let's merge?

[kategengler]

Is #19878 identical except for the targeted branch? (it looks so from a very quick look) If so, I think I'd prefer to merge that one and follow our normal release process.

[NullVoxPopuli]

Yup. I thought targeting the release branch was the thing to do for bug fixes 🙃

[kategengler]

Merged in #19878

Will backport.

[vstefanovic97]

@kategengler Hi! Can this be backported to a patch version for 3.28.* ? Thanks in advance!

[kategengler]

@vstefanovic97 It is included in 4.2.0. #19878 was merged on Jan 7 with a tag of [Bugfix release], which would have it apply to 4.1.0.

I have to defer to others on whether this is important enough to backport to LTS cc @NullVoxPopuli @chancancode

[NullVoxPopuli]

I submitted the fix for use with <template> / strict-mode templates / first-class-component-templates, and because the rfc was only recently merged, I'm 🤷 on if it should be back ported to previous LTS. Any project where I'm using <template> atm (via this addon), is using [email protected]+ so... 🤷

[chancancode]

It’s probably a good idea if it’s easy to do (ie the changes apply cleanly). It would help with polyfills etc.