diff --git a/models/user/user.go b/models/user/user.go
index 3393aac92b..5dd9b8980f 100644
--- a/models/user/user.go
+++ b/models/user/user.go
@@ -955,8 +955,8 @@ func UpdateUserCols(ctx context.Context, u *User, cols ...string) error {
 	return err
 }
 
-// UpdateUserColsWithNoAutotime update user according special columns
-func UpdateUserColsWithNoAutotime(ctx context.Context, u *User, cols ...string) error {
+// UpdateUserColsNoAutotime update user according special columns
+func UpdateUserColsNoAutotime(ctx context.Context, u *User, cols ...string) error {
 	if err := ValidateUser(u, cols...); err != nil {
 		return err
 	}
diff --git a/modules/structs/visible_type.go b/modules/structs/visible_type.go
index 2d9c82b394..45af941aa0 100644
--- a/modules/structs/visible_type.go
+++ b/modules/structs/visible_type.go
@@ -60,10 +60,3 @@ func ExtractKeysFromMapString(in map[string]VisibleType) (keys []string) {
 	}
 	return keys
 }
-
-func ConvertStringToVisibleType(s string) VisibleType {
-	if vt, ok := VisibilityModes[s]; ok {
-		return vt
-	}
-	return VisibleType(-1) // Invalid type
-}
diff --git a/services/org/org.go b/services/org/org.go
index c2e2ff021a..ffb0327d12 100644
--- a/services/org/org.go
+++ b/services/org/org.go
@@ -155,7 +155,7 @@ func ChangeOrganizationVisibility(ctx context.Context, org *org_model.Organizati
 	org.Visibility = visibility
 	// FIXME: If it's a big forks network(forks and sub forks), the database transaction will be too long to fail.
 	return db.WithTx(ctx, func(ctx context.Context) error {
-		if err := user_model.UpdateUserColsWithNoAutotime(ctx, org.AsUser(), "visibility"); err != nil {
+		if err := user_model.UpdateUserColsNoAutotime(ctx, org.AsUser(), "visibility"); err != nil {
 			return err
 		}