bb-drummer / siteapp

js/sys/siteapp.debug.js

/**
 * [Siteapp] - multi-purpose application & CMS theme
 * 
 * Siteapp debugger
 *     
 * @package     [Siteapp]
 * @subpackage  [Siteapp] core
 * @author      Björn Bartels <coding@bjoernbartels.earth>
 * @link        https://gitlab.bjoernbartels.earth/groups/themes
 * @license     http://www.apache.org/licenses/LICENSE-2.0 Apache License, Version 2.0
 * @copyright   copyright (c) 2016 Björn Bartels <coding@bjoernbartels.earth>
 * 
 * @namespace   Siteapp
 * @module      Siteapp.Debugger
 */
/** global: Siteapp */

var Debugger = {
	/* manage debug data */
	debugentries : [],
	
	types : ['log', 'debug', 'info', 'warn', 'error'],
	
	add : function ( oEntry ) {
		var entry = {
			type : oEntry.type || 'info',
			message : oEntry.message || '',
			time : oEntry.time || (new Date())
		};
		this.debugentries.push(entry);
		return (this);
	},

	clear : function ( ) {
		this.debugentries = [];
		return (this);
	},

	toEntry : function ( ) {
		var entry = arguments;
		if (entry.length == 1) {
			entry[1] = entry[0];
			entry[0] = 'info';
		}
		return {
			type : entry[0] || 'info',
			message : entry[1] || '',
			time : (new Date()),
		};
	},
	
	debug : function ( ) { 
		if ( (console || (typeof console.debug == 'function')) && $LCARS.getInstance().config.debug ) {

The variable $LCARS seems to be never declared. If this is a global, consider adding a /** global: $LCARS */ comment.

			console.debug(arguments);
		}
		return (this);
	},

	log : function ( msg ) { var entry = this.toEntry('log', msg); this.add(entry).console(entry); return (this); },
	
	info : function ( msg ) { var entry = this.toEntry('info', msg); this.add(entry).console(entry); return (this); },

	warn : function ( msg ) { var entry = this.toEntry('warn', msg); this.add(entry).console(entry); return (this); },
	
	error : function ( msg ) { var entry = this.toEntry('error', msg); this.add(entry).console(entry); return (this); },
	
	console : function ( data ) {
		if (!console || !data || !data.type || !data.message || !data.time || !$LCARS.getInstance().config.debug) {

The variable $LCARS seems to be never declared. If this is a global, consider adding a /** global: $LCARS */ comment.

			return (this);
		}
		var txt = [
			String(data.type).toUpperCase(), ': ',
			String(data.message), ' - ',
			String(data.time), ''
		].join('');
		if ( (typeof console[data.type] == 'function') ) {
			console[data.type](txt);
		} else {
			//console.//log(txt);
		}
		return (this);
	},
	
	get : function ( ) {
		if (arguments.length === 0) {
			return ( this.debugentries );
		} else if (arguments.length == 1) {
			var entries = [];
			for ( var iEntry in this.debugentries )

A for in loop automatically includes the property of any prototype object, consider checking the key using hasOwnProperty.

				if ( this.debugentries[iEntry].type == arguments[0] ) 

Curly braces around statements make for more readable code and help prevent bugs when you add further statements.

					entries.push( this.debugentries[iEntry] );

Curly braces around statements make for more readable code and help prevent bugs when you add further statements.

			return ( entries );
		}
		return (false);
	},
	
	getInstance : function ( ) {
		return (this);
	},
	
	_app : ( context ) => {
		if (context) {
			this.__app = context;

There is no return statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding return undefined; explicitly.

		} else {
			return this.__app;
		}
    }
	
};

export {Debugger};