Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update detected after custom migration #62

Closed
nylen opened this issue Sep 21, 2019 · 3 comments · Fixed by #63
Closed

Update detected after custom migration #62

nylen opened this issue Sep 21, 2019 · 3 comments · Fixed by #63

Comments

@nylen
Copy link
Contributor

nylen commented Sep 21, 2019

From https://forums.classicpress.net/t/classicpress-1-1-0-rc1-release-notes/1582/3:

migration

Looks like there is some updater state that we need to clear after doing a custom/advanced migration.

Edit to add:

  • This happens sometimes but not every time in my testing
  • If you go to “Dashboard > Updates” and click “Check Again”, then this should clear itself.
@bahiirwa
Copy link

bahiirwa commented Oct 8, 2019

This happens in all my custom migrations. Always have that custom_migration version notice. This still happens with WP 5.3 beta 2 migration to 1.1.0

nylen added a commit that referenced this issue Oct 24, 2019
"WordPress _custom_migration is available! Please update now."

Fixes #62.
@johnalarcon
Copy link

I think what's going on here is that the message is actually showing 2 different things. The first part is saying there's a new version of the migration plugin (which actually leads to a 404, btw), so, that's definitely broken.

The other part of the message is telling you that there's a core update available (because you just downgraded, there's now a core update available), ...and when you visit the updates page, there it is...instead of the migration plugin update you were expecting because of the messaging. About the second half of the message, it should be clearer about what you are updating.

Just a guess here... are these status messages being concatenated, rather than added individually.

@nylen
Copy link
Contributor Author

nylen commented Jan 11, 2020

Belated reply to @johnalarcon's comment above, now that this is fixed:

There is nothing in the migration plugin that would show an update notice for the plugin itself.

This happened because the migration plugin needs to trick the core update process into updating, and to do that, it needs to set the version number that will be installed. Since we don't know the version number of the zip file the user enters (it could be anything), we just use the string _custom_migration.

This data was "sticking around" after migration, so the core update system thought it was still time to install a new version of CP/WP called _custom_migration. The fix is to remove this stale update data after a migration is done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants