diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2022-07-03 15:27:27 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2022-07-03 15:27:27 -0400 |
| commit | 207fc033bd9f7d7f6f8a5609cf2c8caab5ce57f5 (patch) | |
| tree | 8974f983dc80f48037f2f6f2775d5406ed1c4362 | |
| parent | 8533eb181ae452d558b585f9a6b15233e720d447 (diff) | |
| download | postgresql-207fc033bd9f7d7f6f8a5609cf2c8caab5ce57f5.tar.gz postgresql-207fc033bd9f7d7f6f8a5609cf2c8caab5ce57f5.zip | |
Simplify tab completion of extension versions, redux.
After commit 662dbe2c8, psql tab completion didn't conveniently
support the case of "ALTER EXTENSION foo UPDATE". It'd always
add "TO", which is fine if you want to specify a target version
but not if you don't ... and surely the latter is the much more
common case.
To fix, remove "TO" from the initially offered completion; you now
need to press TAB one additional time to get that. We won't try to
duplicate the old behavior of attempting initial completion on the
target version along with TO. It's too squirrelly to get the quoting
right, and this is such an infrequent usage that it doesn't seem worth
expending a lot of effort and special code on.
Noted by Noah Misch. Back-patch to v15.
Discussion: https://postgr.es/m/20220703083217.GB2476530@rfd.leadboat.com
| -rw-r--r-- | src/bin/psql/tab-complete.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.c index bd44a1d55da..89861f6ac02 100644 --- a/src/bin/psql/tab-complete.c +++ b/src/bin/psql/tab-complete.c @@ -1927,7 +1927,7 @@ psql_completion(const char *text, int start, int end) /* ALTER EXTENSION <name> */ else if (Matches("ALTER", "EXTENSION", MatchAny)) - COMPLETE_WITH("ADD", "DROP", "UPDATE TO", "SET SCHEMA"); + COMPLETE_WITH("ADD", "DROP", "UPDATE", "SET SCHEMA"); /* ALTER EXTENSION <name> UPDATE */ else if (Matches("ALTER", "EXTENSION", MatchAny, "UPDATE")) |