fix(drivers/alias): default sort & substitute link (#1917)

* fix(drivers/alias): default sort & substitute link

* fix

* fix

Author: KirCute

drivers/alias/driver.go

@@ -229,6 +229,15 @@ func (d *Alias) List(ctx context.Context, dir model.Obj, args model.ListArgs) ([
 	for _, obj := range objMap {
 		objs = append(objs, obj)
 	}
+	if d.OrderBy == "" {
+		sort := getAllSort(dirs)
+		if sort.OrderBy != "" {
+			model.SortFiles(objs, sort.OrderBy, sort.OrderDirection)
+		}
+		if d.ExtractFolder == "" && sort.ExtractFolder != "" {
+			model.ExtractFolder(objs, sort.ExtractFolder)
+		}
+	}
 	return objs, nil
 }
 
@@ -276,21 +285,38 @@ func (d *Alias) Link(ctx context.Context, file model.Obj, args model.LinkArgs) (
 		}, nil
 	}
 
-	reqPath := d.getBalancedPath(ctx, file)
-	link, fi, err := d.link(ctx, reqPath, args)
+	var link *model.Link
+	var fi model.Obj
+	var err error
+	files := file.(BalancedObjs)
+	if d.ReadConflictPolicy == RandomBalancedRP || d.ReadConflictPolicy == AllRWP {
+		rand.Shuffle(len(files), func(i, j int) {
+			files[i], files[j] = files[j], files[i]
+		})
+	}
+	for _, f := range files {
+		if f == nil {
+			continue
+		}
+		link, fi, err = d.link(ctx, f.GetPath(), args)
+		if err == nil {
+			if link == nil {
+				// 重定向且需要通过代理
+				return &model.Link{
+					URL: fmt.Sprintf("%s/p%s?sign=%s",
+						common.GetApiUrl(ctx),
+						utils.EncodePath(f.GetPath(), true),
+						sign.Sign(f.GetPath())),
+				}, nil
+			}
+			break
+		}
+	}
 	if err != nil {
 		return nil, err
 	}
-	if link == nil {
-		// 重定向且需要通过代理
-		return &model.Link{
-			URL: fmt.Sprintf("%s/p%s?sign=%s",
-				common.GetApiUrl(ctx),
-				utils.EncodePath(reqPath, true),
-				sign.Sign(reqPath)),
-		}, nil
-	}
 	resultLink := *link // 复制一份，避免修改到原始link
+	resultLink.Expiration = nil
 	resultLink.SyncClosers = utils.NewSyncClosers(link)
 	if args.Redirect {
 		return &resultLink, nil

drivers/alias/util.go

@@ -490,3 +490,43 @@ func (d *Alias) extract(ctx context.Context, reqPath string, args model.ArchiveI
 	link, _, err := op.DriverExtract(ctx, storage, reqActualPath, args)
 	return link, err
 }
+
+func getAllSort(dirs []model.Obj) model.Sort {
+	ret := model.Sort{}
+	noSort := false
+	noExtractFolder := false
+	for _, dir := range dirs {
+		if dir == nil {
+			continue
+		}
+		storage, err := fs.GetStorage(dir.GetPath(), &fs.GetStoragesArgs{})
+		if err != nil {
+			continue
+		}
+		if !noSort && storage.GetStorage().OrderBy != "" {
+			if ret.OrderBy == "" {
+				ret.OrderBy = storage.GetStorage().OrderBy
+				ret.OrderDirection = storage.GetStorage().OrderDirection
+				if ret.OrderDirection == "" {
+					ret.OrderDirection = "asc"
+				}
+			} else if ret.OrderBy != storage.GetStorage().OrderBy || ret.OrderDirection != storage.GetStorage().OrderDirection {
+				ret.OrderBy = ""
+				ret.OrderDirection = ""
+				noSort = true
+			}
+		}
+		if !noExtractFolder && storage.GetStorage().ExtractFolder != "" {
+			if ret.ExtractFolder == "" {
+				ret.ExtractFolder = storage.GetStorage().ExtractFolder
+			} else if ret.ExtractFolder != storage.GetStorage().ExtractFolder {
+				ret.ExtractFolder = ""
+				noExtractFolder = true
+			}
+		}
+		if noSort && noExtractFolder {
+			break
+		}
+	}
+	return ret
+}