dpage
commented
5 years ago Comment migrated from Redmine: https://redmine.postgresql.org/issues/4529#note-1 Originally created by Girish Shenoy at 2019-07-29 04:37:49 UTC.
Resolution (formatted): We need to add the following code at line#524 (before we call transaction_status = conn.transaction_status() )
#Procedure/Function output may comes in the form of Notices from the
#database server, so we need to append those outputs with the
#original result.
if status == 'Success' and result is None:
result = conn.status_message()
if (result != 'SELECT 1' or result != 'SELECT 0') and \
result is not None and additional_messages:
result = additional_messages + result
Issue migrated from Redmine: https://redmine.postgresql.org/issues/4529 Originally created by Girish Shenoy at 2019-07-29 04:25:26 UTC.
When we execute an UPDATE query, the messages used to show === UPDATE 1
Query returned successfully in 52 msec.
Now it shows only
Query returned successfully in 52 msec.
Effect: When insert/update/delete statements are fired, we need to search for query rows affected in Query History. This becomes very cumbersome when we are inside a transaction and doing multiple steps. We do not know if the intended number of rows were affected or not. Notifications sent from functions are also not displayed.
Resolution: This change is after commit#710d5206 in web/pgadmin/tools/sqleditor/init.py
We need to add the following code at line#524 (before we call transaction_status = conn.transaction_status() )