LSX_Customizer_Colour_Banner::get_css() - Code Metrics - lightspeeddevelopment/lsx-customizer - Measure and Improve Code Quality continuously with Scrutinizer

LSX_Customizer_Colour_Banner::get_css() B
last analyzed 2022-05-19 07:25 UTC

↳ Parent: LSX_Customizer_Colour_Banner

Complexity

Conditions	8
Paths	4

Size

Total Lines	56
Code Lines	27

Duplication

Lines	0
Ratio	0 %

Importance

Changes	4
Bugs	0	Features	0

Metric	Value
cc	8
eloc	27
c	4
b	0
f	0
nc	4
nop	1
dl	0
loc	56
rs	8.4444

How to fix Long Method

<?php

if ( ! class_exists( 'LSX_Customizer_Colour_Banner' ) ) {


	/**
	 * LSX Customizer Colour Banner Class
	 *
	 * @package   LSX Customizer
	 * @author    LightSpeed
	 * @license   GPL3
	 * @link
	 * @copyright 2016 LightSpeed
	 */
	class LSX_Customizer_Colour_Banner extends LSX_Customizer_Colour {

		/**
		 * Constructor.
		 *
		 * @since 1.0.0
		 */
		public function __construct() {

			add_action( 'after_switch_theme',   array( $this, 'set_theme_mod' ) );

			add_action( 'customize_save_after', array( $this, 'set_theme_mod' ) );

			add_action( 'wp_enqueue_scripts', array( $this, 'enqueue_css' ), 2999 );
		}


		/**
		 * Assign CSS to theme mod.
		 *
		 * @since 1.0.0
		 */
		public function set_theme_mod() {
			$theme_mods = $this->get_theme_mods();
			$styles     = $this->get_css( $theme_mods );

			set_theme_mod( 'lsx_customizer_colour__banner_theme_mod', $styles );
		}


		/**
		 * Enqueues front-end CSS.
		 *
		 * @since 1.0.0
		 */
		public function enqueue_css() {
			$styles_from_theme_mod = get_theme_mod( 'lsx_customizer_colour__banner_theme_mod' );

			if ( is_customize_preview() || false === $styles_from_theme_mod ) {

				$theme_mods = $this->get_theme_mods();
				$styles     = $this->get_css( $theme_mods );

				if ( false === $styles_from_theme_mod ) {

					set_theme_mod( 'lsx_customizer_colour__banner_theme_mod', $styles );
				}
			} else {
				$styles = $styles_from_theme_mod;
			}

			wp_add_inline_style( 'lsx-customizer', $styles );
		}


		/**
		 * Get CSS theme mods.
		 *
		 * @since 1.0.0
		 */
		public function get_theme_mods() {
			$colors = parent::get_color_scheme();

			return apply_filters( 'lsx_customizer_colours_banner', array(
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
);
				'banner_background_color'               => get_theme_mod( 'banner_background_color',               $colors['banner_background_color'] ),

				'banner_text_color'                     => get_theme_mod( 'banner_text_color',                     $colors['banner_text_color'] ),

				'banner_text_image_color'               => get_theme_mod( 'banner_text_image_color',               $colors['banner_text_image_color'] ),

				'banner_breadcrumb_background_color'    => get_theme_mod( 'banner_breadcrumb_background_color',    $colors['banner_breadcrumb_background_color'] ),

				'banner_breadcrumb_text_color'          => get_theme_mod( 'banner_breadcrumb_text_color',          $colors['banner_breadcrumb_text_color'] ),

				'banner_breadcrumb_text_selected_color' => get_theme_mod( 'banner_breadcrumb_text_selected_color', $colors['banner_breadcrumb_text_selected_color'] ),
			) );
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
		}


		/**

		 * Returns CSS.
		 *
		 * @since 1.0.0
		 */
		function get_css( $colors ) {

			global $customizer_colour_names;

			$colors_template = array();

			foreach ( $customizer_colour_names as $key => $value ) {

				$colors_template[ $key ] = '';
			}

			$colors = wp_parse_args( $colors, $colors_template );

			if ( empty( $colors['banner_background_color'] )

				|| empty( $colors['banner_text_color'] )
				|| empty( $colors['banner_text_image_color'] )
				|| empty( $colors['banner_breadcrumb_background_color'] )
				|| empty( $colors['banner_breadcrumb_text_color'] )
				|| empty( $colors['banner_breadcrumb_text_selected_color'] ) ) {
				return '';
			}

			$css = '
				@import "' . get_template_directory() . '/assets/css/scss/global/mixins/banner";

				/**
				 * LSX Customizer - Banner
				 */
				@include banner-colours (
					$bg:                 ' . $colors['banner_background_color'] . ',
					$color:              ' . $colors['banner_text_color'] . ',
					$color-image:        ' . $colors['banner_text_image_color'] . ',
					$breadcrumb-bg:      ' . $colors['banner_breadcrumb_background_color'] . ',
					$breadcrumb-color:   ' . $colors['banner_breadcrumb_text_color'] . ',
					$breadcrumb-current: ' . $colors['banner_breadcrumb_text_selected_color'] . '
				);

				.lsx:not(.single-post) .archive-header .archive-title, .lsx:not(.single-post) .archive-header .page-title, .lsx:not(.single-post) .archive-header *, .lsx:not(.single-post) .archive-header *, .lsx:not(.single-post) .archive-header >p, .lsx:not(.single-post) .archive-header >p, .lsx:not(.single-post):not(.page-has-banner) .archive-header-wrapper .archive-header .archive-title, .lsx:not(.single-post):not(.page-has-banner) .archive-header-wrapper .archive-header>p, .lsx-layout-switcher .lsx-layout-switcher-option {
					color: ' . $colors['banner_text_color'] . ';
				}

				.lsx:not(.single-post) .archive-header.banner-has-custom-bg .archive-title, .lsx:not(.single-post) .archive-header.banner-has-custom-bg .page-title, .lsx:not(.single-post) .archive-header.banner-has-custom-bg *, .lsx:not(.single-post) .archive-header.banner-has-custom-bg *, .lsx:not(.single-post) .archive-header.banner-has-custom-bg >p, .lsx:not(.single-post) .archive-header.banner-has-custom-bg >p, .lsx:not(.single-post):not(.page-has-banner) .archive-header-wrapper .archive-header.banner-has-custom-bg .archive-title, .lsx:not(.single-post):not(.page-has-banner) .archive-header-wrapper .archive-header.banner-has-custom-bg>p, .banner-has-custom-bg .lsx-layout-switcher .lsx-layout-switcher-option {
					color: ' . $colors['banner_text_image_color'] . ';
				}
				.lsx-hero-banner-block {
					background-color: ' . $colors['banner_background_color'] . ';
					.lsx-title-block {
						h1 {
							color: ' . $colors['banner_text_color'] . ';
						}
					}
				}
			';

			$css = apply_filters( 'lsx_customizer_colour_selectors_banner', $css, $colors );
			$css = parent::scss_to_css( $css );

			return $css;
		}


	}

}


