lightspeeddevelopment / to-specials

classes/class-to-specials-schema.php

<?php

Class file names should be based on the class name with "class-" prepended. Expected class-lsx-to-specials-schema.php, but found class-to-specials-schema.php.

/**
 * The Trip Schema for Tours
 *
 * @package tour-operator
 */

/**
 * Returns schema Review data.
 *
 * @since 10.2
 */
class LSX_TO_Specials_Schema extends LSX_TO_Schema_Graph_Piece {

	/**
	 * Constructor.
	 *
	 * @param \WPSEO_Schema_Context $context A value object with context variables.
	 */
	public function __construct( WPSEO_Schema_Context $context ) {

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

Expected 2 blank lines before function; 1 found

		$this->post_type = 'special';

The property post_type does not exist on LSX_TO_Specials_Schema. Did you mean post_types_singular?

		parent::__construct( $context );

The call to LSX_TO_Specials::__construct() has too many arguments starting with $context.

	}

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

	/**
	 * Returns Review data.
	 *
	 * @return array $data Review data.
	 */
	public function generate() {
		$tour_list  = get_post_meta( get_the_ID(), 'tour_to_special', false );

It seems like get_the_ID() can also be of type false; however, parameter $post_id of get_post_meta() does only seem to accept integer, maybe add an additional type check?

		$accom_list = get_post_meta( get_the_ID(), 'accommodation_to_special', false );
		$data       = array(
			'@type'            => array(
				'Offer',
			),
			'@id'              => $this->context->canonical . '#special',

The property context does not exist on LSX_TO_Specials_Schema. Did you maybe forget to declare it?

			'name'             => $this->post->post_title,

The property post does not exist on LSX_TO_Specials_Schema. Did you maybe forget to declare it?

			'description'      => wp_strip_all_tags( $this->post->post_content ),
			'url'              => $this->post_url,

The property post_url does not exist on LSX_TO_Specials_Schema. Did you maybe forget to declare it?

			'mainEntityOfPage' => array(
				'@id' => $this->context->canonical . WPSEO_Schema_IDs::WEBPAGE_HASH,

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

			),
		);

		if ( $this->context->site_represents_reference ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$data['offeredBy'] = $this->context->site_represents_reference;
		}

		$data = $this->add_custom_field( $data, 'availabilityStarts', 'booking_validity_start' );

The method add_custom_field() does not exist on LSX_TO_Specials_Schema.

		$data = $this->add_custom_field( $data, 'availabilityEnds', 'booking_validity_end' );
		$data = $this->add_custom_field( $data, 'priceValidUntil', 'booking_validity_end' );
		$data = $this->add_custom_field( $data, 'priceValidUntil', 'booking_validity_end' );
		$data = $this->get_price( $data );

The method get_price() does not exist on LSX_TO_Specials_Schema.

		$data['itemOffered'] = \lsx\legacy\Schema_Utils::get_item_reviewed( $tour_list, 'Product' );

It seems like $tour_list can also be of type false; however, parameter $items of lsx\legacy\Schema_Utils::get_item_reviewed() does only seem to accept array, maybe add an additional type check?

		$data['itemOffered'] = \lsx\legacy\Schema_Utils::get_item_reviewed( $accom_list, 'Product' );

		$data = $this->add_articles( $data );

The method add_articles() does not exist on LSX_TO_Specials_Schema.

		$data = \lsx\legacy\Schema_Utils::add_image( $data, $this->context );

		return $data;
	}

Expected 2 blank lines after function; 1 found

Expected 1 blank line before closing function brace; 0 found

	/**
	 * Gets the single special post and adds it as a special "Offer".
	 *
	 * @param  array $data An array of offers already added.
	 * @return array $data
	 */
	public function get_price( $data ) {
		$price         = get_post_meta( $this->context->id, 'price', true );

The property context does not exist on LSX_TO_Specials_Schema. Did you maybe forget to declare it?

		$currency      = 'USD';
		$tour_operator = tour_operator();
		if ( is_object( $tour_operator ) && isset( $tour_operator->options['general'] ) && is_array( $tour_operator->options['general'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			if ( isset( $tour_operator->options['general']['currency'] ) && ! empty( $tour_operator->options['general']['currency'] ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$currency = $tour_operator->options['general']['currency'];
			}
		}

No blank line found after control structure

		if ( false !== $price && '' !== $price ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$data['price']         = $price;
			$data['priceCurrency'] = $currency;
			$data['category']      = __( 'Special', 'tour-operator-specials' );
			$data['availability']  = 'https://schema.org/LimitedAvailability';

			$price_type = get_post_meta( $this->context->id, 'price_type', true );
			if ( false !== $price_type && '' !== $price_type && 'none' !== $price_type ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$data['PriceSpecification'] = lsx_to_get_price_type_label( $price_type );
			}
		}

No blank line found after control structure

		return $data;
	}

Expected 2 blank lines after function; 0 found

Expected 1 blank line before closing function brace; 0 found

}