Inspection of "* Fix - Top menu dropdown in large screens (first..." - lightspeeddevelopment/lsx - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( bef6ca...7e2933 )

by Fernando

created 2017-02-15 15:49 UTC

Status

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

@@ -11,7 +11,7 @@  discard block
 block discarded – undo
 
 	<?php lsx_content_wrap_before(); ?>
 
-	<div id="primary" class="content-area <?php echo esc_attr( lsx_main_class() ); ?>">
+	<div id="primary" class="content-area <?php echo esc_attr(lsx_main_class()); ?>">
 		
 		<?php lsx_content_before(); ?>
 
@@ -19,25 +19,25 @@  discard block
 block discarded – undo
 
 		<?php lsx_content_top(); ?>
 
-		<?php if ( have_posts() ) : ?>
+		<?php if (have_posts()) : ?>
 
 			<?php 
-			$layout = get_theme_mod('lsx_layout','2cr');
-			$layout = apply_filters( 'lsx_layout', $layout );
-			if('1c' === $layout){
+			$layout = get_theme_mod('lsx_layout', '2cr');
+			$layout = apply_filters('lsx_layout', $layout);
+			if ('1c' === $layout) {
 				lsx_breadcrumbs();
 			}
 			?>			
 
 			<?php /* Start the Loop */ ?>
-			<?php while ( have_posts() ) : the_post(); ?>
+			<?php while (have_posts()) : the_post(); ?>
 
 				<?php
 					/* Include the Post-Format-specific template for the content.
 					 * If you want to override this in a child theme, then include a file
 					 * called content-___.php (where ___ is the Post Format name) and that will be used instead.
 					 */
-					get_template_part( 'content', get_post_format() );
+					get_template_part('content', get_post_format());
 				?>
 
 			<?php endwhile; ?>
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
 
 		<?php else : ?>
 
-			<?php get_template_part( 'content', 'none' ); ?>
+			<?php get_template_part('content', 'none'); ?>
 
 		<?php endif; ?>
 

Please login to merge, or discard this patch.

Braces +5 added lines, -2 removed lines patch added patch discarded remove patch

@@ -47,9 +47,12 @@
 block discarded – undo
 			
 			<?php lsx_paging_nav(); ?>
 
-		<?php else : ?>
+		<?php else {
+	: ?>
 
-			<?php get_template_part( 'content', 'none' ); ?>
+			<?php get_template_part( 'content', 'none' );
+}
+?>
 
 		<?php endif; ?>
 

Please login to merge, or discard this patch.

index.php 2 patches

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

@@ -15,7 +15,7 @@  discard block
 block discarded – undo
 
 	<?php lsx_content_wrap_before(); ?>
 
-	<div id="primary" class="content-area <?php echo esc_attr( lsx_main_class() ); ?>">
+	<div id="primary" class="content-area <?php echo esc_attr(lsx_main_class()); ?>">
 
 		<?php lsx_content_before(); ?>
 
@@ -23,17 +23,17 @@  discard block
 block discarded – undo
 		
 		<?php lsx_content_top(); ?>
 
-		<?php if ( have_posts() ) : ?>
+		<?php if (have_posts()) : ?>
 
 			<?php /* Start the Loop */ ?>
-			<?php while ( have_posts() ) : the_post(); ?>
+			<?php while (have_posts()) : the_post(); ?>
 
 				<?php
 					/* Include the Post-Format-specific template for the content.
 					 * If you want to override this in a child theme, then include a file
 					 * called content-___.php (where ___ is the Post Format name) and that will be used instead.
 					 */
-					get_template_part( 'content', get_post_format() );
+					get_template_part('content', get_post_format());
 				?>
 
 			<?php endwhile; ?>
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
 			
 		<?php else : ?>
 
-			<?php get_template_part( 'content', 'none' ); ?>
+			<?php get_template_part('content', 'none'); ?>
 
 		<?php endif; ?>
 
@@ -56,5 +56,5 @@  discard block
 block discarded – undo
 
 	<?php lsx_content_wrap_after(); ?>
 
-<?php get_sidebar( 'sidebar' ); ?>
+<?php get_sidebar('sidebar'); ?>
 <?php get_footer();
\ No newline at end of file

Please login to merge, or discard this patch.

Braces +5 added lines, -2 removed lines patch added patch discarded remove patch

@@ -47,9 +47,12 @@
 block discarded – undo
 			
 			<?php lsx_paging_nav(); ?>
 
-		<?php else : ?>
+		<?php else {
+	: ?>
 
-			<?php get_template_part( 'content', 'none' ); ?>
+			<?php get_template_part( 'content', 'none' );
+}
+?>
 
 		<?php endif; ?>
 

Please login to merge, or discard this patch.

header.php 2 patches

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

@@ -12,16 +12,16 @@  discard block
 block discarded – undo
 <html <?php language_attributes(); ?>>
 <head>
 <?php lsx_head_top(); ?>
-<meta charset="<?php bloginfo( 'charset' ); ?>">
+<meta charset="<?php bloginfo('charset'); ?>">
 <link rel="profile" href="http://gmpg.org/xfn/11">
-<link rel="pingback" href="<?php bloginfo( 'pingback_url' ); ?>">
+<link rel="pingback" href="<?php bloginfo('pingback_url'); ?>">
 <?php lsx_head_bottom(); ?>
 <?php wp_head(); ?>
 </head>
-<body <?php body_class( 'lsx' ); ?>>
+<body <?php body_class('lsx'); ?>>
 <?php lsx_body_top(); ?>
 
-<?php if ( !is_singular('landing-page') ) { ?>
+<?php if ( ! is_singular('landing-page')) { ?>
 
 	<?php lsx_header_before(); ?>
 		 	<header class="<?php lsx_header_classes(); ?>">
@@ -57,17 +57,17 @@  discard block
 block discarded – undo
 			    </div>
 
 			    <div class="header-links">
-			    	<strong><?php esc_html_e( 'Get your Tourism Establishment Online', 'lsx' ); ?></strong>
+			    	<strong><?php esc_html_e('Get your Tourism Establishment Online', 'lsx'); ?></strong>
 			    	
 			    	<?php 
-			    	if(is_singular('landing-page')){ 
-			    			$email_address = get_post_meta(get_the_ID(),'email_address',true);
-			    			if(false === $email_address){
+			    	if (is_singular('landing-page')) { 
+			    			$email_address = get_post_meta(get_the_ID(), 'email_address', true);
+			    			if (false === $email_address) {
 								$email_address = '[email protected]';
 							}
 			    	} ?>
 			    	
-			    	<span class="email-address"><?php esc_html_e( 'Questions? Email Us: ', 'lsx' ); ?><a href="mailto:<?php echo esc_attr( $email_address ); ?>"><?php echo esc_html( $email_address ); ?></a></span>
+			    	<span class="email-address"><?php esc_html_e('Questions? Email Us: ', 'lsx'); ?><a href="mailto:<?php echo esc_attr($email_address); ?>"><?php echo esc_html($email_address); ?></a></span>
 			    </div>
 		  	</div>
 

Please login to merge, or discard this patch.

Indentation +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -60,12 +60,12 @@
 block discarded – undo
 			    	<strong><?php esc_html_e( 'Get your Tourism Establishment Online', 'lsx' ); ?></strong>
 			    	
 			    	<?php 
-			    	if(is_singular('landing-page')){ 
-			    			$email_address = get_post_meta(get_the_ID(),'email_address',true);
-			    			if(false === $email_address){
+					if(is_singular('landing-page')){ 
+							$email_address = get_post_meta(get_the_ID(),'email_address',true);
+							if(false === $email_address){
 								$email_address = '[email protected]';
 							}
-			    	} ?>
+					} ?>
 			    	
 			    	<span class="email-address"><?php esc_html_e( 'Questions? Email Us: ', 'lsx' ); ?><a href="mailto:<?php echo esc_attr( $email_address ); ?>"><?php echo esc_html( $email_address ); ?></a></span>
 			    </div>

Please login to merge, or discard this patch.

content-none.php 2 patches

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

@@ -10,22 +10,22 @@
 block discarded – undo
 
 <section class="no-results not-found">
 	<header class="page-header">
-		<h1 class="page-title"><?php esc_html_e( 'Nothing Found', 'lsx' ); ?></h1>
+		<h1 class="page-title"><?php esc_html_e('Nothing Found', 'lsx'); ?></h1>
 	</header><!-- .page-header -->
 
 	<div class="page-content">
-		<?php if ( is_home() && current_user_can( 'publish_posts' ) ) : ?>
+		<?php if (is_home() && current_user_can('publish_posts')) : ?>
 
-			<p><?php esc_html_e( 'Ready to publish your first post?', 'lsx' ); ?> <a href="<?php echo esc_url( admin_url( 'post-new.php' ) ) ?>"><?php esc_html_e( 'Get started here', 'lsx' ); ?></a></p>
+			<p><?php esc_html_e('Ready to publish your first post?', 'lsx'); ?> <a href="<?php echo esc_url(admin_url('post-new.php')) ?>"><?php esc_html_e('Get started here', 'lsx'); ?></a></p>
 
-		<?php elseif ( is_search() ) : ?>
+		<?php elseif (is_search()) : ?>
 
-			<p><?php esc_html_e( 'Sorry, but nothing matched your search terms. Please try again with some different keywords.', 'lsx' ); ?></p>
+			<p><?php esc_html_e('Sorry, but nothing matched your search terms. Please try again with some different keywords.', 'lsx'); ?></p>
 			<?php get_search_form(); ?>
 
 		<?php else : ?>
 
-			<p><?php esc_html_e( 'It seems we can&rsquo;t find what you&rsquo;re looking for. Perhaps searching can help.', 'lsx' ); ?></p>
+			<p><?php esc_html_e('It seems we can&rsquo;t find what you&rsquo;re looking for. Perhaps searching can help.', 'lsx'); ?></p>
 			<?php get_search_form(); ?>
 
 		<?php endif; ?>

Please login to merge, or discard this patch.

Braces +5 added lines, -2 removed lines patch added patch discarded remove patch

@@ -23,9 +23,12 @@
 block discarded – undo
 			<p><?php esc_html_e( 'Sorry, but nothing matched your search terms. Please try again with some different keywords.', 'lsx' ); ?></p>
 			<?php get_search_form(); ?>
 
-		<?php else : ?>
+		<?php else {
+	: ?>
 
-			<p><?php esc_html_e( 'It seems we can&rsquo;t find what you&rsquo;re looking for. Perhaps searching can help.', 'lsx' ); ?></p>
+			<p><?php esc_html_e( 'It seems we can&rsquo;t find what you&rsquo;re looking for. Perhaps searching can help.', 'lsx' );
+}
+?></p>
 			<?php get_search_form(); ?>
 
 		<?php endif; ?>

Please login to merge, or discard this patch.

inc/hooks.php 1 patch

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

@@ -24,11 +24,11 @@  discard block
 block discarded – undo
  * $lsx_supports[] = 'body';
  */
  function lsx_body_top() {
-	 do_action( 'lsx_body_top' );
+	 do_action('lsx_body_top');
  }
 
  function lsx_body_bottom() {
-	 do_action( 'lsx_body_bottom' );
+	 do_action('lsx_body_bottom');
  }
  
 /**
@@ -37,11 +37,11 @@  discard block
 block discarded – undo
 * $lsx_supports[] = 'head';
 */
 function lsx_head_top() {
-	do_action( 'lsx_head_top' );
+	do_action('lsx_head_top');
 }
 
 function lsx_head_bottom() {
-	do_action( 'lsx_head_bottom' );
+	do_action('lsx_head_bottom');
 }
 
 /**
@@ -50,27 +50,27 @@  discard block
 block discarded – undo
 * $lsx_supports[] = 'header';
 */
 function lsx_header_before() {
-	do_action( 'lsx_header_before' );
+	do_action('lsx_header_before');
 }
 
 function lsx_header_after() {
-	do_action( 'lsx_header_after' );
+	do_action('lsx_header_after');
 }
 
 function lsx_header_top() {
-	do_action( 'lsx_header_top' );
+	do_action('lsx_header_top');
 }
 
 function lsx_header_bottom() {
-	do_action( 'lsx_header_bottom' );
+	do_action('lsx_header_bottom');
 }
 
 function lsx_nav_before() {
-	do_action( 'lsx_nav_before' );
+	do_action('lsx_nav_before');
 }
 
 function lsx_nav_after() {
-	do_action( 'lsx_nav_after' );
+	do_action('lsx_nav_after');
 }
 
 /**
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
  */
 
 function lsx_banner_content() {
-	do_action( 'lsx_banner_content' );
+	do_action('lsx_banner_content');
 }
 
 /**
@@ -89,35 +89,35 @@  discard block
 block discarded – undo
 * $lsx_supports[] = 'content';
 */
 function lsx_content_wrap_before() {
-	do_action( 'lsx_content_wrap_before' );
+	do_action('lsx_content_wrap_before');
 }
 
 function lsx_content_wrap_after() {
-	do_action( 'lsx_content_wrap_after' );
+	do_action('lsx_content_wrap_after');
 }
 
 function lsx_content_before() {
-	do_action( 'lsx_content_before' );
+	do_action('lsx_content_before');
 }
 
 function lsx_content_after() {
-	do_action( 'lsx_content_after' );
+	do_action('lsx_content_after');
 }
 
 function lsx_content_top() {
-	do_action( 'lsx_content_top' );
+	do_action('lsx_content_top');
 }
 
 function lsx_content_bottom() {
-	do_action( 'lsx_content_bottom' );
+	do_action('lsx_content_bottom');
 }
 
 function lsx_content_post_meta() {
-	do_action( 'lsx_content_post_meta' );
+	do_action('lsx_content_post_meta');
 }
 
 function lsx_content_post_tags() {
-	do_action( 'lsx_content_post_tags' );
+	do_action('lsx_content_post_tags');
 }
 
 /**
@@ -126,19 +126,19 @@  discard block
 block discarded – undo
 * $lsx_supports[] = 'entry';
 */
 function lsx_entry_before() {
-	do_action( 'lsx_entry_before' );
+	do_action('lsx_entry_before');
 }
 
 function lsx_entry_after() {
-	do_action( 'lsx_entry_after' );
+	do_action('lsx_entry_after');
 }
 
 function lsx_entry_top() {
-	do_action( 'lsx_entry_top' );
+	do_action('lsx_entry_top');
 }
 
 function lsx_entry_bottom() {
-	do_action( 'lsx_entry_bottom' );
+	do_action('lsx_entry_bottom');
 }
 
 /**
@@ -147,11 +147,11 @@  discard block
 block discarded – undo
 * $lsx_supports[] = 'comments';
 */
 function lsx_comments_before() {
-	do_action( 'lsx_comments_before' );
+	do_action('lsx_comments_before');
 }
 
 function lsx_comments_after() {
-	do_action( 'lsx_comments_after' );
+	do_action('lsx_comments_after');
 }
 
 /**
@@ -160,19 +160,19 @@  discard block
 block discarded – undo
 * $lsx_supports[] = 'sidebar';
 */
 function lsx_sidebars_before() {
-	do_action( 'lsx_sidebars_before' );
+	do_action('lsx_sidebars_before');
 }
 
 function lsx_sidebars_after() {
-	do_action( 'lsx_sidebars_after' );
+	do_action('lsx_sidebars_after');
 }
 
 function lsx_sidebar_top() {
-	do_action( 'lsx_sidebar_top' );
+	do_action('lsx_sidebar_top');
 }
 
 function lsx_sidebar_bottom() {
-	do_action( 'lsx_sidebar_bottom' );
+	do_action('lsx_sidebar_bottom');
 }
 
 /**
@@ -181,17 +181,17 @@  discard block
 block discarded – undo
 * $lsx_supports[] = 'footer';
 */
 function lsx_footer_before() {
-	do_action( 'lsx_footer_before' );
+	do_action('lsx_footer_before');
 }
 
 function lsx_footer_after() {
-	do_action( 'lsx_footer_after' );
+	do_action('lsx_footer_after');
 }
 
 function lsx_footer_top() {
-	do_action( 'lsx_footer_top' );
+	do_action('lsx_footer_top');
 }
 
 function lsx_footer_bottom() {
-	do_action( 'lsx_footer_bottom' );
+	do_action('lsx_footer_bottom');
 }
\ No newline at end of file

Please login to merge, or discard this patch.

search.php 2 patches

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

@@ -9,7 +9,7 @@  discard block
 block discarded – undo
 
 	<?php lsx_content_wrap_before(); ?>
 
-	<div id="primary" class="content-area <?php echo esc_attr( lsx_main_class() ); ?>">
+	<div id="primary" class="content-area <?php echo esc_attr(lsx_main_class()); ?>">
 
 		<?php lsx_content_before(); ?>
 
@@ -17,25 +17,25 @@  discard block
 block discarded – undo
 
 		<?php lsx_content_top(); ?>	
 
-		<?php if ( have_posts() ) : global $lsx_archive; $lsx_archive = 1; ?>
+		<?php if (have_posts()) : global $lsx_archive; $lsx_archive = 1; ?>
 		
 			<?php 
-			$layout = get_theme_mod('lsx_layout','2cr');
-			$layout = apply_filters( 'lsx_layout', $layout );
-			if('1c' === $layout){
+			$layout = get_theme_mod('lsx_layout', '2cr');
+			$layout = apply_filters('lsx_layout', $layout);
+			if ('1c' === $layout) {
 				lsx_breadcrumbs();
 			}
 			?>	
 
 			<?php /* Start the Loop */ ?>
-			<?php while ( have_posts() ) : the_post(); ?>
+			<?php while (have_posts()) : the_post(); ?>
 
 				<?php
 					/* Include the Post-Format-specific template for the content.
 					 * If you want to override this in a child theme, then include a file
 					 * called content-___.php (where ___ is the Post Format name) and that will be used instead.
 					 */
-					get_template_part( 'content', get_post_format() );
+					get_template_part('content', get_post_format());
 				?>
 
 			<?php endwhile; ?>
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
 
 		<?php else : ?>
 
-			<?php get_template_part( 'content', 'none' ); ?>
+			<?php get_template_part('content', 'none'); ?>
 
 		<?php endif; $lsx_archive = 0; ?>
 

Please login to merge, or discard this patch.

Braces +5 added lines, -2 removed lines patch added patch discarded remove patch

@@ -42,9 +42,12 @@
 block discarded – undo
 
 			<?php lsx_paging_nav(); ?>
 
-		<?php else : ?>
+		<?php else {
+	: ?>
 
-			<?php get_template_part( 'content', 'none' ); ?>
+			<?php get_template_part( 'content', 'none' );
+}
+?>
 
 		<?php endif; $lsx_archive = 0; ?>
 

Please login to merge, or discard this patch.

inc/wp-bootstrap-navwalker.php 4 patches

Doc Comments -1 removed lines patch added patch discarded remove patch

@@ -30,7 +30,6 @@
 block discarded – undo
 	 * @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 ) {

Please login to merge, or discard this patch.

Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -159,17 +159,17 @@
 block discarded – undo
 	 * @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;
+		if ( ! $element )
+			return;
 
-        $id_field = $this->db_fields['id'];
+		$id_field = $this->db_fields['id'];
 
-        // Display this element.
-        if ( is_object( $args[0] ) )
-           $args[0]->has_children = ! empty( $children_elements[ $element->$id_field ] );
+		// Display this element.
+		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 );
-    }
+		parent::display_element( $element, $children_elements, $max_depth, $depth, $args, $output );
+	}
 
 	/**
 	 * Menu Fallback

Please login to merge, or discard this patch.

Braces +34 added lines, -21 removed lines patch added patch discarded remove patch

@@ -1,5 +1,8 @@  discard block
 block discarded – undo
 <?php
-if ( ! defined( 'ABSPATH' ) ) return; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	return;
+}
+// Exit if accessed directly
 
 class Lsx_Bootstrap_Navwalker extends Walker_Nav_Menu {
 
@@ -71,14 +74,17 @@  discard block
 block discarded – undo
 			
 			$class_names = join( ' ', $classes );
 
-			if ( $args->has_children )
-				$class_names .= ' dropdown';
+			if ( $args->has_children ) {
+							$class_names .= ' dropdown';
+			}
 
-			if ( in_array( 'current-menu-item', $classes ) )
-				$class_names .= ' active';
+			if ( in_array( 'current-menu-item', $classes ) ) {
+							$class_names .= ' active';
+			}
 			
-			if ( in_array( 'current-menu-parent', $classes ) )
-				$class_names .= ' active';			
+			if ( in_array( 'current-menu-parent', $classes ) ) {
+							$class_names .= ' active';
+			}
 			
 			//Check if this is ment to be a "social" type menu
 			$class_names = $class_names ? ' class="' . esc_attr( $class_names ) . '"' : '';
@@ -159,14 +165,16 @@  discard block
 block discarded – undo
 	 * @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;
+        if ( ! $element ) {
+                    return;
+        }
 
         $id_field = $this->db_fields['id'];
 
         // Display this element.
-        if ( is_object( $args[0] ) )
-           $args[0]->has_children = ! empty( $children_elements[ $element->$id_field ] );
+        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 );
     }
@@ -189,29 +197,34 @@  discard block
 block discarded – undo
 			if ( $args['container'] ) {
 				$fb_output = '<' . $args['container'];
 
-				if ( $args['container_id'] )
-					$fb_output .= ' id="' . $args['container_id'] . '"';
+				if ( $args['container_id'] ) {
+									$fb_output .= ' id="' . $args['container_id'] . '"';
+				}
 
-				if ( $args['container_class'] )
-					$fb_output .= ' class="' . $args['container_class'] . '"';
+				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_id'] ) {
+							$fb_output .= ' id="' . $args['menu_id'] . '"';
+			}
 
-			if ( $args['menu_class'] )
-				$fb_output .= ' class="' . $args['menu_class'] . '"';
+			if ( $args['menu_class'] ) {
+							$fb_output .= ' class="' . $args['menu_class'] . '"';
+			}
 
 			$fb_output .= '>';
 			$fb_output .= '<li><a href="' . admin_url( 'nav-menus.php' ) . '">' . esc_html__( 'Add a menu', 'lsx' ) . '</a></li>';
 			$fb_output .= '</ul>';
 
-			if ( $args['container'] )
-				$fb_output .= '</' . $args['container'] . '>';
+			if ( $args['container'] ) {
+							$fb_output .= '</' . $args['container'] . '>';
+			}
 
 			echo wp_kses_post( $fb_output );
 		}

Please login to merge, or discard this patch.

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

@@ -1,5 +1,5 @@  discard block
 block discarded – undo
 <?php
-if ( ! defined( 'ABSPATH' ) ) return; // Exit if accessed directly
+if ( ! defined('ABSPATH')) return; // Exit if accessed directly
 
 class Lsx_Bootstrap_Navwalker extends Walker_Nav_Menu {
 
@@ -10,15 +10,15 @@  discard block
 block discarded – undo
 	 * @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 );
+	public function start_lvl(&$output, $depth = 0, $args = array()) {
+		$indent = str_repeat("\t", $depth);
 		$output .= "\n$indent<ul role=\"menu\" class=\" dropdown-menu\">\n";
 	}
 	
 	/**
 	 * @param string $item Passed by reference. Used to append additional content.
 	 */
-	public function filter_default_pages( &$item ) {
+	public function filter_default_pages(&$item) {
 		
 		return $item;
 	}	
@@ -33,14 +33,14 @@  discard block
 block discarded – undo
 	 * @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 ) : '';	
+	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)){
+		if ( ! isset($item->title)) {
 			return;
 		}
 
@@ -52,62 +52,62 @@  discard block
 block discarded – undo
 		 * 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 ) {
+		if (0 == strcasecmp($item->attr_title, 'divider') && 1 === $depth) {
 			$output .= $indent . '<li role="presentation" class="divider">';
-		} elseif ( 0 == strcasecmp( $item->title, 'divider' ) && 1 === $depth ) {
+		} 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>';
+		} 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 = 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 );
+			$classes = apply_filters('lsx_nav_menu_css_class', array_filter($classes), $item, $args, $depth);
 			
-			$class_names = join( ' ', $classes );
+			$class_names = join(' ', $classes);
 
-			if ( $args->has_children )
+			if ($args->has_children)
 				$class_names .= ' dropdown';
 
-			if ( in_array( 'current-menu-item', $classes ) )
+			if (in_array('current-menu-item', $classes))
 				$class_names .= ' active';
 			
-			if ( in_array( 'current-menu-parent', $classes ) )
+			if (in_array('current-menu-parent', $classes))
 				$class_names .= ' active';			
 			
 			//Check if this is ment to be a "social" type menu
-			$class_names = $class_names ? ' class="' . esc_attr( $class_names ) . '"' : '';
+			$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 ) . '"' : '';
+			$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 .'>';
+			$output .= $indent . '<li' . $id . $value . $class_names . '>';
 
 			$atts = array();
