public inbox for [email protected]
help / color / mirror / Atom feedFrom: Dave Page <[email protected]>
To: Surinder Kumar <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: [pgAdmin4] PATCH: View and Role Node
Date: Thu, 25 Feb 2016 11:03:53 +0000
Message-ID: <CA+OCxowWwR70-w=674UQXfZkAiCnmzdnqj7mCSwGFfT4DymWzw@mail.gmail.com> (raw)
In-Reply-To: <CAM5-9D_08pXJ_G4QpN89TuZR3bDShVk4sFG5Ee8rJ8o0JW1b9w@mail.gmail.com>
References: <CAM5-9D_qRCubJ8CE+7XsNyHKDF=JixqiOCZcZQK+Gr1ATM-OAg@mail.gmail.com>
<CAM5-9D_08pXJ_G4QpN89TuZR3bDShVk4sFG5Ee8rJ8o0JW1b9w@mail.gmail.com>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgadmin-hackers>
Hi
On Wed, Feb 24, 2016 at 1:31 PM, Surinder Kumar <
[email protected]> wrote:
> Hi,
>
> Please find the updated patch with following changes:
>
> 1. Integrated dependents and dependency tabs with view and rule node.
> 2. Added "Create Rule" option for Table node.
> 3. Added node_inode property for rule node as it has no child node.
> 4. Improved javascript code commenting and added pydoc in python files.
> 5. Name column under dependents tab showing wrong name for rule node
> due to typo mistake, It is fixed.
>
> Please review it and Let me know for any comments.
>
Please find an updated patch attached, in which I tweaked some comments and
fixed a few minor issues as I reviewed the code. The following issues need
to be addressed please, then I'll take another look:
- Don't show the definition SQL in the main properties tab.
- Check Options is missing from the properties tab.
- Views were made function in PostgreSQL v6.4! Why only load the module for
9.1+?
- The TRUNCATE ACL flag is not recognised, leading to ACLs being shown as:
arwdUNKNOWNxt
- 4 character indents are not applied consistently to SQL templates
- Why do servers pre-9.3 have VACUUM options in the SQL templates?
- There should be no special casing of Slony schemas (backend_support.sql)
- Don't include \'s on line ends of multiline comments or docstrings.
- I can't create a Materialised View
On the dialogs:
- The "Definition" input box should have a border.
- The "Definition" input box fill the vertical space assigned to it, such
that the border on the left runs top to bottom (see the main SQL tab).
- The "Definition" input box should be displayed below the label at all
times (unlike other controls), to maximise available space
- The SQL tab view is broken for views (tested on PG 9.4), likely due to
the TRUNCATE ACL issue:
2016-02-25 10:59:00,118: INFO werkzeug: 127.0.0.1 - - [25/Feb/2016
10:59:00] "GET /browser/view/sql/1/1/24587/27424/27555 HTTP/1.1" 500 -
Traceback (most recent call last):
File
"/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
line 1836, in __call__
return self.wsgi_app(environ, start_response)
File
"/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
line 1820, in wsgi_app
response = self.make_response(self.handle_exception(e))
File
"/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
line 1403, in handle_exception
reraise(exc_type, exc_value, tb)
File
"/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
line 1817, in wsgi_app
response = self.full_dispatch_request()
File
"/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
line 1477, in full_dispatch_request
rv = self.handle_user_exception(e)
File
"/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
line 1381, in handle_user_exception
reraise(exc_type, exc_value, tb)
File
"/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
line 1475, in full_dispatch_request
rv = self.dispatch_request()
File
"/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/app.py",
line 1461, in dispatch_request
return self.view_functions[rule.endpoint](**req.view_args)
File
"/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/flask/views.py",
line 84, in view
return self.dispatch_request(*args, **kwargs)
File "/Users/dpage/git/pgadmin4/web/pgadmin/browser/utils.py", line 248,
in dispatch_request
return method(*args, **kwargs)
File
"/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_groups/servers/databases/schemas/views/__init__.py",
line 242, in wrap
return f(*args, **kwargs)
File
"/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_groups/servers/databases/schemas/views/__init__.py",
line 646, in sql
result['datacl'] = self.parse_privileges(result['datacl'])
File
"/Users/dpage/git/pgadmin4/web/pgadmin/browser/server_groups/servers/databases/schemas/views/__init__.py",
line 540, in parse_privileges
db_privileges[privilege['privilege_type']])
KeyError: 'UNKNOWN'
Thanks!
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
view thread (7+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected]
Subject: Re: [pgAdmin4] PATCH: View and Role Node
In-Reply-To: <CA+OCxowWwR70-w=674UQXfZkAiCnmzdnqj7mCSwGFfT4DymWzw@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox