LSX_Bootstrap_Navwalker::start_el() - Code Metrics - lightspeeddevelopment/lsx - Measure and Improve Code Quality continuously with Scrutinizer

LSX_Bootstrap_Navwalker::start_el() F
last analyzed 2023-05-03 02:08 UTC

↳ Parent: LSX_Bootstrap_Navwalker

Complexity

Conditions	27
Paths	> 20000

Size

Total Lines	88
Code Lines	51

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	27
eloc	51
nc	65546
nop	5
dl	0
loc	88
rs	0
c	0
b	0
f	0

How to fix Long Method Complexity

<?php
/**
 * LSX functions and definitions - Bootstrap Navigation Walker
 *
 * @package    lsx
 * @subpackage navigation
 * @category   bootstrap-navigation-walker
 */

if ( ! defined( 'ABSPATH' ) ) {
	exit;
}

if ( ! class_exists( 'Walker_Nav_Menu' ) ) {
	return;
}

if ( ! class_exists( 'LSX_Bootstrap_Navwalker' ) ) :

	/**
	 * Cleaner Bootstrap walker
	 *
	 * @package    lsx
	 * @subpackage navigation
	 * @category   bootstrap-navigation-walker
	 */
	class LSX_Bootstrap_Navwalker extends Walker_Nav_Menu {

		/**

		 * Used to append additional content.
		 *
		 * @see Walker::start_lvl()
		 * @since 3.0.0
		 *
		 * @param string $output Passed by reference. Used to append additional content.
		 * @param int    $depth Depth of page. Used for padding.
		 */
		public function start_lvl( &$output, $depth = 0, $args = array() ) {
			$indent  = str_repeat( "\t", $depth );
			$output .= "\n$indent<ul role=\"menu\" class=\" dropdown-menu\">\n";
		}

		/**
		 * Used to append additional content.
		 * @param string $item Passed by reference.

		 */
		public function filter_default_pages( &$item ) {
			return $item;
		}

		/**

		 * @see Walker::start_el()
		 * @since 3.0.0
		 *
		 * @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 int $current_page Menu item ID.

		 * @param object $args

		 */
		public function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0 ) {
			$indent = ( $depth ) ? str_repeat( "\t", $depth ) : '';

			/**
			 * If this is a default menu being called we need to fix
			 * the item object thats coming through.
			 */
			if ( ! isset( $item->title ) ) {
				return;
			}

			/**
			 * Dividers, Headers or Disabled
			 * =============================
			 * Determine whether the item is a Divider, Header, Disabled or regular
			 * menu item. To prevent errors we use the strcasecmp() function to so a
			 * comparison that is not case sensitive. The strcasecmp() function returns
			 * a 0 if the strings are equal.
			 */
			if ( 0 == strcasecmp( $item->attr_title, 'divider' ) && 1 === $depth ) {

				$output .= $indent . '<li role="presentation" class="divider">';
			} elseif ( 0 == strcasecmp( $item->title, 'divider' ) && 1 === $depth ) {

				$output .= $indent . '<li role="presentation" class="divider">';
			} elseif ( 0 == strcasecmp( $item->attr_title, 'dropdown-header' ) && 1 === $depth ) {

				$output .= $indent . '<li role="presentation" class="dropdown-header">' . esc_attr( $item->title );
			} elseif ( 0 == strcasecmp( $item->attr_title, 'disabled' ) ) {

				$output .= $indent . '<li role="presentation" class="disabled"><a href="#">' . esc_attr( $item->title ) . '</a>';
			} else {
				$class_names = '';

				$value       = '';

				$classes = empty( $item->classes ) ? array() : (array) $item->classes;
				$classes[] = 'menu-item-' . $item->ID;

				$classes = apply_filters( 'lsx_nav_menu_css_class', array_filter( $classes ), $item, $args, $depth );

				$class_names = join( ' ', $classes );

				if ( $args->has_children )
// Without braces (not recommended)
if (true)
    doSomething();

// Recommended
if (true) {
    doSomething();
}
					$class_names .= ' dropdown';

				if ( in_array( 'current-menu-item', $classes ) )
// Without braces (not recommended)
if (true)
    doSomething();

// Recommended
if (true) {
    doSomething();
}
					$class_names .= ' active';

				if ( in_array( 'current-menu-parent', $classes ) )
// Without braces (not recommended)
if (true)
    doSomething();

// Recommended
if (true) {
    doSomething();
}
					$class_names .= ' active';

				//Check if this is ment to be a "social" type menu

				$class_names = $class_names ? ' class="' . esc_attr( $class_names ) . '"' : '';

				$id = apply_filters( 'nav_menu_item_id', 'menu-item-' . $item->ID, $item, $args );
				$id = $id ? ' id="' . esc_attr( $id ) . '"' : '';

				$output .= $indent . '<li' . $id . $value . $class_names . '>';

				$atts = array();
				$atts['title']  = ! empty( $item->attr_title ) ? $item->attr_title : $item->title;
				$atts['target'] = ! empty( $item->target ) ? $item->target : '';
				$atts['rel']    = ! empty( $item->xfn ) ? $item->xfn : '';

				// If item has_children add atts to a.
				if ( $args->has_children ) {
					$atts['href']          = ! empty( $item->url ) ? $item->url : '';
					$atts['data-toggle']   = 'dropdown';
					$atts['class']         = 'dropdown-toggle';
					$atts['aria-haspopup'] = 'true';
				} else {
					$atts['href'] = ! empty( $item->url ) ? $item->url : '';
				}

				$atts = apply_filters( 'nav_menu_link_attributes', $atts, $item, $args );

				$attributes = '';
				foreach ( $atts as $attr => $value ) {
					if ( ! empty( $value ) ) {
						$value = ( 'href' === $attr ) ? esc_url( $value ) : esc_attr( $value );
						$attributes .= ' ' . $attr . '="' . $value . '"';
					}
				}

				$item_output = $args->before;

				$item_output .= '<a' . $attributes . '>';
				$item_output .= $args->link_before . apply_filters( 'the_title', $item->title, $item->ID ) . $args->link_after;
				$item_output .= ( $args->has_children && 0 === $depth ) ? ' <span class="caret"></span></a>' : '</a>';
				$item_output .= $args->after;

				$output .= apply_filters( 'walker_nav_menu_start_el', $item_output, $item, $depth, $args );
			}
		}

		/**
		 * Traverse elements to create list from elements.
		 *
		 * Display one element if the element doesn't have any children otherwise,
		 * display the element and its children. Will only traverse up to the max
		 * depth and no ignore elements under that depth.
		 *
		 * This method shouldn't be called directly, use the walk() method instead.
		 *
		 * @see Walker::start_el()
		 * @since 2.5.0
		 *
		 * @param object $element Data object

		 * @param array $children_elements List of elements to continue traversing.

		 * @param int $max_depth Max depth to traverse.

		 * @param int $depth Depth of current element.

		 * @param array $args

		 * @param string $output Passed by reference. Used to append additional content.
		 * @return null Null on failure with no changes to parameters.
		 */
		public function display_element( $element, &$children_elements, $max_depth, $depth, $args, &$output ) {
			if ( ! $element ) {

				return;
			}

			$id_field = $this->db_fields['id'];

			if ( is_object( $args[0] ) ) {
				$args[0]->has_children = ! empty( $children_elements[ $element->$id_field ] );
			}

			parent::display_element( $element, $children_elements, $max_depth, $depth, $args, $output );
		}

		/**
		 * Menu Fallback
		 * =============
		 * If this function is assigned to the wp_nav_menu's fallback_cb variable
		 * and a manu has not been assigned to the theme location in the WordPress
		 * menu manager the function with display nothing to a non-logged in user,
		 * and will add a link to the WordPress menu manager if logged in as an admin.
		 *
		 * @param array $args passed from the wp_nav_menu function.
		 *
		 */

		public static function fallback( $args ) {
			if ( current_user_can( 'manage_options' ) ) {
				$fb_output = null;

				if ( $args['container'] ) {
					$fb_output = '<' . $args['container'];

					if ( $args['container_id'] ) {
						$fb_output .= ' id="' . $args['container_id'] . '"';
					}

					if ( $args['container_class'] ) {
						$fb_output .= ' class="' . $args['container_class'] . '"';
					}

					$fb_output .= '>';
				}

				$fb_output .= '<ul';

				if ( $args['menu_id'] ) {
					$fb_output .= ' id="' . $args['menu_id'] . '"';
				}

				if ( $args['menu_class'] ) {
					$fb_output .= ' class="' . $args['menu_class'] . '"';
				}

				$fb_output .= '>';
				$fb_output .= '<li><a href="' . esc_url( admin_url( 'nav-menus.php' ) ) . '">' . esc_html__( 'Add a menu', 'lsx' ) . '</a></li>';
				$fb_output .= '</ul>';

				if ( $args['container'] ) {
					$fb_output .= '</' . $args['container'] . '>';
				}

				echo wp_kses_post( $fb_output );
			}
		}

	}

