[wp-trac] [WordPress Trac] #24760: Creating default object from empty value in wp-includes/ms-settings.php on line 111

WordPress Trac noreply at wordpress.org
Sun Dec 6 18:06:06 UTC 2015


#24760: Creating default object from empty value in wp-includes/ms-settings.php on
line 111
--------------------------+-------------------------
 Reporter:  conner_bw     |       Owner:  jeremyfelt
     Type:  defect (bug)  |      Status:  reviewing
 Priority:  normal        |   Milestone:  4.5
Component:  Multisite     |     Version:  3.0
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |     Focuses:  multisite
--------------------------+-------------------------
Changes (by jeremyfelt):

 * keywords:  has-patch dev-feedback => has-patch
 * owner:   => jeremyfelt
 * status:  reopened => reviewing
 * milestone:  Future Release => 4.5


Comment:

 Thanks for the patch @uglyrobot! I think we're best off with the second
 patch to fill in missing properties. Now that I've stared at it a bit
 more, I'd like to only set `public` as that's the only obvious thing we
 need. I'm hesitant that populating `domain` and `path` could cause a chain
 reaction that's not currently visible.

 I'm running tests now.

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


More information about the wp-trac mailing list