summaryrefslogtreecommitdiff
path: root/internal/db/bundb/relationship_block.go
diff options
context:
space:
mode:
Diffstat (limited to 'internal/db/bundb/relationship_block.go')
-rw-r--r--internal/db/bundb/relationship_block.go47
1 files changed, 32 insertions, 15 deletions
diff --git a/internal/db/bundb/relationship_block.go b/internal/db/bundb/relationship_block.go
index 2a042bed4..33a3b85fa 100644
--- a/internal/db/bundb/relationship_block.go
+++ b/internal/db/bundb/relationship_block.go
@@ -20,10 +20,10 @@ package bundb
import (
"context"
"errors"
- "fmt"
"github.com/superseriousbusiness/gotosocial/internal/db"
"github.com/superseriousbusiness/gotosocial/internal/gtscontext"
+ "github.com/superseriousbusiness/gotosocial/internal/gtserror"
"github.com/superseriousbusiness/gotosocial/internal/gtsmodel"
"github.com/superseriousbusiness/gotosocial/internal/log"
"github.com/uptrace/bun"
@@ -139,25 +139,42 @@ func (r *relationshipDB) getBlock(ctx context.Context, lookup string, dbQuery fu
return block, nil
}
- // Set the block source account
- block.Account, err = r.state.DB.GetAccountByID(
- gtscontext.SetBarebones(ctx),
- block.AccountID,
- )
- if err != nil {
- return nil, fmt.Errorf("error getting block source account: %w", err)
+ if err := r.state.DB.PopulateBlock(ctx, block); err != nil {
+ return nil, err
}
- // Set the block target account
- block.TargetAccount, err = r.state.DB.GetAccountByID(
- gtscontext.SetBarebones(ctx),
- block.TargetAccountID,
+ return block, nil
+}
+
+func (r *relationshipDB) PopulateBlock(ctx context.Context, block *gtsmodel.Block) error {
+ var (
+ err error
+ errs = gtserror.NewMultiError(2)
)
- if err != nil {
- return nil, fmt.Errorf("error getting block target account: %w", err)
+
+ if block.Account == nil {
+ // Block origin account is not set, fetch from database.
+ block.Account, err = r.state.DB.GetAccountByID(
+ gtscontext.SetBarebones(ctx),
+ block.AccountID,
+ )
+ if err != nil {
+ errs.Appendf("error populating block account: %w", err)
+ }
}
- return block, nil
+ if block.TargetAccount == nil {
+ // Block target account is not set, fetch from database.
+ block.TargetAccount, err = r.state.DB.GetAccountByID(
+ gtscontext.SetBarebones(ctx),
+ block.TargetAccountID,
+ )
+ if err != nil {
+ errs.Appendf("error populating block target account: %w", err)
+ }
+ }
+
+ return errs.Combine()
}
func (r *relationshipDB) PutBlock(ctx context.Context, block *gtsmodel.Block) error {