-			$atts['title']  = ! empty( $item->title )	? $item->title	: '';
-			$atts['target'] = ! empty( $item->target )	? $item->target	: '';
-			$atts['rel']    = ! empty( $item->xfn )		? $item->xfn	: '';
+			$atts['title']  = ! empty($item->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';
+			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['href'] = ! empty($item->url) ? $item->url : '';
 			}
 
-			$atts = apply_filters( 'nav_menu_link_attributes', $atts, $item, $args );
+			$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 );
+			foreach ($atts as $attr => $value) {
+				if ( ! empty($value)) {
+					$value = ('href' === $attr) ? esc_url($value) : esc_attr($value);
 					$attributes .= ' ' . $attr . '="' . $value . '"';
 				}
 			}
@@ -121,19 +121,19 @@  discard block
 block discarded – undo
 			 * if there is a value in the attr_title property. If the attr_title
 			 * property is NOT null we apply it as the class name for the glyphicon.
 			 */
-			if ( ! empty( $item->attr_title ) ) {
-				$item_output .= '<a'. $attributes .'"><span class="glyphicon ' . esc_attr( $item->attr_title ) . '"></span>&nbsp;';
+			if ( ! empty($item->attr_title)) {
+				$item_output .= '<a' . $attributes . '"><span class="glyphicon ' . esc_attr($item->attr_title) . '"></span>&nbsp;';
 			} else {
-				$item_output .= '<a'. $attributes .'>';
+				$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->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 );
+			$output .= apply_filters('walker_nav_menu_start_el', $item_output, $item, $depth, $args);
 		}
 	}
   
