Adds code to update idvisitor if changed in visits_log table, #DEV-18502

[AltamashShaikh]

Description:

Adds code to update idvisitor if changed in visits_log table
Fixes: #DEV-18502, #22857

Review

• Functional review done
• Potential edge cases thought about (behavior of the code with strange input, with strange internal state or possible interactions with other Matomo subsystems)
• Usability review done (is anything maybe unclear or think about anything that would cause people to reach out to support)
• Security review done
• Wording review done
• Code review done
• Tests were added if useful/possible
• Reviewed for breaking changes
• Developer changelog updated if needed
• Documentation added if needed
• Existing documentation updated if needed

[AltamashShaikh]

@matomo-org/core-reviewers Can you guys check if this PR will solve the issue ?

I tested with this scenario

1. I have a goal which converts when URL contains the word goal1, Request URL: http://localhost.demo.com/test.html?utm_campaign=testChangeIdVisitor1&goal1=&beauthera=&gclid=testChangeID3
2. Saw the visitorID in log_visit, log_conversion and log_adclickID table
3. Gave consent _paq.push(['setConsentGiven']);
4. Reloaded the page and verified the idvisitor in all the 3 tables and it appears to be updated now.