Carbon_Fields\Walker\Nav_Menu_Edit_Walker

Complexity

Total Complexity

1

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	3
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	start_el()	0	27	1

<?php

namespace Carbon_Fields\Walker;

/**
 * Walker for the administration nav menu editing.
 *
 * @uses Walker_Nav_Menu_Edit
 */
class Nav_Menu_Edit_Walker extends \Walker_Nav_Menu_Edit {

	/**
	 * Start the element output.
	 *
	 * @param string $output Passed by reference. Used to append additional content.
	 * @param object $item   Menu item data object.
	 * @param int    $depth  Depth of menu item. Used for padding.
	 * @param array  $args   An array of arguments. @see wp_nav_menu()
	 * @param int    $id     Current item ID.
	 */
	public function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0 ) {
		parent::start_el( $output, $item, $depth, $args, $id );

		$flag = '<!--CarbonFields-->';

		// Generates the HTML
		ob_start();
		do_action( 'crb_print_carbon_container_nav_menu_fields_html', $item, $output, $depth, $args, $id );
		echo $flag;

Expected next thing to be a escaping function, not '$flag'

		$fields = ob_get_clean();

		// List of possible insertion markers, this may vary between WP Core versions
		$markers = array(
			preg_quote( '<p class="field-move hide-if-no-js description description-wide">' ),
			preg_quote( '<fieldset class="field-move hide-if-no-js description description-wide">' ),
		);

		// Build the regex
		$regex = sprintf(
			'~(?<!%s)(%s)~',
			preg_quote( $flag, '~' ),
			implode( '|', $markers)

Expected 1 spaces before closing bracket; 0 found

		);

		// Injects the HTML
		$output = preg_replace( $regex, $fields . "$1", $output );

The string literal $1 does not require double quotes, as per coding-style, please use single quotes.

	}

}