Обсуждение: backform.pgadmin: fix for showing multiple errors for radio control [pgadmin4]

Поиск
Список
Период
Сортировка

backform.pgadmin: fix for showing multiple errors for radio control [pgadmin4]

От
Harshal Dhumal
Дата:
Hi,

PFA patch for backform.pgadmin.js

Issue fixed: Backform was showing same error multiple times (equal to no. of radio buttons in radio group) in dialog.

See attached screen shot.
-- 
Harshal Dhumal
Software Engineer 



Вложения

Re: backform.pgadmin: fix for showing multiple errors for radio control [pgadmin4]

От
Ashesh Vashi
Дата:


On Thu, Feb 11, 2016 at 6:44 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi,

PFA patch for backform.pgadmin.js

Issue fixed: Backform was showing same error multiple times (equal to no. of radio buttons in radio group) in dialog.
Approach for this patch is wrong, we should not change the common functionality, and check radio button in it.

That's wrong way to do it.

Though - I've checked-in the fix for it by extending the RadioControl itself.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company



See attached screen shot.
-- 
Harshal Dhumal
Software Engineer 





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