Strontium-90 / StrontiumPjaxBundle

Resources/public/js/pjax.js

(function ($, cookie, exports, domInitializer) {
    'use strict';

    var PJAX_PUSH = 'pjax-push';

    var app = exports.application = {
        PJAX_REDIRECT_TARGET_PARAMETER: 'pjax-redirect-target',
        ROOT_CONTAINER_NAME: 'main',

        linkSelector: 'a[data-pjax],' +
        'a:not([data-toggle]):not([data-behavior]):not([data-skip-pjax]):not([href^="http://"]):not([href^="/_profiler/"]):not([href^="/app_dev.php/_profiler/"])',

        formSelector: 'form:not([w]):not([data-skip-pjax])',

        params: {},

        getPage: function (route, params, target) {
            var req_params = this.params;
            for (var i in params) {

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

                req_params[i] = params[i];
            }

            return this.getUrl(Routing.generate(route, req_params), target);

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

        },

        getPjaxContainer: function (target) {
            return findTargetContainer(target || app.ROOT_CONTAINER_NAME);
        },

        getUrl: function (url, target) {
            var container = this.getPjaxContainer(target);
            url = url || '';

            return $.pjax({
                url: url,
                container: container,
                method: 'get',
                push: toPush(target, 'GET'),
                replace: false
            });
        },

        reload: function (target, url) {
            return this.getUrl(url, target);
        }
    };

    $(function () {
        if ($.support.pjax) {
            $.pjax.defaults.timeout = 50000;

            $(document)
                .on('click', app.linkSelector, onPjaxLinkClick)
                .on('submit', app.formSelector, onPjaxFormSubmit)
                .on('pjax:complete', onPjaxComplete)
                .on('pjax:beforeSend', onPjaxBeforeSend)
                .on('pjax:beforeReplace', onPjaxBeforeReplace);
        }
    });

    function onPjaxLinkClick(event) {
        if (event.isDefaultPrevented()) {
            return;
        }
        var target = findPjaxTargetFor(this);
        var container = findTargetContainer(target);
        var redirectTarget = $(this).data(app.PJAX_REDIRECT_TARGET_PARAMETER);

        $.pjax.click(event, container, {
            target: target,
            redirectTarget: redirectTarget,
            push: toPush(target, 'GET', $(this).data(PJAX_PUSH)),
            replace: false
        });
    }

    function onPjaxFormSubmit(event) {
        if (event.isDefaultPrevented()) {
            return;
        }
        var $form = $(this);
        var target = findPjaxTargetFor(this);
        var targetContainer = findTargetContainer(target);

        /**
         * Если пытаемся отправить форму с файлами,
         * но браузером не поддерживается FormData,
         * тогда просто штатно отправляем форму.
         * Пока так.
         */
        if ('multipart/form-data' === $form.attr('enctype') && window.FormData === undefined) {
            return true;
        }
        var params = {
            target: target,
            redirectTarget: targetContainer.data(app.PJAX_REDIRECT_TARGET_PARAMETER),
            push: toPush(target, $form.attr('method'), $(this).data(PJAX_PUSH)),
            replace: false
        };

        if ($form.attr('enctype') === 'multipart/form-data') {
            $.extend(params, {
                contentType: false,
                processData: false,
                cache: false,
                data: $form.serializeMultipart()
            })
        }

        $.pjax.submit(event, targetContainer, params);

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.

    }

    /**
     * Пушить стейт или нет
     *
     * @param target
     * @param method
     * @param option bool значение атрибута data-pjax-push
     * @returns bool
     */
    function toPush(target, method, option) {
        if (option == undefined) {
            if (method == undefined) {
                method = 'GET';
            }
            method = method.toUpperCase();

            return method == 'GET' && target == app.ROOT_CONTAINER_NAME;
        }

        return option;
    }


    function onPjaxComplete(event, content, status, options) {

The parameter options is not used and could be removed.

The parameter content is not used and could be removed.

The parameter status is not used and could be removed.

        domInitializer.initialize(event.target);
    }

    function onPjaxBeforeReplace(event, contents, options) {
        var redirectedTo,
            redirectCookieTargetName,
            redirectCookieName,
            optionsTransformer,
            generateStateParams,
            redirectTarget = options.redirectTarget;

        if (redirectTarget) {
            redirectCookieTargetName = parsePjaxContainerSelector(options.context.selector);
            optionsTransformer = function (options) {
                return $.extend(options, {
                    context: findTargetContainer(redirectTarget)
                });
            };
            generateStateParams = function (options) {
                return {
                    container: options.context.selector
                };
            };
        } else {
            optionsTransformer = function (options) {
                return options;
            };
            generateStateParams = function (options) {

The parameter options is not used and could be removed.

                return {};
            };
            redirectTarget = redirectCookieTargetName = findPjaxTargetFor(event.target);
        }

        redirectCookieName = 'pjax_redirect_' + redirectCookieTargetName;

        if (undefined !== (redirectedTo = cookie.get(redirectCookieName))) {
            cookie.expire(redirectCookieName);

            options = optionsTransformer(options);

            if (toPush(redirectTarget, 'GET')) {
                $.extend(event.state, {
                    push: true,
                    url: redirectedTo
                }, generateStateParams(options));
                window.history.pushState(event.state, event.state.title, event.state.url);
            }

            if (redirectTarget && redirectTarget != app.PJAX_MODAL_CONTAINER) {
                $(app.PJAX_MODAL_SELECTOR).modal('hide');
            }
        }
    }

    function onPjaxBeforeSend(event, xhr, settings) {
        xhr.setRequestHeader('X-PJAX-Target', settings.target);
        settings.redirectTarget && xhr.setRequestHeader('X-PJAX-Redirect-Target', settings.redirectTarget);
    }

    function findPjaxTargetFor(elem) {
        var $elem = $(elem);

        return $elem.data('pjax')
            || $elem.closest('[data-pjax-container]').data('pjax-container')
            || app.ROOT_CONTAINER_NAME;
    }

    function findTargetContainer(target) {
        var container = $(generatePjaxContainerSelector(target));

        return container.length ? container : null;
    }

    function generatePjaxContainerSelector(name) {
        return '[data-pjax-container="' + name + '"]';
    }

    /**
     * @param {string} selector
     * @returns {string}
     */
    function parsePjaxContainerSelector(selector) {
        return selector.match(/^\[data-pjax-container="(.+?)"\]$/)[1];
    }

    $.fn.serializeMultipart = function () {
        var obj = $(this);
        /* ADD FILE TO PARAM AJAX */
        var formData = new FormData();
        $.each($(obj).find("input[type='file']"), function (i, tag) {
            $.each($(tag)[0].files, function (i, file) {
                formData.append(tag.name, file);
            });
        });
        var params = $(obj).serializeArray();
        $.each(params, function (i, val) {
            formData.append(val.name, val.value);
        });
        return formData;
    };

})(jQuery, Cookies, window, domInitializer);

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

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