Postgres Pro
Downloads
Home   >   mailing lists

Thread: PgAdmin 1.18 possible bug in DROP USER MAPPING

PgAdmin 1.18 possible bug in DROP USER MAPPING

From
Bartosz Dmytrak
Date:
Hello all,
First of all - thanks a lot for new version!

I've noticed wrong quotation in drop user mapping statement generated by pgAdmin:

It is:
DROP USER MAPPING FOR public SERVER Repository@localhost;

It should be:
DROP USER MAPPING FOR public SERVER "Repository@localhost";


Same for ALTER USER MAPPING statement.



Env:
PgAdmin 1.18.0 on Fedora 19 x64

Regards,
Bartek

Re: PgAdmin 1.18 possible bug in DROP USER MAPPING

From
Dave Page
Date:
Dinesh, can you look at this please?

Thanks.

On Mon, Sep 9, 2013 at 10:13 PM, Bartosz Dmytrak <bdmytrak@gmail.com> wrote:
> Hello all,
> First of all - thanks a lot for new version!
>
> I've noticed wrong quotation in drop user mapping statement generated by
> pgAdmin:
>
> It is:
> DROP USER MAPPING FOR public SERVER Repository@localhost;
>
> It should be:
> DROP USER MAPPING FOR public SERVER "Repository@localhost";
>
>
> Same for ALTER USER MAPPING statement.
>
>
>
> Env:
> PgAdmin 1.18.0 on Fedora 19 x64
>
> Regards,
> Bartek



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

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

Re: PgAdmin 1.18 possible bug in DROP USER MAPPING

From
Dinesh Kumar
Date:
Sure Dave.


Dinesh

-- 
Dinesh Kumar
Software Engineer

Ph: +918087463317
Skype ID: dinesh.kumar432
www.enterprisedb.com

Follow us on Twitter
@EnterpriseDB 

Visit EnterpriseDB for tutorials, webinars, whitepapers and more


On Tue, Sep 10, 2013 at 2:22 PM, Dave Page <dpage@pgadmin.org> wrote:
Dinesh, can you look at this please?

Thanks.

On Mon, Sep 9, 2013 at 10:13 PM, Bartosz Dmytrak <bdmytrak@gmail.com> wrote:
> Hello all,
> First of all - thanks a lot for new version!
>
> I've noticed wrong quotation in drop user mapping statement generated by
> pgAdmin:
>
> It is:
> DROP USER MAPPING FOR public SERVER Repository@localhost;
>
> It should be:
> DROP USER MAPPING FOR public SERVER "Repository@localhost";
>
>
> Same for ALTER USER MAPPING statement.
>
>
>
> Env:
> PgAdmin 1.18.0 on Fedora 19 x64
>
> Regards,
> Bartek



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

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

Re: PgAdmin 1.18 possible bug in DROP USER MAPPING

From
Dinesh Kumar
Date:
Hi Dave,

Please find the patch for the reported case.



Dinesh

-- 
Dinesh Kumar
Software Engineer

Ph: +918087463317
Skype ID: dinesh.kumar432
www.enterprisedb.com

Follow us on Twitter
@EnterpriseDB 

Visit EnterpriseDB for tutorials, webinars, whitepapers and more


On Tue, Sep 10, 2013 at 2:27 PM, Dinesh Kumar <dinesh.kumar@enterprisedb.com> wrote:
Sure Dave.


Dinesh

-- 
Dinesh Kumar
Software Engineer
Skype ID: dinesh.kumar432
www.enterprisedb.com

Follow us on Twitter
@EnterpriseDB 

Visit EnterpriseDB for tutorials, webinars, whitepapers and more


On Tue, Sep 10, 2013 at 2:22 PM, Dave Page <dpage@pgadmin.org> wrote:
Dinesh, can you look at this please?

Thanks.

On Mon, Sep 9, 2013 at 10:13 PM, Bartosz Dmytrak <bdmytrak@gmail.com> wrote:
> Hello all,
> First of all - thanks a lot for new version!
>
> I've noticed wrong quotation in drop user mapping statement generated by
> pgAdmin:
>
> It is:
> DROP USER MAPPING FOR public SERVER Repository@localhost;
>
> It should be:
> DROP USER MAPPING FOR public SERVER "Repository@localhost";
>
>
> Same for ALTER USER MAPPING statement.
>
>
>
> Env:
> PgAdmin 1.18.0 on Fedora 19 x64
>
> Regards,
> Bartek



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

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


Attachment

Re: PgAdmin 1.18 possible bug in DROP USER MAPPING

From
Dave Page
Date:
Thanks, committed.


On Wed, Sep 18, 2013 at 2:23 PM, Dinesh Kumar <dinesh.kumar@enterprisedb.com> wrote:
Hi Dave,

Please find the patch for the reported case.



Dinesh

-- 
Dinesh Kumar
Software Engineer
Skype ID: dinesh.kumar432
www.enterprisedb.com

Follow us on Twitter
@EnterpriseDB 

Visit EnterpriseDB for tutorials, webinars, whitepapers and more


On Tue, Sep 10, 2013 at 2:27 PM, Dinesh Kumar <dinesh.kumar@enterprisedb.com> wrote:
Sure Dave.


Dinesh

-- 
Dinesh Kumar
Software Engineer
Skype ID: dinesh.kumar432
www.enterprisedb.com

Follow us on Twitter
@EnterpriseDB 

Visit EnterpriseDB for tutorials, webinars, whitepapers and more


On Tue, Sep 10, 2013 at 2:22 PM, Dave Page <dpage@pgadmin.org> wrote:
Dinesh, can you look at this please?

Thanks.

On Mon, Sep 9, 2013 at 10:13 PM, Bartosz Dmytrak <bdmytrak@gmail.com> wrote:
> Hello all,
> First of all - thanks a lot for new version!
>
> I've noticed wrong quotation in drop user mapping statement generated by
> pgAdmin:
>
> It is:
> DROP USER MAPPING FOR public SERVER Repository@localhost;
>
> It should be:
> DROP USER MAPPING FOR public SERVER "Repository@localhost";
>
>
> Same for ALTER USER MAPPING statement.
>
>
>
> Env:
> PgAdmin 1.18.0 on Fedora 19 x64
>
> Regards,
> Bartek



--
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