WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 6 years ago

#38319 closed enhancement (fixed)

Add $network_id parameter to filters in `get_network_option()`

Reported by: flixos90 Owned by: jeremyfelt
Priority: normal Milestone: 4.7
Component: Options, Meta APIs Version:
Severity: normal Keywords: has-patch has-dev-note
Cc: Focuses: multisite

Description

I just noticed that the filters in get_network_option() do not receive the network ID passed to the function. This makes these filters useless for quite a few use-cases (for example when overriding the process of receiving a specific option via pre_site_option_{$option}).

The network ID should be added as a parameter to all these filters.

I will open similar tickets for the filters and actions in add_network_option(), update_network_option() and delete_network_option().

Attachments (1)

38319.diff​ (2.9 KB ) - added by flixos90 10 years ago.

Download all attachments as: .zip

Change History (5)

@flixos90
10 years ago

#1 @flixos90
10 years ago

  • Keywords has-patch added; needs-patch removed

38319.diff​ introduces the $network_id parameter to the filters in get_network_option().

#2 @jeremyfelt
10 years ago

  • Milestone Awaiting Review → 4.7
  • Owner set to jeremyfelt
  • Status new → reviewing

Good catch. When we moved from get_site_option() to get_network_option() we made it possible to retrieve options from other networks, not just the current network.

Thanks @flixos90.

Related #38320, #38321, #38322.

#3 @jeremyfelt
10 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 38815:

Multisite: Add $network_id parameter to filters in get_network_option().

Props flixos90.
Fixes #38319.

#4 @desrosj
6 years ago

  • Keywords has-dev-note added

Linking the dev note that was published here for reference: ​https://make.wordpress.org/core/2016/11/04/multisite-focused-changes-in-4-7/

Note: See TracTickets for help on using tickets.