lightspeeddevelopment / lsx-health-plan

lsx-health-plan.php

<?php
/*
 * Plugin Name:	LSX Health Plan
 * Plugin URI:	https://github.com/lightspeeddevelopment/lsx-health-plan
 * Description:	LSX Health Plan extension adds a meal and workout plan, with recipes.
 * Author:		LightSpeed
 * Version: 	1.4.0
 * Author URI: 	https://www.lsdev.biz/
 * License: 	GPL3
 * Text Domain: lsx-health-plan
 * Domain Path: /languages/
 */

// If this file is called directly, abort.
if ( ! defined( 'WPINC' ) ) {
	die;
}
define( 'LSX_HEALTH_PLAN_PATH', plugin_dir_path( __FILE__ ) );
define( 'LSX_HEALTH_PLAN_CORE', __FILE__ );
define( 'LSX_HEALTH_PLAN_URL', plugin_dir_url( __FILE__ ) );
define( 'LSX_HEALTH_PLAN_VER', '1.4.0' );

/* ======================= Below is the Plugin Class init ========================= */

require_once LSX_HEALTH_PLAN_PATH . '/classes/class-core.php';

/**
 * Remove unnecessary custom post types
 *
 * @return void
 */
function lsx_remove_extra_meta_box() {
	global $wp_meta_boxes;
	$all_post_types = [ 'plan', 'video', 'workout', 'tip', 'recipe', 'meal' ];
	//remove_meta_box( 'wpseo_meta', $all_post_types, 'normal' );
	remove_meta_box( 'commentsdiv', $all_post_types, 'normal' );
	remove_meta_box( 'commentstatusdiv', $all_post_types, 'normal' );
	remove_meta_box( 'lsx_blocks_title_meta', $all_post_types, 'side' );
}
add_action( 'add_meta_boxes', 'lsx_remove_extra_meta_box', 100 );

/**
 * Create Login page with woocommerce shortcode if the page is not created
 */
if ( ( isset( $_GET['activated'] ) && function_exists( 'wp_verify_nonce' ) && wp_verify_nonce( sanitize_key( $_GET['activated'] ) ) ) && is_admin() ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

Blank line found at start of control structure

	$new_page_title = 'Login';
	// Content to add spacing and woocommerce login shortcode

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

	$new_page_content  = '<!-- wp:lsx-blocks/lsx-container {"containerMaxWidth":600} -->
	<div style="background-color:transparent;padding-left:3%;padding-right:3%;padding-bottom:3%;padding-top:3%;margin-top:3%;margin-bottom:3%" class="wp-block-lsx-blocks-lsx-container aligncenter lsx-block-container"><div class="lsx-container-inside"><div class="lsx-container-content" style="max-width:600px"><!-- wp:paragraph -->
	<p></p>
	<!-- /wp:paragraph -->
	
	<!-- wp:shortcode -->
	[woocommerce_my_account]
	<!-- /wp:shortcode --></div></div></div>
	<!-- /wp:lsx-blocks/lsx-container -->';
	$new_page_template = '';

	$page_check = get_page_by_title( $new_page_title );
	$new_page   = array(
		'post_type'    => 'page',
		'post_title'   => $new_page_title,
		'post_content' => $new_page_content,
		'post_status'  => 'publish',
		'post_author'  => 1,
	);
	if ( ! isset( $page_check->ID ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		$new_page_id = wp_insert_post( $new_page );
		if ( ! empty( $new_page_template ) ) {

The condition empty($new_page_template) is always true.

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			update_post_meta( $new_page_id, '_wp_page_template', $new_page_template );
		}
	}
}

/**
 * Add Login Logout Button to Main Menu
 *
 * @param [type] $items

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

Missing parameter comment

 * @param [type] $args
 * @return void
 */
function lsx_add_login_logout_register_menu( $items, $args ) {

Expected 2 blank lines before function; 1 found

	if ( 'primary' === $args->theme_location ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

		ob_start();
		wp_loginout( get_permalink() );

It seems like get_permalink() can also be of type false; however, parameter $redirect of wp_loginout() does only seem to accept string, maybe add an additional type check?

		$loginoutlink = ob_get_contents();
		ob_end_clean();
		if ( ! is_user_logged_in() ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

			$login_slug = \lsx_health_plan\functions\get_option( 'login_slug', false );
			if ( false === $login_slug ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

				$login_slug = 'login';
			}

No blank line found after control structure

			$items .= '<li class="my-login menu-item"><a rel="nofollow" href="/' . $login_slug . '/">' . __( 'Login', 'lsx-health-plan' ) . '</a></li>';
		} else {
			$items .= '<li class="my-login menu-item">' . $loginoutlink . '</li>';
		}

No blank line found after control structure

		return $items;
	} else {
		return $items;
	}
}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 0 found

add_filter( 'wp_nav_menu_items', 'lsx_add_login_logout_register_menu', 199, 2 );

/**
 * Redirect user after login or redirect
 *
 * @return void
 */
function lsx_login_redirect() {
	$plan_slug = \lsx_health_plan\functions\get_option( 'my_plan_slug', false );
	if ( false === $plan_slug ) {
		$plan_slug = 'my-plan';
	}
	return home_url( $plan_slug );
}
add_filter( 'woocommerce_login_redirect', 'lsx_login_redirect' );

/**
 * Undocumented function
 *
 * @return object lsx_health_plan\classes\Core::get_instance();
 */
function lsx_health_plan() {
	return \lsx_health_plan\classes\Core::get_instance();
}
lsx_health_plan();

/**
 * Creates the svg path
 *
 * @return void
 */
function lsx_get_svg_icon( $icon ) {
	$path = '/assets/images/';

	if ( file_exists( LSX_HEALTH_PLAN_PATH . $path . $icon ) ) {
		// Load and return the contents of the file

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

		return include LSX_HEALTH_PLAN_PATH . $path . $icon;
	}

	// Return a blank string if we can't find the file.
	return '';
}

Expected 1 blank line before closing function brace; 0 found

Expected 2 blank lines after function; 1 found

/**

Doc comment for parameter "$snack" missing

 * Workout Snacks
 *
 * @return void
 */
function lsx_workout_snacks( $snack ) {
	$workout_snack = get_post_meta( get_the_ID(), $snack . '_workout_snack', true );

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?

	if ( ! empty( $workout_snack ) ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

	?>
	<div class="<?php echo esc_html( $snack ); ?>-workout workout-snacks">
			<div class="content-box">
				<?php
				$snack_title = ucfirst( $snack );
				if ( 'pre' === $snack ) {

Expected 0 spaces after opening bracket; 1 found

Expected 0 spaces before closing bracket; 1 found

					/* Translators: %s: snack */
					$title_text = esc_attr_x( 'Pre-Workout Snack', 'pre workout', 'lsx-health-plan' );
				} else {
					/* Translators: %s: snack */
					$title_text = esc_attr_x( 'Post-Workout Snack', 'post workout', 'lsx-health-plan' );
				}

No blank line found after control structure

				$title = sprintf( $title_text, $snack_title );
				?>
				<h3 class="title-lined"><?php echo esc_html( $title ); ?></h3>
				<?php echo wp_kses_post( apply_filters( 'the_content', $workout_snack ) ); ?>
			</div>
		</div>
	<?php
	}
}

Expected 1 blank line before closing function brace; 0 found