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

pull request for devel #45

Closed
wants to merge 12 commits into from
Closed

pull request for devel #45

wants to merge 12 commits into from

Conversation

mariuz
Copy link
Collaborator

@mariuz mariuz commented Sep 11, 2015

No description provided.

@mariuz mariuz mentioned this pull request Sep 11, 2015
@cfingerh
Copy link

Changes:

  1. changes to almost all files to change imports within firebird folder based on https://docs.djangoproject.com/en/1.8/releases/1.8/#database-backend-api
  2. lookup_cast() in file operations.py was modified to
    python def lookup_cast(self, lookup_type, internal_type=None):
    based on answer in http://stackoverflow.com/questions/29508775/django-1-8-issue-with-user-authentication

3.- I could not find the import replacement for the import util in operations.py
python from django.db.backends import BaseDatabaseOperations, util

therfore I had to change
https://github.com/maxirobaina/django-firebird/blob/master/firebird/operations.py#L195
from
python name = '"%s"' % util.truncate_name(name, self.max_name_length())
to
python name = '"%s"' % name

  1. There are issues with migrations. I had to modift runserver.py in the main django code to not check for migrations: remove self.check_migrations() at line 112 in function inner_run

@AdamDavin
Copy link

Good Morning,

I believe '3.' import util is in operations.py for django 1.8 should be:
from django.db.backends import utils

As 'util' has been renamed to 'utils'.

Best Regards,

Adam.

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

5 participants