Shorten table names and re-enable app code checker [$80 awarded]

[ChristophWurst]

Now that Nextcloud support database migration, it should be easy to shorten our table names to be app code checker compliant again.

To do:

• DB migration class
• Tests - if migrations are testable at all
• Bump version number
• Update docs that point to old table names
• Re-enable app code checker on CI

---

The $80 bounty on this issue has been claimed at Bountysource.

[ChristophWurst]

Bummer, migrations don't allow table renaming

[nickvergessen]

Bummer, migrations don't allow table renaming

Little trick:

• Create new table
• Copy everything over

https://github.com/nextcloud/spreed/blob/master/lib/Migration/Version2001Date20171026134605.phphttps://github.com/nextcloud/spreed/blob/master/lib/Migration/Version2001Date20171026134605.php

But yeah, could be simpler

[ChristophWurst]

Little trick:

Create new table
Copy everything over

This is not a real solution for complex relations (e.g. tables that contain referenced primary keys).

https://github.com/nextcloud/spreed/blob/master/lib/Migration/Version2001Date20171026134605.phphttps://github.com/nextcloud/spreed/blob/master/lib/Migration/Version2001Date20171026134605.php

Dead link? 😉

[nickvergessen]

https://github.com/nextcloud/spreed/blob/master/lib/Migration/Version2001Date20171026134605.php

[ChristophWurst]

Thanks. I want to prevent this manual migration as far as possible as it's error-prone (keeping references and also not exhausting the available memory on larger instances).

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and questions.