Automattic / jetpack

projects/plugins/jetpack/modules/wordads/class-wordads.php

Doc Comments +2 added lines, -2 removed lines

@@ -118,7 +118,7 @@ 
 	 * Convenience function for grabbing options from params->options
 	 *
 	 * @param  string $option the option to grab.
-	 * @param  mixed  $default (optional).
+	 * @param  boolean  $default (optional).
 	 * @return option or $default if not set
 	 *
 	 * @since 4.5.0
@@ -710,7 +710,7 @@ 
 	/**
 	 * Check the reasons to bail before we attempt to insert ads.
 	 *
-	 * @return true if we should bail (don't insert ads)
+	 * @return boolean if we should bail (don't insert ads)
 	 *
 	 * @since 4.5.0
 	 */

projects/plugins/jetpack/modules/wordads/php/class-wordads-api.php

Doc Comments +4 added lines, -4 removed lines

@@ -68,7 +68,7 @@ 
 	/**
 	 * Returns status of WordAds approval.
 	 *
-	 * @return boolean true if site is WordAds approved
+	 * @return string true if site is WordAds approved
 	 *
 	 * @since 4.5.0
 	 */
@@ -83,7 +83,7 @@ 
 	/**
 	 * Returns status of WordAds active.
 	 *
-	 * @return boolean true if ads are active on site
+	 * @return string true if ads are active on site
 	 *
 	 * @since 4.5.0
 	 */
@@ -98,7 +98,7 @@ 
 	/**
 	 * Returns status of WordAds house ads.
 	 *
-	 * @return boolean true if WP.com house ads should be shown
+	 * @return string true if WP.com house ads should be shown
 	 *
 	 * @since 4.5.0
 	 */
@@ -114,7 +114,7 @@ 
 	/**
 	 * Returns whether or not this site is safe to run ads on.
 	 *
-	 * @return boolean true if ads shown not be shown on this site.
+	 * @return string true if ads shown not be shown on this site.
 	 *
 	 * @since 6.5.0
 	 */

projects/plugins/jetpack/modules/wordads/php/class-wordads-sidebar-widget.php

Doc Comments +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 	 * @param array $args Widget args.
 	 * @param array $instance The Widget instance.
 	 *
