WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#38356 closed enhancement (fixed)

Replace get_blog_details() usage in add_user_to_blog() with get_site()

Reported by: iamfriendly Owned by: jeremyfelt
Milestone: 4.7 Priority: normal
Severity: normal Version: 3.0
Component: Networks and Sites Keywords: has-patch has-unit-tests
Focuses: multisite Cc:

Description

See https://core.trac.wordpress.org/ticket/37102#comment:17

Number 22 from original ticket.

Should probably add unit tests for add_user_to_blog() at the same time.

Attachments (1)

38356.diff (1.6 KB) - added by flixos90 4 years ago.

Download all attachments as: .zip

Change History (5)

#1 @flixos90
4 years ago

  • Keywords needs-patch needs-unit-tests added

@flixos90
4 years ago

#2 @flixos90
4 years ago

  • Keywords has-patch has-unit-tests added; needs-patch needs-unit-tests removed
  • Milestone changed from Awaiting Review to 4.7
  • Owner set to jeremyfelt
  • Status changed from new to assigned

38356.diff replaces get_blog_details() with get_site() and adds unit tests for add_user_to_blog().

This ticket was mentioned in Slack in #core by jeffpaul. View the logs.


4 years ago

#4 @jeremyfelt
4 years ago

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

In 38903:

Multisite: Replace get_blog_details() in add_user_to_blog() with get_site().

Adds tests for add_user_to_blog().

Props flixos90.
Fixes #38356.

Note: See TracTickets for help on using tickets.