PR suggestion

florelis · Mar 14, 2024 · ce47cf5 · ce47cf5
1 parent 97e9415
commit ce47cf5
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/src/AppInstallerCLICore/Commands/SettingsCommand.cpp b/src/AppInstallerCLICore/Commands/SettingsCommand.cpp
@@ -147,7 +147,6 @@ namespace AppInstaller::CLI
  Utility::LocIndString validOptions = Join(", "_liv, adminSettingList);
 
  if (StringAdminSetting::Unknown == StringToStringAdminSetting(execArgs.GetArg(Execution::Args::Type::SettingName))
-
  {
  throw CommandException(Resource::String::InvalidArgumentValueError(ArgumentCommon::ForType(Execution::Args::Type::SettingName).Name, validOptions));
  }
@@ -198,8 +197,7 @@ namespace AppInstaller::CLI
 
  Utility::LocIndString validOptions = Join(", "_liv, adminSettingList);
 
- if (execArgs.Contains(Execution::Args::Type::SettingName)
- && StringAdminSetting::Unknown == StringToStringAdminSetting(execArgs.GetArg(Execution::Args::Type::SettingName))
+ if (StringAdminSetting::Unknown == StringToStringAdminSetting(execArgs.GetArg(Execution::Args::Type::SettingName))
  && BoolAdminSetting::Unknown == StringToBoolAdminSetting(execArgs.GetArg(Execution::Args::Type::SettingName)))
  {
  throw CommandException(Resource::String::InvalidArgumentValueError(ArgumentCommon::ForType(Execution::Args::Type::SettingName).Name, validOptions));

diff --git a/src/AppInstallerCLICore/Workflows/SettingsFlow.cpp b/src/AppInstallerCLICore/Workflows/SettingsFlow.cpp
@@ -108,7 +108,6 @@ namespace AppInstaller::CLI::Workflow
  context.Reporter.Info() << Resource::String::ResetAdminSettingSucceeded(LocIndString{ adminSettingName }) << std::endl;
  }
  else
-
  {
  context.Reporter.Error() << Resource::String::ResetAdminSettingFailed(LocIndString{ adminSettingName }) << std::endl;
  }