PG11 changes pg_proc.proisagg to prokind to support stored procedures and
authorRobert Treat <rob@xzilla.net>
Fri, 19 Jul 2019 20:40:03 +0000 (16:40 -0400)
committerRobert Treat <rob@xzilla.net>
Fri, 19 Jul 2019 20:40:03 +0000 (16:40 -0400)
window functions, breaking functions, aggregates, and object look-up within
PPA. This commit fixes all three of these areas. In theory we should probably
add some display bits to help users differentiate between these different
types in the UI, but this at least makes the system work again.

Thanks to @decodeJarvis who initially reported the problem and provided a
proposed fix (see PR#24 on the xzilla repo). Since we needed a more
comprehensive fix, I didn't use the code, but his patch was useful for getting
started.

classes/database/Postgres.php
classes/database/Postgres10.php

index 1eb490a0f7f3741aea7bcf7ba9f895b51e5f85b1..466db5d1e563a666da666195de75cef0cf56e92d 100755 (executable)
@@ -727,7 +727,7 @@ class Postgres extends ADODB_base {
                                AND pa.attname ILIKE {$term} AND pa.attnum > 0 AND NOT pa.attisdropped AND pc.relkind IN ('r', 'v') {$where}
                        UNION ALL
                        SELECT 'FUNCTION', pp.oid, pn.nspname, NULL, pp.proname || '(' || pg_catalog.oidvectortypes(pp.proargtypes) || ')' FROM pg_catalog.pg_proc pp, pg_catalog.pg_namespace pn
-                               WHERE pp.pronamespace=pn.oid AND NOT pp.proisagg AND pp.proname ILIKE {$term} {$where}
+                               WHERE pp.pronamespace=pn.oid AND NOT pp.prokind = 'a' AND pp.proname ILIKE {$term} {$where}
                        UNION ALL
                        SELECT 'INDEX', NULL, pn.nspname, pc.relname, pc2.relname FROM pg_catalog.pg_class pc, pg_catalog.pg_namespace pn,
                                pg_catalog.pg_index pi, pg_catalog.pg_class pc2 WHERE pc.relnamespace=pn.oid AND pc.oid=pi.indrelid
@@ -792,7 +792,7 @@ class Postgres extends ADODB_base {
                                UNION ALL
                                SELECT DISTINCT ON (p.proname) 'AGGREGATE', p.oid, pn.nspname, NULL, p.proname FROM pg_catalog.pg_proc p
                                        LEFT JOIN pg_catalog.pg_namespace pn ON p.pronamespace=pn.oid
-                                       WHERE p.proisagg AND p.proname ILIKE {$term} {$where}
+                                       WHERE p.prokind = 'a' AND p.proname ILIKE {$term} {$where}
                                UNION ALL
                                SELECT DISTINCT ON (po.opcname) 'OPCLASS', po.oid, pn.nspname, NULL, po.opcname FROM pg_catalog.pg_opclass po,
                                        pg_catalog.pg_namespace pn WHERE po.opcnamespace=pn.oid
@@ -4179,12 +4179,18 @@ class Postgres extends ADODB_base {
                                pg_catalog.obj_description(p.oid, 'pg_proc') AS procomment,
                                p.proname || ' (' || pg_catalog.oidvectortypes(p.proargtypes) || ')' AS proproto,
                                CASE WHEN p.proretset THEN 'setof ' ELSE '' END || pg_catalog.format_type(p.prorettype, NULL) AS proreturns,
-                               u.usename AS proowner
+                               u.usename AS proowner,
+                               CASE p.prokind
+                                       WHEN 'a' THEN 'agg'
+                                       WHEN 'w' THEN 'window'
+                                       WHEN 'p' THEN 'proc'
+                                       ELSE 'func'
+                               END as protype
                        FROM pg_catalog.pg_proc p
                                INNER JOIN pg_catalog.pg_namespace n ON n.oid = p.pronamespace
                                INNER JOIN pg_catalog.pg_language pl ON pl.oid = p.prolang
                                LEFT JOIN pg_catalog.pg_user u ON u.usesysid = p.proowner
-                       WHERE NOT p.proisagg
+                       WHERE NOT p.prokind = 'a' 
                                AND {$where}
                        ORDER BY p.proname, proresult
                        ";
@@ -5890,7 +5896,7 @@ class Postgres extends ADODB_base {
                                a.agginitval, a.aggsortop, u.usename, pg_catalog.obj_description(p.oid, 'pg_proc') AS aggrcomment
                        FROM pg_catalog.pg_proc p, pg_catalog.pg_namespace n, pg_catalog.pg_user u, pg_catalog.pg_aggregate a
                        WHERE n.oid = p.pronamespace AND p.proowner=u.usesysid AND p.oid=a.aggfnoid
-                               AND p.proisagg AND n.nspname='{$c_schema}'
+                               AND p.prokind = 'a' AND n.nspname='{$c_schema}'
                                AND p.proname='" . $name . "'
                                AND CASE p.proargtypes[0]
                                        WHEN 'pg_catalog.\"any\"'::pg_catalog.regtype THEN ''
@@ -5912,7 +5918,7 @@ class Postgres extends ADODB_base {
                           pg_catalog.obj_description(p.oid, 'pg_proc') AS aggrcomment
                           FROM pg_catalog.pg_proc p, pg_catalog.pg_namespace n, pg_catalog.pg_user u, pg_catalog.pg_aggregate a
                           WHERE n.oid = p.pronamespace AND p.proowner=u.usesysid AND p.oid=a.aggfnoid
-                          AND p.proisagg AND n.nspname='{$c_schema}' ORDER BY 1, 2";
+                          AND p.prokind = 'a' AND n.nspname='{$c_schema}' ORDER BY 1, 2";
 
                return $this->selectSet($sql);
        }
index 17a0812457724b7b263f2f94193f9f4620b290d6..77bbe45588eb75c53d581906816bd40fd818eebe 100644 (file)
@@ -19,6 +19,253 @@ class Postgres10 extends Postgres11 {
                parent::__construct($conn);
        }
 
+
+       /**
+        * Searches all system catalogs to find objects that match a certain name.
+        * @param $term The search term
+        * @param $filter The object type to restrict to ('' means no restriction)
+        * @return A recordset
+        */
+       function findObject($term, $filter) {
+               global $conf;
+
+               /*about escaping:
+                * SET standard_conforming_string is not available before 8.2
+                * So we must use PostgreSQL specific notation :/
+                * E'' notation is not available before 8.1
+                * $$ is available since 8.0
+                * Nothing specific from 7.4
+                **/
+
+               // Escape search term for ILIKE match
+               $this->clean($term);
+               $this->clean($filter);
+               $term = str_replace('_', '\_', $term);
+               $term = str_replace('%', '\%', $term);
+
+               // Exclude system relations if necessary
+               if (!$conf['show_system']) {
+                       // XXX: The mention of information_schema here is in the wrong place, but
+                       // it's the quickest fix to exclude the info schema from 7.4
+                       $where = " AND pn.nspname NOT LIKE \$_PATERN_\$pg\_%\$_PATERN_\$ AND pn.nspname != 'information_schema'";
+                       $lan_where = "AND pl.lanispl";
+               }
+               else {
+                       $where = '';
+                       $lan_where = '';
+               }
+
+               // Apply outer filter
+               $sql = '';
+               if ($filter != '') {
+                       $sql = "SELECT * FROM (";
+               }
+
+               $term = "\$_PATERN_\$%{$term}%\$_PATERN_\$";
+
+               $sql .= "
+                       SELECT 'SCHEMA' AS type, oid, NULL AS schemaname, NULL AS relname, nspname AS name
+                               FROM pg_catalog.pg_namespace pn WHERE nspname ILIKE {$term} {$where}
+                       UNION ALL
+                       SELECT CASE WHEN relkind='r' THEN 'TABLE' WHEN relkind='v' THEN 'VIEW' WHEN relkind='S' THEN 'SEQUENCE' END, pc.oid,
+                               pn.nspname, NULL, pc.relname FROM pg_catalog.pg_class pc, pg_catalog.pg_namespace pn
+                               WHERE pc.relnamespace=pn.oid AND relkind IN ('r', 'v', 'S') AND relname ILIKE {$term} {$where}
+                       UNION ALL
+                       SELECT CASE WHEN pc.relkind='r' THEN 'COLUMNTABLE' ELSE 'COLUMNVIEW' END, NULL, pn.nspname, pc.relname, pa.attname FROM pg_catalog.pg_class pc, pg_catalog.pg_namespace pn,
+                               pg_catalog.pg_attribute pa WHERE pc.relnamespace=pn.oid AND pc.oid=pa.attrelid
+                               AND pa.attname ILIKE {$term} AND pa.attnum > 0 AND NOT pa.attisdropped AND pc.relkind IN ('r', 'v') {$where}
+                       UNION ALL
+                       SELECT 'FUNCTION', pp.oid, pn.nspname, NULL, pp.proname || '(' || pg_catalog.oidvectortypes(pp.proargtypes) || ')' FROM pg_catalog.pg_proc pp, pg_catalog.pg_namespace pn
+                               WHERE pp.pronamespace=pn.oid AND NOT pp.proisagg AND pp.proname ILIKE {$term} {$where}
+                       UNION ALL
+                       SELECT 'INDEX', NULL, pn.nspname, pc.relname, pc2.relname FROM pg_catalog.pg_class pc, pg_catalog.pg_namespace pn,
+                               pg_catalog.pg_index pi, pg_catalog.pg_class pc2 WHERE pc.relnamespace=pn.oid AND pc.oid=pi.indrelid
+                               AND pi.indexrelid=pc2.oid
+                               AND NOT EXISTS (
+                                       SELECT 1 FROM pg_catalog.pg_depend d JOIN pg_catalog.pg_constraint c
+                                       ON (d.refclassid = c.tableoid AND d.refobjid = c.oid)
+                                       WHERE d.classid = pc2.tableoid AND d.objid = pc2.oid AND d.deptype = 'i' AND c.contype IN ('u', 'p')
+                               )
+                               AND pc2.relname ILIKE {$term} {$where}
+                       UNION ALL
+                       SELECT 'CONSTRAINTTABLE', NULL, pn.nspname, pc.relname, pc2.conname FROM pg_catalog.pg_class pc, pg_catalog.pg_namespace pn,
+                               pg_catalog.pg_constraint pc2 WHERE pc.relnamespace=pn.oid AND pc.oid=pc2.conrelid AND pc2.conrelid != 0
+                               AND CASE WHEN pc2.contype IN ('f', 'c') THEN TRUE ELSE NOT EXISTS (
+                                       SELECT 1 FROM pg_catalog.pg_depend d JOIN pg_catalog.pg_constraint c
+                                       ON (d.refclassid = c.tableoid AND d.refobjid = c.oid)
+                                       WHERE d.classid = pc2.tableoid AND d.objid = pc2.oid AND d.deptype = 'i' AND c.contype IN ('u', 'p')
+                               ) END
+                               AND pc2.conname ILIKE {$term} {$where}
+                       UNION ALL
+                       SELECT 'CONSTRAINTDOMAIN', pt.oid, pn.nspname, pt.typname, pc.conname FROM pg_catalog.pg_type pt, pg_catalog.pg_namespace pn,
+                               pg_catalog.pg_constraint pc WHERE pt.typnamespace=pn.oid AND pt.oid=pc.contypid AND pc.contypid != 0
+                               AND pc.conname ILIKE {$term} {$where}
+                       UNION ALL
+                       SELECT 'TRIGGER', NULL, pn.nspname, pc.relname, pt.tgname FROM pg_catalog.pg_class pc, pg_catalog.pg_namespace pn,
+                               pg_catalog.pg_trigger pt WHERE pc.relnamespace=pn.oid AND pc.oid=pt.tgrelid
+                                       AND ( pt.tgconstraint = 0 OR NOT EXISTS
+                                       (SELECT 1 FROM pg_catalog.pg_depend d JOIN pg_catalog.pg_constraint c
+                                       ON (d.refclassid = c.tableoid AND d.refobjid = c.oid)
+                                       WHERE d.classid = pt.tableoid AND d.objid = pt.oid AND d.deptype = 'i' AND c.contype = 'f'))
+                               AND pt.tgname ILIKE {$term} {$where}
+                       UNION ALL
+                       SELECT 'RULETABLE', NULL, pn.nspname AS schemaname, c.relname AS tablename, r.rulename FROM pg_catalog.pg_rewrite r
+                               JOIN pg_catalog.pg_class c ON c.oid = r.ev_class
+                               LEFT JOIN pg_catalog.pg_namespace pn ON pn.oid = c.relnamespace
+                               WHERE c.relkind='r' AND r.rulename != '_RETURN' AND r.rulename ILIKE {$term} {$where}
+                       UNION ALL
+                       SELECT 'RULEVIEW', NULL, pn.nspname AS schemaname, c.relname AS tablename, r.rulename FROM pg_catalog.pg_rewrite r
+                               JOIN pg_catalog.pg_class c ON c.oid = r.ev_class
+                               LEFT JOIN pg_catalog.pg_namespace pn ON pn.oid = c.relnamespace
+                               WHERE c.relkind='v' AND r.rulename != '_RETURN' AND r.rulename ILIKE {$term} {$where}
+               ";
+
+
+               // Add advanced objects if show_advanced is set
+               if ($conf['show_advanced']) {
+                       $sql .= "
+                               UNION ALL
+                               SELECT CASE WHEN pt.typtype='d' THEN 'DOMAIN' ELSE 'TYPE' END, pt.oid, pn.nspname, NULL,
+                                       pt.typname FROM pg_catalog.pg_type pt, pg_catalog.pg_namespace pn
+                                       WHERE pt.typnamespace=pn.oid AND typname ILIKE {$term}
+                                       AND (pt.typrelid = 0 OR (SELECT c.relkind = 'c' FROM pg_catalog.pg_class c WHERE c.oid = pt.typrelid))
+                                       {$where}
+                               UNION ALL
+                               SELECT 'OPERATOR', po.oid, pn.nspname, NULL, po.oprname FROM pg_catalog.pg_operator po, pg_catalog.pg_namespace pn
+                                       WHERE po.oprnamespace=pn.oid AND oprname ILIKE {$term} {$where}
+                               UNION ALL
+                               SELECT 'CONVERSION', pc.oid, pn.nspname, NULL, pc.conname FROM pg_catalog.pg_conversion pc,
+                                       pg_catalog.pg_namespace pn WHERE pc.connamespace=pn.oid AND conname ILIKE {$term} {$where}
+                               UNION ALL
+                               SELECT 'LANGUAGE', pl.oid, NULL, NULL, pl.lanname FROM pg_catalog.pg_language pl
+                                       WHERE lanname ILIKE {$term} {$lan_where}
+                               UNION ALL
+                               SELECT DISTINCT ON (p.proname) 'AGGREGATE', p.oid, pn.nspname, NULL, p.proname FROM pg_catalog.pg_proc p
+                                       LEFT JOIN pg_catalog.pg_namespace pn ON p.pronamespace=pn.oid
+                                       WHERE p.proisagg AND p.proname ILIKE {$term} {$where}
+                               UNION ALL
+                               SELECT DISTINCT ON (po.opcname) 'OPCLASS', po.oid, pn.nspname, NULL, po.opcname FROM pg_catalog.pg_opclass po,
+                                       pg_catalog.pg_namespace pn WHERE po.opcnamespace=pn.oid
+                                       AND po.opcname ILIKE {$term} {$where}
+                       ";
+               }
+               // Otherwise just add domains
+               else {
+                       $sql .= "
+                               UNION ALL
+                               SELECT 'DOMAIN', pt.oid, pn.nspname, NULL,
+                                       pt.typname FROM pg_catalog.pg_type pt, pg_catalog.pg_namespace pn
+                                       WHERE pt.typnamespace=pn.oid AND pt.typtype='d' AND typname ILIKE {$term}
+                                       AND (pt.typrelid = 0 OR (SELECT c.relkind = 'c' FROM pg_catalog.pg_class c WHERE c.oid = pt.typrelid))
+                                       {$where}
+                       ";
+               }
+
+               if ($filter != '') {
+                       // We use like to make RULE, CONSTRAINT and COLUMN searches work
+                       $sql .= ") AS sub WHERE type LIKE '{$filter}%' ";
+               }
+
+               $sql .= "ORDER BY type, schemaname, relname, name";
+
+               return $this->selectSet($sql);
+       }
+
+       /**
+        * Returns a list of all functions in the database
+        * @param $all If true, will find all available functions, if false just those in search path
+        * @param $type If not null, will find all functions with return value = type
+        *
+        * @return All functions
+        */
+       function getFunctions($all = false, $type = null) {
+               if ($all) {
+                       $where = 'pg_catalog.pg_function_is_visible(p.oid)';
+                       $distinct = 'DISTINCT ON (p.proname)';
+
+                       if ($type) {
+                               $where .= " AND p.prorettype = (select oid from pg_catalog.pg_type p where p.typname = 'trigger') ";
+                       }
+               }
+               else {
+                       $c_schema = $this->_schema;
+                       $this->clean($c_schema);
+                       $where = "n.nspname = '{$c_schema}'";
+                       $distinct = '';
+               }
+
+               $sql = "
+                       SELECT
+                               {$distinct}
+                               p.oid AS prooid,
+                               p.proname,
+                               p.proretset,
+                               pg_catalog.format_type(p.prorettype, NULL) AS proresult,
+                               pg_catalog.oidvectortypes(p.proargtypes) AS proarguments,
+                               pl.lanname AS prolanguage,
+                               pg_catalog.obj_description(p.oid, 'pg_proc') AS procomment,
+                               p.proname || ' (' || pg_catalog.oidvectortypes(p.proargtypes) || ')' AS proproto,
+                               CASE WHEN p.proretset THEN 'setof ' ELSE '' END || pg_catalog.format_type(p.prorettype, NULL) AS proreturns,
+                               u.usename AS proowner
+                       FROM pg_catalog.pg_proc p
+                               INNER JOIN pg_catalog.pg_namespace n ON n.oid = p.pronamespace
+                               INNER JOIN pg_catalog.pg_language pl ON pl.oid = p.prolang
+                               LEFT JOIN pg_catalog.pg_user u ON u.usesysid = p.proowner
+                       WHERE NOT p.proisagg
+                               AND {$where}
+                       ORDER BY p.proname, proresult
+                       ";
+
+               return $this->selectSet($sql);
+       }
+
+       /**
+        * Gets all information for an aggregate
+        * @param $name The name of the aggregate
+        * @param $basetype The input data type of the aggregate
+        * @return A recordset
+        */
+       function getAggregate($name, $basetype) {
+               $c_schema = $this->_schema;
+               $this->clean($c_schema);
+               $this->fieldclean($name);
+               $this->fieldclean($basetype);
+
+               $sql = "
+                       SELECT p.proname, CASE p.proargtypes[0]
+                               WHEN 'pg_catalog.\"any\"'::pg_catalog.regtype THEN NULL
+                               ELSE pg_catalog.format_type(p.proargtypes[0], NULL) END AS proargtypes,
+                               a.aggtransfn, format_type(a.aggtranstype, NULL) AS aggstype, a.aggfinalfn,
+                               a.agginitval, a.aggsortop, u.usename, pg_catalog.obj_description(p.oid, 'pg_proc') AS aggrcomment
+                       FROM pg_catalog.pg_proc p, pg_catalog.pg_namespace n, pg_catalog.pg_user u, pg_catalog.pg_aggregate a
+                       WHERE n.oid = p.pronamespace AND p.proowner=u.usesysid AND p.oid=a.aggfnoid
+                               AND p.proisagg AND n.nspname='{$c_schema}'
+                               AND p.proname='" . $name . "'
+                               AND CASE p.proargtypes[0]
+                                       WHEN 'pg_catalog.\"any\"'::pg_catalog.regtype THEN ''
+                                       ELSE pg_catalog.format_type(p.proargtypes[0], NULL)
+                               END ='" . $basetype . "'";
+
+               return $this->selectSet($sql);
+       }
+
+       /**
+        * Gets all aggregates
+        * @return A recordset
+        */
+       function getAggregates() {
+               $c_schema = $this->_schema;
+               $this->clean($c_schema);
+               $sql = "SELECT p.proname, CASE p.proargtypes[0] WHEN 'pg_catalog.\"any\"'::pg_catalog.regtype THEN NULL ELSE
+                          pg_catalog.format_type(p.proargtypes[0], NULL) END AS proargtypes, a.aggtransfn, u.usename,
+                          pg_catalog.obj_description(p.oid, 'pg_proc') AS aggrcomment
+                          FROM pg_catalog.pg_proc p, pg_catalog.pg_namespace n, pg_catalog.pg_user u, pg_catalog.pg_aggregate a
+                          WHERE n.oid = p.pronamespace AND p.proowner=u.usesysid AND p.oid=a.aggfnoid
+                          AND p.proisagg AND n.nspname='{$c_schema}' ORDER BY 1, 2";
+
+               return $this->selectSet($sql);
+       }
+
        // Help functions
 
        function getHelpPages() {