[wp-trac] [WordPress Trac] #49126: Update needed in wp-admin/includes/template.php

WordPress Trac noreply at wordpress.org
Fri Jan 3 18:50:14 UTC 2020


#49126: Update needed in wp-admin/includes/template.php
----------------------------------------------+----------------------------
 Reporter:  hommealone                        |      Owner:  (none)
     Type:  defect (bug)                      |     Status:  new
 Priority:  normal                            |  Milestone:  Awaiting
                                              |  Review
Component:  General                           |    Version:  5.3.2
 Severity:  normal                            |   Keywords:
  Focuses:  administration, coding-standards  |
----------------------------------------------+----------------------------
 I think an update is needed to the function

 {{{
 _wp_admin_html_begin()
 }}}

 in the file wp-admin/includes/template.php beginning at line 2425.

 The output of this function can be seen in the source code of any page in
 the WordPress admin area, right at the top.

 I noticed first that the code contains this:

 Wrong:

 {{{
 <html xmlns="http://www.w3.org/1999/xhtml"
 }}}


 Should be:

 {{{
 <html xmlns="https://www.w3.org/1999/xhtml"
 }}}


 But more importantly, perhaps,
 [https://wordpress.org/support/users/joyously/|@joyously] notes:

         Looking at the URL, even though it redirects to https, it refers
 to http in its content. The other documents do also, when referencing the
 xmlns. And, having an XML namespace on a HTML5 document doesn’t even make
 sense. It should probably be removed entirely, just as the conditional for
 IE8 should be removed since the browser support is for a minimum of IE11
 for quite awhile now.

 (See: [https://wordpress.org/support/topic/error-in-wp-admin-includes-
 template-php/] )

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/49126>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list