Updated Coding Standards.

Author: varunsridharan

includes/class-framework-modules.php

@@ -2,6 +2,7 @@
 
 namespace VSP;
 
+use Varunsridharan\PHP\Autoloader;
 use Varunsridharan\WordPress\Localizer;
 
 if ( ! defined( 'ABSPATH' ) ) {
@@ -106,7 +107,7 @@ protected function _autoloader_init() {
 					'base_path' => $this->plugin_path(),
 					'options'   => array(),
 				) );
-				$this->autoloader = new \Varunsridharan\PHP\Autoloader( $args['namespace'], $args['base_path'], $args['options'] );
+				$this->autoloader = new Autoloader( $args['namespace'], $args['base_path'], $args['options'] );
 			}
 		}
 

includes/class-setup.php

@@ -58,7 +58,8 @@ public static function create_folders( $dir ) {
 
 		foreach ( $files as $file ) {
 			if ( wp_mkdir_p( $file['base'] ) && ! file_exists( vsp_slashit( $file['base'] ) . $file['file'] ) ) {
-				if ( $file_handle = @fopen( vsp_slashit( $file['base'] ) . $file['file'], 'w' ) ) {
+				$file_handle = @fopen( vsp_slashit( $file['base'] ) . $file['file'], 'w' );
+				if ( $file_handle ) {
 					fwrite( $file_handle, $file['content'] );
 					fclose( $file_handle );
 				}

includes/core/class-instance-handler.php

@@ -93,7 +93,7 @@ public function _instance( $class, ...$arguments ) {
 			if ( $this->get_instance( $class ) === false ) {
 				try {
 					$framework_key = ( $this instanceof Framework ) ? static::class : false;
-					$refl          = new \ReflectionClass( $class );
+					$refl          = new ReflectionClass( $class );
 
 					if ( false === $framework_key ) {
 						$framework_key = ( isset( $this->framework_instance ) && ! empty( $this->framework_instance ) ) ? $this->framework_instance : false;

includes/functions/general-functions.php

@@ -1,5 +1,8 @@
 <?php
 
+use VSP\Modules\Logger;
+use VSP\Modules\Logger\File_Handler;
+
 if ( ! defined( 'ABSPATH' ) ) {
 	exit;
 }
@@ -171,12 +174,12 @@ function vsp_get_logger( $subpath = false, $file_name = null, $filesize = false
 		$class      = apply_filters( 'vsp_logging_class', '\VSP\Modules\Logger' );
 		$implements = class_implements( $class );
 		if ( is_array( $implements ) && in_array( 'VSP\Core\Interfaces\Logger', $implements, true ) ) {
-			$logger = ( is_object( $class ) ) ? $class : new $class( array( new \VSP\Modules\Logger\File_Handler( $subpath, $file_name, $filesize ) ) );
+			$logger = ( is_object( $class ) ) ? $class : new $class( array( new File_Handler( $subpath, $file_name, $filesize ) ) );
 		} else {
 			/* translators: 1: class name 2: woocommerce_logging_class 3: WC_Logger_Interface */
 			$smgs = sprintf( __( 'The class %1$s provided by %2$s filter must implement %3$s.', 'vsp-framework' ), '<code>' . esc_html( is_object( $class ) ? get_class( $class ) : $class ) . '</code>', '<code>vsp_logging_class</code>', '<code>\VSP\Core\Interfaces\Logger</code>' );
 			vsp_doing_it_wrong( __FUNCTION__, $smgs, '3.0' );
-			$logger = new \VSP\Modules\Logger( array( new  \VSP\Modules\Logger\File_Handler( $subpath, $file_name, $filesize ) ) );
+			$logger = new Logger( array( new File_Handler( $subpath, $file_name, $filesize ) ) );
 		}
 		return $logger;
 	}
@@ -225,10 +228,10 @@ function vsp_log_msg( $messages = '', $type = 'critical', $handler = false, $con
 			$handler = vsp_logger();
 		}
 
-		if ( $handler instanceof \VSP\Modules\Logger && method_exists( $handler, $type ) ) {
+		if ( $handler instanceof Logger && method_exists( $handler, $type ) ) {
 			$handler->$type( $messages, $context );
 			return true;
-		} elseif ( vsp_logger() instanceof \VSP\Modules\Logger && method_exists( vsp_logger(), $type ) ) {
+		} elseif ( vsp_logger() instanceof Logger && method_exists( vsp_logger(), $type ) ) {
 			$msg = array_merge( array( __( 'Tried To Log A Message But Failed Got unknown Handler', 'vsp-framework' ) ), wp_debug_backtrace_summary( null, 0, false ) );
 			vsp_log_msg( '----------------------------------------------------------------', 'notice', vsp_logger() );
 			vsp_log_msg( $msg, 'critical', vsp_logger() );

includes/modules/class-wponion.php

@@ -3,6 +3,7 @@
 namespace VSP\Modules;
 
 use VSP\Base;
+use function wponion_builder;
 
 if ( ! defined( 'ABSPATH' ) ) {
 	die;
@@ -43,7 +44,7 @@ public function wpo_load() {
 			$this->options['extra_js']   = ( isset( $this->options['extra_js'] ) ) ? $this->options['extra_js'] : array();
 			$this->options['extra_js']   = ( ! is_array( $this->options['extra_js'] ) ) ? array( $this->options['extra_js'] ) : $this->options['extra_js'];
 			$this->options['extra_js'][] = 'vsp_load_core_assets';
-			$options                     = \wponion_builder();
+			$options                     = wponion_builder();
 			$this->plugin()
 				->action( 'settings_options', $options );
 			if ( wpo_is( $options ) ) {

includes/modules/logger/class-file-handler.php

@@ -317,7 +317,7 @@ protected function open( $handle, $mode = 'a' ) {
 				$temphandle = @fopen( $file, 'w+' );
 				@fclose( $temphandle );
 				if ( defined( 'FS_CHMOD_FILE' ) ) {
-					@chmod( $file, FS_CHMOD_FILE ); // phpcs:ignore WordPress.VIP.FileSystemWritesDisallow.chmod_chmod
+					@chmod( $file, FS_CHMOD_FILE );
 				}
 			}
 			$resource = @fopen( $file, $mode );

vsp-bootstrap.php

@@ -34,6 +34,8 @@
  * Domain Path: languages/
  */
 
+use Varunsridharan\PHP\Autoloader;
+
 if ( ! defined( 'ABSPATH' ) ) {
 	exit;
 }
@@ -55,9 +57,7 @@
 			throw new ErrorException( __( 'Framework Autoloader Not Found' ) );
 		}
 
-		new \Varunsridharan\PHP\Autoloader( 'VSP\\', VSP_PATH . 'includes/', array(
-			'prepend' => true,
-		) );
+		new Autoloader( 'VSP\\', VSP_PATH . 'includes/', array( 'prepend' => true ) );
 
 		require_once __DIR__ . '/vsp-functions.php';
 		require_once __DIR__ . '/vsp-hooks.php';

vsp-hooks.php

@@ -1,4 +1,7 @@
 <?php
+
+use VSP\Modules\WordPress\Importers;
+
 if ( ! defined( 'ABSPATH' ) ) {
 	exit;
 }
@@ -11,7 +14,7 @@
 	 * Creates A Instances For WP Importers
 	 */
 	function vsp_on_load() {
-		\VSP\Modules\WordPress\Importers::instance();
+		Importers::instance();
 	}
 }