anrw / classy

app/classy/classy-hierarchy.php

Doc Comments +3 added lines, -2 removed lines

@@ -131,7 +131,8 @@ 
 	 *
 	 * @param  string $type view|scope
 	 * @param  string $type
-	 * @return array
+	 * @param string $page
+	 * @return string|false
 	 */
 	public static function get_available_file( $type = 'view', $page ) {
 
@@ -374,7 +375,7 @@ 
 	/**
 	 * Returns classy template name or boolean if this is not classy template
 	 *
-	 * @return mixed
+	 * @return string|false
 	 */
 	public static function get_classy_template() {
 

Spacing +67 added lines, -67 removed lines

@@ -20,39 +20,39 @@ 
 	 */
 	protected static function check_request() {
 
-		if ( is_404() ) : return '404';
+		if (is_404()) : return '404';
 
-		elseif ( is_search() ) : return 'search';
+		elseif (is_search()) : return 'search';
 
-		elseif ( is_front_page() ) : return 'front-page';
+		elseif (is_front_page()) : return 'front-page';
 
-		elseif ( is_home() ) : return 'home';
+		elseif (is_home()) : return 'home';
 
-		elseif ( is_post_type_archive() ) : return 'post_type_archive';
+		elseif (is_post_type_archive()) : return 'post_type_archive';
 
-		elseif ( is_tax() ) : return 'taxonomy';
+		elseif (is_tax()) : return 'taxonomy';
 
-		elseif ( is_attachment() ) : return 'attachment';
+		elseif (is_attachment()) : return 'attachment';
 
-		elseif ( is_single() ) : return 'single';
+		elseif (is_single()) : return 'single';
 
-		elseif ( self::is_classy_template() ) : return 'classy-template';
+		elseif (self::is_classy_template()) : return 'classy-template';
 
-		elseif ( is_page() ) : return 'page';
+		elseif (is_page()) : return 'page';
 
-		elseif ( is_singular() ) : return 'singular';
+		elseif (is_singular()) : return 'singular';
 
-		elseif ( is_category() ) : return 'category';
+		elseif (is_category()) : return 'category';
 
-		elseif ( is_tag() ) : return 'tag';
+		elseif (is_tag()) : return 'tag';
 
-		elseif ( is_author() ) : return 'author';
+		elseif (is_author()) : return 'author';
 
-		elseif ( is_date() ) : return 'date';
+		elseif (is_date()) : return 'date';
 
-		elseif ( is_archive() ) : return 'archive';
+		elseif (is_archive()) : return 'archive';
 
-		elseif ( is_paged() ) : return 'paged';
+		elseif (is_paged()) : return 'paged';
 
 		else :
 
@@ -69,7 +69,7 @@ 
 	 */
 	public static function get_current_request() {
 
-		if ( null === self::$current_request ) {
+		if (null === self::$current_request) {
 
 			self::$current_request = self::check_request();
 
@@ -87,21 +87,21 @@ 
 	 * @param  string $view path to view ex: "post/archive"
 	 * @return string           full fule path
 	 */
-	public static function get_file_path( $type = 'view', $view ) {
+	public static function get_file_path($type = 'view', $view) {
 
-		$view = str_replace( '.', '/', $view );
+		$view = str_replace('.', '/', $view);
 
-		if ( 'view' == $type ) {
+		if ('view' == $type) {
 
 			$folder = ClassyView::$folder;
 
-			return THEME_PATH . $folder . '/' . $view . '.blade.php';
+			return THEME_PATH.$folder.'/'.$view.'.blade.php';
 
-		} elseif ( 'scope' == $type ) {
+		} elseif ('scope' == $type) {
 
 			$folder = ClassyScope::$folder;
 
-			return THEME_PATH . $folder . '/' . $view . '.php';
+			return THEME_PATH.$folder.'/'.$view.'.php';
 
 		}
 
@@ -115,13 +115,13 @@ 
 	 * @param  string $file in blade path format, ex: layout|header
 	 * @return boolean true|false
 	 */
-	public static function file_exists( $type = 'view', $file ) {
+	public static function file_exists($type = 'view', $file) {
 
-		$file = str_replace( '.', '/', $file );
+		$file = str_replace('.', '/', $file);
 
-		$file_path = self::get_file_path( $type, $file );
+		$file_path = self::get_file_path($type, $file);
 
-		return file_exists( $file_path );
+		return file_exists($file_path);
 
 	}
 
@@ -133,13 +133,13 @@ 
 	 * @param  string $type
 	 * @return array
 	 */
-	public static function get_available_file( $type = 'view', $page ) {
+	public static function get_available_file($type = 'view', $page) {
 
-		$views = self::get_request_hierarchy_list( $page );
+		$views = self::get_request_hierarchy_list($page);
 
-		foreach ( $views as $view ) {
+		foreach ($views as $view) {
 
-			if ( self::file_exists( $type, $view ) ) :
+			if (self::file_exists($type, $view)) :
 
 				return $view;
 
@@ -158,43 +158,43 @@ 
 	 * @param  string $type
 	 * @return array
 	 */
-	private static function get_request_hierarchy_list( $type ) {
+	private static function get_request_hierarchy_list($type) {
 
 		$views = array();
 
 		// Home
 
-		if ( 'home' == $type ) :
+		if ('home' == $type) :
 
 			$views[] = 'home';
 
 			// Single
 
-		elseif ( 'single' == $type ) :
+		elseif ('single' == $type) :
 
 			$post_type = get_post_type();
 
-			$views[] = $post_type . '.single';
+			$views[] = $post_type.'.single';
 
 			$views[] = 'single';
 
 			// Post type
 
-		elseif ( 'post_type_archive' == $type ) :
+		elseif ('post_type_archive' == $type) :
 
 			$post_type = get_post_type();
 
-			$views[] = $post_type . '.archive';
+			$views[] = $post_type.'.archive';
 
 			$views[] = 'archive';
 
 			// Taxonomy
 
-		elseif ( 'taxonomy' == $type ) :
+		elseif ('taxonomy' == $type) :
 
 			$term = get_queried_object();
 
-			if ( ! empty( $term->slug ) ) {
+			if (!empty($term->slug)) {
 
 				$taxonomy = $term->taxonomy;
 
@@ -209,11 +209,11 @@ 
 
 			// Category
 
-		elseif ( 'category' == $type ) :
+		elseif ('category' == $type) :
 
 			$category = get_queried_object();
 
-			if ( ! empty( $category->slug ) ) {
+			if (!empty($category->slug)) {
 				$views[] = "category.{$category->slug}";
 				$views[] = "category.{$category->term_id}";
 			}
@@ -224,23 +224,23 @@ 
 
 			// Attachment
 
-		elseif ( 'attachment' == $type ) :
+		elseif ('attachment' == $type) :
 
 			$attachment = get_queried_object();
 
-			if ( $attachment ) {
+			if ($attachment) {
 
-				if ( false !== strpos( $attachment->post_mime_type, '/' ) ) {
+				if (false !== strpos($attachment->post_mime_type, '/')) {
 
-					list( $type, $subtype ) = explode( '/', $attachment->post_mime_type );
+					list($type, $subtype) = explode('/', $attachment->post_mime_type);
 
 				} else {
 
-					list( $type, $subtype ) = array( $attachment->post_mime_type, '' );
+					list($type, $subtype) = array($attachment->post_mime_type, '');
 
 				}
 
-				if ( ! empty( $subtype ) ) {
+				if (!empty($subtype)) {
 					$views[] = "attachment.{$type}.{$subtype}";
 					$views[] = "attachment.{$subtype}";
 
@@ -259,11 +259,11 @@ 
 
 			// Tag
 
-		elseif ( 'tag' == $type ) :
+		elseif ('tag' == $type) :
 
 			$tag = get_queried_object();
 
-			if ( ! empty( $tag->slug ) ) {
+			if (!empty($tag->slug)) {
 				$views[] = "post.tag.{$tag->slug}";
 				$views[] = "post.tag.{$tag->term_id}";
 
@@ -276,11 +276,11 @@ 
 
 			// Author
 
-		elseif ( 'author' == $type ) :
+		elseif ('author' == $type) :
 
 			$author = get_queried_object();
 
-			if ( $author instanceof WP_User ) {
+			if ($author instanceof WP_User) {
 				$views[] = "post.author.{$author->user_nicename}";
 				$views[] = "post.author.{$author->ID}";
 
@@ -294,7 +294,7 @@ 
 
 			// Front Page
 
-		elseif ( 'front-page' == $type ) :
+		elseif ('front-page' == $type) :
 
 			$views[] = 'front-page.front-page';
 			$views[] = 'front-page';
@@ -302,43 +302,43 @@ 
 			$views[] = 'home.home';
 			$views[] = 'home';
 
-			$views = array_merge( $views, self::get_request_hierarchy_list( 'post_type_archive' ) );
+			$views = array_merge($views, self::get_request_hierarchy_list('post_type_archive'));
 
 			// Page
 
-		elseif ( 'classy-template' == $type ) :
+		elseif ('classy-template' == $type) :
 
 			$template = self::get_classy_template();
 
 			$views[] = $template;
 
-			$views[] = 'page.' . $template;
+			$views[] = 'page.'.$template;
 
-			$views[] = 'template.' . $template;
+			$views[] = 'template.'.$template;
 
 			$views[] = 'page';
 
-		elseif ( 'page' == $type ) :
+		elseif ('page' == $type) :
 
 			$id = get_queried_object_id();
 
-			$pagename = get_query_var( 'pagename' );
+			$pagename = get_query_var('pagename');
 
-			if ( ! $pagename && $id ) {
+			if (!$pagename && $id) {
 				// If a static page is set as the front page, $pagename will not be set. Retrieve it from the queried object
 				$post = get_queried_object();
 
-				if ( $post ) {
+				if ($post) {
 					$pagename = $post->post_name;
 				}
 			}
 
-			if ( $pagename ) {
-				$views[] = 'page.' . $pagename;
+			if ($pagename) {
+				$views[] = 'page.'.$pagename;
 			}
 
-			if ( $id ) {
-				$views[] = 'page.' . $id;
+			if ($id) {
+				$views[] = 'page.'.$id;
 			}
 
 			$views[] = 'page.page';
@@ -380,9 +380,9 @@ 
 
 		$template_slug = get_page_template_slug();
 
-		preg_match( '/classy\-(.*)/', $template_slug, $matches );
+		preg_match('/classy\-(.*)/', $template_slug, $matches);
 
-		if ( ! empty( $matches ) && isset( $matches[1] ) ) { return $matches[1]; }
+		if (!empty($matches) && isset($matches[1])) { return $matches[1]; }
 
 		return false;
 

app/models/classy-post.php

Doc Comments +1 added lines, -1 removed lines

@@ -186,7 +186,7 @@
 	/**
 	 * Returns first attached image id
 	 *
-	 * @return int|boolean
+	 * @return integer|null
 	 */
 	public function get_first_attached_image_id() {
 

Spacing +75 added lines, -75 removed lines

@@ -73,12 +73,12 @@ 
 	 * Main constructor function. If ID won't be provided we will try to find it, based on your query
 	 * @param object/int $post
 	 */
-	public function __construct( $post = null ) {
-		if ( is_integer( $post ) ) {
+	public function __construct($post = null) {
+		if (is_integer($post)) {
 			$this->ID = $post;
 			$this->init();
-		} elseif ( is_a( $post, 'WP_Post' ) ) {
-			$this->import( $post );
+		} elseif (is_a($post, 'WP_Post')) {
+			$this->import($post);
 		}
 	}
 
@@ -88,8 +88,8 @@ 
 	protected function init() {
 		$post = $this->get_object();
 
-		if ( is_a( $post, 'WP_Post' ) ) {
-			$this->import( $post );
+		if (is_a($post, 'WP_Post')) {
+			$this->import($post);
 		}
 	}
 
@@ -100,7 +100,7 @@ 
 	 */
 
 	public function get_object() {
-		$object = get_post( $this->ID );
+		$object = get_post($this->ID);
 
 		return $object;
 	}
@@ -111,10 +111,10 @@ 
 	 * @return boolean
 	 */
 	public 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;
@@ -126,8 +126,8 @@ 
 	 * @return string
 	 */
 	public function get_edit_url() {
-		if ( $this->can_edit() ) {
-			return get_edit_post_link( $this->ID );
+		if ($this->can_edit()) {
+			return get_edit_post_link($this->ID);
 		}
 	}
 
@@ -149,9 +149,9 @@ 
 			'fields' => 'ids',
 		);
 
-		$images = get_children( $attrs );
+		$images = get_children($attrs);
 
-		if ( ! count( $images ) ) {
+		if (!count($images)) {
 			return false;
 		}
 
@@ -170,11 +170,11 @@ 
 
 		$images = $this->get_attached_images();
 
-		if ( $images ) {
+		if ($images) {
 
-			foreach ( $images as $image_id ) {
+			foreach ($images as $image_id) {
 
-				$_return[] = new ClassyImage( $image_id );
+				$_return[] = new ClassyImage($image_id);
 
 			}
 		}
@@ -201,13 +201,13 @@ 
 			'fields' => 'ids',
 		);
 
-		$images = get_children( $attrs );
+		$images = get_children($attrs);
 
-		if ( ! count( $images ) ) {
+		if (!count($images)) {
 			return false;
 		}
 
-		$images = array_values( $images );
+		$images = array_values($images);
 
 		return $images[0];
 
@@ -223,8 +223,8 @@ 
 
 		$image_id = $this->get_first_attached_image_id();
 
-		if ( $image_id ) {
-			return new ClassyImage( $image_id );
+		if ($image_id) {
+			return new ClassyImage($image_id);
 		}
 
 		return new ClassyImage();
@@ -238,12 +238,12 @@ 
 	 */
 	public function thumbnail() {
 
-		if ( function_exists( 'get_post_thumbnail_id' ) ) {
-			$image_id = get_post_thumbnail_id( $this->ID );
+		if (function_exists('get_post_thumbnail_id')) {
+			$image_id = get_post_thumbnail_id($this->ID);
 
-			if ( $image_id ) {
+			if ($image_id) {
 
-				return new ClassyImage( $image_id );
+				return new ClassyImage($image_id);
 
 			}
 		}
@@ -259,7 +259,7 @@ 
 	 * @return string
 	 */
 	public function get_title() {
-		return apply_filters( 'the_title', $this->post_title, $this->ID );
+		return apply_filters('the_title', $this->post_title, $this->ID);
 	}
 
 	/**
@@ -278,24 +278,24 @@ 
 	 * @param  integer $page Page number, in case our post has <!--nextpage--> tags
 	 * @return string        Post content
 	 */
-	public function get_content( $page = 0 ) {
-		if ( 0 == $page && $this->post_content ) {
+	public function get_content($page = 0) {
+		if (0 == $page && $this->post_content) {
 			return $this->post_content;
 		}
 
 		$content = $this->post_content;
 
-		if ( $page ) {
-			$contents = explode( '<!--nextpage-->', $content );
+		if ($page) {
+			$contents = explode('<!--nextpage-->', $content);
 
 			$page--;
 
-			if ( count( $contents ) > $page ) {
-				$content = $contents[ $page ];
+			if (count($contents) > $page) {
+				$content = $contents[$page];
 			}
 		}
 
-		$content = apply_filters( 'the_content', ($content) );
+		$content = apply_filters('the_content', ($content));
 
 		return $content;
 	}
@@ -316,7 +316,7 @@ 
 	 * @return object
 	 */
 	public function get_post_type() {
-		return get_post_type_object( $this->post_type );
+		return get_post_type_object($this->post_type);
 	}
 
 	/**
@@ -325,11 +325,11 @@ 
 	 * @return string
 	 */
 	public function get_permalink() {
-		if ( isset( $this->permalink ) ) {
+		if (isset($this->permalink)) {
 			return $this->permalink;
 		}
 
-		$this->permalink = get_permalink( $this->ID );
+		$this->permalink = get_permalink($this->ID);
 
 		return $this->permalink;
 	}
@@ -355,91 +355,91 @@ 
 	 * @param  boolean $strip    Should we strip tags?
 	 * @return string            Post preview
 	 */
-	public function get_preview( $len = 50, $force = false, $readmore = 'Read More', $strip = true ) {
+	public function get_preview($len = 50, $force = false, $readmore = 'Read More', $strip = true) {
 		$text = '';
 		$trimmed = false;
 
-		if ( isset( $this->post_excerpt ) && strlen( $this->post_excerpt ) ) {
+		if (isset($this->post_excerpt) && strlen($this->post_excerpt)) {
 
-			if ( $force ) {
-				$text = ClassyHelper::trim_words( $this->post_excerpt, $len, false );
+			if ($force) {
+				$text = ClassyHelper::trim_words($this->post_excerpt, $len, false);
 				$trimmed = true;
 			} else {
 				$text = $this->post_excerpt;
 			}
 		}
 
-		if ( ! strlen( $text ) && preg_match( '/<!--\s?more(.*?)?-->/', $this->post_content, $readmore_matches ) ) {
+		if (!strlen($text) && preg_match('/<!--\s?more(.*?)?-->/', $this->post_content, $readmore_matches)) {
 
-			$pieces = explode( $readmore_matches[0], $this->post_content );
+			$pieces = explode($readmore_matches[0], $this->post_content);
 			$text = $pieces[0];
 
-			if ( $force ) {
-				$text = ClassyHelper::trim_words( $text, $len, false );
+			if ($force) {
+				$text = ClassyHelper::trim_words($text, $len, false);
 				$trimmed = true;
 			}
 
-			$text = do_shortcode( $text );
+			$text = do_shortcode($text);
 
 		}
 
-		if ( ! strlen( $text ) ) {
+		if (!strlen($text)) {
 
-			$text = ClassyHelper::trim_words( $this->get_content(), $len, false );
+			$text = ClassyHelper::trim_words($this->get_content(), $len, false);
 			$trimmed = true;
 
 		}
 
-		if ( ! strlen( trim( $text ) ) ) {
+		if (!strlen(trim($text))) {
 
-			return trim( $text );
+			return trim($text);
 
 		}
 
-		if ( $strip ) {
+		if ($strip) {
 
-			$text = trim( strip_tags( $text ) );
+			$text = trim(strip_tags($text));
 
 		}
 
-		if ( strlen( $text ) ) {
+		if (strlen($text)) {
 
-			$text = trim( $text );
-			$last = $text[ strlen( $text ) - 1 ];
+			$text = trim($text);
+			$last = $text[strlen($text) - 1];
 
-			if ( '.' != $last && $trimmed ) {
+			if ('.' != $last && $trimmed) {
 				$text .= ' &hellip; ';
 			}
 
-			if ( ! $strip ) {
-				$last_p_tag = strrpos( $text, '</p>' );
-				if ( false !== $last_p_tag ) {
-					$text = substr( $text, 0, $last_p_tag );
+			if (!$strip) {
+				$last_p_tag = strrpos($text, '</p>');
+				if (false !== $last_p_tag) {
+					$text = substr($text, 0, $last_p_tag);
 				}
-				if ( '.' != $last && $trimmed ) {
+				if ('.' != $last && $trimmed) {
 					$text .= ' &hellip; ';
 				}
 			}
 
-			if ( $readmore && isset( $readmore_matches ) && ! empty( $readmore_matches[1] ) ) {
-				$text .= ' <a href="' . $this->get_permalink() . '" class="read-more">' . trim( $readmore_matches[1] ) . '</a>';
-			} elseif ( $readmore ) {
-				$text .= ' <a href="' . $this->get_permalink() . '" class="read-more">' . trim( $readmore ) . '</a>';
+			if ($readmore && isset($readmore_matches) && !empty($readmore_matches[1])) {
+				$text .= ' <a href="'.$this->get_permalink().'" class="read-more">'.trim($readmore_matches[1]).'</a>';
+			} elseif ($readmore) {
+				$text .= ' <a href="'.$this->get_permalink().'" class="read-more">'.trim($readmore).'</a>';
 			}
 
-			if ( ! $strip ) {
+			if (!$strip) {
 				$text .= '</p>';
 			}
 		}
 
-		return trim( $text );
+		return trim($text);
 	}
 
 	/**
 	 * Returns comments array
 	 * @return array
 	 */
-	public function get_comments( $status = 'approve', $order = 'DESC' ) {
+	public function get_comments($status = 'approve', $order = 'DESC') {
 
 		$_return = array();
 
@@ -449,26 +449,26 @@ 
 			'order' => $order,
 		);
 
-		$comments = get_comments( $args );
+		$comments = get_comments($args);
 
-		foreach ( $comments as $comment ) {
+		foreach ($comments as $comment) {
 
-			$_return[ $comment->comment_ID ] = new ClassyComment( $comment );
+			$_return[$comment->comment_ID] = new ClassyComment($comment);
 
 		}
 
-		foreach ( $_return as $key => $comment ) {
+		foreach ($_return as $key => $comment) {
 
-			if ( $comment->has_parent() ) {
+			if ($comment->has_parent()) {
 
-				$_return[ $comment->comment_parent ]->add_child( $comment );
+				$_return[$comment->comment_parent]->add_child($comment);
 
-				unset( $_return[ $key ] );
+				unset($_return[$key]);
 
 			}
 		}
 
-		return array_values( $_return );
+		return array_values($_return);
 
 	}
 }

app/models/classy-image.php

Spacing +6 added lines, -6 removed lines

@@ -12,10 +12,10 @@ 
 	 * Main constructor function. Requires image id
 	 * @param int $pid
 	 */
-	public function __construct( $pid = null ) {
+	public function __construct($pid = null) {
 
 		// Checks if image with this id exists
-		if ( null !== $pid && wp_get_attachment_image_src( $pid ) ) {
+		if (null !== $pid && wp_get_attachment_image_src($pid)) {
 
 			$this->ID = $pid;
 
@@ -33,7 +33,7 @@ 
 	public static function get_default_image() {
 
 		// You can put here any url
-		return THEME_DIR . '/assets/noimage.png';
+		return THEME_DIR.'/assets/noimage.png';
 
 	}
 
@@ -42,11 +42,11 @@ 
 	 * @param  string $size
 	 * @return string
 	 */
-	public function src( $size = 'medium' ) {
+	public function src($size = 'medium') {
 
-		if ( $this->ID ) {
+		if ($this->ID) {
 
-			$thumb = wp_get_attachment_image_src( $this->ID, $size );
+			$thumb = wp_get_attachment_image_src($this->ID, $size);
 
 			return $thumb[0];