- Notifications
You must be signed in to change notification settings - Fork5.2k
Commit8891433
committed
Fix tab completion for ALTER ROLE|USER ... RESET
Commitc407d54 added tab completion for ALTER ROLE|USER ... RESET,with the intent to offer only the variables actually set on the role.But as soon as the user started typing something, it would start tooffer all possible matching variables.Fix this the same way ALTER DATABASE ... RESET does it, i.e. byproperly considering the prefix.A second issue causing similar symptoms (offering variables that are notactually set for a role) was caused by a match to another pattern. TheALTER DATABASE ... RESET was already excluded, so do the same thing forROLE/USER.Report and fix by Dagfinn Ilmari Mannsåker. Backpatch to 18, same asc407d54.Author: Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>Reviewed-by: jian he <jian.universality@gmail.com>Discussion:https://postgr.es/m/87qzyghw2x.fsf%40wibble.ilmari.orgDiscussion:https://postgr.es/m/87tt4lumqz.fsf%40wibble.ilmari.orgBackpatch-through: 181 parent72c437f commit8891433
1 file changed
+10
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1078 | 1078 | | |
1079 | 1079 | | |
1080 | 1080 | | |
1081 | | - | |
1082 | | - | |
1083 | | - | |
| 1081 | + | |
| 1082 | + | |
| 1083 | + | |
| 1084 | + | |
| 1085 | + | |
| 1086 | + | |
1084 | 1087 | | |
1085 | 1088 | | |
1086 | 1089 | | |
| |||
2508 | 2511 | | |
2509 | 2512 | | |
2510 | 2513 | | |
| 2514 | + | |
| 2515 | + | |
2511 | 2516 | | |
| 2517 | + | |
2512 | 2518 | | |
2513 | 2519 | | |
2514 | 2520 | | |
| |||
4958 | 4964 | | |
4959 | 4965 | | |
4960 | 4966 | | |
4961 | | - | |
| 4967 | + | |
4962 | 4968 | | |
4963 | 4969 | | |
4964 | 4970 | | |
| |||
0 commit comments
Comments
(0)