okaufmann / laravel-web-money-manager-ex

resources/assets/js/bootstrap.js

/**
 * We'll load the axios HTTP library which allows us to easily issue requests
 * to our Laravel back-end. This library automatically handles sending the
 * CSRF token as a header based on the value of the "XSRF" token cookie.
 */

window.axios.defaults.headers.common['X-Requested-With'] = 'XMLHttpRequest';

/**
 * Sett global authorization headers for js libraries
 */
window.axios.defaults.headers.common['Authorization'] = "Bearer " + Laravel.apiToken;

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

$.ajaxSetup({
    beforeSend: function (xhr) {
        xhr.setRequestHeader("Authorization", "Bearer " + Laravel.apiToken);

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

    }
});
kendo.jQuery.ajaxSetup({

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

    beforeSend: function (xhr) {
        xhr.setRequestHeader("Authorization", "Bearer " + Laravel.apiToken);

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

    }
});

/**
 * Next we will register the CSRF Token as a common header with Axios so that
 * all outgoing HTTP requests automatically have it attached. This is just
 * a simple convenience so we don't have to attach every token manually.
 */

let token = document.head.querySelector('meta[name="csrf-token"]');

if (token) {
    window.axios.defaults.headers.common['X-CSRF-TOKEN'] = token.content;
} else {
    console.error('CSRF token not found: https://laravel.com/docs/csrf#csrf-x-csrf-token');
}

/**
 * Echo exposes an expressive API for subscribing to channels and listening
 * for events that are broadcast by Laravel. Echo and event broadcasting
 * allows your team to easily build robust real-time web applications.
 */

// window.Echo = new Echo({
//     broadcaster: 'pusher',
//     key: 'your-pusher-key'
// });

// Material Buttons
$(() => {
    window['mmex'] = window['mmex'] || {};
    window['mmex']['momentLocal'] = (date) => {
        if (!date || _.isEmpty(date)) {

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

            return moment();
        }

        let m = moment(date);
        if (m.isValid()) {
            return m;
        }
        m = moment(date, "DD.MM.YYYY");
        if (m.isValid()) {
            return m;
        }

        throw new Error("No supported date format found for ", date);
    };

    $.material.init();

    autosize($('textarea'));

    $("[autofocus]").focus();

    $(".common-dateinput").each((index, elm) => {
        let val = $(elm).val();
        let date = mmex.momentLocal(val).toDate();

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

        console.log("set date ", date, " out of ", val);

console.log looks like debug code. Are you sure you do not want to remove it?

        new kendo.ui.DateInput($(elm), {

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

The object created with new kendo.ui.DateInput($(elm), {IdentifierNode(value,false): ConditionalExpressionNode(IdentifierNode(val,false),IdentifierNode(date,false),NewExpressionNode(IdentifierNode(Date,false),ListBuffer()))}) is not used but discarded. Consider invoking another function instead of a constructor if you are doing this purely for side effects.

            value: val ? date : new Date()
        });
    });

    $(".common-datepicker").each((index, elm) => {
        let val = $(elm).val();
        let date = mmex.momentLocal(val).toDate();

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

        console.log("set date ", date, " out of ", val);

console.log looks like debug code. Are you sure you do not want to remove it?

        new kendo.ui.DatePicker($(elm), {

The object created with new kendo.ui.DatePicker($(elm), {IdentifierNode(value,false): ConditionalExpressionNode(IdentifierNode(val,false),IdentifierNode(date,false),NewExpressionNode(IdentifierNode(Date,false),ListBuffer()))}) is not used but discarded. Consider invoking another function instead of a constructor if you are doing this purely for side effects.

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

            value: val ? date : new Date()
        });
    });

    $(".common-dropdown-list").each((index, elm) => {
        new kendo.ui.DropDownList($(elm), {

The object created with new kendo.ui.DropDownList($(elm), {IdentifierNode(filter,false): StringLiteralNode(startswith)}) is not used but discarded. Consider invoking another function instead of a constructor if you are doing this purely for side effects.

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

            filter: "startswith",
        });
    });

    $('textarea, input').keyup(function (e) {
        if (e.which == 17) isCtrl = false;

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

The variable isCtrl seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.isCtrl.

    }).keydown(function (e) {
        if (e.which == 17) isCtrl = true;

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

The variable isCtrl seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.isCtrl.

        if (e.which == 13 && isCtrl === true) {

The variable isCtrl does not seem to be initialized in case e.which == 17 on line 102 is false. Are you sure this can never be the case?

There is no return statement if e.which == 13 && isCtrl === true is false. Are you sure this is correct? If so, consider adding return; explicitly.

            $(this).closest('form').submit();
            return false;
        }
    });
});