Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM1595 - "no results to fetch error" displayed if modify definition with other table

2016-09-16 Thread Dave Page
Thanks, applied.

On Fri, Sep 16, 2016 at 11:14 AM, Surinder Kumar
 wrote:
> Hi
>
> Issues fixed:
>
> 1) Creating a m-view with definition and then updating view with new
> definition, it results into an error "No results to fetch".
>
> 2) When definition of m-view is updated, it creates a new m-view with
> different OID, but it wasn't updating the old OID with new OID of tree node
> in tree view. thus on switching to another tab (e.g.: sql), it throws
> message "Cannot find materialized view".
>
> Please find attached patch and review.
>
> Thanks,
> Surinder Kumar
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>



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

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


-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers


[pgadmin-hackers] [pgAdmin4][Patch]: RM1595 - "no results to fetch error" displayed if modify definition with other table

2016-09-16 Thread Surinder Kumar
Hi

*Issues fixed:*

1) Creating a m-view with *definition* and then updating view with* new
definition,* it results into an error *"No results to fetch"*.

2) When definition of m-view is updated, it creates a new m-view with
different OID, but it wasn't updating the old OID with new OID of tree node
in tree view. thus on switching to another tab (e.g.: sql), it throws
message *"Cannot find materialized view"*.

Please find attached patch and review.

Thanks,
Surinder Kumar


RM1595.patch
Description: Binary data

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers