FIX: PR feedbacks

Author: patil-vipul

.stylelintrc.json

@@ -6,5 +6,7 @@
 		"**/*.php",
 		"**/*.svg"
 	],
-	"rules": {}
+	"rules": {
+		"selector-class-pattern": null
+	}
 }

inc/Modules/MediaSharing/MediaActions.php

@@ -136,7 +136,7 @@ public function remove_sync_meta( int $attachment_id ): void {
 			}
 
 			// Make POST request to delete attachment on brand sites.
-			$response = wp_remote_post(
+			$response = wp_safe_remote_post(
 				$site_url . '/wp-json/' . Abstract_REST_Controller::NAMESPACE . '/delete-media-metadata',
 				[
 					'body'      => wp_json_encode(
@@ -350,7 +350,7 @@ public function update_sync_attachments( int $attachment_id ): void {
 			$attachment_data['terms']       = $attachment_terms;
 
 			// Make POST request to update existing attachment on brand sites.
-			wp_remote_post(
+			wp_safe_remote_post(
 				$site_url . $post_request_suffix,
 				[
 					'body'    => [

inc/Modules/Rest/Media_Sharing_Controller.php

@@ -746,7 +746,7 @@ public function sync_media_files( \WP_REST_Request $request ): \WP_Error|\WP_RES
 
 			// Share the attachment metadata with the brand sites.
 			// Get attachment metadata.
-			$attachment_data = wp_get_attachment_metadata( $media['id'] );
+			$attachment_data = wp_get_attachment_metadata( $media['id'] ?? 0 );
 
 			// Get attachment title, alt text, caption and description.
 			$attachment_data['post_title']  = $media['title'];
@@ -767,6 +767,9 @@ public function sync_media_files( \WP_REST_Request $request ): \WP_Error|\WP_RES
 		// Get all registered brand sites to compare endpoint and get API token before sharing media.
 		$all_brand_sites = Settings::get_shared_sites();
 
+		// Success response.
+		$success_response = [];
+
 		foreach ( $brand_sites as $site ) {
 			$site_url = $site;
 
@@ -776,15 +779,13 @@ public function sync_media_files( \WP_REST_Request $request ): \WP_Error|\WP_RES
 
 			if ( empty( $site_url ) ) {
 				$failed_sites[] = [
-					'site'      => $site,
-					'message'   => sprintf(
+					'site'     => $site,
+					'message'  => sprintf(
 					/* translators: %s: site URL */
 						__( 'Invalid site URL: %s', 'onemedia' ),
 						$site_url . $brand_site_prefix
 					),
-					'site_url'  => $site_url . $brand_site_prefix,
-					'token'     => $site_token,
-					'child_key' => Settings::get_api_key(),
+					'site_url' => $site_url . $brand_site_prefix,
 				];
 				continue;
 			}
@@ -800,11 +801,8 @@ public function sync_media_files( \WP_REST_Request $request ): \WP_Error|\WP_RES
 				}
 			}
 
-			// Success response.
-			$success_response = [];
-
 			// Prepare the request to the brand site.
-			$response = wp_remote_post(
+			$response = wp_safe_remote_post(
 				$site_url . $brand_site_prefix,
 				[
 					'headers'   => [
@@ -838,8 +836,6 @@ public function sync_media_files( \WP_REST_Request $request ): \WP_Error|\WP_RES
 					'message'            => $response_body['message'] ?? __( 'Failed to sync media files.', 'onemedia' ),
 					'errors'             => $errors,
 					'site_url'           => $site_url . $brand_site_prefix,
-					'token'              => $site_token,
-					'child_key'          => Settings::get_api_key(),
 					'is_mime_type_error' => $is_mime_type_error,
 				];
 			}
@@ -911,8 +907,6 @@ public function sync_media_files( \WP_REST_Request $request ): \WP_Error|\WP_RES
 					'site'      => $site,
 					'message'   => __( 'Failed to update synced media.', 'onemedia' ),
 					'site_url'  => $site_url . $brand_site_prefix,
-					'token'     => $site_token,
-					'child_key' => Settings::get_api_key(),
 				];
 			}
 		}
@@ -1353,8 +1347,6 @@ public function update_existing_attachment( \WP_REST_Request $request ): \WP_Err
 						'site'      => $site,
 						'message'   => __( 'Invalid site URL.', 'onemedia' ),
 						'site_url'  => $site_url . $brand_site_prefix,
-						'token'     => $site_token,
-						'child_key' => Settings::get_api_key(),
 					];
 					continue;
 				}
@@ -1376,8 +1368,6 @@ public function update_existing_attachment( \WP_REST_Request $request ): \WP_Err
 						'site'      => $site,
 						'message'   => __( 'Invalid child ID data.', 'onemedia' ),
 						'site_url'  => $site_url . $brand_site_prefix,
-						'token'     => $site_token,
-						'child_key' => Settings::get_api_key(),
 					];
 					continue;
 				}
@@ -1407,7 +1397,7 @@ public function update_existing_attachment( \WP_REST_Request $request ): \WP_Err
 				];
 
 				// Prepare the request to the brand site.
-				$response = wp_remote_post(
+				$response = wp_safe_remote_post(
 					$site_url . $brand_site_prefix,
 					[
 						'headers'   => [
@@ -1443,8 +1433,6 @@ public function update_existing_attachment( \WP_REST_Request $request ): \WP_Err
 						'message'            => $response_body['message'] ?? __( 'Failed to sync media files.', 'onemedia' ),
 						'errors'             => $errors,
 						'site_url'           => $site_url . $brand_site_prefix,
-						'token'              => $site_token,
-						'child_key'          => Settings::get_api_key(),
 						'is_mime_type_error' => $is_mime_type_error,
 					];
 				}
@@ -1498,8 +1486,6 @@ public function update_existing_attachment( \WP_REST_Request $request ): \WP_Err
 							'site'      => $site,
 							'message'   => __( 'Failed to update synced media.', 'onemedia' ),
 							'site_url'  => $site_url . $brand_site_prefix,
-							'token'     => $site_token,
-							'child_key' => Settings::get_api_key(),
 						];
 						continue;
 					}