endif;


1			<?php
2			/**
3			* LSX functions and definitions - Bootstrap Navigation Walker
4			*
5			* @package lsx
6			* @subpackage navigation
7			* @category bootstrap-navigation-walker
8			*/
9
10			if ( ! defined( 'ABSPATH' ) ) {
11			exit;
12			}
13
14			if ( ! class_exists( 'Walker_Nav_Menu' ) ) {
15			return;
16			}
17
18			if ( ! class_exists( 'LSX_Bootstrap_Navwalker' ) ) :
19
20			/**
21			* Cleaner Bootstrap walker
22			*
23			* @package lsx
24			* @subpackage navigation
25			* @category bootstrap-navigation-walker
26			*/
27			class LSX_Bootstrap_Navwalker extends Walker_Nav_Menu {
28
29			/**
			0 ignored issues – show Coding Style Documentation introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Doc comment for parameter "$args" missing Loading history...
30			* Used to append additional content.
31			*
32			* @see Walker::start_lvl()
33			* @since 3.0.0
34			*
35			* @param string $output Passed by reference. Used to append additional content.
36			* @param int $depth Depth of page. Used for padding.
37			*/
38			public function start_lvl( &$output, $depth = 0, $args = array() ) {
39			$indent = str_repeat( "\t", $depth );
40			$output .= "\n$indent<ul role=\"menu\" class=\" dropdown-menu\">\n";
41			}
42
43			/**
44			* Used to append additional content.
45			* @param string $item Passed by reference.
			0 ignored issues – show Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report There must be exactly one blank line before the tags in a doc comment Loading history...
46			*/
47			public function filter_default_pages( &$item ) {
48			return $item;
49			}
50
51			/**
			0 ignored issues – show Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Missing short description in doc comment Loading history... Coding Style Documentation introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Doc comment for parameter "$id" missing Loading history...
52			* @see Walker::start_el()
53			* @since 3.0.0
54			*
55			* @param string $output Passed by reference. Used to append additional content.
56			* @param object $item Menu item data object.
57			* @param int $depth Depth of menu item. Used for padding.
			0 ignored issues – show Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Expected 4 spaces after parameter type; 1 found Loading history...
58			* @param int $current_page Menu item ID.
			0 ignored issues – show Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Expected 4 spaces after parameter type; 1 found Loading history... Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Doc comment for parameter $current_page does not match actual variable name $args Loading history...
59			* @param object $args
			0 ignored issues – show Coding Style Documentation introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history... Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Doc comment for parameter $args does not match actual variable name $id Loading history...
60			*/
61			public function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0 ) {
62			$indent = ( $depth ) ? str_repeat( "\t", $depth ) : '';
63
64			/**
65			* If this is a default menu being called we need to fix
66			* the item object thats coming through.
67			*/
68			if ( ! isset( $item->title ) ) {
69			return;
70			}
71
72			/**
73			* Dividers, Headers or Disabled
74			* =============================
75			* Determine whether the item is a Divider, Header, Disabled or regular
76			* menu item. To prevent errors we use the strcasecmp() function to so a
77			* comparison that is not case sensitive. The strcasecmp() function returns
78			* a 0 if the strings are equal.
79			*/
80			if ( 0 == strcasecmp( $item->attr_title, 'divider' ) && 1 === $depth ) {
			0 ignored issues – show introduced 2020-08-30 05:28 UTC by Report Bug Copy Issue Report Found: ==. Use strict comparisons (=== or !==). Loading history...
81			$output .= $indent . '<li role="presentation" class="divider">';
82			} elseif ( 0 == strcasecmp( $item->title, 'divider' ) && 1 === $depth ) {
			0 ignored issues – show introduced 2020-08-30 05:28 UTC by Report Bug Copy Issue Report Found: ==. Use strict comparisons (=== or !==). Loading history...
83			$output .= $indent . '<li role="presentation" class="divider">';
84			} elseif ( 0 == strcasecmp( $item->attr_title, 'dropdown-header' ) && 1 === $depth ) {
			0 ignored issues – show introduced 2020-08-30 05:28 UTC by Report Bug Copy Issue Report Found: ==. Use strict comparisons (=== or !==). Loading history...
85			$output .= $indent . '<li role="presentation" class="dropdown-header">' . esc_attr( $item->title );
86			} elseif ( 0 == strcasecmp( $item->attr_title, 'disabled' ) ) {
			0 ignored issues – show introduced 2020-08-30 05:28 UTC by Report Bug Copy Issue Report Found: ==. Use strict comparisons (=== or !==). Loading history...
87			$output .= $indent . '<li role="presentation" class="disabled"><a href="#">' . esc_attr( $item->title ) . '</a>';
88			} else {
89			$class_names = '';
			0 ignored issues – show Unused Code introduced 2023-04-13 06:59 UTC by Report Bug Copy Issue Report The assignment to `$class_names` is dead and can be removed. Loading history...
90			$value = '';
91
92			$classes = empty( $item->classes ) ? array() : (array) $item->classes;
93			$classes[] = 'menu-item-' . $item->ID;
94
95			$classes = apply_filters( 'lsx_nav_menu_css_class', array_filter( $classes ), $item, $args, $depth );
96
97			$class_names = join( ' ', $classes );
98
99			if ( $args->has_children )
			0 ignored issues – show Coding Style Best Practice introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report It is generally a best practice to always use braces with control structures. Adding braces to control structures avoids accidental mistakes as your code changes: // Without braces (not recommended) if (true) doSomething(); // Recommended if (true) { doSomething(); } Loading history...
100			$class_names .= ' dropdown';
101
102			if ( in_array( 'current-menu-item', $classes ) )
			0 ignored issues – show Coding Style Best Practice introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report It is generally a best practice to always use braces with control structures. Adding braces to control structures avoids accidental mistakes as your code changes: // Without braces (not recommended) if (true) doSomething(); // Recommended if (true) { doSomething(); } Loading history... introduced 2020-08-30 05:28 UTC by Report Bug Copy Issue Report Not using strict comparison for in_array; supply true for third argument. Loading history...
103			$class_names .= ' active';
104
105			if ( in_array( 'current-menu-parent', $classes ) )
			0 ignored issues – show Coding Style Best Practice introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report It is generally a best practice to always use braces with control structures. Adding braces to control structures avoids accidental mistakes as your code changes: // Without braces (not recommended) if (true) doSomething(); // Recommended if (true) { doSomething(); } Loading history... introduced 2020-08-30 05:28 UTC by Report Bug Copy Issue Report Not using strict comparison for in_array; supply true for third argument. Loading history...
106			$class_names .= ' active';
107
108			//Check if this is ment to be a "social" type menu
			0 ignored issues – show Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report No space found before comment text; expected "// Check if this is ment to be a "social" type menu" but found "//Check if this is ment to be a "social" type menu" Loading history... Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Inline comments must end in full-stops, exclamation marks, or question marks Loading history...
109			$class_names = $class_names ? ' class="' . esc_attr( $class_names ) . '"' : '';
110
111			$id = apply_filters( 'nav_menu_item_id', 'menu-item-' . $item->ID, $item, $args );
112			$id = $id ? ' id="' . esc_attr( $id ) . '"' : '';
113
114			$output .= $indent . '<li' . $id . $value . $class_names . '>';
115
116			$atts = array();
117			$atts['title'] = ! empty( $item->attr_title ) ? $item->attr_title : $item->title;
118			$atts['target'] = ! empty( $item->target ) ? $item->target : '';
119			$atts['rel'] = ! empty( $item->xfn ) ? $item->xfn : '';
120
121			// If item has_children add atts to a.
122			if ( $args->has_children ) {
123			$atts['href'] = ! empty( $item->url ) ? $item->url : '';
124			$atts['data-toggle'] = 'dropdown';
125			$atts['class'] = 'dropdown-toggle';
126			$atts['aria-haspopup'] = 'true';
127			} else {
128			$atts['href'] = ! empty( $item->url ) ? $item->url : '';
129			}
130
131			$atts = apply_filters( 'nav_menu_link_attributes', $atts, $item, $args );
132
133			$attributes = '';
134			foreach ( $atts as $attr => $value ) {
135			if ( ! empty( $value ) ) {
136			$value = ( 'href' === $attr ) ? esc_url( $value ) : esc_attr( $value );
137			$attributes .= ' ' . $attr . '="' . $value . '"';
138			}
139			}
140
141			$item_output = $args->before;
142
143			$item_output .= '<a' . $attributes . '>';
144			$item_output .= $args->link_before . apply_filters( 'the_title', $item->title, $item->ID ) . $args->link_after;
145			$item_output .= ( $args->has_children && 0 === $depth ) ? ' <span class="caret"></span></a>' : '</a>';
146			$item_output .= $args->after;
147
148			$output .= apply_filters( 'walker_nav_menu_start_el', $item_output, $item, $depth, $args );
149			}
150			}
151
152			/**
153			* Traverse elements to create list from elements.
154			*
155			* Display one element if the element doesn't have any children otherwise,
156			* display the element and its children. Will only traverse up to the max
157			* depth and no ignore elements under that depth.
158			*
159			* This method shouldn't be called directly, use the walk() method instead.
160			*
161			* @see Walker::start_el()
162			* @since 2.5.0
163			*
164			* @param object $element Data object
			0 ignored issues – show Coding Style Documentation introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Parameter comment must end with a full stop Loading history...
165			* @param array $children_elements List of elements to continue traversing.
			0 ignored issues – show Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Expected 2 spaces after parameter type; 1 found Loading history...
166			* @param int $max_depth Max depth to traverse.
			0 ignored issues – show Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Expected 4 spaces after parameter type; 1 found Loading history...
167			* @param int $depth Depth of current element.
			0 ignored issues – show Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Expected 4 spaces after parameter type; 1 found Loading history...
168			* @param array $args
			0 ignored issues – show Coding Style Documentation introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history... Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Expected 2 spaces after parameter type; 1 found Loading history...
169			* @param string $output Passed by reference. Used to append additional content.
170			* @return null Null on failure with no changes to parameters.
171			*/
172			public function display_element( $element, &$children_elements, $max_depth, $depth, $args, &$output ) {
173			if ( ! $element ) {
			0 ignored issues – show introduced 2020-08-14 10:27 UTC by Report Bug Copy Issue Report `$element` is of type `object`, thus it always evaluated to `true`. Loading history...
174			return;
175			}
176
177			$id_field = $this->db_fields['id'];
178
179			if ( is_object( $args[0] ) ) {
180			$args[0]->has_children = ! empty( $children_elements[ $element->$id_field ] );
181			}
182
183			parent::display_element( $element, $children_elements, $max_depth, $depth, $args, $output );
184			}
185
186			/**
187			* Menu Fallback
188			* =============
189			* If this function is assigned to the wp_nav_menu's fallback_cb variable
190			* and a manu has not been assigned to the theme location in the WordPress
191			* menu manager the function with display nothing to a non-logged in user,
192			* and will add a link to the WordPress menu manager if logged in as an admin.
193			*
194			* @param array $args passed from the wp_nav_menu function.
195			*
196			*/
			0 ignored issues – show Coding Style introduced 2020-08-20 08:08 UTC by Report Bug Copy Issue Report Additional blank lines found at end of doc comment Loading history...
