]> git.mxchange.org Git - friendica.git/blobdiff - src/Content/Item.php
Merge pull request #12976 from annando/npf2
[friendica.git] / src / Content / Item.php
index 87d719719bb6b7ae5576e495f4249e5f69dce99e..2a4bc9e3e6ae2dba54ca0668809f3608678e2eb8 100644 (file)
@@ -223,15 +223,13 @@ class Item
 
                                // select someone by nick in the current network
                                if (!DBA::isResult($contact) && ($network != '')) {
-                                       $condition = ["`nick` = ? AND `network` = ? AND `uid` = ?",
-                                               $name, $network, $profile_uid];
+                                       $condition = ['nick' => $name, 'network' => $network, 'uid' => $profile_uid];
                                        $contact = DBA::selectFirst('contact', $fields, $condition);
                                }
 
                                // select someone by attag in the current network
                                if (!DBA::isResult($contact) && ($network != '')) {
-                                       $condition = ["`attag` = ? AND `network` = ? AND `uid` = ?",
-                                               $name, $network, $profile_uid];
+                                       $condition = ['attag' => $name, 'network' => $network, 'uid' => $profile_uid];
                                        $contact = DBA::selectFirst('contact', $fields, $condition);
                                }
 
@@ -243,13 +241,13 @@ class Item
 
                                // select someone by nick in any network
                                if (!DBA::isResult($contact)) {
-                                       $condition = ["`nick` = ? AND `uid` = ?", $name, $profile_uid];
+                                       $condition = ['nick' => $name, 'uid' => $profile_uid];
                                        $contact = DBA::selectFirst('contact', $fields, $condition);
                                }
 
                                // select someone by attag in any network
                                if (!DBA::isResult($contact)) {
-                                       $condition = ["`attag` = ? AND `uid` = ?", $name, $profile_uid];
+                                       $condition = ['attag' => $name, 'uid' => $profile_uid];
                                        $contact = DBA::selectFirst('contact', $fields, $condition);
                                }
 
@@ -271,7 +269,7 @@ class Item
                                $replaced = true;
                                // create profile link
                                $profile = str_replace(',', '%2c', $profile);
-                               $newtag = $tag_type.'[url=' . $profile . ']' . $newname . '[/url]';
+                               $newtag = $tag_type . '[url=' . $profile . ']' . $newname . '[/url]';
                                $body = str_replace($tag_type . $name, $newtag, $body);
                        }
                }
@@ -295,8 +293,10 @@ class Item
                        $xmlhead = '<?xml version="1.0" encoding="UTF-8" ?>';
 
                        if ($this->activity->match($item['verb'], Activity::TAG)) {
-                               $fields = ['author-id', 'author-link', 'author-name', 'author-network',
-                                       'verb', 'object-type', 'resource-id', 'body', 'plink'];
+                               $fields = [
+                                       'author-id', 'author-link', 'author-name', 'author-network',
+                                       'verb', 'object-type', 'resource-id', 'body', 'plink'
+                               ];
                                $obj = Post::selectFirst($fields, ['uri' => $item['parent-uri']]);
                                if (!DBA::isResult($obj)) {
                                        $this->profiler->stopRecording();
@@ -333,12 +333,12 @@ class Item
                                        default:
                                                if ($obj['resource-id']) {
                                                        $post_type = $this->l10n->t('photo');
-                                                       $m=[]; preg_match("/\[url=([^]]*)\]/", $obj['body'], $m);
-                                                       $rr['plink'] = $m[1];
+                                                       preg_match("/\[url=([^]]*)\]/", $obj['body'], $matches);
+                                                       $rr['plink'] = $matches[1];
                                                } else {
                                                        $post_type = $this->l10n->t('status');
                                                }
-                                               // Let's break everthing ... ;-)
+                                               // Let's break everything ... ;-)
                                                break;
                                }
                                $plink = '[url=' . $obj['plink'] . ']' . $post_type . '[/url]';
@@ -433,7 +433,8 @@ class Item
                        }
 
                        if ((($cid == 0) || ($rel == Contact::FOLLOWER)) &&
-                               in_array($item['network'], Protocol::FEDERATED)) {
+                               in_array($item['network'], Protocol::FEDERATED)
+                       ) {
                                $menu[$this->l10n->t('Connect/Follow')] = 'contact/follow?url=' . urlencode($item['author-link']) . '&auto=1';
                        }
                } else {
@@ -547,7 +548,7 @@ class Item
                                $item['allow_cid'] = '';
                                $item['allow_gid'] = '';
                        }
