View Issue Details

IDProjectCategoryView StatusLast Update
0006907Flow themeThemepublic2019-07-24 22:11
Reporterpbenke Assigned To 
PrioritynormalSeveritytweakReproducibilityalways
Status acknowledgedResolutionopen 
Product Version3.0.2 
Summary0006907: jqBootstrapValidation.js is included twice
DescriptionjqBootstrapValidation.js is included twice.

In the base-template the scripts.min.js is included:

File: source/Application/views/flow/tpl/layout/base.tpl:
[{oxscript include="js/scripts.min.js" priority=1}]

This scripts.min.js is generated by grunt and already contains the build/vendor/jquery-bootstrap-validation/js/jqBootstrapValidation.js:

source/Application/views/flow/grunt/uglify.js:

    theme: {
        files: {
            "out/flow/src/js/scripts.min.js": [
                "build/vendor/bootstrap/js/bootstrap.js",
                "build/vendor/bootstrap-select/js/bootstrap-select.js",
                "build/vendor/jquery-bootstrap-validation/js/jqBootstrapValidation.js",
                "build/vendor/jquery-unveil/js/jquery.unveil.js",
                "build/js/main.js"
            ],
            "out/flow/src/js/pages/compare.min.js": "build/js/pages/compare.js",
            "out/flow/src/js/pages/details.min.js": "build/js/pages/details.js",
            "out/flow/src/js/pages/review.min.js": "build/js/pages/review.js",
            "out/flow/src/js/pages/start.min.js": "build/js/pages/start.js"
        }
    },

But the minified jqBootstrapValidation.min.js is also included on many pages, although it is already there inside the scripts.min.js.

Thanks.
TagsJavaScript

Activities

Alpha-Sys

2019-07-24 22:11

reporter   ~0012950

I created a pull request for the wave theme: https://github.com/OXID-eSales/wave-theme/pull/60