Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-16 Thread Akshay Joshi
Thanks, patch applied. On Thu, Apr 16, 2020 at 1:53 PM Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > Hi, > > Patch looks good to me. > > Thanks, > Khushboo > > On Thu, Apr 16, 2020 at 12:07 PM navnath gadakh < > navnath.gad...@enterprisedb.com> wrote: > >> Hi Khushboo, >> >> Please

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-16 Thread Khushboo Vashi
Hi, Patch looks good to me. Thanks, Khushboo On Thu, Apr 16, 2020 at 12:07 PM navnath gadakh < navnath.gad...@enterprisedb.com> wrote: > Hi Khushboo, > > Please find the modified patch. I have removed the length from the data > types. Test cases also passing on all Postgres versions. > Thanks!

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-15 Thread navnath gadakh
Hi Khushboo, Please find the modified patch. I have removed the length from the data types. Test cases also passing on all Postgres versions. Thanks! On Wed, Apr 15, 2020 at 6:22 PM Ashesh Vashi wrote: > > > On Wed, 15 Apr 2020 at 18:21, Ashesh Vashi > wrote: > >> >> >> On Wed, 15 Apr 2020

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-15 Thread Ashesh Vashi
On Wed, 15 Apr 2020 at 18:21, Ashesh Vashi wrote: > > > On Wed, 15 Apr 2020 at 18:18, Murtuza Zabuawala < > murtuza.zabuaw...@enterprisedb.com> wrote: > >> Hello, >> >> We are sending the data to backend and depending on errors from backend. Any >> thoughts on implementation of basic fronted vali

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-15 Thread Ashesh Vashi
On Wed, 15 Apr 2020 at 18:18, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Hello, > > We are sending the data to backend and depending on errors from backend. Any > thoughts on implementation of basic fronted validations? so that we can > alert user before it clicks on save but

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-15 Thread Murtuza Zabuawala
Hello, We are sending the data to backend and depending on errors from backend. Any thoughts on implementation of basic fronted validations? so that we can alert user before it clicks on save button. On Wed, 15 Apr 2020, 18:08 Dave Page, wrote: > Removing the typecast will almost certainly lea

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-15 Thread Dave Page
Removing the typecast will almost certainly lead to other problems. I think we should just remove the length from it. On Wed, Apr 15, 2020 at 1:33 PM navnath gadakh < navnath.gad...@enterprisedb.com> wrote: > @Dave Page @Akshay Joshi > your input please? > > On Wed, Apr 15, 2020 at 3:13 PM Nee

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-15 Thread Akshay Joshi
Hi Navnath Scenario 3 seems correct to me. No need to type cast. On Wed, Apr 15, 2020 at 6:03 PM navnath gadakh < navnath.gad...@enterprisedb.com> wrote: > @Dave Page @Akshay Joshi > your input please? > > On Wed, Apr 15, 2020 at 3:13 PM Neel Patel > wrote: > >> Hi, >> >> I think we should r

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-15 Thread navnath gadakh
@Dave Page @Akshay Joshi your input please? On Wed, Apr 15, 2020 at 3:13 PM Neel Patel wrote: > Hi, > > I think we should remove the type cast from query during update and > whatever error is thrown should be shown to UI as per scenario 3. > > Thanks, > Neel Patel > > On Wed, Apr 15, 2020 at

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-15 Thread Neel Patel
Hi, I think we should remove the type cast from query during update and whatever error is thrown should be shown to UI as per scenario 3. Thanks, Neel Patel On Wed, Apr 15, 2020 at 3:06 PM Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > > > On Wed, Apr 15, 2020 at 2:48 PM navnath gad

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-15 Thread Aditya Toshniwal
Hi, I think this is not a bug. From what I can see, the query - "UPDATE public.account SET username = 'username-test-123'::character varying(5) WHERE user_id = 1;" will not throw any error from psql. The same query is fired by pgAdmin, hence no error. To get an error we need to remove the typecas

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-15 Thread Khushboo Vashi
On Wed, Apr 15, 2020 at 2:48 PM navnath gadakh < navnath.gad...@enterprisedb.com> wrote: > Hello Hackers, > > > On Tue, Apr 14, 2020 at 5:14 PM Khushboo Vashi < > khushboo.va...@enterprisedb.com> wrote: > >> Hi Navnath, >> >> You have compared the column's internal size with the length of the valu

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-15 Thread navnath gadakh
Hello Hackers, On Tue, Apr 14, 2020 at 5:14 PM Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > Hi Navnath, > > You have compared the column's internal size with the length of the value > given by the user. > For example, column having integer would have internal size 4 and if I > give

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-14 Thread Khushboo Vashi
Hi Navnath, You have compared the column's internal size with the length of the value given by the user. For example, column having integer would have internal size 4 and if I give the value 12121 which is the correct input for the field will fail here because as per your logic column internal siz

Re: [pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-14 Thread Khushboo Vashi
Hi, I am reviewing this patch. Thanks, Khushboo On Tue, Apr 14, 2020 at 4:33 PM navnath gadakh < navnath.gad...@enterprisedb.com> wrote: > Hello Hackers, > Please find the attached patch for below fixes: > > - Added validation for table row data that should not be larger than the > field size.

[pgAdmin][RM5210] pgAdmin4 silently truncates text larger than underlying field size

2020-04-14 Thread navnath gadakh
Hello Hackers, Please find the attached patch for below fixes: - Added validation for table row data that should not be larger than the field size. - Rearrange the existing functions to add validation. - Added test cases. Regards, Navnath Gadakh rm_5210_v1.patch Description: Binary data