From 82f2e7336bd3a0fdf16b845cea76b5191758d24c Mon Sep 17 00:00:00 2001 From: Luis Bosque Date: Thu, 3 Jul 2014 15:33:36 +0200 Subject: [PATCH] Quoted schema references in organization sql --- scripts-available/CDB_Organizations.sql | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/scripts-available/CDB_Organizations.sql b/scripts-available/CDB_Organizations.sql index f15c085..d8e98c1 100644 --- a/scripts-available/CDB_Organizations.sql +++ b/scripts-available/CDB_Organizations.sql @@ -34,8 +34,8 @@ FUNCTION cartodb.CDB_Organization_Add_Table_Read_Permission(from_schema text, ta RETURNS void AS $$ BEGIN - EXECUTE 'GRANT USAGE ON SCHEMA ' || from_schema || ' TO "' || to_role_name || '"'; - EXECUTE 'GRANT SELECT ON ' || from_schema || '.' || table_name || ' TO "' || to_role_name || '"'; + EXECUTE 'GRANT USAGE ON SCHEMA "' || from_schema || '" TO "' || to_role_name || '"'; + EXECUTE 'GRANT SELECT ON "' || from_schema || '".' || table_name || ' TO "' || to_role_name || '"'; END $$ LANGUAGE PLPGSQL VOLATILE; @@ -53,8 +53,8 @@ FUNCTION cartodb.CDB_Organization_Add_Table_Read_Write_Permission(from_schema te RETURNS void AS $$ BEGIN - EXECUTE 'GRANT USAGE ON SCHEMA ' || from_schema || ' TO "' || to_role_name || '"'; - EXECUTE 'GRANT SELECT, INSERT, UPDATE, DELETE ON ' || from_schema || '.' || table_name || ' TO "' || to_role_name || '"'; + EXECUTE 'GRANT USAGE ON SCHEMA "' || from_schema || '" TO "' || to_role_name || '"'; + EXECUTE 'GRANT SELECT, INSERT, UPDATE, DELETE ON "' || from_schema || '".' || table_name || ' TO "' || to_role_name || '"'; END $$ LANGUAGE PLPGSQL VOLATILE; @@ -73,7 +73,7 @@ FUNCTION cartodb.CDB_Organization_Remove_Access_Permission(from_schema text, tab RETURNS void AS $$ BEGIN - EXECUTE 'REVOKE ALL PRIVILEGES ON TABLE ' || from_schema || '.' || table_name || ' FROM "' || to_role_name || '"'; + EXECUTE 'REVOKE ALL PRIVILEGES ON TABLE "' || from_schema || '".' || table_name || ' FROM "' || to_role_name || '"'; -- EXECUTE 'REVOKE USAGE ON SCHEMA ' || from_schema || ' FROM "' || to_role_name || '"'; -- We need to revoke usage on schema only if we are revoking privileges from the last table where to_role_name has -- any permission granted within the schema from_schema