[BUG] create_page_content API adding two cms_pagecontent records instead of one about django-cms HOT 3 OPEN

Two PageContent objects AND two PageURL objects?

from django-cms.

Yes extra records in both, a work around to get to the settings after page creation is to delete one instance of the PageURL instance - as this is what is causing the error (and records are 100% identical).

dupe_pageurl_record = PageUrl.objects.filter(page_id=main_page.pk,language=language).first()
dupe_pageurl_record.delete()

When deleted like this, can access the settings page as expected. There are differences on the two different PageContnet records though.

from django-cms.

@jamesfogg OK, after looking at your code, I think I know the reason:

• create_page already creates a PageContent object
• You need to create or get the page. Important: do not forget to provide a created_by argument.
• Instead of create_pagecontent you'll have to get the existing content by using page.pagecontent_set(manager="admin_manager").latest_content() Only if this should be empty, create a page content object manually.
• From there you get the placeholder.

I expect, if you follow this path, you will only have one page url and only one page content object.

My take-away is that create_pagecontent needs to check if there already is a content object before creating a new one.

from django-cms.