-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Description
Which Umbraco version are you using?
17.0.0
Bug summary
The upgrade screen appears to show the wrong "detected version", where it shows the currently installed version, rather than the version detected from the database's Umbraco.Core.Upgrader.State+Umbraco.Core value.
Specifics
No response
Steps to reproduce
- Create a v13 project using this script:
- Run it once to generate the database
- Create a v17 project using this script:
- Point the v17 project at the v13 database
- Run the project and observe the "Upgrading Umbraco" screen
Expected result / actual result
Expected Result
I would've expected it to say:
Detected current version 13.5.0 ({CC47C751-A81B-489A-A2BC-0240245DB687}), which needs to be upgraded to 17.0.0+da502e0 ({1C38D589-26BB-4A46-9ABE-E4A0DF548A87}).
As the Umbraco migration step GUID maps to the v13.5.0 update:
| To<V_13_5_0.ChangeRedirectUrlToNvarcharMax>("{CC47C751-A81B-489A-A2BC-0240245DB687}"); |
Actual Result
But it actually just says the current installed version, even though that does not match the detected migration plan GUID.
Detected current version 17.0.0 ({CC47C751-A81B-489A-A2BC-0240245DB687}), which needs to be upgraded to 17.0.0+da502e0 ({1C38D589-26BB-4A46-9ABE-E4A0DF548A87}).
Thoughts
I realise that not every minor/patch version has a matching migration plan step, so in my case, my database was actually from a v13.10.1 site, so maybe it would be better if it just stated the detected major version?
E.g:
Detected current version 13 ({CC47C751-A81B-489A-A2BC-0240245DB687}), which needs to be upgraded to 17.0.0+da502e0 ({1C38D589-26BB-4A46-9ABE-E4A0DF548A87}).
I just think that the current wording could be confusing to those who don't understand how Umbraco's migration plans work, or what the GUIDs shown actually mean.
Edit:
I just realised that this also causes the "View report" button to be useless, as it simply opens up a comparison page where it attempts to compare v17 to v17:
