Re: [pgAdmin]: Patch for SonarQube fixes.

From: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
To: Nikhil Mohite <nikhil(dot)mohite(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin]: Patch for SonarQube fixes.
Date: 2021-03-01 12:30:46
Message-ID: CANxoLDe282EoYNXRh1eg=nF=MHyYtcB=txwCK15Ljz16HUgRmQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Thanks, patch applied.

On Mon, Mar 1, 2021 at 4:42 PM Nikhil Mohite <nikhil(dot)mohite(at)enterprisedb(dot)com>
wrote:

> 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*
>>
>

--
*Thanks & Regards*
*Akshay Joshi*
*pgAdmin Hacker | Principal Software Architect*
*EDB Postgres <http://edbpostgres.com>*

*Mobile: +91 976-788-8246*

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2021-03-01 12:30:59 Re: Feature 6270: Allow for --load-servers to replace current server list
Previous Message Akshay Joshi 2021-03-01 12:30:14 pgAdmin 4 commit: Fixed SonarQube issues.