lightspeeddevelopment / lsx

inc/extras.php

Doc Comments +2 added lines

@@ -198,6 +198,7 @@ 
  * @package lsx
  * @subpackage extras
  * @category thumbnails
+ * @param string $size
  */
 function lsx_get_thumbnail($size,$image_src = false){
 	
@@ -252,6 +253,7 @@ 
  * @package lsx
  * @subpackage extras
  * @category thumbnails
+ * @param boolean $image_src
  */
 function lsx_get_attachment_id_from_src($image_src) {
 	global $wpdb;

Braces +30 added lines, -17 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) return; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	return;
+}
+// Exit if accessed directly
 
 /**
  * Add and remove body_class() classes
@@ -204,10 +207,10 @@ 
 	if(false === $image_src){
 		$post_id = get_the_ID(); 
 		$post_thumbnail_id = get_post_thumbnail_id( $post_id );
-	}elseif(false != $image_src	){
+	} elseif(false != $image_src	){
 		if(is_numeric($image_src)){
 			$post_thumbnail_id = $image_src;
-		}else{
+		} else{
 			$post_thumbnail_id = lsx_get_attachment_id_from_src($image_src);
 		}
 	}
@@ -217,7 +220,7 @@ 
 		$srcset = false;
 		$img = wp_get_attachment_image_src($post_thumbnail_id,$size);
 		$img = $img[0];
-	}else{
+	} else{
 		$srcset = true;
 		$img = wp_get_attachment_image_srcset($post_thumbnail_id,$size);
 		if($img == false) {
@@ -327,18 +330,29 @@ 
  */
 function mv_browser_body_class($classes) {
         global $is_lynx, $is_gecko, $is_IE, $is_opera, $is_NS4, $is_safari, $is_chrome, $is_iphone;
-        if($is_lynx) $classes[] = 'lynx';
-        elseif($is_gecko) $classes[] = 'gecko';
-        elseif($is_opera) $classes[] = 'opera';
-        elseif($is_NS4) $classes[] = 'ns4';
-        elseif($is_safari) $classes[] = 'safari';
-        elseif($is_chrome) $classes[] = 'chrome';
-        elseif($is_IE) {
+        if($is_lynx) {
+        	$classes[] = 'lynx';
+        } elseif($is_gecko) {
+        	$classes[] = 'gecko';
+        } elseif($is_opera) {
+        	$classes[] = 'opera';
+        } elseif($is_NS4) {
+        	$classes[] = 'ns4';
+        } elseif($is_safari) {
+        	$classes[] = 'safari';
+        } elseif($is_chrome) {
+        	$classes[] = 'chrome';
+        } elseif($is_IE) {
                 $classes[] = 'ie';
-                if(preg_match('/MSIE ([0-9]+)([a-zA-Z0-9.]+)/', $_SERVER['HTTP_USER_AGENT'], $browser_version))
-                $classes[] = 'ie'.$browser_version[1];
-        } else $classes[] = 'unknown';
-        if($is_iphone) $classes[] = 'iphone';
+                if(preg_match('/MSIE ([0-9]+)([a-zA-Z0-9.]+)/', $_SERVER['HTTP_USER_AGENT'], $browser_version)) {
+                                $classes[] = 'ie'.$browser_version[1];
+                }
+        } else {
+        	$classes[] = 'unknown';
+        }
+        if($is_iphone) {
+        	$classes[] = 'iphone';
+        }
         if ( stristr( $_SERVER['HTTP_USER_AGENT'],"mac") ) {
                  $classes[] = 'osx';
            } elseif ( stristr( $_SERVER['HTTP_USER_AGENT'],"linux") ) {
@@ -396,8 +410,7 @@ 
 
 			if ( ! empty( $pagination ) ) {
 				$excerpt .= $pagination;
-			}
-			else {
+			} else {
 				$excerpt .= '<p><a class="moretag" href="'.get_permalink().'">'.__('Continue reading','lsx').'</a></p>';
 			}
 		}

inc/layout.php

Doc Comments +1 added lines

@@ -5,6 +5,7 @@
  * Layout hooks
  *
  * @package lsx
+ * @param string $class
  */
 
 function lsx_layout_selector( $class, $area = 'site' ) {

Braces +5 added lines, -2 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) return; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	return;
+}
+// Exit if accessed directly
 
 /**
  * Layout hooks
@@ -62,7 +65,7 @@ 
 	$show_on_front = get_option('show_on_front');
 	if('page' == $show_on_front){
 		return lsx_layout_selector( 'main', 'home' );
-	}else{
+	} else{
 		return lsx_layout_selector( 'main', 'site' );
 	}
 

inc/lazyload.php

Doc Comments +3 added lines

@@ -140,6 +140,9 @@
 		return self::$noscripts[(int) $match[1]];
 	}
 
+	/**
+	 * @param string $new_class
+	 */
 	static function add_class( $html_string = '', $new_class ) {
 		$pattern = '/class=[\'"]([^\'"]*)[\'"]/';
 

Braces +6 added lines, -3 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) return;
+if ( ! defined( 'ABSPATH' ) ) {
+	return;
+}
 
 /*
  * LSX Lazy Load Images Class
@@ -25,8 +27,9 @@ 
 	protected static $noscripts   = array();
 
 	static function init() {
-		if ( is_admin() )
-			return;
+		if ( is_admin() ) {
+					return;
+		}
 
 		if ( get_theme_mod( 'lsx_lazyload_status', '1' ) === false ) {
 			self::$enabled = false;

inc/sanitize.php

Doc Comments +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
  * @subpackage sanitize
  *
  * @param array $input
- * @return array $output
+ * @return string|false $output
  */
 function lsx_sanitize_checkbox( $input ) {
 	if ( $input ) {

inc/template-tags.php

Doc Comments +1 added lines, -2 removed lines

@@ -7,7 +7,6 @@ 
  * @author Novrian <me@novrian.info>
  * @copyright (c) 2013. Novrian Y.F.
  * @license MIT License
- * @param string $sep Your custom separator
  */
 function lsx_breadcrumbs() {
   if (!function_exists('yoast_breadcrumb') && !function_exists('woocommerce_breadcrumb')) {
@@ -262,7 +261,7 @@ 
 	/**
 	 * Display navigation to next/previous set of posts when applicable.
 	 *
-	 * @return void
+	 * @return null|boolean
 	 */
 	function lsx_paging_nav() {
 		// Don't print empty markup if there's only one page.

Braces +15 added lines, -11 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) return; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	return;
+}
+// Exit if accessed directly
 
 /**
  * Yoast Breadcrumbs on Twitter Bootstrap
@@ -24,7 +27,7 @@ 
   		woocommerce_breadcrumb();
   		$output = ob_get_clean();
   		$output = str_replace('woocommerce-breadcrumb', 'woocommerce-breadcrumb breadcrumbs-container', $output);
-  }elseif(function_exists('yoast_breadcrumb')){
+  } elseif(function_exists('yoast_breadcrumb')){
 	  	// Default Yoast Breadcrumbs Separator
 	  	$old_sep = '\&raquo\;';
 	  	
@@ -272,9 +275,9 @@ 
 		
 		if(current_theme_supports('infinite-scroll') && class_exists('The_Neverending_Home_Page')){
 			return true;
-		}elseif(function_exists('wp_pagenavi')){
+		} elseif(function_exists('wp_pagenavi')){
 			wp_pagenavi();
-		}else{
+		} else{
 			
 			$labels = array(
 				'next' 		=> '<span class="meta-nav">&larr;</span> '.__( 'Older posts', 'lsx' ),
@@ -354,9 +357,9 @@ 
 
 		if ( function_exists('has_custom_logo') && has_custom_logo() ) {
 			the_custom_logo();
-		}elseif ( function_exists( 'jetpack_has_site_logo' ) && jetpack_has_site_logo() ) {
+		} elseif ( function_exists( 'jetpack_has_site_logo' ) && jetpack_has_site_logo() ) {
 			jetpack_the_site_logo();
-		}else{
+		} else{
 			// shouldn't show both together.. its just strange
 			if(true == get_theme_mod('site_logo_header_text',1)){
 				lsx_site_title();
@@ -498,7 +501,7 @@ 
 		$post_type_object = get_post_type_object($post_type);
 		if(null != $post_type_object){
 			$title = $post_type_object->labels->name;
-		}else{
+		} else{
 			$title = ucwords($post_type);
 		}
 		
@@ -640,8 +643,9 @@ 
  * @category 	urls
  */
 function lsx_get_my_url() {
-	if ( ! preg_match( '/<a\s[^>]*?href=[\'"](.+?)[\'"]/is', get_the_content(), $matches ) )
-		return false;
+	if ( ! preg_match( '/<a\s[^>]*?href=[\'"](.+?)[\'"]/is', get_the_content(), $matches ) ) {
+			return false;
+	}
 
 	return esc_url_raw( $matches[1] );
 }
@@ -658,14 +662,14 @@ 
 	$part = (string) $part;
 	if ( '' !== $part ){
 		$template = "{$slug}-{$part}.php";
-	}else{
+	} else{
 		$template = "{$slug}.php";
 	}
 	$file_path = apply_filters('lsx_content_path',false,$slug,$part);
 
 	if ( false !== $file_path && '' == locate_template( array( $template ) ) && file_exists( $file_path.$template) ) {
 		load_template( $file_path.$template, false );
-	}else{
+	} else{
 		get_template_part($slug,$part);
 	}
 }
\ No newline at end of file

inc/wp_bootstrap_navwalker.php

Doc Comments -1 removed lines

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

Braces +34 added lines, -21 removed lines

@@ -1,5 +1,8 @@ 
 <?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 @@ 
 			
 			$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 @@ 
 	 * @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 );
     }
