Re: [pgAdmin4][RM3575] Right click does not work on database node ifserver name changed - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [pgAdmin4][RM3575] Right click does not work on database node ifserver name changed
Date
Msg-id CA+OCxoywzbQ9NPAhgDUfSkDvSnfxogkg8dadU+JeRFK5GnCg=Q@mail.gmail.com
Whole thread Raw
In response to [pgAdmin4][RM3575] Right click does not work on database node ifserver name changed  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
List pgadmin-hackers
Thanks - patch applied.

On Wed, Jan 9, 2019 at 4:16 PM Aditya Toshniwal
<aditya.toshniwal@enterprisedb.com> wrote:
>
> Hi Hackers,
>
> Attached is the patch to fix the issue where changing the name (or any visible property on tree) of the server node
breaksthe context menu on the database nodes. 
>
> The user information of the server is updated on successful connection. Here, the data is updated on the
aciTreeNode(domNode).But, in the context menu the user data is referred from TreeNode object (a wrapper to
aciTreeNode).After checking the code (and a lot of debugging) it was found that aciTreeNode and TreeNode point to two
differentdata(I don't know why). Every time you change one, it should be synced to other, which is not done everywhere. 
> Changes are done to point both aciTreeNode and TreeNode data to the same data object.
>
> Kindly review.
> --
> Thanks and Regards,
> Aditya Toshniwal
> Software Engineer | EnterpriseDB Software Solutions | Pune
> "Don't Complain about Heat, Plant a tree"



--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Ensure the context menu works after a server isrenam
Next
From: pgAdmin 4 Jenkins
Date:
Subject: Build failed in Jenkins: pgadmin4-master-python35 #848