nirjharlo / complete-google-seo-scan

Plugin/Lib/Analysis/lib/score.php

Spacing +76 added lines, -76 removed lines

@@ -2,7 +2,7 @@ 
 
 namespace NirjharLo\Cgss\Lib\Analysis\Lib;
 
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined('ABSPATH')) exit;
 
 /**
  * From result data found in scan, analyze a score of on-page optimization. Using approximate values
@@ -36,35 +36,35 @@ 
 		$score = $this->exact();
 
 		$star = 0;
-		switch ( true ) {
-			case ( $score >= 560 ):
+		switch (true) {
+			case ($score >= 560):
 				$star = 10;
 				break;
-			case ( $score >= 490 ):
+			case ($score >= 490):
 				$star = 9;
 				break;
-			case ( $score >= 420 ):
+			case ($score >= 420):
 				$star = 8;
 				break;
-			case ( $score >= 350 ):
+			case ($score >= 350):
 				$star = 7;
 				break;
-			case ( $score >= 280 ):
+			case ($score >= 280):
 				$star = 6;
 				break;
-			case ( $score >= 210 ):
+			case ($score >= 210):
 				$star = 5;
 				break;
-			case ( $score >= 140 ):
+			case ($score >= 140):
 				$star = 4;
 				break;
-			case ( $score >= 70 ):
+			case ($score >= 70):
 				$star = 3;
 				break;
-			case ( $score >= 0 ):
+			case ($score >= 0):
 				$star = 2;
 				break;
-			case ( $score <= 0 ):
+			case ($score <= 0):
 				$star = 1;
 				break;
 		}
@@ -77,13 +77,13 @@ 
 		$data = $this->snippet;
 		$score = 0;
 
-		$arr = array( $data['title'], $data['desc'] );
-		foreach ( $arr as $key ) {
-			switch ( true ) {
-				case ( $key && $key != '' ):
+		$arr = array($data['title'], $data['desc']);
+		foreach ($arr as $key) {
+			switch (true) {
+				case ($key && $key != ''):
 					$score += 25;
 					break;
-				case ( ! $key || $key == '' ):
+				case ( ! $key || $key == ''):
 					$score -= 25;
 					break;
 			}
@@ -95,47 +95,47 @@ 
 	//image link to all link ratio and usage of most used word in anchor texts and heading texts: 200, 75
 	public function text() {
 		$data = $this->text;
-		if ( $data ) {
+		if ($data) {
 			$score = 0;
 			$count = $data['count'];
-			switch ( true ) {
-				case ( $count < 200 ):
+			switch (true) {
+				case ($count < 200):
 					$score -= 10;
 					break;
-				case ( $count > 200 ):
+				case ($count > 200):
 					$score += 25;
 					break;
 			}
 			$ratio = $data['ratio'];
-			switch ( true ) {
-				case ( $ratio <= 15 ):
+			switch (true) {
+				case ($ratio <= 15):
 					$score -= 10;
 					break;
-				case ( $ratio <= 70 ):
+				case ($ratio <= 70):
 					$score += 25;
 					break;
-				case ( $ratio > 70 ):
+				case ($ratio > 70):
 					$score -= 10;
 					break;
 			}
-			if ( $data['links'] ) {
+			if ($data['links']) {
 				$links = $data['links'];
-				if ( $count > 0 ) {
-					$link_ratio = ( ( $links['count'] / $count ) * 100 );
-					switch ( true ) {
-						case ( $link_ratio <= 50 ):
+				if ($count > 0) {
+					$link_ratio = (($links['count'] / $count) * 100);
+					switch (true) {
+						case ($link_ratio <= 50):
 							$score += 10;
 							break;
-						case ( $link_ratio > 50 ):
+						case ($link_ratio > 50):
 							$score -= 5;
 							break;
 					}
 				}
-				switch ( true ) {
-					case ( $links['no_text'] < 2 ):
+				switch (true) {
+					case ($links['no_text'] < 2):
 						$score += 10;
 						break;
-					case ( $links['no_text'] > 2 ):
+					case ($links['no_text'] > 2):
 						$score -= 5;
 						break;
 				}
@@ -151,17 +151,17 @@ 
 			$snip_data = $this->snippet;
 			$design_data = $this->design;
 			$img_data = $design_data['image'];
-			$alts = (!empty($img_data['alt_value']) ? implode(' ', $img_data['alt_value']) : []);
-			$url = implode( " ", explode( "-", $snip_data['url'] ) );
-			$arr = array( $snip_data['title'], $snip_data['desc'], $alts, $url, $anch, $hds );
-			foreach ( $arr as $val ) {
-				if ( $val && $val != '' ) {
-					$find_key = $this->key_chk( $val );
-					switch ( true ) {
-						case ( ! $find_key ):
+			$alts = ( ! empty($img_data['alt_value']) ? implode(' ', $img_data['alt_value']) : []);
+			$url = implode(" ", explode("-", $snip_data['url']));
+			$arr = array($snip_data['title'], $snip_data['desc'], $alts, $url, $anch, $hds);
+			foreach ($arr as $val) {
+				if ($val && $val != '') {
+					$find_key = $this->key_chk($val);
+					switch (true) {
+						case ( ! $find_key):
 							$score += 0;
 							break;
-						case ( $find_key > 0 ):
+						case ($find_key > 0):
 							$score += 10;
 							break;
 					}
@@ -176,11 +176,11 @@ 
 	//Score based on alt tags presence in images and top word found in those alt tags: 70, 45
 	public function design() {
 		$data = $this->design;
-		if ( $data ) {
+		if ($data) {
 			$score = 0;
 			$img_data = $data['image'];
 			$no_alt_data = $img_data['no_alt_count'];
-			switch ( $no_alt_data ) {
+			switch ($no_alt_data) {
 				case 0:
 					$score += 25;
 					break;
@@ -190,7 +190,7 @@ 
 			}
 
 			$nested_table = $data['nested_table'];
-			switch ( $nested_table ) {
+			switch ($nested_table) {
 				case 0:
 					$score += 10;
 					break;
@@ -200,7 +200,7 @@ 
 			}
 
 			$tag_style = $data['tag_style'];
-			switch ( $tag_style['count'] ) {
+			switch ($tag_style['count']) {
 				case 0:
 					$score += 10;
 					break;
@@ -212,7 +212,7 @@ 
 			$vport = $data['vport'];
 			$media = $data['media'];
 			$media_ok = $media['ok'];
-			if ( $vport and $media_ok and strlen( $vport ) > 0 and $media_ok > 0 ) {
+			if ($vport and $media_ok and strlen($vport) > 0 and $media_ok > 0) {
 				$score += 25;
 			} else {
 				$score -= 10;
@@ -226,19 +226,19 @@ 
 	//determine score number of http requests and presence of code errors, viewport and social tags: 80, 40
 	public function crawl() {
 		$data = $this->crawl;
-		if ( $data ) {
+		if ($data) {
 			$ip = $data['ip'];
 			$meta_robot = $data['meta_robot'];
 
-			if ( ! $data['cano'] or strlen( $data['cano'] ) != 0 ) {
+			if ( ! $data['cano'] or strlen($data['cano']) != 0) {
 				$cano = 1;
 			} else {
 				$cano = 0;
 			}
-			$arr = array( $data['ssl'], $data['dynamic'], $data['underscore'], $cano, $ip, $data['if_mod'], $meta_robot['ok'] );
+			$arr = array($data['ssl'], $data['dynamic'], $data['underscore'], $cano, $ip, $data['if_mod'], $meta_robot['ok']);
 			$score = 0;
-			foreach ( $arr as $val ) {
-				switch ( $val ) {
+			foreach ($arr as $val) {
+				switch ($val) {
 					case 1:
 						$score += 10;
 						break;
@@ -256,23 +256,23 @@ 
 	//Overview score for url and canonical tag. Excluding robots tag,to be taken care of by jquery: 70, 35
 	public function speed() {
 		$data = $this->speed;
-		if ( $data ) {
+		if ($data) {
 			$score = 0;
 			$down_time = $data['down_time'];
-			switch ( true ) {
-				case ( $down_time > 10 ):
+			switch (true) {
+				case ($down_time > 10):
 					$score -= 5;
 					break;
-				case ( $down_time < 3 ):
+				case ($down_time < 3):
 					$score += 10;
 					break;
-				case ( $down_time <= 5 ):
+				case ($down_time <= 5):
 					$score += 5;
 					break;
 			}
-			$arr = array( $data['gzip'], $data['cache'] );
-			foreach ( $arr as $val ) {
-				switch ( $val ) {
+			$arr = array($data['gzip'], $data['cache']);
+			foreach ($arr as $val) {
+				switch ($val) {
 					case 1:
 						$score += 10;
 						break;
@@ -283,24 +283,24 @@ 
 			}
 			$css = $data['css'];
 			$js = $data['js'];
-			$files_arr = array( $css['count'], $js['count'] );
-			foreach ( $files_arr as $val ) {
-				switch ( true ) {
-					case ( $val <= 10 ):
+			$files_arr = array($css['count'], $js['count']);
+			foreach ($files_arr as $val) {
+				switch (true) {
+					case ($val <= 10):
 						$score += 10;
 						break;
-					case ( $val > 10 ):
+					case ($val > 10):
 						$score -= 5;
 						break;
 				}
 			}
-			$files_size_arr = array( $css['size'], $js['size'] );
-			foreach ( $files_size_arr as $val ) {
-				switch ( true ) {
-					case ( $val <= 100 ):
+			$files_size_arr = array($css['size'], $js['size']);
+			foreach ($files_size_arr as $val) {
+				switch (true) {
+					case ($val <= 100):
 						$score += 10;
 						break;
-					case ( $val > 100 ):
+					case ($val > 100):
 						$score -= 5;
 						break;
 				}
@@ -312,18 +312,18 @@ 
 	}
 
 	//Get the top word found
-	public function key_chk( $txt ) {
+	public function key_chk($txt) {
 		$show = 0;
 		$text = $this->text;
 		$keys = $text['keys'];
-		foreach ( $keys as $key => $val ) {
+		foreach ($keys as $key => $val) {
 
-			$num = substr_count( $key, $txt );
-			if ( $num > 0 ) {
+			$num = substr_count($key, $txt);
+			if ($num > 0) {
 				$show = 1;
 			}
 		}
-		if ( $show == 0 ) {
+		if ($show == 0) {
 			return false;
 		} else {
 			return $show;

Plugin/Lib/Analysis/crawl.php

Spacing +51 added lines, -51 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 namespace NirjharLo\Cgss\Lib\Analysis;
 
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined('ABSPATH')) exit;
 
 use \NirjharLo\Cgss\Lib\Analysis\Lib\Tags;
 use \NirjharLo\Cgss\Lib\Analysis\Lib\Text;
@@ -29,7 +29,7 @@ 
 
 			$this->header = $this->header_check();
 			$ok = $this->header['ok'];
-			if (!$ok) {
+			if ( ! $ok) {
 
 			 	$this->no_header_scan();
 
@@ -38,7 +38,7 @@ 
 				$this->parsed_url = $this->parse_url();
 				$this->body = $this->get_body();
 				$ok = $this->body['ok'];
-				if (!$ok) {
+				if ( ! $ok) {
 
 			 		$this->no_body_scan();
 
@@ -46,8 +46,8 @@ 
 
 					// Prepare the DOM
 					$this->dom = new DOMDocument;
-					libxml_use_internal_errors( true );
-					$this->dom->loadHTML( $this->body['body'] );
+					libxml_use_internal_errors(true);
+					$this->dom->loadHTML($this->body['body']);
 
 					// Parse the DOM
 					$this->title = $this->get_title();
@@ -79,7 +79,7 @@ 
 
 			$toc = microtime(true);
 
-			$this->time = round( ( $toc - $tic ), 3 );
+			$this->time = round(($toc - $tic), 3);
 		}
 
 
@@ -195,7 +195,7 @@ 
 			$js = new Tags();
 			$js->dom = $this->dom;
 			$js->tag = 'script';
-			$js->atts = array( 'src' );
+			$js->atts = array('src');
 
 			$js_list = $js->tag();
 
@@ -214,7 +214,7 @@ 
 			$css = new Tags();
 			$css->dom = $this->dom;
 			$css->tag = 'link';
-			$css->specify = array( 'att' => 'rel', 'val' => 'stylesheet', 'get_att' => 'href' );
+			$css->specify = array('att' => 'rel', 'val' => 'stylesheet', 'get_att' => 'href');
 
 			$css_list = $css->tag();
 
@@ -278,7 +278,7 @@ 
 		public function get_text_vars() {
 
 			$text = new Text();
-			$text->dom =  $this->dom;
+			$text->dom = $this->dom;
 			$text->body_size = $this->body['size'];
 			$text->execute();
 			$text_string = $text->text();
@@ -289,9 +289,9 @@ 
 			$words = $text->words();
 			$text_data['words'] = $words;
 			$text_data['text'] = $text_string;
-			$text_data['count'] = count( $words );
+			$text_data['count'] = count($words);
 			$size = $text->size();
-			$text_data['size'] = round( ( $size / 1024 ), 1 );
+			$text_data['size'] = round(($size / 1024), 1);
 			$text_data['ratio'] = $text->ratio();
 
 			return $text_data;
@@ -301,13 +301,13 @@ 
 		//Get nested table
 		public function get_nested_table() {
 
-			$table = $this->dom->getElementsByTagName( 'table' );
+			$table = $this->dom->getElementsByTagName('table');
 
 			$nested_table_count = 0;
-			if ( $table ) {
-				foreach ( $table as $obj ) {
-					$nested_table = $obj->getElementsByTagName( 'table' );
-					$nested_table_count = ( $nest_table ? $nested_table_count + 1 : $nested_table_count );
+			if ($table) {
+				foreach ($table as $obj) {
+					$nested_table = $obj->getElementsByTagName('table');
+					$nested_table_count = ($nest_table ? $nested_table_count + 1 : $nested_table_count);
 				}
 			}
 
@@ -318,10 +318,10 @@ 
 		//Get heading tags
 		public function get_headings() {
 
-			$all_headings = array( 'h1', 'h2', 'h3', 'h4', 'h5', 'h6' );
+			$all_headings = array('h1', 'h2', 'h3', 'h4', 'h5', 'h6');
 
 			$headings = array();
-			foreach ( $all_headings as $value ) {
+			foreach ($all_headings as $value) {
 
 				$head = new Tags();
 				$head->dom = $this->dom;
@@ -331,10 +331,10 @@ 
 				$headings[$value] = $head_tag;
 			}
 
-			$headings_filtered = array_filter( $headings );
+			$headings_filtered = array_filter($headings);
 
 			$heading = array();
-			$heading['names'] = array_keys( $headings_filtered );
+			$heading['names'] = array_keys($headings_filtered);
 			$heading['content'] = $headings_filtered;
 
 			return $heading;
@@ -367,15 +367,15 @@ 
 					);
 
 			$tag_style_value = array();
-			foreach( $tags as $value ) {
+			foreach ($tags as $value) {
 
 				$tag = new Tags();
 				$tag->dom = $this->dom;
 				$tag->tag = $value;
-				$tag->specify =  array( 'att' => null, 'val' => null, 'get_att' => 'style', );
+				$tag->specify = array('att' => null, 'val' => null, 'get_att' => 'style',);
 
 				$tag_style = $tag->tag();
-				$tag_style_value[] = ( $tag_style ? $value : false );
+				$tag_style_value[] = ($tag_style ? $value : false);
 			}
 
 			$tag_style_filtered = array_filter($tag_style_value);
@@ -393,7 +393,7 @@ 
 			$iframes = new Tags();
 			$iframes->dom = $this->dom;
 			$iframes->tag = 'iframe';
-			$iframes->specify = array( 'att' => null, 'val' => null, 'get_att' => 'src' );
+			$iframes->specify = array('att' => null, 'val' => null, 'get_att' => 'src');
 
 			$get_iframe = $iframes->tag();
 
@@ -407,7 +407,7 @@ 
 			$images = new Tags();
 			$images->dom = $this->dom;
 			$images->tag = 'img';
-			$images->atts = array( 'src', 'alt' );
+			$images->atts = array('src', 'alt');
 
 			$attributes = $images->atts();
 
@@ -430,7 +430,7 @@ 
 			$links = new Tags();
 			$links->dom = $this->dom;
 			$links->tag = 'a';
-			$links->atts = array( 'rel', 'href' );
+			$links->atts = array('rel', 'href');
 
 			$attributes = $links->atts();
 			$anchors = $links->tag();
@@ -446,7 +446,7 @@ 
 				if (filter_var($value, FILTER_VALIDATE_URL)) {
 					$link_num = $link_num + 1;
 
-					if( parse_url($value, PHP_URL_HOST) == $domain ) {
+					if (parse_url($value, PHP_URL_HOST) == $domain) {
 						$ext_link_num = $ext_link_num + 1;
 					}
 				}
@@ -454,8 +454,8 @@ 
 
 			$nofollow_link_num = 0;
 			foreach ($rels as $value) {
-				if( strpos($value, 'nofollow') !== false ) {
-					$nofollow_link_num = $nofollow_link_num +1;
+				if (strpos($value, 'nofollow') !== false) {
+					$nofollow_link_num = $nofollow_link_num + 1;
 				}
 			}
 
@@ -479,15 +479,15 @@ 
 
 			$tags = array('title', 'description', 'url', 'image');
 			$social_tag_val = array();
-			foreach( $tags as $value ) {
+			foreach ($tags as $value) {
 
 				$social_tag = new Tags();
 				$social_tag->dom = $this->dom;
 				$social_tag->tag = 'meta';
-				$social_tag->specify = array('att' => 'property', 'val' => 'og:'.$value, 'get_att' => 'content');
+				$social_tag->specify = array('att' => 'property', 'val' => 'og:' . $value, 'get_att' => 'content');
 				$social_tag_fetch = $social_tag->tag();
 
-				$social_tag_val[$value] = ( $social_tag_fetch ? array_pop($social_tag_fetch) : false );
+				$social_tag_val[$value] = ($social_tag_fetch ? array_pop($social_tag_fetch) : false);
 			}
 
 			return $social_tag_val;
@@ -500,11 +500,11 @@ 
 			$viewport = new Tags();
 			$viewport->dom = $this->dom;
 			$viewport->tag = 'meta';
-			$viewport->specify = array( 'att' => 'name', 'val' => 'viewport', 'get_att' => 'content' );
+			$viewport->specify = array('att' => 'name', 'val' => 'viewport', 'get_att' => 'content');
 			$viewport_tag = $viewport->tag();
 
-			$meta_viewport_val = ( $viewport_tag ? array_pop($viewport_tag) : false );
-			$meta_viewport = ( $meta_viewport_val && $meta_viewport_val != '' ) ? 1 : 0;
+			$meta_viewport_val = ($viewport_tag ? array_pop($viewport_tag) : false);
+			$meta_viewport = ($meta_viewport_val && $meta_viewport_val != '') ? 1 : 0;
 
 			return $meta_viewport;
 		}
@@ -519,11 +519,11 @@ 
 			$canonical = new Tags();
 			$canonical->dom = $this->dom;
 			$canonical->tag = 'meta';
-			$canonical->specify = array( 'att' => 'rel', 'val' => 'canonical', 'get_att' => 'href' );
+			$canonical->specify = array('att' => 'rel', 'val' => 'canonical', 'get_att' => 'href');
 			$canonical_tag = $canonical->tag();
 
-			$meta_canonical['val'] = ( $canonical_tag ? esc_url_raw(array_pop($canonical_tag)) : false );
-			$meta_canonical['ok'] = ( $meta_canonical['val'] && $meta_canonical['val'] != '' ) ? 1 : 0;
+			$meta_canonical['val'] = ($canonical_tag ? esc_url_raw(array_pop($canonical_tag)) : false);
+			$meta_canonical['ok'] = ($meta_canonical['val'] && $meta_canonical['val'] != '') ? 1 : 0;
 
 			return $meta_canonical;
 		}
@@ -538,10 +538,10 @@ 
 			$robot = new Tags();
 			$robot->dom = $this->dom;
 			$robot->tag = 'meta';
-			$robot->specify = array( 'att' => 'name', 'val' => 'robots', 'get_att' => 'content' );
+			$robot->specify = array('att' => 'name', 'val' => 'robots', 'get_att' => 'content');
 			$robot_tag = $robot->tag();
 
-			$meta_robot['val'] = ( $robot_tag ? array_pop($robot_tag) : false );
+			$meta_robot['val'] = ($robot_tag ? array_pop($robot_tag) : false);
 			$meta_robot['ok'] = $meta_robot['val'] && strpos($meta_robot['val'], 'index') !== false && strpos($meta_robot['val'], 'follow') !== false ? 1 : 0;
 
 			return $meta_robot;
@@ -555,10 +555,10 @@ 
 			$description = new Tags();
 			$description->dom = $this->dom;
 			$description->tag = 'meta';
-			$description->specify = array( 'att' => 'name', 'val' => 'description', 'get_att' => 'content' );
+			$description->specify = array('att' => 'name', 'val' => 'description', 'get_att' => 'content');
 			$desc = $description->tag();
 
-			return ( $desc ? array_pop($desc) : false );
+			return ($desc ? array_pop($desc) : false);
 		}
 
 
@@ -571,7 +571,7 @@ 
 			$title_fetch->tag = 'title';
 			$title = $title_fetch->tag();
 
-			return ( $title ? array_pop($title) : false );
+			return ($title ? array_pop($title) : false);
 		}
 
 
@@ -581,13 +581,13 @@ 
 			$response = array();
 
 			$start = microtime(true);
-			$body = @file_get_contents( $this->url, FILE_USE_INCLUDE_PATH );
+			$body = @file_get_contents($this->url, FILE_USE_INCLUDE_PATH);
 			$end = microtime(true);
 
 			$response['ok'] = empty($body) ? false : true;
-			$response['time'] = round( ( $end - $start ), 3 );
+			$response['time'] = round(($end - $start), 3);
 			$response['body'] = $body;
-			$response['size'] = mb_strlen( $body, '8bit' );
+			$response['size'] = mb_strlen($body, '8bit');
 
 			return $response;
 		}
@@ -600,10 +600,10 @@ 
 			$response = array();
 
 			$start = microtime(true);
-			$header = get_headers( $this->url, 1 );
+			$header = get_headers($this->url, 1);
 			$end = microtime(true);
 
-			$response['time'] = round( ( $end - $start ), 3 );
+			$response['time'] = round(($end - $start), 3);
 			$response['response'] = $header[0];
 			$response['ok'] = (strchr($header[0], '200') != false) ? true : false;
 
@@ -622,7 +622,7 @@ 
 			$parse['domain'] = $parsed_url['host'];
 			$parse['ssl'] = $parsed_url['scheme'] == 'http' ? false : true;
 			$parse['dynamic'] = array_key_exists('query', $parsed_url) ? true : false;
-			$parse['underscore'] = ( strpos($this->url, '_') !== false ) ? true : false;
+			$parse['underscore'] = (strpos($this->url, '_') !== false) ? true : false;
 
 			return $parse;
 		}
@@ -632,7 +632,7 @@ 
 		public function no_body_scan() { ?>
 
 			<div class="notice notice-error">
-        		<p><?php _e( 'The page could not be downloaded. Try again', 'cgss' ); ?></p>
+        		<p><?php _e('The page could not be downloaded. Try again', 'cgss'); ?></p>
 			</div>
     	<?PHP
 		}
@@ -642,7 +642,7 @@ 
 		public function no_header_scan() { ?>
 
 			<div class="notice notice-error">
-        		<p><?php _e( 'The header returned is not Ok. Try again', 'cgss' ); ?></p>
+        		<p><?php _e('The header returned is not Ok. Try again', 'cgss'); ?></p>
 			</div>
     	<?PHP
 		}

Plugin/Lib/Action/insight.php

Spacing +27 added lines, -27 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 namespace NirjharLo\Cgss\Lib\Action;
 
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined('ABSPATH')) exit;
 
 
 /**
@@ -12,12 +12,12 @@ 
 
 		public function __construct() {
 
-			try{
+			try {
 				$this->data = $this->fetch();
 				$this->compile();
 				$this->save();
-			} catch(Exception $e) {
-				echo __( 'Something went wrong', 'cgss') . ' ' . $e->get_message();
+			} catch (Exception $e) {
+				echo __('Something went wrong', 'cgss') . ' ' . $e->get_message();
 			}
 		}
 
@@ -39,7 +39,7 @@ 
 				$this->social,
 			);
 			foreach ($result as $key => $value) {
-				$sql = $wpdb->prepare("UPDATE {$wpdb->prefix}cgss_insight SET remark = %s WHERE ID = %d", $value, ($key+1));
+				$sql = $wpdb->prepare("UPDATE {$wpdb->prefix}cgss_insight SET remark = %s WHERE ID = %d", $value, ($key + 1));
 				$update = $wpdb->query($sql);
 			}
 		}
@@ -50,9 +50,9 @@ 
 
 			$score = $this->data['score'];
 			$this->count = (count($score) == 0 ? 1 : count($score));
-			$avarage_score = (count($score) == 0 ? 0 : round(array_sum($score)/count($score), 0));
+			$avarage_score = (count($score) == 0 ? 0 : round(array_sum($score) / count($score), 0));
 
-			$this->score = sprintf(__('Avarage SEO score is %d out of 10', 'cgss'),$avarage_score);
+			$this->score = sprintf(__('Avarage SEO score is %d out of 10', 'cgss'), $avarage_score);
 			$this->snippet = $this->snippet_analyze();
 			$this->text = $this->text_analyze();
 			$this->links = $this->link_analyze();
@@ -72,13 +72,13 @@ 
 			foreach ($snippets as $snippet) {
 				$title = $snippet['title'];
 				$desc = $snippet['desc'];
-				if (!empty($title) && !empty($desc)) {
+				if ( ! empty($title) && ! empty($desc)) {
 					$snip_count++;
 				}
 			}
 			$snip_fraction = $this->count - $snip_count;
 
-			$output = ($snip_fraction == 0) ? __( 'All snippets are ok', 'cgss' ) : sprintf(_n('%d page', '%d pages', $snip_fraction, 'cgss'), $snip_fraction) . ' ' . __( 'have incomplete snippets', 'cgss' );
+			$output = ($snip_fraction == 0) ? __('All snippets are ok', 'cgss') : sprintf(_n('%d page', '%d pages', $snip_fraction, 'cgss'), $snip_fraction) . ' ' . __('have incomplete snippets', 'cgss');
 			return $output;
 		}
 
@@ -90,7 +90,7 @@ 
 			$count = round(array_sum(array_column($text, 'count')) / $this->count, 0);
 			$ratio = round(array_sum(array_column($text, 'ratio')) / $this->count, 2);
 
-			$output = sprintf(__('Avarage', 'cgss') . ' ' . _n('%d word is','%d words are',$count,'cgss') . ' ' . __( 'found per page and avarage text to HTML ratio is %d', 'cgss'),$count,$ratio) . '%';
+			$output = sprintf(__('Avarage', 'cgss') . ' ' . _n('%d word is', '%d words are', $count, 'cgss') . ' ' . __('found per page and avarage text to HTML ratio is %d', 'cgss'), $count, $ratio) . '%';
 			return $output;
 		}
 
@@ -103,10 +103,10 @@ 
 			$count = round(array_sum(array_column($links, 'count')) / $this->count, 0);
 			$external = round(array_sum(array_column($links, 'external')) / $this->count, 0);
 			$nofollow = round(array_sum(array_column($links, 'external')) / $this->count, 0);
-			$external_percentage = ($count == 0 ? 0 : round(($external/$count)*100, 0));
-			$nofollow_percentage = ($count == 0 ? 0 : round(($nofollow/$count)*100, 0));
+			$external_percentage = ($count == 0 ? 0 : round(($external / $count) * 100, 0));
+			$nofollow_percentage = ($count == 0 ? 0 : round(($nofollow / $count) * 100, 0));
 
-			$output = sprintf(__('Avarage', 'cgss') . ' '.  _n( '%d link is','%d links are', $count, 'cgss' ) . ' ' . __('found per page. %d&#37; are external and %d&#37; are nofollow among them.', 'cgss'),$count,$external_percentage,$nofollow_percentage);
+			$output = sprintf(__('Avarage', 'cgss') . ' ' . _n('%d link is', '%d links are', $count, 'cgss') . ' ' . __('found per page. %d&#37; are external and %d&#37; are nofollow among them.', 'cgss'), $count, $external_percentage, $nofollow_percentage);
 			return $output;
 		}
 
@@ -120,7 +120,7 @@ 
 			$key_collect = array();
 			$percent_collect = array();
 			foreach ($keywords as $keyword) {
-				$keys = (!empty($keyword) ? array_keys($keyword) : []);
+				$keys = ( ! empty($keyword) ? array_keys($keyword) : []);
 				$top_key = $keys[0];
 				$key_collect[] = count(explode(' ', $top_key));
 				$percent_collect[] = $keyword[$top_key];
@@ -129,7 +129,7 @@ 
 			$key_count = round(array_sum($key_collect) / $this->count, 1);
 			$percent = round(array_sum($percent_collect) / $this->count, 1);
 
-			$output = sprintf(__('Avarage foucs keyword is', 'cgss') . ' ' . _n( '%d word','%d words',$key_count, 'cgss' ) . ' ' . __('long. Keyword frequency of %d&#37;','cgss'),$key_count,$percent);
+			$output = sprintf(__('Avarage foucs keyword is', 'cgss') . ' ' . _n('%d word', '%d words', $key_count, 'cgss') . ' ' . __('long. Keyword frequency of %d&#37;', 'cgss'), $key_count, $percent);
 
 			return $output;
 		}
@@ -144,14 +144,14 @@ 
 			$image_count = array_sum(array_column($images, 'count'));
 			$no_alt_image = array_sum(array_column($images, 'no_alt_count'));
 
-			$avg_image = round(($image_count/$this->count), 0);
+			$avg_image = round(($image_count / $this->count), 0);
 
-			$output = sprintf(__('Avarage', 'cgss') . ' ' . _n( '%d image is', '%d images are', $avg_image, 'cgss' ) . ' ' . __( 'found per page.', 'cgss'),$avg_image) . ' ';
+			$output = sprintf(__('Avarage', 'cgss') . ' ' . _n('%d image is', '%d images are', $avg_image, 'cgss') . ' ' . __('found per page.', 'cgss'), $avg_image) . ' ';
 			if ($no_alt_image == 0) {
 				$output .= ' ' . __('All of them are optimized', 'cgss');
 			} else {
-				$no_alt_percent = round(($no_alt_image/$image_count)*100, 0);
-				$output .= ' ' . sprintf(__('%d&#37; among them doesn\'t have alt tag', 'cgss'),$no_alt_percent);
+				$no_alt_percent = round(($no_alt_image / $image_count) * 100, 0);
+				$output .= ' ' . sprintf(__('%d&#37; among them doesn\'t have alt tag', 'cgss'), $no_alt_percent);
 			}
 			return $output;
 
@@ -167,8 +167,8 @@ 
 			if ($vport == $this->count) {
 				$output = __('All pages are mobile optimized', 'cgss');
 			} else {
-				$no_mobile_percent = round(($vport/$this->count)*100, 0);
-				$output = sprintf(__('%d&#37; pages aren\'t mobile optimized', 'cgss'),$no_mobile_percent);
+				$no_mobile_percent = round(($vport / $this->count) * 100, 0);
+				$output = sprintf(__('%d&#37; pages aren\'t mobile optimized', 'cgss'), $no_mobile_percent);
 			}
 
 			return $output;
@@ -183,10 +183,10 @@ 
 			$res_time = array_sum(array_column($speed, 'res_time'));
 			$down_time = array_sum(array_column($speed, 'down_time'));
 
-			$avg_res_time = round(($res_time/$this->count) * 1000, 0);
-			$avg_down_time = round(($down_time/$this->count) * 1000, 0);
+			$avg_res_time = round(($res_time / $this->count) * 1000, 0);
+			$avg_down_time = round(($down_time / $this->count) * 1000, 0);
 
-			$output = sprintf( __('Average response time is %d miliseconds and average download time is %d miliseconds', 'cgss'), $avg_res_time, $avg_down_time);
+			$output = sprintf(__('Average response time is %d miliseconds and average download time is %d miliseconds', 'cgss'), $avg_res_time, $avg_down_time);
 
 			return $output;
 		}
@@ -200,7 +200,7 @@ 
 			$fb_share = array_sum(array_column($social, 'fb_share'));
 			$fb_like = array_sum(array_column($social, 'fb_like'));
 
-			$output = sprintf( __('Total', 'cgss') . ': ' . _n( '%d share', '%d shares', $fb_share, 'cgss' ) . ' ' . __( 'in Facebook', 'cgss' ). ' | ' . _n( '%d Facebook like', '%d Facebook likes', $fb_like, 'cgss' ), $fb_share, $fb_like);
+			$output = sprintf(__('Total', 'cgss') . ': ' . _n('%d share', '%d shares', $fb_share, 'cgss') . ' ' . __('in Facebook', 'cgss') . ' | ' . _n('%d Facebook like', '%d Facebook likes', $fb_like, 'cgss'), $fb_share, $fb_like);
 
 			return $output;
 		}
@@ -211,12 +211,12 @@ 
 
 			global $wpdb;
 			$sql = "SELECT ID FROM {$wpdb->prefix}posts WHERE post_type!='attachment'";
-			$ids = $wpdb->get_results( $sql, 'ARRAY_A' );;
+			$ids = $wpdb->get_results($sql, 'ARRAY_A'); ;
 
 			$data_piece = array();
 			foreach ($ids as $id) {
 
-				$meta = get_post_meta( $id['ID'], 'cgss_scan_result', true );
+				$meta = get_post_meta($id['ID'], 'cgss_scan_result', true);
 				if ($meta) {
 					$data_piece[] = $meta;
 				}

Plugin/Lib/Action/scan.php

Spacing +48 added lines, -48 removed lines

@@ -2,7 +2,7 @@ 
 
 namespace NirjharLo\Cgss\Lib\Action;
 
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined('ABSPATH')) exit;
 
 use \NirjharLo\Cgss\Lib\Analysis\Crawl;
 
@@ -15,14 +15,14 @@ 
 		public function __construct() {
 
 			$post_id = intval($_GET['scan']);
-			$url = get_permalink( $post_id );
+			$url = get_permalink($post_id);
 
 			$crawl = new Crawl();
 			$crawl->url = esc_url_raw($url);
 			$crawl->execute();
 			$this->result = $crawl->result();
 
-			update_post_meta( $post_id, 'cgss_scan_result', $this->result );
+			update_post_meta($post_id, 'cgss_scan_result', $this->result);
 
 			$this->render();
 		}
@@ -37,7 +37,7 @@ 
 			$fb_like = $social['fb_like'];
 
 			$score_html =
-			'<span>'.sprintf(__('Overall SEO score %d out of 10. Facebook shares: %d and likes: %d', 'cgss' ), $score, $fb_share, $fb_like ).'</span>';
+			'<span>' . sprintf(__('Overall SEO score %d out of 10. Facebook shares: %d and likes: %d', 'cgss'), $score, $fb_share, $fb_like) . '</span>';
 
 			return $score_html;
 		}
@@ -58,17 +58,17 @@ 
 			$not_found = __('NOT_FOUND', 'cgss');
 			$snippet_html =
 			'<ul>
-				<li><strong>'.__('Search snippet', 'cgss').':</strong>
+				<li><strong>'.__('Search snippet', 'cgss') . ':</strong>
 					<ul>
-						<li>'.sprintf(__('Title: %s', 'cgss'),($title ? $title : $not_found)).'</li>
-						<li>'.sprintf(__('Description: %s', 'cgss'),($desc ? $desc : $not_found)).'</li>
+						<li>'.sprintf(__('Title: %s', 'cgss'), ($title ? $title : $not_found)) . '</li>
+						<li>'.sprintf(__('Description: %s', 'cgss'), ($desc ? $desc : $not_found)) . '</li>
 					</ul>
 				</li>
-				<li><strong>'.__('Social snippet(OGP)', 'cgss').':</strong>
+				<li><strong>'.__('Social snippet(OGP)', 'cgss') . ':</strong>
 					<ul>
-						<li>'.sprintf(__('Title: %s', 'cgss'),($social_title ? $social_title : $not_found)).'</li>
-						<li>'.sprintf(__('Description: %s', 'cgss'),( $social_desc ? $social_desc : $not_found)).'</li>
-						<li>'.sprintf(__('Image: %s', 'cgss'),( $social_image ? $social_image : $not_found)).'</li>
+						<li>'.sprintf(__('Title: %s', 'cgss'), ($social_title ? $social_title : $not_found)) . '</li>
+						<li>'.sprintf(__('Description: %s', 'cgss'), ($social_desc ? $social_desc : $not_found)) . '</li>
+						<li>'.sprintf(__('Image: %s', 'cgss'), ($social_image ? $social_image : $not_found)) . '</li>
 					</ul></li>
 			</ul>';
 
@@ -80,12 +80,12 @@ 
 		public function text_display() {
 
 			$text = $this->result['text'];
-			$keys = (!empty($text['keys']) ? implode(', ', array_keys($text['keys'])) : '');
+			$keys = ( ! empty($text['keys']) ? implode(', ', array_keys($text['keys'])) : '');
 			$count = $text['count'];
 			$ratio = $text['ratio'];
 
 			$htags = $text['htags'];
-			$headings = (!empty($htags['names']) ? implode( ', ', $htags['names'] ) : '');
+			$headings = ( ! empty($htags['names']) ? implode(', ', $htags['names']) : '');
 
 			$links = $text['links'];
 			$link_count = $links['count'];
@@ -94,11 +94,11 @@ 
 
 			$text_html =
 			'<ul>
-				<li>'.sprintf(__('Keywords: %s','cgss'), $keys).'</li>
-				<li>'.sprintf(__('Number of words: %s','cgss'), $count).'</li>
-				<li>'.sprintf(__('Text to html ratio: %s percent','cgss'), $ratio).'</li>
-				<li>'.sprintf(__('Heading tags in text hierarchy: %s','cgss'), $headings).'</li>
-				<li>'.sprintf(__('Total Links: %d, Nofollow Links %d, External Links: %d','cgss'), $link_count, $link_nofollow, $link_external).'</li>
+				<li>'.sprintf(__('Keywords: %s', 'cgss'), $keys) . '</li>
+				<li>'.sprintf(__('Number of words: %s', 'cgss'), $count) . '</li>
+				<li>'.sprintf(__('Text to html ratio: %s percent', 'cgss'), $ratio) . '</li>
+				<li>'.sprintf(__('Heading tags in text hierarchy: %s', 'cgss'), $headings) . '</li>
+				<li>'.sprintf(__('Total Links: %d, Nofollow Links %d, External Links: %d', 'cgss'), $link_count, $link_nofollow, $link_external) . '</li>
 			</ul>';
 
 			return $text_html;
@@ -115,7 +115,7 @@ 
 
 			$tag_style = $design['tag_style'];
 			$tag_style_count = $tag_style['count'];
-			$tag_style_attributes = implode( ', ', $tag_style['tags'] );
+			$tag_style_attributes = implode(', ', $tag_style['tags']);
 
 			$nested_table = $design['nested_table'];
 			$vport = $design['vport'];
@@ -126,11 +126,11 @@ 
 
 			$design_html =
 			'<ul>
-				<li>'.sprintf(__('Total images: %d and images without alt tags: %d','cgss'), $image_count, $image_no_alt_count).'</li>
-				<li>'.sprintf(__('iframe: %d','cgss'), $iframe).'</li>
-				<li>'.sprintf(__('Nested Tables: %d','cgss'), $nested_table).'</li>
-				<li>'.sprintf(__('Tags with style attribute: %s(%s)','cgss'), $tag_style_count, $tag_style_attributes).'</li>
-				<li>'.sprintf(__('Mobile optimization: Viewport Tag: %d, @media Queries: %d','cgss'), $vport, $media).'</li>
+				<li>'.sprintf(__('Total images: %d and images without alt tags: %d', 'cgss'), $image_count, $image_no_alt_count) . '</li>
+				<li>'.sprintf(__('iframe: %d', 'cgss'), $iframe) . '</li>
+				<li>'.sprintf(__('Nested Tables: %d', 'cgss'), $nested_table) . '</li>
+				<li>'.sprintf(__('Tags with style attribute: %s(%s)', 'cgss'), $tag_style_count, $tag_style_attributes) . '</li>
+				<li>'.sprintf(__('Mobile optimization: Viewport Tag: %d, @media Queries: %d', 'cgss'), $vport, $media) . '</li>
 			</ul>';
 
 			return $design_html;
@@ -152,17 +152,17 @@ 
 			$alive = $crawl['alive'];
 			$robot = $crawl['meta_robot'];
 
-			$on = __( 'on', 'cgss' );
-			$off = __( 'off', 'cgss' );
+			$on = __('on', 'cgss');
+			$off = __('off', 'cgss');
 
 			$crawl_html =
 			'<ul>
-				<li>'.sprintf(__('Your URL parameters: SSL security: %s, Static url: %s, underscores in Url: %s','cgss'), (!$ssl ? $off : $on ), (!$dynamic ? $off : $on ), (!$underscore ? $off : $on )).'</li>
-				<li>'.sprintf(__('If modified since header: %s','cgss'), ($if_mod == 0 ? $off : $on )).'</li>
-				<li>'.sprintf(__('Canonical Url: %s','cgss'), ($cano == 0 ? $off : $on )).'</li>
-				<li>'.sprintf(__('Meta Robot: %s','cgss'), (!$robot['ok'] ? $off : $on )).'</li>
-				<li>'.sprintf(__('IP Forward: %s','cgss'), ($ip ? $ip : $off )).'</li>
-				<li>'.sprintf(__('Keep alive connection: %s','cgss'), ($alive == 0 ? $off : $on )).'</li>
+				<li>'.sprintf(__('Your URL parameters: SSL security: %s, Static url: %s, underscores in Url: %s', 'cgss'), ( ! $ssl ? $off : $on), ( ! $dynamic ? $off : $on), ( ! $underscore ? $off : $on)) . '</li>
+				<li>'.sprintf(__('If modified since header: %s', 'cgss'), ($if_mod == 0 ? $off : $on)) . '</li>
+				<li>'.sprintf(__('Canonical Url: %s', 'cgss'), ($cano == 0 ? $off : $on)) . '</li>
+				<li>'.sprintf(__('Meta Robot: %s', 'cgss'), ( ! $robot['ok'] ? $off : $on)) . '</li>
+				<li>'.sprintf(__('IP Forward: %s', 'cgss'), ($ip ? $ip : $off)) . '</li>
+				<li>'.sprintf(__('Keep alive connection: %s', 'cgss'), ($alive == 0 ? $off : $on)) . '</li>
 			</ul>';
 
 			return $crawl_html;
@@ -181,16 +181,16 @@ 
 			$css = $speed['css'];
 			$js = $speed['js'];
 
-			$on = __( 'on', 'cgss' );
-			$off = __( 'off', 'cgss' );
+			$on = __('on', 'cgss');
+			$off = __('off', 'cgss');
 
 			$speed_html =
 			'<ul>
-				<li>'. sprintf( __( 'Header response time %s s and page downloading time %s s', 'cgss' ), $res_time, $down_time ). '</li>
-				<li>'.sprintf(__('gZip compression: %s','cgss'), ($gzip ? $off : $on )).'</li>
-				<li>'.sprintf(__('Browser caching: %s','cgss'), ($cache ? $off : $on )).'</li>
-				<li>'.sprintf(__('Requests made by CSS is %s and by JS is %s','cgss'), $css['count'], $js['count']).'</li>
-				<li>'.sprintf(__('Resources can be compressed by %s kb for CSS and %s kb for JS','cgss'), $css['compress_num'], $js['compress_num']).'</li>
+				<li>'. sprintf(__('Header response time %s s and page downloading time %s s', 'cgss'), $res_time, $down_time) . '</li>
+				<li>'.sprintf(__('gZip compression: %s', 'cgss'), ($gzip ? $off : $on)) . '</li>
+				<li>'.sprintf(__('Browser caching: %s', 'cgss'), ($cache ? $off : $on)) . '</li>
+				<li>'.sprintf(__('Requests made by CSS is %s and by JS is %s', 'cgss'), $css['count'], $js['count']) . '</li>
+				<li>'.sprintf(__('Resources can be compressed by %s kb for CSS and %s kb for JS', 'cgss'), $css['compress_num'], $js['compress_num']) . '</li>
 			</ul>';
 
 			return $speed_html;
@@ -200,19 +200,19 @@ 
 		// Render the HTML
 		public function render() {
 
-			$this->score_html	= $this->score_html();
-			$this->snippets_html	= $this->snippet_display();
+			$this->score_html = $this->score_html();
+			$this->snippets_html = $this->snippet_display();
 			$this->text_html = $this->text_display();
 			$this->design_html = $this->design_display();
 			$this->crawl_html = $this->crawl_display();
 			$this->speed_html = $this->speed_display();
 
-			$this->box( null, null, $this->score_html );
-			$this->box( __( 'Snippets', 'cgss' ), $this->dashicon('align-none'), $this->snippets_html );
-			$this->box( __( 'Text & Links', 'cgss' ), $this->dashicon('text'), $this->text_html );
-			$this->box( __( 'Design', 'cgss' ), $this->dashicon('smartphone'), $this->design_html );
-			$this->box( __( 'Crawl', 'cgss' ), $this->dashicon('randomize'), $this->crawl_html );
-			$this->box( __( 'Speed', 'cgss' ), $this->dashicon('clock'), $this->speed_html );
+			$this->box(null, null, $this->score_html);
+			$this->box(__('Snippets', 'cgss'), $this->dashicon('align-none'), $this->snippets_html);
+			$this->box(__('Text & Links', 'cgss'), $this->dashicon('text'), $this->text_html);
+			$this->box(__('Design', 'cgss'), $this->dashicon('smartphone'), $this->design_html);
+			$this->box(__('Crawl', 'cgss'), $this->dashicon('randomize'), $this->crawl_html);
+			$this->box(__('Speed', 'cgss'), $this->dashicon('clock'), $this->speed_html);
 		}
 
 
@@ -232,6 +232,6 @@ 
 
 		public function dashicon($icon) {
 
-			return '<span class="dashicons dashicons-'.$icon.'"></span>';
+			return '<span class="dashicons dashicons-' . $icon . '"></span>';
 		}
 	} ?>