Stop immediately if the target enters the range during ApproachTarget

[TaranDahl]

Stop immediately if the target enters the range during ApproachTarget

• In vanilla, the ApproachTarget will simply exit and do nothing if the target is in range. This will cause your units to approach the target unnecessarily.
• Now you can change this behavior by the following flag.

In rulesmd.ini:

[General]
ApproachTarget.StopWhenInRange=false  ; boolean

[github-actions]

Nightly build for this pull request:

• compiled-dll-3b8f72cab50ae30cd82cfe686587706a13fd2d2f.zip
  These artifacts will expire in 90 days and will not be available for download after that time.

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

[TaranDahl]

The AttackMove.StopWhenTargetAcquired has been deprecated because it was implemented in an incorrect way. It should be ApproachTarget instead of AttackMove that needs to be modified. It has now been replaced by ApproachTarget.StopWhenInRange. The prerequisite for it to take full effect is #2084.

The AttackMove.PursuitTarget has been deprecated because it was implemented in an incorrect way. It should be InAttackMoveKeepRange (implemented by #2081) and ApproachTarget (implemented by #1998) that need to be modified.

[Coronia]

given there were quite a lot of controversial for our previous tag renaming (such as new Trajectory and the recent major renaming), I think we should still stick to the principal of no tag name change unless necessary. For this PR I think we don't have to depreciate those AttackMove tag, could just redirect their implementaion to the new one.

[TaranDahl]

deprecated because it was implemented in an incorrect way

I don't think we should keep something that is incorrect. This is not a simple tag renaming. They are removed because the function itself is incorrect.

[TaranDahl]

Bounty: 10 CNY for the 1st reviewer who gives valid review

[DeathFishAtEase]

Changelog entry position error

[DeathFishAtEase]

I see that it has already added the Tested label and is not a draft, so should this PR already be in a completed state?

I don't think we should keep something that is incorrect. This is not a simple tag renaming. They are removed because the function itself is incorrect.

The documentation for the discarded INI flag and its related content does not seem to have been removed yet, which should also be part of the pending work for this PR, because I see that they have already been removed in the changes to the source code.