pgAdmin 4 commit: Fixed following issues reported by SonarQube: - Mailing list pgadmin-hackers

From Akshay Joshi
Subject pgAdmin 4 commit: Fixed following issues reported by SonarQube:
Date
Msg-id E1jl7xi-0004Tb-Bt@gothos.postgresql.org
Whole thread Raw
List pgadmin-hackers
Fixed following issues reported by SonarQube:
1) Replace this if-then-else statement by a single return statement. (clumsy)
2) 'switch' statements should have at least 3 'case' clauses. (bad practice)

Branch
------
master

Details
-------
https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=d43518cb3c1a393af544f2a81633c20b06e1e00c
Author: Satish V <satish.v@enterprisedb.com>

Modified Files
--------------
.../schemas/functions/static/js/function.js        |  5 ++--
.../schemas/functions/static/js/procedure.js       |  7 +----
.../functions/static/js/trigger_function.js        |  5 ++--
.../schemas/tables/columns/static/js/column.js     | 32 ++++++----------------
.../static/js/compound_trigger.js                  |  6 +---
.../check_constraint/static/js/check_constraint.js |  6 +---
.../static/js/exclusion_constraint.js              |  3 +-
.../index_constraint/static/js/primary_key.js      |  6 +---
8 files changed, 17 insertions(+), 53 deletions(-)


pgadmin-hackers by date:

Previous
From: Satish V
Date:
Subject: Reg-[Sonarqube]CodeSmell issues
Next
From: Akshay Joshi
Date:
Subject: Re: Reg-[Sonarqube]CodeSmell issues