status: when extracting arguments from `opts`, use the same default values
authorKyle Lippincott <spectral@google.com>
Tue, 07 Dec 2021 15:53:55 -0800
changeset 48461 eaad68294904
parent 48460 c51408b92b88
child 48462 5bc3ce09c872
status: when extracting arguments from `opts`, use the same default values Sometimes other code, such as commit when using `commands.commit.post-status`, calls `commands.status()` without going through the normal dispatch mechanism that would typically fill in the args to be something besides None. As a "defense in depth" mechanism for a bug where Mercurial would crash if both `commands.commit.post-status` and `experimental.directaccess` were enabled, let's sanitize these values to be identical to the values they would have when the user invoked this method from the commandline. Differential Revision: https://phab.mercurial-scm.org/D11884
mercurial/commands.py
--- a/mercurial/commands.py	Tue Dec 07 15:48:22 2021 -0800
+++ b/mercurial/commands.py	Tue Dec 07 15:53:55 2021 -0800
@@ -6873,9 +6873,9 @@
 
     cmdutil.check_at_most_one_arg(opts, 'rev', 'change')
     opts = pycompat.byteskwargs(opts)
-    revs = opts.get(b'rev')
-    change = opts.get(b'change')
-    terse = opts.get(b'terse')
+    revs = opts.get(b'rev', [])
+    change = opts.get(b'change', b'')
+    terse = opts.get(b'terse', _NOTTERSE)
     if terse is _NOTTERSE:
         if revs:
             terse = b''