Thanks, applied. On Friday, October 14, 2016, Surinder Kumar <surinder.ku...@enterprisedb.com> wrote:
> Hi > > *Following are the issues fixed in nodes:* > > 1) If we create/update a node with non-default schema, It should return > selected schema id in return response. but default schema id is returned > every time due to which it throws error in properties panel. > Fixed in Domains, Collation, Types, Views & Table node. > > 2) Incorrect parent id of object node is returned from *nodes method* due > to which wrong parent id is passed while updating object and > thus node didn't get refreshed. > Fixed in FTS Configuration, FTS Parser nodes. > > Also, I have kept changes of first patch which are essential to refresh > node every time. Without that patch nodes properties panel updates only > sometimes. > > Please find attached patch. Please review and let me know for comments. > > Thanks > Surinder Kumar > > > > On Fri, Sep 23, 2016 at 6:00 PM, Dave Page <dp...@pgadmin.org > <javascript:_e(%7B%7D,'cvml','dp...@pgadmin.org');>> wrote: > >> Umm, no it wasn't - sorry. >> >> I see the same issue with Types. Can you fix that, and check all other >> nodes as well please? >> >> Thanks. >> >> On Fri, Sep 23, 2016 at 1:29 PM, Dave Page <dp...@pgadmin.org >> <javascript:_e(%7B%7D,'cvml','dp...@pgadmin.org');>> wrote: >> > Thanks, applied. >> > >> > On Fri, Sep 23, 2016 at 12:05 PM, Surinder Kumar >> > <surinder.ku...@enterprisedb.com >> <javascript:_e(%7B%7D,'cvml','surinder.ku...@enterprisedb.com');>> wrote: >> >> Hi, >> >> >> >> Please find updated patch with changes: >> >> 1) On debugging through JS files, the issue was in synonym update >> method >> >> which wasn't returning node object. >> >> 2) retrieving schema name in node.sql for creating node object in >> update >> >> method. >> >> >> >> Please review and let me know for comments. >> >> >> >> On Fri, Sep 23, 2016 at 2:44 PM, Dave Page <dp...@pgadmin.org >> <javascript:_e(%7B%7D,'cvml','dp...@pgadmin.org');>> wrote: >> >>> >> >>> Hi >> >>> >> >>> On Fri, Sep 23, 2016 at 7:39 AM, Surinder Kumar >> >>> <surinder.ku...@enterprisedb.com >> <javascript:_e(%7B%7D,'cvml','surinder.ku...@enterprisedb.com');>> wrote: >> >>> > Hi >> >>> > >> >>> > Issue: >> >>> > on updating node, we deselect and then again select the node >> updated to >> >>> > refresh the panel. but it needs some delay of few milliseconds >> between >> >>> > deselect and select to fix this issue. >> >>> > >> >>> > Please find attached patch and review. >> >>> >> >>> This does not resolve the issue for me. I tested using a synonym to a >> >>> package on EPAS 9.5, by changing the target package name. >> >>> >> >>> >> >>> -- >> >>> Dave Page >> >>> Blog: http://pgsnake.blogspot.com >> >>> Twitter: @pgsnake >> >>> >> >>> EnterpriseDB UK: http://www.enterprisedb.com >> >>> The Enterprise PostgreSQL Company >> >> >> >> >> > >> > >> > >> > -- >> > Dave Page >> > Blog: http://pgsnake.blogspot.com >> > Twitter: @pgsnake >> > >> > EnterpriseDB UK: http://www.enterprisedb.com >> > The Enterprise PostgreSQL Company >> >> >> >> -- >> Dave Page >> Blog: http://pgsnake.blogspot.com >> Twitter: @pgsnake >> >> EnterpriseDB UK: http://www.enterprisedb.com >> The Enterprise PostgreSQL Company >> > > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
