refactor: notification synchronizer

resources/templates/sync/actions.php

@@ -16,18 +16,10 @@
 ?>
 
 <div class="button-group">
-	<a
-		href="#"
-		class="button button-secondary button-icon notification-sync-all"
-		data-type="json"
-	>
+	<a href="#" class="button button-secondary button-icon notification-sync-all" data-type="json">
 		<span class="dashicons dashicons-download"></span> <?php esc_html_e('Save all to JSON'); ?>
 	</a>
-	<a
-		href="#"
-		class="button button-secondary button-icon notification-sync-all"
-		data-type="wordpress"
-	>
+	<a href="#" class="button button-secondary button-icon notification-sync-all" data-type="wordpress">
 		<span class="dashicons dashicons-upload"></span> <?php esc_html_e('Load all to WordPress'); ?>
 	</a>
 </div>

resources/templates/sync/disabled.php

@@ -18,8 +18,8 @@
 	// Translators: %s Function name.
 	__(
 		sprintf(
-			'Synchronization is disabled. You can enable it with %s function',
-			'<code>notification_sync()</code>'
+			'Synchronization is disabled. You can enable it with %s',
+			'<code>\BracketSpace\Notification\Core\Sync::enable()</code>'
 		),
 		'notification'
 	)

resources/templates/sync/notifications-empty.php

@@ -13,4 +13,4 @@
  * @var \BracketSpace\Notification\Dependencies\Micropackage\Templates\Template $this Template instance.
  */
 
-esc_html_e('All Notifications are synchronized and up to date.');
+esc_html_e('All Notifications are synchronized and up to date.', 'notification');

resources/templates/sync/notifications.php

@@ -29,23 +29,12 @@
 		<tr>
 			<td class="title">
 				<?php if (isset($item['post_id'])) : ?>
-				<a
-					href="
-					<?php
-					echo esc_url(
-						(string)get_edit_post_link(
-							$item['post_id'],
-							'admin'
-						)
-					);
-					?>
-					"
-				>
+					<a href="<?php echo esc_url((string)get_edit_post_link($item['post_id'], 'admin')); ?>">
 				<?php endif ?>
 					<?php echo esc_html($item['notification']->getTitle()); ?>
-					<?php if (isset($item['post_id'])) : ?>
-				</a>
-					<?php endif ?>
+				<?php if (isset($item['post_id'])) : ?>
+					</a>
+				<?php endif ?>
 			</td>
 			<td class="hash">
 				<code><?php echo esc_html($hash); ?></code>

src/Admin/Sync.php

@@ -48,11 +48,10 @@ public function settings($settings)
 				],
 				'render' => [new CoreFields\Message(), 'input'],
 				'sanitize' => [new CoreFields\Message(), 'sanitize'],
-				'description' => __('Bulk actions for the table below.'),
 			]
 		);
 
-		if (!CoreSync::isSyncing()) {
+		if (! CoreSync::isSyncing()) {
 			return;
 		}
 

src/Core/Sync.php

@@ -37,7 +37,7 @@ class Sync
 	 */
 	public static function getAllJson()
 	{
-		if (!self::isSyncing()) {
+		if (! self::isSyncing()) {
 			return [];
 		}
 
@@ -87,7 +87,7 @@ public function loadLocalJson()
 				$notification = Notification::from('json', $json);
 
 				if ($notification->isEnabled()) {
-					Register::notification($notification);
+					Register::notificationIfNewer($notification);
 				}
 			} catch (\Throwable $e) {
 				// Do nothing.
@@ -170,7 +170,7 @@ public function deleteLocalJson($postId)
 	//phpcs:ignore SlevomatCodingStandard.TypeHints.NullableTypeForNullDefaultValue.NullabilitySymbolRequired
 	public static function enable(string $path = null)
 	{
-		if (!$path) {
+		if (! $path) {
 			$path = trailingslashit(get_stylesheet_directory()) . 'notifications';
 		}
 
@@ -187,11 +187,11 @@ public static function enable(string $path = null)
 
 		$fs = static::getSyncFs();
 
-		if (!$fs) {
+		if (! $fs) {
 			return;
 		}
 
-		if (!$fs->exists('') || !$fs->is_dir('')) {
+		if (! $fs->exists('') || ! $fs->is_dir('')) {
 			$fs->mkdir('');
 		}
 
@@ -200,10 +200,7 @@ public static function enable(string $path = null)
 		}
 
 		$fs->touch('index.php');
-		$fs->put_contents(
-			'index.php',
-			'<?php' . "\r\n" . '// Keep this file here.' . "\r\n"
-		);
+		$fs->put_contents('index.php', '<?php' . "\r\n" . '// Keep this file here.' . "\r\n");
 	}
 
 	/**
@@ -236,7 +233,7 @@ public static function getSyncPath()
 	 */
 	public static function getSyncFs()
 	{
-		if (!static::isSyncing()) {
+		if (! static::isSyncing()) {
 			return null;
 		}
 

src/Integration/WordPressIntegration.php

@@ -52,7 +52,7 @@ public function loadDatabaseNotifications()
 		 */
 		$notifications = $cache->collect(static fn() => NotificationDatabaseService::getAll());
 
-		array_map([Register::class, 'notification'], $notifications);
+		array_map([Register::class, 'notificationIfNewer'], $notifications);
 	}
 
 	/**

src/Utils/Settings/Fields/SyncTable.php

@@ -64,13 +64,7 @@ public function input($field)
 
 				$wpNotification = $collection[$notification->getHash()]['notification'];
 
-				if (
-					version_compare(
-						(string)$wpNotification->getVersion(),
-						(string)$notification->getVersion(),
-						'>='
-					)
-				) {
+				if (version_compare((string)$wpNotification->getVersion(), (string)$notification->getVersion(), '>=')) {
 					$collection[$notification->getHash()]['up_to_date'] = true;
 				}
 			} else {
@@ -85,7 +79,7 @@ public function input($field)
 
 		// Filter synchronized.
 		foreach ($collection as $key => $data) {
-			if (!$data['up_to_date']) {
+			if (! $data['up_to_date']) {
 				continue;
 			}