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

Posts content is not a mediumText (ref #1155) #1801

Closed
luceos opened this issue Jun 24, 2019 · 3 comments · Fixed by #2793
Closed

Posts content is not a mediumText (ref #1155) #1801

luceos opened this issue Jun 24, 2019 · 3 comments · Fixed by #2793
Labels
Milestone

Comments

@luceos
Copy link
Member

luceos commented Jun 24, 2019

Even though we merged a PR #1155 in 2017, this migration is not working. The issue seems to relate to something in dbal where changing from text to medium or longtext has no effect.

We need to increase the size of our content.

Ref: doctrine/dbal#2566

@franzliedke
Copy link
Contributor

@luceos This only fixed the problem for new installs, but not for existing databases, which would have already run this migration.

Never change migrations that have been released already.

I would wager that discuss probably still has a text column here. 😕

@franzliedke franzliedke reopened this Aug 28, 2019
@franzliedke
Copy link
Contributor

Hmm, discuss has mediumtext. Something is wrong here.

wzdiyb pushed a commit to wzdiyb/core that referenced this issue Feb 16, 2020
@askvortsov1
Copy link
Sponsor Member

Let's create a new migration that uses the comment hack? Linked GH issue doesn't seem to show progress, but if that gets the job done...

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

Successfully merging a pull request may close this issue.

3 participants