FooGallery_Blocks::enqueue_block_editor_assets() - Code Metrics - Inspection of "include masonry" - fooplugins/foogallery - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — feature/gutenberg ( 6ad327...ccecaf )

by Brad

created 2018-11-02 13:29 UTC

FooGallery_Blocks::enqueue_block_editor_assets() B

↳ Parent: FooGallery_Blocks

Complexity

Conditions	4
Paths	6

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
nc	6
nop	0
dl	0
loc	57
rs	8.9381
c	0
b	0
f	0

How to fix Long Method

<?php
/**
 * FooGallery Blocks Initializer
 *
 * Enqueue CSS/JS of all the FooGallery blocks.
 *
 * @since   1.0.0
 * @package CGB
 */

if ( ! class_exists( 'FooGallery_Blocks' ) ) {
	class FooGallery_Blocks {
namespace YourVendor;

class YourClass { }

		function __construct() {

			//Frontend block assets.
			add_action( 'enqueue_block_assets', array( $this, 'enqueue_block_assets' ) );

			//Backend editor block assets.
			add_action( 'enqueue_block_editor_assets', array( $this, 'enqueue_block_editor_assets' ) );

			add_action( 'init', array( $this, 'php_block_init' ) );
		}

		/**
		 * Enqueue Gutenberg block assets for backend editor.
		 *
		 * `wp-blocks`: includes block type registration and related functions.
		 * `wp-element`: includes the WordPress Element abstraction for describing the structure of your blocks.
		 * `wp-i18n`: To internationalize the block's text.
		 *
		 * @since 1.0.0
		 */
		function enqueue_block_editor_assets() {


			//enqueue foogallery dependencies
			wp_enqueue_script( 'masonry' );
			foogallery_enqueue_core_gallery_template_script();
			foogallery_enqueue_core_gallery_template_style();

			$deps = array(
				'wp-blocks',
				'wp-i18n',
				'wp-element',
				'wp-components',
				'wp-editor',
				'foogallery-core'
			);

			$js_url = plugins_url( 'gutenberg/dist/blocks.build.js', dirname( __FILE__ ) );

			// Scripts.
			wp_enqueue_script(
				'foogallery-block-js', // Handle.
				$js_url, // Block.build.js: We register the block here. Built with Webpack.
				$deps, // Dependencies, defined above.
				// filemtime( plugin_dir_path( __DIR__ ) . 'dist/blocks.build.js' ), // Version: filemtime — Gets file modification time.

				true // Enqueue the script in the footer.
			);

			// Styles.
			wp_enqueue_style(
				'foogallery-block-editor-css', // Handle.
				plugins_url( 'gutenberg/dist/blocks.editor.build.css', dirname( __FILE__ ) ), // Block editor CSS.
				array( 'wp-edit-blocks', 'foogallery-core' ) // Dependency to include the CSS after it.
				// filemtime( plugin_dir_path( __DIR__ ) . 'dist/blocks.editor.build.css' ) // Version: filemtime — Gets file modification time.

			);

			$local_data = false;

			if ( function_exists( 'wp_get_jed_locale_data' ) ) {
				$local_data = wp_get_jed_locale_data( 'foogallery' );
			} else if ( function_exists( 'gutenberg_get_jed_locale_data' ) ) {
				$local_data = gutenberg_get_jed_locale_data( 'foogallery' );
			}

			if ( false !== $local_data ) {

				/*
				 * Pass already loaded translations to our JavaScript.
				 *
				 * This happens _before_ our JavaScript runs, afterwards it's too late.
				 */
				wp_add_inline_script(
					'foogallery-block-js',
					'wp.i18n.setLocaleData( ' . json_encode( $local_data ) . ', "foogallery" );',
					'before'
				);
			}
		}

		/**
		 * Enqueue Gutenberg block assets for both frontend + backend.
		 *
		 * `wp-blocks`: includes block type registration and related functions.
		 *
		 * @since 1.0.0
		 */
		function enqueue_block_assets() {

			// Styles.
			wp_enqueue_style(
				'foogallery-block-css',
				plugins_url( 'gutenberg/dist/blocks.style.build.css', dirname( __FILE__ ) ),
				array( 'wp-blocks' )
			);
		}

		/**
		 * Register our block and shortcode.
		 */
		function php_block_init() {

			//get out quickly if no Gutenberg
			if ( !function_exists( 'register_block_type' ) ) {
				return;
			}

			// Register our block, and explicitly define the attributes we accept.
			register_block_type(
				'fooplugins/foogallery', array(
				'attributes' => array(
					'id' => array(
						'type' => 'number',
						'default' => 0
					),
				),
				'render_callback' => array( $this, 'render_block' ),
			));
		}

		function render_block( $attributes ) {

			$foogallery_id = $attributes['id'];
			$args = array(
				'id' => $foogallery_id
			);
			//create new instance of template engine
			$engine = new FooGallery_Template_Loader();

			ob_start();

			$engine->render_template( $args );

			$output_string = ob_get_contents();
			ob_end_clean();
			return !empty($output_string) ? $output_string : null;
		}
	}
}


