diff options
author | 2021-11-22 08:46:19 +0100 | |
---|---|---|
committer | 2021-11-22 08:46:19 +0100 | |
commit | f8630348b4c14215d87921962a2acbe8d3b6c981 (patch) | |
tree | 8a06b1e91ca408de2451a79fee44de831456d729 /internal/processing/federation/getstatusreplies.go | |
parent | tidy up database creation logic a bit (#317) (diff) | |
download | gotosocial-f8630348b4c14215d87921962a2acbe8d3b6c981.tar.xz |
Enable stricter linting with golangci-lint (#316)
* update golangci-lint
* add golangci config file w/ more linters
* correct issues flagged by stricter linters
* add more generous timeout for golangci-lint
* add some style + formatting guidelines
* move timeout to config file
* go fmt
Diffstat (limited to 'internal/processing/federation/getstatusreplies.go')
-rw-r--r-- | internal/processing/federation/getstatusreplies.go | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/internal/processing/federation/getstatusreplies.go b/internal/processing/federation/getstatusreplies.go index 28c46269e..03fd734a6 100644 --- a/internal/processing/federation/getstatusreplies.go +++ b/internal/processing/federation/getstatusreplies.go @@ -82,10 +82,9 @@ func (p *processor) GetStatusReplies(ctx context.Context, requestedUsername stri // 1. we're asked for the whole collection and not a page -- we can just return the collection, with no items, but a link to 'first' page. // 2. we're asked for a page but only_other_accounts has not been set in the query -- so we should just return the first page of the collection, with no items. // 3. we're asked for a page, and only_other_accounts has been set, and min_id has optionally been set -- so we need to return some actual items! - - if !page { + switch { + case !page: // scenario 1 - // get the collection collection, err := p.tc.StatusToASRepliesCollection(ctx, s, onlyOtherAccounts) if err != nil { @@ -96,9 +95,8 @@ func (p *processor) GetStatusReplies(ctx context.Context, requestedUsername stri if err != nil { return nil, gtserror.NewErrorInternalError(err) } - } else if page && requestURL.Query().Get("only_other_accounts") == "" { + case page && requestURL.Query().Get("only_other_accounts") == "": // scenario 2 - // get the collection collection, err := p.tc.StatusToASRepliesCollection(ctx, s, onlyOtherAccounts) if err != nil { @@ -109,7 +107,7 @@ func (p *processor) GetStatusReplies(ctx context.Context, requestedUsername stri if err != nil { return nil, gtserror.NewErrorInternalError(err) } - } else { + default: // scenario 3 // get immediate children replies, err := p.db.GetStatusChildren(ctx, s, true, minID) |