WordPress.org

Make WordPress Core

Opened 2 months ago

Last modified 3 days ago

#41285 reviewing defect (bug)

Remove $site_id and $public multisite globals that are no longer used

Reported by: earnjam Owned by: jeremyfelt
Milestone: 4.9 Priority: normal
Severity: normal Version:
Component: Networks and Sites Keywords: has-patch
Focuses: multisite Cc:

Description

As @jeremyfelt noted in #39419 and #34217, there are two globals used in ms-settings.php that are no longer used anywhere else by core, $site_id and $public.

Can we explore removing these?

Attachments (1)

41285.patch (512 bytes) - added by tejas5989 2 months ago.

Download all attachments as: .zip

Change History (10)

@tejas5989
2 months ago

#1 @tejas5989
2 months ago

  • Keywords has-patch added

I have added patch of removing $public.

#2 @SergeyBiryukov
2 months ago

  • Milestone changed from Awaiting Review to 4.9

This ticket was mentioned in Slack in #core-multisite by spacedmonkey. View the logs.


8 weeks ago

This ticket was mentioned in Slack in #core-multisite by spacedmonkey. View the logs.


3 weeks ago

This ticket was mentioned in Slack in #core-multisite by flixos90. View the logs.


11 days ago

#6 @jeremyfelt
11 days ago

  • Owner set to jeremyfelt
  • Status changed from new to reviewing

This ticket was mentioned in Slack in #core-multisite by jeremyfelt. View the logs.


10 days ago

#8 @jeremyfelt
10 days ago

Thanks for the patch, @tejas5989!

It looks like we need to be careful with $site_id. VaultPress is using it for something, likely in the WPCOM environment. If anyone has details on the safeness of removing that, please chime in. :)

I've poked around quite a bit and can't find anyone using $public. I feel pretty comfortable about removing that one, but will keep digging.

This ticket was mentioned in Slack in #core-multisite by jeremyfelt. View the logs.


3 days ago

Note: See TracTickets for help on using tickets.