Skip to content

Commit 46dd594

Browse files
committed
Update alias and macro create/delete to use pfeedback for non-essential success cases
1 parent e68b481 commit 46dd594

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

cmd2/cmd2.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3862,7 +3862,7 @@ def _alias_create(self, args: argparse.Namespace) -> None:
38623862

38633863
# Set the alias
38643864
result = "overwritten" if args.name in self.aliases else "created"
3865-
self.poutput(f"Alias '{args.name}' {result}")
3865+
self.pfeedback(f"Alias '{args.name}' {result}")
38663866

38673867
self.aliases[args.name] = value
38683868
self.last_result = True
@@ -3891,15 +3891,15 @@ def _alias_delete(self, args: argparse.Namespace) -> None:
38913891

38923892
if args.all:
38933893
self.aliases.clear()
3894-
self.poutput("All aliases deleted")
3894+
self.pfeedback("All aliases deleted")
38953895
elif not args.names:
38963896
self.perror("Either --all or alias name(s) must be specified")
38973897
self.last_result = False
38983898
else:
38993899
for cur_name in utils.remove_duplicates(args.names):
39003900
if cur_name in self.aliases:
39013901
del self.aliases[cur_name]
3902-
self.poutput(f"Alias '{cur_name}' deleted")
3902+
self.pfeedback(f"Alias '{cur_name}' deleted")
39033903
else:
39043904
self.perror(f"Alias '{cur_name}' does not exist")
39053905

@@ -4150,7 +4150,7 @@ def _macro_create(self, args: argparse.Namespace) -> None:
41504150

41514151
# Set the macro
41524152
result = "overwritten" if args.name in self.macros else "created"
4153-
self.poutput(f"Macro '{args.name}' {result}")
4153+
self.pfeedback(f"Macro '{args.name}' {result}")
41544154

41554155
self.macros[args.name] = Macro(name=args.name, value=value, minimum_arg_count=max_arg_num, args=macro_args)
41564156
self.last_result = True
@@ -4179,15 +4179,15 @@ def _macro_delete(self, args: argparse.Namespace) -> None:
41794179

41804180
if args.all:
41814181
self.macros.clear()
4182-
self.poutput("All macros deleted")
4182+
self.pfeedback("All macros deleted")
41834183
elif not args.names:
41844184
self.perror("Either --all or macro name(s) must be specified")
41854185
self.last_result = False
41864186
else:
41874187
for cur_name in utils.remove_duplicates(args.names):
41884188
if cur_name in self.macros:
41894189
del self.macros[cur_name]
4190-
self.poutput(f"Macro '{cur_name}' deleted")
4190+
self.pfeedback(f"Macro '{cur_name}' deleted")
41914191
else:
41924192
self.perror(f"Macro '{cur_name}' does not exist")
41934193

0 commit comments

Comments
 (0)