🐛 fix delete account when change name

drivers/189.go

@@ -67,8 +67,9 @@ func (c Cloud189) Items() []Item {
 }
 
 func (c Cloud189) Save(account *model.Account, old *model.Account) error {
-	if old != nil {
+	if old != nil && old.Name != account.Name {
 		delete(client189Map, old.Name)
+		model.DeleteAccountFromMap(old.Name)
 	}
 	if err := c.Login(account); err != nil {
 		account.Status = err.Error()

drivers/alidrive.go

@@ -349,7 +349,9 @@ func (a AliDrive) RefreshToken(account *model.Account) error {
 func (a AliDrive) Save(account *model.Account, old *model.Account) error {
 	if old != nil {
 		conf.Cron.Remove(cron.EntryID(old.CronId))
-		model.DeleteAccountFromMap(old.Name)
+		if old.Name != account.Name {
+			model.DeleteAccountFromMap(old.Name)
+		}
 	}
 	if account.RootFolder == "" {
 		account.RootFolder = "root"

drivers/driver.go

@@ -9,9 +9,9 @@ import (
 
 type Driver interface {
 	Items() []Item
+	Save(account *model.Account, old *model.Account) error
 	Path(path string, account *model.Account) (*model.File, []*model.File, error)
 	Link(path string, account *model.Account) (string, error)
-	Save(account *model.Account, old *model.Account) error
 	Proxy(c *gin.Context)
 	Preview(path string, account *model.Account) (interface{}, error)
 	// TODO

drivers/native.go

@@ -17,7 +17,7 @@ type Native struct {
 }
 
 func (n Native) Preview(path string, account *model.Account) (interface{}, error) {
-	return nil,fmt.Errorf("no need")
+	return nil, fmt.Errorf("no need")
 }
 
 func (n Native) Items() []Item {
@@ -36,6 +36,9 @@ func (n Native) Proxy(c *gin.Context) {
 }
 
 func (n Native) Save(account *model.Account, old *model.Account) error {
+	if old != nil {
+		model.DeleteAccountFromMap(old.Name)
+	}
 	log.Debugf("save a account: [%s]", account.Name)
 	if !utils.Exists(account.RootFolder) {
 		account.Status = fmt.Sprintf("[%s] not exist", account.RootFolder)
@@ -73,7 +76,7 @@ func (n Native) Path(path string, account *model.Account) (*model.File, []*model
 				Size:      f.Size(),
 				Type:      0,
 				UpdatedAt: &time,
-				Driver: "Native",
+				Driver:    "Native",
 			}
 			if f.IsDir() {
 				file.Type = conf.FOLDER
@@ -94,7 +97,7 @@ func (n Native) Path(path string, account *model.Account) (*model.File, []*model
 		Size:      f.Size(),
 		Type:      utils.GetFileType(filepath.Ext(f.Name())),
 		UpdatedAt: &time,
-		Driver: "Native",
+		Driver:    "Native",
 	}
 	return file, nil, nil
 }

drivers/onedrive.go

@@ -273,7 +273,9 @@ func (o Onedrive) Save(account *model.Account, old *model.Account) error {
 	}
 	if old != nil {
 		conf.Cron.Remove(cron.EntryID(old.CronId))
-		model.DeleteAccountFromMap(old.Name)
+		if old.Name != account.Name {
+			model.DeleteAccountFromMap(old.Name)
+		}
 	}
 	account.RootFolder = utils.ParsePath(account.RootFolder)
 	err := o.RefreshToken(account)