fix linting

Author: nickchomey

plugins/dominant-color-images/helper.php

@@ -140,7 +140,6 @@ function dominant_color_get_dominant_color_data( int $attachment_id ) {
 	}
 	$dominant_color_data['dominant_color'] = $hex;
 
-
 	return $dominant_color_data;
 }
 
@@ -152,7 +151,7 @@ function dominant_color_get_dominant_color_data( int $attachment_id ) {
  * @param int $attachment_id Attachment ID.
  * @return array|null Attachment metadata array or null if not an image or metadata unavailable.
  */
-function dominant_color_get_attachment_metadata( int $attachment_id ) {
+function dominant_color_get_attachment_metadata( int $attachment_id ): ?array {
 
 	if ( ! wp_attachment_is_image( $attachment_id ) ) {
 		return null;

plugins/dominant-color-images/tests/test-dominant-color-image-editor-gd.php

@@ -77,33 +77,40 @@ public function test_has_transparency_no_image(): void {
 	 * @covers ::get_dominant_color
 	 */
 	public function test_get_dominant_color_success(): void {
-		$im = imagecreatetruecolor( 1, 1 );
+		$im  = imagecreatetruecolor( 1, 1 );
 		$red = imagecolorallocate( $im, 255, 0, 0 );
 		imagefill( $im, 0, 0, $red );
 
-		$editor = new Dominant_Color_Image_Editor_GD( null );
+		$editor     = new Dominant_Color_Image_Editor_GD( null );
 		$reflection = new ReflectionClass( $editor );
-		$property = $reflection->getProperty( 'image' );
+		$property   = $reflection->getProperty( 'image' );
 		$property->setAccessible( true );
 		$property->setValue( $editor, $im );
 
 		$result = $editor->get_dominant_color();
 
 		$this->assertIsArray( $result );
-		$this->assertSame( array( 'r' => 255, 'g' => 0, 'b' => 0 ), $result );
+		$this->assertSame(
+			array(
+				'r' => 255,
+				'g' => 0,
+				'b' => 0,
+			),
+			$result
+		);
 	}
 
 	/**
 	 * @covers ::has_transparency
 	 */
 	public function test_has_no_transparency(): void {
-		$im = imagecreatetruecolor( 1, 1 );
+		$im  = imagecreatetruecolor( 1, 1 );
 		$red = imagecolorallocate( $im, 255, 0, 0 );
 		imagefill( $im, 0, 0, $red );
 
-		$editor = new Dominant_Color_Image_Editor_GD( null );
+		$editor     = new Dominant_Color_Image_Editor_GD( null );
 		$reflection = new ReflectionClass( $editor );
-		$property = $reflection->getProperty( 'image' );
+		$property   = $reflection->getProperty( 'image' );
 		$property->setAccessible( true );
 		$property->setValue( $editor, $im );
 
@@ -116,13 +123,13 @@ public function test_has_no_transparency(): void {
 	 * @covers ::has_transparency
 	 */
 	public function test_has_transparency_with_transparency(): void {
-		$im = imagecreatetruecolor( 1, 1 );
+		$im          = imagecreatetruecolor( 1, 1 );
 		$alpha_color = imagecolorallocatealpha( $im, 255, 0, 0, 64 );
 		imagefill( $im, 0, 0, $alpha_color );
 
-		$editor = new Dominant_Color_Image_Editor_GD( null );
+		$editor     = new Dominant_Color_Image_Editor_GD( null );
 		$reflection = new ReflectionClass( $editor );
-		$property = $reflection->getProperty( 'image' );
+		$property   = $reflection->getProperty( 'image' );
 		$property->setAccessible( true );
 		$property->setValue( $editor, $im );
 

plugins/dominant-color-images/tests/test-dominant-color-image-editor-imagick.php

@@ -152,7 +152,14 @@ public function test_get_dominant_color_success(): void {
 
 		$result = $editor->get_dominant_color();
 
-		$this->assertEquals( array( 'r' => 255, 'g' => 0, 'b' => 0 ), $result );
+		$this->assertEquals(
+			array(
+				'r' => 255,
+				'g' => 0,
+				'b' => 0,
+			),
+			$result
+		);
 	}
 
 	/**

plugins/dominant-color-images/tests/test-dominant-color.php

@@ -26,7 +26,7 @@ public function test_dominant_color_metadata( string $image_path, array $expecte
 		$this->assertEmpty( $dominant_color_metadata );
 
 		// Creating attachment.
-		$attachment_id = self::factory()->attachment->create_upload_object( $image_path );
+		$attachment_id           = self::factory()->attachment->create_upload_object( $image_path );
 		$dominant_color_metadata = dominant_color_metadata( array(), $attachment_id );
 		$this->assertArrayHasKey( 'dominant_color', $dominant_color_metadata );
 		$this->assertNotEmpty( $dominant_color_metadata['dominant_color'] );
@@ -76,7 +76,7 @@ public function test_has_transparency_metadata( string $image_path, array $expec
 		$transparency_metadata = dominant_color_metadata( array(), 1 );
 		$this->assertEmpty( $transparency_metadata );
 
-		$attachment_id = self::factory()->attachment->create_upload_object( $image_path );
+		$attachment_id         = self::factory()->attachment->create_upload_object( $image_path );
 		$transparency_metadata = dominant_color_metadata( array(), $attachment_id );
 		$this->assertArrayHasKey( 'has_transparency', $transparency_metadata );
 		$this->assertSame( $expected_transparency, $transparency_metadata['has_transparency'] );