Re: SonarQube Fixes #3 + RM5985[pgAdmin4] Bgprocess watcher dialog box disappers and error occurs after server is removed

From: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
To: Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: SonarQube Fixes #3 + RM5985[pgAdmin4] Bgprocess watcher dialog box disappers and error occurs after server is removed
Date: 2020-11-12 12:39:34
Message-ID: CANxoLDfKjVjsrJZuGGimPBaT2HeYudZZBac3dY-O5k7Agn9Giw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Thanks, patch applied.

On Thu, Nov 12, 2020 at 2:15 AM Rahul Shirsat <
rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:

> Hi Hackers,
>
> Please find below attached patch which fixes:
> 1. Bgprocess watcher dialog box disappearing after server is removed
> 2. Removed 3 dots on Storage Manager Dialog box (Storage Manager... to
> Storage Manager)
> 3. Previous sonarqube issues carry forward in this patch.
>
> Rest I haven't got any user_old specific issues while removing the orphan
> process, I think, programmatically, it is handled.
>
> Note: Previous sonarqube issue in this mail trace is also considered in
> this patch due to many similar file changes.
>
> On Tue, Nov 10, 2020 at 1:59 PM Rahul Shirsat <
> rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Minor sonar issues too fixed. Find the updated patch.
>>
>> On Mon, Nov 9, 2020 at 2:53 PM Rahul Shirsat <
>> rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Hackers,
>>>
>>> Please find the attached patch to fix the SonarQube code smells having
>>> rules:
>>>
>>> - Refactor this function to reduce its Cognitive Complexity from 24
>>> to the 15 allowed.
>>>
>>> Also, corrected one of the backup test cases which were failing in
>>> server mode.
>>>
>>> --
>>> *Rahul Shirsat*
>>> Software Engineer | EnterpriseDB Corporation.
>>>
>>
>>
>> --
>> *Rahul Shirsat*
>> Software Engineer | EnterpriseDB Corporation.
>>
>
>
> --
> *Rahul Shirsat*
> Software Engineer | EnterpriseDB Corporation.
>

--
*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 2020-11-12 12:39:49 Re: Patch for SonarQube code scan fixes
Previous Message Akshay Joshi 2020-11-12 12:39:24 Re: [pgAdmin][RM-5974]: Debugger tab title (placeholders) does not apply to debugger opened in the new browser tab.