Revert "Team: Include user UID in GetTeamMembers() output" (#102519)

Revert "Team: Include user UID in GetTeamMembers() output (#102373)"

This reverts commit e5b6b7b370.

pkg/services/team/model.go

@@ -177,7 +177,6 @@ type TeamMemberDTO struct {
 	TeamID     int64          `json:"teamId" xorm:"team_id"`
 	TeamUID    string         `json:"teamUID" xorm:"uid"`
 	UserID     int64          `json:"userId" xorm:"user_id"`
-	UserUID    string         `json:"userUID" xorm:"user_uid"`
 	External   bool           `json:"-"`
 	AuthModule string         `json:"auth_module"`
 	Email      string         `json:"email"`

pkg/services/team/teamimpl/store.go

@@ -551,17 +551,18 @@ func (ss *xormStore) getTeamMembers(ctx context.Context, query *team.GetTeamMemb
 		if query.External {
 			sess.Where("team_member.external=?", ss.db.GetDialect().BooleanValue(true))
 		}
-		sess.Select("team_member.org_id," +
-			"team_member.team_id," +
-			"team_member.user_id," +
-			"\"user\".email," +
-			"\"user\".name," +
-			"\"user\".login," +
-			"\"user\".uid as user_uid," +
-			"team_member.external," +
-			"team_member.permission," +
-			"user_auth.auth_module," +
-			"team.uid")
+		sess.Cols(
+			"team_member.org_id",
+			"team_member.team_id",
+			"team_member.user_id",
+			"user.email",
+			"user.name",
+			"user.login",
+			"team_member.external",
+			"team_member.permission",
+			"user_auth.auth_module",
+			"team.uid",
+		)
 		sess.Asc("user.login", "user.email")
 
 		err := sess.Find(&queryResult)

pkg/services/team/teamimpl/store_test.go

@@ -58,7 +58,6 @@ func TestIntegrationTeamCommandsAndQueries(t *testing.T) {
 
 		t.Run("Given saved users and two teams", func(t *testing.T) {
 			var userIds []int64
-			var userUIDs []string
 			const testOrgID int64 = 1
 			var team1, team2 team.Team
 			var usr *user.User
@@ -75,7 +74,6 @@ func TestIntegrationTeamCommandsAndQueries(t *testing.T) {
 					usr, err = userSvc.Create(context.Background(), &userCmd)
 					require.NoError(t, err)
 					userIds = append(userIds, usr.ID)
-					userUIDs = append(userUIDs, usr.UID)
 				}
 				team1, err = teamSvc.CreateTeam(context.Background(), "group1 name", "test1@test.com", testOrgID)
 				require.NoError(t, err)
@@ -110,13 +108,9 @@ func TestIntegrationTeamCommandsAndQueries(t *testing.T) {
 				require.NoError(t, err)
 				require.Equal(t, 2, len(q1Result))
 				require.Equal(t, q1Result[0].TeamID, team1.ID)
-				require.Contains(t, userIds[:2], q1Result[0].UserID)
-				require.Contains(t, userUIDs[:2], q1Result[0].UserUID)
 				require.Equal(t, q1Result[0].Login, "loginuser0")
 				require.Equal(t, q1Result[0].OrgID, testOrgID)
 				require.Equal(t, q1Result[1].TeamID, team1.ID)
-				require.Contains(t, userIds[:2], q1Result[1].UserID)
-				require.Contains(t, userUIDs[:2], q1Result[1].UserUID)
 				require.Equal(t, q1Result[1].Login, "loginuser1")
 				require.Equal(t, q1Result[1].OrgID, testOrgID)
 				require.Equal(t, q1Result[1].External, true)

public/api-merged.json

@@ -21220,9 +21220,6 @@
         "userId": {
           "type": "integer",
           "format": "int64"
-        },
-        "userUID": {
-          "type": "string"
         }
       }
     },

public/openapi3.json

@@ -11281,9 +11281,6 @@
           "userId": {
             "format": "int64",
             "type": "integer"
-          },
-          "userUID": {
-            "type": "string"
           }
         },
         "type": "object"