Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#40277 closed defect (bug) (fixed)

Adding page created with the dropdown-pages settings to menu creates Custom Link instead of Page

Reported by: danielmilner Owned by: westonruter
Priority: normal Milestone: 4.7.4
Component: Customize Version: 4.7
Severity: normal Keywords: has-patch commit fixed-major
Cc: Focuses:

Description

How to recreate the issue in Customize:

  1. Create a new page using a dropdown-pages setting that has the allow_addition option.
  2. Switch to the menu panel, and add this newly created page to a Menu. (It shows up as a Page)
  3. Save changes and exit the Customize screen.

View the menu on the front-end. The menu item was added to the menu as a Custom Link instead of a Page.

Attachments (1)

40277.0.diff​ (612 bytes ) - added by westonruter 10 years ago.

Download all attachments as: .zip

Change History (9)

#1 @danielmilner
10 years ago

This only happens during the Customize session where the Page was created. If you Save/Exit and return to the Customize screen, the menu item will be added correctly.

@westonruter
10 years ago

#2 @westonruter
10 years ago

  • Keywords has-patch needs-testing added
  • Milestone Awaiting Review → 4.7.4
  • Version 4.7.3 → 4.7

@danielmilner good catch. Would you give 40277.0.diff​ a try? Looks like we got the values for type and object reversed.

#3 @lukecavanagh
10 years ago

40277.0.diff​ seems to fix the issue, when through the same steps and added the newly created page to an existing menu in the Customizer, when checking the site front-end the page added to the main menu was a page and not a custom link.

#4 @danielmilner
10 years ago

I concur with @lukecavanagh. 40277.0.diff​ appears to fix the issue.

#5 @westonruter
10 years ago

  • Keywords commit added; needs-testing removed
  • Owner set to westonruter
  • Status new → accepted

#6 @westonruter
10 years ago

  • Resolution → fixed
  • Status accepted → closed

In 40380:

Customize: Fix reversal of nav menu item's type and object properties for page stub added in customizer.

Amends [38906].
See #38164.
Fixes #40277.

#7 @westonruter
10 years ago

  • Keywords fixed-major added
  • Resolution fixed
  • Status closed → reopened

Re-opening for consideration in 4.7.4

#8 @swissspidy
10 years ago

  • Resolution → fixed
  • Status reopened → closed

In 40383:

Customize: Fix reversal of nav menu item's type and object properties for
page stub added in customizer.

Amends [38906].
See #38164.
Fixes #40277.

Merges [40380] to the 4.7 branch.

Note: See TracTickets for help on using tickets.