feat(street): 公开详情改为runway同形嵌套；审核页合并为图片流

2026-09-21 23:24:00 +08:00
parent 49bd7b74c6
commit 85ad3a3f68
5 changed files with 119 additions and 108 deletions
--- a/internal/service/street_snap_group_test.go
+++ b/internal/service/street_snap_group_test.go
@ -7,12 +7,13 @@ import (
 	"fashionapi/internal/pkg/imgurl"
 )

-// TestGroupSnapImages 验证主副图分组与「副图的副图」上浮归并。
+// TestBuildSnapImages 验证公开详情的 image 只含主图、副图嵌套在其主图的 Detail 下，
+// 且「副图的副图」上浮归并到该副图所在组的主图。
 //
 //	5 张图：A 主，B 副(挂 A)；C 主；D 副(挂 C)；E 副(挂 B → 应上浮归并到 A)
 //
-// 期望 2 组：第 0 组 = A + [B, E]，第 1 组 = C + [D]。
-func TestGroupSnapImages(t *testing.T) {
+// 期望 images 只有 2 张主图：A(Detail=[B,E], LookIndex=1)、C(Detail=[D], LookIndex=2)。
+func TestBuildSnapImages(t *testing.T) {
 	s := &streetSnapService{img: imgurl.New("", "", "")}
 	imgs := []model.StreetSnapImage{
 		{ID: 1, Image: "A", Name: "A", IsDetail: 0, ParentImageID: 0},
@ -22,46 +23,45 @@ func TestGroupSnapImages(t *testing.T) {
 		{ID: 5, Image: "E", Name: "E", IsDetail: 1, ParentImageID: 2}, // 2 是副图 → 上浮到 A
 	}

-	flat, groups := s.groupSnapImages(imgs)
-	if len(flat) != 5 {
-		t.Fatalf("flat 应 5 张，实际 %d", len(flat))
-	}
-	if len(groups) != 2 {
-		t.Fatalf("groups 应为 2 组，实际 %d", len(groups))
+	out := s.buildSnapImages(imgs)
+	if len(out) != 2 {
+		t.Fatalf("images 应只含 2 张主图，实际 %d", len(out))
 	}

-	g0 := groups[0]
-	if g0.Image.Name != "A" {
-		t.Fatalf("第 0 组主图应为 A，实际 %s", g0.Image.Name)
+	a := out[0]
+	if a.Name != "A" || a.IsDetail != 0 || a.LookIndex != 1 {
+		t.Fatalf("第 1 张应为 A 主图 LookIndex=1，实际 name=%s is_detail=%d look=%d", a.Name, a.IsDetail, a.LookIndex)
 	}
-	if len(g0.Details) != 2 {
-		t.Fatalf("第 0 组副图应为 2 张，实际 %d", len(g0.Details))
+	if len(a.Detail) != 2 || a.Detail[0].Name != "B" || a.Detail[1].Name != "E" {
+		t.Fatalf("A 的副图应依序为 [B,E]，实际 %d 张", len(a.Detail))
 	}
-	if g0.Details[0].Name != "B" || g0.Details[1].Name != "E" {
-		t.Fatalf("第 0 组副图顺序应为 [B, E]，实际 %s, %s", g0.Details[0].Name, g0.Details[1].Name)
+	for _, d := range a.Detail {
+		if d.IsDetail != 1 || d.LookIndex != 1 {
+			t.Fatalf("副图 %s 的 is_detail/look_index 应为 1/1，实际 %d/%d", d.Name, d.IsDetail, d.LookIndex)
+		}
 	}

-	g1 := groups[1]
-	if g1.Image.Name != "C" {
-		t.Fatalf("第 1 组主图应为 C，实际 %s", g1.Image.Name)
+	c := out[1]
+	if c.Name != "C" || c.LookIndex != 2 {
+		t.Fatalf("第 2 张应为 C 主图 LookIndex=2，实际 name=%s look=%d", c.Name, c.LookIndex)
 	}
-	if len(g1.Details) != 1 || g1.Details[0].Name != "D" {
-		t.Fatalf("第 1 组副图应为 [D]，实际 %d 张", len(g1.Details))
+	if len(c.Detail) != 1 || c.Detail[0].Name != "D" {
+		t.Fatalf("C 的副图应为 [D]，实际 %d 张", len(c.Detail))
 	}
 }

-// TestGroupSnapImagesOrphan 父引用指向失效行（不存在）时，该图按主图渲染，不丢图。
-func TestGroupSnapImagesOrphan(t *testing.T) {
+// TestBuildSnapImagesOrphan 父引用指向失效行（不存在）时，该图按主图渲染，不丢图。
+func TestBuildSnapImagesOrphan(t *testing.T) {
 	s := &streetSnapService{img: imgurl.New("", "", "")}
 	imgs := []model.StreetSnapImage{
 		{ID: 1, Image: "A", Name: "A", IsDetail: 1, ParentImageID: 99}, // 父 99 不存在
 		{ID: 2, Image: "B", Name: "B", IsDetail: 0, ParentImageID: 0},
 	}
-	flat, groups := s.groupSnapImages(imgs)
-	if len(flat) != 2 || len(groups) != 2 {
-		t.Fatalf("孤儿副图应自成主图：flat=%d groups=%d", len(flat), len(groups))
+	out := s.buildSnapImages(imgs)
+	if len(out) != 2 {
+		t.Fatalf("孤儿副图应自成主图：images=%d", len(out))
 	}
-	if groups[0].Image.Name != "A" || len(groups[0].Details) != 0 {
-		t.Fatalf("A 应作为主图且无副图，实际 %s details=%d", groups[0].Image.Name, len(groups[0].Details))
+	if out[0].Name != "A" || out[0].IsDetail != 0 || len(out[0].Detail) != 0 {
+		t.Fatalf("A 应作为无副图的主图，实际 name=%s is_detail=%d details=%d", out[0].Name, out[0].IsDetail, len(out[0].Detail))
 	}
 }
--- a/internal/service/street_snap_service.go
+++ b/internal/service/street_snap_service.go
@ -134,24 +134,22 @@ func (s *streetSnapService) Detail(ctx context.Context, id string) (*dto.PublicS
 		return nil, internalErr(err.Error())
 	}

-	flat, groups := s.groupSnapImages(imgs)
 	return &dto.PublicStreetSnapDetail{
 		UID:    hashid.EncodeWithType(snap.ID, hashid.TypeSnap),
 		Title:  snap.Title,
 		Cover:  s.img.Compose(snap.Cover),
-		Images: flat,
-		Groups: groups,
+		Images: s.buildSnapImages(imgs),
 	}, nil
 }

-// groupSnapImages 把街拍图片按主副图分组：主图各成一组，副图归入其（存活）主图组。
-// 父引用指向「副图」时沿链向上归并到该副图所在组的主图；父行缺失（被软删）则把该图按主图渲染（不丢图）。
-// 返回的 flat（扁平）与 groups 同源同序：groups 中每张主图对应的 Details 顺序，等于 flat 中该主图之后的副图出现顺序。
-func (s *streetSnapService) groupSnapImages(imgs []model.StreetSnapImage) ([]dto.PublicArticleImage, []dto.PublicStreetSnapGroup) {
+// buildSnapImages 把街拍图片组装成「主图 + 其副图」的嵌套结构（与走秀详情同形）：
+// Images 只含主图（is_detail=0），每张主图的副图挂在其 Detail 子数组下。
+// 父引用指向「副图」时沿链向上归并到该副图所在组的主图；父行缺失 / 成环则把该图按主图渲染（不丢图）。
+// LookIndex 是合成的 1-based 主图序号（街拍不落库组序号）。
+func (s *streetSnapService) buildSnapImages(imgs []model.StreetSnapImage) []dto.PublicArticleImage {
 	if len(imgs) == 0 {
-		return nil, nil
+		return nil
 	}
-	flat := s.toPublicSnapImages(imgs)

 	byID := make(map[uint32]*model.StreetSnapImage, len(imgs))
 	for i := range imgs {
@ -178,26 +176,35 @@ func (s *streetSnapService) groupSnapImages(imgs []model.StreetSnapImage) ([]dto
 		}
 	}

-	groups := make([]dto.PublicStreetSnapGroup, 0, len(imgs))
+	// 第一遍：按 sort_order 建立主图（根），并记录「主图行 id → 输出下标」，副图排在主图之前也能正确归位。
+	out := make([]dto.PublicArticleImage, 0, len(imgs))
 	index := make(map[uint32]int, len(imgs))
-	for i, im := range imgs {
-		root := resolveRoot(im.ID)
-		isDetail := root != im.ID
-		ref := flat[i]
-		if !isDetail {
-			groups = append(groups, dto.PublicStreetSnapGroup{Image: ref})
-			index[im.ID] = len(groups) - 1
+	for _, im := range imgs {
+		if resolveRoot(im.ID) != im.ID {
 			continue
 		}
-		if gi, ok := index[root]; ok {
-			groups[gi].Details = append(groups[gi].Details, ref)
-		} else {
-			// 兜底（理论不发生）：主图未先出现，自成一组
-			groups = append(groups, dto.PublicStreetSnapGroup{Image: ref})
-			index[im.ID] = len(groups) - 1
-		}
+		pub := s.toPublicSnapImage(im)
+		pub.LookIndex = uint32(len(out) + 1) // 合成的 1-based 主图序号
+		index[im.ID] = len(out)
+		out = append(out, pub)
 	}
-	return flat, groups
+
+	// 第二遍：把副图挂到所属主图的 Detail 下（LookIndex 取所属主图的值）。
+	for _, im := range imgs {
+		root := resolveRoot(im.ID)
+		if root == im.ID {
+			continue
+		}
+		gi, ok := index[root]
+		if !ok {
+			continue // 理论不发生（孤儿已按主图渲染）
+		}
+		pub := s.toPublicSnapImage(im)
+		pub.IsDetail = 1
+		pub.LookIndex = out[gi].LookIndex
+		out[gi].Detail = append(out[gi].Detail, pub)
+	}
+	return out
 }

 // IDs 返回全部街拍的编码 uid（供 SSG 枚举详情页路径使用，对外暴露的即编码串）。
@ -229,16 +236,21 @@ func (s *streetSnapService) toPublicSnapImages(imgs []model.StreetSnapImage) []d
 	}
 	out := make([]dto.PublicArticleImage, 0, len(imgs))
 	for _, im := range imgs {
-		out = append(out, dto.PublicArticleImage{
-			ID:    hashid.EncodeWithType(im.ID, hashid.TypeSnapImage),
-			Image: s.img.Compose(im.Image),
-			Thumb: s.img.ComposeThumb(im.Image),
-			Name:  im.Name,
-		})
+		out = append(out, s.toPublicSnapImage(im))
 	}
 	return out
 }

+// toPublicSnapImage 把单张图片实体映射为对外结构（url + 名称 + 图片自身的编码 id）。
+func (s *streetSnapService) toPublicSnapImage(im model.StreetSnapImage) dto.PublicArticleImage {
+	return dto.PublicArticleImage{
+		ID:    hashid.EncodeWithType(im.ID, hashid.TypeSnapImage),
+		Image: s.img.Compose(im.Image),
+		Thumb: s.img.ComposeThumb(im.Image),
+		Name:  im.Name,
+	}
+}
+
 // toThumbSnapImages 与 toPublicSnapImages 同构，但图片地址走「列表缩略图」样式（ComposeThumb），
 // 仅供列表卡片 / 图条等小尺寸场景使用；详情页与大图灯箱仍用 toPublicSnapImages（展示样式）。
 func (s *streetSnapService) toThumbSnapImages(imgs []model.StreetSnapImage) []dto.PublicArticleImage {