From: | Joao De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io> |
---|---|
To: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][RM#3271] Update to the latest 3.x version of jQuery |
Date: | 2018-05-23 14:11:58 |
Message-ID: | CAE+jja=WjvZtHKmPwWTpujj451SOc_5AVtstdaesB4j8msFZQQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
HI Aditya,
Good job porting the app from the ancient version of JQuery to the latest
build :D
The patch in general looks good, and the patches-bot says all tests pass.
The only thing that we would like to point out is this piece of code:
/* jQuery has removed selector property in version 3.x
* To allow JS test cases to run, the below function is
* used to set the selector property
*/
function setPropAndSelector(selector, propName, propVal) {
let tmpObj = $(selector);
tmpObj['selector'] = selector;
tmpObj.prop(propName,propVal);
return tmpObj;
}
As tests are also part of the code we believe that when there is a change
that affect testing we should change the tests to work accordingly. So our
suggestion is, instead of creating this function, why don’t we use a CSS
selector in the tests to do the clicking or even use the HTML5 tag
data-test-selector?
This way the tests will also be coherent with the upgrade from JQuery 1 to 3
Thanks
Victoria & Joao
On Wed, May 23, 2018 at 6:44 AM Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
> Hi Hackers,
>
> Please find the updated patch for upgrading the jQuery version to 3.x. The
> patch is only for pgAdmin4 code changes replacing old deprecated functions
> with new replacement.
> We need to look into external libraries used for any vulnerabilities.
> Request you to kindly review.
>
> Thanks and Regards,
> Aditya Toshniwal
> Software Engineer | EnterpriseDB Software Solutions | Pune
> "Don't Complain about Heat, Plant a tree"
>
> On Mon, May 14, 2018 at 1:23 PM, Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Please hold on with the patch. Will send the updated patch soon.
>> Apologies.
>>
>>
>>
>> Thanks and Regards,
>> Aditya Toshniwal
>> Software Engineer | EnterpriseDB Software Solutions | Pune
>> "Don't Complain about Heat, Plant a tree"
>>
>> On Mon, May 14, 2018 at 12:52 PM, Aditya Toshniwal <
>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Hackers,
>>>
>>> PFA patch for upgrading jQuery version to 3.3.1 from current version
>>> 1.12.4. Patch includes replacing deprecated functions of jquery to latest
>>> one.
>>> Kindly review.
>>>
>>> Thanks and Regards,
>>> Aditya Toshniwal
>>> Software Engineer | EnterpriseDB Software Solutions | Pune
>>> "Don't Complain about Heat, Plant a tree"
>>>
>>
>>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Joao De Almeida Pereira | 2018-05-23 14:14:40 | Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree |
Previous Message | Joao De Almeida Pereira | 2018-05-23 13:36:49 | Re: Implement geospatial data viewer in pgAdmin4 for PostGIS |