[wp-trac] [WordPress Trac] #56886: Although users can set a new site's visibility for search engines, the super admin can't :)

[WordPress Trac]

#56886: Although users can set a new site's visibility for search engines, the
super admin can't :)
-----------------------------------------+------------------------
 Reporter:  imath                        |       Owner:  (none)
     Type:  defect (bug)                 |      Status:  new
 Priority:  normal                       |   Milestone:  6.2
Component:  Networks and Sites           |     Version:  trunk
 Severity:  normal                       |  Resolution:
 Keywords:  has-patch changes-requested  |     Focuses:  multisite
-----------------------------------------+------------------------
Changes (by costdev):

 * keywords:  has-patch => has-patch changes-requested


Comment:

 The patch needs some updates.

 - The markup has changed for this field. See [https://github.com/WordPress
 /wordpress-develop/blob/trunk/src/wp-admin/install.php#L181-L205 the
 current markup] on the install page. This includes hooks.
 - The patch seems to be processing all of the `$blog[meta]`, which goes
 outside the scope of this ticket.
 - `intval()` should be replaced with `(int)`.
 - `wp_unslash()` shouldn't be necessary here. It's being converted to an
 integer.

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