Thread: [pgAdmin4][Patch]: RM #3461 Properties/SQL panel contents does notupdates when user perform Refresh operation on the node
[pgAdmin4][Patch]: RM #3461 Properties/SQL panel contents does notupdates when user perform Refresh operation on the node
From
Akshay Joshi
Date:
Hi Hackers,
--

Attached is the patch to fix RM #3461 "Properties/SQL panel contents does not updates when user perform Refresh operation on the node". Please review it.
Akshay Joshi
Sr. Software Architect

Phone: +91 20-3058-9517
Mobile: +91 976-788-8246
Mobile: +91 976-788-8246
Attachment
Re: [pgAdmin4][Patch]: RM #3461 Properties/SQL panel contents doesnot updates when user perform Refresh operation on the node
From
Dave Page
Date:
Thanks, patch applied.
On Mon, Aug 13, 2018 at 12:01 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Hackers,Attached is the patch to fix RM #3461 "Properties/SQL panel contents does not updates when user perform Refresh operation on the node". Please review it.--Akshay JoshiSr. Software ArchitectPhone: +91 20-3058-9517
Mobile: +91 976-788-8246
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Re: [pgAdmin4][Patch]: RM #3461 Properties/SQL panel contents does not updates when user perform Refresh operation on the node
From
Murtuza Zabuawala
Date:
Hi Akshay,
I've tested and found one issue,
Steps:
1) Expand to any Table (eg: 'abc') with a Trigger ('trig_on_abc')
2) Right click on that 'trig_on_abc' trigger and open "Properties" dialog, Goto > "Definition" tab
3) Toggle the value of 'Trigger enabled?'
4) Go to the modified "SQL" tab and just copy the generated SQL & close the dialog (do not save, just close it)
5) Open the "Query tool" on the same and paste the SQL from the previous step and execute it (commit it)
6) Now again goto that 'trig_on_abc' in the browser tree and right click and refresh.
7) You will see that icon is not getting updated on the "Refresh", though I'm receiving proper icon in the response.
(Attaching the screenshot)
Expected: Let's say If I've disabled the Trigger via SQL then Disable trigger icon should get reflected when we Refresh.
--
Regards,
On Mon, Aug 13, 2018 at 6:18 PM, Dave Page <dpage@pgadmin.org> wrote:
Thanks, patch applied.On Mon, Aug 13, 2018 at 12:01 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote: Hi Hackers,Attached is the patch to fix RM #3461 "Properties/SQL panel contents does not updates when user perform Refresh operation on the node". Please review it.--Akshay JoshiSr. Software ArchitectPhone: +91 20-3058-9517
Mobile: +91 976-788-8246--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment
Re: [pgAdmin4][Patch]: RM #3461 Properties/SQL panel contents doesnot updates when user perform Refresh operation on the node
From
Ashesh Vashi
Date:
On Mon, Aug 13, 2018 at 9:56 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi Akshay,I've tested and found one issue,Steps:1) Expand to any Table (eg: 'abc') with a Trigger ('trig_on_abc')2) Right click on that 'trig_on_abc' trigger and open "Properties" dialog, Goto > "Definition" tab3) Toggle the value of 'Trigger enabled?'4) Go to the modified "SQL" tab and just copy the generated SQL & close the dialog (do not save, just close it)5) Open the "Query tool" on the same and paste the SQL from the previous step and execute it (commit it)6) Now again goto that 'trig_on_abc' in the browser tree and right click and refresh.7) You will see that icon is not getting updated on the "Refresh", though I'm receiving proper icon in the response.(Attaching the screenshot)Expected: Let's say If I've disabled the Trigger via SQL then Disable trigger icon should get reflected when we Refresh.
It is not result of this patch.
This patch only refreshes the SQL, statistics, dependents & dependencies panel for the selected tree item.
Please send a separate patch for the same (if time permits).
-- Thanks, Ashesh
--Regards,On Mon, Aug 13, 2018 at 6:18 PM, Dave Page <dpage@pgadmin.org> wrote:Thanks, patch applied.On Mon, Aug 13, 2018 at 12:01 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote: Hi Hackers,Attached is the patch to fix RM #3461 "Properties/SQL panel contents does not updates when user perform Refresh operation on the node". Please review it.--Akshay JoshiSr. Software ArchitectPhone: +91 20-3058-9517
Mobile: +91 976-788-8246--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company