BootstrapUtil.initPopover - Code Metrics - Inspection of "Merge branch 'next-21195/fix-profiler-requirement'..." - shopware/platform - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — trunk ( 08307f...9f0935 )

by Christian

created 2022-10-13 10:01 UTC

BootstrapUtil.initPopover A

↳ Parent: BootstrapUtil

Complexity

Conditions

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	9
c	0
b	0
f	0
dl	0
loc	14
rs	9.95

import Feature from 'src/helper/feature.helper';

/**
 * @deprecated tag:v6.5.0 - The bootstrap `data-toggle` attribute will be renamed to `data-bs-toggle`
 * @see https://getbootstrap.com/docs/5.0/migration/#javascript
 * @type {string}
 */
const BS_TOGGLE_ATTR = Feature.isActive('v6.5.0.0') ? 'data-bs-toggle' : 'data-toggle';

const TOOLTIP_SELECTOR = `[${BS_TOGGLE_ATTR}="tooltip"]`;
const POPOVER_SELECTOR = `[${BS_TOGGLE_ATTR}="popover"]`;

export default class BootstrapUtil {

    /**
     * Initialize Tooltip plugin everywhere
     * @see https://getbootstrap.com/docs/4.3/components/tooltips/#example-enable-tooltips-everywhere
     */
    static initTooltip() {
        /** @deprecated tag:v6.5.0 - Bootstrap v5 uses native elements to init Tooltip plugin */
        if (Feature.isActive('v6.5.0.0')) {
            return new bootstrap.Tooltip(document.body, {

                selector: TOOLTIP_SELECTOR,
            });
        } else {

            $('body').tooltip({
                selector: TOOLTIP_SELECTOR,
            });

        }
    }

    /**
     * Initialize Popover plugin everywhere
     * @see https://getbootstrap.com/docs/4.3/components/popovers/#example-enable-popovers-everywhere
     */
    static initPopover() {
        /** @deprecated tag:v6.5.0 - Bootstrap v5 uses native elements to init Popover plugin */
        if (Feature.isActive('v6.5.0.0')) {
            new bootstrap.Popover(document.body, {

                selector: POPOVER_SELECTOR,
                trigger: 'focus',
            });
        } else {
            $('body').popover({
                selector: POPOVER_SELECTOR,
                trigger: 'focus',
            });
        }
    }

    static initBootstrapPlugins() {
        this.initTooltip();
        this.initPopover();
    }
}


1			import Feature from 'src/helper/feature.helper';
2
3			/**
4			* @deprecated tag:v6.5.0 - The bootstrap `data-toggle` attribute will be renamed to `data-bs-toggle`
5			* @see https://getbootstrap.com/docs/5.0/migration/#javascript
6			* @type {string}
7			*/
8			const BS_TOGGLE_ATTR = Feature.isActive('v6.5.0.0') ? 'data-bs-toggle' : 'data-toggle';
9
10			const TOOLTIP_SELECTOR = `[${BS_TOGGLE_ATTR}="tooltip"]`;
11			const POPOVER_SELECTOR = `[${BS_TOGGLE_ATTR}="popover"]`;
12
13			export default class BootstrapUtil {
14
15			/**
16			* Initialize Tooltip plugin everywhere
17			* @see https://getbootstrap.com/docs/4.3/components/tooltips/#example-enable-tooltips-everywhere
18			*/
19			static initTooltip() {
20			/** @deprecated tag:v6.5.0 - Bootstrap v5 uses native elements to init Tooltip plugin */
21			if (Feature.isActive('v6.5.0.0')) {
22			return new bootstrap.Tooltip(document.body, {
			0 ignored issues – show Bug introduced 2022-10-13 10:15 UTC by Report Bug Copy Issue Report The variable `bootstrap` seems to be never declared. If this is a global, consider adding a `/** global: bootstrap */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
23			selector: TOOLTIP_SELECTOR,
24			});
25			} else {
			0 ignored issues – show Comprehensibility introduced 2022-10-13 10:15 UTC by Report Bug Copy Issue Report `else` is not necessary here since all `if` branches return, consider removing it to reduce nesting and make code more readable. Loading history...
26			$('body').tooltip({
27			selector: TOOLTIP_SELECTOR,
28			});
			0 ignored issues – show Best Practice introduced 2022-10-13 10:15 UTC by Report Bug Copy Issue Report 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. Loading history...
29			}
30			}
31
32			/**
33			* Initialize Popover plugin everywhere
34			* @see https://getbootstrap.com/docs/4.3/components/popovers/#example-enable-popovers-everywhere
35			*/
36			static initPopover() {
37			/** @deprecated tag:v6.5.0 - Bootstrap v5 uses native elements to init Popover plugin */
38			if (Feature.isActive('v6.5.0.0')) {
39			new bootstrap.Popover(document.body, {
			0 ignored issues – show Bug introduced 2022-10-13 10:15 UTC by Report Bug Copy Issue Report The variable `bootstrap` seems to be never declared. If this is a global, consider adding a `/** global: bootstrap */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history... Unused Code Best Practice introduced 2022-10-13 10:15 UTC by Report Bug Copy Issue Report The object created with `new bootstrap.Popover(do...ngLiteralNode(focus))})` is not used but discarded. Consider invoking another function instead of a constructor if you are doing this purely for side effects. Loading history...
40			selector: POPOVER_SELECTOR,
41			trigger: 'focus',
42			});
43			} else {
44			$('body').popover({
45			selector: POPOVER_SELECTOR,
46			trigger: 'focus',
47			});
48			}
49			}
50
51			static initBootstrapPlugins() {
52			this.initTooltip();
53			this.initPopover();
54			}
55			}
56

shopware / platform

Push — trunk ( 08307f...9f0935 )

BootstrapUtil.initPopover A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like