From 2e5c93fdba0197a3d2de4ffd836ee35ed31c4580 Mon Sep 17 00:00:00 2001
From: kim <grufwub@gmail.com>
Date: Fri, 26 Apr 2024 12:14:50 +0100
Subject: [PATCH] make if else clauses easier to parse

---
 internal/federation/federatingdb/delete.go | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/internal/federation/federatingdb/delete.go b/internal/federation/federatingdb/delete.go
index 448a57e0f..f13a6f9b1 100644
--- a/internal/federation/federatingdb/delete.go
+++ b/internal/federation/federatingdb/delete.go
@@ -60,8 +60,11 @@ func (f *federatingDB) Delete(ctx context.Context, id *url.URL) error {
 		receiving,
 		uriStr,
 	)
-	if err != nil || ok { // handles success
+	if err != nil {
 		return err
+	} else if ok {
+		// success!
+		return nil
 	}
 
 	// Attempt to delete status.
@@ -70,8 +73,11 @@ func (f *federatingDB) Delete(ctx context.Context, id *url.URL) error {
 		receiving,
 		uriStr,
 	)
-	if err != nil || ok { // handles success
+	if err != nil {
 		return err
+	} else if ok {
+		// success!
+		return nil
 	}
 
 	// Log at debug level, as lots of these could indicate federation