Delete `GetOrgsByUserID`

internal/db/org.go

@@ -11,21 +11,6 @@ import (
 	"xorm.io/xorm"
 )
 
-func getOrgsByUserID(sess *xorm.Session, userID int64, showAll bool) ([]*User, error) {
-	orgs := make([]*User, 0, 10)
-	if !showAll {
-		sess.And("`org_user`.is_public=?", true)
-	}
-	return orgs, sess.And("`org_user`.uid=?", userID).
-		Join("INNER", "`org_user`", "`org_user`.org_id=`user`.id").Find(&orgs)
-}
-
-// GetOrgsByUserID returns a list of organizations that the given user ID
-// has joined.
-func GetOrgsByUserID(userID int64, showAll bool) ([]*User, error) {
-	return getOrgsByUserID(x.NewSession(), userID, showAll)
-}
-
 // getOwnedOrgsByUserID returns a list of organizations are owned by given user ID.
 func getOwnedOrgsByUserID(sess *xorm.Session, userID int64) ([]*User, error) {
 	orgs := make([]*User, 0, 10)

internal/route/user/profile.go

@@ -42,9 +42,15 @@ func Profile(c *context.Context, puser *context.ParamsUser) {
 	c.PageIs("UserProfile")
 	c.Data["Owner"] = puser
 
-	orgs, err := db.GetOrgsByUserID(puser.ID, c.IsLogged && (c.User.IsAdmin || c.User.ID == puser.ID))
+	orgs, err := db.Organizations.List(
+		c.Req.Context(),
+		db.ListOrganizationsOptions{
+			MemberID:              puser.ID,
+			IncludePrivateMembers: c.IsLogged && (c.User.IsAdmin || c.User.ID == puser.ID),
+		},
+	)
 	if err != nil {
-		c.Error(err, "get organizations by user ID")
+		c.Error(err, "list organizations by user ID")
 		return
 	}
 

internal/route/user/setting.go

@@ -555,9 +555,15 @@ func SettingsOrganizations(c *context.Context) {
 	c.Title("settings.orgs")
 	c.PageIs("SettingsOrganizations")
 
-	orgs, err := db.GetOrgsByUserID(c.User.ID, true)
+	orgs, err := db.Organizations.List(
+		c.Req.Context(),
+		db.ListOrganizationsOptions{
+			MemberID:              c.User.ID,
+			IncludePrivateMembers: true,
+		},
+	)
 	if err != nil {
-		c.Errorf(err, "get organizations by user ID")
+		c.Errorf(err, "list organizations by user ID")
 		return
 	}
 	c.Data["Orgs"] = orgs