pgAdmin 4 commit: Sorting the data during tree data loading using the n

From: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
To: pgadmin-hackers(at)postgresql(dot)org
Subject: pgAdmin 4 commit: Sorting the data during tree data loading using the n
Date: 2016-08-29 14:40:09
Message-ID: E1beNjN-00059A-M8@gothos.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Sorting the data during tree data loading using the natural sort
algorithm.

This patch takes care of:
* Consistent behaviour during create, update operation on any node.
- It should return the node data during creating a new object, or
updating the existing one.
* Now that - we have consistent behaviour during these operations, we
can consistently move, update the tree item based the node data from
the server.
* Implemented the methods for each node to get the information about
that particular node only.
* Using the above changes to implement the 'refresh' operation on tree
node properly.

I must thank Surinder, and Harshal for helping me implement the 'node'
method, and also changed the behaviour of create & update methods to
return the node data for most of the nodes.

Fixes #1244

Branch
------
master

Details
-------
http://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=b6f307256b7a48944d9e5a198d0d461b3ac5c5d5

Modified Files
--------------
libraries.txt | 1 +
web/pgadmin/browser/collection.py | 2 +-
web/pgadmin/browser/server_groups/__init__.py | 50 +-
.../browser/server_groups/servers/__init__.py | 16 +-
.../server_groups/servers/databases/__init__.py | 34 +-
.../servers/databases/casts/__init__.py | 127 ++-
.../servers/databases/event_triggers/__init__.py | 51 +-
.../event_triggers/sql/9.3_plus/nodes.sql | 3 +
.../servers/databases/extensions/__init__.py | 133 ++--
.../databases/foreign_data_wrappers/__init__.py | 253 +++---
.../foreign_servers/__init__.py | 264 +++---
.../foreign_servers/user_mapping/__init__.py | 260 +++---
.../servers/databases/languages/__init__.py | 155 ++--
.../servers/databases/schemas/__init__.py | 104 ++-
.../databases/schemas/catalog_objects/__init__.py | 40 +
.../schemas/catalog_objects/columns/__init__.py | 6 +-
.../catalog_object/sql/pg/9.1_plus/nodes.sql | 4 +
.../catalog_object/sql/ppas/9.1_plus/nodes.sql | 4 +
.../databases/schemas/collations/__init__.py | 169 ++--
.../templates/collation/sql/9.1_plus/get_oid.sql | 11 +-
.../templates/collation/sql/9.1_plus/nodes.sql | 4 +
.../servers/databases/schemas/domains/__init__.py | 298 +++----
.../schemas/domains/domain_constraints/__init__.py | 48 ++
.../domain_constraints/sql/9.1_plus/properties.sql | 5 +
.../domain_constraints/sql/9.2_plus/properties.sql | 5 +
.../templates/domains/sql/9.1_plus/node.sql | 4 +
.../templates/domains/sql/9.2_plus/node.sql | 4 +
.../databases/schemas/foreign_tables/__init__.py | 272 ++++---
.../templates/foreign_tables/sql/9.1_plus/node.sql | 4 +
.../templates/foreign_tables/sql/9.2_plus/node.sql | 4 +
.../templates/foreign_tables/sql/9.5_plus/node.sql | 4 +
.../schemas/fts_configurations/__init__.py | 338 ++++----
.../fts_configuration/sql/9.1_plus/properties.sql | 7 +-
.../fts_configuration/sql/9.1_plus/update.sql | 4 +-
.../databases/schemas/fts_dictionaries/__init__.py | 320 ++++----
.../fts_dictionary/sql/9.1_plus/properties.sql | 9 +-
.../databases/schemas/fts_parser/__init__.py | 310 ++++----
.../templates/fts_parser/sql/9.1_plus/nodes.sql | 4 +-
.../fts_parser/sql/9.1_plus/properties.sql | 10 +-
.../databases/schemas/fts_templates/__init__.py | 399 +++++-----
.../templates/fts_template/sql/9.1_plus/nodes.sql | 4 +-
.../fts_template/sql/9.1_plus/properties.sql | 9 +-
.../templates/fts_template/sql/9.1_plus/update.sql | 4 +-
.../databases/schemas/functions/__init__.py | 497 ++++++------
.../templates/function/pg/sql/9.1_plus/get_oid.sql | 2 +-
.../templates/function/pg/sql/9.1_plus/node.sql | 5 +
.../function/pg/sql/9.1_plus/properties.sql | 5 +-
.../templates/function/pg/sql/9.2_plus/get_oid.sql | 2 +-
.../templates/function/pg/sql/9.2_plus/node.sql | 5 +
.../function/pg/sql/9.2_plus/properties.sql | 3 +-
.../templates/function/pg/sql/9.5_plus/get_oid.sql | 2 +-
.../templates/function/pg/sql/9.5_plus/node.sql | 5 +
.../function/pg/sql/9.5_plus/properties.sql | 5 +-
.../function/ppas/sql/9.1_plus/get_oid.sql | 2 +-
.../templates/function/ppas/sql/9.1_plus/node.sql | 5 +
.../function/ppas/sql/9.1_plus/properties.sql | 3 +-
.../function/ppas/sql/9.2_plus/get_oid.sql | 2 +-
.../templates/function/ppas/sql/9.2_plus/node.sql | 5 +
.../function/ppas/sql/9.2_plus/properties.sql | 3 +-
.../function/ppas/sql/9.5_plus/get_oid.sql | 2 +-
.../templates/function/ppas/sql/9.5_plus/node.sql | 5 +
.../function/ppas/sql/9.5_plus/properties.sql | 3 +-
.../procedure/ppas/sql/9.1_plus/get_oid.sql | 2 +-
.../templates/procedure/ppas/sql/9.1_plus/node.sql | 5 +
.../procedure/ppas/sql/9.1_plus/properties.sql | 3 +-
.../procedure/ppas/sql/9.2_plus/get_oid.sql | 2 +-
.../templates/procedure/ppas/sql/9.2_plus/node.sql | 5 +
.../procedure/ppas/sql/9.2_plus/properties.sql | 3 +-
.../procedure/ppas/sql/9.5_plus/get_oid.sql | 2 +-
.../templates/procedure/ppas/sql/9.5_plus/node.sql | 5 +
.../procedure/ppas/sql/9.5_plus/properties.sql | 3 +-
.../trigger_function/pg/sql/9.1_plus/get_oid.sql | 2 +-
.../trigger_function/pg/sql/9.1_plus/node.sql | 5 +
.../pg/sql/9.1_plus/properties.sql | 3 +-
.../trigger_function/pg/sql/9.2_plus/get_oid.sql | 2 +-
.../trigger_function/pg/sql/9.2_plus/node.sql | 7 +-
.../pg/sql/9.2_plus/properties.sql | 3 +-
.../trigger_function/pg/sql/9.5_plus/get_oid.sql | 2 +-
.../trigger_function/pg/sql/9.5_plus/node.sql | 5 +
.../pg/sql/9.5_plus/properties.sql | 3 +-
.../trigger_function/ppas/sql/9.1_plus/get_oid.sql | 2 +-
.../trigger_function/ppas/sql/9.1_plus/node.sql | 5 +
.../ppas/sql/9.1_plus/properties.sql | 3 +-
.../trigger_function/ppas/sql/9.2_plus/get_oid.sql | 2 +-
.../trigger_function/ppas/sql/9.2_plus/node.sql | 5 +
.../ppas/sql/9.2_plus/properties.sql | 3 +-
.../trigger_function/ppas/sql/9.5_plus/get_oid.sql | 2 +-
.../trigger_function/ppas/sql/9.5_plus/node.sql | 5 +
.../ppas/sql/9.5_plus/properties.sql | 3 +-
.../servers/databases/schemas/packages/__init__.py | 230 +++---
.../schemas/packages/edbfuncs/__init__.py | 27 +-
.../templates/edbfunc/ppas/9.1_plus/node.sql | 8 +-
.../templates/edbfunc/ppas/9.2_plus/node.sql | 7 +-
.../templates/edbproc/ppas/9.1_plus/node.sql | 7 +-
.../templates/edbproc/ppas/9.2_plus/node.sql | 7 +-
.../databases/schemas/packages/edbvars/__init__.py | 8 +-
.../edbvars/templates/edbvar/ppas/node.sql | 3 +
.../databases/schemas/sequences/__init__.py | 162 ++--
.../templates/sequence/sql/9.1_plus/get_oid.sql | 8 +-
.../templates/sequence/sql/9.1_plus/nodes.sql | 13 +-
.../servers/databases/schemas/tables/__init__.py | 50 +-
.../databases/schemas/tables/column/__init__.py | 139 ++--
.../schemas/tables/constraints/__init__.py | 14 +-
.../constraints/check_constraint/__init__.py | 240 ++++--
.../constraints/exclusion_constraint/__init__.py | 235 ++++--
.../js/exclusion_constraint.js | 3 -
.../tables/constraints/foreign_key/__init__.py | 275 ++++---
.../constraints/index_constraint/__init__.py | 167 ++--
.../databases/schemas/tables/indexes/__init__.py | 96 ++-
.../databases/schemas/tables/rules/__init__.py | 114 +--
.../tables/templates/column/sql/9.1_plus/nodes.sql | 8 +-
.../tables/templates/column/sql/9.2_plus/nodes.sql | 8 +-
.../schemas/tables/templates/rules/sql/nodes.sql | 13 +
.../table/sql/9.1_plus/get_schema_oid.sql | 9 +
.../table/sql/9.5_plus/get_schema_oid.sql | 9 +
.../databases/schemas/tables/triggers/__init__.py | 80 +-
.../servers/databases/schemas/types/__init__.py | 105 ++-
.../types/templates/type/sql/9.1_plus/nodes.sql | 3 +
.../servers/databases/schemas/views/__init__.py | 566 ++++++-------
.../databases/schemas/views/children/__init__.py | 11 +
.../schemas/views/templates/mview/js/mview.js | 6 +-
.../templates/mview/pg/9.3_plus/sql/get_oid.sql | 9 +
.../templates/mview/pg/9.3_plus/sql/nodes.sql | 13 +
.../templates/mview/pg/9.4_plus/sql/get_oid.sql | 9 +
.../templates/mview/pg/9.4_plus/sql/nodes.sql | 13 +
.../templates/mview/ppas/9.3_plus/sql/get_oid.sql | 9 +
.../templates/mview/ppas/9.3_plus/sql/nodes.sql | 13 +
.../schemas/views/templates/view/js/view.js | 6 +-
.../templates/view/pg/9.1_plus/sql/get_oid.sql | 9 +
.../views/templates/view/pg/9.1_plus/sql/nodes.sql | 13 +
.../templates/view/pg/9.2_plus/sql/get_oid.sql | 9 +
.../views/templates/view/pg/9.2_plus/sql/nodes.sql | 13 +
.../templates/view/pg/9.3_plus/sql/get_oid.sql | 9 +
.../views/templates/view/pg/9.3_plus/sql/nodes.sql | 13 +
.../templates/view/pg/9.4_plus/sql/get_oid.sql | 9 +
.../views/templates/view/pg/9.4_plus/sql/nodes.sql | 13 +
.../templates/view/ppas/9.1_plus/sql/get_oid.sql | 9 +
.../templates/view/ppas/9.1_plus/sql/nodes.sql | 13 +
.../templates/view/ppas/9.2_plus/sql/get_oid.sql | 9 +
.../templates/view/ppas/9.2_plus/sql/nodes.sql | 13 +
.../templates/view/ppas/9.3_plus/sql/get_oid.sql | 9 +
.../templates/view/ppas/9.3_plus/sql/nodes.sql | 13 +
.../templates/view/ppas/9.4_plus/sql/get_oid.sql | 9 +
.../templates/view/ppas/9.4_plus/sql/nodes.sql | 13 +
.../servers/resource_groups/__init__.py | 130 +--
.../templates/resource_groups/sql/nodes.sql | 6 +
.../server_groups/servers/tablespaces/__init__.py | 86 +-
.../servers/templates/servers/servers.js | 3 +
.../browser/templates/browser/js/browser.js | 881 ++++++++++++++++++++-
.../browser/templates/browser/js/collection.js | 3 +-
.../browser/templates/browser/js/messages.js | 2 +-
web/pgadmin/browser/templates/browser/js/node.js | 332 ++------
web/pgadmin/misc/depends/static/js/depends.js | 16 +-
web/pgadmin/misc/sql/static/js/sql.js | 13 +-
.../misc/statistics/static/js/statistics.js | 20 +-
web/pgadmin/static/js/pgadmin.js | 46 ++
156 files changed, 5363 insertions(+), 3464 deletions(-)

Browse pgadmin-hackers by date

  From Date Subject
Next Message Ashesh Vashi 2016-08-29 14:55:38 pgAdmin 4 commit: Fixes issues specific to SYNONYMS
Previous Message Murtuza Zabuawala 2016-08-29 14:19:50 PATCH: To fix the issue in cast node (pgAdmin4)