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

Add 'bigint unsigned' mapping to Doctrine SqlitePlatform #48

Merged
merged 1 commit into from
Sep 12, 2013

Conversation

bartv2
Copy link
Contributor

@bartv2 bartv2 commented Aug 18, 2013

No description provided.

@DeepDiver1975
Copy link
Contributor

@bartv2 how are we handling this with respect to upstream bug reporting and future updates of doctrine? THX

@bartv2
Copy link
Contributor Author

bartv2 commented Aug 25, 2013

After some testing in OC i create PR's for upstream. 1 is already merged, others will get there

@cosenal
Copy link

cosenal commented Aug 26, 2013

👍 Yes, it fixes owncloud/core#4392

@BernhardPosselt
Copy link

👍

BernhardPosselt pushed a commit that referenced this pull request Sep 12, 2013
Add 'bigint unsigned' mapping to Doctrine SqlitePlatform
@BernhardPosselt BernhardPosselt merged commit 708fb9f into master Sep 12, 2013
@BernhardPosselt BernhardPosselt deleted the fix-missing-mapping branch September 12, 2013 14:51
@AdamWill
Copy link

This patch does not appear to have been sent upstream. I see neither an upstream commit nor a pull request for it. Could it please be submitted? We don't want OC and upstream DBAL to diverge...

@bartv2
Copy link
Contributor Author

bartv2 commented Jan 17, 2014

This is fixed with doctrine/dbal@d5fc8a6

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.

5 participants