Missing methods get_author and get_modified_date in admin

[corentinbettiol]

Hi, an error is raised when enabling djangocms-versionning with djangocms-alias, due to those lines:

djangocms-alias/djangocms_alias/admin.py

Lines 43 to 45 in 67fb17d

	if djangocms_versioning_enabled:
	alias_admin_list_display.insert(-1, "get_author")
	alias_admin_list_display.insert(-1, "get_modified_date")

[...]
The value of 'list_display[4]' refers to 'get_author' which is not a callable or attribute of 'AliasAdmin',
[...]

[wfehr]

Getting the same when trying to upgrade alias version 2.0.5 to 3.0.x

[corentinbettiol]

Hi @wfehr, that's because djangocms-versioning should add those methods automatically if it's configured (and present before djangocms-alias in your INSTALLED_APPS).

However, we deactivated versioning for djangocms-alias and the bug is gone (but versioning is gone too).

Here's how to disable it:

# add this to your settings.py
VERSIONING_ALIAS_MODELS_ENABLED = False

[fsbraun]

@wfehr @corentinbettiol Has this been resolved?

[wfehr]

@wfehr @corentinbettiol Has this been resolved?

Nope - still occurs with version 3.0.2.

Other latest versions:

• cms 5.0.4
• moderation 2.2.3
• versioning 2.4.0

[fsbraun]

@wfehr Thanks!

Maybe it's got to do with moderation? Can you temporarily remove moderation from installed apps to check if the issue persists?

[wfehr]

@wfehr Thanks!

Maybe it's got to do with moderation? Can you temporarily remove moderation from installed apps to check if the issue persists?

disabling moderation doesn't solve this - the error(s) still occurs.

the "faulty" code is added with enabled versioning (as shown by the issue post above).

[wfehr]

also just to mention it: switching around the installed apps (like versioning above alias or below) doesn't change it as well :)

[fsbraun]

Still flabbergasts me since I cannot reproduce the issue. There is a severe style issue in that code block anyways. In #307 I've moved the detection of the author and modified date columns into the admin class.

[wfehr]

Still flabbergasts me since I cannot reproduce the issue. There is a severe style issue in that code block anyways. In #307 I've moved the detection of the author and modified date columns into the admin class.

I am able to install it with this fix 👍

[wfehr]

it doesn't show the 2 fields in admin display though - but I don't know why that still is the case 😅
maybe the same reason why you can't reproduce this and I got the error in the first place - so still a unknown variable :)

[fsbraun]

Can you share a screenshot of your change list view?

[wfehr]

Can you share a screenshot of your change list view?

Of course:

Both author and state are missing.
But again: I assume the problem still exists on why I couldn't install 3.0.2 in the first place 🤔
With the changes the exception itself is prevented, allowing to upgrade, but the error itself is still present.

But again: I don't know whats wrong with my setup that this occurs - if you can't reproduce that behaviour :)

[fsbraun]

That's what's also puzzling me. The action buttons are there, so versioning injects itself correctly into the admin.