1			<?php
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report This file is missing a doc comment. Loading history...
2			if ( ! class_exists( 'LSX_Customizer_Colour_Banner' ) ) {
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 0 spaces after opening bracket; 1 found Loading history... Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 0 spaces before closing bracket; 1 found Loading history...
3
4			/**
5			* LSX Customizer Colour Banner Class
6			*
7			* @package LSX Customizer
8			* @author LightSpeed
9			* @license GPL3
10			* @link
11			* @copyright 2016 LightSpeed
12			*/
13			class LSX_Customizer_Colour_Banner extends LSX_Customizer_Colour {
14
15			/**
16			* Constructor.
17			*
18			* @since 1.0.0
19			*/
20			public function __construct() {
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 2 blank lines before function; 1 found Loading history...
21			add_action( 'after_switch_theme', array( $this, 'set_theme_mod' ) );
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 1 space after comma in argument list; 3 found Loading history...
22			add_action( 'customize_save_after', array( $this, 'set_theme_mod' ) );
23
24			add_action( 'wp_enqueue_scripts', array( $this, 'enqueue_css' ), 2999 );
25			}
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 1 blank line before closing function brace; 0 found Loading history... Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 1 found Loading history...
26
27			/**
28			* Assign CSS to theme mod.
29			*
30			* @since 1.0.0
31			*/
32			public function set_theme_mod() {
33			$theme_mods = $this->get_theme_mods();
34			$styles = $this->get_css( $theme_mods );
35
36			set_theme_mod( 'lsx_customizer_colour__banner_theme_mod', $styles );
37			}
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 1 blank line before closing function brace; 0 found Loading history... Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 1 found Loading history...
38
39			/**
40			* Enqueues front-end CSS.
41			*
42			* @since 1.0.0
43			*/
44			public function enqueue_css() {
45			$styles_from_theme_mod = get_theme_mod( 'lsx_customizer_colour__banner_theme_mod' );
46
47			if ( is_customize_preview() \|\| false === $styles_from_theme_mod ) {
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 0 spaces after opening bracket; 1 found Loading history... Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 0 spaces before closing bracket; 1 found Loading history...
48			$theme_mods = $this->get_theme_mods();
49			$styles = $this->get_css( $theme_mods );
50
51			if ( false === $styles_from_theme_mod ) {
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 0 spaces after opening bracket; 1 found Loading history... Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 0 spaces before closing bracket; 1 found Loading history...
52			set_theme_mod( 'lsx_customizer_colour__banner_theme_mod', $styles );
53			}
54			} else {
55			$styles = $styles_from_theme_mod;
56			}
57
58			wp_add_inline_style( 'lsx-customizer', $styles );
59			}
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 1 blank line before closing function brace; 0 found Loading history... Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 1 found Loading history...
60
61			/**
62			* Get CSS theme mods.
63			*
64			* @since 1.0.0
65			*/
66			public function get_theme_mods() {
67			$colors = parent::get_color_scheme();
68
69			return apply_filters( 'lsx_customizer_colours_banner', array(
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report The opening parenthesis of a multi-line function call should be the last content on the line. Loading history... Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report For multi-line function calls, each argument should be on a separate line. For a function calls that spawns multiple lines, the coding style suggests to split arguments to separate lines like this: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); Loading history...
70			'banner_background_color' => get_theme_mod( 'banner_background_color', $colors['banner_background_color'] ),
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 1 space after comma in argument list; 15 found Loading history...
71			'banner_text_color' => get_theme_mod( 'banner_text_color', $colors['banner_text_color'] ),
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 1 space after comma in argument list; 21 found Loading history...
72			'banner_text_image_color' => get_theme_mod( 'banner_text_image_color', $colors['banner_text_image_color'] ),
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 1 space after comma in argument list; 15 found Loading history...
73			'banner_breadcrumb_background_color' => get_theme_mod( 'banner_breadcrumb_background_color', $colors['banner_breadcrumb_background_color'] ),
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 1 space after comma in argument list; 4 found Loading history...
74			'banner_breadcrumb_text_color' => get_theme_mod( 'banner_breadcrumb_text_color', $colors['banner_breadcrumb_text_color'] ),
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 1 space after comma in argument list; 10 found Loading history...
75			'banner_breadcrumb_text_selected_color' => get_theme_mod( 'banner_breadcrumb_text_selected_color', $colors['banner_breadcrumb_text_selected_color'] ),
76			) );
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report For multi-line function calls, the closing parenthesis should be on a new line. If a function call spawns multiple lines, the coding standard suggests to move the closing parenthesis to a new line: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); // Closing parenthesis on a new line. Loading history...
77			}
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 1 blank line before closing function brace; 0 found Loading history... Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 1 found Loading history...
78
79			/**
			0 ignored issues – show Coding Style Documentation introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Doc comment for parameter "$colors" missing Loading history...
80			* Returns CSS.
81			*
82			* @since 1.0.0
83			*/
84			function get_css( $colors ) {
			0 ignored issues – show Best Practice introduced 2020-08-14 21:35 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history... Comprehensibility Best Practice introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report It is recommend to declare an explicit visibility for `get_css`. Generally, we recommend to declare visibility for all methods in your source code. This has the advantage of clearly communication to other developers, and also yourself, how this method should be consumed. If you are not sure which visibility to choose, it is a good idea to start with the most restrictive visibility, and then raise visibility as needed, i.e. start with `private`, and only raise it to `protected` if a sub-class needs to have access, or `public` if an external class needs access. Loading history...
85			global $customizer_colour_names;
86
87			$colors_template = array();
88
89			foreach ( $customizer_colour_names as $key => $value ) {
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 0 spaces after opening bracket; 1 found Loading history... Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 0 spaces before closing bracket; 1 found Loading history...
90			$colors_template[ $key ] = '';
91			}
92
93			$colors = wp_parse_args( $colors, $colors_template );
94
95			if ( empty( $colors['banner_background_color'] )
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 0 spaces after opening bracket; 1 found Loading history...
96			\|\| empty( $colors['banner_text_color'] )
97			\|\| empty( $colors['banner_text_image_color'] )
98			\|\| empty( $colors['banner_breadcrumb_background_color'] )
99			\|\| empty( $colors['banner_breadcrumb_text_color'] )
100			\|\| empty( $colors['banner_breadcrumb_text_selected_color'] ) ) {
101			return '';
102			}
103
104			$css = '
105			@import "' . get_template_directory() . '/assets/css/scss/global/mixins/banner";
106
107			/**
108			* LSX Customizer - Banner
109			*/
110			@include banner-colours (
111			$bg: ' . $colors['banner_background_color'] . ',
112			$color: ' . $colors['banner_text_color'] . ',
113			$color-image: ' . $colors['banner_text_image_color'] . ',
114			$breadcrumb-bg: ' . $colors['banner_breadcrumb_background_color'] . ',
115			$breadcrumb-color: ' . $colors['banner_breadcrumb_text_color'] . ',
116			$breadcrumb-current: ' . $colors['banner_breadcrumb_text_selected_color'] . '
117			);
118
119			.lsx:not(.single-post) .archive-header .archive-title, .lsx:not(.single-post) .archive-header .page-title, .lsx:not(.single-post) .archive-header , .lsx:not(.single-post) .archive-header , .lsx:not(.single-post) .archive-header >p, .lsx:not(.single-post) .archive-header >p, .lsx:not(.single-post):not(.page-has-banner) .archive-header-wrapper .archive-header .archive-title, .lsx:not(.single-post):not(.page-has-banner) .archive-header-wrapper .archive-header>p, .lsx-layout-switcher .lsx-layout-switcher-option {
120			color: ' . $colors['banner_text_color'] . ';
121			}
122
123			.lsx:not(.single-post) .archive-header.banner-has-custom-bg .archive-title, .lsx:not(.single-post) .archive-header.banner-has-custom-bg .page-title, .lsx:not(.single-post) .archive-header.banner-has-custom-bg , .lsx:not(.single-post) .archive-header.banner-has-custom-bg , .lsx:not(.single-post) .archive-header.banner-has-custom-bg >p, .lsx:not(.single-post) .archive-header.banner-has-custom-bg >p, .lsx:not(.single-post):not(.page-has-banner) .archive-header-wrapper .archive-header.banner-has-custom-bg .archive-title, .lsx:not(.single-post):not(.page-has-banner) .archive-header-wrapper .archive-header.banner-has-custom-bg>p, .banner-has-custom-bg .lsx-layout-switcher .lsx-layout-switcher-option {
124			color: ' . $colors['banner_text_image_color'] . ';
125			}
126			.lsx-hero-banner-block {
127			background-color: ' . $colors['banner_background_color'] . ';
128			.lsx-title-block {
129			h1 {
130			color: ' . $colors['banner_text_color'] . ';
131			}
132			}
133			}
134			';
135
136			$css = apply_filters( 'lsx_customizer_colour_selectors_banner', $css, $colors );
137			$css = parent::scss_to_css( $css );
138
139			return $css;
140			}
			0 ignored issues – show Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 1 blank line before closing function brace; 0 found Loading history... Coding Style introduced 2020-08-27 17:03 UTC by Report Bug Copy Issue Report Expected 2 blank lines after function; 1 found Loading history...
141
142			}
143
144			}
145

lightspeeddevelopment / lsx-customizer

LSX_Customizer_Colour_Banner::get_css() B last analyzed 2022-05-19 07:25 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like

LSX_Customizer_Colour_Banner::get_css() B
last analyzed 2022-05-19 07:25 UTC