@@ -192,29 +200,34 @@ 
 			if ( $container ) {
 				$fb_output = '<' . $container;
 
-				if ( $container_id )
-					$fb_output .= ' id="' . $container_id . '"';
+				if ( $container_id ) {
+									$fb_output .= ' id="' . $container_id . '"';
+				}
 
-				if ( $container_class )
-					$fb_output .= ' class="' . $container_class . '"';
+				if ( $container_class ) {
+									$fb_output .= ' class="' . $container_class . '"';
+				}
 
 				$fb_output .= '>';
 			}
 
 			$fb_output .= '<ul';
 
-			if ( $menu_id )
-				$fb_output .= ' id="' . $menu_id . '"';
+			if ( $menu_id ) {
+							$fb_output .= ' id="' . $menu_id . '"';
+			}
 
-			if ( $menu_class )
-				$fb_output .= ' class="' . $menu_class . '"';
+			if ( $menu_class ) {
+							$fb_output .= ' class="' . $menu_class . '"';
+			}
 
 			$fb_output .= '>';
 			$fb_output .= '<li><a href="' . admin_url( 'nav-menus.php' ) . '">'.__('Add a menu','lsx').'</a></li>';
 			$fb_output .= '</ul>';
 
-			if ( $container )
-				$fb_output .= '</' . $container . '>';
+			if ( $container ) {
+							$fb_output .= '</' . $container . '>';
+			}
 
 			echo $fb_output;
 		}

