Wrong "use" for config

[friendica.git] / src / Database / PostUpdate.php

diff --git a/src/Database/PostUpdate.php b/src/Database/PostUpdate.php
index e4bfa710aa3ee8e359a819cb85de5cf835de82f1..4ce89d49e5b67c1f6d97c51c2f30d7c1455e20a2 100644 (file)
--- a/src/Database/PostUpdate.php
+++ b/src/Database/PostUpdate.php
@@ -5,9 +5,9 @@
 namespace Friendica\Database;
 
 use Friendica\Core\Config;
-use Friendica\Database\DBM;
 use Friendica\Model\Contact;
-use dba;
+use Friendica\Model\Item;
+use Friendica\Model\PermissionSet;
 
 require_once 'include/dba.php';
 
@@ -30,6 +30,9 @@ class PostUpdate
                if (!self::update1206()) {
                        return;
                }
+               if (!self::update1279()) {
+                       return;
+               }
        }
 
        /**
@@ -120,7 +123,8 @@ class PostUpdate
                logger("Start", LOGGER_DEBUG);
 
                // Check if the first step is done (Setting "author-id" and "owner-id" in the item table)
-               $r = dba::select('item', ['author-link', 'owner-link', 'uid'], ['author-id' => 0, 'owner-id' => 0], ['limit' => 1000]);
+               $fields = ['author-link', 'author-name', 'author-avatar', 'owner-link', 'owner-name', 'owner-avatar', 'network', 'uid'];
+               $r = dba::select('item', $fields, ['author-id' => 0, 'owner-id' => 0], ['limit' => 1000]);
                if (!$r) {
                        // Are there unfinished entries in the thread table?
                        $r = q("SELECT COUNT(*) AS `total` FROM `thread`
@@ -162,8 +166,13 @@ class PostUpdate
 
                // Set the "author-id" and "owner-id" in the item table and add a new public contact entry if needed
                foreach ($item_arr as $item) {
-                       $author_id = Contact::getIdForURL($item["author-link"]);
-                       $owner_id = Contact::getIdForURL($item["owner-link"]);
+                       $default = ['url' => $item['author-link'], 'name' => $item['author-name'],
+                               'photo' => $item['author-avatar'], 'network' => $item['network']];
+                       $author_id = Contact::getIdForURL($item["author-link"], 0, false, $default);
+
+                       $default = ['url' => $item['owner-link'], 'name' => $item['owner-name'],
+                               'photo' => $item['owner-avatar'], 'network' => $item['network']];
+                       $owner_id = Contact::getIdForURL($item["owner-link"], 0, false, $default);
 
                        if ($author_id == 0) {
                                $author_id = -1;
@@ -211,4 +220,80 @@ class PostUpdate
                logger("Done", LOGGER_DEBUG);
                return true;
        }
+
+       /**
+        * @brief update the item related tables
+        *
+        * @return bool "true" when the job is done
+        */
+       private static function update1279()
+       {
+               // Was the script completed?
+               if (Config::get("system", "post_update_version") >= 1279) {
+                       return true;
+               }
+
+               $id = Config::get("system", "post_update_version_1279_id", 0);
+
+               logger("Start from item " . $id, LOGGER_DEBUG);
+
+               $fields = array_merge(Item::MIXED_CONTENT_FIELDLIST, ['network', 'author-id', 'owner-id', 'tag', 'file',
+                       'author-name', 'author-avatar', 'author-link', 'owner-name', 'owner-avatar', 'owner-link', 'id',
+                       'uid', 'allow_cid', 'allow_gid', 'deny_cid', 'deny_gid', 'psid', 'post-type', 'bookmark', 'type',
+                       'inform']);
+
+               $start_id = $id;
+               $rows = 0;
+               $condition = ["`id` > ?", $id];
+               $params = ['order' => ['id'], 'limit' => 10000];
+               $items = Item::select($fields, $condition, $params);
+               while ($item = Item::fetch($items)) {
+                       $id = $item['id'];
+
+                       if (empty($item['author-id'])) {
+                               $default = ['url' => $item['author-link'], 'name' => $item['author-name'],
+                                       'photo' => $item['author-avatar'], 'network' => $item['network']];
+
+                               $item['author-id'] = Contact::getIdForURL($item["author-link"], 0, false, $default);
+                       }
+
+                       if (empty($item['owner-id'])) {
+                               $default = ['url' => $item['owner-link'], 'name' => $item['owner-name'],
+                                       'photo' => $item['owner-avatar'], 'network' => $item['network']];
+
+                               $item['owner-id'] = Contact::getIdForURL($item["owner-link"], 0, false, $default);
+                       }
+
+                       if (empty($item['psid'])) {
+                               $item['psid'] = PermissionSet::fetchIDForPost($item);
+                       }
+
+                       if ($item['post-type'] == 0) {
+                               if (!empty($item['type']) && ($item['type'] == 'note')) {
+                                       $item['post-type'] = Item::PT_PERSONAL_NOTE;
+                               } elseif (!empty($item['type']) && ($item['type'] == 'photo')) {
+                                       $item['post-type'] = Item::PT_IMAGE;
+                               } elseif (!empty($item['bookmark']) && $item['bookmark']) {
+                                       $item['post-type'] = Item::PT_PAGE;
+                               }
+                       }
+
+                       Item::update($item, ['id' => $id]);
+
+                       ++$rows;
+               }
+               dba::close($items);
+
+               Config::set("system", "post_update_version_1279_id", $id);
+
+               logger("Processed rows: " . $rows . " - last processed item:  " . $id, LOGGER_DEBUG);
+
+               if ($start_id == $id) {
+                       Config::set("system", "post_update_version", 1279);
+                       logger("Done", LOGGER_DEBUG);
+                       return true;
+               }
+
+               return false;
+       }
 }