Thread: [pgAdmin4][Patch]: Fixed RM #2779 - Lost field size

[pgAdmin4][Patch]: Fixed RM #2779 - Lost field size

From
Khushboo Vashi
Date:
Hi,

Please find the attached patch to fix RM #2779: Lost field size.

While editing the table column, if we change collate then, the length and the precision are lost.
Also, while changing column type, it does not honour the length and precision.

Thanks,
Khushboo
Attachment

Re: [pgAdmin4][Patch]: Fixed RM #2779 - Lost field size

From
Dave Page
Date:
Hi

On Wed, Dec 6, 2017 at 5:45 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached patch to fix RM #2779: Lost field size.

While editing the table column, if we change collate then, the length and the precision are lost.
Also, while changing column type, it does not honour the length and precision.

Now it loses the collation if you edit the length. 

:-(

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

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

Re: [pgAdmin4][Patch]: Fixed RM #2779 - Lost field size

From
Khushboo Vashi
Date:


On Mon, Dec 11, 2017 at 5:40 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Wed, Dec 6, 2017 at 5:45 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached patch to fix RM #2779: Lost field size.

While editing the table column, if we change collate then, the length and the precision are lost.
Also, while changing column type, it does not honour the length and precision.

Now it loses the collation if you edit the length. 

I have tested it, it's not loosing the collation.
Please let me know the steps to reproduce this.
:-(

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

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

Re: [pgAdmin4][Patch]: Fixed RM #2779 - Lost field size

From
Khushboo Vashi
Date:


On Mon, Dec 11, 2017 at 5:45 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:


On Mon, Dec 11, 2017 at 5:40 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Wed, Dec 6, 2017 at 5:45 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached patch to fix RM #2779: Lost field size.

While editing the table column, if we change collate then, the length and the precision are lost.
Also, while changing column type, it does not honour the length and precision.

Now it loses the collation if you edit the length. 

I have tested it, it's not loosing the collation.
Please let me know the steps to reproduce this.
Reproduced. Will send the patch. 
:-(
 
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

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


Re: [pgAdmin4][Patch]: Fixed RM #2779 - Lost field size

From
Khushboo Vashi
Date:

On Mon, Dec 11, 2017 at 6:03 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:


On Mon, Dec 11, 2017 at 5:45 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:


On Mon, Dec 11, 2017 at 5:40 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Wed, Dec 6, 2017 at 5:45 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached patch to fix RM #2779: Lost field size.

While editing the table column, if we change collate then, the length and the precision are lost.
Also, while changing column type, it does not honour the length and precision.

Now it loses the collation if you edit the length. 

Fixed. 
Please find the attached patch. 
I have tested it, it's not loosing the collation.
Please let me know the steps to reproduce this.
Reproduced. Will send the patch. 
:-(
 
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

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



Attachment

Re: [pgAdmin4][Patch]: Fixed RM #2779 - Lost field size

From
Dave Page
Date:
Thanks, patch applied.

On Mon, Dec 11, 2017 at 12:46 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:

On Mon, Dec 11, 2017 at 6:03 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:


On Mon, Dec 11, 2017 at 5:45 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:


On Mon, Dec 11, 2017 at 5:40 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Wed, Dec 6, 2017 at 5:45 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached patch to fix RM #2779: Lost field size.

While editing the table column, if we change collate then, the length and the precision are lost.
Also, while changing column type, it does not honour the length and precision.

Now it loses the collation if you edit the length. 

Fixed. 
Please find the attached patch. 
I have tested it, it's not loosing the collation.
Please let me know the steps to reproduce this.
Reproduced. Will send the patch. 
:-(
 
--
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