Completed
Pull Request — master (#20016)
by Yaroslav
167:46 queued 157:55
created
projects/plugins/jetpack/modules/shortcodes/instagram.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -145,7 +145,7 @@
 block discarded – undo
145 145
  * @param string $url  URL of the content to be embedded.
146 146
  * @param array  $atts Shortcode attributes.
147 147
  *
148
- * @return array $params Array of parameters to be used in Instagram query.
148
+ * @return string $params Array of parameters to be used in Instagram query.
149 149
  */
150 150
 function jetpack_instagram_get_allowed_parameters( $url, $atts = array() ) {
151 151
 	global $content_width;
Please login to merge, or discard this patch.
projects/plugins/jetpack/modules/theme-tools/site-logo/inc/functions.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -165,6 +165,7 @@
 block discarded – undo
165 165
  * Sanitize the string of classes used for header text.
166 166
  * Limit to A-Z,a-z,0-9,(space),(comma),_,-
167 167
  *
168
+ * @param string $classes
168 169
  * @return string Sanitized string of CSS classes.
169 170
  */
170 171
 function jetpack_sanitize_header_text_classes( $classes ) {
Please login to merge, or discard this patch.
jetpack/tests/php/json-api/test-class.json-api-plugins-endpoints.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -161,6 +161,9 @@
 block discarded – undo
161 161
 		return 'direct';
162 162
 	}
163 163
 
164
+	/**
165
+	 * @param string $dir
166
+	 */
164 167
 	function rmdir( $dir ) {
165 168
 		foreach ( scandir( $dir ) as $file ) {
166 169
 			if ( is_dir( $file ) )
Please login to merge, or discard this patch.
plugins/jetpack/tests/php/sync/test_class.jetpack-sync-callables.php 1 patch
Doc Comments   +8 added lines, -2 removed lines patch added patch discarded remove patch
@@ -149,6 +149,9 @@  discard block
 block discarded – undo
149 149
 		return array( 'test' );
150 150
 	}
151 151
 
152
+	/**
153
+	 * @param string $name
154
+	 */
152 155
 	function assertCallableIsSynced( $name, $value ) {
153 156
 		$this->assertEqualsObject( $value, $this->server_replica_storage->get_callable( $name ), 'Function ' . $name . ' didn\'t have the expected value of ' . json_encode( $value ) );
154 157
 	}
@@ -635,6 +638,9 @@  discard block
 block discarded – undo
635 638
 		$this->assert_sanitized_post_type_default( $sanitized, $label );
636 639
 	}
637 640
 
641
+	/**
642
+	 * @param string $label
643
+	 */
638 644
 	function assert_sanitized_post_type_default( $sanitized, $label ) {
639 645
 		$this->assertEquals( $label, $sanitized->name );
640 646
 		$this->assertEquals( 'Posts', $sanitized->label );
@@ -1201,7 +1207,7 @@  discard block
 block discarded – undo
1201 1207
 	/**
1202 1208
 	 * Test getting a hosting provider by a known function
1203 1209
 	 *
1204
-	 * @return bool
1210
+	 * @return boolean|null
1205 1211
 	 */
1206 1212
 	public function test_get_hosting_provider_by_known_function() {
1207 1213
 
@@ -1310,7 +1316,7 @@  discard block
 block discarded – undo
1310 1316
 /**
1311 1317
  * Create a recursive object.
1312 1318
  *
1313
- * @return object
1319
+ * @return stdClass
1314 1320
  */
1315 1321
 function jetpack_recursive_banana() {
1316 1322
 	$banana        = new stdClass();
Please login to merge, or discard this patch.
projects/plugins/jetpack/tests/php/sync/test_class.jetpack-sync-options.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -286,6 +286,9 @@
 block discarded – undo
286 286
 		$this->assertTrue( in_array( 'foo_option_bar', $whitelist ) );
287 287
 	}
288 288
 
289
+	/**
290
+	 * @param string $option_name
291
+	 */
289 292
 	function assertOptionIsSynced( $option_name, $value ) {
290 293
 		$this->assertEqualsObject( $value, $this->server_replica_storage->get_option( $option_name ), 'Option ' . $option_name . ' did\'t have the extected value of ' . json_encode( $value ) );
291 294
 	}
Please login to merge, or discard this patch.
projects/plugins/jetpack/tests/php/sync/test_class.jetpack-sync-themes.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -401,6 +401,9 @@
 block discarded – undo
401 401
 		$this->assertEquals( $event->args[0]['title'], 'My Widget' );
402 402
 	}
403 403
 
404
+	/**
405
+	 * @param string $slug
406
+	 */
404 407
 	private function install_theme( $slug ) {
405 408
 		require_once ABSPATH . 'wp-admin/includes/theme-install.php';
406 409
 		require_once ABSPATH . 'wp-admin/includes/class-wp-upgrader.php';
Please login to merge, or discard this patch.
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.