Fix undefined variable versionNumberRegex

[fer-av]

versionNumberRegex Not define!!!

Questions	Answers
Description?	No anda Expert mode -> Canal: directorio local
Type?	bug fix
BC breaks?	yes
Deprecations?	yes
Fixed ticket?
Sponsor company	eflyshop
How to test?	hice el cambio y actualice mi sitio!

[ps-jarvis]

Hello @fer-av!

This is your first pull request on autoupgrade repository of the PrestaShop project.

Thank you, and welcome to this Open Source community!

[sonarqubecloud]

Quality Gate failed

Failed conditions
1 Security Hotspot
C Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

[M0rgan01]

@fer-av Hello! Thanks for the contribution, and for restoring the missing variable!

However, the upgrade.js file will be completely refactored into typescript for the introduction of a new UI on the next major release.

On the other hand it is possible to target the 6.x branch, which may have minor or patch releases in the future.

[Quetzacoalt91]

This variable is indeed not used anymore in the upgrade.js file on the dev branch (https://github.com/PrestaShop/autoupgrade/pull/902/files#diff-35ef954ff7799e931a3e99706a3bca463d56a3c77729d5755311b9ebcb93c1acL837), and the incoming version will provide a new UI.

I changed the base branch for 6.x but it requires a rebase on your end.

[sonarqubecloud]

Quality Gate failed

Failed conditions
1 Security Hotspot
C Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

[AureRita]

Hi @fer-av

Thank you for your PR, I tested it and it seems to works as you can see :

recording.13.webm

Tested from :
8.0.4 to 8.2
8.2 to 9.0.0
8.0.4 to 9.0.0

Because the PR seems to works as expected, It's QA ✔️

Thank you