@@ -69,6 +69,9 @@ discard block |
||
| 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 |
||
| 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( |
@@ -312,6 +312,10 @@ |
||
| 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 ) { |
@@ -1032,6 +1032,9 @@ |
||
| 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, |
@@ -13,6 +13,7 @@ |
||
| 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 ) { |
@@ -103,7 +103,7 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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; |
@@ -639,6 +639,7 @@ discard block |
||
| 639 | 639 | * Fetch latest Jetpack version. |
| 640 | 640 | * |
| 641 | 641 | * @param bool $is_dev_version - If JP Dev version is being queried. |
| 642 | + * @return string |
|
| 642 | 643 | */ |
| 643 | 644 | public static function get_new_jetpack_version( $is_dev_version = false ) { |
| 644 | 645 | $manifest = self::get_beta_manifest(); |
@@ -714,6 +715,7 @@ discard block |
||
| 714 | 715 | * |
| 715 | 716 | * @param string $branch - Branch. |
| 716 | 717 | * @param string $section - Section. |
| 718 | + * @return string |
|
| 717 | 719 | */ |
| 718 | 720 | public static function get_install_url( $branch = null, $section = null ) { |
| 719 | 721 | if ( is_null( $section ) ) { |
@@ -769,7 +771,7 @@ discard block |
||
| 769 | 771 | /** |
| 770 | 772 | * Get JP plugin data. |
| 771 | 773 | * |
| 772 | - * @param mixed $plugin_file - JP or JP Dev plugin path. |
|
| 774 | + * @param string $plugin_file - JP or JP Dev plugin path. |
|
| 773 | 775 | */ |
| 774 | 776 | public static function get_jetpack_plugin_info( $plugin_file = null ) { |
| 775 | 777 | |
@@ -1418,7 +1420,7 @@ discard block |
||
| 1418 | 1420 | /** |
| 1419 | 1421 | * Checks if a dir is empty. |
| 1420 | 1422 | * |
| 1421 | - * @param [type] $dir The absolute directory path to check. |
|
| 1423 | + * @param string $dir The absolute directory path to check. |
|
| 1422 | 1424 | * @return boolean |
| 1423 | 1425 | */ |
| 1424 | 1426 | public static function is_dir_empty( $dir ) { |
@@ -729,6 +729,7 @@ |
||
| 729 | 729 | * |
| 730 | 730 | * @param string $branch - Branch that's ready for update. |
| 731 | 731 | * @param string $section - What kind of branch we're updated (master, rc, pr). |
| 732 | + * @return string |
|
| 732 | 733 | */ |
| 733 | 734 | public static function update_action_url( $branch, $section ) { |
| 734 | 735 | $query = array( |