Fix automation pattern regressions in #3352

[PhysSong]

Fixes #3952 by passing true to putValue()'s ignoreSurroundingPoint argument.
Its default value is false, which makes positions of automation points incorrect or remove some points which should stay on its position. Thus, passing true fixes this problem.

[zonkmachine]

This will work for sure. I still can't test this but the code is solid.

[zonkmachine]

I looked at the other instances of putValue(...) and I seem to have missed more than these ones and will have another look.

[PhysSong]

@zonkmachine As far as I know, ignoreSurroundingPoints is not used except in automation editor. I think its default value should be true(the behavior before the argument is introduced).

[zonkmachine]

I think its default value should be true(the behavior before the argument is introduced).

Agreed. const bool ignoreSurroundingPoints = true );. That's the original behavior and where it's needed we already pass controlKey.

[PhysSong]

Okay. Is it better to drop current commit and change the default value instead?

[zonkmachine]

Yes, I think so. This looks like a one-liner. I did a quick test and it worked fine.

[PhysSong]

@zonkmachine I just pushed new commit.

[zonkmachine]

Still approved. Tested, works well.