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

Can't migrate database on fresh install with cookiecutter-django. #537

Open
3 tasks done
flaviobarros opened this issue May 1, 2018 · 2 comments
Open
3 tasks done

Comments

@flaviobarros
Copy link

flaviobarros commented May 1, 2018

Actual behavior

I have just created a basic project with cookiecutter-django. After having a working site I tried to install zinnia following documentation but It doens't migrate the database. I receive the following error:

LookupError: Model 'users.User' not registered.

Expected behavior

The migration could be done without any error.

Steps to reproduce the issue

  1. Install cookiecutter-django basic project (Mailgun, Heroku and etc). After having a working site
  2. Install zinnia following documentation
  3. try migrate

python manage.py migrate

Specifications

  • Zinnia version: 0.20
  • Django version: 2.0.4
  • Python version: 3.5.2
  • Operating system: Linux Mint 18.2

Disclaimer

Before submitting an issue make sure you have:

  • Read the guidelines for contributing.
  • Checked for duplicate issues.
  • Not a support request.
@flaviobarros
Copy link
Author

I have tested with cookiecutter-django multiple versions, using django 1.11, 1.10 and 1.9. Didn't work with any of them.

@Admoroux
Copy link

Admoroux commented Mar 13, 2019

In zinnia/models/author.py:

def safe_get_user_model():
    """
    Safe loading of the User model, customized or not.
    """
    user_app, user_model = settings.AUTH_USER_MODEL.split('.')
    return apps.get_registered_model(user_app, user_model)

I changed it as:

[...]
from django.contrib.auth import get_user_model
[...]

def safe_get_user_model():
    """
    Safe loading of the User model, customized or not.
    """
    # user_app, user_model = settings.AUTH_USER_MODEL.split('.')
    # return apps.get_registered_model(user_app, user_model)
    return get_user_model()

and runs perfectly.

I'm trying to test it in a clean installation. If it works, i'll make a pull request with it.

Admoroux added a commit to Admoroux/django-blog-zinnia that referenced this issue Mar 14, 2019
This change fixes issue Can't migrate database on fresh install with cookiecutter-django. Fantomas42#537, opened by flaviobarros. This way, User model is allways notified right using Django Auth system.
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

No branches or pull requests

2 participants