Fix normalizeVersion() levels on 8.x

[Seb33300]

Questions	Answers
Description?	Fix db upgrade from 8.x
Type?	bug fix
BC breaks?	yes
Deprecations?	no
Fixed ticket?	Fixes PrestaShop/PrestaShop#34987
Sponsor company	WEB AREA
How to test?	Please check in the ticket

[matthieu-rolland]

Hello,

It seems like the QA couldn't reproduce the bug you're fixing... do you thing you can give them more context in the issue ? (your environment, anything specific that could help them reproduce ?)

[Seb33300]

I don't understand why QA cannot reproduce as I can reproduce by following the provided steps.

But I think the code fixed is easy to understand.
Currently it returns all version of PS with 4 levels (8.0.0.0, 8.1.0.0 ...)

[Seb33300]

We have different staging environment of the same project, and I am not able to reproduce the issue on another environment.
But i still can reproduce it on the first one.
I will investigate and try to understand why...

[Seb33300]

@matthieu-rolland I updated the ticket to add more information.

It's a bit tricky. The issue happened because I tried to update to 8.1.2, and the autoupgrade module updated my database to 8.1.3

[AureRita]

Hi @Seb33300

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

Untitled_.Jan.15.2024.3_50.PM.webm

But we still have Error as we can see :

@PrestaShop/tech-council do you think this error can be ignored ? because on the issue, at the start we don't have error

Except it, it'll be a QA ✔️

Thank you

[Seb33300]

The version number is properly detected.
And yes, the error is how it's supposed to show

[Seb33300]

Previously you also had an error showing incorrect version number:

[kpodemski]

thank you @Seb33300