From: | Nikhil Mohite <nikhil(dot)mohite(at)enterprisedb(dot)com> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin]: Patch for SonarQube fixes. |
Date: | 2021-03-01 11:12:39 |
Message-ID: | CAOBg0APj74-4QGoq5+xs+d2Qq3ynVPQMQnnfPdXzXtSCTOJvYQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Akshay,
Sorry for the inconvenience, Please find the attached patch.
Regards,
Nikhil Mohite.
On Mon, Mar 1, 2021 at 4:37 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi Nikhil
>
> You forgot to attach the patch :)
>
> On Mon, Mar 1, 2021 at 4:12 PM Nikhil Mohite <
> nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:
>
>> Hi Team,
>>
>> I have fixed a few sonarQube issues, PFA patch
>> Details as follows:
>>
>> 1. Back-form (backform.pgadmin.js):
>>
>> - TypeError can be thrown as "$tabContent" might be null or undefined
>> here.
>>
>> 2. User Management (user_management.js):
>>
>> - 'self' is already declared in the upper scope.
>> - 'res' is already declared in the upper scope.
>> - 'self' is already declared in the upper scope.
>> - 'ownershipModel' is already declared in the upper scope.
>>
>>
>> --
>> *Thanks & Regards,*
>> *Nikhil Mohite*
>> *Software Engineer.*
>> *EDB Postgres* <https://www.enterprisedb.com/>
>> *Mob.No: +91-7798364578.*
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
> *pgAdmin Hacker | Principal Software Architect*
> *EDB Postgres <http://edbpostgres.com>*
>
> *Mobile: +91 976-788-8246*
>
Attachment | Content-Type | Size |
---|---|---|
SonarQubeCodeSmellFixes.patch | application/octet-stream | 4.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Alessandro De Maria | 2021-03-01 11:12:40 | Re: Feature 6270: Allow for --load-servers to replace current server list |
Previous Message | Akshay Joshi | 2021-03-01 11:07:28 | Re: [pgAdmin]: Patch for SonarQube fixes. |