From: | Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com> |
---|---|
To: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
Cc: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)lists(dot)postgresql(dot)org>, Dave Page <dave(dot)page(at)enterprisedb(dot)com> |
Subject: | Re: pgAdmin 4 commit: Fixed an issue where the validation error message is |
Date: | 2020-12-03 06:58:37 |
Message-ID: | CAKtn9dMD_U1fFFyNX2zUsfdQdbfJ_VzzBhTpOd_BngQebS2vOg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Thu, Dec 3, 2020 at 11:18 AM Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
> Hello,
>
> Another observation,
>
> + if(msg && handler) {
> + msg = msg || _.values(handler.errorModel.attributes)[0];
> +
> + handler.trigger('pgadmin-session:model:invalid', msg, handler);
> + $(parentTr).addClass('error-in-grid');
> + }
> + return this;
> + }, 120);
> + },
>
> Here we are only making the model invalid, there is no logic to make it
> valid again.
>
If there are multiple model errors in the list, and if user changes any of
them, it used to clear every other invalid models in the list. To avoid
this to happen, only invalid models are highlighted, rest others are taken
care of by the child valid events running in parallel.
>
>
>
> On Thu, Dec 3, 2020 at 11:09 AM Murtuza Zabuawala <
> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>
>> Hello,
>>
>> What is the reason for adding delay *setTimeout(<func>, 120)*?
>>
>> This is causing issues in automation testing and makes buttons
>> enable/disable with slight delay.
>>
>> Timeout was added for the reason that, even if the current changed
model is valid, it should also check the rest of the invalid models in the
list. Slight delay was added, so even if the current model becomes valid,
it should highlight the invalid one iterating over the inner model list.
> --
>> Regards,
>> Murtuza Zabuawala
>> *EDB*
>> *POWER TO POSTGRES*
>> https://www.edbpostgres.com
>>
>>
>> On Fri, Oct 30, 2020 at 3:10 PM Akshay Joshi <
>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>
>>> Fixed an issue where the validation error message is shown twice. Fixes
>>> #5736
>>>
>>> Branch
>>> ------
>>> master
>>>
>>> Details
>>> -------
>>>
>>> https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=183c83f0d2b2cb8dd33f9e8c41a030bc4cdaa4b2
>>> Author: Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com>
>>>
>>> Modified Files
>>> --------------
>>> docs/en_US/release_notes_4_28.rst | 1 +
>>> web/pgadmin/browser/static/js/datamodel.js | 75
>>> +++++++++++++++++++++++++++++-
>>> web/pgadmin/static/js/backform.pgadmin.js | 24 ++++------
>>> web/pgadmin/static/scss/_alert.scss | 6 +++
>>> 4 files changed, 90 insertions(+), 16 deletions(-)
>>>
>>>
--
*Rahul Shirsat*
Software Engineer | EnterpriseDB Corporation.
From | Date | Subject | |
---|---|---|---|
Next Message | Murtuza Zabuawala | 2020-12-03 07:17:46 | Re: pgAdmin 4 commit: Fixed an issue where the validation error message is |
Previous Message | Murtuza Zabuawala | 2020-12-03 05:47:52 | Re: pgAdmin 4 commit: Fixed an issue where the validation error message is |