Inspection of "* Fix - Fixes duplicated breadcrumb" - lightspeeddevelopment/lsx - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( d53b10...8df284 )

by Fernando

created 2017-10-10 15:13 UTC

Status

Spacing +98 added lines, -98 removed lines patch added patch discarded remove patch

@@ -6,11 +6,11 @@  discard block
 block discarded – undo
  * @subpackage layout
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! function_exists( 'lsx_layout_selector' ) ) :
+if ( ! function_exists('lsx_layout_selector')) :
 
 	/**
 	 * Layout selector.
@@ -18,14 +18,14 @@  discard block
 block discarded – undo
 	 * @package    lsx
 	 * @subpackage layout
 	 */
-	function lsx_layout_selector( $class, $area = 'site' ) {
+	function lsx_layout_selector($class, $area = 'site') {
 		$return_class = '';
-		$layout       = get_theme_mod( 'lsx_layout', '2cr' );
-		$layout       = apply_filters( 'lsx_layout', $layout );
+		$layout       = get_theme_mod('lsx_layout', '2cr');
+		$layout       = apply_filters('lsx_layout', $layout);
 		$default_size = 'sm';
-		$size         = apply_filters( 'lsx_bootstrap_column_size', $default_size );
+		$size         = apply_filters('lsx_bootstrap_column_size', $default_size);
 
-		switch ( $layout ) {
+		switch ($layout) {
 			case '1c':
 				$main_class    = 'col-' . $size . '-12';
 				$sidebar_class = 'col-' . $size . '-12';
@@ -44,12 +44,12 @@  discard block
 block discarded – undo
 				break;
 		}
 
-		if ( 'main' === $class ) {
-			$return_class = apply_filters( 'lsx_layout_selector', $main_class, $class, $layout, $size );
+		if ('main' === $class) {
+			$return_class = apply_filters('lsx_layout_selector', $main_class, $class, $layout, $size);
 		}
 
-		if ( 'sidebar' === $class ) {
-			$return_class = apply_filters( 'lsx_layout_selector', $sidebar_class, $class, $layout, $size );
+		if ('sidebar' === $class) {
+			$return_class = apply_filters('lsx_layout_selector', $sidebar_class, $class, $layout, $size);
 		}
 
 		return $return_class;
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
 
 endif;
 
-if ( ! function_exists( 'lsx_main_class' ) ) :
+if ( ! function_exists('lsx_main_class')) :
 
 	/**
 	 * .main classes.
@@ -66,12 +66,12 @@  discard block
 block discarded – undo
 	 * @subpackage layout
 	 */
 	function lsx_main_class() {
-		return lsx_layout_selector( 'main' );
+		return lsx_layout_selector('main');
 	}
 
 endif;
 
-if ( ! function_exists( 'lsx_sidebar_class' ) ) :
+if ( ! function_exists('lsx_sidebar_class')) :
 
 	/**
 	 * .sidebar classes.
@@ -80,12 +80,12 @@  discard block
 block discarded – undo
 	 * @subpackage layout
 	 */
 	function lsx_sidebar_class() {
-		return lsx_layout_selector( 'sidebar' );
+		return lsx_layout_selector('sidebar');
 	}
 
 endif;
 
-if ( ! function_exists( 'lsx_header_classes' ) ) :
+if ( ! function_exists('lsx_header_classes')) :
 
 	/**
 	 * Output the classes for the header.
@@ -93,19 +93,19 @@  discard block
 block discarded – undo
 	 * @package    lsx
 	 * @subpackage layout
 	 */
-	function lsx_header_classes( $additional = false ) {
+	function lsx_header_classes($additional = false) {
 		$classes = 'banner navbar navbar-default';
 
-		if ( false !== $additional ) {
+		if (false !== $additional) {
 			$classes .= ' ' . $additional;
 		}
 
-		echo esc_attr( $classes );
+		echo esc_attr($classes);
 	}
 
 endif;
 
-if ( ! function_exists( 'lsx_top_menu_classes' ) ) :
+if ( ! function_exists('lsx_top_menu_classes')) :
 
 	/**
 	 * Output the classes for the top-menu.
@@ -113,19 +113,19 @@  discard block
 block discarded – undo
 	 * @package    lsx
 	 * @subpackage layout
 	 */
-	function lsx_top_menu_classes( $additional = false ) {
+	function lsx_top_menu_classes($additional = false) {
 		$classes = 'top-menu-default';
 
-		if ( false !== $additional ) {
+		if (false !== $additional) {
 			$classes .= ' ' . $additional;
 		}
 
-		echo esc_attr( $classes );
+		echo esc_attr($classes);
 	}
 
 endif;
 
-if ( ! function_exists( 'lsx_add_footer_sidebar_area' ) ) :
+if ( ! function_exists('lsx_add_footer_sidebar_area')) :
 
 	/**
 	 * Output the Footer CTA and/pr Footer Widgets.
@@ -134,34 +134,34 @@  discard block
 block discarded – undo
 	 * @subpackage layout
 	 */
 	function lsx_add_footer_sidebar_area() {
-		if ( is_active_sidebar( 'sidebar-footer-cta' ) ) : ?>
+		if (is_active_sidebar('sidebar-footer-cta')) : ?>
 			<div id="footer-cta">
 				<div class="container">
 					<div class="lsx-full-width">
 						<div class="lsx-hero-unit">
-							<?php dynamic_sidebar( 'sidebar-footer-cta' ); ?>
+							<?php dynamic_sidebar('sidebar-footer-cta'); ?>
 						</div>
 					</div>
 				</div>
 			</div>
 		<?php endif; ?>
 
-		<?php if ( is_active_sidebar( 'sidebar-footer' ) ) : ?>
+		<?php if (is_active_sidebar('sidebar-footer')) : ?>
 			<div id="footer-widgets">
 				<div class="container">
 					<div class="row">
-						<?php dynamic_sidebar( 'sidebar-footer' ); ?>
+						<?php dynamic_sidebar('sidebar-footer'); ?>
 					</div>
 				</div>
 			</div>
 		<?php endif;
 	}
 
-	add_action( 'lsx_footer_before', 'lsx_add_footer_sidebar_area' );
+	add_action('lsx_footer_before', 'lsx_add_footer_sidebar_area');
 
 endif;
 
-if ( ! function_exists( 'lsx_global_header' ) ) :
+if ( ! function_exists('lsx_global_header')) :
 
 	/**
 	 * Displays the global header.
@@ -174,21 +174,21 @@  discard block
 block discarded – undo
 		// 	return;
 		// }
 
-		$show_on_front  = get_option( 'show_on_front' );
+		$show_on_front  = get_option('show_on_front');
 		$queried_object = get_queried_object();
 		$default_size   = 'sm';
-		$size           = apply_filters( 'lsx_bootstrap_column_size', $default_size );
+		$size           = apply_filters('lsx_bootstrap_column_size', $default_size);
 
-		if ( true === apply_filters( 'lsx_global_header_disable', false ) ) :
+		if (true === apply_filters('lsx_global_header_disable', false)) :
 			// Display only the breadcrumbs
 			?>
-			<div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
+			<div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
 				<?php lsx_global_header_inner_bottom(); ?>
 			</div>
 			<?php
-		elseif ( is_page() && ( 'page' !== $show_on_front || ! is_front_page() ) ) :
+		elseif (is_page() && ('page' !== $show_on_front || ! is_front_page())) :
 			?>
-			<div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
+			<div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
 				<header class="archive-header">
 					<h1 class="archive-title"><?php the_title(); ?></h1>
 				</header>
@@ -196,9 +196,9 @@  discard block
 block discarded – undo
 				<?php lsx_global_header_inner_bottom(); ?>
 			</div>
 			<?php
-		elseif ( is_single() && ! is_singular( 'post' ) ) :
+		elseif (is_single() && ! is_singular('post')) :
 			?>
-			<div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
+			<div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
 				<header class="archive-header">
 					<h1 class="archive-title"><?php the_title(); ?></h1>
 				</header>
@@ -206,15 +206,15 @@  discard block
 block discarded – undo
 				<?php lsx_global_header_inner_bottom(); ?>
 			</div>
 			<?php
-		elseif ( is_search() ) :
+		elseif (is_search()) :
 			?>
-			<div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
+			<div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
 				<header class="archive-header">
 					<h1 class="archive-title">
 						<?php
 							printf(
 								/* Translators: %s: search term/query */
-								esc_html__( 'Search Results for: %s', 'lsx' ),
+								esc_html__('Search Results for: %s', 'lsx'),
 								'<span>' . get_search_query() . '</span>'
 							);
 						?>
@@ -224,12 +224,12 @@  discard block
 block discarded – undo
 				<?php lsx_global_header_inner_bottom(); ?>
 			</div>
 			<?php
-		elseif ( is_author() ) :
+		elseif (is_author()) :
 			$author = get_the_author();
-			$author_avatar = get_avatar( get_the_author_meta( 'ID' ), 256 );
+			$author_avatar = get_avatar(get_the_author_meta('ID'), 256);
 			$author_bio = get_the_archive_description();
 			?>
-			<div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
+			<div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
 				<header class="archive-header">
 					<h1 class="archive-title"><?php the_archive_title(); ?></h1>
 				</header>
@@ -237,13 +237,13 @@  discard block
 block discarded – undo
 				<?php lsx_global_header_inner_bottom(); ?>
 			</div>
 			<?php
-		elseif ( is_archive() ) :
+		elseif (is_archive()) :
 			?>
-			<div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
+			<div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
 				<header class="archive-header">
 					<h1 class="archive-title">
-						<?php if ( has_post_format() && ! is_category() && ! is_tag() && ! is_date() && ! is_tax( 'post_format' ) ) { ?>
-							<?php the_archive_title( esc_html__( 'Type:', 'lsx' ) ); ?>
+						<?php if (has_post_format() && ! is_category() && ! is_tag() && ! is_date() && ! is_tax('post_format')) { ?>
+							<?php the_archive_title(esc_html__('Type:', 'lsx')); ?>
 						<?php } else { ?>
 							<?php the_archive_title(); ?>
 						<?php } ?>
@@ -255,31 +255,31 @@  discard block
 block discarded – undo
 				<?php lsx_global_header_inner_bottom(); ?>
 			</div>
 			<?php
-		elseif ( 'page' === $show_on_front && (int) get_option( 'page_for_posts' ) === $queried_object->ID ) :
+		elseif ('page' === $show_on_front && (int) get_option('page_for_posts') === $queried_object->ID) :
 			?>
-			<div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
+			<div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
 				<header class="archive-header">
-					<h1 class="archive-title"><?php esc_html_e( 'Blog', 'lsx' ); ?></h1>
+					<h1 class="archive-title"><?php esc_html_e('Blog', 'lsx'); ?></h1>
 				</header>
 
 				<?php lsx_global_header_inner_bottom(); ?>
 			</div>
 			<?php
-		elseif ( ! is_singular( 'post' ) ) :
+		elseif ( ! is_singular('post')) :
 			// Display only the breadcrumbs
 			?>
-			<div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
+			<div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
 				<?php lsx_global_header_inner_bottom(); ?>
 			</div>
 			<?php
 		endif;
 	}
 
-	add_action( 'lsx_content_wrap_before', 'lsx_global_header' );
+	add_action('lsx_content_wrap_before', 'lsx_global_header');
 
 endif;
 
-if ( ! function_exists( 'lsx_author_extra_info' ) ) :
+if ( ! function_exists('lsx_author_extra_info')) :
 
 	/**
 	 * Displays the author extra info.
@@ -289,58 +289,58 @@  discard block
 block discarded – undo
 	 */
 	function lsx_author_extra_info() {
 		$default_size   = 'sm';
-		$size           = apply_filters( 'lsx_bootstrap_column_size', $default_size );
+		$size           = apply_filters('lsx_bootstrap_column_size', $default_size);
 
-		if ( is_author() ) :
-			$author_id         = get_the_author_meta( 'ID' );
+		if (is_author()) :
+			$author_id         = get_the_author_meta('ID');
 			$author            = get_the_author();
-			$author_avatar     = get_avatar( $author_id, 256 );
+			$author_avatar     = get_avatar($author_id, 256);
 			$author_bio        = get_the_archive_description();
-			$author_url        = get_the_author_meta( 'url', $author_id );
-			$author_email      = get_the_author_meta( 'email', $author_id );
-			$author_facebook   = get_the_author_meta( 'facebook', $author_id );
-			$author_twitter    = get_the_author_meta( 'twitter', $author_id );
-			$author_googleplus = get_the_author_meta( 'googleplus', $author_id );
+			$author_url        = get_the_author_meta('url', $author_id);
+			$author_email      = get_the_author_meta('email', $author_id);
+			$author_facebook   = get_the_author_meta('facebook', $author_id);
+			$author_twitter    = get_the_author_meta('twitter', $author_id);
+			$author_googleplus = get_the_author_meta('googleplus', $author_id);
 			?>
-			<div class="col-<?php echo esc_attr( $size ); ?>-12">
+			<div class="col-<?php echo esc_attr($size); ?>-12">
 				<div class="archive-author-data">
-					<figure class="archive-author-avatar"><?php echo wp_kses_post( $author_avatar ); ?></figure>
+					<figure class="archive-author-avatar"><?php echo wp_kses_post($author_avatar); ?></figure>
 
-					<?php if ( ! empty( $author_url ) || ! empty( $author_email ) || ! empty( $author_facebook ) || ! empty( $author_twitter ) || ! empty( $author_googleplus ) ) : ?>
+					<?php if ( ! empty($author_url) || ! empty($author_email) || ! empty($author_facebook) || ! empty($author_twitter) || ! empty($author_googleplus)) : ?>
 						<div class="archive-author-social-links">
-							<?php if ( ! empty( $author_url ) ) : ?>
-								<a href="<?php echo esc_url( $author_url ); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-url"><i class="fa fa-link" aria-hidden="true"></i></a>
+							<?php if ( ! empty($author_url)) : ?>
+								<a href="<?php echo esc_url($author_url); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-url"><i class="fa fa-link" aria-hidden="true"></i></a>
 							<?php endif; ?>
 
-							<?php if ( ! empty( $author_email ) ) : ?>
-								<a href="mailto:<?php echo esc_attr( $author_email ); ?>" class="archive-author-social-link archive-author-social-link-email"><i class="fa fa-envelope" aria-hidden="true"></i></a>
+							<?php if ( ! empty($author_email)) : ?>
+								<a href="mailto:<?php echo esc_attr($author_email); ?>" class="archive-author-social-link archive-author-social-link-email"><i class="fa fa-envelope" aria-hidden="true"></i></a>
 							<?php endif; ?>
 
-							<?php if ( ! empty( $author_facebook ) ) : ?>
-								<a href="<?php echo esc_url( $author_facebook ); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-facebook"><i class="fa fa-facebook" aria-hidden="true"></i></a>
+							<?php if ( ! empty($author_facebook)) : ?>
+								<a href="<?php echo esc_url($author_facebook); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-facebook"><i class="fa fa-facebook" aria-hidden="true"></i></a>
 							<?php endif; ?>
 
-							<?php if ( ! empty( $author_twitter ) ) : ?>
-								<a href="https://twitter.com/<?php echo esc_attr( $author_twitter ); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-twitter"><i class="fa fa-twitter" aria-hidden="true"></i></a>
+							<?php if ( ! empty($author_twitter)) : ?>
+								<a href="https://twitter.com/<?php echo esc_attr($author_twitter); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-twitter"><i class="fa fa-twitter" aria-hidden="true"></i></a>
 							<?php endif; ?>
 
-							<?php if ( ! empty( $author_googleplus ) ) : ?>
-								<a href="<?php echo esc_url( $author_googleplus ); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-googleplus"><i class="fa fa-google-plus" aria-hidden="true"></i></a>
+							<?php if ( ! empty($author_googleplus)) : ?>
+								<a href="<?php echo esc_url($author_googleplus); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-googleplus"><i class="fa fa-google-plus" aria-hidden="true"></i></a>
 							<?php endif; ?>
 						</div>
 					<?php endif; ?>
 
-					<?php if ( ! empty( $author_bio ) ) : ?>
-						<h2 class="archive-author-title text-center"><?php esc_html_e( 'About the author', 'lsx' ); ?></h2>
-						<p class="archive-author-bio"><?php echo wp_kses_post( $author_bio ); ?></p>
+					<?php if ( ! empty($author_bio)) : ?>
+						<h2 class="archive-author-title text-center"><?php esc_html_e('About the author', 'lsx'); ?></h2>
+						<p class="archive-author-bio"><?php echo wp_kses_post($author_bio); ?></p>
 					<?php endif; ?>
 
 					<h2 class="archive-author-posts text-center">
 						<?php
 							printf(
 								/* Translators: %s: author name */
-								esc_html__( 'All posts by %s', 'lsx' ),
-								esc_html( $author )
+								esc_html__('All posts by %s', 'lsx'),
+								esc_html($author)
 							);
 						?>
 					</h2>
@@ -350,11 +350,11 @@  discard block
 block discarded – undo
 		endif;
 	}
 
-	add_action( 'lsx_content_wrap_before', 'lsx_author_extra_info', 11 );
+	add_action('lsx_content_wrap_before', 'lsx_author_extra_info', 11);
 
 endif;
 
-if ( ! function_exists( 'lsx_post_header' ) ) :
+if ( ! function_exists('lsx_post_header')) :
 
 	/**
 	 * Displays the post header.
@@ -364,21 +364,21 @@  discard block
 block discarded – undo
 	 */
 	function lsx_post_header() {
 		$default_size  = 'sm';
-		$size          = apply_filters( 'lsx_bootstrap_column_size', $default_size );
+		$size          = apply_filters('lsx_bootstrap_column_size', $default_size);
 
-		if ( is_singular( 'post' ) ) :
+		if (is_singular('post')) :
 			$format = get_post_format();
 
-			if ( false === $format ) {
+			if (false === $format) {
 				$format = 'standard';
 			}
 
-			$format = lsx_translate_format_to_fontawesome( $format );
+			$format = lsx_translate_format_to_fontawesome($format);
 			?>
-			<div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
+			<div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
 				<header class="archive-header">
 					<h1 class="archive-title">
-						<i class="format-link fa fa-<?php echo esc_attr( $format ); ?>"></i>
+						<i class="format-link fa fa-<?php echo esc_attr($format); ?>"></i>
 						<span><?php the_title(); ?></span>
 					</h1>
 				</header>
@@ -387,11 +387,11 @@  discard block
 block discarded – undo
 		endif;
 	}
 
-	add_action( 'lsx_entry_top', 'lsx_post_header' );
+	add_action('lsx_entry_top', 'lsx_post_header');
 
 endif;
 
-if ( ! function_exists( 'lsx_add_viewport_meta_tag' ) ) :
+if ( ! function_exists('lsx_add_viewport_meta_tag')) :
 
 	/**
 	 * Add Viewport Meta Tag to head.
@@ -405,11 +405,11 @@  discard block
 block discarded – undo
 		<?php
 	}
 
-	add_action( 'wp_head', 'lsx_add_viewport_meta_tag' );
+	add_action('wp_head', 'lsx_add_viewport_meta_tag');
 
 endif;
 
-if ( ! function_exists( 'lsx_header_search_form' ) ) :
+if ( ! function_exists('lsx_header_search_form')) :
 
 	/**
 	 * Add a search form to just above the nav menu.
@@ -418,13 +418,13 @@  discard block
 block discarded – undo
 	 * @subpackage layout
 	 */
 	function lsx_header_search_form() {
-		$search_form = get_theme_mod( 'lsx_header_search', false );
+		$search_form = get_theme_mod('lsx_header_search', false);
 
-		if ( false !== $search_form || is_customize_preview() ) {
-			get_search_form( true );
+		if (false !== $search_form || is_customize_preview()) {
+			get_search_form(true);
 		}
 	}
 
-	add_action( 'lsx_nav_before', 'lsx_header_search_form', 0 );
+	add_action('lsx_nav_before', 'lsx_header_search_form', 0);
 
 endif;

Please login to merge, or discard this patch.

		@@ -6,11 +6,11 @@ discard block
		block discarded – undo
6	6	* @subpackage layout
7	7	*/
8	8
9		-if ( ! defined( 'ABSPATH' ) ) {
	9	+if ( ! defined('ABSPATH')) {
10	10	exit;
11	11	}
12	12
13		-if ( ! function_exists( 'lsx_layout_selector' ) ) :
	13	+if ( ! function_exists('lsx_layout_selector')) :
14	14
15	15	/**
16	16	* Layout selector.
		@@ -18,14 +18,14 @@ discard block
		block discarded – undo
18	18	* @package lsx
19	19	* @subpackage layout
20	20	*/
21		- function lsx_layout_selector( $class, $area = 'site' ) {
	21	+ function lsx_layout_selector($class, $area = 'site') {
22	22	$return_class = '';
23		- $layout = get_theme_mod( 'lsx_layout', '2cr' );
24		- $layout = apply_filters( 'lsx_layout', $layout );
	23	+ $layout = get_theme_mod('lsx_layout', '2cr');
	24	+ $layout = apply_filters('lsx_layout', $layout);
25	25	$default_size = 'sm';
26		- $size = apply_filters( 'lsx_bootstrap_column_size', $default_size );
	26	+ $size = apply_filters('lsx_bootstrap_column_size', $default_size);
27	27
28		- switch ( $layout ) {
	28	+ switch ($layout) {
29	29	case '1c':
30	30	$main_class = 'col-' . $size . '-12';
31	31	$sidebar_class = 'col-' . $size . '-12';
		@@ -44,12 +44,12 @@ discard block
		block discarded – undo
44	44	break;
45	45	}
46	46
47		- if ( 'main' === $class ) {
48		- $return_class = apply_filters( 'lsx_layout_selector', $main_class, $class, $layout, $size );
	47	+ if ('main' === $class) {
	48	+ $return_class = apply_filters('lsx_layout_selector', $main_class, $class, $layout, $size);
49	49	}
50	50
51		- if ( 'sidebar' === $class ) {
52		- $return_class = apply_filters( 'lsx_layout_selector', $sidebar_class, $class, $layout, $size );
	51	+ if ('sidebar' === $class) {
	52	+ $return_class = apply_filters('lsx_layout_selector', $sidebar_class, $class, $layout, $size);
53	53	}
54	54
55	55	return $return_class;
		@@ -57,7 +57,7 @@ discard block
		block discarded – undo
57	57
58	58	endif;
59	59
60		-if ( ! function_exists( 'lsx_main_class' ) ) :
	60	+if ( ! function_exists('lsx_main_class')) :
61	61
62	62	/**
63	63	* .main classes.
		@@ -66,12 +66,12 @@ discard block
		block discarded – undo
66	66	* @subpackage layout
67	67	*/
68	68	function lsx_main_class() {
69		- return lsx_layout_selector( 'main' );
	69	+ return lsx_layout_selector('main');
70	70	}
71	71
72	72	endif;
73	73
74		-if ( ! function_exists( 'lsx_sidebar_class' ) ) :
	74	+if ( ! function_exists('lsx_sidebar_class')) :
75	75
76	76	/**
77	77	* .sidebar classes.
		@@ -80,12 +80,12 @@ discard block
		block discarded – undo
80	80	* @subpackage layout
81	81	*/
82	82	function lsx_sidebar_class() {
83		- return lsx_layout_selector( 'sidebar' );
	83	+ return lsx_layout_selector('sidebar');
84	84	}
85	85
86	86	endif;
87	87
88		-if ( ! function_exists( 'lsx_header_classes' ) ) :
	88	+if ( ! function_exists('lsx_header_classes')) :
89	89
90	90	/**
91	91	* Output the classes for the header.
		@@ -93,19 +93,19 @@ discard block
		block discarded – undo
93	93	* @package lsx
94	94	* @subpackage layout
95	95	*/
96		- function lsx_header_classes( $additional = false ) {
	96	+ function lsx_header_classes($additional = false) {
97	97	$classes = 'banner navbar navbar-default';
98	98
99		- if ( false !== $additional ) {
	99	+ if (false !== $additional) {
100	100	$classes .= ' ' . $additional;
101	101	}
102	102
103		- echo esc_attr( $classes );
	103	+ echo esc_attr($classes);
104	104	}
105	105
106	106	endif;
107	107
108		-if ( ! function_exists( 'lsx_top_menu_classes' ) ) :
	108	+if ( ! function_exists('lsx_top_menu_classes')) :
109	109
110	110	/**
111	111	* Output the classes for the top-menu.
		@@ -113,19 +113,19 @@ discard block
		block discarded – undo
113	113	* @package lsx
114	114	* @subpackage layout
115	115	*/
116		- function lsx_top_menu_classes( $additional = false ) {
	116	+ function lsx_top_menu_classes($additional = false) {
117	117	$classes = 'top-menu-default';
118	118
119		- if ( false !== $additional ) {
	119	+ if (false !== $additional) {
120	120	$classes .= ' ' . $additional;
121	121	}
122	122
123		- echo esc_attr( $classes );
	123	+ echo esc_attr($classes);
124	124	}
125	125
126	126	endif;
127	127
128		-if ( ! function_exists( 'lsx_add_footer_sidebar_area' ) ) :
	128	+if ( ! function_exists('lsx_add_footer_sidebar_area')) :
129	129
130	130	/**
131	131	* Output the Footer CTA and/pr Footer Widgets.
		@@ -134,34 +134,34 @@ discard block
		block discarded – undo
134	134	* @subpackage layout
135	135	*/
136	136	function lsx_add_footer_sidebar_area() {
137		- if ( is_active_sidebar( 'sidebar-footer-cta' ) ) : ?>
	137	+ if (is_active_sidebar('sidebar-footer-cta')) : ?>
138	138	<div id="footer-cta">
139	139	<div class="container">
140	140	<div class="lsx-full-width">
141	141	<div class="lsx-hero-unit">
142		- <?php dynamic_sidebar( 'sidebar-footer-cta' ); ?>
	142	+ <?php dynamic_sidebar('sidebar-footer-cta'); ?>
143	143	</div>
144	144	</div>
145	145	</div>
146	146	</div>
147	147	<?php endif; ?>
148	148
149		- <?php if ( is_active_sidebar( 'sidebar-footer' ) ) : ?>
	149	+ <?php if (is_active_sidebar('sidebar-footer')) : ?>
150	150	<div id="footer-widgets">
151	151	<div class="container">
152	152	<div class="row">
153		- <?php dynamic_sidebar( 'sidebar-footer' ); ?>
	153	+ <?php dynamic_sidebar('sidebar-footer'); ?>
154	154	</div>
155	155	</div>
156	156	</div>
157	157	<?php endif;
158	158	}
159	159
160		- add_action( 'lsx_footer_before', 'lsx_add_footer_sidebar_area' );
	160	+ add_action('lsx_footer_before', 'lsx_add_footer_sidebar_area');
161	161
162	162	endif;
163	163
164		-if ( ! function_exists( 'lsx_global_header' ) ) :
	164	+if ( ! function_exists('lsx_global_header')) :
165	165
166	166	/**
167	167	* Displays the global header.
		@@ -174,21 +174,21 @@ discard block
		block discarded – undo
174	174	// return;
175	175	// }
176	176
177		- $show_on_front = get_option( 'show_on_front' );
	177	+ $show_on_front = get_option('show_on_front');
178	178	$queried_object = get_queried_object();
179	179	$default_size = 'sm';
180		- $size = apply_filters( 'lsx_bootstrap_column_size', $default_size );
	180	+ $size = apply_filters('lsx_bootstrap_column_size', $default_size);
181	181
182		- if ( true === apply_filters( 'lsx_global_header_disable', false ) ) :
	182	+ if (true === apply_filters('lsx_global_header_disable', false)) :
183	183	// Display only the breadcrumbs
184	184	?>
185		- <div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
	185	+ <div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
186	186	<?php lsx_global_header_inner_bottom(); ?>
187	187	</div>
188	188	<?php
189		- elseif ( is_page() && ( 'page' !== $show_on_front \|\| ! is_front_page() ) ) :
	189	+ elseif (is_page() && ('page' !== $show_on_front \|\| ! is_front_page())) :
190	190	?>
191		- <div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
	191	+ <div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
192	192	<header class="archive-header">
193	193	<h1 class="archive-title"><?php the_title(); ?></h1>
194	194	</header>
		@@ -196,9 +196,9 @@ discard block
		block discarded – undo
196	196	<?php lsx_global_header_inner_bottom(); ?>
197	197	</div>
198	198	<?php
199		- elseif ( is_single() && ! is_singular( 'post' ) ) :
	199	+ elseif (is_single() && ! is_singular('post')) :
200	200	?>
201		- <div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
	201	+ <div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
202	202	<header class="archive-header">
203	203	<h1 class="archive-title"><?php the_title(); ?></h1>
204	204	</header>
		@@ -206,15 +206,15 @@ discard block
		block discarded – undo
206	206	<?php lsx_global_header_inner_bottom(); ?>
207	207	</div>
208	208	<?php
209		- elseif ( is_search() ) :
	209	+ elseif (is_search()) :
210	210	?>
211		- <div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
	211	+ <div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
212	212	<header class="archive-header">
213	213	<h1 class="archive-title">
214	214	<?php
215	215	printf(
216	216	/* Translators: %s: search term/query */
217		- esc_html__( 'Search Results for: %s', 'lsx' ),
	217	+ esc_html__('Search Results for: %s', 'lsx'),
218	218	'<span>' . get_search_query() . '</span>'
219	219	);
220	220	?>
		@@ -224,12 +224,12 @@ discard block
		block discarded – undo
224	224	<?php lsx_global_header_inner_bottom(); ?>
225	225	</div>
226	226	<?php
227		- elseif ( is_author() ) :
	227	+ elseif (is_author()) :
228	228	$author = get_the_author();
229		- $author_avatar = get_avatar( get_the_author_meta( 'ID' ), 256 );
	229	+ $author_avatar = get_avatar(get_the_author_meta('ID'), 256);
230	230	$author_bio = get_the_archive_description();
231	231	?>
232		- <div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
	232	+ <div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
233	233	<header class="archive-header">
234	234	<h1 class="archive-title"><?php the_archive_title(); ?></h1>
235	235	</header>
		@@ -237,13 +237,13 @@ discard block
		block discarded – undo
237	237	<?php lsx_global_header_inner_bottom(); ?>
238	238	</div>
239	239	<?php
240		- elseif ( is_archive() ) :
	240	+ elseif (is_archive()) :
241	241	?>
242		- <div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
	242	+ <div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
243	243	<header class="archive-header">
244	244	<h1 class="archive-title">
245		- <?php if ( has_post_format() && ! is_category() && ! is_tag() && ! is_date() && ! is_tax( 'post_format' ) ) { ?>
246		- <?php the_archive_title( esc_html__( 'Type:', 'lsx' ) ); ?>
	245	+ <?php if (has_post_format() && ! is_category() && ! is_tag() && ! is_date() && ! is_tax('post_format')) { ?>
	246	+ <?php the_archive_title(esc_html__('Type:', 'lsx')); ?>
247	247	<?php } else { ?>
248	248	<?php the_archive_title(); ?>
249	249	<?php } ?>
		@@ -255,31 +255,31 @@ discard block
		block discarded – undo
255	255	<?php lsx_global_header_inner_bottom(); ?>
256	256	</div>
257	257	<?php
258		- elseif ( 'page' === $show_on_front && (int) get_option( 'page_for_posts' ) === $queried_object->ID ) :
	258	+ elseif ('page' === $show_on_front && (int) get_option('page_for_posts') === $queried_object->ID) :
259	259	?>
260		- <div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
	260	+ <div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
261	261	<header class="archive-header">
262		- <h1 class="archive-title"><?php esc_html_e( 'Blog', 'lsx' ); ?></h1>
	262	+ <h1 class="archive-title"><?php esc_html_e('Blog', 'lsx'); ?></h1>
263	263	</header>
264	264
265	265	<?php lsx_global_header_inner_bottom(); ?>
266	266	</div>
267	267	<?php
268		- elseif ( ! is_singular( 'post' ) ) :
	268	+ elseif ( ! is_singular('post')) :
269	269	// Display only the breadcrumbs
270	270	?>
271		- <div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
	271	+ <div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
272	272	<?php lsx_global_header_inner_bottom(); ?>
273	273	</div>
274	274	<?php
275	275	endif;
276	276	}
277	277
278		- add_action( 'lsx_content_wrap_before', 'lsx_global_header' );
	278	+ add_action('lsx_content_wrap_before', 'lsx_global_header');
279	279
280	280	endif;
281	281
282		-if ( ! function_exists( 'lsx_author_extra_info' ) ) :
	282	+if ( ! function_exists('lsx_author_extra_info')) :
283	283
284	284	/**
285	285	* Displays the author extra info.
		@@ -289,58 +289,58 @@ discard block
		block discarded – undo
289	289	*/
290	290	function lsx_author_extra_info() {
291	291	$default_size = 'sm';
292		- $size = apply_filters( 'lsx_bootstrap_column_size', $default_size );
	292	+ $size = apply_filters('lsx_bootstrap_column_size', $default_size);
293	293
294		- if ( is_author() ) :
295		- $author_id = get_the_author_meta( 'ID' );
	294	+ if (is_author()) :
	295	+ $author_id = get_the_author_meta('ID');
296	296	$author = get_the_author();
297		- $author_avatar = get_avatar( $author_id, 256 );
	297	+ $author_avatar = get_avatar($author_id, 256);
298	298	$author_bio = get_the_archive_description();
299		- $author_url = get_the_author_meta( 'url', $author_id );
300		- $author_email = get_the_author_meta( 'email', $author_id );
301		- $author_facebook = get_the_author_meta( 'facebook', $author_id );
302		- $author_twitter = get_the_author_meta( 'twitter', $author_id );
303		- $author_googleplus = get_the_author_meta( 'googleplus', $author_id );
	299	+ $author_url = get_the_author_meta('url', $author_id);
	300	+ $author_email = get_the_author_meta('email', $author_id);
	301	+ $author_facebook = get_the_author_meta('facebook', $author_id);
	302	+ $author_twitter = get_the_author_meta('twitter', $author_id);
	303	+ $author_googleplus = get_the_author_meta('googleplus', $author_id);
304	304	?>
305		- <div class="col-<?php echo esc_attr( $size ); ?>-12">
	305	+ <div class="col-<?php echo esc_attr($size); ?>-12">
306	306	<div class="archive-author-data">
307		- <figure class="archive-author-avatar"><?php echo wp_kses_post( $author_avatar ); ?></figure>
	307	+ <figure class="archive-author-avatar"><?php echo wp_kses_post($author_avatar); ?></figure>
308	308
309		- <?php if ( ! empty( $author_url ) \|\| ! empty( $author_email ) \|\| ! empty( $author_facebook ) \|\| ! empty( $author_twitter ) \|\| ! empty( $author_googleplus ) ) : ?>
	309	+ <?php if ( ! empty($author_url) \|\| ! empty($author_email) \|\| ! empty($author_facebook) \|\| ! empty($author_twitter) \|\| ! empty($author_googleplus)) : ?>
310	310	<div class="archive-author-social-links">
311		- <?php if ( ! empty( $author_url ) ) : ?>
312		- <a href="<?php echo esc_url( $author_url ); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-url"><i class="fa fa-link" aria-hidden="true"></i></a>
	311	+ <?php if ( ! empty($author_url)) : ?>
	312	+ <a href="<?php echo esc_url($author_url); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-url"><i class="fa fa-link" aria-hidden="true"></i></a>
313	313	<?php endif; ?>
314	314
315		- <?php if ( ! empty( $author_email ) ) : ?>
316		- <a href="mailto:<?php echo esc_attr( $author_email ); ?>" class="archive-author-social-link archive-author-social-link-email"><i class="fa fa-envelope" aria-hidden="true"></i></a>
	315	+ <?php if ( ! empty($author_email)) : ?>
	316	+ <a href="mailto:<?php echo esc_attr($author_email); ?>" class="archive-author-social-link archive-author-social-link-email"><i class="fa fa-envelope" aria-hidden="true"></i></a>
317	317	<?php endif; ?>
318	318
319		- <?php if ( ! empty( $author_facebook ) ) : ?>
320		- <a href="<?php echo esc_url( $author_facebook ); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-facebook"><i class="fa fa-facebook" aria-hidden="true"></i></a>
	319	+ <?php if ( ! empty($author_facebook)) : ?>
	320	+ <a href="<?php echo esc_url($author_facebook); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-facebook"><i class="fa fa-facebook" aria-hidden="true"></i></a>
321	321	<?php endif; ?>
322	322
323		- <?php if ( ! empty( $author_twitter ) ) : ?>
324		- <a href="https://twitter.com/<?php echo esc_attr( $author_twitter ); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-twitter"><i class="fa fa-twitter" aria-hidden="true"></i></a>
	323	+ <?php if ( ! empty($author_twitter)) : ?>
	324	+ <a href="https://twitter.com/<?php echo esc_attr($author_twitter); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-twitter"><i class="fa fa-twitter" aria-hidden="true"></i></a>
325	325	<?php endif; ?>
326	326
327		- <?php if ( ! empty( $author_googleplus ) ) : ?>
328		- <a href="<?php echo esc_url( $author_googleplus ); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-googleplus"><i class="fa fa-google-plus" aria-hidden="true"></i></a>
	327	+ <?php if ( ! empty($author_googleplus)) : ?>
	328	+ <a href="<?php echo esc_url($author_googleplus); ?>" target="_blank" rel="nofollow" class="archive-author-social-link archive-author-social-link-googleplus"><i class="fa fa-google-plus" aria-hidden="true"></i></a>
329	329	<?php endif; ?>
330	330	</div>
331	331	<?php endif; ?>
332	332
333		- <?php if ( ! empty( $author_bio ) ) : ?>
334		- <h2 class="archive-author-title text-center"><?php esc_html_e( 'About the author', 'lsx' ); ?></h2>
335		- <p class="archive-author-bio"><?php echo wp_kses_post( $author_bio ); ?></p>
	333	+ <?php if ( ! empty($author_bio)) : ?>
	334	+ <h2 class="archive-author-title text-center"><?php esc_html_e('About the author', 'lsx'); ?></h2>
	335	+ <p class="archive-author-bio"><?php echo wp_kses_post($author_bio); ?></p>
336	336	<?php endif; ?>
337	337
338	338	<h2 class="archive-author-posts text-center">
339	339	<?php
340	340	printf(
341	341	/* Translators: %s: author name */
342		- esc_html__( 'All posts by %s', 'lsx' ),
343		- esc_html( $author )
	342	+ esc_html__('All posts by %s', 'lsx'),
	343	+ esc_html($author)
344	344	);
345	345	?>
346	346	</h2>
		@@ -350,11 +350,11 @@ discard block
		block discarded – undo
350	350	endif;
351	351	}
352	352
353		- add_action( 'lsx_content_wrap_before', 'lsx_author_extra_info', 11 );
	353	+ add_action('lsx_content_wrap_before', 'lsx_author_extra_info', 11);
354	354
355	355	endif;
356	356
357		-if ( ! function_exists( 'lsx_post_header' ) ) :
	357	+if ( ! function_exists('lsx_post_header')) :
358	358
359	359	/**
360	360	* Displays the post header.
		@@ -364,21 +364,21 @@ discard block
		block discarded – undo
364	364	*/
365	365	function lsx_post_header() {
366	366	$default_size = 'sm';
367		- $size = apply_filters( 'lsx_bootstrap_column_size', $default_size );
	367	+ $size = apply_filters('lsx_bootstrap_column_size', $default_size);
368	368
369		- if ( is_singular( 'post' ) ) :
	369	+ if (is_singular('post')) :
370	370	$format = get_post_format();
371	371
372		- if ( false === $format ) {
	372	+ if (false === $format) {
373	373	$format = 'standard';
374	374	}
375	375
376		- $format = lsx_translate_format_to_fontawesome( $format );
	376	+ $format = lsx_translate_format_to_fontawesome($format);
377	377	?>
378		- <div class="archive-header-wrapper col-<?php echo esc_attr( $size ); ?>-12">
	378	+ <div class="archive-header-wrapper col-<?php echo esc_attr($size); ?>-12">
379	379	<header class="archive-header">
380	380	<h1 class="archive-title">
381		- <i class="format-link fa fa-<?php echo esc_attr( $format ); ?>"></i>
	381	+ <i class="format-link fa fa-<?php echo esc_attr($format); ?>"></i>
382	382	<span><?php the_title(); ?></span>
383	383	</h1>
384	384	</header>
		@@ -387,11 +387,11 @@ discard block
		block discarded – undo
387	387	endif;
388	388	}
389	389
390		- add_action( 'lsx_entry_top', 'lsx_post_header' );
	390	+ add_action('lsx_entry_top', 'lsx_post_header');
391	391
392	392	endif;
393	393
394		-if ( ! function_exists( 'lsx_add_viewport_meta_tag' ) ) :
	394	+if ( ! function_exists('lsx_add_viewport_meta_tag')) :
395	395
396	396	/**
397	397	* Add Viewport Meta Tag to head.
		@@ -405,11 +405,11 @@ discard block
		block discarded – undo
405	405	<?php
406	406	}
407	407
408		- add_action( 'wp_head', 'lsx_add_viewport_meta_tag' );
	408	+ add_action('wp_head', 'lsx_add_viewport_meta_tag');
409	409
410	410	endif;
411	411
412		-if ( ! function_exists( 'lsx_header_search_form' ) ) :
	412	+if ( ! function_exists('lsx_header_search_form')) :
413	413
414	414	/**
415	415	* Add a search form to just above the nav menu.
		@@ -418,13 +418,13 @@ discard block
		block discarded – undo
418	418	* @subpackage layout
419	419	*/
420	420	function lsx_header_search_form() {
421		- $search_form = get_theme_mod( 'lsx_header_search', false );
	421	+ $search_form = get_theme_mod('lsx_header_search', false);
422	422
423		- if ( false !== $search_form \|\| is_customize_preview() ) {
424		- get_search_form( true );
	423	+ if (false !== $search_form \|\| is_customize_preview()) {
	424	+ get_search_form(true);
425	425	}
426	426	}
427	427
428		- add_action( 'lsx_nav_before', 'lsx_header_search_form', 0 );
	428	+ add_action('lsx_nav_before', 'lsx_header_search_form', 0);
429	429
430	430	endif;

lightspeeddevelopment / lsx

Push — master ( d53b10...8df284 )

Status

Category

Spacing +98 added lines, -98 removed lines patch added patch discarded remove patch