[wp-trac] [WordPress Trac] #42506: Customize: Custom Link URL validation for nav menu items doesn't allow jump links

WordPress Trac noreply at wordpress.org
Fri Nov 10 23:40:44 UTC 2017


#42506: Customize: Custom Link URL validation for nav menu items doesn't allow jump
links
-------------------------------------------------+-------------------------
 Reporter:  ravipatel                            |       Owner:
     Type:  defect (bug)                         |  westonruter
 Priority:  normal                               |      Status:  closed
Component:  Menus                                |   Milestone:  4.9
 Severity:  normal                               |     Version:  4.9
 Keywords:  has-patch needs-testing needs-unit-  |  Resolution:  fixed
  tests dev-reviewed                             |     Focuses:  javascript
-------------------------------------------------+-------------------------
Changes (by westonruter):

 * status:  accepted => closed
 * resolution:   => fixed


Comment:

 In [changeset:"42153"]:
 {{{
 #!CommitTicketReference repository="" revision="42153"
 Customize: Restore ability to add Custom Link nav menu items for jump
 links and other URLs that were previously allowed.

 Simplify regular expression for checking URL validity to just do basic
 checks to confirm the value looks like a URL. Leave the complete
 validation to the server-side logic in
 `WP_Customize_Nav_Menu_Item_Setting::sanitize()` to avoid having to
 maintain two separate codebases for validating URLs.

 Props westonruter, SergeyBiryukov for testing.
 Amends [41697].
 See #32816.
 Fixes #42506 for trunk.
 }}}

--
Ticket URL: <https://core.trac.wordpress.org/ticket/42506#comment:12>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list