refactor: Decompose all redudant parameters into Operations struct
This commit is contained in:
@@ -86,9 +86,7 @@ var archiveCmd = &cobra.Command{
|
||||
tm.Close,
|
||||
|
||||
metadataPersister,
|
||||
)
|
||||
|
||||
if _, err := ops.Archive(
|
||||
config.PipeConfig{
|
||||
Compression: viper.GetString(compressionFlag),
|
||||
Encryption: viper.GetString(encryptionFlag),
|
||||
@@ -101,11 +99,14 @@ var archiveCmd = &cobra.Command{
|
||||
},
|
||||
|
||||
viper.GetInt(recordSizeFlag),
|
||||
|
||||
logging.NewLogger().PrintHeader,
|
||||
)
|
||||
|
||||
if _, err := ops.Archive(
|
||||
viper.GetString(fromFlag),
|
||||
viper.GetString(compressionLevelFlag),
|
||||
viper.GetBool(overwriteFlag),
|
||||
|
||||
logging.NewLogger().PrintHeader,
|
||||
); err != nil {
|
||||
return err
|
||||
}
|
||||
|
||||
@@ -73,9 +73,7 @@ var deleteCmd = &cobra.Command{
|
||||
tm.Close,
|
||||
|
||||
metadataPersister,
|
||||
)
|
||||
|
||||
return ops.Delete(
|
||||
config.PipeConfig{
|
||||
Compression: viper.GetString(compressionFlag),
|
||||
Encryption: viper.GetString(encryptionFlag),
|
||||
@@ -88,10 +86,11 @@ var deleteCmd = &cobra.Command{
|
||||
},
|
||||
|
||||
viper.GetInt(recordSizeFlag),
|
||||
viper.GetString(nameFlag),
|
||||
|
||||
logging.NewLogger().PrintHeader,
|
||||
)
|
||||
|
||||
return ops.Delete(viper.GetString(nameFlag))
|
||||
},
|
||||
}
|
||||
|
||||
|
||||
@@ -69,9 +69,7 @@ var moveCmd = &cobra.Command{
|
||||
tm.Close,
|
||||
|
||||
metadataPersister,
|
||||
)
|
||||
|
||||
return ops.Move(
|
||||
config.PipeConfig{
|
||||
Compression: viper.GetString(compressionFlag),
|
||||
Encryption: viper.GetString(encryptionFlag),
|
||||
@@ -84,11 +82,11 @@ var moveCmd = &cobra.Command{
|
||||
},
|
||||
|
||||
viper.GetInt(recordSizeFlag),
|
||||
viper.GetString(fromFlag),
|
||||
viper.GetString(toFlag),
|
||||
|
||||
logging.NewLogger().PrintHeader,
|
||||
)
|
||||
|
||||
return ops.Move(viper.GetString(fromFlag), viper.GetString(toFlag))
|
||||
},
|
||||
}
|
||||
|
||||
|
||||
@@ -73,9 +73,7 @@ var restoreCmd = &cobra.Command{
|
||||
tm.Close,
|
||||
|
||||
metadataPersister,
|
||||
)
|
||||
|
||||
return ops.Restore(
|
||||
config.PipeConfig{
|
||||
Compression: viper.GetString(compressionFlag),
|
||||
Encryption: viper.GetString(encryptionFlag),
|
||||
@@ -88,11 +86,14 @@ var restoreCmd = &cobra.Command{
|
||||
},
|
||||
|
||||
viper.GetInt(recordSizeFlag),
|
||||
|
||||
logging.NewLogger().PrintHeader,
|
||||
)
|
||||
|
||||
return ops.Restore(
|
||||
viper.GetString(fromFlag),
|
||||
viper.GetString(toFlag),
|
||||
viper.GetBool(flattenFlag),
|
||||
|
||||
logging.NewLogger().PrintHeader,
|
||||
)
|
||||
},
|
||||
}
|
||||
|
||||
@@ -76,9 +76,7 @@ var updateCmd = &cobra.Command{
|
||||
tm.Close,
|
||||
|
||||
metadataPersister,
|
||||
)
|
||||
|
||||
if _, err := ops.Update(
|
||||
config.PipeConfig{
|
||||
Compression: viper.GetString(compressionFlag),
|
||||
Encryption: viper.GetString(encryptionFlag),
|
||||
@@ -91,11 +89,14 @@ var updateCmd = &cobra.Command{
|
||||
},
|
||||
|
||||
viper.GetInt(recordSizeFlag),
|
||||
viper.GetString(fromFlag),
|
||||
viper.GetBool(overwriteFlag),
|
||||
viper.GetString(compressionLevelFlag),
|
||||
|
||||
logging.NewLogger().PrintHeader,
|
||||
)
|
||||
|
||||
if _, err := ops.Update(
|
||||
viper.GetString(fromFlag),
|
||||
viper.GetString(compressionLevelFlag),
|
||||
viper.GetBool(overwriteFlag),
|
||||
); err != nil {
|
||||
return err
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user