@@ -157,17 +157,17 @@  discard block
 block discarded – undo
 	 * @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 )
+	public function display_element($element, &$children_elements, $max_depth, $depth, $args, &$output) {
+        if ( ! $element)
             return;
 
         $id_field = $this->db_fields['id'];
 
         // Display this element.
-        if ( is_object( $args[0] ) )
-           $args[0]->has_children = ! empty( $children_elements[ $element->$id_field ] );
+        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 );
+        parent::display_element($element, $children_elements, $max_depth, $depth, $args, $output);
     }
 
 	/**
@@ -181,17 +181,17 @@  discard block
 block discarded – undo
 	 * @param array $args passed from the wp_nav_menu function.
 	 *
 	 */
-	public static function fallback( $args ) {
-		if ( current_user_can( 'manage_options' ) ) {
+	public static function fallback($args) {
+		if (current_user_can('manage_options')) {
 			$fb_output = null;
 
-			if ( $args['container'] ) {
+			if ($args['container']) {
 				$fb_output = '<' . $args['container'];
 
-				if ( $args['container_id'] )
+				if ($args['container_id'])
 					$fb_output .= ' id="' . $args['container_id'] . '"';
 
-				if ( $args['container_class'] )
+				if ($args['container_class'])
 					$fb_output .= ' class="' . $args['container_class'] . '"';
 
 				$fb_output .= '>';
@@ -199,20 +199,20 @@  discard block
 block discarded – undo
 
 			$fb_output .= '<ul';
 
-			if ( $args['menu_id'] )
+			if ($args['menu_id'])
 				$fb_output .= ' id="' . $args['menu_id'] . '"';
 
-			if ( $args['menu_class'] )
+			if ($args['menu_class'])
 				$fb_output .= ' class="' . $args['menu_class'] . '"';
 
 			$fb_output .= '>';
-			$fb_output .= '<li><a href="' . admin_url( 'nav-menus.php' ) . '">' . esc_html__( 'Add a menu', 'lsx' ) . '</a></li>';
+			$fb_output .= '<li><a href="' . admin_url('nav-menus.php') . '">' . esc_html__('Add a menu', 'lsx') . '</a></li>';
 			$fb_output .= '</ul>';
 
-			if ( $args['container'] )
+			if ($args['container'])
 				$fb_output .= '</' . $args['container'] . '>';
 
-			echo wp_kses_post( $fb_output );
+			echo wp_kses_post($fb_output);
 		}
 	}
 }
@@ -221,9 +221,9 @@  discard block
 block discarded – undo
 /**
  * Add in our custom classes to the menus
  */
-function wpml_nav_language_switcher_fix( $items , $args ) {
-	$items = str_replace('menu-item-language-current','menu-item-language-current dropdown',$items);
-	$items = str_replace('submenu-languages','submenu-languages dropdown-menu',$items);
+function wpml_nav_language_switcher_fix($items, $args) {
+	$items = str_replace('menu-item-language-current', 'menu-item-language-current dropdown', $items);
+	$items = str_replace('submenu-languages', 'submenu-languages dropdown-menu', $items);
 	return $items;
 }
-add_filter( 'wp_nav_menu_items', 'wpml_nav_language_switcher_fix', 10, 2 );
+add_filter('wp_nav_menu_items', 'wpml_nav_language_switcher_fix', 10, 2);

Please login to merge, or discard this patch.

page-templates/template-archives.php 1 patch

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

@@ -5,7 +5,7 @@  discard block
 block discarded – undo
 
 	<?php lsx_content_wrap_before(); ?>
 
-	<div id="primary" class="content-area <?php echo esc_attr( lsx_main_class() ); ?>">
+	<div id="primary" class="content-area <?php echo esc_attr(lsx_main_class()); ?>">
 
 		<?php lsx_content_before(); ?>
 
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
 
 			<?php lsx_content_top(); ?>
 
-			<?php while ( have_posts() ) : the_post(); ?>
+			<?php while (have_posts()) : the_post(); ?>
 
 				<?php lsx_entry_before(); ?>
 
@@ -22,28 +22,28 @@  discard block
 block discarded – undo
 					<?php lsx_entry_top(); ?>
 
 					<div class="entry-content">
-						<h2><?php esc_html_e( 'The Last 30 Posts', 'lsx' ); ?></h3>
+						<h2><?php esc_html_e('The Last 30 Posts', 'lsx'); ?></h3>
                         <ul>
-                            <?php $loop = new WP_Query( array( 'posts_per_page' => 30 ) ); ?>
-                            <?php if ( $loop->have_posts() ) { while ( $loop->have_posts() ) { $loop->the_post(); ?>
+                            <?php $loop = new WP_Query(array('posts_per_page' => 30)); ?>
+                            <?php if ($loop->have_posts()) { while ($loop->have_posts()) { $loop->the_post(); ?>
                                 <?php $loop->is_home = false; ?>
-                                <li><a href="<?php the_permalink(); ?>"><?php the_title(); ?></a> - <?php the_time( get_option( 'date_format' ) ); ?> - <?php echo esc_html( $post->comment_count ); ?> <?php esc_html_e( 'comments', 'lsx' ); ?></li>
+                                <li><a href="<?php the_permalink(); ?>"><?php the_title(); ?></a> - <?php the_time(get_option('date_format')); ?> - <?php echo esc_html($post->comment_count); ?> <?php esc_html_e('comments', 'lsx'); ?></li>
                             <?php } } wp_reset_postdata(); ?>
                         </ul>
 
-                        <h2><?php esc_html_e( 'Categories', 'lsx' ); ?></h3>
+                        <h2><?php esc_html_e('Categories', 'lsx'); ?></h3>
 
                         <ul>
-                            <?php wp_list_categories( 'title_li=&hierarchical=0&show_count=1' ); ?>
+                            <?php wp_list_categories('title_li=&hierarchical=0&show_count=1'); ?>
                         </ul>
 
-                        <h2><?php esc_html_e( 'Monthly Archives', 'lsx' ); ?></h3>
+                        <h2><?php esc_html_e('Monthly Archives', 'lsx'); ?></h3>
 
                         <ul>
-                            <?php wp_get_archives( 'type=monthly&show_post_count=1' ); ?>
+                            <?php wp_get_archives('type=monthly&show_post_count=1'); ?>
                         </ul>
 					</div><!-- .entry-content -->
-					<?php edit_post_link( esc_html__( 'Edit', 'lsx' ), '<footer class="entry-meta"><span class="edit-link">', '</span></footer>' ); ?>
+					<?php edit_post_link(esc_html__('Edit', 'lsx'), '<footer class="entry-meta"><span class="edit-link">', '</span></footer>'); ?>
 
 					<?php lsx_entry_bottom(); ?>
 					

Please login to merge, or discard this patch.

page-templates/template-portfolio.php 2 patches

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

@@ -17,8 +17,8 @@  discard block
 block discarded – undo
 
 			<?php lsx_content_top(); ?>
 
-			<?php if(have_posts()) { ?>
-				<?php while(have_posts()) { the_post(); ?>
+			<?php if (have_posts()) { ?>
+				<?php while (have_posts()) { the_post(); ?>
 					<div class="entry-content">
 						
 						<?php the_content(); ?>
@@ -29,10 +29,10 @@  discard block
 block discarded – undo
 			<?php } ?>
 
 			<?php
-				if ( get_query_var( 'paged' ) ) :
-					$paged = get_query_var( 'paged' );
-				elseif ( get_query_var( 'page' ) ) :
-					$paged = get_query_var( 'page' );
+				if (get_query_var('paged')) :
+					$paged = get_query_var('paged');
+				elseif (get_query_var('page')) :
+					$paged = get_query_var('page');
 				else :
 					$paged = 1;
 				endif;
@@ -41,18 +41,18 @@  discard block
 block discarded – undo
 					'post_type'      => 'jetpack-portfolio',
 					'posts_per_page' => 99,
 				);
-				$project_query = new WP_Query ( $args );
-				if ( post_type_exists( 'jetpack-portfolio' ) && $project_query -> have_posts() ) :
+				$project_query = new WP_Query($args);
+				if (post_type_exists('jetpack-portfolio') && $project_query -> have_posts()) :
 			?>
 			
 				<?php lsx_portfolio_sorter(); ?>
 
 				<div class="filter-items-wrapper lsx-portfolio-wrapper">
 					<div class="filter-items-container lsx-portfolio masonry">
-						<?php while ( $project_query -> have_posts() ) : $project_query -> the_post(); ?>
+						<?php while ($project_query -> have_posts()) : $project_query -> the_post(); ?>
 
-							<?php if(has_post_thumbnail()) { ?>
-								<?php get_template_part( 'content', 'portfolio' ); ?>
+							<?php if (has_post_thumbnail()) { ?>
+								<?php get_template_part('content', 'portfolio'); ?>
 							<?php } ?>
 
 						<?php endwhile; ?>
@@ -63,17 +63,17 @@  discard block
 block discarded – undo
 
 				<section class="no-results not-found">
 					<header class="page-header">
-						<h1 class="page-title"><?php esc_html_e( 'Nothing Found', 'lsx' ); ?></h1>
+						<h1 class="page-title"><?php esc_html_e('Nothing Found', 'lsx'); ?></h1>
 					</header><!-- .page-header -->
 
 					<div class="page-content">
-						<?php if ( current_user_can( 'publish_posts' ) ) : ?>
+						<?php if (current_user_can('publish_posts')) : ?>
 
-							<p><?php esc_html_e( 'Ready to publish your first project?', 'lsx' ); ?> <a href="<?php echo esc_url( admin_url( 'post-new.php?post_type=jetpack-portfolio' ) ) ?>"><?php esc_html_e( 'Get started here', 'lsx' ); ?></a></p>
+							<p><?php esc_html_e('Ready to publish your first project?', 'lsx'); ?> <a href="<?php echo esc_url(admin_url('post-new.php?post_type=jetpack-portfolio')) ?>"><?php esc_html_e('Get started here', 'lsx'); ?></a></p>
 
 						<?php else : ?>
 
-							<p><?php esc_html_e( 'It seems we can&rsquo;t find what you&rsquo;re looking for. Perhaps searching can help.', 'lsx' ); ?></p>
+							<p><?php esc_html_e('It seems we can&rsquo;t find what you&rsquo;re looking for. Perhaps searching can help.', 'lsx'); ?></p>
 							<?php get_search_form(); ?>
 
 						<?php endif; ?>

Please login to merge, or discard this patch.

Braces +13 added lines, -5 removed lines patch added patch discarded remove patch

@@ -33,8 +33,10 @@  discard block
 block discarded – undo
 					$paged = get_query_var( 'paged' );
 				elseif ( get_query_var( 'page' ) ) :
 					$paged = get_query_var( 'page' );
-				else :
+				else {
+					:
 					$paged = 1;
+				}
 				endif;
 
 				$args = array(
@@ -59,11 +61,14 @@  discard block
 block discarded – undo
 					</div>
 				</div><!-- .portfolio-wrapper -->
 
-			<?php else : ?>
+			<?php else {
+	: ?>
 
 				<section class="no-results not-found">
 					<header class="page-header">
-						<h1 class="page-title"><?php esc_html_e( 'Nothing Found', 'lsx' ); ?></h1>
+						<h1 class="page-title"><?php esc_html_e( 'Nothing Found', 'lsx' );
+}
+?></h1>
 					</header><!-- .page-header -->
 
 					<div class="page-content">
@@ -71,9 +76,12 @@  discard block
 block discarded – undo
 
 							<p><?php esc_html_e( 'Ready to publish your first project?', 'lsx' ); ?> <a href="<?php echo esc_url( admin_url( 'post-new.php?post_type=jetpack-portfolio' ) ) ?>"><?php esc_html_e( 'Get started here', 'lsx' ); ?></a></p>
 
-						<?php else : ?>
+						<?php else {
+	: ?>
 
-							<p><?php esc_html_e( 'It seems we can&rsquo;t find what you&rsquo;re looking for. Perhaps searching can help.', 'lsx' ); ?></p>
+							<p><?php esc_html_e( 'It seems we can&rsquo;t find what you&rsquo;re looking for. Perhaps searching can help.', 'lsx' );
+}
+?></p>
 							<?php get_search_form(); ?>
 
 						<?php endif; ?>

Please login to merge, or discard this patch.

		@@ -11,7 +11,7 @@ discard block
		block discarded – undo
11	11
12	12	<?php lsx_content_wrap_before(); ?>
13	13
14		- <div id="primary" class="content-area <?php echo esc_attr( lsx_main_class() ); ?>">
	14	+ <div id="primary" class="content-area <?php echo esc_attr(lsx_main_class()); ?>">
15	15
16	16	<?php lsx_content_before(); ?>
17	17
		@@ -19,25 +19,25 @@ discard block
		block discarded – undo
19	19
20	20	<?php lsx_content_top(); ?>
21	21
22		- <?php if ( have_posts() ) : ?>
	22	+ <?php if (have_posts()) : ?>
23	23
24	24	<?php
25		- $layout = get_theme_mod('lsx_layout','2cr');
26		- $layout = apply_filters( 'lsx_layout', $layout );
27		- if('1c' === $layout){
	25	+ $layout = get_theme_mod('lsx_layout', '2cr');
	26	+ $layout = apply_filters('lsx_layout', $layout);
	27	+ if ('1c' === $layout) {
28	28	lsx_breadcrumbs();
29	29	}
30	30	?>
31	31
32	32	<?php /* Start the Loop */ ?>
33		- <?php while ( have_posts() ) : the_post(); ?>
	33	+ <?php while (have_posts()) : the_post(); ?>
34	34
35	35	<?php
36	36	/* Include the Post-Format-specific template for the content.
37	37	* If you want to override this in a child theme, then include a file
38	38	* called content-___.php (where ___ is the Post Format name) and that will be used instead.
39	39	*/
40		- get_template_part( 'content', get_post_format() );
	40	+ get_template_part('content', get_post_format());
41	41	?>
42	42
43	43	<?php endwhile; ?>
		@@ -46,7 +46,7 @@ discard block
		block discarded – undo
46	46
47	47	<?php else : ?>
48	48
49		- <?php get_template_part( 'content', 'none' ); ?>
	49	+ <?php get_template_part('content', 'none'); ?>
50	50
51	51	<?php endif; ?>
52	52

		@@ -47,9 +47,12 @@
		block discarded – undo
47	47
48	48	<?php lsx_paging_nav(); ?>
49	49
50		- <?php else : ?>
	50	+ <?php else {
	51	+ : ?>
51	52
52		- <?php get_template_part( 'content', 'none' ); ?>
	53	+ <?php get_template_part( 'content', 'none' );
	54	+}
	55	+?>
53	56
54	57	<?php endif; ?>
55	58

		@@ -15,7 +15,7 @@ discard block
		block discarded – undo
15	15
16	16	<?php lsx_content_wrap_before(); ?>
17	17
18		- <div id="primary" class="content-area <?php echo esc_attr( lsx_main_class() ); ?>">
	18	+ <div id="primary" class="content-area <?php echo esc_attr(lsx_main_class()); ?>">
19	19
20	20	<?php lsx_content_before(); ?>
21	21
		@@ -23,17 +23,17 @@ discard block
		block discarded – undo
23	23
24	24	<?php lsx_content_top(); ?>
25	25
26		- <?php if ( have_posts() ) : ?>
	26	+ <?php if (have_posts()) : ?>
27	27
28	28	<?php /* Start the Loop */ ?>
29		- <?php while ( have_posts() ) : the_post(); ?>
	29	+ <?php while (have_posts()) : the_post(); ?>
30	30
31	31	<?php
32	32	/* Include the Post-Format-specific template for the content.
33	33	* If you want to override this in a child theme, then include a file
34	34	* called content-___.php (where ___ is the Post Format name) and that will be used instead.
35	35	*/
36		- get_template_part( 'content', get_post_format() );
	36	+ get_template_part('content', get_post_format());
37	37	?>
38	38
39	39	<?php endwhile; ?>
		@@ -42,7 +42,7 @@ discard block
		block discarded – undo
42	42
43	43	<?php else : ?>
44	44
45		- <?php get_template_part( 'content', 'none' ); ?>
	45	+ <?php get_template_part('content', 'none'); ?>
46	46
47	47	<?php endif; ?>
48	48
		@@ -56,5 +56,5 @@ discard block
		block discarded – undo
56	56
57	57	<?php lsx_content_wrap_after(); ?>
58	58
59		-<?php get_sidebar( 'sidebar' ); ?>
	59	+<?php get_sidebar('sidebar'); ?>
60	60	<?php get_footer();
61	61	\ No newline at end of file

		@@ -12,16 +12,16 @@ discard block
		block discarded – undo
12	12	<html <?php language_attributes(); ?>>
13	13	<head>
14	14	<?php lsx_head_top(); ?>
15		-<meta charset="<?php bloginfo( 'charset' ); ?>">
	15	+<meta charset="<?php bloginfo('charset'); ?>">
16	16	<link rel="profile" href="http://gmpg.org/xfn/11">
17		-<link rel="pingback" href="<?php bloginfo( 'pingback_url' ); ?>">
	17	+<link rel="pingback" href="<?php bloginfo('pingback_url'); ?>">
18	18	<?php lsx_head_bottom(); ?>
19	19	<?php wp_head(); ?>
20	20	</head>
21		-<body <?php body_class( 'lsx' ); ?>>
	21	+<body <?php body_class('lsx'); ?>>
22	22	<?php lsx_body_top(); ?>
23	23
24		-<?php if ( !is_singular('landing-page') ) { ?>
	24	+<?php if ( ! is_singular('landing-page')) { ?>
25	25
26	26	<?php lsx_header_before(); ?>
27	27	<header class="<?php lsx_header_classes(); ?>">
		@@ -57,17 +57,17 @@ discard block
		block discarded – undo
57	57	</div>
58	58
59	59	<div class="header-links">
60		- <strong><?php esc_html_e( 'Get your Tourism Establishment Online', 'lsx' ); ?></strong>
	60	+ <strong><?php esc_html_e('Get your Tourism Establishment Online', 'lsx'); ?></strong>
61	61
62	62	<?php
63		- if(is_singular('landing-page')){
64		- $email_address = get_post_meta(get_the_ID(),'email_address',true);
65		- if(false === $email_address){
	63	+ if (is_singular('landing-page')) {
	64	+ $email_address = get_post_meta(get_the_ID(), 'email_address', true);
	65	+ if (false === $email_address) {
66	66	$email_address = '[email protected]';
67	67	}
68	68	} ?>
69	69
70		- <span class="email-address"><?php esc_html_e( 'Questions? Email Us: ', 'lsx' ); ?><a href="mailto:<?php echo esc_attr( $email_address ); ?>"><?php echo esc_html( $email_address ); ?></a></span>
	70	+ <span class="email-address"><?php esc_html_e('Questions? Email Us: ', 'lsx'); ?><a href="mailto:<?php echo esc_attr($email_address); ?>"><?php echo esc_html($email_address); ?></a></span>
71	71	</div>
72	72	</div>
73	73

		@@ -60,12 +60,12 @@
		block discarded – undo
60	60	<strong><?php esc_html_e( 'Get your Tourism Establishment Online', 'lsx' ); ?></strong>
61	61
62	62	<?php
63		- if(is_singular('landing-page')){
64		- $email_address = get_post_meta(get_the_ID(),'email_address',true);
65		- if(false === $email_address){
	63	+ if(is_singular('landing-page')){
	64	+ $email_address = get_post_meta(get_the_ID(),'email_address',true);
	65	+ if(false === $email_address){
66	66	$email_address = '[email protected]';
67	67	}
68		- } ?>
	68	+ } ?>
69	69
70	70	<span class="email-address"><?php esc_html_e( 'Questions? Email Us: ', 'lsx' ); ?><a href="mailto:<?php echo esc_attr( $email_address ); ?>"><?php echo esc_html( $email_address ); ?></a></span>
71	71	</div>

		@@ -10,22 +10,22 @@
		block discarded – undo
10	10
11	11	<section class="no-results not-found">
12	12	<header class="page-header">
13		- <h1 class="page-title"><?php esc_html_e( 'Nothing Found', 'lsx' ); ?></h1>
	13	+ <h1 class="page-title"><?php esc_html_e('Nothing Found', 'lsx'); ?></h1>
14	14	</header><!-- .page-header -->
15	15
16	16	<div class="page-content">
17		- <?php if ( is_home() && current_user_can( 'publish_posts' ) ) : ?>
	17	+ <?php if (is_home() && current_user_can('publish_posts')) : ?>
18	18
19		- <p><?php esc_html_e( 'Ready to publish your first post?', 'lsx' ); ?> <a href="<?php echo esc_url( admin_url( 'post-new.php' ) ) ?>"><?php esc_html_e( 'Get started here', 'lsx' ); ?></a></p>
	19	+ <p><?php esc_html_e('Ready to publish your first post?', 'lsx'); ?> <a href="<?php echo esc_url(admin_url('post-new.php')) ?>"><?php esc_html_e('Get started here', 'lsx'); ?></a></p>
20	20
21		- <?php elseif ( is_search() ) : ?>
	21	+ <?php elseif (is_search()) : ?>
22	22
23		- <p><?php esc_html_e( 'Sorry, but nothing matched your search terms. Please try again with some different keywords.', 'lsx' ); ?></p>
	23	+ <p><?php esc_html_e('Sorry, but nothing matched your search terms. Please try again with some different keywords.', 'lsx'); ?></p>
24	24	<?php get_search_form(); ?>
25	25
26	26	<?php else : ?>
27	27
28		- <p><?php esc_html_e( 'It seems we can’t find what you’re looking for. Perhaps searching can help.', 'lsx' ); ?></p>
	28	+ <p><?php esc_html_e('It seems we can’t find what you’re looking for. Perhaps searching can help.', 'lsx'); ?></p>
29	29	<?php get_search_form(); ?>
30	30
31	31	<?php endif; ?>

		@@ -23,9 +23,12 @@
		block discarded – undo
23	23	<p><?php esc_html_e( 'Sorry, but nothing matched your search terms. Please try again with some different keywords.', 'lsx' ); ?></p>
24	24	<?php get_search_form(); ?>
25	25
26		- <?php else : ?>
	26	+ <?php else {
	27	+ : ?>
27	28
28		- <p><?php esc_html_e( 'It seems we can’t find what you’re looking for. Perhaps searching can help.', 'lsx' ); ?></p>
	29	+ <p><?php esc_html_e( 'It seems we can’t find what you’re looking for. Perhaps searching can help.', 'lsx' );
	30	+}
	31	+?></p>
29	32	<?php get_search_form(); ?>
30	33
31	34	<?php endif; ?>

		@@ -24,11 +24,11 @@ discard block
		block discarded – undo
24	24	* $lsx_supports[] = 'body';
25	25	*/
26	26	function lsx_body_top() {
27		- do_action( 'lsx_body_top' );
	27	+ do_action('lsx_body_top');
28	28	}
29	29
30	30	function lsx_body_bottom() {
31		- do_action( 'lsx_body_bottom' );
	31	+ do_action('lsx_body_bottom');
32	32	}
33	33
34	34	/**
		@@ -37,11 +37,11 @@ discard block
		block discarded – undo
37	37	* $lsx_supports[] = 'head';
38	38	*/
39	39	function lsx_head_top() {
40		- do_action( 'lsx_head_top' );
	40	+ do_action('lsx_head_top');
41	41	}
42	42
43	43	function lsx_head_bottom() {
44		- do_action( 'lsx_head_bottom' );
	44	+ do_action('lsx_head_bottom');
45	45	}
46	46
47	47	/**
		@@ -50,27 +50,27 @@ discard block
		block discarded – undo
50	50	* $lsx_supports[] = 'header';
51	51	*/
52	52	function lsx_header_before() {
53		- do_action( 'lsx_header_before' );
	53	+ do_action('lsx_header_before');
54	54	}
55	55
56	56	function lsx_header_after() {
57		- do_action( 'lsx_header_after' );
	57	+ do_action('lsx_header_after');
58	58	}
59	59
60	60	function lsx_header_top() {
61		- do_action( 'lsx_header_top' );
	61	+ do_action('lsx_header_top');
62	62	}
63	63
64	64	function lsx_header_bottom() {
65		- do_action( 'lsx_header_bottom' );
	65	+ do_action('lsx_header_bottom');
66	66	}
67	67
68	68	function lsx_nav_before() {
69		- do_action( 'lsx_nav_before' );
	69	+ do_action('lsx_nav_before');
70	70	}
71	71
72	72	function lsx_nav_after() {
73		- do_action( 'lsx_nav_after' );
	73	+ do_action('lsx_nav_after');
74	74	}
75	75
76	76	/**
		@@ -80,7 +80,7 @@ discard block
		block discarded – undo
80	80	*/
81	81
82	82	function lsx_banner_content() {
83		- do_action( 'lsx_banner_content' );
	83	+ do_action('lsx_banner_content');
84	84	}
85	85
86	86	/**
		@@ -89,35 +89,35 @@ discard block
		block discarded – undo
89	89	* $lsx_supports[] = 'content';
90	90	*/
91	91	function lsx_content_wrap_before() {
92		- do_action( 'lsx_content_wrap_before' );
	92	+ do_action('lsx_content_wrap_before');
93	93	}
94	94
95	95	function lsx_content_wrap_after() {
96		- do_action( 'lsx_content_wrap_after' );
	96	+ do_action('lsx_content_wrap_after');
97	97	}
98	98
99	99	function lsx_content_before() {
100		- do_action( 'lsx_content_before' );
	100	+ do_action('lsx_content_before');
101	101	}
102	102
103	103	function lsx_content_after() {
104		- do_action( 'lsx_content_after' );
	104	+ do_action('lsx_content_after');
105	105	}
106	106
107	107	function lsx_content_top() {
108		- do_action( 'lsx_content_top' );
	108	+ do_action('lsx_content_top');
109	109	}
110	110
111	111	function lsx_content_bottom() {
112		- do_action( 'lsx_content_bottom' );
	112	+ do_action('lsx_content_bottom');
113	113	}
114	114
115	115	function lsx_content_post_meta() {
116		- do_action( 'lsx_content_post_meta' );
	116	+ do_action('lsx_content_post_meta');
117	117	}
118	118
119	119	function lsx_content_post_tags() {
120		- do_action( 'lsx_content_post_tags' );
	120	+ do_action('lsx_content_post_tags');
121	121	}
122	122
123	123	/**
		@@ -126,19 +126,19 @@ discard block
		block discarded – undo
126	126	* $lsx_supports[] = 'entry';
127	127	*/
128	128	function lsx_entry_before() {
129		- do_action( 'lsx_entry_before' );
	129	+ do_action('lsx_entry_before');
130	130	}
131	131
132	132	function lsx_entry_after() {
133		- do_action( 'lsx_entry_after' );
	133	+ do_action('lsx_entry_after');
134	134	}
135	135
136	136	function lsx_entry_top() {
137		- do_action( 'lsx_entry_top' );
	137	+ do_action('lsx_entry_top');
138	138	}
139	139
140	140	function lsx_entry_bottom() {
141		- do_action( 'lsx_entry_bottom' );
	141	+ do_action('lsx_entry_bottom');
142	142	}
143	143
144	144	/**
		@@ -147,11 +147,11 @@ discard block
		block discarded – undo
147	147	* $lsx_supports[] = 'comments';
148	148	*/
149	149	function lsx_comments_before() {
150		- do_action( 'lsx_comments_before' );
	150	+ do_action('lsx_comments_before');
151	151	}
152	152
153	153	function lsx_comments_after() {
154		- do_action( 'lsx_comments_after' );
	154	+ do_action('lsx_comments_after');
155	155	}
156	156
157	157	/**
		@@ -160,19 +160,19 @@ discard block
		block discarded – undo
160	160	* $lsx_supports[] = 'sidebar';
161	161	*/
162	162	function lsx_sidebars_before() {
163		- do_action( 'lsx_sidebars_before' );
	163	+ do_action('lsx_sidebars_before');
164	164	}
165	165
166	166	function lsx_sidebars_after() {
167		- do_action( 'lsx_sidebars_after' );
	167	+ do_action('lsx_sidebars_after');
168	168	}
169	169
170	170	function lsx_sidebar_top() {
171		- do_action( 'lsx_sidebar_top' );
	171	+ do_action('lsx_sidebar_top');
172	172	}
173	173
174	174	function lsx_sidebar_bottom() {
175		- do_action( 'lsx_sidebar_bottom' );
	175	+ do_action('lsx_sidebar_bottom');
176	176	}
177	177
178	178	/**
		@@ -181,17 +181,17 @@ discard block
		block discarded – undo
181	181	* $lsx_supports[] = 'footer';
182	182	*/
183	183	function lsx_footer_before() {
184		- do_action( 'lsx_footer_before' );
	184	+ do_action('lsx_footer_before');
185	185	}
186	186
187	187	function lsx_footer_after() {
188		- do_action( 'lsx_footer_after' );
	188	+ do_action('lsx_footer_after');
189	189	}
190	190
191	191	function lsx_footer_top() {
192		- do_action( 'lsx_footer_top' );
	192	+ do_action('lsx_footer_top');
193	193	}
194	194
195	195	function lsx_footer_bottom() {
196		- do_action( 'lsx_footer_bottom' );
	196	+ do_action('lsx_footer_bottom');
197	197	}
198	198	\ No newline at end of file

		@@ -9,7 +9,7 @@ discard block
		block discarded – undo
9	9
10	10	<?php lsx_content_wrap_before(); ?>
11	11
12		- <div id="primary" class="content-area <?php echo esc_attr( lsx_main_class() ); ?>">
	12	+ <div id="primary" class="content-area <?php echo esc_attr(lsx_main_class()); ?>">
13	13
14	14	<?php lsx_content_before(); ?>
15	15
		@@ -17,25 +17,25 @@ discard block
		block discarded – undo
17	17
18	18	<?php lsx_content_top(); ?>
19	19
20		- <?php if ( have_posts() ) : global $lsx_archive; $lsx_archive = 1; ?>
	20	+ <?php if (have_posts()) : global $lsx_archive; $lsx_archive = 1; ?>
21	21
22	22	<?php
23		- $layout = get_theme_mod('lsx_layout','2cr');
24		- $layout = apply_filters( 'lsx_layout', $layout );
25		- if('1c' === $layout){
	23	+ $layout = get_theme_mod('lsx_layout', '2cr');
	24	+ $layout = apply_filters('lsx_layout', $layout);
	25	+ if ('1c' === $layout) {
26	26	lsx_breadcrumbs();
27	27	}
28	28	?>
29	29
30	30	<?php /* Start the Loop */ ?>
31		- <?php while ( have_posts() ) : the_post(); ?>
	31	+ <?php while (have_posts()) : the_post(); ?>
32	32
33	33	<?php
34	34	/* Include the Post-Format-specific template for the content.
35	35	* If you want to override this in a child theme, then include a file
36	36	* called content-___.php (where ___ is the Post Format name) and that will be used instead.
37	37	*/
38		- get_template_part( 'content', get_post_format() );
	38	+ get_template_part('content', get_post_format());
39	39	?>
40	40
41	41	<?php endwhile; ?>
		@@ -44,7 +44,7 @@ discard block
		block discarded – undo
44	44
45	45	<?php else : ?>
46	46
47		- <?php get_template_part( 'content', 'none' ); ?>
	47	+ <?php get_template_part('content', 'none'); ?>
48	48
49	49	<?php endif; $lsx_archive = 0; ?>
50	50

		@@ -42,9 +42,12 @@
		block discarded – undo
42	42
43	43	<?php lsx_paging_nav(); ?>
44	44
45		- <?php else : ?>
	45	+ <?php else {
	46	+ : ?>
46	47
47		- <?php get_template_part( 'content', 'none' ); ?>
	48	+ <?php get_template_part( 'content', 'none' );
	49	+}
	50	+?>
48	51
49	52	<?php endif; $lsx_archive = 0; ?>
50	53

		@@ -30,7 +30,6 @@
		block discarded – undo
30	30	* @param string $output Passed by reference. Used to append additional content.
31	31	* @param object $item Menu item data object.
32	32	* @param int $depth Depth of menu item. Used for padding.
33		- * @param int $current_page Menu item ID.
34	33	* @param object $args
35	34	*/
36	35	public function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0 ) {

		@@ -159,17 +159,17 @@
		block discarded – undo
159	159	* @return null Null on failure with no changes to parameters.
160	160	*/
161	161	public function display_element( $element, &$children_elements, $max_depth, $depth, $args, &$output ) {
162		- if ( ! $element )
163		- return;
	162	+ if ( ! $element )
	163	+ return;
164	164
165		- $id_field = $this->db_fields['id'];
	165	+ $id_field = $this->db_fields['id'];
166	166
167		- // Display this element.
168		- if ( is_object( $args[0] ) )
169		- $args[0]->has_children = ! empty( $children_elements[ $element->$id_field ] );
	167	+ // Display this element.
	168	+ if ( is_object( $args[0] ) )
	169	+ $args[0]->has_children = ! empty( $children_elements[ $element->$id_field ] );
170	170
171		- parent::display_element( $element, $children_elements, $max_depth, $depth, $args, $output );
172		- }
	171	+ parent::display_element( $element, $children_elements, $max_depth, $depth, $args, $output );
	172	+ }
173	173
174	174	/**
175	175	* Menu Fallback

		@@ -1,5 +1,8 @@ discard block
		block discarded – undo
1	1	<?php
2		-if ( ! defined( 'ABSPATH' ) ) return; // Exit if accessed directly
	2	+if ( ! defined( 'ABSPATH' ) ) {
	3	+ return;
	4	+}
	5	+// Exit if accessed directly
3	6
4	7	class Lsx_Bootstrap_Navwalker extends Walker_Nav_Menu {
5	8
		@@ -71,14 +74,17 @@ discard block
		block discarded – undo
71	74
72	75	$class_names = join( ' ', $classes );
73	76
74		- if ( $args->has_children )
75		- $class_names .= ' dropdown';
	77	+ if ( $args->has_children ) {
	78	+ $class_names .= ' dropdown';
	79	+ }
76	80
77		- if ( in_array( 'current-menu-item', $classes ) )
78		- $class_names .= ' active';
	81	+ if ( in_array( 'current-menu-item', $classes ) ) {
	82	+ $class_names .= ' active';
	83	+ }
79	84
80		- if ( in_array( 'current-menu-parent', $classes ) )
81		- $class_names .= ' active';
	85	+ if ( in_array( 'current-menu-parent', $classes ) ) {
	86	+ $class_names .= ' active';
	87	+ }
82	88
83	89	//Check if this is ment to be a "social" type menu
84	90	$class_names = $class_names ? ' class="' . esc_attr( $class_names ) . '"' : '';
		@@ -159,14 +165,16 @@ discard block
		block discarded – undo
159	165	* @return null Null on failure with no changes to parameters.
160	166	*/
161	167	public function display_element( $element, &$children_elements, $max_depth, $depth, $args, &$output ) {
162		- if ( ! $element )
163		- return;
	168	+ if ( ! $element ) {
	169	+ return;
	170	+ }
164	171
165	172	$id_field = $this->db_fields['id'];
166	173
167	174	// Display this element.
168		- if ( is_object( $args[0] ) )
169		- $args[0]->has_children = ! empty( $children_elements[ $element->$id_field ] );
	175	+ if ( is_object( $args[0] ) ) {
	176	+ $args[0]->has_children = ! empty( $children_elements[ $element->$id_field ] );
	177	+ }
170	178
171	179	parent::display_element( $element, $children_elements, $max_depth, $depth, $args, $output );
172	180	}
		@@ -189,29 +197,34 @@ discard block
		block discarded – undo
189	197	if ( $args['container'] ) {
190	198	$fb_output = '<' . $args['container'];
191	199
192		- if ( $args['container_id'] )
193		- $fb_output .= ' id="' . $args['container_id'] . '"';
	200	+ if ( $args['container_id'] ) {
	201	+ $fb_output .= ' id="' . $args['container_id'] . '"';
	202	+ }
194	203
195		- if ( $args['container_class'] )
196		- $fb_output .= ' class="' . $args['container_class'] . '"';
	204	+ if ( $args['container_class'] ) {
	205	+ $fb_output .= ' class="' . $args['container_class'] . '"';
	206	+ }
197	207
198	208	$fb_output .= '>';
199	209	}
200	210
201	211	$fb_output .= '<ul';
202	212
203		- if ( $args['menu_id'] )
204		- $fb_output .= ' id="' . $args['menu_id'] . '"';
	213	+ if ( $args['menu_id'] ) {
	214	+ $fb_output .= ' id="' . $args['menu_id'] . '"';
	215	+ }
205	216
206		- if ( $args['menu_class'] )
207		- $fb_output .= ' class="' . $args['menu_class'] . '"';
	217	+ if ( $args['menu_class'] ) {
	218	+ $fb_output .= ' class="' . $args['menu_class'] . '"';
	219	+ }
208	220
209	221	$fb_output .= '>';
210	222	$fb_output .= '<li><a href="' . admin_url( 'nav-menus.php' ) . '">' . esc_html__( 'Add a menu', 'lsx' ) . '</a></li>';
211	223	$fb_output .= '</ul>';
212	224
213		- if ( $args['container'] )
214		- $fb_output .= '</' . $args['container'] . '>';
	225	+ if ( $args['container'] ) {
	226	+ $fb_output .= '</' . $args['container'] . '>';
	227	+ }
215	228
216	229	echo wp_kses_post( $fb_output );
217	230	}

		@@ -1,5 +1,5 @@ discard block
		block discarded – undo
1	1	<?php
2		-if ( ! defined( 'ABSPATH' ) ) return; // Exit if accessed directly
	2	+if ( ! defined('ABSPATH')) return; // Exit if accessed directly
3	3
4	4	class Lsx_Bootstrap_Navwalker extends Walker_Nav_Menu {
5	5
		@@ -10,15 +10,15 @@ discard block
		block discarded – undo
10	10	* @param string $output Passed by reference. Used to append additional content.
11	11	* @param int $depth Depth of page. Used for padding.
12	12	*/
13		- public function start_lvl( &$output, $depth = 0, $args = array() ) {
14		- $indent = str_repeat( "\t", $depth );
	13	+ public function start_lvl(&$output, $depth = 0, $args = array()) {
	14	+ $indent = str_repeat("\t", $depth);
15	15	$output .= "\n$indent<ul role=\"menu\" class=\" dropdown-menu\">\n";
16	16	}
17	17
18	18	/**
19	19	* @param string $item Passed by reference. Used to append additional content.
20	20	*/
21		- public function filter_default_pages( &$item ) {
	21	+ public function filter_default_pages(&$item) {
22	22
23	23	return $item;
24	24	}
		@@ -33,14 +33,14 @@ discard block
		block discarded – undo
33	33	* @param int $current_page Menu item ID.
34	34	* @param object $args
35	35	*/
36		- public function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0 ) {
37		- $indent = ( $depth ) ? str_repeat( "\t", $depth ) : '';
	36	+ public function start_el(&$output, $item, $depth = 0, $args = array(), $id = 0) {
	37	+ $indent = ($depth) ? str_repeat("\t", $depth) : '';
38	38
39	39	/**
40	40	* If this is a default menu being called we need to fix
41	41	* the item object thats coming through.
42	42	*/
43		- if(!isset($item->title)){
	43	+ if ( ! isset($item->title)) {
44	44	return;
45	45	}
46	46
		@@ -52,62 +52,62 @@ discard block
		block discarded – undo
52	52	* comparison that is not case sensitive. The strcasecmp() function returns
53	53	* a 0 if the strings are equal.
54	54	*/
55		- if ( 0 == strcasecmp( $item->attr_title, 'divider' ) && 1 === $depth ) {
	55	+ if (0 == strcasecmp($item->attr_title, 'divider') && 1 === $depth) {
56	56	$output .= $indent . '<li role="presentation" class="divider">';
57		- } elseif ( 0 == strcasecmp( $item->title, 'divider' ) && 1 === $depth ) {
	57	+ } elseif (0 == strcasecmp($item->title, 'divider') && 1 === $depth) {
58	58	$output .= $indent . '<li role="presentation" class="divider">';
59		- } elseif ( 0 == strcasecmp( $item->attr_title, 'dropdown-header' ) && 1 === $depth ) {
60		- $output .= $indent . '<li role="presentation" class="dropdown-header">' . esc_attr( $item->title );
61		- } elseif ( 0 == strcasecmp( $item->attr_title, 'disabled' ) ) {
62		- $output .= $indent . '<li role="presentation" class="disabled"><a href="#">' . esc_attr( $item->title ) . '</a>';
	59	+ } elseif (0 == strcasecmp($item->attr_title, 'dropdown-header') && 1 === $depth) {
	60	+ $output .= $indent . '<li role="presentation" class="dropdown-header">' . esc_attr($item->title);
	61	+ } elseif (0 == strcasecmp($item->attr_title, 'disabled')) {
	62	+ $output .= $indent . '<li role="presentation" class="disabled"><a href="#">' . esc_attr($item->title) . '</a>';
63	63	} else {
64	64	$class_names = $value = '';
65	65
66		- $classes = empty( $item->classes ) ? array() : (array) $item->classes;
	66	+ $classes = empty($item->classes) ? array() : (array) $item->classes;
67	67	$classes[] = 'menu-item-' . $item->ID;
68	68
69		- $classes = apply_filters( 'lsx_nav_menu_css_class', array_filter( $classes ), $item, $args , $depth );
	69	+ $classes = apply_filters('lsx_nav_menu_css_class', array_filter($classes), $item, $args, $depth);
70	70
71		- $class_names = join( ' ', $classes );
	71	+ $class_names = join(' ', $classes);
72	72
73		- if ( $args->has_children )
	73	+ if ($args->has_children)
74	74	$class_names .= ' dropdown';
75	75
76		- if ( in_array( 'current-menu-item', $classes ) )
	76	+ if (in_array('current-menu-item', $classes))
77	77	$class_names .= ' active';
78	78
79		- if ( in_array( 'current-menu-parent', $classes ) )
	79	+ if (in_array('current-menu-parent', $classes))
80	80	$class_names .= ' active';
81	81
82	82	//Check if this is ment to be a "social" type menu
83		- $class_names = $class_names ? ' class="' . esc_attr( $class_names ) . '"' : '';
	83	+ $class_names = $class_names ? ' class="' . esc_attr($class_names) . '"' : '';
84	84
85		- $id = apply_filters( 'nav_menu_item_id', 'menu-item-'. $item->ID, $item, $args );
86		- $id = $id ? ' id="' . esc_attr( $id ) . '"' : '';
	85	+ $id = apply_filters('nav_menu_item_id', 'menu-item-' . $item->ID, $item, $args);
	86	+ $id = $id ? ' id="' . esc_attr($id) . '"' : '';
87	87
88		- $output .= $indent . '<li' . $id . $value . $class_names .'>';
	88	+ $output .= $indent . '<li' . $id . $value . $class_names . '>';
89	89
90	90	$atts = array();
91		- $atts['title'] = ! empty( $item->title ) ? $item->title : '';
92		- $atts['target'] = ! empty( $item->target ) ? $item->target : '';
93		- $atts['rel'] = ! empty( $item->xfn ) ? $item->xfn : '';
	91	+ $atts['title'] = ! empty($item->title) ? $item->title : '';
	92	+ $atts['target'] = ! empty($item->target) ? $item->target : '';
	93	+ $atts['rel'] = ! empty($item->xfn) ? $item->xfn : '';
94	94
95	95	// If item has_children add atts to a.
96		- if ( $args->has_children ) {
97		- $atts['href'] = ! empty( $item->url ) ? $item->url : '';
98		- $atts['data-toggle'] = 'dropdown';
99		- $atts['class'] = 'dropdown-toggle';
100		- $atts['aria-haspopup'] = 'true';
	96	+ if ($args->has_children) {
	97	+ $atts['href'] = ! empty($item->url) ? $item->url : '';
	98	+ $atts['data-toggle'] = 'dropdown';
	99	+ $atts['class'] = 'dropdown-toggle';
	100	+ $atts['aria-haspopup'] = 'true';
101	101	} else {
102		- $atts['href'] = ! empty( $item->url ) ? $item->url : '';
	102	+ $atts['href'] = ! empty($item->url) ? $item->url : '';
103	103	}
104	104
105		- $atts = apply_filters( 'nav_menu_link_attributes', $atts, $item, $args );
	105	+ $atts = apply_filters('nav_menu_link_attributes', $atts, $item, $args);
106	106
107	107	$attributes = '';
108		- foreach ( $atts as $attr => $value ) {
109		- if ( ! empty( $value ) ) {
110		- $value = ( 'href' === $attr ) ? esc_url( $value ) : esc_attr( $value );
	108	+ foreach ($atts as $attr => $value) {
	109	+ if ( ! empty($value)) {
	110	+ $value = ('href' === $attr) ? esc_url($value) : esc_attr($value);
111	111	$attributes .= ' ' . $attr . '="' . $value . '"';
112	112	}
113	113	}
		@@ -121,19 +121,19 @@ discard block
		block discarded – undo
121	121	* if there is a value in the attr_title property. If the attr_title
122	122	* property is NOT null we apply it as the class name for the glyphicon.
123	123	*/
124		- if ( ! empty( $item->attr_title ) ) {
125		- $item_output .= '<a'. $attributes .'"><span class="glyphicon ' . esc_attr( $item->attr_title ) . '"></span> ';
	124	+ if ( ! empty($item->attr_title)) {
	125	+ $item_output .= '<a' . $attributes . '"><span class="glyphicon ' . esc_attr($item->attr_title) . '"></span> ';
126	126	} else {
127		- $item_output .= '<a'. $attributes .'>';
	127	+ $item_output .= '<a' . $attributes . '>';
128	128	}
129	129
130		- $item_output .= $args->link_before . apply_filters( 'the_title', $item->title, $item->ID ) . $args->link_after;
131		- $item_output .= ( $args->has_children && 0 === $depth ) ? ' <span class="caret"></span></a>' : '</a>';
	130	+ $item_output .= $args->link_before . apply_filters('the_title', $item->title, $item->ID) . $args->link_after;
	131	+ $item_output .= ($args->has_children && 0 === $depth) ? ' <span class="caret"></span></a>' : '</a>';
132	132	$item_output .= $args->after;
133	133
134	134
135	135
136		- $output .= apply_filters( 'walker_nav_menu_start_el', $item_output, $item, $depth, $args );
	136	+ $output .= apply_filters('walker_nav_menu_start_el', $item_output, $item, $depth, $args);
137	137	}
138	138	}
139	139
		@@ -157,17 +157,17 @@ discard block
		block discarded – undo
157	157	* @param string $output Passed by reference. Used to append additional content.
158	158	* @return null Null on failure with no changes to parameters.
159	159	*/
160		- public function display_element( $element, &$children_elements, $max_depth, $depth, $args, &$output ) {
161		- if ( ! $element )
	160	+ public function display_element($element, &$children_elements, $max_depth, $depth, $args, &$output) {
	161	+ if ( ! $element)
162	162	return;
163	163
164	164	$id_field = $this->db_fields['id'];
165	165
166	166	// Display this element.
167		- if ( is_object( $args[0] ) )
168		- $args[0]->has_children = ! empty( $children_elements[ $element->$id_field ] );
	167	+ if (is_object($args[0]))
	168	+ $args[0]->has_children = ! empty($children_elements[$element->$id_field]);
169	169
170		- parent::display_element( $element, $children_elements, $max_depth, $depth, $args, $output );
	170	+ parent::display_element($element, $children_elements, $max_depth, $depth, $args, $output);
171	171	}
172	172
173	173	/**
		@@ -181,17 +181,17 @@ discard block
		block discarded – undo
181	181	* @param array $args passed from the wp_nav_menu function.
182	182	*
183	183	*/
184		- public static function fallback( $args ) {
185		- if ( current_user_can( 'manage_options' ) ) {
	184	+ public static function fallback($args) {
	185	+ if (current_user_can('manage_options')) {
186	186	$fb_output = null;
187	187
188		- if ( $args['container'] ) {
	188	+ if ($args['container']) {
189	189	$fb_output = '<' . $args['container'];
190	190
191		- if ( $args['container_id'] )
	191	+ if ($args['container_id'])
192	192	$fb_output .= ' id="' . $args['container_id'] . '"';
193	193
194		- if ( $args['container_class'] )
	194	+ if ($args['container_class'])
195	195	$fb_output .= ' class="' . $args['container_class'] . '"';
196	196
197	197	$fb_output .= '>';
		@@ -199,20 +199,20 @@ discard block
		block discarded – undo
199	199
200	200	$fb_output .= '<ul';
201	201
202		- if ( $args['menu_id'] )
	202	+ if ($args['menu_id'])
203	203	$fb_output .= ' id="' . $args['menu_id'] . '"';
204	204
205		- if ( $args['menu_class'] )
	205	+ if ($args['menu_class'])
206	206	$fb_output .= ' class="' . $args['menu_class'] . '"';
207	207
208	208	$fb_output .= '>';
209		- $fb_output .= '<li><a href="' . admin_url( 'nav-menus.php' ) . '">' . esc_html__( 'Add a menu', 'lsx' ) . '</a></li>';
	209	+ $fb_output .= '<li><a href="' . admin_url('nav-menus.php') . '">' . esc_html__('Add a menu', 'lsx') . '</a></li>';
210	210	$fb_output .= '</ul>';
211	211
212		- if ( $args['container'] )
	212	+ if ($args['container'])
213	213	$fb_output .= '</' . $args['container'] . '>';
214	214
215		- echo wp_kses_post( $fb_output );
	215	+ echo wp_kses_post($fb_output);
216	216	}
217	217	}
218	218	}
		@@ -221,9 +221,9 @@ discard block
		block discarded – undo
221	221	/**
222	222	* Add in our custom classes to the menus
223	223	*/
224		-function wpml_nav_language_switcher_fix( $items , $args ) {
225		- $items = str_replace('menu-item-language-current','menu-item-language-current dropdown',$items);
226		- $items = str_replace('submenu-languages','submenu-languages dropdown-menu',$items);
	224	+function wpml_nav_language_switcher_fix($items, $args) {
	225	+ $items = str_replace('menu-item-language-current', 'menu-item-language-current dropdown', $items);
	226	+ $items = str_replace('submenu-languages', 'submenu-languages dropdown-menu', $items);
227	227	return $items;
228	228	}
229		-add_filter( 'wp_nav_menu_items', 'wpml_nav_language_switcher_fix', 10, 2 );
	229	+add_filter('wp_nav_menu_items', 'wpml_nav_language_switcher_fix', 10, 2);

		@@ -5,7 +5,7 @@ discard block
		block discarded – undo
5	5
6	6	<?php lsx_content_wrap_before(); ?>
7	7
8		- <div id="primary" class="content-area <?php echo esc_attr( lsx_main_class() ); ?>">
	8	+ <div id="primary" class="content-area <?php echo esc_attr(lsx_main_class()); ?>">
9	9
10	10	<?php lsx_content_before(); ?>
11	11
		@@ -13,7 +13,7 @@ discard block
		block discarded – undo
13	13
14	14	<?php lsx_content_top(); ?>
15	15
16		- <?php while ( have_posts() ) : the_post(); ?>
	16	+ <?php while (have_posts()) : the_post(); ?>
17	17
18	18	<?php lsx_entry_before(); ?>
19	19
		@@ -22,28 +22,28 @@ discard block
		block discarded – undo
22	22	<?php lsx_entry_top(); ?>
23	23
24	24	<div class="entry-content">
25		- <h2><?php esc_html_e( 'The Last 30 Posts', 'lsx' ); ?></h3>
	25	+ <h2><?php esc_html_e('The Last 30 Posts', 'lsx'); ?></h3>
26	26	<ul>
27		- <?php $loop = new WP_Query( array( 'posts_per_page' => 30 ) ); ?>
28		- <?php if ( $loop->have_posts() ) { while ( $loop->have_posts() ) { $loop->the_post(); ?>
	27	+ <?php $loop = new WP_Query(array('posts_per_page' => 30)); ?>
	28	+ <?php if ($loop->have_posts()) { while ($loop->have_posts()) { $loop->the_post(); ?>
29	29	<?php $loop->is_home = false; ?>
30		- <li><a href="<?php the_permalink(); ?>"><?php the_title(); ?></a> - <?php the_time( get_option( 'date_format' ) ); ?> - <?php echo esc_html( $post->comment_count ); ?> <?php esc_html_e( 'comments', 'lsx' ); ?></li>
	30	+ <li><a href="<?php the_permalink(); ?>"><?php the_title(); ?></a> - <?php the_time(get_option('date_format')); ?> - <?php echo esc_html($post->comment_count); ?> <?php esc_html_e('comments', 'lsx'); ?></li>
31	31	<?php } } wp_reset_postdata(); ?>
32	32	</ul>
33	33
34		- <h2><?php esc_html_e( 'Categories', 'lsx' ); ?></h3>
	34	+ <h2><?php esc_html_e('Categories', 'lsx'); ?></h3>
35	35
36	36	<ul>
37		- <?php wp_list_categories( 'title_li=&hierarchical=0&show_count=1' ); ?>
	37	+ <?php wp_list_categories('title_li=&hierarchical=0&show_count=1'); ?>
38	38	</ul>
39	39
40		- <h2><?php esc_html_e( 'Monthly Archives', 'lsx' ); ?></h3>
	40	+ <h2><?php esc_html_e('Monthly Archives', 'lsx'); ?></h3>
41	41
42	42	<ul>
43		- <?php wp_get_archives( 'type=monthly&show_post_count=1' ); ?>
	43	+ <?php wp_get_archives('type=monthly&show_post_count=1'); ?>
44	44	</ul>
45	45	</div><!-- .entry-content -->
46		- <?php edit_post_link( esc_html__( 'Edit', 'lsx' ), '<footer class="entry-meta"><span class="edit-link">', '</span></footer>' ); ?>
	46	+ <?php edit_post_link(esc_html__('Edit', 'lsx'), '<footer class="entry-meta"><span class="edit-link">', '</span></footer>'); ?>
47	47
48	48	<?php lsx_entry_bottom(); ?>
49	49

		@@ -17,8 +17,8 @@ discard block
		block discarded – undo
17	17
18	18	<?php lsx_content_top(); ?>
19	19
20		- <?php if(have_posts()) { ?>
21		- <?php while(have_posts()) { the_post(); ?>
	20	+ <?php if (have_posts()) { ?>
	21	+ <?php while (have_posts()) { the_post(); ?>
22	22	<div class="entry-content">
23	23
24	24	<?php the_content(); ?>
		@@ -29,10 +29,10 @@ discard block
		block discarded – undo
29	29	<?php } ?>
30	30
31	31	<?php
32		- if ( get_query_var( 'paged' ) ) :
33		- $paged = get_query_var( 'paged' );
34		- elseif ( get_query_var( 'page' ) ) :
35		- $paged = get_query_var( 'page' );
	32	+ if (get_query_var('paged')) :
	33	+ $paged = get_query_var('paged');
	34	+ elseif (get_query_var('page')) :
	35	+ $paged = get_query_var('page');
36	36	else :
37	37	$paged = 1;
38	38	endif;
		@@ -41,18 +41,18 @@ discard block
		block discarded – undo
41	41	'post_type' => 'jetpack-portfolio',
42	42	'posts_per_page' => 99,
43	43	);
44		- $project_query = new WP_Query ( $args );
45		- if ( post_type_exists( 'jetpack-portfolio' ) && $project_query -> have_posts() ) :
	44	+ $project_query = new WP_Query($args);
	45	+ if (post_type_exists('jetpack-portfolio') && $project_query -> have_posts()) :
46	46	?>
47	47
48	48	<?php lsx_portfolio_sorter(); ?>
49	49
50	50	<div class="filter-items-wrapper lsx-portfolio-wrapper">
51	51	<div class="filter-items-container lsx-portfolio masonry">
52		- <?php while ( $project_query -> have_posts() ) : $project_query -> the_post(); ?>
	52	+ <?php while ($project_query -> have_posts()) : $project_query -> the_post(); ?>
53	53
54		- <?php if(has_post_thumbnail()) { ?>
55		- <?php get_template_part( 'content', 'portfolio' ); ?>
	54	+ <?php if (has_post_thumbnail()) { ?>
	55	+ <?php get_template_part('content', 'portfolio'); ?>
56	56	<?php } ?>
57	57
58	58	<?php endwhile; ?>
		@@ -63,17 +63,17 @@ discard block
		block discarded – undo
63	63
64	64	<section class="no-results not-found">
65	65	<header class="page-header">
66		- <h1 class="page-title"><?php esc_html_e( 'Nothing Found', 'lsx' ); ?></h1>
	66	+ <h1 class="page-title"><?php esc_html_e('Nothing Found', 'lsx'); ?></h1>
67	67	</header><!-- .page-header -->
68	68
69	69	<div class="page-content">
70		- <?php if ( current_user_can( 'publish_posts' ) ) : ?>
	70	+ <?php if (current_user_can('publish_posts')) : ?>
71	71
72		- <p><?php esc_html_e( 'Ready to publish your first project?', 'lsx' ); ?> <a href="<?php echo esc_url( admin_url( 'post-new.php?post_type=jetpack-portfolio' ) ) ?>"><?php esc_html_e( 'Get started here', 'lsx' ); ?></a></p>
	72	+ <p><?php esc_html_e('Ready to publish your first project?', 'lsx'); ?> <a href="<?php echo esc_url(admin_url('post-new.php?post_type=jetpack-portfolio')) ?>"><?php esc_html_e('Get started here', 'lsx'); ?></a></p>
73	73
74	74	<?php else : ?>
75	75
76		- <p><?php esc_html_e( 'It seems we can’t find what you’re looking for. Perhaps searching can help.', 'lsx' ); ?></p>
	76	+ <p><?php esc_html_e('It seems we can’t find what you’re looking for. Perhaps searching can help.', 'lsx'); ?></p>
77	77	<?php get_search_form(); ?>
78	78
79	79	<?php endif; ?>

		@@ -33,8 +33,10 @@ discard block
		block discarded – undo
33	33	$paged = get_query_var( 'paged' );
34	34	elseif ( get_query_var( 'page' ) ) :
35	35	$paged = get_query_var( 'page' );
36		- else :
	36	+ else {
	37	+ :
37	38	$paged = 1;
	39	+ }
38	40	endif;
39	41
40	42	$args = array(
		@@ -59,11 +61,14 @@ discard block
		block discarded – undo
59	61	</div>
60	62	</div><!-- .portfolio-wrapper -->
61	63
62		- <?php else : ?>
	64	+ <?php else {
	65	+ : ?>
63	66
64	67	<section class="no-results not-found">
65	68	<header class="page-header">
66		- <h1 class="page-title"><?php esc_html_e( 'Nothing Found', 'lsx' ); ?></h1>
	69	+ <h1 class="page-title"><?php esc_html_e( 'Nothing Found', 'lsx' );
	70	+}
	71	+?></h1>
67	72	</header><!-- .page-header -->
68	73
69	74	<div class="page-content">
		@@ -71,9 +76,12 @@ discard block
		block discarded – undo
71	76
72	77	<p><?php esc_html_e( 'Ready to publish your first project?', 'lsx' ); ?> <a href="<?php echo esc_url( admin_url( 'post-new.php?post_type=jetpack-portfolio' ) ) ?>"><?php esc_html_e( 'Get started here', 'lsx' ); ?></a></p>
73	78
74		- <?php else : ?>
	79	+ <?php else {
	80	+ : ?>
75	81
76		- <p><?php esc_html_e( 'It seems we can’t find what you’re looking for. Perhaps searching can help.', 'lsx' ); ?></p>
	82	+ <p><?php esc_html_e( 'It seems we can’t find what you’re looking for. Perhaps searching can help.', 'lsx' );
	83	+}
	84	+?></p>
77	85	<?php get_search_form(); ?>
78	86
79	87	<?php endif; ?>

lightspeeddevelopment / lsx

Push — master ( bef6ca...7e2933 )

Status

Category

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

Braces +5 added lines, -2 removed lines patch added patch discarded remove patch

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

Braces +5 added lines, -2 removed lines patch added patch discarded remove patch

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

Indentation +4 added lines, -4 removed lines patch added patch discarded remove patch

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

Braces +5 added lines, -2 removed lines patch added patch discarded remove patch

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

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

Braces +5 added lines, -2 removed lines patch added patch discarded remove patch

Doc Comments -1 removed lines patch added patch discarded remove patch

Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

Braces +34 added lines, -21 removed lines patch added patch discarded remove patch

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

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

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

Braces +13 added lines, -5 removed lines patch added patch discarded remove patch