-               } elseif ($setPermissions && ($item['gravity'] == ItemModel::GRAVITY_PARENT)) {
+               } elseif ($setPermissions) {
                        if (empty($receivers)) {
                                // For security reasons direct posts without any receiver will be posts to yourself
                                $self = Contact::selectFirst(['id'], ['uid' => $item['uid'], 'self' => true]);
@@ -684,11 +685,11 @@ class Item
 
                // If it is a reshared post then reformat it to avoid display problems with two share elements
                if (!empty($shared)) {
-                       if (!empty($shared['guid']) && ($encaspulated_share = $this->createSharedPostByGuid($shared['guid'], true))) {
+                       if (!empty($shared['guid']) && ($encapsulated_share = $this->createSharedPostByGuid($shared['guid'], true))) {
                                if (!empty(BBCode::fetchShareAttributes($item['body']))) {
-                                       $item['body'] = preg_replace("/\[share.*?\](.*)\[\/share\]/ism", $encaspulated_share, $item['body']);
+                                       $item['body'] = preg_replace("/\[share.*?\](.*)\[\/share\]/ism", $encapsulated_share, $item['body']);
                                } else {
-                                       $item['body'] .= $encaspulated_share;
+                                       $item['body'] .= $encapsulated_share;
                                }
                        }
                        $item['body'] = HTML::toBBCode(BBCode::convertForUriId($item['uri-id'], $item['body'], BBCode::ACTIVITYPUB));
@@ -891,7 +892,7 @@ class Item
 
        public function moveAttachmentsFromBodyToAttach(array $post): array
        {
-               if (!preg_match_all('/(\[attachment\]([0-9]+)\[\/attachment\])/',$post['body'], $match)) {
+               if (!preg_match_all('/(\[attachment\]([0-9]+)\[\/attachment\])/', $post['body'], $match)) {
                        return $post;
                }
 
@@ -903,11 +904,17 @@ class Item
                        if ($post['attach']) {
                                $post['attach'] .= ',';
                        }
-                       $post['attach'] .= Post\Media::getAttachElement($this->baseURL->get() . '/attach/' . $attachment['id'],
-                               $attachment['filesize'], $attachment['filetype'], $attachment['filename'] ?? '');
-
-                       $fields = ['allow_cid' => $post['allow_cid'], 'allow_gid' => $post['allow_gid'],
-                                       'deny_cid' => $post['deny_cid'], 'deny_gid' => $post['deny_gid']];
+                       $post['attach'] .= Post\Media::getAttachElement(
+                               $this->baseURL . '/attach/' . $attachment['id'],
+                               $attachment['filesize'],
+                               $attachment['filetype'],
+                               $attachment['filename'] ?? ''
+                       );
+
+                       $fields = [
+                               'allow_cid' => $post['allow_cid'], 'allow_gid' => $post['allow_gid'],
+                               'deny_cid' => $post['deny_cid'], 'deny_gid' => $post['deny_gid']
+                       ];
                        $condition = ['id' => $attachment_id];
                        Attach::update($fields, $condition);
                }
@@ -925,8 +932,9 @@ class Item
 
                // embedded bookmark or attachment in post? set bookmark flag
                $data = BBCode::getAttachmentData($post['body']);
-               if ((preg_match_all("/\[bookmark\=([^\]]*)\](.*?)\[\/bookmark\]/ism", $post['body'], $match, PREG_SET_ORDER) || isset($data['type']))
-                       && ($post['post-type'] != ItemModel::PT_PERSONAL_NOTE)) {
+               if ((preg_match_all("/\[bookmark\=([^\]]*)\](.*?)\[\/bookmark\]/ism", $post['body'], $match, PREG_SET_ORDER) || !empty($data['type']))
+                       && ($post['post-type'] != ItemModel::PT_PERSONAL_NOTE)
+               ) {
                        $post['post-type'] = ItemModel::PT_PAGE;
                        $post['object-type'] = Activity\ObjectType::BOOKMARK;
                }
@@ -934,34 +942,25 @@ class Item
                // Setting the object type if not defined before
                if (empty($post['object-type'])) {
                        $post['object-type'] = ($post['gravity'] == ItemModel::GRAVITY_PARENT) ? Activity\ObjectType::NOTE : Activity\ObjectType::COMMENT;
-
-                       $objectdata = BBCode::getAttachedData($post['body']);
-
-                       if ($objectdata['type'] == 'link') {
-                               $post['object-type'] = Activity\ObjectType::BOOKMARK;
-                       } elseif ($objectdata['type'] == 'video') {
-                               $post['object-type'] = Activity\ObjectType::VIDEO;
-                       } elseif ($objectdata['type'] == 'photo') {
-                               $post['object-type'] = Activity\ObjectType::IMAGE;
-                       }
                }
                return $post;
        }
 
        public function initializePost(array $post): array
        {
-               $post['network']   = Protocol::DFRN;
-               $post['protocol']  = Conversation::PARCEL_DIRECT;
-               $post['direction'] = Conversation::PUSH;
-               $post['received']  = DateTimeFormat::utcNow();
-               $post['origin']    = true;
-               $post['wall']      = $post['wall'] ?? true;
-               $post['guid']      = $post['guid'] ?? System::createUUID();
-               $post['verb']      = $post['verb'] ?? Activity::POST;
-
-               if (empty($post['uri'])) {
-                       $post['thr-parent'] = $post['uri'] = ItemModel::newURI($post['guid']);
-                       $post['gravity']    = ItemModel::GRAVITY_PARENT;
+               $post['network']    = Protocol::DFRN;
+               $post['protocol']   = Conversation::PARCEL_DIRECT;
+               $post['direction']  = Conversation::PUSH;
+               $post['received']   = DateTimeFormat::utcNow();
+               $post['origin']     = true;
+               $post['wall']       = $post['wall'] ?? true;
+               $post['guid']       = $post['guid'] ?? System::createUUID();
+               $post['verb']       = $post['verb'] ?? Activity::POST;
+               $post['uri']        = $post['uri'] ?? ItemModel::newURI($post['guid']);
+               $post['thr-parent'] = $post['thr-parent'] ?? $post['uri'];
+
+               if (empty($post['gravity'])) {
+                       $post['gravity'] = ($post['uri'] == $post['thr-parent']) ? ItemModel::GRAVITY_PARENT : ItemModel::GRAVITY_COMMENT;
                }
 
                $owner = User::getOwnerDataById($post['uid']);
@@ -1038,8 +1037,14 @@ class Item
                                continue;
                        }
 
-                       $this->emailer->send(new ItemCCEMail($this->app, $this->l10n, $this->baseURL,
-                               $post, $address, $author['thumb'] ?? ''));
+                       $this->emailer->send(new ItemCCEMail(
+                               $this->app,
+                               $this->l10n,
+                               $this->baseURL,
+                               $post,
+                               $address,
+                               $author['thumb'] ?? ''
+                       ));
                }
        }
 }