lightspeeddevelopment / to-vehicles

classes/class-lsx-to-vehicles-admin.php

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

/**
 * Main plugin class.
 *
 * @package LSX_TO_Vehicles_Admin
 * @author  LightSpeed
 */

There must be no blank lines after the class comment

class LSX_TO_Vehicles_Admin extends LSX_TO_Vehicles {

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

Expected 2 blank lines before function; 1 found

		add_action( 'init', array( $this, 'register_post_types' ) );
		add_filter( 'cmb_meta_boxes', array( $this, 'register_metaboxes' ) );

		//add_filter( 'lsx_get_post-types_configs', array( $this, 'post_type_config' ), 10, 1 );

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

No space found before comment text; expected "// add_filter( 'lsx_get_post-types_configs', array( $this, 'post_type_config' ), 10, 1 );" but found "//add_filter( 'lsx_get_post-types_configs', array( $this, 'post_type_config' ), 10, 1 );"

		//add_filter( 'lsx_get_metaboxes_configs', array( $this, 'meta_box_config' ), 10, 1 );

No space found before comment text; expected "// add_filter( 'lsx_get_metaboxes_configs', array( $this, 'meta_box_config' ), 10, 1 );" but found "//add_filter( 'lsx_get_metaboxes_configs', array( $this, 'meta_box_config' ), 10, 1 );"

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

		add_filter( 'lsx_to_destination_custom_fields', array( $this, 'custom_fields' ) );
		add_filter( 'lsx_to_tour_custom_fields', array( $this, 'custom_fields' ) );
		add_filter( 'lsx_to_accommodation_custom_fields', array( $this, 'custom_fields' ) );
		add_filter( 'lsx_to_team_custom_fields', array( $this, 'custom_fields' ) );
		add_filter( 'lsx_to_special_custom_fields', array( $this, 'custom_fields' ) );
		add_filter( 'lsx_to_activity_custom_fields', array( $this, 'custom_fields' ) );
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**

Doc comment for parameter "$objects" missing

	 * Register the activity post type config
	 *
	 * @param  $objects

Missing parameter name

