WordPress.org
Get WordPress

Make WordPress Core

Opened 8 years ago

Closed 8 years ago

Last modified 4 years ago

#38322 closed enhancement (fixed)

Add $network_id parameter to hooks in `delete_network_option()`

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

Description

The actions in delete_network_option() do not receive the network ID passed to the function which makes using them worthless for several use-cases. The parameter should be added.

Attachments (1)

38322.diff​ (1.7 KB) - added by flixos90 8 years ago.

Download all attachments as: .zip

Change History (5)

@flixos90
8 years ago

#1 @flixos90
8 years ago

  • Keywords has-patch added; needs-patch removed

38322.diff​ introduces the $network_id parameter to the hooks in delete_network_option().

#2 @jeremyfelt
8 years ago

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

Looks good, thanks @flixos90.

Related: #38319, #38320, #38321.

#3 @jeremyfelt
8 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 38818:

Multisite: Add $network_id parameter to hooks in delete_network_option().

Props flixos90.
Fixes #38322.

#4 @desrosj
4 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.