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

From Nikhil Mohite
Subject Patch for SonarQube code scan fixes
Date
Msg-id CAOBg0AP50o3PH7LQiH8u1LGR8QmkA+FM9REL6m5D42CGxtRcog@mail.gmail.com
Whole thread Raw
Responses Re: Patch for SonarQube code scan fixes  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
List pgadmin-hackers
Hi Team,

I have fixed some code smell issues in the SonarQube scan, PFA patch.
Details as follows:
1. Functions files:
  • Remove the unused local variable "args"
2. node.js file in browser/static:
  • "j" hides or potentially hides a variable declared in an outer scope at line 1080.
3. FTS Templates files:
  • Refactor this function to reduce its Cognitive Complexity from 22 to the 15 allowed.
4. FTS Parsers files:
  • Refactor this function to reduce its Cognitive Complexity from 22 to the 15 allowed.
  • Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed.
5. FTS Dictionaries files:
  • Refactor this function to reduce its Cognitive Complexity from 25 to the 15 allowed.
  • Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed.
6. FTS Configurations files:
  • Refactor this function to reduce its Cognitive Complexity from 22 to the 15 allowed.
  • Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed.

Regards,
Nikhil Mohite.
Attachment

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: Fix PEP8
Next
From: Ashesh Vashi
Date:
Subject: Re: Patch for SonarQube code scan fixes