Merge pull request #22 from newfold-labs/update/one-notice-only

Only render one notice (the latest) in wp admin
newfold-labs · Apr 25, 2024 · 3c8a64c · 3c8a64c
2 parents 0f90aef + 475da95
commit 3c8a64c
Showing 1 changed file with 10 additions and 10 deletions.
diff --git a/includes/AdminNotices.php b/includes/AdminNotices.php
@@ -46,17 +46,17 @@ public static function maybeRenderAdminNotices() {
 		self::openContainer();
 
 		if ( $collection->count() ) {
-			$collection->each(
-				function ( Notification $notification ) use ( $page ) {
-					if ( $notification->shouldShow( 'wp-admin-notice', array( 'page' => $page ) ) ) {
-						?>
-                        <div class="newfold-notice" data-id="<?php echo esc_attr( $notification->id ); ?>">
-							<?php echo $notification->content; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped ?>
-                        </div>
-						<?php
-					}
+			// Only show the latest notification that should be shown on this page
+			foreach ( $collection as $notification ) {
+				if ( $notification->shouldShow( 'wp-admin-notice', array( 'page' => $page ) ) ) {
+					?>
+					<div class="newfold-notice" data-id="<?php echo esc_attr( $notification->id ); ?>">
+						<?php echo $notification->content; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped ?>
+					</div>
+					<?php
+					break;
 				}
-			);
+			}
 		}
 
 		self::closeContainer();