jarednova / timber

lib/image/timber-image-operation-retina.php

Braces +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
      *                               (ex: /src/var/www/wp-content/uploads/my-pic@2x.jpg)
      * @return bool                  true if everything went fine, false otherwise
      */
-    function run($load_filename, $save_filename){
+    function run($load_filename, $save_filename) {
         $image = wp_get_image_editor( $load_filename );
         if ( !is_wp_error( $image ) ) {
             $current_size = $image->get_size();

Spacing +13 added lines, -13 removed lines

@@ -17,7 +17,7 @@ 
      * Construct our operation
      * @param float   $factor to multiply original dimensions by
      */
-    function __construct($factor) {
+    function __construct( $factor ) {
         $this->factor = $factor;
     }
 
@@ -28,7 +28,7 @@ 
      * @param   string    $src_extension    the extension (ex: .jpg)
      * @return  string    the final filename to be used (ex: my-awesome-pic@2x.jpg)
      */
-    function filename($src_filename, $src_extension) {
+    function filename( $src_filename, $src_extension ) {
         $newbase = $src_filename . '@' . $this->factor . 'x'; // add @2x, @3x, @1.5x, etc.
         $new_name = $newbase . '.' . $src_extension;
         return $new_name;
@@ -44,30 +44,30 @@ 
      *                               (ex: /src/var/www/wp-content/uploads/my-pic@2x.jpg)
      * @return bool                  true if everything went fine, false otherwise
      */
-    function run($load_filename, $save_filename){
-        $image = wp_get_image_editor( $load_filename );
-        if ( !is_wp_error( $image ) ) {
+    function run( $load_filename, $save_filename ) {
+        $image = wp_get_image_editor($load_filename);
+        if ( !is_wp_error($image) ) {
             $current_size = $image->get_size();
             $src_w = $current_size['width'];
             $src_h = $current_size['height'];
             // Get ratios
             $w = $src_w * $this->factor;
             $h = $src_h * $this->factor;
-            $image->crop( 0, 0, $src_w, $src_h, $w, $h );
-            $result = $image->save( $save_filename );
-            if ( is_wp_error( $result ) ) {
+            $image->crop(0, 0, $src_w, $src_h, $w, $h);
+            $result = $image->save($save_filename);
+            if ( is_wp_error($result) ) {
             	// @codeCoverageIgnoreStart
-				TimberHelper::error_log( 'Error resizing image' );
-				TimberHelper::error_log( $result );
+				TimberHelper::error_log('Error resizing image');
+				TimberHelper::error_log($result);
 				return false;
 				// @codeCoverageIgnoreEnd
             } else {
                 return true;
             }
-        } else if ( isset( $image->error_data['error_loading_image'] ) ) {
-            TimberHelper::error_log( 'Error loading ' . $image->error_data['error_loading_image'] );
+        } else if ( isset($image->error_data['error_loading_image']) ) {
+            TimberHelper::error_log('Error loading ' . $image->error_data['error_loading_image']);
         } else {
-            TimberHelper::error_log( $image );
+            TimberHelper::error_log($image);
         }
         return false;
     }

lib/integrations/timber-command.php

Braces +5 added lines, -5 removed lines

@@ -6,14 +6,14 @@ 
  */
 class TimberCommand {
 
-    public static function clear_cache($mode = 'all'){
-        if (is_array($mode)){
+    public static function clear_cache($mode = 'all') {
+        if (is_array($mode)) {
             $mode = reset($mode);
         }
         if ($mode == 'all') {
             $twig_cache = self::clear_cache_twig();
             $timber_cache = self::clear_cache_timber();
-            if ($twig_cache && $timber_cache){
+            if ($twig_cache && $timber_cache) {
                 return true;
             }
         } else if ($mode == 'twig') {
@@ -23,12 +23,12 @@ 
         }
     }
 
-    static function clear_cache_timber(){
+    static function clear_cache_timber() {
         $loader = new TimberLoader();
         return $loader->clear_cache_timber();
     }
 
-    static function clear_cache_twig(){
+    static function clear_cache_twig() {
         $loader = new TimberLoader();
         return $loader->clear_cache_twig();
     }

Spacing +8 added lines, -8 removed lines

@@ -6,29 +6,29 @@
  */
 class TimberCommand {
 
-    public static function clear_cache($mode = 'all'){
-        if (is_array($mode)){
+    public static function clear_cache( $mode = 'all' ) {
+        if ( is_array($mode) ) {
             $mode = reset($mode);
         }
-        if ($mode == 'all') {
+        if ( $mode == 'all' ) {
             $twig_cache = self::clear_cache_twig();
             $timber_cache = self::clear_cache_timber();
-            if ($twig_cache && $timber_cache){
+            if ( $twig_cache && $timber_cache ) {
                 return true;
             }
-        } else if ($mode == 'twig') {
+        } else if ( $mode == 'twig' ) {
             return self::clear_cache_twig();
-        } else if ($mode == 'timber') {
+        } else if ( $mode == 'timber' ) {
             return self::clear_cache_timber();
         }
     }
 
-    static function clear_cache_timber(){
+    static function clear_cache_timber() {
         $loader = new TimberLoader();
         return $loader->clear_cache_timber();
     }
 
-    static function clear_cache_twig(){
+    static function clear_cache_twig() {
         $loader = new TimberLoader();
         return $loader->clear_cache_twig();
     }

lib/integrations/wpcli-timber.php

Braces +3 added lines, -3 removed lines

@@ -25,9 +25,9 @@ 
      *    wp timber clear_cache_twig
      *
      */
-    function clear_cache_twig(){
+    function clear_cache_twig() {
         $clear = TimberCommand::clear_cache_twig();
-        if ($clear){
+        if ($clear) {
             WP_CLI::success('Cleared contents of twig cache');
         } else {
             WP_CLI::warning('Failed to clear twig cache');
@@ -45,7 +45,7 @@ 
     function clear_cache_timber() {
         $clear = TimberCommand::clear_cache_timber();
         $message = 'Failed to clear timber cache';
-        if ($clear){
+        if ($clear) {
             $message = "Cleared contents of Timber's Cache";
             WP_CLI::success($message);
         } else {

Spacing +5 added lines, -5 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if (!class_exists('WP_CLI_Command')) {
+if ( !class_exists('WP_CLI_Command') ) {
 	return;
 }
 
@@ -13,7 +13,7 @@ 
      *    wp timber clear_cache
      *
      */
-    public function clear_cache($mode = 'all') {
+    public function clear_cache( $mode = 'all' ) {
         TimberCommand::clear_cache($mode);
     }
 
@@ -25,9 +25,9 @@ 
      *    wp timber clear_cache_twig
      *
      */
-    function clear_cache_twig(){
+    function clear_cache_twig() {
         $clear = TimberCommand::clear_cache_twig();
-        if ($clear){
+        if ( $clear ) {
             WP_CLI::success('Cleared contents of twig cache');
         } else {
             WP_CLI::warning('Failed to clear twig cache');
@@ -45,7 +45,7 @@ 
     function clear_cache_timber() {
         $clear = TimberCommand::clear_cache_timber();
         $message = 'Failed to clear timber cache';
-        if ($clear){
+        if ( $clear ) {
             $message = "Cleared contents of Timber's Cache";
             WP_CLI::success($message);
         } else {

lib/timber-core.php

Braces +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
             foreach ( $info as $key => $value ) {
                 if ( !empty( $key ) && $force ) {
                     $this->$key = $value;
-                } else if ( !empty( $key ) && !method_exists($this, $key) ){
+                } else if ( !empty( $key ) && !method_exists($this, $key) ) {
                     $this->$key = $value;
                 }
             }

Spacing +11 added lines, -11 removed lines

@@ -12,7 +12,7 @@ 
      * @return boolean
      */
     function __isset( $field ) {
-        if ( isset( $this->$field ) ) {
+        if ( isset($this->$field) ) {
             return $this->$field;
         }
         return false;
@@ -23,7 +23,7 @@ 
      * @return mixed
      */
     function __call( $field, $args ) {
-        return $this->__get( $field );
+        return $this->__get($field);
     }
 
     /**
@@ -35,7 +35,7 @@ 
     	if ( property_exists($this, $field) ) {
     		return $this->$field;
     	}
-        if ( method_exists($this, 'meta') && $meta_value = $this->meta( $field ) ) {
+        if ( method_exists($this, 'meta') && $meta_value = $this->meta($field) ) {
             return $this->$field = $meta_value;
         }
         if ( method_exists($this, $field) ) {
@@ -56,14 +56,14 @@ 
      * @param array|object $info an object or array you want to grab data from to attach to the Timber object
      */
     function import( $info, $force = false ) {
-        if ( is_object( $info ) ) {
-            $info = get_object_vars( $info );
+        if ( is_object($info) ) {
+            $info = get_object_vars($info);
         }
-        if ( is_array( $info ) ) {
+        if ( is_array($info) ) {
             foreach ( $info as $key => $value ) {
-                if ( !empty( $key ) && $force ) {
+                if ( !empty($key) && $force ) {
                     $this->$key = $value;
-                } else if ( !empty( $key ) && !method_exists($this, $key) ){
+                } else if ( !empty($key) && !method_exists($this, $key) ) {
                     $this->$key = $value;
                 }
             }
@@ -77,7 +77,7 @@ 
      * @param mixed   $value
      */
     function update( $key, $value ) {
-        update_metadata( $this->object_type, $this->ID, $key, $value );
+        update_metadata($this->object_type, $this->ID, $key, $value);
     }
 
     /**
@@ -94,10 +94,10 @@ 
      * @return bool
      */
     function can_edit() {
-        if ( !function_exists( 'current_user_can' ) ) {
+        if ( !function_exists('current_user_can') ) {
             return false;
         }
-        if ( current_user_can( 'edit_post', $this->ID ) ) {
+        if ( current_user_can('edit_post', $this->ID) ) {
             return true;
         }
         return false;

lib/timber-helper.php

Braces +3 added lines, -2 removed lines

@@ -48,8 +48,9 @@
 
 			// lock timeout shouldn't be higher than 5 seconds, unless
 			// remote calls with high timeouts are made here
-			if ( $enable_transients )
-				self::_lock_transient( $slug, $lock_timeout );
+			if ( $enable_transients ) {
+							self::_lock_transient( $slug, $lock_timeout );
+			}
 
 			$data = $callback();
 

Spacing +101 added lines, -101 removed lines

@@ -26,17 +26,17 @@ 
 	 * @return mixed
 	 */
 	public static function transient( $slug, $callback, $transient_time = 0, $lock_timeout = 5, $force = false ) {
-		$slug = apply_filters( 'timber/transient/slug', $slug );
+		$slug = apply_filters('timber/transient/slug', $slug);
 
-		$enable_transients = ( $transient_time === false || ( defined( 'WP_DISABLE_TRANSIENTS' ) && WP_DISABLE_TRANSIENTS ) ) ? false : true;
-		$data = $enable_transients ? get_transient( $slug ) : false;
+		$enable_transients = ($transient_time === false || (defined('WP_DISABLE_TRANSIENTS') && WP_DISABLE_TRANSIENTS)) ? false : true;
+		$data = $enable_transients ? get_transient($slug) : false;
 
 		if ( false === $data ) {
 
-			if ( $enable_transients && self::_is_transient_locked( $slug ) ) {
+			if ( $enable_transients && self::_is_transient_locked($slug) ) {
 
-				$force = apply_filters( 'timber_force_transients', $force );
-				$force = apply_filters( 'timber_force_transient_' . $slug, $force );
+				$force = apply_filters('timber_force_transients', $force);
+				$force = apply_filters('timber_force_transient_' . $slug, $force);
 
 				if ( !$force ) {
 					//the server is currently executing the process.
@@ -50,13 +50,13 @@ 
 			// lock timeout shouldn't be higher than 5 seconds, unless
 			// remote calls with high timeouts are made here
 			if ( $enable_transients )
-				self::_lock_transient( $slug, $lock_timeout );
+				self::_lock_transient($slug, $lock_timeout);
 
 			$data = $callback();
 
 			if ( $enable_transients ) {
-				set_transient( $slug, $data, $transient_time );
-				self::_unlock_transient( $slug );
+				set_transient($slug, $data, $transient_time);
+				self::_unlock_transient($slug);
 			}
 
 		}
@@ -71,7 +71,7 @@ 
 	 * @param integer $lock_timeout
 	 */
 	static function _lock_transient( $slug, $lock_timeout ) {
-		set_transient( $slug . '_lock', true, $lock_timeout );
+		set_transient($slug . '_lock', true, $lock_timeout);
 	}
 
 	/**
@@ -79,7 +79,7 @@ 
 	 * @param string $slug
 	 */
 	static function _unlock_transient( $slug ) {
-		delete_transient( $slug . '_lock', true );
+		delete_transient($slug . '_lock', true);
 	}
 
 	/**
@@ -87,7 +87,7 @@ 
 	 * @param string $slug
 	 */
 	static function _is_transient_locked( $slug ) {
-		return (bool)get_transient( $slug . '_lock' );
+		return (bool) get_transient($slug . '_lock');
 	}
 
 	/* These are for measuring page render time */
@@ -99,7 +99,7 @@ 
 	 */
 	public static function start_timer() {
 		$time = microtime();
-		$time = explode( ' ', $time );
+		$time = explode(' ', $time);
 		$time = $time[1] + $time[0];
 		return $time;
 	}
@@ -117,10 +117,10 @@ 
 	 */
 	public static function stop_timer( $start ) {
 		$time = microtime();
-		$time = explode( ' ', $time );
+		$time = explode(' ', $time);
 		$time = $time[1] + $time[0];
 		$finish = $time;
-		$total_time = round( ( $finish - $start ), 4 );
+		$total_time = round(($finish - $start), 4);
 		return $total_time . ' seconds.';
 	}
 
@@ -153,9 +153,9 @@ 
 	 * @param array   $args
 	 * @return string
 	 */
-	public static function ob_function( $function, $args = array( null ) ) {
+	public static function ob_function( $function, $args = array(null) ) {
 		ob_start();
-		call_user_func_array( $function, $args );
+		call_user_func_array($function, $args);
 		$data = ob_get_contents();
 		ob_end_clean();
 		return $data;
@@ -170,7 +170,7 @@ 
 	 * @return TimberFunctionWrapper
 	 */
 	public static function function_wrapper( $function_name, $defaults = array(), $return_output_buffer = false ) {
-		return new TimberFunctionWrapper( $function_name, $defaults, $return_output_buffer );
+		return new TimberFunctionWrapper($function_name, $defaults, $return_output_buffer);
 	}
 
 	/**
@@ -183,10 +183,10 @@ 
 		if ( !WP_DEBUG ) {
 			return;
 		}
-		if ( is_object( $arg ) || is_array( $arg ) ) {
-			$arg = print_r( $arg, true );
+		if ( is_object($arg) || is_array($arg) ) {
+			$arg = print_r($arg, true);
 		}
-		return error_log( $arg );
+		return error_log($arg);
 	}
 
 	/**
@@ -197,8 +197,8 @@ 
 	 * @return string
 	 */
 	public static function get_wp_title( $separator = ' ', $seplocation = 'left' ) {
-		$separator = apply_filters( 'timber_wp_title_seperator', $separator );
-		return trim( wp_title( $separator, false, $seplocation ) );
+		$separator = apply_filters('timber_wp_title_seperator', $separator);
+		return trim(wp_title($separator, false, $seplocation));
 	}
 
 	/* Text Utilities
@@ -215,33 +215,33 @@ 
 	 */
 	public static function trim_words( $text, $num_words = 55, $more = null, $allowed_tags = 'p a span b i br blockquote' ) {
 		if ( null === $more ) {
-			$more = __( '…' );
+			$more = __('…');
 		}
 		$original_text = $text;
 		$allowed_tag_string = '';
-		foreach ( explode( ' ', apply_filters( 'timber/trim_words/allowed_tags', $allowed_tags ) ) as $tag ) {
+		foreach ( explode(' ', apply_filters('timber/trim_words/allowed_tags', $allowed_tags)) as $tag ) {
 			$allowed_tag_string .= '<' . $tag . '>';
 		}
-		$text = strip_tags( $text, $allowed_tag_string );
+		$text = strip_tags($text, $allowed_tag_string);
 		/* translators: If your word count is based on single characters (East Asian characters), enter 'characters'. Otherwise, enter 'words'. Do not translate into your own language. */
-		if ( 'characters' == _x( 'words', 'word count: words or characters?' ) && preg_match( '/^utf\-?8$/i', get_option( 'blog_charset' ) ) ) {
-			$text = trim( preg_replace( "/[\n\r\t ]+/", ' ', $text ), ' ' );
-			preg_match_all( '/./u', $text, $words_array );
-			$words_array = array_slice( $words_array[0], 0, $num_words + 1 );
+		if ( 'characters' == _x('words', 'word count: words or characters?') && preg_match('/^utf\-?8$/i', get_option('blog_charset')) ) {
+			$text = trim(preg_replace("/[\n\r\t ]+/", ' ', $text), ' ');
+			preg_match_all('/./u', $text, $words_array);
+			$words_array = array_slice($words_array[0], 0, $num_words + 1);
 			$sep = '';
 		} else {
-			$words_array = preg_split( "/[\n\r\t ]+/", $text, $num_words + 1, PREG_SPLIT_NO_EMPTY );
+			$words_array = preg_split("/[\n\r\t ]+/", $text, $num_words + 1, PREG_SPLIT_NO_EMPTY);
 			$sep = ' ';
 		}
-		if ( count( $words_array ) > $num_words ) {
-			array_pop( $words_array );
-			$text = implode( $sep, $words_array );
+		if ( count($words_array) > $num_words ) {
+			array_pop($words_array);
+			$text = implode($sep, $words_array);
 			$text = $text . $more;
 		} else {
-			$text = implode( $sep, $words_array );
+			$text = implode($sep, $words_array);
 		}
-		$text = self::close_tags( $text );
-		return apply_filters( 'wp_trim_words', $text, $num_words, $more, $original_text );
+		$text = self::close_tags($text);
+		return apply_filters('wp_trim_words', $text, $num_words, $more, $original_text);
 	}
 
 	/**
@@ -252,27 +252,27 @@ 
 	 */
 	public static function close_tags( $html ) {
 		//put all opened tags into an array
-		preg_match_all( '#<([a-z]+)(?: .*)?(?<![/|/ ])>#iU', $html, $result );
+		preg_match_all('#<([a-z]+)(?: .*)?(?<![/|/ ])>#iU', $html, $result);
 		$openedtags = $result[1];
 		//put all closed tags into an array
-		preg_match_all( '#</([a-z]+)>#iU', $html, $result );
+		preg_match_all('#</([a-z]+)>#iU', $html, $result);
 		$closedtags = $result[1];
-		$len_opened = count( $openedtags );
+		$len_opened = count($openedtags);
 		// all tags are closed
-		if ( count( $closedtags ) == $len_opened ) {
+		if ( count($closedtags) == $len_opened ) {
 			return $html;
 		}
-		$openedtags = array_reverse( $openedtags );
+		$openedtags = array_reverse($openedtags);
 		// close tags
 		for ( $i = 0; $i < $len_opened; $i++ ) {
-			if ( !in_array( $openedtags[$i], $closedtags ) ) {
+			if ( !in_array($openedtags[$i], $closedtags) ) {
 				$html .= '</' . $openedtags[$i] . '>';
 			} else {
-				unset( $closedtags[array_search( $openedtags[$i], $closedtags )] );
+				unset($closedtags[array_search($openedtags[$i], $closedtags)]);
 			}
 		}
-		$html = str_replace(array('</br>','</hr>','</wbr>'), '', $html);
-		$html = str_replace(array('<br>','<hr>','<wbr>'), array('<br />','<hr />','<wbr />'), $html);
+		$html = str_replace(array('</br>', '</hr>', '</wbr>'), '', $html);
+		$html = str_replace(array('<br>', '<hr>', '<wbr>'), array('<br />', '<hr />', '<wbr />'), $html);
 		return $html;
 	}
 
@@ -287,15 +287,15 @@ 
 	 */
 	public static function get_posts_by_meta( $key, $value ) {
 		global $wpdb;
-		$query = $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = %s AND meta_value = %s", $key, $value );
-		$results = $wpdb->get_col( $query );
+		$query = $wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = %s AND meta_value = %s", $key, $value);
+		$results = $wpdb->get_col($query);
 		$pids = array();
 		foreach ( $results as $result ) {
-			if ( get_post( $result ) ) {
+			if ( get_post($result) ) {
 				$pids[] = $result;
 			}
 		}
-		if ( count( $pids ) ) {
+		if ( count($pids) ) {
 			return $pids;
 		}
 		return 0;
@@ -311,10 +311,10 @@ 
 	 */
 	public static function get_post_by_meta( $key, $value ) {
 		global $wpdb;
-		$query = $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = %s AND meta_value = %s ORDER BY post_id", $key, $value );
-		$results = $wpdb->get_col( $query );
+		$query = $wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = %s AND meta_value = %s ORDER BY post_id", $key, $value);
+		$results = $wpdb->get_col($query);
 		foreach ( $results as $result ) {
-			if ( $result && get_post( $result ) ) {
+			if ( $result && get_post($result) ) {
 				return $result;
 			}
 		}
@@ -329,8 +329,8 @@ 
 	 */
 	public static function get_term_id_by_term_taxonomy_id( $ttid ) {
 		global $wpdb;
-		$query = $wpdb->prepare( "SELECT term_id FROM $wpdb->term_taxonomy WHERE term_taxonomy_id = %s", $ttid );
-		return $wpdb->get_var( $query );
+		$query = $wpdb->prepare("SELECT term_id FROM $wpdb->term_taxonomy WHERE term_taxonomy_id = %s", $ttid);
+		return $wpdb->get_var($query);
 	}
 
 	/* Object Utilities
@@ -344,7 +344,7 @@ 
 	 * @return void
 	 */
 	public static function osort( &$array, $prop ) {
-		usort( $array, function ( $a, $b ) use ( $prop ) {
+		usort($array, function( $a, $b ) use ($prop) {
 				return $a->$prop > $b->$prop ? 1 : -1;
 			} );
 	}
@@ -356,10 +356,10 @@ 
 	 * @return bool
 	 */
 	public static function is_array_assoc( $arr ) {
-		if ( !is_array( $arr ) ) {
+		if ( !is_array($arr) ) {
 			return false;
 		}
-		return (bool)count( array_filter( array_keys( $arr ), 'is_string' ) );
+		return (bool) count(array_filter(array_keys($arr), 'is_string'));
 	}
 
 	/**
@@ -371,8 +371,8 @@ 
 	public static function array_to_object( $array ) {
 		$obj = new stdClass;
 		foreach ( $array as $k => $v ) {
-			if ( is_array( $v ) ) {
-				$obj->{$k} = self::array_to_object( $v ); //RECURSION
+			if ( is_array($v) ) {
+				$obj->{$k} = self::array_to_object($v); //RECURSION
 			} else {
 				$obj->{$k} = $v;
 			}
@@ -389,10 +389,10 @@ 
 	 * @return bool|int
 	 */
 	public static function get_object_index_by_property( $array, $key, $value ) {
-		if ( is_array( $array ) ) {
+		if ( is_array($array) ) {
 			$i = 0;
 			foreach ( $array as $arr ) {
-				if ( is_array( $arr ) ) {
+				if ( is_array($arr) ) {
 					if ( $arr[$key] == $value ) {
 						return $i;
 					}
@@ -417,15 +417,15 @@ 
 	 * @throws Exception
 	 */
 	public static function get_object_by_property( $array, $key, $value ) {
-		if ( is_array( $array ) ) {
+		if ( is_array($array) ) {
 			foreach ( $array as $arr ) {
 				if ( $arr->$key == $value ) {
 					return $arr;
 				}
 			}
 		} else {
-			throw new InvalidArgumentException( '$array is not an array, got:' );
-			TimberHelper::error_log( $array );
+			throw new InvalidArgumentException('$array is not an array, got:');
+			TimberHelper::error_log($array);
 		}
 	}
 
@@ -437,8 +437,8 @@ 
 	 * @return array
 	 */
 	public static function array_truncate( $array, $len ) {
-		if ( sizeof( $array ) > $len ) {
-			$array = array_splice( $array, 0, $len );
+		if ( sizeof($array) > $len ) {
+			$array = array_splice($array, 0, $len);
 		}
 		return $array;
 	}
@@ -453,11 +453,11 @@ 
 	 * @return bool
 	 */
 	public static function is_true( $value ) {
-		if ( isset( $value ) ) {
-			if (is_string($value)) {
+		if ( isset($value) ) {
+			if ( is_string($value) ) {
 				$value = strtolower($value);
 			}
-			if ( ($value == 'true' || $value === 1 || $value === '1' || $value == true) && $value !== false && $value !== 'false') {
+			if ( ($value == 'true' || $value === 1 || $value === '1' || $value == true) && $value !== false && $value !== 'false' ) {
 				return true;
 			}
 		}
@@ -471,7 +471,7 @@ 
 	 * @return bool
 	 */
 	public static function iseven( $i ) {
-		return ( $i % 2 ) == 0;
+		return ($i % 2) == 0;
 	}
 
 	/**
@@ -481,7 +481,7 @@ 
 	 * @return bool
 	 */
 	public static function isodd( $i ) {
-		return ( $i % 2 ) != 0;
+		return ($i % 2) != 0;
 	}
 
 	/* Links, Forms, Etc. Utilities
@@ -496,7 +496,7 @@ 
 	 * @return string
 	 */
 	public static function get_comment_form( $post_id = null, $args = array() ) {
-		return self::ob_function( 'comment_form', array( $args, $post_id ) );
+		return self::ob_function('comment_form', array($args, $post_id));
 	}
 
 	/**
@@ -513,28 +513,28 @@ 
 			'current' => 0,
 			'show_all' => false,
 			'prev_next' => false,
-			'prev_text' => __( '&laquo; Previous' ),
-			'next_text' => __( 'Next &raquo;' ),
+			'prev_text' => __('&laquo; Previous'),
+			'next_text' => __('Next &raquo;'),
 			'end_size' => 1,
 			'mid_size' => 2,
 			'type' => 'array',
 			'add_args' => false, // array of query args to add
 			'add_fragment' => ''
 		);
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 		// Who knows what else people pass in $args
-		$args['total'] = intval( (int)$args['total'] );
+		$args['total'] = intval((int) $args['total']);
 		if ( $args['total'] < 2 ) {
 			return array();
 		}
-		$args['current'] = (int)$args['current'];
-		$args['end_size'] = 0 < (int)$args['end_size'] ? (int)$args['end_size'] : 1; // Out of bounds?  Make it the default.
-		$args['mid_size'] = 0 <= (int)$args['mid_size'] ? (int)$args['mid_size'] : 2;
-		$args['add_args'] = is_array( $args['add_args'] ) ? $args['add_args'] : false;
+		$args['current'] = (int) $args['current'];
+		$args['end_size'] = 0 < (int) $args['end_size'] ? (int) $args['end_size'] : 1; // Out of bounds?  Make it the default.
+		$args['mid_size'] = 0 <= (int) $args['mid_size'] ? (int) $args['mid_size'] : 2;
+		$args['add_args'] = is_array($args['add_args']) ? $args['add_args'] : false;
 		$page_links = array();
 		$dots = false;
 		for ( $n = 1; $n <= $args['total']; $n++ ) {
-			$n_display = number_format_i18n( $n );
+			$n_display = number_format_i18n($n);
 			if ( $n == $args['current'] ) {
 				$page_links[] = array(
 					'class' => 'page-number page-numbers current',
@@ -545,18 +545,18 @@ 
 				);
 				$dots = true;
 			} else {
-				if ( $args['show_all'] || ( $n <= $args['end_size'] || ( $args['current'] && $n >= $args['current'] - $args['mid_size'] && $n <= $args['current'] + $args['mid_size'] ) || $n > $args['total'] - $args['end_size'] ) ) {
-					$link = str_replace( '%_%', 1 == $n ? '' : $args['format'], $args['base'] );
-					$link = str_replace( '%#%', $n, $link );
-					$link = trailingslashit( $link ) . ltrim( $args['add_fragment'], '/' );
+				if ( $args['show_all'] || ($n <= $args['end_size'] || ($args['current'] && $n >= $args['current'] - $args['mid_size'] && $n <= $args['current'] + $args['mid_size']) || $n > $args['total'] - $args['end_size']) ) {
+					$link = str_replace('%_%', 1 == $n ? '' : $args['format'], $args['base']);
+					$link = str_replace('%#%', $n, $link);
+					$link = trailingslashit($link) . ltrim($args['add_fragment'], '/');
 					if ( $args['add_args'] ) {
-						$link = rtrim( add_query_arg( $args['add_args'], $link ), '/' );
+						$link = rtrim(add_query_arg($args['add_args'], $link), '/');
 					}
 					$link = str_replace(' ', '+', $link);
-					$link = untrailingslashit( $link );
+					$link = untrailingslashit($link);
 					$page_links[] = array(
 						'class' => 'page-number page-numbers',
-						'link' => esc_url( apply_filters( 'paginate_links', $link ) ),
+						'link' => esc_url(apply_filters('paginate_links', $link)),
 						'title' => $n_display,
 						'name' => $n_display,
 						'current' => $args['current'] == $n
@@ -565,7 +565,7 @@ 
 				} elseif ( $dots && !$args['show_all'] ) {
 					$page_links[] = array(
 						'class' => 'dots',
-						'title' => __( '&hellip;' )
+						'title' => __('&hellip;')
 					);
 					$dots = false;
 				}
@@ -585,7 +585,7 @@ 
 	 * @deprecated 0.18.0
 	 */
 	static function is_url( $url ) {
-		return TimberURLHelper::is_url( $url );
+		return TimberURLHelper::is_url($url);
 	}
 
 	/**
@@ -599,70 +599,70 @@ 
 	 * @deprecated 0.18.0
 	 */
 	static function get_rel_url( $url, $force = false ) {
-		return TimberURLHelper::get_rel_url( $url, $force );
+		return TimberURLHelper::get_rel_url($url, $force);
 	}
 
 	/**
 	 * @deprecated 0.18.0
 	 */
 	static function is_local( $url ) {
-		return TimberURLHelper::is_local( $url );
+		return TimberURLHelper::is_local($url);
 	}
 
 	/**
 	 * @deprecated 0.18.0
 	 */
 	static function get_full_path( $src ) {
-		return TimberURLHelper::get_full_path( $src );
+		return TimberURLHelper::get_full_path($src);
 	}
 
 	/**
 	 * @deprecated 0.18.0
 	 */
 	static function get_rel_path( $src ) {
-		return TimberURLHelper::get_rel_path( $src );
+		return TimberURLHelper::get_rel_path($src);
 	}
 
 	/**
 	 * @deprecated 0.18.0
 	 */
 	static function remove_double_slashes( $url ) {
-		return TimberURLHelper::remove_double_slashes( $url );
+		return TimberURLHelper::remove_double_slashes($url);
 	}
 
 	/**
 	 * @deprecated 0.18.0
 	 */
 	static function prepend_to_url( $url, $path ) {
-		return TimberURLHelper::prepend_to_url( $url, $path );
+		return TimberURLHelper::prepend_to_url($url, $path);
 	}
 
 	/**
 	 * @deprecated 0.18.0
 	 */
 	static function preslashit( $path ) {
-		return TimberURLHelper::preslashit( $path );
+		return TimberURLHelper::preslashit($path);
 	}
 
 	/**
 	 * @deprecated 0.18.0
 	 */
 	static function is_external( $url ) {
-		return TimberURLHelper::is_external( $url );
+		return TimberURLHelper::is_external($url);
 	}
 
 	/**
 	 * @deprecated 0.18.0
 	 */
 	static function download_url( $url, $timeout = 300 ) {
-		return TimberURLHelper::download_url( $url, $timeout );
+		return TimberURLHelper::download_url($url, $timeout);
 	}
 
 	/**
 	 * @deprecated 0.18.0
 	 */
 	static function get_params( $i = -1 ) {
-		return TimberURLHelper::get_params( $i );
+		return TimberURLHelper::get_params($i);
 	}
 
 }

lib/timber-image-helper.php

Braces +2 added lines, -1 removed lines

@@ -325,7 +325,8 @@
 		);
 		$upload_dir = wp_upload_dir();
 		$tmp = $url;
-		if ( 0 === strpos($tmp, ABSPATH) ) { // we've been given a dir, not an url
+		if ( 0 === strpos($tmp, ABSPATH) ) {
+// we've been given a dir, not an url
 			$result['absolute'] = true;
 			if ( 0 === strpos($tmp, $upload_dir['basedir']) ) {
 				$result['base']= self::BASE_UPLOADS; // upload based

Spacing +73 added lines, -73 removed lines

@@ -72,8 +72,8 @@ 
 			$w = $_wp_additional_image_sizes[$size]['width'];
 			$h = $_wp_additional_image_sizes[$size]['height'];
 		} else if ( in_array($size, array('thumbnail', 'medium', 'large')) ) {
-			$w = get_option($size.'_size_w');
-			$h = get_option($size.'_size_h');
+			$w = get_option($size . '_size_w');
+			$h = get_option($size . '_size_h');
 		}
 		if ( isset($w) && isset($h) && ($w || $h) ) {
 			return array('w' => $w, 'h' => $h);
@@ -106,7 +106,7 @@ 
 			return false;
 		}
 		//its a gif so test
-		if( !($fh = @fopen($file, 'rb')) ) {
+		if ( !($fh = @fopen($file, 'rb')) ) {
 		  	return false;
 		}
 		$count = 0;
@@ -118,7 +118,7 @@ 
 
 		// We read through the file til we reach the end of the file, or we've found
 		// at least 2 frame headers
-		while(!feof($fh) && $count < 2) {
+		while ( !feof($fh) && $count < 2 ) {
 			$chunk = fread($fh, 1024 * 100); //read 100kb at a time
 			$count += preg_match_all('#\x00\x21\xF9\x04.{4}\x00[\x2C\x21]#s', $chunk, $matches);
 	    }
@@ -159,13 +159,13 @@ 
 	 * Deletes all resized versions of an image when the source is deleted
 	 */
 	protected static function add_actions() {
-		add_action( 'delete_attachment', function ( $post_id ) {
-			$post = get_post( $post_id );
-			$image_types = array( 'image/jpeg', 'image/png', 'image/gif', 'image/jpg' );
-			if ( in_array( $post->post_mime_type, $image_types ) ) {
-				$attachment = new TimberImage( $post_id );
+		add_action('delete_attachment', function( $post_id ) {
+			$post = get_post($post_id);
+			$image_types = array('image/jpeg', 'image/png', 'image/gif', 'image/jpg');
+			if ( in_array($post->post_mime_type, $image_types) ) {
+				$attachment = new TimberImage($post_id);
 				if ( $attachment->file_loc ) {
-					TimberImageHelper::delete_generated_files( $attachment->file_loc );
+					TimberImageHelper::delete_generated_files($attachment->file_loc);
 				}
 			}
 		} );
@@ -176,9 +176,9 @@ 
 	 * for example /wp-content or /content
 	 */
 	protected static function add_constants() {
-		if ( !defined( 'WP_CONTENT_SUBDIR' ) ) {
-			$wp_content_path = str_replace( home_url(), '', WP_CONTENT_URL );
-			define( 'WP_CONTENT_SUBDIR', $wp_content_path );
+		if ( !defined('WP_CONTENT_SUBDIR') ) {
+			$wp_content_path = str_replace(home_url(), '', WP_CONTENT_URL);
+			define('WP_CONTENT_SUBDIR', $wp_content_path);
 		}
 	}
 
@@ -188,8 +188,8 @@ 
 	 * @return void
 	 */
 	static function add_filters() {
-		add_filter( 'upload_dir', function ( $arr ) {
-			$arr['relative'] = str_replace( home_url(), '', $arr['baseurl'] );
+		add_filter('upload_dir', function( $arr ) {
+			$arr['relative'] = str_replace(home_url(), '', $arr['baseurl']);
 			return $arr;
 		} );
 	}
@@ -201,15 +201,15 @@ 
 	 *	                            or: http://example.org/wp-content/uploads/2015/my-pic.jpg
 	 */
 	static function delete_generated_files( $local_file ) {
-		if (TimberURLHelper::is_absolute( $local_file ) ) {
-			$local_file = TimberURLHelper::url_to_file_system( $local_file );
+		if ( TimberURLHelper::is_absolute($local_file) ) {
+			$local_file = TimberURLHelper::url_to_file_system($local_file);
 		}
-		$info = pathinfo( $local_file );
+		$info = pathinfo($local_file);
 		$dir = $info['dirname'];
 		$ext = $info['extension'];
 		$filename = $info['filename'];
-		self::process_delete_generated_files( $filename, $ext, $dir, '-[0-9999999]*', '-[0-9]*x[0-9]*-c-[a-z]*.' );
-		self::process_delete_generated_files( $filename, $ext, $dir, '-lbox-[0-9999999]*', '-lbox-[0-9]*x[0-9]*-[a-zA-Z0-9]*.' );
+		self::process_delete_generated_files($filename, $ext, $dir, '-[0-9999999]*', '-[0-9]*x[0-9]*-c-[a-z]*.');
+		self::process_delete_generated_files($filename, $ext, $dir, '-lbox-[0-9999999]*', '-lbox-[0-9]*x[0-9]*-[a-zA-Z0-9]*.');
 	}
 
 	/**
@@ -228,12 +228,12 @@ 
 	 */
 	protected static function process_delete_generated_files( $filename, $ext, $dir, $search_pattern, $match_pattern ) {
 		$searcher = '/' . $filename . $search_pattern;
-		foreach ( glob( $dir . $searcher ) as $found_file ) {
-			$regexdir = str_replace( '/', '\/', $dir );
-			$pattern = '/' . ( $regexdir ) . '\/' . $filename . $match_pattern . $ext . '/';
-			$match = preg_match( $pattern, $found_file );
+		foreach ( glob($dir . $searcher) as $found_file ) {
+			$regexdir = str_replace('/', '\/', $dir);
+			$pattern = '/' . ($regexdir) . '\/' . $filename . $match_pattern . $ext . '/';
+			$match = preg_match($pattern, $found_file);
 			if ( $match ) {
-				unlink( $found_file );
+				unlink($found_file);
 			}
 		}
 	}
@@ -247,7 +247,7 @@ 
 	 */
 	public static function get_server_location( $url ) {
 		// if we're already an absolute dir, just return
-		if ( 0 === strpos( $url, ABSPATH ) ) {
+		if ( 0 === strpos($url, ABSPATH) ) {
 			return $url;
 		}
 		// otherwise, analyze URL then build mapping path
@@ -266,11 +266,11 @@ 
 		$upload = wp_upload_dir();
 		$dir = $upload['path'];
 		$filename = $file;
-		$file = parse_url( $file );
-		$path_parts = pathinfo( $file['path'] );
-		$basename = md5( $filename );
+		$file = parse_url($file);
+		$path_parts = pathinfo($file['path']);
+		$basename = md5($filename);
 		$ext = 'jpg';
-		if ( isset( $path_parts['extension'] ) ) {
+		if ( isset($path_parts['extension']) ) {
 			$ext = $path_parts['extension'];
 		}
 		return $dir . '/' . $basename . '.' . $ext;
@@ -283,27 +283,27 @@ 
 	 * @return string the URL to the downloaded file
 	 */
 	public static function sideload_image( $file ) {
-		$loc = self::get_sideloaded_file_loc( $file );
-		if ( file_exists( $loc ) ) {
-			return TimberURLHelper::preslashit( TimberURLHelper::get_rel_path( $loc ) );
+		$loc = self::get_sideloaded_file_loc($file);
+		if ( file_exists($loc) ) {
+			return TimberURLHelper::preslashit(TimberURLHelper::get_rel_path($loc));
 		}
 		// Download file to temp location
-		if ( !function_exists( 'download_url' ) ) {
+		if ( !function_exists('download_url') ) {
 			require_once ABSPATH . '/wp-admin/includes/file.php';
 		}
-		$tmp = download_url( $file );
-		preg_match( '/[^\?]+\.(jpe?g|jpe|gif|png)\b/i', $file, $matches );
+		$tmp = download_url($file);
+		preg_match('/[^\?]+\.(jpe?g|jpe|gif|png)\b/i', $file, $matches);
 		$file_array = array();
-		$file_array['name'] = basename( $matches[0] );
+		$file_array['name'] = basename($matches[0]);
 		$file_array['tmp_name'] = $tmp;
 		// If error storing temporarily, unlink
-		if ( is_wp_error( $tmp ) ) {
-			@unlink( $file_array['tmp_name'] );
+		if ( is_wp_error($tmp) ) {
+			@unlink($file_array['tmp_name']);
 			$file_array['tmp_name'] = '';
 		}
 		// do the validation and storage stuff
-		$locinfo = pathinfo( $loc );
-		$file = wp_upload_bits( $locinfo['basename'], null, file_get_contents( $file_array['tmp_name'] ) );
+		$locinfo = pathinfo($loc);
+		$file = wp_upload_bits($locinfo['basename'], null, file_get_contents($file_array['tmp_name']));
 		return $file['url'];
 	}
 
@@ -315,7 +315,7 @@ 
 	 * @param  string $url an URL (absolute or relative) pointing to an image
 	 * @return array       an array (see keys in code below)
 	 */
-	private static function analyze_url($url) {
+	private static function analyze_url( $url ) {
 		$result = array(
 			'url' => $url, // the initial url
 			'absolute' => TimberURLHelper::is_absolute($url), // is the url absolute or relative (to home_url)
@@ -330,23 +330,23 @@ 
 		if ( 0 === strpos($tmp, ABSPATH) ) { // we've been given a dir, not an url
 			$result['absolute'] = true;
 			if ( 0 === strpos($tmp, $upload_dir['basedir']) ) {
-				$result['base']= self::BASE_UPLOADS; // upload based
+				$result['base'] = self::BASE_UPLOADS; // upload based
 				$tmp = str_replace($upload_dir['basedir'], '', $tmp);
 			}
 			if ( 0 === strpos($tmp, WP_CONTENT_DIR) ) {
-				$result['base']= self::BASE_CONTENT; // content based
+				$result['base'] = self::BASE_CONTENT; // content based
 				$tmp = str_replace(WP_CONTENT_DIR, '', $tmp);
 			}
 		} else {
-			if (!$result['absolute']) {
-				$tmp = home_url().$tmp;
+			if ( !$result['absolute'] ) {
+				$tmp = home_url() . $tmp;
 			}
-			if (0 === strpos($tmp, $upload_dir['baseurl'])) {
-				$result['base']= self::BASE_UPLOADS; // upload based
+			if ( 0 === strpos($tmp, $upload_dir['baseurl']) ) {
+				$result['base'] = self::BASE_UPLOADS; // upload based
 				$tmp = str_replace($upload_dir['baseurl'], '', $tmp);
 			}
-			if (0 === strpos($tmp, content_url())) {
-				$result['base']= self::BASE_CONTENT; // content-based
+			if ( 0 === strpos($tmp, content_url()) ) {
+				$result['base'] = self::BASE_CONTENT; // content-based
 				$tmp = str_replace(content_url(), '', $tmp);
 			}
 		}
@@ -368,20 +368,20 @@ 
 	 * @param  bool   $absolute should the returned URL be absolute (include protocol+host), or relative
 	 * @return string           the URL
 	 */
-	private static function _get_file_url($base, $subdir, $filename, $absolute) {
+	private static function _get_file_url( $base, $subdir, $filename, $absolute ) {
 		$url = '';
-		if( self::BASE_UPLOADS == $base ) {
+		if ( self::BASE_UPLOADS == $base ) {
 			$upload_dir = wp_upload_dir();
 			$url = $upload_dir['baseurl'];
 		}
-		if( self::BASE_CONTENT == $base ) {
+		if ( self::BASE_CONTENT == $base ) {
 			$url = content_url();
 		}
-		if(!empty($subdir)) {
+		if ( !empty($subdir) ) {
 			$url .= $subdir;
 		}
-		$url .= '/'.$filename;
-		if(!$absolute) {
+		$url .= '/' . $filename;
+		if ( !$absolute ) {
 			$url = str_replace(home_url(), '', $url);
 		}
 		// $url = TimberURLHelper::remove_double_slashes( $url);
@@ -396,19 +396,19 @@ 
 	 * @param  string $filename file name, including extension (but no path)
 	 * @return string           the file location
 	 */
-	private static function _get_file_path($base, $subdir, $filename) {
+	private static function _get_file_path( $base, $subdir, $filename ) {
 		$path = '';
-		if(self::BASE_UPLOADS == $base) {
+		if ( self::BASE_UPLOADS == $base ) {
 			$upload_dir = wp_upload_dir();
 			$path = $upload_dir['basedir'];
 		}
-		if(self::BASE_CONTENT == $base) {
+		if ( self::BASE_CONTENT == $base ) {
 			$path = WP_CONTENT_DIR;
 		}
-		if(!empty($subdir)) {
+		if ( !empty($subdir) ) {
 			$path .= $subdir;
 		}
-		$path .= '/'.$filename;
+		$path .= '/' . $filename;
 		return $path;
 	}
 
@@ -427,14 +427,14 @@ 
 	 *
 	 */
 	private static function _operate( $src, $op, $force = false ) {
-		if ( empty( $src ) ) {
+		if ( empty($src) ) {
 			return '';
 		}
 		$external = false;
 
 		// if external image, load it first
-		if ( TimberURLHelper::is_external_content( $src ) ) {
-			$src = self::sideload_image( $src );
+		if ( TimberURLHelper::is_external_content($src) ) {
+			$src = self::sideload_image($src);
 			$external = true;
 		}
 		// break down URL into components
@@ -457,19 +457,19 @@ 
 			$au['basename']
 		);
 		// if already exists...
-		if ( file_exists( $new_server_path ) ) {
+		if ( file_exists($new_server_path) ) {
 			if ( $force ) {
 				// Force operation - warning: will regenerate the image on every pageload, use for testing purposes only!
-				unlink( $new_server_path );
+				unlink($new_server_path);
 			} else {
 				// return existing file (caching)
 				return $new_url;
 			}
 		}
 		// otherwise generate result file
-		if($op->run($old_server_path, $new_server_path)) {
-			if( get_class( $op ) === 'TimberImageOperationResize' && $external ) {
-				$new_url = strtolower( $new_url );
+		if ( $op->run($old_server_path, $new_server_path) ) {
+			if ( get_class($op) === 'TimberImageOperationResize' && $external ) {
+				$new_url = strtolower($new_url);
 			}
 			return $new_url;
 		} else {
@@ -481,7 +481,7 @@ 
 
 // -- the below methods are just used for unit testing the URL generation code
 //
-	static function get_letterbox_file_url($url, $w, $h, $color) {
+	static function get_letterbox_file_url( $url, $w, $h, $color ) {
 		$au = self::analyze_url($url);
 		$op = new TimberImageOperationLetterbox($w, $h, $color);
 		$new_url = self::_get_file_url(
@@ -492,7 +492,7 @@ 
 		);
 		return $new_url;
 	}
-	public static function get_letterbox_file_path($url, $w, $h, $color ) {
+	public static function get_letterbox_file_path( $url, $w, $h, $color ) {
 		$au = self::analyze_url($url);
 		$op = new TimberImageOperationLetterbox($w, $h, $color);
 		$new_path = self::_get_file_path(
@@ -502,7 +502,7 @@ 
 		);
 		return $new_path;
 	}
-	static function get_resize_file_url($url, $w, $h, $crop) {
+	static function get_resize_file_url( $url, $w, $h, $crop ) {
 		$au = self::analyze_url($url);
 		$op = new TimberImageOperationResize($w, $h, $crop);
 		$new_url = self::_get_file_url(
@@ -513,7 +513,7 @@ 
 		);
 		return $new_url;
 	}
-	static function get_resize_file_path($url, $w, $h, $crop) {
+	static function get_resize_file_path( $url, $w, $h, $crop ) {
 		$au = self::analyze_url($url);
 		$op = new TimberImageOperationResize($w, $h, $crop);
 		$new_path = self::_get_file_path(

lib/timber-integrations.php

Spacing +4 added lines, -4 removed lines

@@ -7,16 +7,16 @@
 class TimberIntegrations {
     public static function init() {
 
-        add_action( 'init', array( __CLASS__, 'maybe_init_acftimber' ) );
+        add_action('init', array(__CLASS__, 'maybe_init_acftimber'));
 
-        if ( class_exists( 'WP_CLI_Command' ) ) {
-            WP_CLI::add_command( 'timber', 'Timber_WP_CLI_Command' );
+        if ( class_exists('WP_CLI_Command') ) {
+            WP_CLI::add_command('timber', 'Timber_WP_CLI_Command');
         }
     }
 
     public static function maybe_init_acftimber() {
 
-        if ( class_exists( 'ACF' ) ) {
+        if ( class_exists('ACF') ) {
             new ACFTimber();
         }
 

lib/timber-loader.php

Braces +20 added lines, -21 removed lines

@@ -258,7 +258,7 @@ 
 		return $twig;
 	}
 
-	public function clear_cache_timber($cache_mode = self::CACHE_USE_DEFAULT){
+	public function clear_cache_timber($cache_mode = self::CACHE_USE_DEFAULT) {
 		//_transient_timberloader
 		$object_cache = false;
 		if (isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache'])) {
@@ -277,7 +277,7 @@ 
 			return true;
 		} else if (self::CACHE_OBJECT === $cache_mode && $object_cache) {
 			global $wp_object_cache;
-			if (isset($wp_object_cache->cache[self::CACHEGROUP])){
+			if (isset($wp_object_cache->cache[self::CACHEGROUP])) {
 				unset($wp_object_cache->cache[self::CACHEGROUP]);
 				return true;
 			}
@@ -289,7 +289,7 @@ 
 		$twig = $this->get_twig();
 		$twig->clearCacheFiles();
 		$cache = $twig->getCache();
-		if ($cache){
+		if ($cache) {
 			self::rrmdir($twig->getCache());
 			return true;
 		}
@@ -348,14 +348,13 @@ 
 		$value = false;
 
 		$trans_key = substr($group . '_' . $key, 0, self::TRANS_KEY_LEN);
-		if (self::CACHE_TRANSIENT === $cache_mode)
-			$value = get_transient($trans_key);
-
-		elseif (self::CACHE_SITE_TRANSIENT === $cache_mode)
-			$value = get_site_transient($trans_key);
-
-		elseif (self::CACHE_OBJECT === $cache_mode && $object_cache)
-			$value = wp_cache_get($key, $group);
+		if (self::CACHE_TRANSIENT === $cache_mode) {
+					$value = get_transient($trans_key);
+		} elseif (self::CACHE_SITE_TRANSIENT === $cache_mode) {
+					$value = get_site_transient($trans_key);
+		} elseif (self::CACHE_OBJECT === $cache_mode && $object_cache) {
+					$value = wp_cache_get($key, $group);
+		}
 
 		return $value;
 	}
@@ -375,20 +374,20 @@ 
 			$object_cache = true;
 		}
 
-		if ((int)$expires < 1)
-			$expires = 0;
+		if ((int)$expires < 1) {
+					$expires = 0;
+		}
 
 		$cache_mode = self::_get_cache_mode($cache_mode);
 		$trans_key = substr($group . '_' . $key, 0, self::TRANS_KEY_LEN);
 
-		if (self::CACHE_TRANSIENT === $cache_mode)
-			set_transient($trans_key, $value, $expires);
-
-		elseif (self::CACHE_SITE_TRANSIENT === $cache_mode)
-			set_site_transient($trans_key, $value, $expires);
-
-		elseif (self::CACHE_OBJECT === $cache_mode && $object_cache)
-			wp_cache_set($key, $value, $group, $expires);
+		if (self::CACHE_TRANSIENT === $cache_mode) {
+					set_transient($trans_key, $value, $expires);
+		} elseif (self::CACHE_SITE_TRANSIENT === $cache_mode) {
+					set_site_transient($trans_key, $value, $expires);
+		} elseif (self::CACHE_OBJECT === $cache_mode && $object_cache) {
+					wp_cache_set($key, $value, $group, $expires);
+		}
 
 		return $value;
 	}

Spacing +69 added lines, -69 removed lines

@@ -26,7 +26,7 @@ 
 	/**
 	 * @param bool|string   $caller the calling directory or false
 	 */
-	function __construct($caller = false) {
+	function __construct( $caller = false ) {
 		$this->locations = $this->get_locations($caller);
 		$this->cache_mode = apply_filters('timber_cache_mode', $this->cache_mode);
 		$this->cache_mode = apply_filters('timber/cache/mode', $this->cache_mode);
@@ -39,10 +39,10 @@ 
 	 * @param string        $cache_mode
 	 * @return bool|string
 	 */
-	function render($file, $data = null, $expires = false, $cache_mode = self::CACHE_USE_DEFAULT) {
+	function render( $file, $data = null, $expires = false, $cache_mode = self::CACHE_USE_DEFAULT ) {
 		// Different $expires if user is anonymous or logged in
-		if (is_array($expires)) {
-			if (is_user_logged_in() && isset($expires[1])) {
+		if ( is_array($expires) ) {
+			if ( is_user_logged_in() && isset($expires[1]) ) {
 				$expires = $expires[1];
 			} else {
 				$expires = $expires[0];
@@ -51,15 +51,15 @@ 
 
 		$key = null;
 		$output = false;
-		if (false !== $expires) {
+		if ( false !== $expires ) {
 			ksort($data);
 			$key = md5($file . json_encode($data));
 			$output = $this->get_cache($key, self::CACHEGROUP, $cache_mode);
 		}
 
-		if (false === $output || null === $output) {
+		if ( false === $output || null === $output ) {
 			$twig = $this->get_twig();
-			if (strlen($file)) {
+			if ( strlen($file) ) {
 				$loader = $this->get_loader();
 				$result = $loader->getCacheKey($file);
 				do_action('timber_loader_render_file', $result);
@@ -69,7 +69,7 @@ 
 			$output = $twig->render($file, $data);
 		}
 
-		if (false !== $output && false !== $expires && null !== $key) {
+		if ( false !== $output && false !== $expires && null !== $key ) {
 			$this->set_cache($key, $output, self::CACHEGROUP, $expires, $cache_mode);
 		}
 		$output = apply_filters('timber_output', $output);
@@ -80,11 +80,11 @@ 
 	 * @param array $filenames
 	 * @return bool
 	 */
-	public function choose_template($filenames) {
-		if (is_array($filenames)) {
+	public function choose_template( $filenames ) {
+		if ( is_array($filenames) ) {
 			/* its an array so we have to figure out which one the dev wants */
-			foreach ($filenames as $filename) {
-				if (self::template_exists($filename)) {
+			foreach ( $filenames as $filename ) {
+				if ( self::template_exists($filename) ) {
 					return $filename;
 				}
 			}
@@ -97,10 +97,10 @@ 
 	 * @param string $file
 	 * @return bool
 	 */
-	protected function template_exists($file) {
-		foreach ($this->locations as $dir) {
+	protected function template_exists( $file ) {
+		foreach ( $this->locations as $dir ) {
 			$look_for = trailingslashit($dir) . $file;
-			if (file_exists($look_for)) {
+			if ( file_exists($look_for) ) {
 				return true;
 			}
 		}
@@ -114,17 +114,17 @@ 
 		$theme_locs = array();
 		$child_loc = get_stylesheet_directory();
 		$parent_loc = get_template_directory();
-		if (DIRECTORY_SEPARATOR == '\\') {
+		if ( DIRECTORY_SEPARATOR == '\\' ) {
 			$child_loc = str_replace('/', '\\', $child_loc);
 			$parent_loc = str_replace('/', '\\', $parent_loc);
 		}
 		$theme_locs[] = $child_loc;
-		foreach ($this->get_locations_theme_dir() as $dirname) {
+		foreach ( $this->get_locations_theme_dir() as $dirname ) {
 			$theme_locs[] = trailingslashit($child_loc) . trailingslashit($dirname);
 		}
-		if ($child_loc != $parent_loc) {
+		if ( $child_loc != $parent_loc ) {
 			$theme_locs[] = $parent_loc;
-			foreach ($this->get_locations_theme_dir() as $dirname) {
+			foreach ( $this->get_locations_theme_dir() as $dirname ) {
 				$theme_locs[] = trailingslashit($parent_loc) . trailingslashit($dirname);
 			}
 		}
@@ -138,7 +138,7 @@ 
 	 * @return string[] the names of directores, ie: array('templats', 'views');
 	 */
 	private function get_locations_theme_dir() {
-		if (is_string(Timber::$dirname)) {
+		if ( is_string(Timber::$dirname) ) {
 			return array(Timber::$dirname);
 		}
 		return Timber::$dirname;
@@ -149,13 +149,13 @@ 
 	 */
 	function get_locations_user() {
 		$locs = array();
-		if (isset(Timber::$locations)) {
-			if (is_string(Timber::$locations)) {
+		if ( isset(Timber::$locations) ) {
+			if ( is_string(Timber::$locations) ) {
 				Timber::$locations = array(Timber::$locations);
 			}
-			foreach (Timber::$locations as $tloc) {
+			foreach ( Timber::$locations as $tloc ) {
 				$tloc = realpath($tloc);
-				if (is_dir($tloc)) {
+				if ( is_dir($tloc) ) {
 					$locs[] = $tloc;
 				}
 			}
@@ -167,16 +167,16 @@ 
 	 * @param bool|string   $caller the calling directory
 	 * @return array
 	 */
-	function get_locations_caller($caller = false) {
+	function get_locations_caller( $caller = false ) {
 		$locs = array();
-		if ($caller && is_string($caller)) {
+		if ( $caller && is_string($caller) ) {
 			$caller = trailingslashit($caller);
-			if (is_dir($caller)) {
+			if ( is_dir($caller) ) {
 				$locs[] = $caller;
 			}
-			foreach ($this->get_locations_theme_dir() as $dirname) {
+			foreach ( $this->get_locations_theme_dir() as $dirname ) {
 				$caller_sub = $caller . trailingslashit($dirname);
-				if (is_dir($caller_sub)) {
+				if ( is_dir($caller_sub) ) {
 					$locs[] = $caller_sub;
 				}
 			}
@@ -188,7 +188,7 @@ 
 	 * @param bool|string   $caller the calling directory (or false)
 	 * @return array
 	 */
-	function get_locations($caller = false) {
+	function get_locations( $caller = false ) {
 		//prioirty: user locations, caller (but not theme), child theme, parent theme, caller
 		$locs = array();
 		$locs = array_merge($locs, $this->get_locations_user());
@@ -208,16 +208,16 @@ 
 	 */
 	function get_loader() {
 		$paths = array();
-		foreach ($this->locations as $loc) {
+		foreach ( $this->locations as $loc ) {
 			$loc = realpath($loc);
-			if (is_dir($loc)) {
+			if ( is_dir($loc) ) {
 				$loc = realpath($loc);
 				$paths[] = $loc;
 			} else {
 				//error_log($loc.' is not a directory');
 			}
 		}
-		if (!ini_get('open_basedir')) {
+		if ( !ini_get('open_basedir') ) {
 			$paths[] = '/';
 		} else {
 			$paths[] = ABSPATH;
@@ -233,15 +233,15 @@ 
 	function get_twig() {
 		$loader = $this->get_loader();
 		$params = array('debug' => WP_DEBUG, 'autoescape' => false);
-		if (isset(Timber::$autoescape)) {
+		if ( isset(Timber::$autoescape) ) {
 			$params['autoescape'] = Timber::$autoescape;
 		}
-		if (Timber::$cache === true) {
+		if ( Timber::$cache === true ) {
 			Timber::$twig_cache = true;
 		}
-		if (Timber::$twig_cache) {
-			$twig_cache_loc = apply_filters( 'timber/cache/location', TIMBER_LOC . '/cache/twig' );
-			if (!file_exists($twig_cache_loc)) {
+		if ( Timber::$twig_cache ) {
+			$twig_cache_loc = apply_filters('timber/cache/location', TIMBER_LOC . '/cache/twig');
+			if ( !file_exists($twig_cache_loc) ) {
 				mkdir($twig_cache_loc, 0777, true);
 			}
 			$params['cache'] = $twig_cache_loc;
@@ -258,26 +258,26 @@ 
 		return $twig;
 	}
 
-	public function clear_cache_timber($cache_mode = self::CACHE_USE_DEFAULT){
+	public function clear_cache_timber( $cache_mode = self::CACHE_USE_DEFAULT ) {
 		//_transient_timberloader
 		$object_cache = false;
-		if (isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache'])) {
+		if ( isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache']) ) {
 			$object_cache = true;
 		}
 		$cache_mode = $this->_get_cache_mode($cache_mode);
-		if (self::CACHE_TRANSIENT === $cache_mode) {
+		if ( self::CACHE_TRANSIENT === $cache_mode ) {
 			global $wpdb;
 			$query = $wpdb->prepare("DELETE FROM $wpdb->options WHERE option_name LIKE '%s'", '_transient_timberloader_%');
-			$wpdb->query( $query );
+			$wpdb->query($query);
 			return true;
-		} else if (self::CACHE_SITE_TRANSIENT === $cache_mode) {
+		} else if ( self::CACHE_SITE_TRANSIENT === $cache_mode ) {
 			global $wpdb;
 			$query = $wpdb->prepare("DELETE FROM $wpdb->options WHERE option_name LIKE '%s'", '_transient_timberloader_%');
-			$wpdb->query( $query );
+			$wpdb->query($query);
 			return true;
-		} else if (self::CACHE_OBJECT === $cache_mode && $object_cache) {
+		} else if ( self::CACHE_OBJECT === $cache_mode && $object_cache ) {
 			global $wp_object_cache;
-			if (isset($wp_object_cache->cache[self::CACHEGROUP])){
+			if ( isset($wp_object_cache->cache[self::CACHEGROUP]) ) {
 				unset($wp_object_cache->cache[self::CACHEGROUP]);
 				return true;
 			}
@@ -289,7 +289,7 @@ 
 		$twig = $this->get_twig();
 		$twig->clearCacheFiles();
 		$cache = $twig->getCache();
-		if ($cache){
+		if ( $cache ) {
 			self::rrmdir($twig->getCache());
 			return true;
 		}
@@ -299,16 +299,16 @@ 
 	/**
 	 * @param string|false $dirPath
 	 */
-	public static function rrmdir($dirPath) {
-		if (! is_dir($dirPath)) {
+	public static function rrmdir( $dirPath ) {
+		if ( !is_dir($dirPath) ) {
 			throw new InvalidArgumentException("$dirPath must be a directory");
 		}
-		if (substr($dirPath, strlen($dirPath) - 1, 1) != '/') {
+		if ( substr($dirPath, strlen($dirPath) - 1, 1) != '/' ) {
 			$dirPath .= '/';
 		}
 		$files = glob($dirPath . '*', GLOB_MARK);
-		foreach ($files as $file) {
-			if (is_dir($file)) {
+		foreach ( $files as $file ) {
+			if ( is_dir($file) ) {
 				self::rrmdir($file);
 			} else {
 				unlink($file);
@@ -323,9 +323,9 @@ 
 	private function _get_cache_extension() {
 
 		$key_generator   = new \Timber\Cache\KeyGenerator();
-		$cache_provider  = new \Timber\Cache\WPObjectCacheAdapter( $this );
-		$cache_strategy  = new \Asm89\Twig\CacheExtension\CacheStrategy\GenerationalCacheStrategy( $cache_provider, $key_generator );
-		$cache_extension = new \Asm89\Twig\CacheExtension\Extension( $cache_strategy );
+		$cache_provider  = new \Timber\Cache\WPObjectCacheAdapter($this);
+		$cache_strategy  = new \Asm89\Twig\CacheExtension\CacheStrategy\GenerationalCacheStrategy($cache_provider, $key_generator);
+		$cache_extension = new \Asm89\Twig\CacheExtension\Extension($cache_strategy);
 
 		return $cache_extension;
 	}
@@ -336,10 +336,10 @@ 
 	 * @param string $cache_mode
 	 * @return bool
 	 */
-	public function get_cache($key, $group = self::CACHEGROUP, $cache_mode = self::CACHE_USE_DEFAULT) {
+	public function get_cache( $key, $group = self::CACHEGROUP, $cache_mode = self::CACHE_USE_DEFAULT ) {
 		$object_cache = false;
 
-		if (isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache'])) {
+		if ( isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache']) ) {
 			$object_cache = true;
 		}
 
@@ -348,13 +348,13 @@ 
 		$value = false;
 
 		$trans_key = substr($group . '_' . $key, 0, self::TRANS_KEY_LEN);
-		if (self::CACHE_TRANSIENT === $cache_mode)
+		if ( self::CACHE_TRANSIENT === $cache_mode )
 			$value = get_transient($trans_key);
 
-		elseif (self::CACHE_SITE_TRANSIENT === $cache_mode)
+		elseif ( self::CACHE_SITE_TRANSIENT === $cache_mode )
 			$value = get_site_transient($trans_key);
 
-		elseif (self::CACHE_OBJECT === $cache_mode && $object_cache)
+		elseif ( self::CACHE_OBJECT === $cache_mode && $object_cache )
 			$value = wp_cache_get($key, $group);
 
 		return $value;
@@ -368,26 +368,26 @@ 
 	 * @param string $cache_mode
 	 * @return string|boolean
 	 */
-	public function set_cache($key, $value, $group = self::CACHEGROUP, $expires = 0, $cache_mode = self::CACHE_USE_DEFAULT) {
+	public function set_cache( $key, $value, $group = self::CACHEGROUP, $expires = 0, $cache_mode = self::CACHE_USE_DEFAULT ) {
 		$object_cache = false;
 
-		if (isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache'])) {
+		if ( isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache']) ) {
 			$object_cache = true;
 		}
 
-		if ((int)$expires < 1)
+		if ( (int) $expires < 1 )
 			$expires = 0;
 
 		$cache_mode = self::_get_cache_mode($cache_mode);
 		$trans_key = substr($group . '_' . $key, 0, self::TRANS_KEY_LEN);
 
-		if (self::CACHE_TRANSIENT === $cache_mode)
+		if ( self::CACHE_TRANSIENT === $cache_mode )
 			set_transient($trans_key, $value, $expires);
 
-		elseif (self::CACHE_SITE_TRANSIENT === $cache_mode)
+		elseif ( self::CACHE_SITE_TRANSIENT === $cache_mode )
 			set_site_transient($trans_key, $value, $expires);
 
-		elseif (self::CACHE_OBJECT === $cache_mode && $object_cache)
+		elseif ( self::CACHE_OBJECT === $cache_mode && $object_cache )
 			wp_cache_set($key, $value, $group, $expires);
 
 		return $value;
@@ -397,13 +397,13 @@ 
 	 * @param string $cache_mode
 	 * @return string
 	 */
-	private function _get_cache_mode($cache_mode) {
-		if (empty($cache_mode) || self::CACHE_USE_DEFAULT === $cache_mode) {
+	private function _get_cache_mode( $cache_mode ) {
+		if ( empty($cache_mode) || self::CACHE_USE_DEFAULT === $cache_mode ) {
 			$cache_mode = $this->cache_mode;
 		}
 
 		// Fallback if self::$cache_mode did not get a valid value
-		if (!in_array($cache_mode, self::$cache_modes)) {
+		if ( !in_array($cache_mode, self::$cache_modes) ) {
 			$cache_mode = self::CACHE_OBJECT;
 		}
 

lib/timber-menu.php

Braces +5 added lines, -5 removed lines

@@ -101,7 +101,7 @@ 
 		$menu = wp_get_nav_menu_items($menu_id);
 		if ($menu) {
 			_wp_menu_item_classes_by_context($menu);
-			if (is_array($menu)){
+			if (is_array($menu)) {
 				$menu = self::order_children($menu);
 			}
 			$this->items = $menu;
@@ -123,7 +123,7 @@ 
 				$mi->__title = $mi->post_title;
 			}
 			_wp_menu_item_classes_by_context($menu);
-			if (is_array($menu)){
+			if (is_array($menu)) {
 				$menu = self::order_children($menu);
 			}
 			$this->items = $menu;
@@ -201,13 +201,13 @@ 
 				$item->__title = $item->title;
 				unset($item->title);
 			}
-			if(isset($item->ID)){
-				if (is_object($item) && get_class($item) == 'WP_Post'){
+			if(isset($item->ID)) {
+				if (is_object($item) && get_class($item) == 'WP_Post') {
 					$old_menu_item = $item;
 					$item = new $this->PostClass($item);
 				}
 				$menu_item = new $this->MenuItemClass($item);
-				if (isset($old_menu_item)){
+				if (isset($old_menu_item)) {
 					$menu_item->import_classes($old_menu_item);
 				}
 				$index[$item->ID] = $menu_item;

Spacing +33 added lines, -33 removed lines

@@ -75,18 +75,18 @@ 
 	/**
 	 * @param int|string $slug
 	 */
-	function __construct($slug = 0) {
+	function __construct( $slug = 0 ) {
 		$locations = get_nav_menu_locations();
-		if ($slug != 0 && is_numeric($slug)) {
+		if ( $slug != 0 && is_numeric($slug) ) {
 			$menu_id = $slug;
-		} else if (is_array($locations) && count($locations)) {
+		} else if ( is_array($locations) && count($locations) ) {
 			$menu_id = $this->get_menu_id_from_locations($slug, $locations);
-		} else if ($slug === false) {
+		} else if ( $slug === false ) {
 			$menu_id = false;
 		} else {
 			$menu_id = $this->get_menu_id_from_terms($slug);
 		}
-		if ($menu_id) {
+		if ( $menu_id ) {
 			$this->init($menu_id);
 		} else {
 			$this->init_as_page_menu();
@@ -97,11 +97,11 @@ 
 	 * @internal
 	 * @param int $menu_id
 	 */
-	protected function init($menu_id) {
+	protected function init( $menu_id ) {
 		$menu = wp_get_nav_menu_items($menu_id);
-		if ($menu) {
+		if ( $menu ) {
 			_wp_menu_item_classes_by_context($menu);
-			if (is_array($menu)){
+			if ( is_array($menu) ) {
 				$menu = self::order_children($menu);
 			}
 			$this->items = $menu;
@@ -118,12 +118,12 @@ 
 	 */
 	protected function init_as_page_menu() {
 		$menu = get_pages();
-		if ($menu) {
-			foreach($menu as $mi) {
+		if ( $menu ) {
+			foreach ( $menu as $mi ) {
 				$mi->__title = $mi->post_title;
 			}
 			_wp_menu_item_classes_by_context($menu);
-			if (is_array($menu)){
+			if ( is_array($menu) ) {
 				$menu = self::order_children($menu);
 			}
 			$this->items = $menu;
@@ -136,14 +136,14 @@ 
 	 * @param array $locations
 	 * @return integer
 	 */
-	protected function get_menu_id_from_locations($slug, $locations) {
-		if ($slug === 0) {
+	protected function get_menu_id_from_locations( $slug, $locations ) {
+		if ( $slug === 0 ) {
 			$slug = $this->get_menu_id_from_terms($slug);
 		}
-		if (is_numeric($slug)) {
+		if ( is_numeric($slug) ) {
 			$slug = array_search($slug, $locations);
 		}
-		if (isset($locations[$slug])) {
+		if ( isset($locations[$slug]) ) {
 			$menu_id = $locations[$slug];
 			return $menu_id;
 		}
@@ -154,21 +154,21 @@ 
 	 * @param int $slug
 	 * @return int
 	 */
-	protected function get_menu_id_from_terms($slug = 0) {
-		if (!is_numeric($slug) && is_string($slug)) {
+	protected function get_menu_id_from_terms( $slug = 0 ) {
+		if ( !is_numeric($slug) && is_string($slug) ) {
 			//we have a string so lets search for that
 			$menu_id = get_term_by('slug', $slug, 'nav_menu');
-			if ($menu_id) {
+			if ( $menu_id ) {
 				return $menu_id;
 			}
 			$menu_id = get_term_by('name', $slug, 'nav_menu');
-			if ($menu_id) {
+			if ( $menu_id ) {
 				return $menu_id;
 			}
 		}
 		$menus = get_terms('nav_menu', array('hide_empty' => true));
-		if (is_array($menus) && count($menus)) {
-			if (isset($menus[0]->term_id)) {
+		if ( is_array($menus) && count($menus) ) {
+			if ( isset($menus[0]->term_id) ) {
 				return $menus[0]->term_id;
 			}
 		}
@@ -180,9 +180,9 @@ 
 	 * @param int $parent_id
 	 * @return TimberMenuItem|null
 	 */
-	function find_parent_item_in_menu($menu_items, $parent_id) {
-		foreach ($menu_items as &$item) {
-			if ($item->ID == $parent_id) {
+	function find_parent_item_in_menu( $menu_items, $parent_id ) {
+		foreach ( $menu_items as &$item ) {
+			if ( $item->ID == $parent_id ) {
 				return $item;
 			}
 		}
@@ -193,29 +193,29 @@ 
 	 * @param array $items
 	 * @return array
 	 */
-	protected function order_children($items) {
+	protected function order_children( $items ) {
 		$index = array();
 		$menu = array();
-		foreach ($items as $item) {
-			if (isset($item->title)) {
+		foreach ( $items as $item ) {
+			if ( isset($item->title) ) {
 				//items from wp can come with a $title property which conflicts with methods
 				$item->__title = $item->title;
 				unset($item->title);
 			}
-			if(isset($item->ID)){
-				if (is_object($item) && get_class($item) == 'WP_Post'){
+			if ( isset($item->ID) ) {
+				if ( is_object($item) && get_class($item) == 'WP_Post' ) {
 					$old_menu_item = $item;
 					$item = new $this->PostClass($item);
 				}
 				$menu_item = new $this->MenuItemClass($item);
-				if (isset($old_menu_item)){
+				if ( isset($old_menu_item) ) {
 					$menu_item->import_classes($old_menu_item);
 				}
 				$index[$item->ID] = $menu_item;
 			}
 		}
-		foreach ($index as $item) {
-			if (isset($item->menu_item_parent) && $item->menu_item_parent && isset($index[$item->menu_item_parent])) {
+		foreach ( $index as $item ) {
+			if ( isset($item->menu_item_parent) && $item->menu_item_parent && isset($index[$item->menu_item_parent]) ) {
 				$index[$item->menu_item_parent]->add_child($item);
 			} else {
 				$menu[] = $item;
@@ -228,7 +228,7 @@ 
 	 * @return array
 	 */
 	function get_items() {
-		if (is_array($this->items)) {
+		if ( is_array($this->items) ) {
 			return $this->items;
 		}
 		return array();