Postgres Pro
Downloads
Home   >   mailing lists

Re: Patch for SonarQube code scan fixes - Mailing list pgadmin-hackers

From Akshay Joshi
Subject Re: Patch for SonarQube code scan fixes
Date
Msg-id CANxoLDcg4Czh51o6-g25k=6eR+hMcKwwG5WNT15MK-C2j1+sYg@mail.gmail.com
Whole thread Raw
In response to Patch for SonarQube code scan fixes  (Nikhil Mohite <nikhil.mohite@enterprisedb.com>)
List pgadmin-hackers
Tree view
Thanks, patch applied.

On Tue, Nov 10, 2020 at 3:44 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:
Hi Team,

I have fixed some code smell issues in the SonarQube scan, PFA patch.
Details as follows:

1. Schema dif dependency:
  • 'isChecked' is already declared in the upper scope.
2. User management:
  • Refactor this function to reduce its Cognitive Complexity from 27 to the 15 allowed.
3. Debugger:
  • Refactor this function to reduce its Cognitive Complexity from 30 to the 15 allowed.
  • Refactor this function to reduce its Cognitive Complexity from 45 to the 15 allowed.
  • Refactor this function to reduce its Cognitive Complexity from 50 to the 15 allowed.





--
Thanks & Regards,
Nikhil Mohite
Software Engineer.
Mob.No: +91-7798364578.


--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Principal Software Architect
EDB Postgres
Mobile: +91 976-788-8246

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: SonarQube Fixes #3 + RM5985[pgAdmin4] Bgprocess watcher dialog box disappers and error occurs after server is removed
Next
From: Akshay Joshi
Date:
Subject: Re: [pgAdmin][RM-5983]:Add server type icon on new connection dialog.