Automattic / jetpack

packages/capabilities/src/Capabilities/class-wpfilterrule.php

Doc Comments +4 added lines

@@ -24,6 +24,10 @@
 	private $required_value;
 
 	// phpcs:ignore Squiz.Commenting.FunctionComment.Missing
+
+	/**
+	 * @param string $filter_name
+	 */
 	public function __construct( $filter_name, $required_value, $initial_value = false ) {
 		$this->filter_name    = $filter_name;
 		$this->required_value = $required_value;

packages/capabilities/src/class-capabilities.php

Doc Comments +16 added lines

@@ -24,21 +24,37 @@
 	}
 
 	// phpcs:ignore Squiz.Commenting.FunctionComment.Missing
+
+	/**
+	 * @param string $name
+	 */
 	public static function get( $name ) {
 		return self::$capabilities[ $name ];
 	}
 
 	// phpcs:ignore Squiz.Commenting.FunctionComment.Missing
+
+	/**
+	 * @param string $name
+	 */
 	public static function granted( $name ) {
 		return isset( self::$capabilities[ $name ] ) ? self::$capabilities[ $name ]->check()->granted() : false;
 	}
 
 	// phpcs:ignore Squiz.Commenting.FunctionComment.Missing
+
+	/**
+	 * @param string $name
+	 */
 	public static function build( $name ) {
 		return ( new Capabilities\Builder() )->create()->register( $name );
 	}
 
 	// phpcs:ignore Squiz.Commenting.FunctionComment.Missing
+
+	/**
+	 * @param Capabilities\AggregateRule $capability
+	 */
 	public static function register( $capability, $name ) {
 		self::$capabilities[ $name ] = $capability;
 	}

packages/capabilities/tests/php/test_Capabilities.php

Doc Comments +13 added lines

@@ -22,12 +22,16 @@ 
 
 	/**
 	 * Utility functions
+	 * @param string $role
 	 */
 	protected function setUserRole( $role ) {
 		$user = wp_get_current_user();
 		$user->set_role( $role );
 	}
 
+	/**
+	 * @param string $cap
+	 */
 	protected function addUserCapability( $cap ) {
 		$user = wp_get_current_user();
 		$user->add_cap( $cap );
@@ -102,6 +106,9 @@ 
 		$this->assertFalse( $capability->check()->granted() );
 	}
 
+	/**
+	 * @param string $product_slug
+	 */
 	private function mockJetpackPlan( $product_slug ) {
 		$this->current_product_slug = $product_slug;
 
@@ -134,6 +141,9 @@ 
 		$this->assertFalse( $capability->check()->granted() );
 	}
 
+	/**
+	 * @param string $supports_slug
+	 */
 	private function mockJetpackPlanSupports( $supports_slug ) {
 		$this->current_supports_slug = $supports_slug;
 
@@ -217,6 +227,9 @@ 
 		$this->assertTrue( $capability->check()->granted() );
 	}
 
+	/**
+	 * @param boolean $is_active
+	 */
 	private function mockJetpackIsActive( $is_active ) {
 		$this->current_is_active = $is_active;