Skip to content
This repository has been archived by the owner on May 10, 2023. It is now read-only.

Fixing case sensitive fk #88

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Nepherhotep
Copy link

The script can't handle indexes and constraints for camel cased columns. The fix just adds quotes to support it (the same as it is done for table names).

@mattlqx
Copy link

mattlqx commented Nov 2, 2015

👍 thanks! This fixed the rather egregious problems I was having with columns named "userID" and the like.

Why this hasn't been merged in over 3 months, I have no idea.

@Nepherhotep
Copy link
Author

It looks like the repo is abandoned. By the way, I faced with another issue - for Django 1.8 it's required to use fk constraints with options "DEFERRABLE INITIALLY DEFERRED".
I didn't create pull request as the one above wasn't merged, just fixed it in my database with migration altering existing fk contraints. But I suggest before going live to update the script as well (may be there are pull requests, I don't know).

@kworr
Copy link
Collaborator

kworr commented Nov 4, 2015

Hey, I'm listening. :)
I'm not active as I'm not working with MySQL, PostgreSQL or Python currently. Checking each patch is taking time, and I'm the kind of person that prefers defer the task if can't be sure of quality.
PS: I'm planning to expand my home network. If there would be a place for test MySQL server I'll try to come back. But again, first I'll add some test's to check that I'm not breaking anything with patches.

@Nepherhotep
Copy link
Author

:)
Well, it's opensource anyway, nobody owes to maintain repo. This script is a great tool, personally for me it helped to migrate a big production project from MySQL, so great thanks for sharing it to community.
I suppose it is worth to add in description - maintainer help wanted :)

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

Successfully merging this pull request may close these issues.

3 participants