[FB4] ALTER TRIGGER дважды определяет sql security

[dmitry-lipetsk]

ALTER TRIGGER trigname
[ACTIVE | INACTIVE]
[{BEFORE | AFTER} <mutation_list>]
[POSITION number]
[SQL SECURITY {DEFINER | INVOKER} | DROP SQL SECURITY]
[<routine-body>]

Определение routine-body так же содержит правила для SQL SECURITY.

Получается что мы может определить эти правила дважды.

[sim1984]

Пока не ясно как этот поправить.
Вообще SQL SECURITY является частью спецификации PSQL тела модуля и только его. Для внешних модулей это не работает.
С другой стороны только у триггеров есть возможность изменить SQL SECURITY не вводя тело триггера заново, а потому здесь эта часть записана.

[dmitry-lipetsk]

У Марка в английской доке есть нормальная идея :

ALTER TRIGGER trigname
  [ACTIVE | INACTIVE]
  [{BEFORE | AFTER} <mutation_list>]
  [POSITION number]
  [{<psql_trigger> | <external-module-body>}]

<psql_trigger> ::=
  [<sql_security>]
  [<psql-routine-body>]

<sql_security> ::=
    SQL SECURITY {INVOKER | DEFINER} | DROP SQL SECURITY

Я бы эту идею доработал.

Сначала избавляемся от <sql_security>. В теле модуля у нас уже есть <rights-clause>, можно на него сослаться:

<psql_trigger> ::=
  [ DROP SQL SECURITY | <rights-clause> ]
  [<psql-routine-body>]

<psql_trigger> тоже как-то не очень здесь вписывается.

Тут надо что-то типа , по аналогии с <psql-routine-spec>.

Кстати, почему бы просто не заюзать <new-psql-routine-spec>?

То есть получается так:

ALTER TRIGGER trigname
  [ACTIVE | INACTIVE]
  [{BEFORE | AFTER} <mutation_list>]
  [POSITION number]
  [{<new-psql-routine-spec> | <external-module-body>}]

<new-psql-routine-spec> ::=
  [ DROP SQL SECURITY | <rights-clause> ]
  [<psql-routine-body>]

Можно еще лесника позвать. Пусть объяснит как правильно должно быть :)

[dmitry-lipetsk]

Попытаюсь угадать, что предложит лесник :)

ALTER TRIGGER trigname
[ACTIVE | INACTIVE]
[{BEFORE | AFTER} <mutation_list>]
[POSITION number]
[{[SQL SECURITY {DEFINER | INVOKER} | DROP SQL SECURITY] [<psql-routine-body>] | <external-module-body>}]

UPD. C 3-го раза смог написать :). Вариант с <new-psql-routine-spec> выглядит понятнее.

[dmitry-lipetsk]

Еще идея.

Вместо <new-psql-routine-spec> - <alter-psql-routine-spec>.

Наверное так будет лучше - "DROP SQL SECURITY" это вроде как операция/команда, а не сущность.