LP#1568046: Fix some mistakes in previous commit.
authorJason Stephenson <jstephenson@mvlcstaff.org>
Fri, 8 Apr 2016 19:30:32 +0000 (15:30 -0400)
committerBen Shum <ben@evergreener.net>
Wed, 4 May 2016 16:57:34 +0000 (12:57 -0400)
Looks like a few thinkos crept in from it being dashed off in a hurry.

Signed-off-by: Jason Stephenson <jstephenson@mvlcstaff.org>
Signed-off-by: Ben Shum <ben@evergreener.net>
Open-ILS/src/sql/Pg/006.schema.permissions.sql
Open-ILS/src/sql/Pg/upgrade/XXXX.function.grp-tree-functions-via-cte.sql

index 9c310fe..c351ba8 100644 (file)
@@ -124,12 +124,12 @@ $$ LANGUAGE SQL STABLE ROWS 1;
 CREATE OR REPLACE FUNCTION permission.grp_descendants( INT ) RETURNS SETOF permission.grp_tree AS $$
     WITH RECURSIVE descendant_depth AS (
         SELECT  gr.id,
-                gr.parent,
+                gr.parent
           FROM  permission.grp_tree gr
           WHERE gr.id = $1
             UNION ALL
         SELECT  gr.id,
-                gr.parent,
+                gr.parent
           FROM  permission.grp_tree gr
                 JOIN descendant_depth dd ON (dd.id = gr.parent)
     ) SELECT gr.* FROM permission.grp_tree gr JOIN descendant_depth USING (id);
@@ -137,26 +137,26 @@ $$ LANGUAGE SQL ROWS 1;
 
 CREATE OR REPLACE FUNCTION permission.grp_tree_full_path ( INT ) RETURNS SETOF permission.grp_tree AS $$
         SELECT  *
-          FROM  permission.grp_tree_ancestors($1)
+          FROM  permission.grp_ancestors($1)
                         UNION
         SELECT  *
-          FROM  permission.grp_tree_descendants($1);
+          FROM  permission.grp_descendants($1);
 $$ LANGUAGE SQL STABLE ROWS 1;
 
 CREATE OR REPLACE FUNCTION permission.grp_tree_combined_ancestors ( INT, INT ) RETURNS SETOF permission.grp_tree AS $$
         SELECT  *
-          FROM  permission.grp_tree_ancestors($1)
+          FROM  permission.grp_ancestors($1)
                         UNION
         SELECT  *
-          FROM  permission.grp_tree_ancestors($2);
+          FROM  permission.grp_ancestors($2);
 $$ LANGUAGE SQL STABLE ROWS 1;
 
 CREATE OR REPLACE FUNCTION permission.grp_tree_common_ancestors ( INT, INT ) RETURNS SETOF permission.grp_tree AS $$
         SELECT  *
-          FROM  permission.grp_tree_ancestors($1)
+          FROM  permission.grp_ancestors($1)
                         INTERSECT
         SELECT  *
-          FROM  permission.grp_tree_ancestors($2);
+          FROM  permission.grp_ancestors($2);
 $$ LANGUAGE SQL STABLE ROWS 1;
 
 CREATE OR REPLACE FUNCTION permission.usr_perms ( INT ) RETURNS SETOF permission.usr_perm_map AS $$
index 7646e99..28dc061 100644 (file)
@@ -19,12 +19,12 @@ $$ LANGUAGE SQL ROWS 1;
 CREATE OR REPLACE FUNCTION permission.grp_descendants( INT ) RETURNS SETOF permission.grp_tree AS $$
     WITH RECURSIVE descendant_depth AS (
         SELECT  gr.id,
-                gr.parent,
+                gr.parent
           FROM  permission.grp_tree gr
           WHERE gr.id = $1
             UNION ALL
         SELECT  gr.id,
-                gr.parent,
+                gr.parent
           FROM  permission.grp_tree gr
                 JOIN descendant_depth dd ON (dd.id = gr.parent)
     ) SELECT gr.* FROM permission.grp_tree gr JOIN descendant_depth USING (id);
@@ -34,26 +34,26 @@ $$ LANGUAGE SQL ROWS 1;
 
 CREATE OR REPLACE FUNCTION permission.grp_tree_full_path ( INT ) RETURNS SETOF permission.grp_tree AS $$
         SELECT  *
-          FROM  permission.grp_tree_ancestors($1)
+          FROM  permission.grp_ancestors($1)
                         UNION
         SELECT  *
-          FROM  permission.grp_tree_descendants($1);
+          FROM  permission.grp_descendants($1);
 $$ LANGUAGE SQL STABLE ROWS 1;
 
 CREATE OR REPLACE FUNCTION permission.grp_tree_combined_ancestors ( INT, INT ) RETURNS SETOF permission.grp_tree AS $$
         SELECT  *
-          FROM  permission.grp_tree_ancestors($1)
+          FROM  permission.grp_ancestors($1)
                         UNION
         SELECT  *
-          FROM  permission.grp_tree_ancestors($2);
+          FROM  permission.grp_ancestors($2);
 $$ LANGUAGE SQL STABLE ROWS 1;
 
 CREATE OR REPLACE FUNCTION permission.grp_tree_common_ancestors ( INT, INT ) RETURNS SETOF permission.grp_tree AS $$
         SELECT  *
-          FROM  permission.grp_tree_ancestors($1)
+          FROM  permission.grp_ancestors($1)
                         INTERSECT
         SELECT  *
-          FROM  permission.grp_tree_ancestors($2);
+          FROM  permission.grp_ancestors($2);
 $$ LANGUAGE SQL STABLE ROWS 1;
 
 COMMIT;