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

Move all core tables to be created using Doctrine Migrations #26924

Closed
24 tasks
PVince81 opened this issue Jan 11, 2017 · 7 comments
Closed
24 tasks

Move all core tables to be created using Doctrine Migrations #26924

PVince81 opened this issue Jan 11, 2017 · 7 comments

Comments

@PVince81
Copy link
Contributor

PVince81 commented Jan 11, 2017

Core

  • appconfig
  • storages
  • mounts
  • mimetypes
  • filecache
  • group_user
  • group_admin
  • groups
  • preferences
  • properties
  • share
  • jobs
  • users
  • authtoken
  • vcategory
  • vcategory_to_object
  • systemtag
  • systemtag_object_mapping
  • systemtag_group
  • privatedata
  • file_locks
  • comments
  • comments_read_markers
  • credentials

Apps

TODO

@DeepDiver1975 @butonic FYI

By the way, if we want to change fileid of oc_filecache to bigint we'll first need to port its schema to be a migration. (#26901)

@PVince81 PVince81 added this to the backlog milestone Jan 11, 2017
@DeepDiver1975
Copy link
Member

DeepDiver1975 commented Jan 12, 2017

Move all core tables to be created using Doctrine Migrations

I'd not do this

@butonic
Copy link
Member

butonic commented Jan 12, 2017

@DeepDiver1975 do what not? use bigint? move to migrations for all tables? come on ... I know you are on sick leave but don't make us guess ... we will certainly get it wrong and your mood will worsen as a result of us doing stupid things ... 🍪 ?

@DeepDiver1975
Copy link
Member

but don't make us guess

come on - you know be for such long time ..... 😉

@PVince81
Copy link
Contributor Author

@phisch I believe you likely already covered a few during the migration implementation ?

@ownclouders
Copy link
Contributor

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

(This is an automated comment from GitMate.io.

@PVince81
Copy link
Contributor Author

PVince81 commented Aug 8, 2018

did we not do this already ? @DeepDiver1975 @VicDeo

or only parts of it ?

@VicDeo
Copy link
Member

VicDeo commented Aug 8, 2018

@PVince81 the answer is "no", as soon as https://github.com/owncloud/core/blob/master/db_structure.xml still exists

@DeepDiver1975 DeepDiver1975 closed this as not planned Won't fix, can't repro, duplicate, stale Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants