summaryrefslogtreecommitdiff
path: root/internal/typeutils
diff options
context:
space:
mode:
authorLibravatar kim <89579420+NyaaaWhatsUpDoc@users.noreply.github.com>2024-08-02 12:15:11 +0000
committerLibravatar GitHub <noreply@github.com>2024-08-02 13:15:11 +0100
commit0f734a24100383171a866fccc194dea578141d74 (patch)
treea2cfdebeb1098ef39a8b2907a0933ac7f18a3ca3 /internal/typeutils
parent[chore] add back exif-terminator and use only for jpeg,png,webp (#3161) (diff)
downloadgotosocial-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/typeutils')
-rw-r--r--internal/typeutils/internaltofrontend.go12
1 files changed, 5 insertions, 7 deletions
diff --git a/internal/typeutils/internaltofrontend.go b/internal/typeutils/internaltofrontend.go
index b85d24eee..682d76775 100644
--- a/internal/typeutils/internaltofrontend.go
+++ b/internal/typeutils/internaltofrontend.go
@@ -113,13 +113,11 @@ func (c *Converter) AccountToAPIAccountSensitive(ctx context.Context, a *gtsmode
}
// Ensure account stats populated.
- if a.Stats == nil {
- if err := c.state.DB.PopulateAccountStats(ctx, a); err != nil {
- return nil, gtserror.Newf(
- "error getting stats for account %s: %w",
- a.ID, err,
- )
- }
+ if err := c.state.DB.PopulateAccountStats(ctx, a); err != nil {
+ return nil, gtserror.Newf(
+ "error getting stats for account %s: %w",
+ a.ID, err,
+ )
}
// Populate the account's role permissions bitmap and highlightedness from its public role.