Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#47666 closed defect (bug) (fixed)

wrong url on http://site.com/wp-admin/nav-menus.php

Reported by: alexeyskr's profile alexeyskr Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.3 Priority: normal
Severity: normal Version:
Component: Menus Keywords:
Focuses: administration Cc:

Description

Hello!
We've noticed that if we try to modify the menu items -> http://site.com/wp-admin/nav-menus.php Javascript is not working.
The reason is that in the following file - https://github.com/WordPress/WordPress/blob/master/wp-admin/includes/class-walker-nav-menu-edit.php - the end of link a href= gets several tabs (see line 156) ->
"wp-admin / nav-menus.php? action = move-up-menu-item & amp; menu-item = 4955 & amp; _wpnonce = 46deb17c5d% 09% 09% 09% 09% 09% 09% 09".
Thank you!

Change History (7)

#1 @SergeyBiryukov
5 years ago

  • Component changed from General to Menus
  • Focuses administration added
  • Milestone changed from Awaiting Review to 5.3
  • Owner set to SergeyBiryukov
  • Status changed from new to accepted

#2 @SergeyBiryukov
5 years ago

Hi @alexeyskr, welcome to WordPress Trac! Thanks for the report, I was able to confirm the issue.

#3 @SergeyBiryukov
5 years ago

Introduced in [42343].

#4 @SergeyBiryukov
5 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 45614:

Menus: Remove extra whitespace from URLs in Walker_Nav_Menu_Edit::start_el().

Props alexeyskr, SergeyBiryukov.
Fixes #47666.

#5 follow-up: @alexeyskr
5 years ago

Thank you! Will it be added in the next release?

#6 in reply to: ↑ 5 @knutsp
5 years ago

Replying to alexeyskr:

Thank you! Will it be added in the next release?

When a changeset (r45614) is committed (to trunk, the "alpha" version of a possible future release), it will (unless the next release is already branched, which is not the case at this point).

#7 @alexeyskr
5 years ago

Thank you very much!

Note: See TracTickets for help on using tickets.