pgAdmin 4 commit: Infrastructure and changes to the Query Tool for real

From: Dave Page <dpage(at)pgadmin(dot)org>
To: pgadmin-hackers(at)postgresql(dot)org
Subject: pgAdmin 4 commit: Infrastructure and changes to the Query Tool for real
Date: 2018-07-05 10:38:52
Message-ID: E1fb1f2-0006aT-PR@gothos.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Infrastructure and changes to the Query Tool for realtime preference handling. Refs #3294

Highlights of this patch include:
- Changes will affect SQL Editors in Create dialog boxes, SQL tab of the main screen, Query tool, History entries in the query tool, Query tool opened in New Tab/Window
- All the components of SQL editor will refer to single source of preferences which is cached in the Browser object. All other redundant ajax get preference calls are removed.
- SQL editor will not refer template JS variables anymore, once all the references are removed the template variables will also be removed.
- Code refactoring wherever possible.
- Covered JS test cases wherever possible.

Branch
------
master

Details
-------
https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=bdb7e3fde2ff3e3895181469ce24d533e5c7cfbf
Author: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>

Modified Files
--------------
web/pgadmin/browser/__init__.py | 11 -
web/pgadmin/browser/static/css/browser.css | 7 +
web/pgadmin/browser/static/js/browser.js | 71 +---
web/pgadmin/preferences/static/js/preferences.js | 12 +-
web/pgadmin/static/js/backform.pgadmin.js | 73 +++-
web/pgadmin/static/js/keyboard_shortcuts.js | 66 ++-
.../static/js/sqleditor/query_tool_actions.js | 52 ---
web/pgadmin/static/js/sqleditor_utils.js | 27 +-
.../static/jsx/history/detail/code_mirror.jsx | 5 +
.../jsx/history/detail/history_detail_query.jsx | 2 +
web/pgadmin/static/jsx/history/query_history.jsx | 5 +-
web/pgadmin/static/jsx/react_shapes.jsx | 6 +
web/pgadmin/tools/datagrid/__init__.py | 40 +-
web/pgadmin/tools/datagrid/static/js/datagrid.js | 38 +-
.../tools/datagrid/templates/datagrid/index.html | 93 ++---
.../tools/sqleditor/static/css/sqleditor.css | 16 +
web/pgadmin/tools/sqleditor/static/js/sqleditor.js | 445 +++++++--------------
.../tools/sqleditor/tests/test_pref_utilities.py | 100 -----
.../sqleditor/utils/query_tool_preferences.py | 120 ------
.../javascript/history/query_history_spec.jsx | 15 +-
.../sqleditor/keyboard_shortcuts_spec.js | 141 ++++---
web/regression/javascript/sqleditor_utils_spec.js | 8 +
web/webpack.shim.js | 2 +
web/webpack.test.config.js | 1 +
24 files changed, 517 insertions(+), 839 deletions(-)

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2018-07-05 10:39:29 Re: [pgAdmin4][RM3294] User need to reset the layout to see the changed preferences parameters
Previous Message Akshay Joshi 2018-07-05 10:21:18 Re: [pgAdmin4][Patch]: RM #3397 Add support for JIT stats in EXPLAIN output in PG11