197			public static function fallback( $args ) {
198			if ( current_user_can( 'manage_options' ) ) {
199			$fb_output = null;
200
201			if ( $args['container'] ) {
202			$fb_output = '<' . $args['container'];
203
204			if ( $args['container_id'] ) {
205			$fb_output .= ' id="' . $args['container_id'] . '"';
206			}
207
208			if ( $args['container_class'] ) {
209			$fb_output .= ' class="' . $args['container_class'] . '"';
210			}
211
212			$fb_output .= '>';
213			}
214
215			$fb_output .= '<ul';
216
217			if ( $args['menu_id'] ) {
218			$fb_output .= ' id="' . $args['menu_id'] . '"';
219			}
220
221			if ( $args['menu_class'] ) {
222			$fb_output .= ' class="' . $args['menu_class'] . '"';
223			}
224
225			$fb_output .= '>';
226			$fb_output .= '<li><a href="' . esc_url( admin_url( 'nav-menus.php' ) ) . '">' . esc_html__( 'Add a menu', 'lsx' ) . '</a></li>';
227			$fb_output .= '</ul>';
228
229			if ( $args['container'] ) {
230			$fb_output .= '</' . $args['container'] . '>';
231			}
232
233			echo wp_kses_post( $fb_output );
234			}
235			}
236
237			}
238
239			endif;
240

lightspeeddevelopment / lsx

LSX_Bootstrap_Navwalker::start_el() F last analyzed 2023-05-03 02:08 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like

LSX_Bootstrap_Navwalker::start_el() F
last analyzed 2023-05-03 02:08 UTC