On Wed, Jul 1, 2020 at 5:23 PM Nikhil Mohite <nikhil.moh...@enterprisedb.com> wrote:
> 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. > > This is a common file between pgAdmin & PEM. Please send a patch to the pgadmin team. > 3. FTS Templates files: > > - Refactor this function to reduce its Cognitive Complexity from 22 to > the 15 allowed. > > This is a common file between pgAdmin & PEM. Please send a patch to the pgadmin team. > 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. > > This is a common file between pgAdmin & PEM. Please send a patch to the pgadmin team. > 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. > > This is a common file between pgAdmin & PEM. Please send a patch to the pgadmin team. > 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. > > This is a common file between pgAdmin & PEM. Please send a patch to the pgadmin team. -- Thanks, Ashesh > > Regards, > Nikhil Mohite. >
