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

Compare version vs. OC_VersionCanBeUpgradedFrom before making irreversible changes #9499

Closed
deborahgu opened this issue May 16, 2018 · 4 comments

Comments

@deborahgu
Copy link

Steps to reproduce

  1. Start with any Owncloud version that's document as being migrateable in the migration document, but is not in the current release's current version.php file's VersionCanBeUpgradedFrom.
  2. Run migration either using the automatic migration tool or the steps in the migration document
  3. Realize the documents weren't accurate and cry noisily into your beer because you now need to restore your database and all of your files. (Or alternately manually edit version.php, which I did because I verified it was safe to go from 10.0.8. But I shouldn't have.)

Expected behaviour

When the migration tool starts, it should access the version.php file and check if this will be a permissible upgrade path before it clobbers the database or the local /var/www/owncloud directory.

Actual behaviour

When the migration tool starts, it migrates the database and the local /var/www/owncloud directory. Then, at the end, it checks to see if this is permitted and throws an error if it wasn't. This forces users to have to recover the database dump and the files from a backup directory, and restart a fairly complex process.

@MorrisJobke
Copy link
Member

cc @jospoortvliet - ref #9493

@nextcloud-bot nextcloud-bot added the stale Ticket or PR with no recent activity label Jun 20, 2018
@nextcloud-bot
Copy link
Member

Hey, this issue has been closed because the label stale is set and there were no updates for 14 days. Feel free to reopen this issue if you deem it appropriate.

(This is an automated comment from GitMate.io.)

@nextcloud-bot nextcloud-bot removed the stale Ticket or PR with no recent activity label Jul 5, 2018
@deborahgu
Copy link
Author

This is still a valid bug.

@MorrisJobke MorrisJobke reopened this Jul 5, 2018
@ChristophWurst
Copy link
Member

I'm closing this ticket because it hasn't received any updates in a long time. If this is still relevant, feel free to re-open or file a new ticket. If you do so make sure to fill out the issue template. I can't tell what version of Nextcloud you're referring to based on the information you provided.

Thank you.

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

No branches or pull requests

4 participants