Merge pull request #4565 from pixelfed/staging

Staging

CHANGELOG.md

@@ -46,6 +46,8 @@
 - Update ProfileController, allow albums in atom feed. Closes #4561. Fixes #4526 ([1c105a6c](https://github.com/pixelfed/pixelfed/commit/1c105a6c))
 - Update admin users view, fix website value. Closes #4557 ([c469d475](https://github.com/pixelfed/pixelfed/commit/c469d475))
 - Update StatusStatelessTransformer, allow unlisted reblogs ([1c13b518](https://github.com/pixelfed/pixelfed/commit/1c13b518))
+- Update ApiV1Controller, hydrate reblog state in home timeline ([13bdaa2e](https://github.com/pixelfed/pixelfed/commit/13bdaa2e))
+- Update Timeline component, improve reblog support ([29de91e5](https://github.com/pixelfed/pixelfed/commit/29de91e5))
 -  ([](https://github.com/pixelfed/pixelfed/commit/))
 
 ## [v0.11.8 (2023-05-29)](https://github.com/pixelfed/pixelfed/compare/v0.11.7...v0.11.8)

app/Http/Controllers/Api/ApiV1Controller.php

@@ -2199,6 +2199,14 @@ class ApiV1Controller extends Controller
 			->filter(function($status) {
 				return $status && isset($status['account']);
 			})
+            ->map(function($status) use($pid) {
+                if(!empty($status['reblog'])) {
+                    $status['reblog']['favourited'] = (bool) LikeService::liked($pid, $status['reblog']['id']);
+                    $status['reblog']['reblogged'] = (bool) ReblogService::get($pid, $status['reblog']['id']);
+                }
+
+                return $status;
+            })
 			->take($limit)
 			->values();
 		} else {
@@ -2242,6 +2250,14 @@ class ApiV1Controller extends Controller
 			->filter(function($status) {
 				return $status && isset($status['account']);
 			})
+            ->map(function($status) use($pid) {
+                if(!empty($status['reblog'])) {
+                    $status['reblog']['favourited'] = (bool) LikeService::liked($pid, $status['reblog']['id']);
+                    $status['reblog']['reblogged'] = (bool) ReblogService::get($pid, $status['reblog']['id']);
+                }
+
+                return $status;
+            })
 			->take($limit)
 			->values();
 		}

app/Jobs/SharePipeline/SharePipeline.php

@@ -49,7 +49,10 @@ class SharePipeline implements ShouldQueue
 	public function handle()
 	{
 		$status = $this->status;
-		$parent = $this->status->parent();
+		$parent = Status::find($this->status->reblog_of_id);
+        if(!$parent) {
+            return;
+        }
 		$actor = $status->profile;
 		$target = $parent->profile;
 
@@ -84,7 +87,7 @@ class SharePipeline implements ShouldQueue
 
 	public function remoteAnnounceDeliver()
 	{
-		if(config_cache('federation.activitypub.enabled') == false) {
+		if(config('app.env') !== 'production' || config_cache('federation.activitypub.enabled') == false) {
 			return true;
 		}
 		$status = $this->status;

app/Jobs/SharePipeline/UndoSharePipeline.php

@@ -61,7 +61,7 @@ class UndoSharePipeline implements ShouldQueue
 			return;
 		}
 
-		if(config_cache('federation.activitypub.enabled') == false) {
+		if(config('app.env') !== 'production' || config_cache('federation.activitypub.enabled') == false) {
 			return $status->delete();
 		} else {
 			return $this->remoteAnnounceDeliver();
@@ -70,7 +70,8 @@ class UndoSharePipeline implements ShouldQueue
 
 	public function remoteAnnounceDeliver()
 	{
-		if(config_cache('federation.activitypub.enabled') == false) {
+		if(config('app.env') !== 'production' || config_cache('federation.activitypub.enabled') == false) {
+            $status->delete();
 			return 1;
 		}
 

resources/assets/components/partials/TimelineStatus.vue

@@ -3,18 +3,20 @@
         <div class="card shadow-sm" style="border-radius: 15px;">
             <post-header
                 :profile="profile"
-                :status="status"
+                :status="shadowStatus"
+                :is-reblog="isReblog"
+                :reblog-account="reblogAccount"
                 @menu="openMenu"
                 @follow="follow"
                 @unfollow="unfollow" />
 
             <post-content
                 :profile="profile"
-                :status="status" />
+                :status="shadowStatus" />
 
             <post-reactions
             	v-if="reactionBar"
-                :status="status"
+                :status="shadowStatus"
                 :profile="profile"
                 :admin="admin"
                 v-on:like="like"
@@ -29,7 +31,7 @@
 
             <div v-if="showCommentDrawer" class="card-footer rounded-bottom border-0" style="background: rgba(0,0,0,0.02);z-index: 3;">
                 <comment-drawer
-                    :status="status"
+                    :status="shadowStatus"
                     :profile="profile"
                     v-on:handle-report="handleReport"
                     v-on:counter-change="counterChange"
@@ -90,14 +92,14 @@
         },
 
         mounted() {
-            this.license = this.status.media_attachments && this.status.media_attachments.length ?
+            this.license = this.shadowStatus.media_attachments && this.shadowStatus.media_attachments.length ?
-                this.status
+                this.shadowStatus
                 .media_attachments
                 .filter(m => m.hasOwnProperty('license') && m.license && m.license.hasOwnProperty('id'))
                 .map(m => m.license)[0] : false;
             this.admin = window._sharedData.user.is_admin;
-            this.owner = this.status.account.id == window._sharedData.user.id;
+            this.owner = this.shadowStatus.account.id == window._sharedData.user.id;
-            if(this.status.reply_count && this.autoloadComments && this.status.comments_disabled === false) {
+            if(this.shadowStatus.reply_count && this.autoloadComments && this.shadowStatus.comments_disabled === false) {
                 setTimeout(() => {
                     this.showCommentDrawer = true;
                 }, 1000);
@@ -127,6 +129,24 @@
                 get() {
                     return this.$store.state.newReactions;
                 },
+            },
+
+            isReblog: {
+                get() {
+                    return this.status.reblog != null;
+                }
+            },
+
+            reblogAccount: {
+                get() {
+                    return this.status.reblog ? this.status.account : null;
+                }
+            },
+
+            shadowStatus: {
+                get() {
+                    return this.status.reblog ? this.status.reblog : this.status;
+                }
             }
         },
 
@@ -137,7 +157,7 @@
                 handler: function(o, n) {
                     this.isBookmarking = false;
                 }
-            }
+            },
         },
 
         methods: {

resources/assets/components/sections/Timeline.vue

@@ -361,17 +361,30 @@
 
             likeStatus(index) {
                 let status = this.feed[index];
+                if(status.reblog) {
+                    status = status.reblog;
+                    let state = status.favourited;
+                    let count = status.favourites_count;
+                    this.feed[index].reblog.favourites_count = count + 1;
+                    this.feed[index].reblog.favourited = !status.favourited;
+                } else {
                     let state = status.favourited;
                     let count = status.favourites_count;
                     this.feed[index].favourites_count = count + 1;
                     this.feed[index].favourited = !status.favourited;
+                }
 
                 axios.post('/api/v1/statuses/' + status.id + '/favourite')
                 .then(res => {
                     //
                 }).catch(err => {
+                    if(status.reblog) {
+                        this.feed[index].reblog.favourites_count = count;
+                        this.feed[index].reblog.favourited = false;
+                    } else {
                         this.feed[index].favourites_count = count;
                         this.feed[index].favourited = false;
+                    }
 
                     let el = document.createElement('p');
                     el.classList.add('text-left');
@@ -413,17 +426,30 @@
 
             unlikeStatus(index) {
                 let status = this.feed[index];
+                if(status.reblog) {
+                    status = status.reblog;
+                    let state = status.favourited;
+                    let count = status.favourites_count;
+                    this.feed[index].reblog.favourites_count = count - 1;
+                    this.feed[index].reblog.favourited = !status.favourited;
+                } else {
                     let state = status.favourited;
                     let count = status.favourites_count;
                     this.feed[index].favourites_count = count - 1;
                     this.feed[index].favourited = !status.favourited;
+                }
 
                 axios.post('/api/v1/statuses/' + status.id + '/unfavourite')
                 .then(res => {
                     //
                 }).catch(err => {
+                    if(status.reblog && status.pf_type == 'share') {
+                        this.feed[index].reblog.favourites_count = count;
+                        this.feed[index].reblog.favourited = false;
+                    } else {
                         this.feed[index].favourites_count = count;
                         this.feed[index].favourited = false;
+                    }
                 })
             },
 
@@ -445,7 +471,8 @@
 
             openLikesModal(idx) {
                 this.postIndex = idx;
-                this.likesModalPost = this.feed[this.postIndex];
+                let post = this.feed[this.postIndex];
+                this.likesModalPost = post.reblog ? post.reblog : post;
                 this.showLikesModal = true;
                 this.$nextTick(() => {
                     this.$refs.likesModal.open();
@@ -454,7 +481,8 @@
 
             openSharesModal(idx) {
                 this.postIndex = idx;
-                this.sharesModalPost = this.feed[this.postIndex];
+                let post = this.feed[this.postIndex];
+                this.sharesModalPost = post.reblog ? post.reblog : post;
                 this.showSharesModal = true;
                 this.$nextTick(() => {
                     this.$refs.sharesModal.open();
@@ -492,19 +520,32 @@
             },
 
             counterChange(index, type) {
+                let post = this.feed[index];
                 switch(type) {
                     case 'comment-increment':
+                        if(post.reblog != null) {
+                            this.feed[index].reblog.reply_count = this.feed[index].reblog.reply_count + 1;
+                        } else {
                             this.feed[index].reply_count = this.feed[index].reply_count + 1;
+                        }
                     break;
 
                     case 'comment-decrement':
+                        if(post.reblog != null) {
+                            this.feed[index].reblog.reply_count = this.feed[index].reblog.reply_count - 1;
+                        } else {
                             this.feed[index].reply_count = this.feed[index].reply_count - 1;
+                        }
                     break;
                 }
             },
 
             openCommentLikesModal(post) {
+                if(post.reblog != null) {
+                    this.likesModalPost = post.reblog;
+                } else {
                     this.likesModalPost = post;
+                }
                 this.showLikesModal = true;
                 this.$nextTick(() => {
                     this.$refs.likesModal.open();
@@ -513,33 +554,59 @@
 
             shareStatus(index) {
                 let status = this.feed[index];
+                if(status.reblog) {
+                    status = status.reblog;
+                    let state = status.reblogged;
+                    let count = status.reblogs_count;
+                    this.feed[index].reblog.reblogs_count = count + 1;
+                    this.feed[index].reblog.reblogged = !status.reblogged;
+                } else {
                     let state = status.reblogged;
                     let count = status.reblogs_count;
                     this.feed[index].reblogs_count = count + 1;
                     this.feed[index].reblogged = !status.reblogged;
+                }
 
                 axios.post('/api/v1/statuses/' + status.id + '/reblog')
                 .then(res => {
                     //
                 }).catch(err => {
+                    if(status.reblog) {
+                        this.feed[index].reblog.reblogs_count = count;
+                        this.feed[index].reblog.reblogged = false;
+                    } else {
                         this.feed[index].reblogs_count = count;
                         this.feed[index].reblogged = false;
+                    }
                 })
             },
 
             unshareStatus(index) {
                 let status = this.feed[index];
+                if(status.reblog) {
+                    status = status.reblog;
+                    let state = status.reblogged;
+                    let count = status.reblogs_count;
+                    this.feed[index].reblog.reblogs_count = count - 1;
+                    this.feed[index].reblog.reblogged = !status.reblogged;
+                } else {
                     let state = status.reblogged;
                     let count = status.reblogs_count;
                     this.feed[index].reblogs_count = count - 1;
                     this.feed[index].reblogged = !status.reblogged;
+                }
 
                 axios.post('/api/v1/statuses/' + status.id + '/unreblog')
                 .then(res => {
                     //
                 }).catch(err => {
+                    if(status.reblog) {
+                        this.feed[index].reblog.reblogs_count = count;
+                        this.feed[index].reblog.reblogged = false;
+                    } else {
                         this.feed[index].reblogs_count = count;
                         this.feed[index].reblogged = false;
+                    }
                 })
             },
 
@@ -554,11 +621,19 @@
             handleBookmark(index) {
                 let p = this.feed[index];
 
+                if(p.reblog) {
+                    p = p.reblog;
+                }
+
                 axios.post('/i/bookmark', {
                     item: p.id
                 })
                 .then(res => {
+                    if(this.feed[index].reblog) {
+                        this.feed[index].reblog.bookmarked = !p.bookmarked;
+                    } else {
                         this.feed[index].bookmarked = !p.bookmarked;
+                    }
                 })
                 .catch(err => {
                     // this.feed[index].bookmarked = false;
@@ -571,8 +646,17 @@
             },
 
             follow(index) {
-                // this.feed[index].relationship.following = true;
+                if(this.feed[index].reblog) {
-
+                    axios.post('/api/v1/accounts/' + this.feed[index].reblog.account.id + '/follow')
+                    .then(res => {
+                        this.$store.commit('updateRelationship', [res.data]);
+                        this.updateProfile({ following_count: this.profile.following_count + 1 });
+                        this.feed[index].reblog.account.followers_count = this.feed[index].reblog.account.followers_count + 1;
+                    }).catch(err => {
+                        swal('Oops!', 'An error occured when attempting to follow this account.', 'error');
+                        this.feed[index].reblog.relationship.following = false;
+                    });
+                } else {
                     axios.post('/api/v1/accounts/' + this.feed[index].account.id + '/follow')
                     .then(res => {
                         this.$store.commit('updateRelationship', [res.data]);
@@ -582,11 +666,21 @@
                         swal('Oops!', 'An error occured when attempting to follow this account.', 'error');
                         this.feed[index].relationship.following = false;
                     });
+                }
             },
 
             unfollow(index) {
-                // this.feed[index].relationship.following = false;
+                if(this.feed[index].reblog) {
-
+                    axios.post('/api/v1/accounts/' + this.feed[index].reblog.account.id + '/unfollow')
+                    .then(res => {
+                        this.$store.commit('updateRelationship', [res.data]);
+                        this.updateProfile({ following_count: this.profile.following_count - 1 });
+                        this.feed[index].reblog.account.followers_count = this.feed[index].reblog.account.followers_count - 1;
+                    }).catch(err => {
+                        swal('Oops!', 'An error occured when attempting to unfollow this account.', 'error');
+                        this.feed[index].reblog.relationship.following = true;
+                    });
+                } else {
                     axios.post('/api/v1/accounts/' + this.feed[index].account.id + '/unfollow')
                     .then(res => {
                         this.$store.commit('updateRelationship', [res.data]);
@@ -596,6 +690,7 @@
                         swal('Oops!', 'An error occured when attempting to unfollow this account.', 'error');
                         this.feed[index].relationship.following = true;
                     });
+                }
             },
 
             updateProfile(delta) {