I'm trying to split the models.py of my app into several files:

My first guess was do this:

myproject/
    settings.py
    manage.py
    urls.py
    __init__.py
    app1/
        views.py
        __init__.py
        models/
            __init__.py
            model1.py
            model2.py
    app2/
        views.py
        __init__.py
        models/
            __init__.py
            model3.py
            model4.py

This doesn't work, then i found this, but in this solution i still have a problem, when i run python manage.py sqlall app1 I got something like:

BEGIN;
CREATE TABLE "product_product" (
    "id" serial NOT NULL PRIMARY KEY,
    "store_id" integer NOT NULL
)
;
-- The following references should be added but depend on non-existent tables:
-- ALTER TABLE "product_product" ADD CONSTRAINT "store_id_refs_id_3e117eef" FOREIGN KEY     ("store_id") REFERENCES "store_store" ("id") DEFERRABLE INITIALLY DEFERRED;
CREATE INDEX "product_product_store_id" ON "product_product" ("store_id");
COMMIT;

I'm not pretty sure about this, but i'm worried aboout the part The following references should be added but depend on non-existent tables:

This is my model1.py file:

from django.db import models

class Store(models.Model):
    class Meta:
        app_label = "store"

This is my model3.py file:

from django.db import models

from store.models import Store

class Product(models.Model):
    store = models.ForeignKey(Store)
    class Meta:
        app_label = "product"

And apparently works but i got the comment in alter table and if I try this, same thing happens:

class Product(models.Model):
    store = models.ForeignKey('store.Store')
    class Meta:
        app_label = "product"

So, should I run the alter for references manually? this may bring me problems with south?

up vote 22 down vote accepted

I can't even begin to imagine why you'd want to do this. But I'll assume you've got a good reason. If I needed to do this for some reason, I'd do the following:

myproject/
    ...
    app1/
        views.py
        __init__.py
        models.py
        submodels/
            __init__.py
            model1.py
            model2.py
    app2/
        views.py
        __init__.py
        models.py
        submodels/
            __init__.py
            model3.py
            model4.py

Then

#myproject/app1/models.py:
    from submodels/model1.py import *
    from submodels/model2.py import *

#myproject/app2/models.py:
    from submodels/model3.py import *
    from submodels/model4.py import *

But, if you don't have a good reason, put model1 and model2 directly in app1/models.py and model3 and model4 in app2/models.py

---second part---

This is app1/submodels/model1.py file:

from django.db import models
class Store(models.Model):
    class Meta:
        app_label = "store"

Thus correct your model3 file:

from django.db import models
from app1.models import Store

class Product(models.Model):
    store = models.ForeignKey(Store)
    class Meta:
        app_label = "product"

Edited, in case this comes up again for someone: Check out django-schedule for an example of a project that does just this. https://github.com/thauber/django-schedule/tree/master/schedule/models https://github.com/thauber/django-schedule/

  • 1
    With respect to this answer i got another problem when in models2.py make something like from product.models import Product: ImportError: No module named models – diegueus9 Jun 14 '11 at 21:30
  • 49
    you would do it this way to maintain one class per file. – worc Nov 25 '13 at 18:53
  • 34
    The "why" would be the desire to reduce the size of a massive models.py file. I recently did this when mine grew to over 15k lines of code. Great writeup though. The process is fairly simple. The main caveat being you have to remember to define an explicit app_label, since Django extracts this from the immediate module by default. – Cerin Mar 28 '14 at 19:55
  • 1
    Giving this a try in 2016. Is the second part of this post still needed? I only moved the classes to separate files, wrote my __init__.py and everything seems to be working okay. I did not have to correct my model files. I'm able to fetch and create objects from the shell and the django admin. I've been trying django for like a week so I'm wondering if the newest version now allows for this to happen? – Vic Jun 10 '16 at 23:48
  • 2
    @Vic: The app_label in the Meta class is no longer necessary with newer versions of Django. See code.djangoproject.com/wiki/CookBookSplitModelsToFiles – jrial Jun 20 '16 at 7:56

For anyone on Django 1.9, it is now supported by the framework without defining the class meta data.

https://docs.djangoproject.com/en/1.9/topics/db/models/#organizing-models-in-a-package

NOTE: For Django 2, it's still the same

The manage.py startapp command creates an application structure that includes a models.py file. If you have many models, organizing them in separate files may be useful.

To do so, create a models package. Remove models.py and create a myapp/models/ directory with an __init__.py file and the files to store your models. You must import the models in the __init__.py file.

So, in your case, for a structure like

app1/
    views.py
    __init__.py
    models/
        __init__.py
        model1.py
        model2.py
app2/
    views.py
    __init__.py
    models/
        __init__.py
        model3.py
        model4.py

You only need to do

#myproject/app1/models/__init__.py:
from .model1 import Model1
from .model2 import Model2

#myproject/app2/models/__init__.py:
from .model3 import Model3
from .model4 import Model4

A note against importing all the classes:

Explicitly importing each model rather than using from .models import * has the advantages of not cluttering the namespace, making code more readable, and keeping code analysis tools useful.

  • 1
    In case someone is wondering if it's still up to date with Django 2.0 docs.djangoproject.com/en/2.0/topics/db/models/… – NaturalBornCamper Mar 13 at 4:12
  • 2
    NOTE: This usually does not work properly when you have started with models.py and want to migrate later. In that case, you have to delete your migrations and your database :/ Or to manually solve all errors in all migration files – tuergeist Mar 15 at 14:08

I've actually come across a tutorial for exactly what you're asking about, you can view it here:

http://paltman.com/breaking-apart-models-in-django/

One key point that's probably relevant - you may want to use the db_table field on the Meta class to point the relocated classes back at their own table.

I can confirm this approach is working in Django 1.3

Your Answer

 

By clicking "Post Your Answer", you acknowledge that you have read our updated terms of service, privacy policy and cookie policy, and that your continued use of the website is subject to these policies.

Not the answer you're looking for? Browse other questions tagged or ask your own question.