From 86a59db711fa893362cfccdee3cd4a08e16ccdb5 Mon Sep 17 00:00:00 2001
From: Vyr Cossont <VyrCossont@users.noreply.github.com>
Date: Tue, 23 Jul 2024 12:51:57 -0700
Subject: Remove content and related fields from boosts (#3131)

These duplicate the content of the target and aren't necessary for anything.

- Stops copying some fields from target when boosting or processing a remote boost
- Adds a migration to null out existing duplicate data
- Updates tests
---
 .../20240722222556_remove_boost_content.go         | 56 ++++++++++++++++++++++
 1 file changed, 56 insertions(+)
 create mode 100644 internal/db/bundb/migrations/20240722222556_remove_boost_content.go

(limited to 'internal/db')

diff --git a/internal/db/bundb/migrations/20240722222556_remove_boost_content.go b/internal/db/bundb/migrations/20240722222556_remove_boost_content.go
new file mode 100644
index 000000000..e9226478b
--- /dev/null
+++ b/internal/db/bundb/migrations/20240722222556_remove_boost_content.go
@@ -0,0 +1,56 @@
+// GoToSocial
+// Copyright (C) GoToSocial Authors admin@gotosocial.org
+// SPDX-License-Identifier: AGPL-3.0-or-later
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU Affero General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU Affero General Public License for more details.
+//
+// You should have received a copy of the GNU Affero General Public License
+// along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+package migrations
+
+import (
+	"context"
+
+	"github.com/superseriousbusiness/gotosocial/internal/gtsmodel"
+	"github.com/uptrace/bun"
+)
+
+// Boosts previously duplicated some columns from their targets.
+// This isn't necessary, so we remove them here.
+// Admins may want to vacuum after running this migration.
+func init() {
+	up := func(ctx context.Context, db *bun.DB) error {
+		return db.RunInTx(ctx, nil, func(ctx context.Context, tx bun.Tx) error {
+			_, err := tx.NewUpdate().
+				Model((*gtsmodel.Status)(nil)).
+				Where("boost_of_id IS NOT NULL").
+				SetColumn("content", "NULL").
+				SetColumn("content_warning", "NULL").
+				SetColumn("text", "NULL").
+				SetColumn("language", "NULL").
+				SetColumn("sensitive", "FALSE").
+				Exec(ctx)
+
+			return err
+		})
+	}
+
+	down := func(ctx context.Context, db *bun.DB) error {
+		return db.RunInTx(ctx, nil, func(ctx context.Context, tx bun.Tx) error {
+			return nil
+		})
+	}
+
+	if err := Migrations.Register(up, down); err != nil {
+		panic(err)
+	}
+}
-- 
cgit v1.2.3