	 * @return   array
	 */
	public function post_type_config( $objects ) {

Expected 0 blank lines after opening function brace; 1 found

		foreach ( $this->post_types as $key => $label ) {

The expression $this->post_types of type boolean is not traversable.

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			if ( file_exists( LSX_TO_VEHICLES_PATH . 'includes/post-types/config-' . $key . '.php' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$objects[ $key ] = include LSX_TO_VEHICLES_PATH . 'includes/post-types/config-' . $key . '.php';
			}
		}

		return 	$objects;

Language constructs must be followed by a single space; expected 1 space but found "··"

Spaces must be used for mid-line alignment; tabs are not allowed

	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**

Doc comment for parameter "$meta_boxes" missing

	 * Register the activity metabox config
	 *
	 * @param  $meta_boxes

Missing parameter name

	 * @return   array
	 */
	public function meta_box_config( $meta_boxes ) {
		foreach ( $this->post_types as $key => $label ) {

The expression $this->post_types of type boolean is not traversable.

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			if ( file_exists( LSX_TO_VEHICLES_PATH . 'includes/metaboxes/config-' . $key . '.php' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$meta_boxes[ $key ] = include LSX_TO_VEHICLES_PATH . 'includes/metaboxes/config-' . $key . '.php';
			}
		}

No blank line found after control structure

		return 	$meta_boxes;

Language constructs must be followed by a single space; expected 1 space but found "··"

Spaces must be used for mid-line alignment; tabs are not allowed

	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

	/**
	 * Register the landing pages post type.
	 */
	public function register_post_types() {

Expected 0 blank lines after opening function brace; 1 found

		$labels = array(
			'name'               => _x( 'Vehicles', 'to-vehicles' ),
			'singular_name'      => _x( 'Vehicle', 'to-vehicles' ),
			'add_new'            => _x( 'Add New', 'to-vehicles' ),
			'add_new_item'       => _x( 'Add New Vehicle', 'to-vehicles' ),
			'edit_item'          => _x( 'Edit Vehicle', 'to-vehicles' ),
			'new_item'           => _x( 'New Vehicle', 'to-vehicles' ),
			'all_items'          => _x( 'Vehicles', 'to-vehicles' ),
			'view_item'          => _x( 'View Vehicle', 'to-vehicles' ),
			'search_items'       => _x( 'Search Vehicles', 'to-vehicles' ),
			'not_found'          => _x( 'No vehicles found', 'to-vehicles' ),
			'not_found_in_trash' => _x( 'No vehicles found in Trash', 'to-vehicles' ),
			'parent_item_colon'  => '',
			'menu_name'          => _x( 'Vehicles', 'to-vehicles' ),
		);

		$args = array(
			'menu_icon'          => 'dashicons-performance',
			'labels'             => $labels,
			'public'             => true,
			'publicly_queryable' => true,
			'show_ui'            => true,
			'show_in_menu'       => 'tour-operator',
			'menu_position'      => 80,
			'query_var'          => true,
			'rewrite'            => array(
				'slug' => 'vehicle',
			),
			'capability_type'    => 'post',
			'has_archive'        => 'vehicles',
			'hierarchical'       => true,
			'show_in_rest'       => true,
			'supports'           => array( 'title', 'editor', 'thumbnail', 'excerpt', 'custom-fields' ),
		);

		register_post_type( 'vehicle', $args );

	}

Expected 2 blank lines after function; 1 found

	function register_metaboxes( array $meta_boxes ) {

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

Missing doc comment for function register_metaboxes()

It is recommend to declare an explicit visibility for register_metaboxes.

Expected 0 blank lines after opening function brace; 1 found

		$fields[] = array(
			'id'   => 'title',
			'name' => 'General',
			'type' => 'title',
		);
		$fields[] = array(
			'id'   => 'featured',
			'name' => 'Featured',
			'type' => 'checkbox',
		);
		if ( ! class_exists( 'LSX_Banners' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$fields[] = array(
				'id'   => 'tagline',
				'name' => 'Tagline',
				'type' => 'text',
			);
		}

No blank line found after control structure

		$fields[] = array(
			'id'   => 'code',
			'name' => 'Code',
			'type' => 'text',
		);
		$fields[] = array(
			'id'      => 'gearbox',
			'name'    => 'Gearbox Type',
			'type'    => 'radio',
			'options' => array(
				'Automatic' => 'Automatic',
				'Manual'    => 'Manual',
			),
		);
		$fields[] = array(
			'id'      => 'gears',
			'name'    => 'Gears',
			'type'    => 'radio',
			'options' => array(
				'4',
				'5',
				'6',
				'7',
			),
		);
		$fields[] = array(
			'id'   => 'vehicle_type',
			'name' => 'Vehicle Type',
			'type' => 'text',
		);
		$fields[] = array(
			'id'   => 'price',
			'name' => 'Price Guide',
			'type' => 'text',
		);
		$fields[] = array(
			'id'      => 'engine_type',
			'name'    => 'Engine Type',
			'type'    => 'radio',
			'options' => array(
				'Diesel' => 'Diesel',
				'Petrol' => 'Petrol',
			),
		);
		$fields[] = array(
			'id'   => 'engine_size',
			'name' => 'Engine Size',
			'type' => 'text',
		);
		$fields[] = array(
			'id'   => 'seating',
			'name' => 'Seats',
			'type' => 'text',
		);
		$fields[] = array(
			'id'      => 'features',
			'name'    => 'Features',
			'type'    => 'wysiwyg',
			'options' => array(
				'editor_height' => '100',
			),
		);
		$fields[] = array(
			'id'   => 'gallery_title',
			'name' => 'Gallery',
			'type' => 'title',
		);

		//Galleries Block

No space found before comment text; expected "// Galleries Block" but found "//Galleries Block"

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

		$fields[] = array(
			'id'   => 'gallery_title',
			'name' => esc_html__( 'Gallery', 'tour-operator' ),
			'type' => 'title',
		);
		$fields[] = array(
			'id'                  => 'gallery',
			'name'                => '',
			'type'                => 'image',
			'repeatable'          => true,
			'show_size'           => false,
			'sortable'            => true,
			'string-repeat-field' => esc_html__( 'Add new image', 'tour-operator' ),
		);
		if ( class_exists( 'Envira_Gallery' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$fields[] = array(
				'id'   => 'envira_title',
				'name' => esc_html__( 'Envira Gallery', 'tour-operator' ),
				'type' => 'title',
			);
			$fields[] = array(
				'id'         => 'envira_gallery',
				'name'       => esc_html__( 'Envira Gallery', 'to-galleries' ),
				'type'       => 'post_select',
				'use_ajax'   => false,
				'query'      => array(
					'post_type'      => 'envira',
					'nopagin'        => true,
					'posts_per_page' => '-1',
					'orderby'        => 'title',
					'order'          => 'ASC',
				),
				'allow_none' => true,
			);
			if ( class_exists( 'Envira_Videos' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$fields[] = array(
					'id'         => 'envira_video',
					'name'       => esc_html__( 'Envira Video Gallery', 'to-galleries' ),
					'type'       => 'post_select',
					'use_ajax'   => false,
					'query'      => array(
						'post_type'      => 'envira',
						'nopagin'        => true,
						'posts_per_page' => '-1',
						'orderby'        => 'title',
						'order'          => 'ASC',
					),
					'allow_none' => true,
				);
			}
		}

		if ( class_exists( 'LSX_Field_Pattern' ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$fields = array_merge( $fields, LSX_Field_Pattern::videos() );

The type LSX_Field_Pattern was not found. Maybe you did not declare it correctly or list all dependencies?

		}

		$fields[] = array(
			'id'   => 'accommodation_title',
			'name' => 'Accommodation',
			'type' => 'title',
		);
		$fields[] = array(
			'id'         => 'accommodation_to_vehicle',
			'name'       => 'Accommodations related with this vehicle',
			'type'       => 'post_select',
			'use_ajax'   => false,
			'query'      => array(
				'post_type'      => 'accommodation',
				'nopagin'        => true,
				'posts_per_page' => 1000,

Detected high pagination limit, posts_per_page is set to 1000

				'orderby'        => 'title',
				'order'          => 'ASC',
			),
			'repeatable' => true,
			'sortable'   => true,
			'allow_none' => true,
		);
		$fields[] = array(
			'id'   => 'activity_title',
			'name' => 'Activities',
			'type' => 'title',
		);
		$fields[] = array(
			'id'         => 'activity_to_vehicle',
			'name'       => 'Activities related with this vehicle',
			'type'       => 'post_select',
			'use_ajax'   => false,
			'query'      => array(
				'post_type'      => 'activity',
				'nopagin'        => true,
				'posts_per_page' => 1000,

Detected high pagination limit, posts_per_page is set to 1000

				'orderby'        => 'title',
				'order'          => 'ASC',
			),
			'repeatable' => true,
			'sortable'   => true,
			'allow_none' => true,
		);
		$fields[] = array(
			'id'   => 'destinations_title',
			'name' => 'Destinations',
			'type' => 'title',
		);
		$fields[] = array(
			'id'         => 'destination_to_vehicle',
			'name'       => 'Destinations related with this vehicle',
			'type'       => 'post_select',
			'use_ajax'   => false,
			'query'      => array(
				'post_type'      => 'destination',
				'nopagin'        => true,
				'posts_per_page' => 1000,

Detected high pagination limit, posts_per_page is set to 1000

				'orderby'        => 'title',
				'order'          => 'ASC',
			),
			'repeatable' => true,
			'sortable'   => true,
			'allow_none' => true,
		);
		$fields[] = array(
			'id'   => 'review_title',
			'name' => 'Reviews',
			'type' => 'title',
		);
		$fields[] = array(
			'id'         => 'review_to_vehicle',
			'name'       => 'Reviews related with this vehicle',
			'type'       => 'post_select',
			'use_ajax'   => false,
			'query'      => array(
				'post_type'      => 'reviews',
				'nopagin'        => true,
				'posts_per_page' => 1000,

Detected high pagination limit, posts_per_page is set to 1000

				'orderby'        => 'title',
				'order'          => 'ASC',
			),
			'repeatable' => true,
			'sortable'   => true,
			'allow_none' => true,
		);
		$fields[] = array(
			'id'   => 'specials_title',
			'name' => 'Specials',
			'type' => 'title',
		);
		$fields[] = array(
			'id'         => 'special_to_vehicle',
			'name'       => 'Specials related with this vehicle',
			'type'       => 'post_select',
			'use_ajax'   => false,
			'query'      => array(
				'post_type'      => 'special',
				'nopagin'        => true,
				'posts_per_page' => 1000,

Detected high pagination limit, posts_per_page is set to 1000

				'orderby'        => 'title',
				'order'          => 'ASC',
			),
			'repeatable' => true,
			'sortable'   => true,
			'allow_none' => true,
		);
		$fields[] = array(
			'id'   => 'team_title',
			'name' => 'Team Members',
			'type' => 'title',
		);
		$fields[] = array(
			'id'         => 'team_to_vehicle',
			'name'       => 'Team members related with this vehicle',
			'type'       => 'post_select',
			'use_ajax'   => false,
			'query'      => array(
				'post_type'      => 'team',
				'nopagin'        => true,
				'posts_per_page' => 1000,

Detected high pagination limit, posts_per_page is set to 1000

				'orderby'        => 'title',
				'order'          => 'ASC',
			),
			'repeatable' => true,
			'sortable'   => true,
			'allow_none' => true,
		);
		$fields[] = array(
			'id'   => 'tours_title',
			'name' => 'Tours',
			'type' => 'title',
		);
		$fields[] = array(
			'id'         => 'tour_to_vehicle',
			'name'       => 'Tours related with this vehicle',
			'type'       => 'post_select',
			'use_ajax'   => false,
			'query'      => array(
				'post_type'      => 'tour',
				'nopagin'        => true,
				'posts_per_page' => 1000,

Detected high pagination limit, posts_per_page is set to 1000

				'orderby'        => 'title',
				'order'          => 'ASC',
			),
			'repeatable' => true,
			'sortable'   => true,
			'allow_none' => true,
		);

		$fields = apply_filters( 'lsx_to_vehicle_custom_fields', $fields );

		$meta_boxes[] = array(
			'title'  => 'Tour Operator Plugin',
			'pages'  => 'vehicle',
			'fields' => $fields,
		);
		return $meta_boxes;

	}

Expected 2 blank lines after function; 1 found

	/**

Doc comment for parameter "$fields" missing

	 * Adds in the gallery fields to the Tour Operators Post Types.
	 */
	public function custom_fields( $fields ) {
		global $post, $typenow, $current_screen;

		$post_type = false;
		if ( $post && $post->post_type ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$post_type = $post->post_type;
		} elseif ( $typenow ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$post_type = $typenow;
		} elseif ( $current_screen && $current_screen->post_type ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$post_type = $current_screen->post_type;
		} elseif ( isset( $_REQUEST['post_type'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Processing form data without nonce verification.

Expected 0 spaces before closing bracket; 1 found

			$post_type = sanitize_key( $_REQUEST['post_type'] );

Processing form data without nonce verification.

		} elseif ( isset( $_REQUEST['post'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Processing form data without nonce verification.

Expected 0 spaces before closing bracket; 1 found

			$post_type = get_post_type( sanitize_key( $_REQUEST['post'] ) );

Processing form data without nonce verification.

sanitize_key($_REQUEST['post']) of type string is incompatible with the type WP_Post|integer|null expected by parameter $post of get_post_type().

		}

No blank line found after control structure

		//$post_type = get_post_type();

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

No space found before comment text; expected "// $post_type = get_post_type();" but found "//$post_type = get_post_type();"

		if ( false !== $post_type ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$fields[]  = array(

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

				'id'   => 'vehicle_title',
				'name' => 'Vehicles',
				'type' => 'title',
				'cols' => 12,
			);
			$fields[]  = array(

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

				'id'         => 'vehicle_to_' . $post_type,
				'name'       => 'Vehicles related with this ' . $post_type,
				'type'       => 'post_select',
				'use_ajax'   => false,
				'query'      => array(
					'post_type'      => 'vehicle',
					'nopagin'        => true,
					'posts_per_page' => '-1',
					'orderby'        => 'title',
					'order'          => 'ASC',
				),
				'repeatable' => true,
				'allow_none' => true,
				'cols'       => 12,
			);
		}

No blank line found after control structure

		return $fields;
	}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 0 found

}
new LSX_TO_Vehicles_Admin();