LSX_TO_Vehicles

Complexity

Total Complexity

38

Size/Duplication

Total Lines	217
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	plugin_path()	0	5	3
A	activated_plugin()	0	10	5
A	set_vars()	0	8	1
A	__construct()	0	28	3
A	taxonomies_plural_filter()	0	7	4
A	post_types_filter()	0	7	4
A	register_activation_hook()	0	3	3
A	taxonomies_filter()	0	7	4
A	post_types_slugs_filter()	0	7	2
A	post_types_singular_filter()	0	7	4
A	load_plugin_textdomain()	0	2	1
A	register_activation_hook_check()	0	7	2
A	lsx_to_search_integration()	0	3	1
A	theme_allowed_post_type_banners()	0	5	1

<?php
/**
 * LSX_TO_Vehicles
 *
 * @package   LSX_TO_Vehicles
 * @author    LightSpeed
 * @license   GPL-3.0+
 * @link
 * @copyright 2016 LightSpeedDevelopment
 */

There must be exactly one blank line after the file comment

if ( ! class_exists( 'LSX_TO_Vehicles' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	/**
	 * Main plugin class.
	 *
	 * @package LSX_TO_Vehicles
	 * @author  LightSpeed
	 */
	class LSX_TO_Vehicles {

		/**
		 * The plugins id
		 */

Missing @var tag in member variable comment

		public $plugin_slug = 'to-vehicles';

		/**
		 * The post types the plugin registers
		 */

Missing @var tag in member variable comment

		public $post_types = false;

		/**
		 * The singular post types the plugin registers
		 */

Missing @var tag in member variable comment

		public $post_types_singular = false;

		/**
		 * An array of the post types slugs plugin registers
		 */

Missing @var tag in member variable comment

		public $post_type_slugs = false;

		/**
		 * The taxonomies the plugin registers
		 */

Missing @var tag in member variable comment

		public $taxonomies = false;

		/**
		 * The taxonomies the plugin registers (plural)
		 */

Missing @var tag in member variable comment

		public $taxonomies_plural = false;

		/**
		 * Constructor
		 */
		public function __construct() {

Expected 2 blank lines before function; 1 found

			//Set the variables

No space found before comment text; expected "// Set the variables" but found "//Set the variables"

Inline comments must end in full-stops, exclamation marks, or question marks

			$this->set_vars();
			$this->lsx_to_search_integration();

			// Make TO last plugin to load

Inline comments must end in full-stops, exclamation marks, or question marks

			add_action( 'activated_plugin', array( $this, 'activated_plugin' ) );

			add_action( 'init', array( $this, 'load_plugin_textdomain' ) );

			if ( false !== $this->post_types ) {

The condition false !== $this->post_types is always true.

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				add_filter( 'lsx_to_framework_post_types', array( $this, 'post_types_filter' ) );
				add_filter( 'lsx_to_post_types', array( $this, 'post_types_filter' ) );
				add_filter( 'lsx_to_post_types_singular', array( $this, 'post_types_singular_filter' ) );
				add_filter( 'lsx_to_settings_path', array( $this, 'plugin_path' ), 10, 2 );
			}

No blank line found after control structure

			if ( false !== $this->taxonomies ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				add_filter( 'lsx_to_framework_taxonomies', array( $this, 'taxonomies_filter' ) );
				add_filter( 'lsx_to_framework_taxonomies_plural', array( $this, 'taxonomies_plural_filter' ) );
			}

			require_once( LSX_TO_VEHICLES_PATH . '/classes/class-lsx-to-vehicles-admin.php' );

"require_once" is a statement not a function; no parentheses are required

			require_once( LSX_TO_VEHICLES_PATH . '/classes/class-lsx-to-vehicles-frontend.php' );

"require_once" is a statement not a function; no parentheses are required

			require_once( LSX_TO_VEHICLES_PATH . '/includes/template-tags.php' );

"require_once" is a statement not a function; no parentheses are required

			// flush_rewrite_rules()

67% of this comment could be valid code. Did you maybe forget this after debugging?

Inline comments must end in full-stops, exclamation marks, or question marks

			register_activation_hook( LSX_TO_VEHICLES_CORE, array( $this, 'register_activation_hook' ) );
			add_action( 'admin_init', array( $this, 'register_activation_hook_check' ) );
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**
		 * Include the post type for the search integration
		 */
		public function lsx_to_search_integration() {
			add_filter( 'lsx_to_search_post_types', array( $this, 'post_types_filter' ) );
			add_filter( 'lsx_to_search_taxonomies', array( $this, 'taxonomies_filter' ) );
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**
		 * Load the plugin text domain for translation.
		 */
		public function load_plugin_textdomain() {
			load_plugin_textdomain( 'to-vehicles', false, basename( LSX_TO_VEHICLES_PATH ) . '/languages' );
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**
		 * Sets the plugins variables
		 */
		public function set_vars() {
			$this->post_types = array(

Equals sign not aligned with surrounding assignments; expected 10 spaces but found 1 space

				'vehicle' => __( 'Vehicles', 'to-vehicles' ),
			);
			$this->post_types_singular = array(
				'vehicle' => __( 'Vehicle', 'to-vehicles' ),
			);
			$this->post_type_slugs = array_keys( $this->post_types );

Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space

		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**

Doc comment for parameter "$path" missing

Doc comment for parameter "$post_type" missing

		 * Adds our post types to an array via a filter
		 */
		public function plugin_path( $path, $post_type ) {
			if ( false !== $this->post_types && array_key_exists( $post_type, $this->post_types ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

$this->post_types of type true is incompatible with the type ArrayObject|array expected by parameter $array of array_key_exists().

				$path = LSX_TO_VEHICLES_PATH;
			}

No blank line found after control structure

			return $path;
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**

Doc comment for parameter "$post_types" missing

		 * Adds our post types to an array via a filter
		 */
		public function post_types_slugs_filter( $post_types ) {
			if ( is_array( $post_types ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$post_types = array_merge( $post_types, $this->post_type_slugs );

$this->post_type_slugs of type boolean is incompatible with the type array expected by parameter $arrays of array_merge().

			} else {
				$post_types = $this->post_type_slugs;
			}

No blank line found after control structure

			return $post_types;
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**

Doc comment for parameter "$post_types" missing

		 * Adds our post types to an array via a filter
		 */
		public function post_types_filter( $post_types ) {
			if ( is_array( $post_types ) && is_array( $this->post_types ) ) {

The condition is_array($this->post_types) is always false.

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$post_types = array_merge( $post_types, $this->post_types );
			} elseif ( is_array( $this->post_types ) ) {

The condition is_array($this->post_types) is always false.

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$post_types = $this->post_types;
			}

No blank line found after control structure

			return $post_types;
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**

Doc comment for parameter "$post_types_singular" missing

		 * Adds our post types to an array via a filter
		 */
		public function post_types_singular_filter( $post_types_singular ) {
			if ( is_array( $post_types_singular ) && is_array( $this->post_types_singular ) ) {

The condition is_array($this->post_types_singular) is always false.

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$post_types_singular = array_merge( $post_types_singular, $this->post_types_singular );
			} elseif ( is_array( $this->post_types_singular ) ) {

The condition is_array($this->post_types_singular) is always false.

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$post_types_singular = $this->post_types_singular;
			}

No blank line found after control structure

			return $post_types_singular;
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**

Doc comment for parameter "$taxonomies" missing

		 * Adds our taxonomies to an array via a filter
		 */
		public function taxonomies_filter( $taxonomies ) {
			if ( is_array( $taxonomies ) && is_array( $this->taxonomies ) ) {

The condition is_array($this->taxonomies) is always false.

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$taxonomies = array_merge( $taxonomies, $this->taxonomies );
			} elseif ( is_array( $this->taxonomies ) ) {

The condition is_array($this->taxonomies) is always false.

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$taxonomies = $this->taxonomies;
			}

No blank line found after control structure

			return $taxonomies;
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**

Doc comment for parameter "$taxonomies_plural" missing

		 * Adds our taxonomies_plural to an array via a filter
		 */
		public function taxonomies_plural_filter( $taxonomies_plural ) {
			if ( is_array( $taxonomies_plural ) && is_array( $this->taxonomies_plural ) ) {

The condition is_array($this->taxonomies_plural) is always false.

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$taxonomies_plural = array_merge( $taxonomies_plural, $this->taxonomies_plural );
			} elseif ( is_array( $this->taxonomies_plural ) ) {

The condition is_array($this->taxonomies_plural) is always false.

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$taxonomies_plural = $this->taxonomies_plural;
			}

No blank line found after control structure

			return $taxonomies_plural;
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**
		 * Make TO last plugin to load.
		 */
		public function activated_plugin() {
			if ( get_option( 'active_plugins' ) === $plugins ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$search = preg_grep( '/.*\/tour-operator\.php/', $plugins );
				$key = array_search( $search, $plugins );

Equals sign not aligned with surrounding assignments; expected 4 spaces but found 1 space

Not using strict comparison for array_search; supply true for third argument.

				if ( is_array( $search ) && count( $search ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					foreach ( $search as $key => $path ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

						array_splice( $plugins, $key, 1 );
						array_push( $plugins, $path );
						update_option( 'active_plugins', $plugins );
					}
				}
			}
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**
		 * On plugin activation
		 */
		public function register_activation_hook() {
			if ( ! is_network_admin() && ! isset( $_GET['activate-multi'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Processing form data without nonce verification.

Expected 0 spaces before closing bracket; 1 found

				set_transient( '_tour_operators_vehicles_flush_rewrite_rules', 1, 30 );
			}
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**
		 * On plugin activation (check)
		 */
		public function register_activation_hook_check() {
			if ( ! get_transient( '_tour_operators_vehicles_flush_rewrite_rules' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				return;
			}

			delete_transient( '_tour_operators_vehicles_flush_rewrite_rules' );
			flush_rewrite_rules();
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

		/**

Doc comment for parameter "$post_types" missing

		 * Enabled banners for the additional post types
		 *
		 * @package    theme
		 * @subpackage setup
		 * @category   banners
		 *
		 * @param   $post_types array()

Missing parameter name

		 * @return  $post_types array()

The doc comment $post_types at position 0 could not be parsed: Unknown type name '$post_types' at position 0 in $post_types.

		 */
		function theme_allowed_post_type_banners( $post_types ) {

It is generally recommended to explicitly declare the visibility for methods.

It is recommend to declare an explicit visibility for theme_allowed_post_type_banners.

			$post_types[] = 'summary';
			$post_types[] = 'gallery';
			$post_types[] = 'video';
			return $post_types;
		}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	}
	new LSX_TO_Vehicles();
}