On Tue, Aug 18, 2015 at 6:07 AM, Daniel Verite <daniel@manitou-mail.org> wrote:
> Hi,
>
> When tab-completing after GRANT EXECUTE, currently psql injects
> "PROCEDURE", rather than the expected "ON".
>
> The code for completing with "ON" is there, but it's not reached due to
> falling earlier into another branch, one that handles CREATE TRIGGER.
>
> A trivial patch is attached. It adds the condition that if EXECUTE is
> preceded by GRANT itself preceded by nothing, then that completion
> with PROCEDURE is skipped.
>
> I've looked at fixing it more directly, by testing if the EXECUTE
> is part of a CREATE TRIGGER, but it didn't seem fitting to go
> looking backwards that many words into the string (more
> than the 5 words suggested by the rest of the code).
You should consider adding it to the next CF:
https://commitfest.postgresql.org/6/
--
Michael
