Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 15 additions & 7 deletions src/wp-includes/class-wp-network-query.php
Original file line number Diff line number Diff line change
Expand Up @@ -21,15 +21,23 @@ class WP_Network_Query {
* SQL for database query.
*
* @since 4.6.0
* @var string

This comment was marked as duplicate.

* @var string SQL statement for the last performed network query.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Descriptions aren't appropriate here. If you really want to add one, it would go on a separate paragraph between SQL for database query. and @since 4.6.0. I don't think it is needed, though.

Same goes for the following cases.

Suggested change
* @var string SQL statement for the last performed network query.
* @var string

*/
public $request;

/**
* SQL query clauses.
*
* @since 4.6.0
* @var array
* @var array{
* select: string,
* from: string,
* where: array,
* groupby: string,
* orderby: string,
* limits: string
* }
* Associative array of SQL query clauses used to build the request.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Associative array of SQL query clauses used to build the request.

*/
protected $sql_clauses = array(
'select' => '',
Expand All @@ -44,39 +52,39 @@ class WP_Network_Query {
* Query vars set by the user.
*
* @since 4.6.0
* @var array
* @var array<string, mixed> Query variables for the current query.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* @var array<string, mixed> Query variables for the current query.
* @var array<string, mixed>

*/
public $query_vars;

/**
* Default values for query vars.
*
* @since 4.6.0
* @var array
* @var array<string, mixed> Default values for query variables.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* @var array<string, mixed> Default values for query variables.
* @var array<string, mixed>

*/
public $query_var_defaults;

/**
* List of networks located by the query.
*
* @since 4.6.0
* @var array
* @var WP_Network[]|int[] Array of WP_Network objects or network IDs, depending on 'fields' parameter.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* @var WP_Network[]|int[] Array of WP_Network objects or network IDs, depending on 'fields' parameter.
* @var WP_Network[]|int[]

*/
public $networks;

/**
* The amount of found networks for the current query.
*
* @since 4.6.0
* @var int
* @var int Number of networks found for the current query.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* @var int Number of networks found for the current query.
* @var int

*/
public $found_networks = 0;

/**
* The number of pages.
*
* @since 4.6.0
* @var int
* @var int Number of pages of results for the current query.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* @var int Number of pages of results for the current query.
* @var int

*/
public $max_num_pages = 0;

Expand Down
Loading