1			<?php
2			/**
3			* FooGallery Blocks Initializer
4			*
5			* Enqueue CSS/JS of all the FooGallery blocks.
6			*
7			* @since 1.0.0
8			* @package CGB
9			*/
10
11			if ( ! class_exists( 'FooGallery_Blocks' ) ) {
12			class FooGallery_Blocks {
			0 ignored issues – show Coding Style Compatibility introduced 2017-10-08 18:52 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
13
14			function __construct() {
			0 ignored issues – show Best Practice introduced 2018-10-22 04:26 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...
15			//Frontend block assets.
16			add_action( 'enqueue_block_assets', array( $this, 'enqueue_block_assets' ) );
17
18			//Backend editor block assets.
19			add_action( 'enqueue_block_editor_assets', array( $this, 'enqueue_block_editor_assets' ) );
20
21			add_action( 'init', array( $this, 'php_block_init' ) );
22			}
23
24			/**
25			* Enqueue Gutenberg block assets for backend editor.
26			*
27			* `wp-blocks`: includes block type registration and related functions.
28			* `wp-element`: includes the WordPress Element abstraction for describing the structure of your blocks.
29			* `wp-i18n`: To internationalize the block's text.
30			*
31			* @since 1.0.0
32			*/
33			function enqueue_block_editor_assets() {
			0 ignored issues – show Best Practice introduced 2018-10-22 04:26 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...
34
35			//enqueue foogallery dependencies
36			wp_enqueue_script( 'masonry' );
37			foogallery_enqueue_core_gallery_template_script();
38			foogallery_enqueue_core_gallery_template_style();
39
40			$deps = array(
41			'wp-blocks',
42			'wp-i18n',
43			'wp-element',
44			'wp-components',
45			'wp-editor',
46			'foogallery-core'
47			);
48
49			$js_url = plugins_url( 'gutenberg/dist/blocks.build.js', dirname( __FILE__ ) );
50
51			// Scripts.
52			wp_enqueue_script(
53			'foogallery-block-js', // Handle.
54			$js_url, // Block.build.js: We register the block here. Built with Webpack.
55			$deps, // Dependencies, defined above.
56			// filemtime( plugin_dir_path( __DIR__ ) . 'dist/blocks.build.js' ), // Version: filemtime — Gets file modification time.
			0 ignored issues – show Unused Code Comprehensibility introduced 2018-10-16 20:26 UTC by Report Bug Copy Issue Report `39%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
57			true // Enqueue the script in the footer.
58			);
59
60			// Styles.
61			wp_enqueue_style(
62			'foogallery-block-editor-css', // Handle.
63			plugins_url( 'gutenberg/dist/blocks.editor.build.css', dirname( __FILE__ ) ), // Block editor CSS.
64			array( 'wp-edit-blocks', 'foogallery-core' ) // Dependency to include the CSS after it.
65			// filemtime( plugin_dir_path( __DIR__ ) . 'dist/blocks.editor.build.css' ) // Version: filemtime — Gets file modification time.
			0 ignored issues – show Unused Code Comprehensibility introduced 2018-10-16 20:26 UTC by Report Bug Copy Issue Report `36%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
66			);
67
68			$local_data = false;
69
70			if ( function_exists( 'wp_get_jed_locale_data' ) ) {
71			$local_data = wp_get_jed_locale_data( 'foogallery' );
72			} else if ( function_exists( 'gutenberg_get_jed_locale_data' ) ) {
73			$local_data = gutenberg_get_jed_locale_data( 'foogallery' );
74			}
75
76			if ( false !== $local_data ) {
77
78			/*
79			* Pass already loaded translations to our JavaScript.
80			*
81			* This happens _before_ our JavaScript runs, afterwards it's too late.
82			*/
83			wp_add_inline_script(
84			'foogallery-block-js',
85			'wp.i18n.setLocaleData( ' . json_encode( $local_data ) . ', "foogallery" );',
86			'before'
87			);
88			}
89			}
90
91			/**
92			* Enqueue Gutenberg block assets for both frontend + backend.
93			*
94			* `wp-blocks`: includes block type registration and related functions.
95			*
96			* @since 1.0.0
97			*/
98			function enqueue_block_assets() {
			0 ignored issues – show Best Practice introduced 2018-10-22 04:26 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...
99			// Styles.
100			wp_enqueue_style(
101			'foogallery-block-css',
102			plugins_url( 'gutenberg/dist/blocks.style.build.css', dirname( __FILE__ ) ),
103			array( 'wp-blocks' )
104			);
105			}
106
107			/**
108			* Register our block and shortcode.
109			*/
110			function php_block_init() {
			0 ignored issues – show Best Practice introduced 2018-11-02 14:27 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...
111			//get out quickly if no Gutenberg
112			if ( !function_exists( 'register_block_type' ) ) {
113			return;
114			}
115
116			// Register our block, and explicitly define the attributes we accept.
117			register_block_type(
118			'fooplugins/foogallery', array(
119			'attributes' => array(
120			'id' => array(
121			'type' => 'number',
122			'default' => 0
123			),
124			),
125			'render_callback' => array( $this, 'render_block' ),
126			));
127			}
128
129			function render_block( $attributes ) {
			0 ignored issues – show Best Practice introduced 2018-10-23 19:10 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...
130			$foogallery_id = $attributes['id'];
131			$args = array(
132			'id' => $foogallery_id
133			);
134			//create new instance of template engine
135			$engine = new FooGallery_Template_Loader();
136
137			ob_start();
138
139			$engine->render_template( $args );
140
141			$output_string = ob_get_contents();
142			ob_end_clean();
143			return !empty($output_string) ? $output_string : null;
144			}
145			}
146			}
147

fooplugins / foogallery

GitHub Access Token became invalid

Push — feature/gutenberg ( 6ad327...ccecaf )

FooGallery_Blocks::enqueue_block_editor_assets() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like