From 82ebcf66a5caa4a7fb120d42cf7fa726e2e5bb1a Mon Sep 17 00:00:00 2001 From: mngshm Date: Mon, 24 Feb 2025 17:12:13 +0530 Subject: [PATCH] fix: allow null values in storage_limit column --- server/migrations/96_storage_table_families.up.sql | 2 +- server/pkg/controller/family/admin.go | 2 +- server/pkg/repo/family.go | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/server/migrations/96_storage_table_families.up.sql b/server/migrations/96_storage_table_families.up.sql index d41daeef51..8f3e88b0f3 100644 --- a/server/migrations/96_storage_table_families.up.sql +++ b/server/migrations/96_storage_table_families.up.sql @@ -1,2 +1,2 @@ ALTER TABLE families -ADD COLUMN storage_limit BIGINT; \ No newline at end of file +ADD COLUMN storage_limit BIGINT NULL; \ No newline at end of file diff --git a/server/pkg/controller/family/admin.go b/server/pkg/controller/family/admin.go index c944c0cfc4..e08475d89b 100644 --- a/server/pkg/controller/family/admin.go +++ b/server/pkg/controller/family/admin.go @@ -232,7 +232,7 @@ func (c *Controller) ModifyMemberStorage(ctx context.Context, actorUserID int64, } } - modifyStorageErr := c.FamilyRepo.ModifyMemberStorage(ctx, actorUserID, member.ID, storageLimit) + modifyStorageErr := c.FamilyRepo.ModifyMemberStorage(ctx, member.ID, storageLimit) if modifyStorageErr != nil { return stacktrace.Propagate(modifyStorageErr, "Failed to modify members storage") } diff --git a/server/pkg/repo/family.go b/server/pkg/repo/family.go index fad14d89aa..b9cdfb4cbc 100644 --- a/server/pkg/repo/family.go +++ b/server/pkg/repo/family.go @@ -197,7 +197,7 @@ func (repo *FamilyRepository) RemoveMember(ctx context.Context, adminID int64, m } // UpdateStorage is used to set Pre-existing Members Storage Limit. -func (repo *FamilyRepository) ModifyMemberStorage(ctx context.Context, adminID int64, id uuid.UUID, storageLimit *int64) error { +func (repo *FamilyRepository) ModifyMemberStorage(ctx context.Context, id uuid.UUID, storageLimit *int64) error { _, err := repo.DB.Exec(`UPDATE families SET storage_limit=$1 where id=$2`, storageLimit, id) if err != nil { return stacktrace.Propagate(err, "Could not update Members Storage Limit")