Remove unused notice

av3nger · Nov 10, 2023 · a9bb8b1 · a9bb8b1
1 parent 6dd09f4
commit a9bb8b1
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 62 deletions.
diff --git a/app/class-admin.php b/app/class-admin.php
@@ -53,7 +53,6 @@ public function __construct() {
 		add_action( 'admin_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
 
 		add_action( 'admin_menu', array( $this, 'register_menu' ) );
-		add_action( 'admin_notices', array( $this, 'show_notice' ) );
 		add_filter( 'plugin_action_links_cf-images/cf-images.php', array( $this, 'settings_link' ) );
 
 		if ( wp_doing_ajax() ) {
@@ -173,43 +172,6 @@ public function register_menu() {
 		);
 	}
 
-	/**
-	 * Show notice.
-	 *
-	 * @since 1.0.0
-	 */
-	public function show_notice() {
-		if ( false !== $this->get_error() ) {
-			$message = sprintf( /* translators: %1$s - error message, %2$d - error code */
-				esc_html__( '%1$s (code: %2$d)', 'cf-images' ),
-				esc_html( $this->get_error()->get_error_message() ),
-				(int) $this->get_error()->get_error_code()
-			);
-
-			$this->render_notice( $message, 'error' );
-		}
-	}
-
-	/**
-	 * Render notice.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @param string $message Notice message.
-	 * @param string $type    Notice type.
-	 */
-	public function render_notice( string $message, string $type = 'success' ) {
-		?>
-		<div class="cf-images-notifications">
-			<div class="notice notice-<?php echo esc_attr( $type ); ?>" id="cf-images-notice">
-				<p>
-					<?php echo esc_html( $message ); ?>
-				</p>
-			</div>
-		</div>
-		<?php
-	}
-
 	/**
 	 * Render page.
 	 *

diff --git a/assets/_src/js/core.js b/assets/_src/js/core.js
@@ -9,7 +9,7 @@
 
 import { post } from './helpers/post';
 
-( function( $ ) {
+(function ($) {
 	'use strict';
 
 	/**
@@ -28,38 +28,31 @@ import { post } from './helpers/post';
 		cf_images_compress: '.cf-images-compress', // Compress image.
 	};
 
-	/**
-	 * Auto hide any pending notices.
-	 *
-	 * @since 1.0.0
-	 */
-	$( document ).ready( function() {
-		setTimeout( () => $( '#cf-images-notice' ).slideUp( 'slow' ), 5000 );
-	} );
-
 	/**
 	 * Register Ajax actions.
 	 */
-	Object.keys( ajaxActions ).forEach( ( action ) => {
-		$( document ).on( 'click', ajaxActions[ action ], function( e ) {
+	Object.keys(ajaxActions).forEach((action) => {
+		$(document).on('click', ajaxActions[action], function (e) {
 			e.preventDefault();
 
 			const { inProgress, offloadError } = CFImages.strings;
 
-			const divStatus = $( this ).closest( '.cf-images-status' );
-			divStatus.html( inProgress + '<span class="spinner is-active"></span>' );
+			const divStatus = $(this).closest('.cf-images-status');
+			divStatus.html(
+				inProgress + '<span class="spinner is-active"></span>'
+			);
 
-			post( action, $( this ).data( 'id' ) )
-				.then( ( response ) => {
-					if ( ! response.success ) {
+			post(action, $(this).data('id'))
+				.then((response) => {
+					if (!response.success) {
 						const message = response.data || offloadError;
-						divStatus.html( message );
+						divStatus.html(message);
 						return;
 					}
 
-					divStatus.html( response.data );
-				} )
-				.catch( window.console.log );
-		} );
-	} );
-}( jQuery ) );
+					divStatus.html(response.data);
+				})
+				.catch(window.console.log);
+		});
+	});
+})(jQuery);