[Visual] VisualModelImpl: clean apply transformations

[fredroy]

and dont commit (i.e call updateVisual) automaticallly.

Every time apply{scale, roatation, translation} was called, a updateVisual() was made.
So in this case for example it was done thrice whereas two calls are useless.

It is a bit breaking functionally but I dont know how to make it "warnable"/deprecated/etc.
I dont think lots of people are extending and modifying this part of the code so it should be not really painful (hopefully 🫢)

[with-all-tests]

---

By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).

---

Reviewers will merge this pull-request only if

• it builds with SUCCESS for all platforms on the CI.
• it does not generate new warnings.
• it does not generate new unit test failures.
• it does not generate new scene test failures.
• it does not break API compatibility.
• it is more than 1 week old (or has fast-merge label).

[fredroy]

[ci-build][with-all-tests]