sidebar.php

Braces +6 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
 		
 		if('posts' == $show_on_front && is_home()){
 			$sidebar = 'home';
-		}else{
+		} else{
 			$sidebar = 'sidebar-1';
 		}
 	}
@@ -38,9 +38,12 @@ 
 	
 		<div id="secondary" class="widget-area <?php echo esc_attr(lsx_sidebar_class()); ?>" role="complementary">
 		
-	<?php else : ?>
+	<?php else {
+	: ?>
 	
-		<div id="secondary" class="widget-area <?php echo esc_attr(lsx_sidebar_class()); ?>" role="complementary">
+		<div id="secondary" class="widget-area <?php echo esc_attr(lsx_sidebar_class());
+}
+?>" role="complementary">
 		
 	<?php endif ; ?>
 

sidebar-sitemap.php

Braces +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 	$layout = get_theme_mod('lsx_layout','2cr');
 	if('posts' == $show_on_front && is_home()){
 		$sidebar = 'home';
-	}else{
+	} else{
 		$sidebar = 'sidebar-1';
 	}
 	

page-templates/template-portfolio.php

Braces +13 added lines, -5 removed lines

@@ -33,8 +33,10 @@ 
 					$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 @@ 
 					</div>
 				</div><!-- .portfolio-wrapper -->
 
-			<?php else : ?>
+			<?php else {
+	: ?>
 
 				<section class="no-results not-found">
 					<header class="page-header">
-						<h1 class="page-title"><?php _e( 'Nothing Found', 'lsx' ); ?></h1>
+						<h1 class="page-title"><?php _e( 'Nothing Found', 'lsx' );
+}
+?></h1>
 					</header><!-- .page-header -->
 
 					<div class="page-content">
@@ -71,9 +76,12 @@ 
 
 							<p><?php printf( __( 'Ready to publish your first project? <a href="%1$s">Get started here</a>.', 'lsx' ), esc_url( admin_url( 'post-new.php?post_type=jetpack-portfolio' ) ) ); ?></p>
 
-						<?php else : ?>
+						<?php else {
+	: ?>
 
-							<p><?php _e( 'It seems we can&rsquo;t find what you&rsquo;re looking for. Perhaps searching can help.', 'lsx' ); ?></p>
+							<p><?php _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; ?>