diff options
| author |  kim <89579420+NyaaaWhatsUpDoc@users.noreply.github.com> | 2024-08-02 12:15:11 +0000 |
|---|---|---|
| committer |  GitHub <noreply@github.com> | 2024-08-02 13:15:11 +0100 |
| commit | 0f734a24100383171a866fccc194dea578141d74 (patch) | |
| tree | a2cfdebeb1098ef39a8b2907a0933ac7f18a3ca3 /internal/processing/fedi | |
| parent | [chore] add back exif-terminator and use only for jpeg,png,webp (#3161) (diff) | |
| download | gotosocial-0f734a24100383171a866fccc194dea578141d74.tar.xz | |
[chore] move PopulateAccountStats() nil check often performed into function itself (#3158)
* move PopulateAccountStats() nil check often performed into function itself
* fix test to take in mind we don't repopulate account stats if not-nil
Diffstat (limited to 'internal/processing/fedi')
| -rw-r--r-- | internal/processing/fedi/collections.go | 24 |
1 files changed, 9 insertions, 15 deletions
diff --git a/internal/processing/fedi/collections.go b/internal/processing/fedi/collections.go index a1fc5e1f4..e81d3e269 100644 --- a/internal/processing/fedi/collections.go +++ b/internal/processing/fedi/collections.go @@ -70,11 +70,9 @@ func (p *Processor) OutboxGet( } // Ensure we have stats for this account. - if receivingAcct.Stats == nil { - if err := p.state.DB.PopulateAccountStats(ctx, receivingAcct); err != nil { - err := gtserror.Newf("error getting stats for account %s: %w", receivingAcct.ID, err) - return nil, gtserror.NewErrorInternalError(err) - } + if err := p.state.DB.PopulateAccountStats(ctx, receivingAcct); err != nil { + err := gtserror.Newf("error getting stats for account %s: %w", receivingAcct.ID, err) + return nil, gtserror.NewErrorInternalError(err) } var obj vocab.Type @@ -200,11 +198,9 @@ func (p *Processor) FollowersGet( } // Ensure we have stats for this account. - if receivingAcct.Stats == nil { - if err := p.state.DB.PopulateAccountStats(ctx, receivingAcct); err != nil { - err := gtserror.Newf("error getting stats for account %s: %w", receivingAcct.ID, err) - return nil, gtserror.NewErrorInternalError(err) - } + if err := p.state.DB.PopulateAccountStats(ctx, receivingAcct); err != nil { + err := gtserror.Newf("error getting stats for account %s: %w", receivingAcct.ID, err) + return nil, gtserror.NewErrorInternalError(err) } var obj vocab.Type @@ -314,11 +310,9 @@ func (p *Processor) FollowingGet(ctx context.Context, requestedUser string, page } // Ensure we have stats for this account. - if receivingAcct.Stats == nil { - if err := p.state.DB.PopulateAccountStats(ctx, receivingAcct); err != nil { - err := gtserror.Newf("error getting stats for account %s: %w", receivingAcct.ID, err) - return nil, gtserror.NewErrorInternalError(err) - } + if err := p.state.DB.PopulateAccountStats(ctx, receivingAcct); err != nil { + err := gtserror.Newf("error getting stats for account %s: %w", receivingAcct.ID, err) + return nil, gtserror.NewErrorInternalError(err) } var obj vocab.Type |