Automattic / jetpack

projects/plugins/jetpack/tests/php/sync/test_class.jetpack-sync-updates.php

Doc Comments +6 added lines

@@ -69,6 +69,9 @@ 
 		$this->assertTrue( (bool) $has_action );
 	}
 
+	/**
+	 * @param string $new_version
+	 */
 	function new_plugin_response( $new_version ) {
 		return (object) array(
 			'response' => array(
@@ -131,6 +134,9 @@ 
 		$this->assertTrue( (bool) $has_action );
 	}
 
+	/**
+	 * @param string $new_version
+	 */
 	public function new_theme_response( $new_version ) {
 		return (object) array(
 			'response' => array(

projects/plugins/jetpack/tests/php/sync/test_class.jetpack-sync-woocommerce.php

Doc Comments +4 added lines

@@ -312,6 +312,10 @@
 		}
 	}
 
+	/**
+	 * @param string $expected_meta_key
+	 * @param integer $expected_meta_value
+	 */
 	private function assertHasObjectMetaValue( $metas, $order_item_id, $expected_meta_key, $expected_meta_value ) {
 		$has_meta_entry = false;
 		foreach( $metas as $meta ) {

projects/plugins/jetpack/tests/php/sync/test_interface.jetpack-sync-replicastore.php

Doc Comments +3 added lines

@@ -1032,6 +1032,9 @@
 		return $return;
 	}
 
+	/**
+	 * @param string $slug
+	 */
 	private function ensure_synced_taxonomy( $store, $slug, $type = 'post' ) {
 		register_taxonomy(
 			$slug,

projects/plugins/jetpack/tests/php/lib/wpcom-helper-functions.php

Doc Comments +1 added lines

@@ -13,6 +13,7 @@
 
 /**
  * Wrapper around `WP_REST_Request` constructor. Injects a `sites/$blog_id` into endpoint string for WPCOM environment
+ * @param string $path
  */
 function wp_rest_request( $method, $path ) {
 	if ( defined( 'IS_WPCOM' ) && IS_WPCOM ) {

projects/plugins/jetpack/_inc/lib/core-api/wpcom-fields/class-wpcom-rest-api-v2-attachment-videopress-data.php

Doc Comments +5 added lines, -5 removed lines

@@ -103,7 +103,7 @@ 
 	 *
 	 * @param object $attachment The attachment object.
 	 *
-	 * @return false|int
+	 * @return boolean
 	 */
 	public function is_video( $attachment ) {
 		return isset( $attachment->post_mime_type ) && wp_startswith( $attachment->post_mime_type, 'video/' );
@@ -113,8 +113,8 @@ 
 	 * Removes the jetpack_videopress field from the response if the
 	 * given attachment is not a video.
 	 *
-	 * @param WP_REST_Response $response Response from the attachment endpoint.
-	 * @param WP_Post          $attachment The original attachment object.
+	 * @param stdClass $response Response from the attachment endpoint.
+	 * @param stdClass          $attachment The original attachment object.
 	 *
 	 * @return mixed
 	 */
@@ -146,7 +146,7 @@ 
 	 * @param mixed           $object Response from the attachment endpoint.
 	 * @param WP_REST_Request $request Request to the attachment endpoint.
 	 *
-	 * @return true
+	 * @return boolean
 	 */
 	public function get_permission_check( $object, $request ) { // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
 		return true;
@@ -160,7 +160,7 @@ 
 	 * @param WP_Post         $object The attachment object.
 	 * @param WP_REST_Request $request Request to the attachment endpoint.
 	 *
-	 * @return true
+	 * @return boolean
 	 */
 	public function update_permission_check( $value, $object, $request ) { // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
 		return true;

projects/plugins/beta/jetpack-beta-admin.php

Doc Comments +4 added lines, -4 removed lines

@@ -111,7 +111,7 @@ 
 	/**
 	 * Checks if autoupdates and email notifications are toggled.
 	 *
-	 * @param var $option - Which option is being toggled.
+	 * @param string $option - Which option is being toggled.
 	 */
 	public static function is_toggle_action( $option ) {
 		return (
@@ -357,7 +357,7 @@ 
 	/**
 	 * Handles list of available Jetpack tags to select specific Jetpack version number.
 	 *
-	 * @param var  $header - Title of tag.
+	 * @param string  $header - Title of tag.
 	 * @param var  $tag - Jetpack tag (for selecting a specific version of Jetpack).
 	 * @param var  $url - Download link for Jetpack version.
 	 * @param var  $section - The kind of version we're switching to (in this case 'tags').
@@ -411,7 +411,7 @@ 
 	 * Handles the activation buttons.
 	 *
 	 * @param object $branch - Specifies which branch.
-	 * @param var    $section - The kind of branch we're switching to (stable, rc, master, pr).
+	 * @param var|null    $section - The kind of branch we're switching to (stable, rc, master, pr).
 	 */
 	public static function activate_button( $branch, $section ) {
 		if ( is_object( $section ) && $branch === 'master' ) {
@@ -611,7 +611,7 @@ 
 	 * Display autoupdate and email notification toggles
 	 *
 	 * @param var  $name - name of toggle.
-	 * @param var  $option - Which toggle (autoupdates, email_notification).
+	 * @param string  $option - Which toggle (autoupdates, email_notification).
 	 * @param bool $value - If toggle is active or not.
 	 */
 	public static function show_toggle( $name, $option, $value ) {

projects/plugins/beta/class-jetpack-beta.php

Doc Comments +4 added lines, -2 removed lines

@@ -639,6 +639,7 @@ 
 	 * Fetch latest Jetpack version.
 	 *
 	 * @param bool $is_dev_version - If JP Dev version is being queried.
+	 * @return string
 	 */
 	public static function get_new_jetpack_version( $is_dev_version = false ) {
 		$manifest = self::get_beta_manifest();
@@ -714,6 +715,7 @@ 
 	 *
 	 * @param string $branch  - Branch.
 	 * @param string $section - Section.
+	 * @return string
 	 */
 	public static function get_install_url( $branch = null, $section = null ) {
 		if ( is_null( $section ) ) {
@@ -769,7 +771,7 @@ 
 	/**
 	 * Get JP plugin data.
 	 *
-	 * @param mixed $plugin_file - JP or JP Dev plugin path.
+	 * @param string $plugin_file - JP or JP Dev plugin path.
 	 */
 	public static function get_jetpack_plugin_info( $plugin_file = null ) {
 
@@ -1418,7 +1420,7 @@ 
 	/**
 	 * Checks if a dir is empty.
 	 *
-	 * @param [type] $dir The absolute directory path to check.
+	 * @param string $dir The absolute directory path to check.
 	 * @return boolean
 	 */
 	public static function is_dir_empty( $dir ) {