Automattic / jetpack

packages/tracking/legacy/class.tracks-client.php

Doc Comments +2 added lines, -1 removed lines

@@ -48,7 +48,7 @@ 
 
 	/**
 	 * record_event
-	 * @param  mixed  $event Event object to send to Tracks. An array will be cast to object. Required.
+	 * @param  Jetpack_Tracks_Event  $event Event object to send to Tracks. An array will be cast to object. Required.
 	 *                       Properties are included directly in the pixel query string after light validation.
 	 * @return mixed         True on success, WP_Error on failure
 	 */
@@ -71,6 +71,7 @@ 
 
 	/**
 	 * Synchronously request the pixel
+	 * @param string $pixel
 	 */
 	static function record_pixel( $pixel ) {
 		// Add the Request Timestamp and URL terminator just before the HTTP request.

packages/status/tests/php/test_Status.php

Doc Comments +2 added lines, -4 removed lines

@@ -143,7 +143,6 @@ 
 	/**
 	 * Mock a set of filters.
 	 *
-	 * @param array $args Array of filters with their arguments.
 	 * @return phpmock\Mock The mock object.
 	 */
 	protected function mock_filters( $filters = array() ) {
@@ -153,8 +152,7 @@ 
 	/**
 	 * Mock a set of constants.
 	 *
-	 * @param array $args Array of sets with constants and their respective values.
-	 * @return phpmock\Mock The mock object.
+	 * @return phpmock\Mock[] The mock object.
 	 */
 	protected function mock_constants( $constants = array() ) {
 		$prepare_constant = function( $constant ) {
@@ -171,7 +169,7 @@ 
 	 * Mock a global function and make it return a certain value.
 	 *
 	 * @param string $function_name Name of the function.
-	 * @param mixed  $return_value  Return value of the function.
+	 * @param string  $return_value  Return value of the function.
 	 * @return phpmock\Mock The mock object.
 	 */
 	protected function mock_function( $function_name, $return_value = null ) {

_inc/lib/plugins.php

Doc Comments +3 added lines

@@ -95,6 +95,9 @@
 		return (array) $upgrader->skin->get_upgrade_messages();
 	}
 
+	 /**
+	  * @param string $plugin_slug
+	  */
 	 protected static function generate_wordpress_org_plugin_download_link( $plugin_slug ) {
 		return "https://downloads.wordpress.org/plugin/$plugin_slug.latest-stable.zip";
 	 }

packages/connection/tests/php/Manager.php

Doc Comments +1 added lines, -1 removed lines

@@ -115,7 +115,7 @@
 	 * Mock a global function and make it return a certain value.
 	 *
 	 * @param string $function_name Name of the function.
-	 * @param mixed  $return_value  Return value of the function.
+	 * @param integer  $return_value  Return value of the function.
 	 * @return phpmock\Mock The mock object.
 	 */
 	protected function mock_function( $function_name, $return_value = null ) {

packages/sync/src/modules/Callables.php

Doc Comments +3 added lines

@@ -171,6 +171,9 @@
 		update_option( 'jetpack_plugin_api_action_links', $plugins_action_links );
 	}
 
+	/**
+	 * @param integer $checksum
+	 */
 	public function should_send_callable( $callable_checksums, $name, $checksum ) {
 		$idc_override_callables = array(
 			'main_network_site',

packages/connection/src/Manager.php

Doc Comments +2 added lines, -2 removed lines

@@ -106,7 +106,7 @@ 
 	 * Is the user the connection owner.
 	 *
 	 * @param Integer $user_id the user identifier.
-	 * @return Boolean is the user the connection owner?
+	 * @return integer is the user the connection owner?
 	 */
 	public function is_connection_owner( $user_id ) {
 		return $user_id;
@@ -570,7 +570,7 @@ 
 	 *
 	 * @param int|false    $user_id   false: Return the Blog Token. int: Return that user's User Token.
 	 * @param string|false $token_key If provided, check that the token matches the provided input.
-	 * @param bool|true    $suppress_errors If true, return a falsy value when the token isn't found; When false, return a descriptive WP_Error when the token isn't found.
+	 * @param boolean    $suppress_errors If true, return a falsy value when the token isn't found; When false, return a descriptive WP_Error when the token isn't found.
 	 *
 	 * @return object|false
 	 */

packages/sync/legacy/class.jetpack-sync-settings.php

Doc Comments +9 added lines

@@ -164,6 +164,9 @@ 
 		self::set_is_sending( null );
 	}
 
+	/**
+	 * @param boolean|null $is_importing
+	 */
 	static function set_importing( $is_importing ) {
 		// set to NULL to revert to WP_IMPORTING, the standard behavior
 		self::$is_importing = $is_importing;
@@ -198,6 +201,9 @@ 
 		return (bool) self::$is_syncing || ( defined( 'REST_API_REQUEST' ) && REST_API_REQUEST );
 	}
 
+	/**
+	 * @param boolean|null $is_syncing
+	 */
 	static function set_is_syncing( $is_syncing ) {
 		self::$is_syncing = $is_syncing;
 	}
@@ -206,6 +212,9 @@ 
 		return (bool) self::$is_sending;
 	}
 
+	/**
+	 * @param null|boolean $is_sending
+	 */
 	static function set_is_sending( $is_sending ) {
 		self::$is_sending = $is_sending;
 	}

packages/sync/src/Sender.php

Doc Comments +10 added lines

@@ -73,6 +73,9 @@ 
 		return (float) get_option( self::NEXT_SYNC_TIME_OPTION_NAME . '_' . $queue_name, 0 );
 	}
 
+	/**
+	 * @param integer $time
+	 */
 	public function set_next_sync_time( $time, $queue_name ) {
 		return update_option( self::NEXT_SYNC_TIME_OPTION_NAME . '_' . $queue_name, $time, true );
 	}
@@ -103,6 +106,9 @@ 
 		return $this->do_sync_and_set_delays( $this->sync_queue );
 	}
 
+	/**
+	 * @param \Jetpack_Sync_Queue $queue
+	 */
 	public function do_sync_and_set_delays( $queue ) {
 		// don't sync if importing
 		if ( defined( 'WP_IMPORTING' ) && WP_IMPORTING ) {
@@ -360,6 +366,10 @@ 
 	}
 
 	// in seconds
+
+	/**
+	 * @param double $seconds
+	 */
 	function set_max_dequeue_time( $seconds ) {
 		$this->max_dequeue_time = $seconds;
 	}

packages/sync/src/modules/Users.php

Doc Comments +3 added lines

@@ -99,6 +99,9 @@
 		return $user;
 	}
 
+	/**
+	 * @param \WP_User|null $user
+	 */
 	public function expand_user( $user ) {
 		if ( ! is_object( $user ) ) {
 			return null;