Automattic / jetpack

json-endpoints/jetpack/class.jetpack-json-api-plugins-endpoint.php

Doc Comments +3 added lines

@@ -170,6 +170,9 @@
 	}
 
 
+	/**
+	 * @param string $plugin
+	 */
 	protected function validate_plugin( $plugin ) {
 		if ( ! isset( $plugin) || empty( $plugin ) ) {
 			return new WP_Error( 'missing_plugin', __( 'You are required to specify a plugin to activate.', 'jetpack' ), 400 );

_inc/lib/debugger/class-jetpack-cxn-tests.php

Doc Comments +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 	/**
 	 * Helper function to look up the expected master user and return the local WP_User.
 	 *
-	 * @return WP_User Jetpack's expected master user.
+	 * @return string Jetpack's expected master user.
 	 */
 	protected function helper_retrieve_local_master_user() {
 		$master_user = Jetpack_Options::get_option( 'master_user' );

json-endpoints/jetpack/class.jetpack-json-api-sync-endpoint.php

Doc Comments +3 added lines

@@ -228,6 +228,9 @@
 		);
 	}
 
+	/**
+	 * @param Queue $queue
+	 */
 	protected function get_buffer( $queue, $number_of_items ) {
 		$start = time();
 		$max_duration = 5; // this will try to get the buffer

packages/sync/src/Listener.php

Doc Comments +3 added lines

@@ -177,6 +177,9 @@
 		$queue->add_all( $data_to_enqueue );
 	}
 
+	/**
+	 * @param Queue $queue
+	 */
 	function enqueue_action( $current_filter, $args, $queue ) {
 		// don't enqueue an action during the outbound http request - this prevents recursion
 		if ( Settings::is_sending() ) {

packages/sync/src/Queue.php

Doc Comments +6 added lines

@@ -240,6 +240,9 @@ 
 		return true;
 	}
 
+	/**
+	 * @param Queue_Buffer $buffer
+	 */
 	function close( $buffer, $ids_to_remove = null ) {
 		$is_valid = $this->validate_checkout( $buffer );
 
@@ -344,6 +347,9 @@ 
 		return false;
 	}
 
+	/**
+	 * @param string $checkout_id
+	 */
 	private function set_checkout_id( $checkout_id ) {
 		global $wpdb;
 

packages/sync/src/Sender.php

Doc Comments +10 added lines

@@ -70,6 +70,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 );
 	}
@@ -100,6 +103,9 @@ 
 		return $this->do_sync_and_set_delays( $this->sync_queue );
 	}
 
+	/**
+	 * @param Queue $queue
+	 */
 	public function do_sync_and_set_delays( $queue ) {
 		// don't sync if importing
 		if ( defined( 'WP_IMPORTING' ) && WP_IMPORTING ) {
@@ -357,6 +363,10 @@ 
 	}
 
 	// in seconds
+
+	/**
+	 * @param double $seconds
+	 */
 	function set_max_dequeue_time( $seconds ) {
 		$this->max_dequeue_time = $seconds;
 	}

packages/tracking/src/Tracking.php

Doc Comments -1 removed lines

@@ -183,7 +183,6 @@
 	/**
 	 * Record an event in Tracks - this is the preferred way to record events from PHP.
 	 *
-	 * @param mixed  $identity username, user_id, or WP_user object
 	 * @param string $event_name The name of the event
 	 * @param array  $properties Custom properties to send with the event
 	 * @param int    $event_timestamp_millis The time in millis since 1970-01-01 00:00:00 when the event occurred

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 ) {

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
 	 */