#39071 closed defect (bug) (fixed)
Pages created as theme starter content aren't saved if a page is created from Customizer
Reported by: | dlh | Owned by: | westonruter |
---|---|---|---|
Milestone: | 4.7 | Priority: | high |
Severity: | normal | Version: | 4.7 |
Component: | Customize | Keywords: | has-patch commit dev-reviewed |
Focuses: | Cc: |
Description
To replicate on a "fresh site" running Twenty Seventeen:
- Open the Customizer, and navigate directly to "Theme Options"
- Use "Add New Page" to add a page to a front page section
- Publish the changes
The pages intended to be starter content will still be auto-draft
s. The links to those pages will still be in the menu; returning to the frontend and attempting to visit them results in a 404.
Attachments (1)
Change History (9)
#1
@
8 years ago
- Keywords has-patch needs-testing added
- Milestone changed from Awaiting Review to 4.7
- Owner set to westonruter
- Priority changed from normal to high
- Status changed from new to accepted
This ticket was mentioned in Slack in #core-customize by westonruter. View the logs.
8 years ago
#3
@
8 years ago
39071.0.diff fixes the issue for me.
This ticket was mentioned in Slack in #core by helen. View the logs.
8 years ago
Note: See
TracTickets for help on using
tickets.
@dlh excellent catch! The problem is that when starter content is applied, the
nav_menus_created_posts
setting is initially non-empty. Nevertheless, the nav menus logic is expecting that it would be empty as seen in theinsertedAutoDrafts
array which would keep track of the newly-created nav menus in the current customizer session. Once saving, it was intended that this would be used to empty out the list but the logic is faulty. We should only be manipulating the one setting and not trying to keep a separate copy around.