@@ -101,7 +101,7 @@ discard block |
||
101 | 101 | * Update the media post grabbing the post values from |
102 | 102 | * the `attrs` parameter |
103 | 103 | * |
104 | - * @param {Number} $media_id - post media ID |
|
104 | + * @param integer $media_id - post media ID |
|
105 | 105 | * @param {Object} $attrs - `attrs` parameter sent from the client in the request body |
106 | 106 | * @return |
107 | 107 | */ |
@@ -202,7 +202,7 @@ discard block |
||
202 | 202 | * Save the given temporal file in a local folder. |
203 | 203 | * |
204 | 204 | * @param {Array} $file_array |
205 | - * @param {Number} $media_id |
|
205 | + * @param integer $media_id |
|
206 | 206 | * @return {Array|WP_Error} An array with information about the new file saved or a WP_Error is something went wrong. |
207 | 207 | */ |
208 | 208 | private function save_temporary_file( $file_array, $media_id ) { |
@@ -277,7 +277,7 @@ discard block |
||
277 | 277 | /** |
278 | 278 | * Get the image from a remote url and then save it locally. |
279 | 279 | * |
280 | - * @param {Number} $media_id - media post ID |
|
280 | + * @param integer $media_id - media post ID |
|
281 | 281 | * @param {String} $url - image URL to save locally |
282 | 282 | * @return {Array|WP_Error} An array with information about the new file saved or a WP_Error is something went wrong. |
283 | 283 | */ |
@@ -505,6 +505,9 @@ |
||
505 | 505 | public $service = ''; |
506 | 506 | public $total = 0; |
507 | 507 | |
508 | + /** |
|
509 | + * @param integer $total |
|
510 | + */ |
|
508 | 511 | public function __construct( $id, $total ) { |
509 | 512 | $services = new Sharing_Service(); |
510 | 513 | $this->id = esc_html( $id ); |
@@ -86,7 +86,7 @@ discard block |
||
86 | 86 | /** |
87 | 87 | * Return an array of allowed mime_type items used to upload a media file. |
88 | 88 | * |
89 | - * @return array mime_type array |
|
89 | + * @return string[] mime_type array |
|
90 | 90 | */ |
91 | 91 | static function get_allowed_mime_types( $default_mime_types ) { |
92 | 92 | return array_unique( array_merge( $default_mime_types, array( |
@@ -253,7 +253,7 @@ discard block |
||
253 | 253 | * @param object $media_item - media post object |
254 | 254 | * @param file $file - file recently added |
255 | 255 | * @param bool $has_original_media - condition is the original media has been already added |
256 | - * @return bool `true` if the item has been added. Otherwise `false`. |
|
256 | + * @return false|null `true` if the item has been added. Otherwise `false`. |
|
257 | 257 | */ |
258 | 258 | public static function register_revision( $media_item, $file, $has_original_media ) { |
259 | 259 | if ( is_wp_error( $file ) || ! $has_original_media ) { |
@@ -281,6 +281,9 @@ discard block |
||
281 | 281 | return $original; |
282 | 282 | } |
283 | 283 | |
284 | + /** |
|
285 | + * @param string $pathname |
|
286 | + */ |
|
284 | 287 | public static function delete_file( $pathname ) { |
285 | 288 | if ( ! file_exists( $pathname ) || ! is_file( $pathname ) ) { |
286 | 289 | // let's touch a fake file to try to `really` remove the media file |
@@ -296,7 +299,7 @@ discard block |
||
296 | 299 | * |
297 | 300 | * @param number $media_id - media post ID |
298 | 301 | * @param string $filename - basename of the file ( name-of-file.ext ) |
299 | - * @return bool `true` is the file has been removed, `false` if not. |
|
302 | + * @return boolean|null `true` is the file has been removed, `false` if not. |
|
300 | 303 | */ |
301 | 304 | private static function delete_media_history_file( $media_id, $filename ) { |
302 | 305 | $attached_path = get_attached_file( $media_id ); |
@@ -366,6 +369,7 @@ discard block |
||
366 | 369 | * |
367 | 370 | * @param number $media_id - media post ID |
368 | 371 | * @param number [$limit] - maximun amount of items. 20 as default. |
372 | + * @param integer $limit |
|
369 | 373 | * @return array items removed from `revision_history` |
370 | 374 | */ |
371 | 375 | public static function limit_revision_history( $media_id, $limit = null) { |
@@ -175,7 +175,7 @@ |
||
175 | 175 | /** |
176 | 176 | * Back end function to abstract the xmlrpc function calls to wpcom. |
177 | 177 | * |
178 | - * @param $endpoint |
|
178 | + * @param string $endpoint |
|
179 | 179 | * @param $error_message |
180 | 180 | */ |
181 | 181 | function __process_ajax_proxy_request( $endpoint, $error_message ) { // phpcs:ignore |
@@ -270,7 +270,7 @@ discard block |
||
270 | 270 | * Returns the number of decimal places on string representing a price. |
271 | 271 | * |
272 | 272 | * @param string $number Price to check. |
273 | - * @return number number of decimal places. |
|
273 | + * @return null|integer number of decimal places. |
|
274 | 274 | */ |
275 | 275 | private function get_decimal_places( $number ) { |
276 | 276 | $parts = explode( '.', $number ); |
@@ -367,8 +367,7 @@ discard block |
||
367 | 367 | /** |
368 | 368 | * Gets the latests field value from either the old instance or the new instance. |
369 | 369 | * |
370 | - * @param array $mixed Array of values for the new form instance. |
|
371 | - * @param array $mixed Array of values for the old form instance. |
|
370 | + * @param string $field |
|
372 | 371 | * @return mixed $mixed Field value. |
373 | 372 | */ |
374 | 373 | private function get_latest_field_value( $new_instance, $old_instance, $field) { |
@@ -382,6 +381,7 @@ discard block |
||
382 | 381 | * it returns the default values. |
383 | 382 | * |
384 | 383 | * @param int Product Post ID. |
384 | + * @param integer $product_post_id |
|
385 | 385 | * @return array $fields Product Fields from the Product Post. |
386 | 386 | */ |
387 | 387 | private function get_product_from_post( $product_post_id ) { |
@@ -64,6 +64,7 @@ discard block |
||
64 | 64 | |
65 | 65 | /** |
66 | 66 | * Remove a Publicize connection |
67 | + * @param boolean $service_name |
|
67 | 68 | */ |
68 | 69 | function disconnect( $service_name, $connection_id, $_blog_id = false, $_user_id = false, $force_delete = false ) { |
69 | 70 | return Jetpack_Keyring_Service_Helper::disconnect( $service_name, $connection_id, $_blog_id, $_user_id, $force_delete ); |
@@ -559,6 +560,9 @@ discard block |
||
559 | 560 | $this->options_save_other( 'google_plus' ); |
560 | 561 | } |
561 | 562 | |
563 | + /** |
|
564 | + * @param string $service_name |
|
565 | + */ |
|
562 | 566 | function options_save_other( $service_name ) { |
563 | 567 | // Nonce check |
564 | 568 | check_admin_referer( 'save_' . $service_name . '_token_' . $_REQUEST['connection'] ); |
@@ -87,7 +87,7 @@ |
||
87 | 87 | * Construct object from image. |
88 | 88 | * |
89 | 89 | * @param optional $type (hex, rgb, hsv) |
90 | - * @return color as a string formatted as $type |
|
90 | + * @return false|string as a string formatted as $type |
|
91 | 91 | * |
92 | 92 | */ |
93 | 93 | function color( $type = 'hex' ) { |
@@ -68,7 +68,7 @@ discard block |
||
68 | 68 | /** |
69 | 69 | * Returns status of WordAds approval. |
70 | 70 | * |
71 | - * @return boolean true if site is WordAds approved |
|
71 | + * @return string true if site is WordAds approved |
|
72 | 72 | * |
73 | 73 | * @since 4.5.0 |
74 | 74 | */ |
@@ -83,7 +83,7 @@ discard block |
||
83 | 83 | /** |
84 | 84 | * Returns status of WordAds active. |
85 | 85 | * |
86 | - * @return boolean true if ads are active on site |
|
86 | + * @return string true if ads are active on site |
|
87 | 87 | * |
88 | 88 | * @since 4.5.0 |
89 | 89 | */ |
@@ -98,7 +98,7 @@ discard block |
||
98 | 98 | /** |
99 | 99 | * Returns status of WordAds house ads. |
100 | 100 | * |
101 | - * @return boolean true if WP.com house ads should be shown |
|
101 | + * @return string true if WP.com house ads should be shown |
|
102 | 102 | * |
103 | 103 | * @since 4.5.0 |
104 | 104 | */ |
@@ -114,7 +114,7 @@ discard block |
||
114 | 114 | /** |
115 | 115 | * Returns whether or not this site is safe to run ads on. |
116 | 116 | * |
117 | - * @return boolean true if ads shown not be shown on this site. |
|
117 | + * @return string true if ads shown not be shown on this site. |
|
118 | 118 | * |
119 | 119 | * @since 6.5.0 |
120 | 120 | */ |
@@ -49,7 +49,7 @@ discard block |
||
49 | 49 | * Convenience function for grabbing options from params->options |
50 | 50 | * |
51 | 51 | * @param string $option the option to grab |
52 | - * @param mixed $default (optional) |
|
52 | + * @param boolean $default (optional) |
|
53 | 53 | * @return option or $default if not set |
54 | 54 | * |
55 | 55 | * @since 4.5.0 |
@@ -494,7 +494,7 @@ discard block |
||
494 | 494 | /** |
495 | 495 | * Check the reasons to bail before we attempt to insert ads. |
496 | 496 | * |
497 | - * @return true if we should bail (don't insert ads) |
|
497 | + * @return boolean if we should bail (don't insert ads) |
|
498 | 498 | * |
499 | 499 | * @since 4.5.0 |
500 | 500 | */ |