SKarajic / LoLNode

es/utils.js

import { RegionDomains } from './regions';

This code seems to be duplicated in your project.

import * as https from 'https';
export function getJSON(url, callback) {
    https.get(url, function (res) {
        var statusCode = res.statusCode;
        if (statusCode !== 200) {
            res.resume();
            callback(null, 'error');
        }
        res.setEncoding('utf8');
        var rawData = '';
        res.on('data', function (chunk) { rawData += chunk; });
        res.on('end', function () {
            try {
                var parsedData = JSON.parse(rawData);
                callback({ headers: res.headers, statusCode: res.statusCode, data: parsedData });
            }
            catch (e) {
                callback(null, e);
            }
        });
    }).on('error', function (e) {
        callback(null, e);
    });
}
export function getRegion(region) {
    var val;
    switch (region.toUpperCase()) {

As per coding-style, switch statements should have a default case.

        case 'EUNE':
            val = RegionDomains.EUNE;
            break;
        case 'EUW':
            val = RegionDomains.EUW;
            break;
        case 'OCE':
            val = RegionDomains.OCE;
            break;
        case 'LAN':
            val = RegionDomains.LAN;
            break;
        case 'LAS':
            val = RegionDomains.LAS;
            break;
        case 'RU':
            val = RegionDomains.RU;
            break;
        case 'BR':
            val = RegionDomains.BR;
            break;
        case 'TR':
            val = RegionDomains.TR;
            break;
        case 'NA':
            val = RegionDomains.NA;
            break;
        case 'KR':
            val = RegionDomains.KR;
            break;
        case 'JP':
            val = RegionDomains.JP;
            break;
    }
    return val;

The variable val seems to not be initialized for all possible execution paths.

}
//# sourceMappingURL=utils.js.map