WordPress.org

Make WordPress Core

Opened 35 hours ago

Last modified 25 hours ago

#30145 reopened task (blessed)

Introduce wp_is_trusted_network()

Reported by: jeremyfelt Owned by: jeremyfelt
Milestone: 4.1 Priority: normal
Severity: normal Version:
Component: Networks and Sites Keywords:
Focuses: multisite Cc:

Description

wp_is_trusted_network() is a first step in establishing the concepts around trusted and untrusted networks in multisite as part of the potential roadmap.

At first, the function will always return false. This will allow us to start plugging it into various parts of core to better determine how to call something "open" (as in registration, with untrusted users).

In the future, it will likely default to false with a filter to modify on a per network basis.

Attachments (1)

30145.diff (487 bytes) - added by jeremyfelt 35 hours ago.

Download all attachments as: .zip

Change History (6)

jeremyfelt35 hours ago

comment:1 jeremyfelt35 hours ago

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

comment:2 jeremyfelt35 hours ago

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

In 30071:

Introduce wp_is_trusted_network()

A first step to establish concepts around trusted and untrusted networks.

  • Will always return false as default.
  • Will one day have a filter.

Fixes #30145

comment:3 johnbillion31 hours ago

  • Keywords has-patch removed
  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Type changed from enhancement to task (blessed)

Discussion points:

  • Should this be moved out of ms-functions.php?
  • Should it be renamed is_trusted_network() (minus the wp_ prefix) to match most of the other conditionals?

comment:4 knutsp30 hours ago

This function must always exist, and should by default return true for single installs, and false for multisite. No more global functions without a wp_ prefix, please. is_trusted_network() may have been used already, somewhere.

Then to determine on what conditions this should return true for multisite, like a constant, or a site option.

After that we can start implementing it to make multisite go in the direction we want.

Just my humble opinion, or my two cents, if you wish.

comment:5 extendwings25 hours ago

I agree with @johnbilion's 2 points.

I guess is_trusted_network() should return false in single site. Because developers can use is_multisite() to check multisite or not.

Note: See TracTickets for help on using tickets.