[pgAdmin4][patch] error message is incorrect in domains

Поиск
Список
Период
Сортировка
От zhangjie2@fujitsu.com
Тема [pgAdmin4][patch] error message is incorrect in domains
Дата
Msg-id TYWPR01MB7678393FB87082348AEDA114F9A79@TYWPR01MB7678.jpnprd01.prod.outlook.com
обсуждение исходный текст
Ответы Re: [pgAdmin4][patch] error message is incorrect in domains  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Список pgadmin-hackers
Hi, all

In domains's create function, res is used for all error messages.
I don't think this is correct.
On line 13, doid should be used as the error message.Because the execution result of line 11 is doid.
On line 21, scid should be used as the error message.Because the execution result of line 19 is scid.
There are similar problems in other functions.
Here is a patch for domains.
Please review.

file: pgadmin4\web\pgadmin\browser\server_groups\servers\databases\schemas\domains\__init__.py
----------------------------------------------------------------------------
   def create(self, gid, sid, did, scid):
        ......
 1       status, res = self.conn.execute_scalar(SQL)   
 2       if not status:
 3           return internal_server_error(errormsg=res) ※1   res is OK
 4
 5       # We need oid to add object in tree at browser, below sql will
 6       # gives the same
 7       SQL = render_template("/".join([self.template_path,
 8                                       self._OID_SQL]),
 9                             basensp=data['basensp'],
 10                             name=data['name'])
 11       status, doid = self.conn.execute_scalar(SQL)
 12       if not status:
 13           return internal_server_error(errormsg=res)   ※2  res => doid
 14
 15       # Get updated schema oid
 16       SQL = render_template("/".join([self.template_path,
 17                                       self._OID_SQL]),
 18                             doid=doid)
 19       status, scid = self.conn.execute_scalar(SQL)
 20       if not status:
 21           return internal_server_error(errormsg=res)   ※3  res => scid 
----------------------------------------------------------------------------------

Вложения

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Khushboo Vashi
Дата:
Сообщение: Re: [pgAdmin4][Patch] - RM #6129 - Port browser tree to React
Следующее
От: Akshay Joshi
Дата:
Сообщение: pgAdmin 4 commit: Correct the variable names in the error messages for