Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

linting utils.js #461

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

linting utils.js #461

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions packages/metadata-core/src/objs.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,10 +74,10 @@ export class BaseDataObj {
const {_obj} = this;
const res = _obj ? _obj[f] : '';

if(f == 'type' && typeof res == 'object') {
if(f === 'type' && typeof res === 'object') {
return res;
}
else if(f == 'ref') {
else if(f === 'ref') {
return res;
}
else if(mf.is_ref) {
Expand Down Expand Up @@ -114,7 +114,7 @@ export class BaseDataObj {
else if(mf.digits) {
return utils.fix_number(_obj[f], !mf.hasOwnProperty('str_len'));
}
else if(mf.types[0] == 'boolean') {
else if(mf.types[0] === 'boolean') {
return utils.fix_boolean(_obj[f]);
}
else {
Expand Down Expand Up @@ -161,7 +161,7 @@ export class BaseDataObj {
if(mf.digits && typeof v === 'number' || mf.hasOwnProperty('str_len') && typeof v === 'string' && !utils.is_guid(v)) {
_obj[f] = v;
}
else if(typeof v === 'boolean' && mf.types.indexOf('boolean') != -1) {
else if(typeof v === 'boolean' && mf.types.indexOf('boolean') !== -1) {
_obj[f] = v;
}
else if(mf.date_part && v instanceof Date) {
Expand All @@ -170,7 +170,7 @@ export class BaseDataObj {
else {
_obj[f] = utils.fix_guid(v);

if(utils.is_data_obj(v) && mf.types.indexOf(v._manager.class_name) != -1) {
if(utils.is_data_obj(v) && mf.types.indexOf(v._manager.class_name) !== -1) {

}
else {
Expand Down Expand Up @@ -212,7 +212,7 @@ export class BaseDataObj {
else if(mf.digits) {
_obj[f] = utils.fix_number(v, !mf.hasOwnProperty('str_len'));
}
else if(mf.types[0] == 'boolean') {
else if(mf.types[0] === 'boolean') {
_obj[f] = utils.fix_boolean(v);
}
else {
Expand All @@ -236,7 +236,7 @@ export class BaseDataObj {
* @private
*/
_setter(f, v) {
if(this._obj[f] != v) {
if(this._obj[f] !== v) {
this.__notify(f);
this.__setter(f, v);
}
Expand Down
51 changes: 26 additions & 25 deletions packages/metadata-core/src/utils.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -310,6 +310,7 @@ const utils = {
* @method date_add_day
* @param date {Date} - исходная дата
* @param days {Number} - число дней, добавляемых к дате (может быть отрицательным)
* @param reset_time {boolean} - сбросить время в рамках дня на начало суток
* @return {Date}
*/
date_add_day(date, days, reset_time) {
Expand Down Expand Up @@ -339,7 +340,7 @@ const utils = {
*
* @method is_guid
* @param v {*} - проверяемое значение
* @return {Boolean} - true, если значение соответствует регурярному выражению guid
* @return {Boolean} - true, если значение соответствует регулярному выражению guid
*/
is_guid(v) {
if (typeof v !== 'string' || v.length < 36) {
Expand All @@ -364,7 +365,7 @@ const utils = {
},

/**
* ### Проверяет, является ли значенние Data-объектным типом
* ### Проверяет, является ли значение Data-объектным типом
*
* @method is_data_obj
* @param v {*} - проверяемое значение
Expand All @@ -375,7 +376,7 @@ const utils = {
},

/**
* ### Проверяет, является ли значенние Data-документом
* ### Проверяет, является ли значение Data-документом
*
* @method is_doc_obj
* @param v {*} - проверяемое значение
Expand All @@ -386,7 +387,7 @@ const utils = {
},

/**
* ### Проверяет, является ли значенние менеджером объектов данных
* ### Проверяет, является ли значение менеджером объектов данных
*
* @method is_data_mgr
* @param v {*} - проверяемое значение
Expand All @@ -397,7 +398,7 @@ const utils = {
},

/**
* ### Проверяет, является ли значенние менеджером перечисления
* ### Проверяет, является ли значение менеджером перечисления
*
* @method is_enm_mgr
* @param v {*} - проверяемое значение
Expand All @@ -408,7 +409,7 @@ const utils = {
},

/**
* ### Проверяет, является ли значенние табличной частью или строкой табличной части
* ### Проверяет, является ли значение табличной частью или строкой табличной части
*
* @method is_tabular
* @param v {*} - проверяемое значение
Expand All @@ -424,10 +425,10 @@ const utils = {
* @method is_equal
* @param v1 {DataObj|String}
* @param v2 {DataObj|String}
* @return {boolean} - true, если значенния эквивалентны
* @return {boolean} - true, если значения эквивалентны
*/
is_equal(v1, v2) {
if (v1 == v2) {
if (v1 === v2) {
return true;
}
else if (typeof v1 === 'string' && typeof v2 === 'string' && v1.trim() === v2.trim()) {
Expand All @@ -436,7 +437,7 @@ const utils = {
else if (typeof v1 === typeof v2) {
return false;
}
return (this.fix_guid(v1, false) == this.fix_guid(v2, false));
return (this.fix_guid(v1, false) === this.fix_guid(v2, false));
},

/**
Expand Down Expand Up @@ -486,14 +487,14 @@ const utils = {
*/
get_and_show_blob(url, post_data, method) {

function show_blob(req) {
url = window.URL.createObjectURL(req.response);
function show_blob(request) {
url = window.URL.createObjectURL(request.response);
const wnd_print = window.open(url, 'wnd_print', 'menubar=no,toolbar=no,location=no,status=no,directories=no,resizable=yes,scrollbars=yes');
wnd_print.onload = () => window.URL.revokeObjectURL(url);
return wnd_print;
}

const req = (!method || (typeof method == 'string' && method.toLowerCase().indexOf('post') != -1)) ?
const req = (!method || (typeof method == 'string' && method.toLowerCase().indexOf('post') !== -1)) ?
this.post_ex(url,
typeof post_data == 'object' ? JSON.stringify(post_data) : post_data,
true,
Expand Down Expand Up @@ -539,7 +540,7 @@ const utils = {
function exclude_cpy(f) {
if (!(exclude && exclude.includes(f))) {
// копируем в dst свойства src, кроме тех, которые унаследованы от Object
if ((typeof tobj[f] == 'undefined') || (tobj[f] != src[f])) {
if ((typeof tobj[f] === 'undefined') || (tobj[f] !== src[f])) {
obj[f] = src[f];
}
}
Expand Down Expand Up @@ -665,7 +666,7 @@ const utils = {
const {ne, gt, gte, lt, lte, nin, inh, ninh, lke, nlk} = comparison_types;
switch (comparison_type) {
case ne:
return left != right;
return left !== right;
case gt:
return left > right;
case gte:
Expand All @@ -676,36 +677,36 @@ const utils = {
return left <= right;
case nin:
if(Array.isArray(left) && !Array.isArray(right)) {
return left.indexOf(right) == -1;
return left.indexOf(right) === -1;
}
else if(!Array.isArray(left) && Array.isArray(right)) {
return right.indexOf(left) == -1;
return right.indexOf(left) === -1;
}
else if(!Array.isArray(left) && !Array.isArray(right)) {
return right != left;
return right !== left;
}
break;
case comparison_types.in:
if(Array.isArray(left) && !Array.isArray(right)) {
return left.indexOf(right) != -1;
return left.indexOf(right) !== -1;
}
else if(!Array.isArray(left) && Array.isArray(right)) {
return right.indexOf(left) != -1;
return right.indexOf(left) !== -1;
}
else if(!Array.isArray(left) && !Array.isArray(right)) {
return left == right;
return left === right;
}
break;
case inh:
return utils.is_data_obj(left) ? left._hierarchy(right) : left == right;
return utils.is_data_obj(left) ? left._hierarchy(right) : left === right;
case ninh:
return utils.is_data_obj(left) ? !left._hierarchy(right) : left != right;
return utils.is_data_obj(left) ? !left._hierarchy(right) : left !== right;
case lke:
return left.indexOf && right && left.indexOf(right) !== -1;
case nlk:
return left.indexOf && left.indexOf(right) === -1;
default:
return left == right;
return left === right;
}
},

Expand Down Expand Up @@ -752,7 +753,7 @@ const utils = {


// пропускаем служебные свойства
if (j.substr(0, 1) == '_') {
if (j.substr(0, 1) === '_') {
if(j === '_search' && sel.fields && sel.value) {
ok = sel.value.every((str) => sel.fields.some((fld) => utils._like(o[fld], str)));
if(!ok) {
Expand Down Expand Up @@ -1059,7 +1060,7 @@ utils.__define('blank', {
v = this.date;
else if (mtype['digits'])
v = 0;
else if (mtype.types && mtype.types[0] == 'boolean')
else if (mtype.types && mtype.types[0] === 'boolean')
v = false;
else
v = '';
Expand Down