Only log when there is data

diff --git a/src/Model/GContact.php b/src/Model/GContact.php
index dc62ec460f7120dc54887fb0f063635685d362a7..a9233e8cf29ae0d7927d1e36617e9baaa06b2fee 100644 (file)
--- a/src/Model/GContact.php
+++ b/src/Model/GContact.php
@@ -1288,28 +1288,32 @@ class GContact
 
                if (!empty($apcontact['followers']) && is_string($apcontact['followers'])) {
                        $followers = ActivityPub::fetchItems($apcontact['followers']);
-                       Logger::info('Discover AP followers', ['url' => $url, 'contacts' => count($followers)]);
-                       foreach ($followers as $follower) {
-                               if (DBA::exists('gcontact', ['nurl' => Strings::normaliseLink(($follower))])) {
-                                       continue;
+                       if (!empty($followers)) {
+                               Logger::info('Discover AP followers', ['url' => $url, 'contacts' => count($followers)]);
+                               foreach ($followers as $follower) {
+                                       if (DBA::exists('gcontact', ['nurl' => Strings::normaliseLink(($follower))])) {
+                                               continue;
+                                       }
+                                       Logger::info('Discover new AP contact', ['url' => $follower]);
+                                       Worker::add(PRIORITY_LOW, 'UpdateGContact', $follower);
                                }
-                               Logger::info('Discover new AP contact', ['url' => $follower]);
-                               Worker::add(PRIORITY_LOW, 'UpdateGContact', $follower);
+                               Logger::info('AP followers discovery finished', ['url' => $url]);
                        }
-                       Logger::info('AP followers discovery finished', ['url' => $url]);
                }
 
                if (!empty($apcontact['following']) && is_string($apcontact['following'])) {
                        $followings = ActivityPub::fetchItems($apcontact['following']);
-                       Logger::info('Discover AP followings', ['url' => $url, 'contacts' => count($followings)]);
-                       foreach ($followings as $following) {
-                               if (DBA::exists('gcontact', ['nurl' => Strings::normaliseLink(($following))])) {
-                                       continue;
+                       if (!empty($followings)) {
+                               Logger::info('Discover AP followings', ['url' => $url, 'contacts' => count($followings)]);
+                               foreach ($followings as $following) {
+                                       if (DBA::exists('gcontact', ['nurl' => Strings::normaliseLink(($following))])) {
+                                               continue;
+                                       }
+                                       Logger::info('Discover new AP contact', ['url' => $following]);
+                                       Worker::add(PRIORITY_LOW, 'UpdateGContact', $following);
                                }
-                               Logger::info('Discover new AP contact', ['url' => $following]);
-                               Worker::add(PRIORITY_LOW, 'UpdateGContact', $following);
+                               Logger::info('AP followings discovery finished', ['url' => $url]);
                        }
-                       Logger::info('AP followings discovery finished', ['url' => $url]);
                }