Showing 45 of 53 total issues
Function use
has 36 lines of code (exceeds 25 allowed). Consider refactoring. Open
Poplar.prototype.use = function(name, options) {
var apiBuilder = name;
if (!(apiBuilder instanceof ApiBuilder)) {
apiBuilder = new ApiBuilder(name, options);
Function errorHandler
has 36 lines of code (exceeds 25 allowed). Consider refactoring. Open
RestAdapter.errorHandler = function(options) {
options = options || {};
return function restErrorHandler(err, req, res, next) {
if (typeof options.errorHandler === 'function') {
try {
Function execHooks
has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring. Open
Poplar.prototype.execHooks = function(when, method, ctx, next) {
var methodName = method.fullName();
// change state
var currentState = util.format('%s.%s', when, methodName);
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function restErrorHandler
has 33 lines of code (exceeds 25 allowed). Consider refactoring. Open
return function restErrorHandler(err, req, res, next) {
if (typeof options.errorHandler === 'function') {
try {
options.errorHandler(err, req, res, next);
} catch (e) {
Function coerceAll
has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring. Open
function coerceAll(obj) {
var type = Array.isArray(obj) ? 'array' : typeof obj;
var i;
var n;
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function _eval
has 31 lines of code (exceeds 25 allowed). Consider refactoring. Open
function _eval(source, path, value) {
if (path && _.isString(path)) {
// split path by '[', '.', ']' and then remove empty string
var list = _.compact(path.trim().split(/\[|\.|\]/));
Function execHooks
has 29 lines of code (exceeds 25 allowed). Consider refactoring. Open
Poplar.prototype.execHooks = function(when, method, ctx, next) {
var methodName = method.fullName();
// change state
var currentState = util.format('%s.%s', when, methodName);
Function parse
has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring. Open
Entity.prototype.parse = function(input, options, converter) {
debug('parsing %j with options %j and converter', input, options);
var originalObj;
var result = {};
var self = this;
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function ApiMethod
has 27 lines of code (exceeds 25 allowed). Consider refactoring. Open
function ApiMethod(name, options, fn) {
this.__original = [name, options, fn];
this.fn = fn;
Function performValidator
has 27 lines of code (exceeds 25 allowed). Consider refactoring. Open
var performValidator = function(name, val, validatorOpts, validatorName) {
validatorOpts = validatorOpts || {};
// if validator is a custom function, then execute it
// else find cooresponding validator in built in Validator
Function getType
has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring. Open
ApiMethod.getType = function(val) {
var type = typeof val;
switch (type) {
case 'undefined':
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function createHandler
has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring. Open
RestAdapter.prototype.createHandler = function() {
var router = express.Router(this.options);
var adapter = this;
var methods = this.api.allMethods();
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function convert
has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring. Open
Dynamic.convert = function(val, toType, ctx) {
if (Array.isArray(toType)) {
if (!Array.isArray(val)) {
if (val === undefined || val === '') {
val = [];
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function coerceAccepts
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
function coerceAccepts(uarg, desc) {
var name = desc.name || desc.arg;
var targetType = convertToBasicType(desc.type);
var targetTypeIsArray = _.isArray(targetType) && targetType.length === 1;
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function add
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
Entity.prototype.add = function() {
// ...names, options, fn
var message = '\'%s\' is not a valid string';
var self = this;
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function ApiMethod
has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring. Open
function ApiMethod(name, options, fn) {
this.__original = [name, options, fn];
this.fn = fn;
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Avoid too many return
statements within this function. Open
if (isint.test(str) && str.charAt(0) !== '0') return parseInt(str, 10);
Avoid too many return
statements within this function. Open
return callback(err);
Avoid too many return
statements within this function. Open
return cb(e);
Avoid too many return
statements within this function. Open
return str;