Re: [pgAdmin4] [Patch]: Language Module

Hi Akshay,

Please find the review comments below for Language node.

- On postgres9.1 language node failed with below error when clicked on properties or sql tab.
   function expression in FROM cannot refer to other relations of same query level LINE 15: aclexplode(lanacl) d ^

- Error when tried to update language name, owner, comment.

File "/home/edb/PRJ/pgadmin4/web/pgadmin/browser/utils.py", line 277, in dispatch_request
    return method(*args, **kwargs)
  File "/home/edb/PRJ/pgadmin4/web/pgadmin/browser/server_groups/servers/databases/languages/__init__.py", line 139, in wrap
    return f(*args, **kwargs)
  File "/home/edb/PRJ/pgadmin4/web/pgadmin/browser/server_groups/servers/databases/languages/__init__.py", line 224, in update
    sql = self.get_sql(gid, sid, data, did, lid)
TypeError: get_sql() takes from 2 to 3 positional arguments but 6 were given

 

- In pgadmin3 it shows drop down for language name while in pgadmin4 in edit mode it shows input text.

Need some clarification whether user is allowed to enter any language name or can only select from predefined language names.

Regards,
~Harshal

-- 

Harshal Dhumal

Software Engineer 

EenterpriseDB

On Mon, Feb 8, 2016 at 6:17 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:

Hi All

Attached is the patch for the Language Module. Please review it and if it looks good then please commit it. 

--

Akshay Joshi

Principal Software Engineer 

Phone: +91 20-3058-9517
Mobile: +91 976-788-8246

--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers