dip @axlerk @zhigalov

tasks/dip/refactoring/error-formatter/object-error-formatter.js

@@ -0,0 +1,11 @@
+var browser = require('../util/browser');
+
+module.exports = function (options) {
+    return {
+        type: 'handle',
+        hostname: location.hostname,
+        message: options.message,
+        location: options.file + ':' + options.line + ':' + (options.column || 0),
+        browser: browser
+    }
+}

tasks/dip/refactoring/error-formatter/string-error-formatter.js

@@ -0,0 +1,10 @@
+var browser = require('../util/browser');
+
+module.exports = function (options) {
+    return
+        'type: ' + 'handle' +
+        '; hostname: ' + location.hostname +
+        '; message: ' + options.message +
+        '; location: ' + options.file + ':' + options.line + ':' + (options.column || 0) +
+        '; browser: ' + browser;
+}

tasks/dip/refactoring/error-handlers/debounce-error-logger.js

@@ -0,0 +1,17 @@
+var ErrorLogger = require('./error-logger'),
+        _ = require('lodash'),
+        debounce = require(debounce);
+
+var DebounceErrorLogger = function (options) {
+    this._repeatTime = options.repeatTime;
+    this._needFirst = options.needFirst;
+    this._adapter = options.adapter;
+};
+
+_.extend(DebounceErrorLogger.prototype, ErrorLogger.prototype, {
+    handle: debounce(function (options) {
+        this._adapter(options);
+    }, this._repeatTime, this._needFirst)
+});
+
+module.exports = DebounceErrorLogger;

tasks/dip/refactoring/error-handlers/error-logger.js

@@ -0,0 +1,8 @@
+var ErrorLogger = function (options) {
+};
+
+ErrorLogger.prototype.handle = function (options) {
+    new Error('Not implemented exception');
+};
+
+module.exports = ErrorLogger;

tasks/dip/refactoring/error-handlers/full-error-logger.js

@@ -0,0 +1,10 @@
+var ErrorLogger = require('./error-logger'),
+    _ = require('lodash');
+
+var FullErrorLogger = function (options) {
+    this._adapter = options.adapter;
+}
+
+_.extend(FullErrorLogger.prototype, ErrorLogger.prototype, { handle: adapter })
+
+module.exports = FullErrorLogger;

tasks/dip/refactoring/logger.js

@@ -1,22 +1,21 @@
-var debounce = require('debounce');
-
-var browser = navigator.userAgent.match(/[a-zA-Z]+\/[0-9.A-F]+/g)
-    .filter(function (string) {
-        return !/^(AppleWebKit|WebKit|Safari|Gecko|Mozilla)/.test(string);
-    })
-    .map(function (string) {
-        return string.replace(/^Version/, 'Safari');
-    })
-    .join(' ');
+var ErrorLogger = require('./error-handlers/debounce-error-logger'),
+        repeatTime = 1000,
+        needFirst = true,
+        formatter = ('./error-formater/object-error-formatter'),
+        adapter = require('./network-adapters/yaCounter')(yaCounter12208345, formatter);
 
 module.exports = function () {
-    window.onerror = debounce(function (message, file, line, column) {
-        yaCounter12208345.params(
-            'error',
-            location.hostname,
-            message,
-            file + ':' + line + ':' + (column || 0),
-            browser
-        );
-    }, 1000, true);
+    var logger = new ErrorLogger({
+        repeatTime: repeatTime,
+        needFirst: needFirst,
+        adapter: adapter
+    });
+    window.onerror = function (message, file, line, column) {
+        logger.handle({
+            message: message,
+            file: file,
+            line: line,
+            column: column
+        });
+    };
 };

tasks/dip/refactoring/network-adapters/localstorage.js

@@ -0,0 +1,16 @@
+function isAvailableLocalStorage() {
+    try {
+        return 'localStorage' in window && window['localStorage'] !== null;
+    } catch (e) {
+        return false;
+    }
+}
+
+module.exports = function (formatter) {
+    var noop = function () {
+    };
+    return isAvailableLocalStorage() ? function (options) {
+        var data = formatter(options);
+        localStorage["log" + (+new Date())] = data;
+    } : noop;
+}

tasks/dip/refactoring/network-adapters/yaCouner.js

@@ -0,0 +1,13 @@
+module.exports = function (counter, formatter) {
+    return function (options) {
+        var data = formatter(options);
+
+        counter.params(
+            data.type,
+            data.hostname,
+            data.message,
+            data.location,
+            data.browser
+        );
+    }
+}

tasks/dip/refactoring/util/browser.js

@@ -0,0 +1,10 @@
+var Browser = navigator.userAgent.match(/[a-zA-Z]+\/[0-9.A-F]+/g)
+    .filter(function (string) {
+        return !/^(AppleWebKit|WebKit|Safari|Gecko|Mozilla)/.test(string);
+    })
+    .map(function (string) {
+        return string.replace(/^Version/, 'Safari');
+    })
+    .join(' ');
+
+module.exports = Browser;