-	 * @return bool|void
+	 * @return false|null
 	 */
 	public function widget( $args, $instance ) {
 		global $wordads;

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

Doc Comments +1 added lines, -1 removed lines

@@ -389,7 +389,7 @@
 	/**
 	 * Set the disconnect capability for multisite.
 	 *
-	 * @param array $caps The capabilities array.
+	 * @param string[] $caps The capabilities array.
 	 */
 	public function set_multisite_disconnect_cap( $caps ) {
 		// Can individual site admins manage their own connection?

projects/plugins/jetpack/modules/contact-form/grunion-contact-form.php

Doc Comments +48 added lines, -2 removed lines

@@ -352,6 +352,9 @@ 
 		return Grunion_Contact_Form::parse( $atts, do_blocks( $content ) );
 	}
 
+	/**
+	 * @param string $type
+	 */
 	public static function block_attributes_to_shortcode_attributes( $atts, $type ) {
 		$atts['type'] = $type;
 		if ( isset( $atts['className'] ) ) {
@@ -403,6 +406,10 @@ 
 		$atts = self::block_attributes_to_shortcode_attributes( $atts, 'checkbox-multiple' );
 		return Grunion_Contact_Form::parse_contact_field( $atts, $content );
 	}
+
+	/**
+	 * @param string $content
+	 */
 	public static function gutenblock_render_field_radio( $atts, $content ) {
 		$atts = self::block_attributes_to_shortcode_attributes( $atts, 'radio' );
 		return Grunion_Contact_Form::parse_contact_field( $atts, $content );
@@ -2045,7 +2052,6 @@ 
 	 *
 	 * @see ::style()
 	 * @internal
-	 * @param bool $style
 	 */
 	static function _style_on() {
 		return self::style( true );
@@ -2236,6 +2242,7 @@ 
 	 *
 	 * @param int                         $feedback_id
 	 * @param object Grunion_Contact_Form $form
+	 * @param Grunion_Contact_Form $form
 	 *
 	 * @return string $message
 	 */
@@ -2394,7 +2401,7 @@ 
 	 *
 	 * @param array       $attributes Key => Value pairs as parsed by shortcode_parse_atts()
 	 * @param string|null $content The shortcode's inner content: [contact-field]$content[/contact-field]
-	 * @return HTML for the contact form field
+	 * @return string for the contact form field
 	 */
 	static function parse_contact_field( $attributes, $content ) {
 		// Don't try to parse contact form fields if not inside a contact form
@@ -3523,6 +3530,9 @@ 
 
 	}
 
+	/**
+	 * @param string $type
+	 */
 	function render_input_field( $type, $id, $value, $class, $placeholder, $required ) {
 		return "<input
 					type='". esc_attr( $type ) ."'
@@ -3534,24 +3544,40 @@ 
 				/>\n";
 	}
 
+	/**
+	 * @param string $class
+	 * @param string $placeholder
+	 */
 	function render_email_field( $id, $label, $value, $class, $required, $required_field_text, $placeholder ) {
 		$field = $this->render_label( 'email', $id, $label, $required, $required_field_text );
 		$field .= $this->render_input_field( 'email', $id, $value, $class, $placeholder, $required );
 		return $field;
 	}
 
+	/**
+	 * @param string $class
+	 * @param string $placeholder
+	 */
 	function render_telephone_field( $id, $label, $value, $class, $required, $required_field_text, $placeholder ) {
 		$field = $this->render_label( 'telephone', $id, $label, $required, $required_field_text );
 		$field .= $this->render_input_field( 'tel', $id, $value, $class, $placeholder, $required );
 		return $field;
 	}
 
+	/**
+	 * @param string $class
+	 * @param string $placeholder
+	 */
 	function render_url_field( $id, $label, $value, $class, $required, $required_field_text, $placeholder ) {
 		$field = $this->render_label( 'url', $id, $label, $required, $required_field_text );
 		$field .= $this->render_input_field( 'url', $id, $value, $class, $placeholder, $required );
 		return $field;
 	}
 
+	/**
+	 * @param string $class
+	 * @param string $placeholder
+	 */
 	function render_textarea_field( $id, $label, $value, $class, $required, $required_field_text, $placeholder ) {
 		$field = $this->render_label( 'textarea', 'contact-form-comment-' . $id, $label, $required, $required_field_text );
 		$field .= "<textarea
@@ -3566,6 +3592,9 @@ 
 		return $field;
 	}
 
+	/**
+	 * @param string $class
+	 */
 	function render_radio_field( $id, $label, $value, $class, $required, $required_field_text ) {
 		$field = $this->render_label( '', $id, $label, $required, $required_field_text );
 		foreach ( (array) $this->get_attribute( 'options' ) as $optionIndex => $option ) {
@@ -3587,6 +3616,9 @@ 
 		return $field;
 	}
 
+	/**
+	 * @param string $class
+	 */
 	function render_checkbox_field( $id, $label, $value, $class, $required, $required_field_text ) {
 		$field = "<label class='grunion-field-label checkbox" . ( $this->is_error() ? ' form-error' : '' ) . "'>";
 			$field .= "\t\t<input type='checkbox' name='" . esc_attr( $id ) . "' value='" . esc_attr__( 'Yes', 'jetpack' ) . "' " . $class . checked( (bool) $value, true, false ) . ' ' . ( $required ? "required aria-required='true'" : '' ) . "/> \n";
@@ -3619,6 +3651,9 @@ 
 		return $field;
 	}
 
+	/**
+	 * @param string $class
+	 */
 	function render_checkbox_multiple_field( $id, $label, $value, $class, $required, $required_field_text  ) {
 		$field = $this->render_label( '', $id, $label, $required, $required_field_text );
 		foreach ( (array) $this->get_attribute( 'options' ) as $optionIndex => $option ) {
@@ -3634,6 +3669,9 @@ 
 		return $field;
 	}
 
+	/**
+	 * @param string $class
+	 */
 	function render_select_field( $id, $label, $value, $class, $required, $required_field_text ) {
 		$field = $this->render_label( 'select', $id, $label, $required, $required_field_text );
 		$field  .= "\t<select name='" . esc_attr( $id ) . "' id='" . esc_attr( $id ) . "' " . $class . ( $required ? "required aria-required='true'" : '' ) . ">\n";
@@ -3651,6 +3689,10 @@ 
 		return $field;
 	}
 
+	/**
+	 * @param string $class
+	 * @param string $placeholder
+	 */
 	function render_date_field( $id, $label, $value, $class, $required, $required_field_text, $placeholder ) {
 
 		$field = $this->render_label( 'date', $id, $label, $required, $required_field_text );
@@ -3681,6 +3723,10 @@ 
 		return $field;
 	}
 
+	/**
+	 * @param string $class
+	 * @param string $placeholder
+	 */
 	function render_default_field( $id, $label, $value, $class, $required, $required_field_text, $placeholder, $type ) {
 		$field = $this->render_label( $type, $id, $label, $required, $required_field_text );
 		$field .= $this->render_input_field( 'text', $id, $value, $class, $placeholder, $required );

projects/plugins/jetpack/modules/shortcodes/youtube.php

Doc Comments +1 added lines, -1 removed lines

@@ -382,7 +382,7 @@
  *
  * @param array $query_args The query args of the URL.
  *
- * @return array The width and height of the shortcode.
+ * @return integer[] The width and height of the shortcode.
  */
 function jetpack_shortcode_youtube_dimensions( $query_args ) {
 	global $content_width;

projects/plugins/jetpack/modules/sitemaps/sitemap-buffer.php

Doc Comments +1 added lines, -1 removed lines

@@ -263,7 +263,7 @@
 	 * @param DOMElement  $parent (optional) an element to which new children should be added.
 	 * @param DOMDocument $root (optional) the parent document.
 	 *
-	 * @return string|DOMDocument The rendered XML string or an object if root element is specified.
+	 * @return null|DOMNode The rendered XML string or an object if root element is specified.
 	 */
 	protected function array_to_xml_string( $array, $parent = null, $root = null ) {
 		$return_string = false;

projects/plugins/jetpack/modules/videopress/utility-functions.php

Doc Comments +1 added lines, -1 removed lines

@@ -308,7 +308,7 @@
  *
  * @since 4.4
  * @param string $title
- * @return int|WP_Error
+ * @return string
  */
 function videopress_create_new_media_item( $title, $guid = null ) {
 	$post = array(

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;