AssertionError thrown when mixing managers

[dtnmr]

Hi !

I'm using this segment of the docs in order to implement translatable MPTT models. (By the way the MPTTManager seems to have been renamed to TreeManager)

However an AssertionError is thrown here due to my Django version (1.10.2)

Is there a way to overcome this, or the feature is not ready for 1.10 yet ?

[spectras]

Hello,

The assertion error is because Django 1.10 no longer supports this way of assigning model managers. It introduced a new API instead, using default_manager_name meta option (and an equivalent for base manager name). Therefore, the _add_to_class way of changing managers doesn't work anymore.

Are you using the new API to assign your manager to your model? If so, can you post a copy of the model and its meta options?

[orf]

Hi, I ran into this error, perhaps the docs should be updated?

I had to add the following code to my models:

objects = TranslationManager()
base_objects = models.Manager()

class Meta:
    base_manager_name = 'base_objects'

[spectras]

Interesting. TranslationManager is hvad's one or a custom one?

[orf]

It was hvads default one.

[spectras]

I should add a tag troublesome. It's not a bug per se, though working around this on the hvad side would be an option. I have trouble wrapping my head around the new API for customizing the manager hierarchy. Even though it was changed by Django team at hvad's request (first version was completely incompatible with hvad), another pass on that is due.

While trying to figure out how to handle that, your workaround seems good.