Completed
Push — add/beta-plugin-code-standard ( 36bcad...2258ef )
by
unknown
374:47 queued 364:56
created
projects/plugins/jetpack/tests/php/sync/test_class.jetpack-sync-updates.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -69,6 +69,9 @@  discard block
 block discarded – undo
69 69
 		$this->assertTrue( (bool) $has_action );
70 70
 	}
71 71
 
72
+	/**
73
+	 * @param string $new_version
74
+	 */
72 75
 	function new_plugin_response( $new_version ) {
73 76
 		return (object) array(
74 77
 			'response' => array(
@@ -131,6 +134,9 @@  discard block
 block discarded – undo
131 134
 		$this->assertTrue( (bool) $has_action );
132 135
 	}
133 136
 
137
+	/**
138
+	 * @param string $new_version
139
+	 */
134 140
 	public function new_theme_response( $new_version ) {
135 141
 		return (object) array(
136 142
 			'response' => array(
Please login to merge, or discard this patch.
plugins/jetpack/tests/php/sync/test_class.jetpack-sync-woocommerce.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -312,6 +312,10 @@
 block discarded – undo
312 312
 		}
313 313
 	}
314 314
 
315
+	/**
316
+	 * @param string $expected_meta_key
317
+	 * @param integer $expected_meta_value
318
+	 */
315 319
 	private function assertHasObjectMetaValue( $metas, $order_item_id, $expected_meta_key, $expected_meta_value ) {
316 320
 		$has_meta_entry = false;
317 321
 		foreach( $metas as $meta ) {
Please login to merge, or discard this patch.
plugins/jetpack/tests/php/sync/test_interface.jetpack-sync-replicastore.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -1032,6 +1032,9 @@
 block discarded – undo
1032 1032
 		return $return;
1033 1033
 	}
1034 1034
 
1035
+	/**
1036
+	 * @param string $slug
1037
+	 */
1035 1038
 	private function ensure_synced_taxonomy( $store, $slug, $type = 'post' ) {
1036 1039
 		register_taxonomy(
1037 1040
 			$slug,
Please login to merge, or discard this patch.
projects/plugins/jetpack/tests/php/lib/wpcom-helper-functions.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -13,6 +13,7 @@
 block discarded – undo
13 13
 
14 14
 /**
15 15
  * Wrapper around `WP_REST_Request` constructor. Injects a `sites/$blog_id` into endpoint string for WPCOM environment
16
+ * @param string $path
16 17
  */
17 18
 function wp_rest_request( $method, $path ) {
18 19
 	if ( defined( 'IS_WPCOM' ) && IS_WPCOM ) {
Please login to merge, or discard this patch.
wpcom-fields/class-wpcom-rest-api-v2-attachment-videopress-data.php 1 patch
Doc Comments   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
 	 *
104 104
 	 * @param object $attachment The attachment object.
105 105
 	 *
106
-	 * @return false|int
106
+	 * @return boolean
107 107
 	 */
108 108
 	public function is_video( $attachment ) {
109 109
 		return isset( $attachment->post_mime_type ) && wp_startswith( $attachment->post_mime_type, 'video/' );
@@ -113,8 +113,8 @@  discard block
 block discarded – undo
113 113
 	 * Removes the jetpack_videopress field from the response if the
114 114
 	 * given attachment is not a video.
115 115
 	 *
116
-	 * @param WP_REST_Response $response Response from the attachment endpoint.
117
-	 * @param WP_Post          $attachment The original attachment object.
116
+	 * @param stdClass $response Response from the attachment endpoint.
117
+	 * @param stdClass          $attachment The original attachment object.
118 118
 	 *
119 119
 	 * @return mixed
120 120
 	 */
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
 	 * @param mixed           $object Response from the attachment endpoint.
147 147
 	 * @param WP_REST_Request $request Request to the attachment endpoint.
148 148
 	 *
149
-	 * @return true
149
+	 * @return boolean
150 150
 	 */
151 151
 	public function get_permission_check( $object, $request ) { // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
152 152
 		return true;
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
 	 * @param WP_Post         $object The attachment object.
161 161
 	 * @param WP_REST_Request $request Request to the attachment endpoint.
162 162
 	 *
163
-	 * @return true
163
+	 * @return boolean
164 164
 	 */
165 165
 	public function update_permission_check( $value, $object, $request ) { // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
166 166
 		return true;
Please login to merge, or discard this patch.
projects/plugins/beta/class-jetpack-beta.php 1 patch
Doc Comments   +18 added lines, -1 removed lines patch added patch discarded remove patch
@@ -543,6 +543,9 @@  discard block
 block discarded – undo
543 543
 		return self::get_jetpack_plugin_version();
544 544
 	}
545 545
 
546
+	/**
547
+	 * @return string
548
+	 */
546 549
 	static function get_new_jetpack_version( $is_dev_version = false ) {
547 550
 		$manifest = self::get_beta_manifest();
548 551
 		if ( $is_dev_version ) {
@@ -645,6 +648,9 @@  discard block
 block discarded – undo
645 648
 		return self::get_jetpack_plugin_info( JETPACK_DEV_PLUGIN_FILE );
646 649
 	}
647 650
 
651
+	/**
652
+	 * @param string $plugin_file
653
+	 */
648 654
 	static function get_jetpack_plugin_info( $plugin_file = null ) {
649 655
 
650 656
 		if ( is_null( $plugin_file ) ) {
@@ -678,6 +684,10 @@  discard block
 block discarded – undo
678 684
 		return self::get_remote_data( JETPACK_ORG_API_URL, 'org_data' );
679 685
 	}
680 686
 
687
+	/**
688
+	 * @param string $url
689
+	 * @param string $transient
690
+	 */
681 691
 	static function get_remote_data( $url, $transient, $bypass = false ) {
682 692
 		$prefix = 'jetpack_beta_';
683 693
 		$cache  = get_site_transient( $prefix . $transient );
@@ -800,6 +810,9 @@  discard block
 block discarded – undo
800 810
 		return $current_manifest_data;
801 811
 	}
802 812
 
813
+	/**
814
+	 * @param string $plugin_folder
815
+	 */
803 816
 	static function proceed_to_install_and_activate( $url, $plugin_folder = JETPACK_DEV_PLUGIN_SLUG, $section ) {
804 817
 		self::proceed_to_install( $url, $plugin_folder, $section );
805 818
 
@@ -854,6 +867,10 @@  discard block
 block discarded – undo
854 867
 		return false;
855 868
 	}
856 869
 
870
+	/**
871
+	 * @param string $current_plugin
872
+	 * @param string $replace_with_plugin
873
+	 */
857 874
 	static function replace_active_plugin( $current_plugin, $replace_with_plugin = null, $force_activate = false ) {
858 875
 		if ( self::is_network_active() ) {
859 876
 			$new_active_plugins     = array();
@@ -1180,7 +1197,7 @@  discard block
 block discarded – undo
1180 1197
 	/**
1181 1198
 	 * Checks if a dir is empty
1182 1199
 	 *
1183
-	 * @param [type] $dir The absolute directory path to check
1200
+	 * @param string $dir The absolute directory path to check
1184 1201
 	 * @return boolean
1185 1202
 	 */
1186 1203
 	static function is_dir_empty( $dir ) {
Please login to merge, or discard this patch.
projects/plugins/beta/jetpack-beta-admin.php 1 patch
Doc Comments   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
 	/**
112 112
 	 * Checks if autoupdates and email notifications are toggled.
113 113
 	 *
114
-	 * @param var $option - Which option is being toggled.
114
+	 * @param string $option - Which option is being toggled.
115 115
 	 */
116 116
 	public static function is_toggle_action( $option ) {
117 117
 		return (
@@ -357,7 +357,7 @@  discard block
 block discarded – undo
357 357
 	/**
358 358
 	 * Handles list of available Jetpack tags to select specific Jetpack version number.
359 359
 	 *
360
-	 * @param var  $header - Title of tag.
360
+	 * @param string  $header - Title of tag.
361 361
 	 * @param var  $tag - Jetpack tag (for selecting a specific version of Jetpack).
362 362
 	 * @param var  $url - Download link for Jetpack version.
363 363
 	 * @param var  $section - The kind of version we're switching to (in this case 'tags').
@@ -411,7 +411,7 @@  discard block
 block discarded – undo
411 411
 	 * Handles the activation buttons.
412 412
 	 *
413 413
 	 * @param object $branch - Specifies which branch.
414
-	 * @param var    $section - The kind of branch we're switching to (stable, rc, master, pr).
414
+	 * @param var|null    $section - The kind of branch we're switching to (stable, rc, master, pr).
415 415
 	 */
416 416
 	public static function activate_button( $branch, $section ) {
417 417
 		if ( is_object( $section ) && $branch === 'master' ) {
@@ -611,7 +611,7 @@  discard block
 block discarded – undo
611 611
 	 * Display autoupdate and email notification toggles
612 612
 	 *
613 613
 	 * @param var  $name - name of toggle.
614
-	 * @param var  $option - Which toggle (autoupdates, email_notification).
614
+	 * @param string  $option - Which toggle (autoupdates, email_notification).
615 615
 	 * @param bool $value - If toggle is active or not.
616 616
 	 */
617 617
 	public static function show_toggle( $name, $option, $value ) {
Please login to merge, or discard this patch.