Hi, Please find attached patch for the updated Privileges macros under Schema node.
The changes are required due to previous commit (8a7ec6b45221f042bc39c9bce2c577e12b43cc3a) regarding "parse_priv_to_db" function. Thanks, Khushboo
diff --git a/web/pgadmin/browser/server_groups/servers/databases/schemas/templates/macros/functions/privilege.macros b/web/pgadmin/browser/server_groups/servers/databases/schemas/templates/macros/functions/privilege.macros index 075e768..08d1198 100644 --- a/web/pgadmin/browser/server_groups/servers/databases/schemas/templates/macros/functions/privilege.macros +++ b/web/pgadmin/browser/server_groups/servers/databases/schemas/templates/macros/functions/privilege.macros @@ -1,12 +1,12 @@ {##############################################} {# Macros for Privileges (functions module) #} {##############################################} -{% macro SET(conn, type, role, param, priv, with_grant, schema, func_args) -%} -{% if priv %} -GRANT {{ priv }} ON {{ type }} {{ conn|qtIdent(schema, param) }}({{func_args}}) TO {{ conn|qtIdent(role) }}; +{% macro SET(conn, type, role, param, privs, with_grant_privs, schema, func_args) -%} +{% if privs %} +GRANT {{ privs|join(', ') }} ON {{ type }} {{ conn|qtIdent(schema, param) }}({{func_args}}) TO {{conn|qtIdent(role) }}; {% endif %} -{% if with_grant %} -GRANT {{ with_grant }} ON {{ type }} {{ conn|qtIdent(schema, param) }}({{func_args}}) TO {{ conn|qtIdent(role) }} WITH GRANT OPTION; +{% if with_grant_privs %} +GRANT {{ with_grant_privs|join(', ') }} ON {{ type }} {{ conn|qtIdent(schema, param) }}({{func_args}}) TO {{ conn|qtIdent(role) }} WITH GRANT OPTION; {% endif %} {%- endmacro %} {% macro UNSETALL(conn, type, role, param, schema, func_args) -%} diff --git a/web/pgadmin/browser/server_groups/servers/databases/schemas/templates/macros/schemas/privilege.macros b/web/pgadmin/browser/server_groups/servers/databases/schemas/templates/macros/schemas/privilege.macros index cec2e53..183ec2f 100644 --- a/web/pgadmin/browser/server_groups/servers/databases/schemas/templates/macros/schemas/privilege.macros +++ b/web/pgadmin/browser/server_groups/servers/databases/schemas/templates/macros/schemas/privilege.macros @@ -1,12 +1,12 @@ {##########################################} {# Macros for schema and its child nodes #} {##########################################} -{% macro SET(conn, type, role, param, priv, with_grant, schema) -%} -{% if priv %} -GRANT {{ priv }} ON {{ type }} {{ conn|qtIdent(schema, param) }} TO {{ conn|qtIdent(role) }}; +{% macro SET(conn, type, role, param, privs, with_grant_privs, schema) -%} +{% if privs %} +GRANT {{ privs|join(', ') }} ON {{ type }} {{ conn|qtIdent(schema, param) }} TO {{ conn|qtIdent(role) }}; {% endif %} -{% if with_grant %} -GRANT {{ with_grant }} ON {{ type }} {{ conn|qtIdent(schema, param) }} TO {{ conn|qtIdent(role) }} WITH GRANT OPTION; +{% if with_grant_privs %} +GRANT {{ with_grant_privs|join(', ') }} ON {{ type }} {{ conn|qtIdent(schema, param) }} TO {{ conn|qtIdent(role) }} WITH GRANT OPTION; {% endif %} {%- endmacro %} {% macro UNSETALL(conn, type, role, param, schema) -%}
-- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers