Merge pull request #11558 from annando/public-contact

[friendica.git] / src / Console / MoveToAvatarCache.php

diff --git a/src/Console/MoveToAvatarCache.php b/src/Console/MoveToAvatarCache.php
index b4684859713daeeea81a435a03be4ae22d6945b8..08c30c21530646804195436d90445e11b0a7edb8 100644 (file)
--- a/src/Console/MoveToAvatarCache.php
+++ b/src/Console/MoveToAvatarCache.php
@@ -86,55 +86,77 @@ HELP;
                $total    = $this->dba->count('contact', $condition);
                $contacts = $this->dba->select('contact', ['id', 'avatar', 'photo', 'uri-id', 'url', 'avatar'], $condition, ['order' => ['id']]);
                while ($contact = $this->dba->fetch($contacts)) {
-                       $valid = true;
-                       echo ++$count . '/' . $total . "\t" . $contact['id'] . "\t" . $contact['url'] . "\t";
+                       $this->out(++$count . '/' . $total . "\t" . $contact['id'] . "\t" . $contact['url'] . "\t", false);
                        $resourceid = Photo::ridFromURI($contact['photo']);
                        if (empty($resourceid)) {
-                               echo $this->l10n->t('no resource in photo %s', $contact['photo']) . "\n";
-                               $valid = false;
+                               $this->out($this->l10n->t('no resource in photo %s', $contact['photo']) . ' ', false);
                        }
 
-                       if ($valid) {
-                               echo '1';
-                               $photo = Photo::selectFirst([], ['resource-id' => $resourceid], ['order' => ['scale']]);
-                               if (empty($photo)) {
-                                       echo ' ' . $this->l10n->t('no photo with id %s', $resourceid) . "\n";
-                                       $valid = false;
-                               }
+                       $this->storeAvatar($resourceid, $contact, false);
+               }
+
+               $count  = 0;
+               $totals = $this->dba->p("SELECT COUNT(DISTINCT(`resource-id`)) AS `total` FROM `photo` WHERE `contact-id` != ? AND `photo-type` = ?;", 0, Photo::CONTACT_AVATAR);
+               $total  = $this->dba->fetch($totals)['total'] ?? 0;
+               $photos = $this->dba->p("SELECT `resource-id`, MAX(`contact-id`) AS `contact-id` FROM `photo` WHERE `contact-id` != ? AND `photo-type` = ? GROUP BY `resource-id`;", 0, Photo::CONTACT_AVATAR);
+               while ($photo = $this->dba->fetch($photos)) {
+                       $contact = Contact::getById($photo['contact-id'], ['id', 'avatar', 'photo', 'uri-id', 'url', 'avatar']);
+                       if (empty($contact)) {
+                               continue;
                        }
+                       $this->out(++$count . '/' . $total . "\t" . $contact['id'] . "\t" . $contact['url'] . "\t", false);
+                       $this->storeAvatar($photo['resource-id'], $contact, true);
+               }
+               return 0;
+       }
 
-                       if ($valid) {
-                               echo '2';
-                               $imgdata = Photo::getImageDataForPhoto($photo);
-                               if (empty($imgdata)) {
-                                       echo ' ' . $this->l10n->t('no image data for photo with id %s', $resourceid) . "\n";
-                                       $valid = false;
-                               }
+       private function storeAvatar(string $resourceid, array $contact, bool $quit_on_invalid)
+       {
+               $valid = !empty($resourceid);
+               if ($valid) {
+                       $this->out('1', false);
+                       $photo = Photo::selectFirst([], ['resource-id' => $resourceid], ['order' => ['scale']]);
+                       if (empty($photo)) {
+                               $this->out(' ' . $this->l10n->t('no photo with id %s', $resourceid) . ' ', false);
+                               $valid = false;
                        }
+               }
 
-                       if ($valid) {
-                               echo '3';
-                               $image = new Image($imgdata, Images::getMimeTypeByData($imgdata));
-                               if (!$image->isValid()) {
-                                       echo ' ' . $this->l10n->t('invalid image for id %s', $resourceid) . "\n";
-                                       $valid = false;
-                               }
+               if ($valid) {
+                       $this->out('2', false);
+                       $imgdata = Photo::getImageDataForPhoto($photo);
+                       if (empty($imgdata)) {
+                               $this->out(' ' . $this->l10n->t('no image data for photo with id %s', $resourceid) . ' ', false);
+                               $valid = false;
                        }
+               }
 
-                       if ($valid) {
-                               echo '4';
-                               $fields = Avatar::storeAvatarByImage($contact, $image);
-                       } else {
-                               $fields = ['photo' => '', 'thumb' => '', 'micro' => ''];
+               if ($valid) {
+                       $this->out('3', false);
+                       $image = new Image($imgdata, Images::getMimeTypeByData($imgdata));
+                       if (!$image->isValid()) {
+                               $this->out(' ' . $this->l10n->t('invalid image for id %s', $resourceid) . ' ', false);
+                               $valid = false;
                        }
+               }
+
+               if ($valid) {
+                       $this->out('4', false);
+                       $fields = Avatar::storeAvatarByImage($contact, $image);
+               } else {
+                       $fields = ['photo' => '', 'thumb' => '', 'micro' => ''];
+               }
 
-                       echo '5';
-                       Contact::update($fields, ['uri-id' => $contact['uri-id']]);
-                       echo '6';
+               if ($quit_on_invalid && $fields['photo'] == '') {
+                       $this->out(' ' . $this->l10n->t('Quit on invalid photo %s', $contact['avatar']));
                        Photo::delete(['resource-id' => $resourceid]);
-                       echo ' ' . $fields['photo'] . "\n";
+                       return;
                }
 
-               return 0;
+               $this->out('5', false);
+               Contact::update($fields, ['uri-id' => $contact['uri-id']]);
+               $this->out('6', false);
+               Photo::delete(['resource-id' => $resourceid]);
+               $this->out(' ' . $fields['photo']);
        }
 }