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

Dimension columns default null #11032

Closed

Conversation

spacenate
Copy link
Contributor

@spacenate spacenate commented Dec 18, 2016

Refs #6207. There were only a few dimensions that did not allow NULL values ($ ack 'columnType\s*=.*NOT NULL').

It looks like there are some migrations to be done, I am looking in to that now. Oh neat, migration was actually handled automatically.

@tsteur
Copy link
Member

tsteur commented Dec 18, 2016

They are not null on purpose as they are required by Piwik in order to function properly. I'm pretty sure #6207 was actually fixed just recently and most dimensions apart from these were set to null. Did you create this PR because you saw the issue #6207 or did you have a specific problem?

@spacenate
Copy link
Contributor Author

The former - I was looking through issues attached to the 3.0.0 milestone, and found #6207. I did find it curious how few dimensions did not allow null 😛 If #6207 is all set, feel free to close this PR.

@tsteur
Copy link
Member

tsteur commented Dec 19, 2016

Sorry about that, it was fixed in #10492 . Usually we fix all related issues but rarely we may forget or do not find all of the related issues.

Really appreciate your work there though. Thanks for this 👍

I will close the issue now. Looking forward to your next PR. Merry Christmas

@tsteur tsteur closed this Dec 19, 2016
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 this pull request may close these issues.

None yet

3 participants