seb86 / woocommerce-subscribe-to-all-the-things-signup-trial-add-on

includes/class-wcsatt-stt-display.php

Spacing +36 added lines, -36 removed lines

@@ -16,22 +16,22 @@ 
 	 */
 	public static function init() {
 		// Adds the sign up fee and trial data to the price html on the 'wcsatt_overridden_subscription_prices_product' filter.
-		add_filter( 'wcsatt_overridden_subscription_prices_product', __CLASS__ . '::add_sub_scheme_data_price_html', 10, 3 );
+		add_filter('wcsatt_overridden_subscription_prices_product', __CLASS__.'::add_sub_scheme_data_price_html', 10, 3);
 
 		// Adds the extra subscription scheme data to the product object on the 'wcsatt_converted_product_for_scheme_option' filter.
-		add_filter( 'wcsatt_converted_product_for_scheme_option', __CLASS__ . '::sub_product_scheme_option', 10, 2 );
+		add_filter('wcsatt_converted_product_for_scheme_option', __CLASS__.'::sub_product_scheme_option', 10, 2);
 
 		// Filters the price string to include the sign up fee and/or trial to pass per scheme option on the 'wcsatt_single_product_subscription_scheme_price_html' filter.
-		add_filter( 'wcsatt_single_product_subscription_scheme_price_html', __CLASS__ . '::get_price_string', 10, 2 );
+		add_filter('wcsatt_single_product_subscription_scheme_price_html', __CLASS__.'::get_price_string', 10, 2);
 
 		// Filters the lowest price string to include the sign up fee on the 'wcsatt_get_single_product_lowest_price_string' filter.
-		add_filter( 'wcsatt_get_single_product_lowest_price_string', __CLASS__ . '::get_lowest_price_string', 10, 2 );
+		add_filter('wcsatt_get_single_product_lowest_price_string', __CLASS__.'::get_lowest_price_string', 10, 2);
 
 		// Filters the lowest price subscription scheme data on the 'wcsatt_get_lowest_price_sub_scheme_data' filter.
-		add_filter( 'wcsatt_get_lowest_price_sub_scheme_data', __CLASS__ . '::get_lowest_price_sub_scheme_data', 10, 2 );
+		add_filter('wcsatt_get_lowest_price_sub_scheme_data', __CLASS__.'::get_lowest_price_sub_scheme_data', 10, 2);
 
 		// Adds the sign-up and/or trial data to the subscription scheme prices on the 'wcsatt_subscription_scheme_prices' filter.
-		add_filter( 'wcsatt_subscription_scheme_prices', __CLASS__ . '::add_subscription_scheme_prices', 10, 2 );
+		add_filter('wcsatt_subscription_scheme_prices', __CLASS__.'::add_subscription_scheme_prices', 10, 2);
 	}
 
 	/**
@@ -44,17 +44,17 @@ 
 	 * @param  WC_Product $product
 	 * @return string
 	 */
-	public static function add_sub_scheme_data_price_html( $_product, $subscription_scheme, $product ) {
-		if ( isset( $subscription_scheme[ 'subscription_sign_up_fee' ] ) ) {
-			$_product->subscription_sign_up_fee = $subscription_scheme[ 'subscription_sign_up_fee' ];
+	public static function add_sub_scheme_data_price_html($_product, $subscription_scheme, $product) {
+		if (isset($subscription_scheme['subscription_sign_up_fee'])) {
+			$_product->subscription_sign_up_fee = $subscription_scheme['subscription_sign_up_fee'];
 		}
 
-		if ( isset( $subscription_scheme[ 'subscription_trial_length' ] ) ) {
-			$_product->subscription_trial_length = $subscription_scheme[ 'subscription_trial_length' ];
+		if (isset($subscription_scheme['subscription_trial_length'])) {
+			$_product->subscription_trial_length = $subscription_scheme['subscription_trial_length'];
 		}
 
-		if ( isset( $subscription_scheme[ 'subscription_trial_period' ] ) ) {
-			$_product->subscription_trial_period = $subscription_scheme[ 'subscription_trial_period' ];
+		if (isset($subscription_scheme['subscription_trial_period'])) {
+			$_product->subscription_trial_period = $subscription_scheme['subscription_trial_period'];
 		}
 
 		return $_product;
@@ -71,14 +71,14 @@ 
 	 * @param  array  $subscription_scheme
 	 * @return object
 	 */
-	public static function sub_product_scheme_option( $_cloned, $subscription_scheme ) {
-		if ( isset( $subscription_scheme[ 'subscription_sign_up_fee' ] ) && $subscription_scheme[ 'subscription_sign_up_fee' ] > 0 ) {
-			$_cloned->subscription_sign_up_fee = $subscription_scheme[ 'subscription_sign_up_fee' ];
+	public static function sub_product_scheme_option($_cloned, $subscription_scheme) {
+		if (isset($subscription_scheme['subscription_sign_up_fee']) && $subscription_scheme['subscription_sign_up_fee'] > 0) {
+			$_cloned->subscription_sign_up_fee = $subscription_scheme['subscription_sign_up_fee'];
 		}
 
-		if ( isset( $subscription_scheme[ 'subscription_trial_length' ] ) && 0 != $subscription_scheme[ 'subscription_trial_length' ] ) {
-			$_cloned->subscription_trial_length = $subscription_scheme[ 'subscription_trial_length' ];
-			$_cloned->subscription_trial_period = $subscription_scheme[ 'subscription_trial_period' ];
+		if (isset($subscription_scheme['subscription_trial_length']) && 0 != $subscription_scheme['subscription_trial_length']) {
+			$_cloned->subscription_trial_length = $subscription_scheme['subscription_trial_length'];
+			$_cloned->subscription_trial_period = $subscription_scheme['subscription_trial_period'];
 		}
 
 		return $_cloned;
@@ -94,13 +94,13 @@ 
 	 * @param  array $subscription_scheme
 	 * @return array
 	 */
-	public static function get_price_string( $prices, $subscription_scheme ) {
-		if ( isset( $subscription_scheme[ 'subscription_sign_up_fee' ] ) && $subscription_scheme[ 'subscription_sign_up_fee' ] > 0 ) {
-			$prices[ 'sign_up_fee' ] = $subscription_scheme[ 'subscription_sign_up_fee' ];
+	public static function get_price_string($prices, $subscription_scheme) {
+		if (isset($subscription_scheme['subscription_sign_up_fee']) && $subscription_scheme['subscription_sign_up_fee'] > 0) {
+			$prices['sign_up_fee'] = $subscription_scheme['subscription_sign_up_fee'];
 		}
 
-		if ( isset( $subscription_scheme[ 'subscription_trial_length' ] ) && 0 != $subscription_scheme[ 'subscription_trial_length' ] ) {
-			$prices[ 'trial_length' ] = true;
+		if (isset($subscription_scheme['subscription_trial_length']) && 0 != $subscription_scheme['subscription_trial_length']) {
+			$prices['trial_length'] = true;
 		}
 
 		return $prices;
@@ -116,9 +116,9 @@ 
 	 * @param  array $lowest_subscription_scheme
 	 * @return array
 	 */
-	public static function get_lowest_price_string( $prices,  $lowest_subscription_scheme ) {
-		if ( isset( $lowest_subscription_scheme[ 'sign_up_fee' ] ) && $lowest_subscription_scheme[ 'sign_up_fee' ] > 0 ) {
-			$prices[ 'sign_up_fee' ] = $lowest_subscription_scheme[ 'sign_up_fee' ];
+	public static function get_lowest_price_string($prices, $lowest_subscription_scheme) {
+		if (isset($lowest_subscription_scheme['sign_up_fee']) && $lowest_subscription_scheme['sign_up_fee'] > 0) {
+			$prices['sign_up_fee'] = $lowest_subscription_scheme['sign_up_fee'];
 		}
 
 		return $prices;
@@ -133,8 +133,8 @@ 
 	 * @param array $lowest_scheme
 	 * @return array
 	 */
-	public static function get_lowest_price_sub_scheme_data( $data, $lowest_scheme ) {
-		if ( isset( $lowest_scheme['subscription_sign_up_fee'] ) && $lowest_scheme['subscription_sign_up_fee'] > 0 ) {
+	public static function get_lowest_price_sub_scheme_data($data, $lowest_scheme) {
+		if (isset($lowest_scheme['subscription_sign_up_fee']) && $lowest_scheme['subscription_sign_up_fee'] > 0) {
 			$data['sign_up_fee'] = $lowest_scheme['subscription_sign_up_fee'];
 		}
 
@@ -150,17 +150,17 @@ 
 	 * @param  array $subscription_scheme
 	 * @return array
 	 */
-	public static function add_subscription_scheme_prices( $prices, $subscription_scheme ) {
-		if ( isset( $subscription_scheme[ 'subscription_sign_up_fee' ] ) ) {
-			$prices[ 'sign_up_fee' ] = $subscription_scheme[ 'subscription_sign_up_fee' ];
+	public static function add_subscription_scheme_prices($prices, $subscription_scheme) {
+		if (isset($subscription_scheme['subscription_sign_up_fee'])) {
+			$prices['sign_up_fee'] = $subscription_scheme['subscription_sign_up_fee'];
 		}
 
-		if ( isset( $subscription_scheme[ 'subscription_trial_length' ] ) ) {
-			$prices[ 'trial_length' ] = $subscription_scheme[ 'subscription_trial_length' ];
+		if (isset($subscription_scheme['subscription_trial_length'])) {
+			$prices['trial_length'] = $subscription_scheme['subscription_trial_length'];
 		}
 
-		if ( isset( $subscription_scheme[ 'subscription_trial_period' ] ) ) {
-			$prices[ 'trial_period' ] = $subscription_scheme[ 'subscription_trial_period' ];
+		if (isset($subscription_scheme['subscription_trial_period'])) {
+			$prices['trial_period'] = $subscription_scheme['subscription_trial_period'];
 		}
 
 		return $prices;

includes/class-wcsatt-stt-cart.php

Spacing +13 added lines, -13 removed lines

@@ -16,7 +16,7 @@ 
 	 */
 	public static function init() {
 		// Overrides the price of the subscription for sign up fee and/or trial on the 'wcsatt_cart_item' filter.
-		add_filter( 'wcsatt_cart_item', __CLASS__ . '::update_cart_item_sub_data', 10, 1 );
+		add_filter('wcsatt_cart_item', __CLASS__.'::update_cart_item_sub_data', 10, 1);
 	}
 
 	/**
@@ -28,38 +28,38 @@ 
 	 * @param  array $cart_item
 	 * @return array
 	 */
-	public static function update_cart_item_sub_data( $cart_item ) {
-		$active_scheme = WCS_ATT_Schemes::get_active_subscription_scheme( $cart_item );
+	public static function update_cart_item_sub_data($cart_item) {
+		$active_scheme = WCS_ATT_Schemes::get_active_subscription_scheme($cart_item);
 
-		$subscription_prices = WCS_ATT_Scheme_Prices::get_active_subscription_scheme_prices( $cart_item, $active_scheme );
+		$subscription_prices = WCS_ATT_Scheme_Prices::get_active_subscription_scheme_prices($cart_item, $active_scheme);
 
-		if ( $active_scheme && $cart_item['data']->is_converted_to_sub == 'yes' ) {
+		if ($active_scheme && $cart_item['data']->is_converted_to_sub == 'yes') {
 
 			// Subscription Price
 			$price = $cart_item['data']->subscription_price;
 
 			// Is there a sign up fee?
-			$sign_up_fee = isset( $subscription_prices['sign_up_fee'] ) ? $subscription_prices['sign_up_fee'] : '';
+			$sign_up_fee = isset($subscription_prices['sign_up_fee']) ? $subscription_prices['sign_up_fee'] : '';
 
 			// Checks if the cart item is a supported bundle type child.
-			$container_key = WCS_ATT_Integrations::has_bundle_type_container( $cart_item );
+			$container_key = WCS_ATT_Integrations::has_bundle_type_container($cart_item);
 
 			// If the cart item is a child item then reset the sign-up fee.
-			if ( false !== $container_key ) { $sign_up_fee = ''; }
+			if (false !== $container_key) { $sign_up_fee = ''; }
 
 			// Put both the subscription price and the sign-up fee together.
-			$new_price = round( $price + $sign_up_fee, wc_get_price_decimals() );
+			$new_price = round($price + $sign_up_fee, wc_get_price_decimals());
 
-			if ( $sign_up_fee > 0 ) {
+			if ($sign_up_fee > 0) {
 				$cart_item['data']->initial_amount = $new_price;
 				$cart_item['data']->subscription_sign_up_fee = $sign_up_fee;
 			}
 
-			$trial_length = isset( $subscription_prices['trial_length'] ) ? $subscription_prices['trial_length'] : 0;
-			$trial_period = isset( $subscription_prices['trial_period'] ) ? $subscription_prices['trial_period'] : '';
+			$trial_length = isset($subscription_prices['trial_length']) ? $subscription_prices['trial_length'] : 0;
+			$trial_period = isset($subscription_prices['trial_period']) ? $subscription_prices['trial_period'] : '';
 
 			// If a trial length is more than zero then set the conditions for the cart.
-			if ( $trial_length > 0 ) {
+			if ($trial_length > 0) {
 				$cart_item['data']->subscription_trial_length = $trial_length;
 				$cart_item['data']->subscription_trial_period = $trial_period;
 			} else {

includes/admin/class-wcsatt-stt-admin.php

Spacing +46 added lines, -46 removed lines

@@ -16,16 +16,16 @@ 
 	 */
 	public static function init() {
 		// Admin scripts and styles.
-		add_action( 'admin_enqueue_scripts', __CLASS__ . '::admin_style' );
+		add_action('admin_enqueue_scripts', __CLASS__.'::admin_style');
 
 		// Adds to the default values for subscriptions schemes content.
-		add_filter( 'wcsatt_default_subscription_scheme', __CLASS__ . '::subscription_schemes_content', 10, 1 );
+		add_filter('wcsatt_default_subscription_scheme', __CLASS__.'::subscription_schemes_content', 10, 1);
 
 		// Subscription scheme options displayed on the 'wcsatt_subscription_scheme_product_content' action.
-		add_action( 'wcsatt_subscription_scheme_product_content', __CLASS__ . '::wcsatt_stt_fields', 15, 3 );
+		add_action('wcsatt_subscription_scheme_product_content', __CLASS__.'::wcsatt_stt_fields', 15, 3);
 
 		// Filter the subscription scheme data to process the sign up and trial options on the ''wcsatt_subscription_scheme_process_scheme_data' filter.
-		add_filter( 'wcsatt_subscription_scheme_process_scheme_data', __CLASS__ . '::wcsatt_stt_process_scheme_data', 10, 2 );
+		add_filter('wcsatt_subscription_scheme_process_scheme_data', __CLASS__.'::wcsatt_stt_process_scheme_data', 10, 2);
 	}
 
 	/**
@@ -38,9 +38,9 @@ 
 		$screen    = get_current_screen();
 		$screen_id = $screen ? $screen->id : '';
 
-		if ( in_array( $screen_id, array( 'edit-product', 'product' ) ) ) {
-			wp_register_style( 'wcsatt_stt_writepanel', WCSATT_STT::plugin_url() . '/assets/css/wcsatt-stt-write-panel.css', array( 'woocommerce_admin_styles' ), WCSATT_STT::VERSION );
-			wp_enqueue_style( 'wcsatt_stt_writepanel' );
+		if (in_array($screen_id, array('edit-product', 'product'))) {
+			wp_register_style('wcsatt_stt_writepanel', WCSATT_STT::plugin_url().'/assets/css/wcsatt-stt-write-panel.css', array('woocommerce_admin_styles'), WCSATT_STT::VERSION);
+			wp_enqueue_style('wcsatt_stt_writepanel');
 		}
 
 	} // END admin_style()
@@ -53,14 +53,14 @@ 
 	 * @param  array $defaults
 	 * @return void
 	 */
-	public static function subscription_schemes_content( $defaults ) {
+	public static function subscription_schemes_content($defaults) {
 		$new_defaults = array(
 			'subscription_sign_up_fee'  => '',
 			'subscription_trial_length' => '',
 			'subscription_trial_period' => ''
 		);
 
-		return array_merge( $new_defaults, $defaults );
+		return array_merge($new_defaults, $defaults);
 	} // END subscription_schemes_content()
 
 	/**
@@ -73,11 +73,11 @@ 
 	 * @param  int   $post_id
 	 * @return void
 	 */
-	public static function wcsatt_stt_fields( $index, $scheme_data, $post_id ) {
-		if ( ! empty( $scheme_data ) ) {
-			$subscription_sign_up_fee = ! empty( $scheme_data[ 'subscription_sign_up_fee' ] ) ? $scheme_data[ 'subscription_sign_up_fee' ] : '';
-			$subscription_trial_length = isset( $scheme_data[ 'subscription_trial_length' ] ) ? $scheme_data[ 'subscription_trial_length' ] : 0;
-			$subscription_trial_period = isset( $scheme_data[ 'subscription_trial_period' ] ) ? $scheme_data[ 'subscription_trial_period' ] : '';
+	public static function wcsatt_stt_fields($index, $scheme_data, $post_id) {
+		if (!empty($scheme_data)) {
+			$subscription_sign_up_fee = !empty($scheme_data['subscription_sign_up_fee']) ? $scheme_data['subscription_sign_up_fee'] : '';
+			$subscription_trial_length = isset($scheme_data['subscription_trial_length']) ? $scheme_data['subscription_trial_length'] : 0;
+			$subscription_trial_period = isset($scheme_data['subscription_trial_period']) ? $scheme_data['subscription_trial_period'] : '';
 		} else {
 			$subscription_sign_up_fee = '';
 			$subscription_trial_length = 0;
@@ -87,47 +87,47 @@ 
 		echo '<div class="options_group subscription_scheme_product_data sign_up_trial_scheme">';
 
 		// Sign-up Fee
-		woocommerce_wp_text_input( array(
+		woocommerce_wp_text_input(array(
 			'id'          => '_subscription_sign_up_fee',
 			'class'       => 'wc_input_subscription_intial_price',
 			// translators: %s is a currency symbol / code
-			'label'       => sprintf( __( 'Sign-up Fee (%s)', WCSATT_STT::TEXT_DOMAIN ), get_woocommerce_currency_symbol() ),
-			'placeholder' => _x( 'e.g. 9.90', 'example price', WCSATT_STT::TEXT_DOMAIN ),
-			'description' => __( 'Optionally include an amount to be charged at the outset of the subscription. The sign-up fee will be charged immediately, even if the product has a free trial or the payment dates are synced.', WCSATT_STT::TEXT_DOMAIN ),
+			'label'       => sprintf(__('Sign-up Fee (%s)', WCSATT_STT::TEXT_DOMAIN), get_woocommerce_currency_symbol()),
+			'placeholder' => _x('e.g. 9.90', 'example price', WCSATT_STT::TEXT_DOMAIN),
+			'description' => __('Optionally include an amount to be charged at the outset of the subscription. The sign-up fee will be charged immediately, even if the product has a free trial or the payment dates are synced.', WCSATT_STT::TEXT_DOMAIN),
 			'desc_tip'    => true,
 			'type'        => 'text',
 			'custom_attributes' => array(
 				'step' => 'any',
 				'min'  => '0',
 			),
-			'name'        => 'wcsatt_schemes[' . $index . '][subscription_sign_up_fee]',
+			'name'        => 'wcsatt_schemes['.$index.'][subscription_sign_up_fee]',
 			'value'       => $subscription_sign_up_fee
-		) );
+		));
 
 		echo '<div class="subscription_trial">';
 
 		// Trial Length
-		woocommerce_wp_text_input( array(
+		woocommerce_wp_text_input(array(
 			'id'          => '_subscription_trial_length',
 			'class'       => 'wc_input_subscription_trial_length',
-			'label'       => __( 'Free Trial', WCSATT_STT::TEXT_DOMAIN ),
-			'name'        => 'wcsatt_schemes[' . $index . '][subscription_trial_length]',
+			'label'       => __('Free Trial', WCSATT_STT::TEXT_DOMAIN),
+			'name'        => 'wcsatt_schemes['.$index.'][subscription_trial_length]',
 			'value'       => $subscription_trial_length
-		) );
+		));
 
 		// Trial Period
-		woocommerce_wp_select( array(
+		woocommerce_wp_select(array(
 			'id'          => '_subscription_trial_period',
 			'class'       => 'wc_input_subscription_trial_period',
-			'label'       => __( 'Subscription Trial Period', WCSATT_STT::TEXT_DOMAIN ),
+			'label'       => __('Subscription Trial Period', WCSATT_STT::TEXT_DOMAIN),
 			'options'     => wcs_get_available_time_periods(),
 			// translators: placeholder is trial period validation message if passed an invalid value (e.g. "Trial period can not exceed 4 weeks")
-			'description' => sprintf( _x( 'An optional period of time to wait before charging the first recurring payment. Any sign up fee will still be charged at the outset of the subscription. %s', 'Trial period dropdown\'s description in pricing fields', WCSATT_STT::TEXT_DOMAIN ), WC_Subscriptions_Admin::get_trial_period_validation_message() ),
+			'description' => sprintf(_x('An optional period of time to wait before charging the first recurring payment. Any sign up fee will still be charged at the outset of the subscription. %s', 'Trial period dropdown\'s description in pricing fields', WCSATT_STT::TEXT_DOMAIN), WC_Subscriptions_Admin::get_trial_period_validation_message()),
 			'desc_tip'    => true,
-			'value'       => WC_Subscriptions_Product::get_trial_period( $post_id ), // Explicitly set value in to ensure backward compatibility
-			'name'        => 'wcsatt_schemes[' . $index . '][subscription_trial_period]',
+			'value'       => WC_Subscriptions_Product::get_trial_period($post_id), // Explicitly set value in to ensure backward compatibility
+			'name'        => 'wcsatt_schemes['.$index.'][subscription_trial_period]',
 			'value'       => $subscription_trial_period
-	) );
+	));
 
 		echo '</div>';
 
@@ -144,38 +144,38 @@ 
 	 * @param  string $product_type
 	 * @return void
 	 */
-	public static function wcsatt_stt_process_scheme_data( $posted_scheme, $product_type ) {
+	public static function wcsatt_stt_process_scheme_data($posted_scheme, $product_type) {
 		// Copy variable type fields.
-		if ( 'variable' == $product_type ) {
-			if ( isset( $posted_scheme[ 'subscription_sign_up_fee_variable' ] ) ) {
-				$posted_scheme[ 'subscription_sign_up_fee' ] = $posted_scheme[ 'subscription_sign_up_fee_variable' ];
+		if ('variable' == $product_type) {
+			if (isset($posted_scheme['subscription_sign_up_fee_variable'])) {
+				$posted_scheme['subscription_sign_up_fee'] = $posted_scheme['subscription_sign_up_fee_variable'];
 			}
-			if ( isset( $posted_scheme[ 'subscription_trial_length_variable' ] ) ) {
-				$posted_scheme[ 'subscription_trial_length' ] = $posted_scheme[ 'subscription_trial_length_variable' ];
+			if (isset($posted_scheme['subscription_trial_length_variable'])) {
+				$posted_scheme['subscription_trial_length'] = $posted_scheme['subscription_trial_length_variable'];
 			}
-			if ( isset( $posted_scheme[ 'subscription_trial_period_variable' ] ) ) {
-				$posted_scheme[ 'subscription_trial_period' ] = $posted_scheme[ 'subscription_trial_period_variable'];
+			if (isset($posted_scheme['subscription_trial_period_variable'])) {
+				$posted_scheme['subscription_trial_period'] = $posted_scheme['subscription_trial_period_variable'];
 			}
 		}
 
 		// Format subscription sign up fee.
-		if ( isset( $posted_scheme[ 'subscription_sign_up_fee' ] ) ) {
-			$posted_scheme[ 'subscription_sign_up_fee' ] = ( $posted_scheme[ 'subscription_sign_up_fee' ] === '' ) ? '' : wc_format_decimal( $posted_scheme[ 'subscription_sign_up_fee' ] );
+		if (isset($posted_scheme['subscription_sign_up_fee'])) {
+			$posted_scheme['subscription_sign_up_fee'] = ($posted_scheme['subscription_sign_up_fee'] === '') ? '' : wc_format_decimal($posted_scheme['subscription_sign_up_fee']);
 		}
 
 		// Make sure trial period is within allowable range.
 		$subscription_ranges = wcs_get_subscription_ranges();
-		$max_trial_length = count( $subscription_ranges[ $posted_scheme[ 'subscription_trial_period' ] ] ) - 1;
+		$max_trial_length = count($subscription_ranges[$posted_scheme['subscription_trial_period']]) - 1;
 
 		// Format subscription trial length.
-		if ( isset( $posted_scheme[ 'subscription_trial_length' ] ) && $posted_scheme[ 'subscription_trial_length' ] > $max_trial_length ) {
-			$posted_scheme[ 'subscription_trial_length' ] = ( $posted_scheme[ 'subscription_trial_length' ] === '' ) ? '' : absint( $posted_scheme[ 'subscription_trial_length' ] );
+		if (isset($posted_scheme['subscription_trial_length']) && $posted_scheme['subscription_trial_length'] > $max_trial_length) {
+			$posted_scheme['subscription_trial_length'] = ($posted_scheme['subscription_trial_length'] === '') ? '' : absint($posted_scheme['subscription_trial_length']);
 		}
 
 		// Format subscription trial period.
-		$trial_periods = apply_filters( 'wcsatt_stt_trial_periods', array( 'day', 'week', 'month', 'year' ) );
-		if ( isset( $posted_scheme[ 'subscription_trial_period' ] ) && in_array( $posted_scheme[ 'subscription_trial_period' ], $trial_periods ) ) {
-			$posted_scheme[ 'subscription_trial_period' ] = trim( $posted_scheme[ 'subscription_trial_period' ] );
+		$trial_periods = apply_filters('wcsatt_stt_trial_periods', array('day', 'week', 'month', 'year'));
+		if (isset($posted_scheme['subscription_trial_period']) && in_array($posted_scheme['subscription_trial_period'], $trial_periods)) {
+			$posted_scheme['subscription_trial_period'] = trim($posted_scheme['subscription_trial_period']);
 		}
 
 		return $posted_scheme;

wc-satt-stt.php

Spacing +25 added lines, -25 removed lines

@@ -17,9 +17,9 @@ 
  * License: GNU General Public License v3.0
  * License URI: http://www.gnu.org/licenses/gpl-3.0.html
  */
-if ( ! defined('ABSPATH') ) exit; // Exit if accessed directly.
+if (!defined('ABSPATH')) exit; // Exit if accessed directly.
 
-if ( ! class_exists( 'WCSATT_STT' ) ) {
+if (!class_exists('WCSATT_STT')) {
 	class WCSATT_STT {
 
 		/* Plugin version. */
@@ -52,7 +52,7 @@ 
 		 * @since 1.0.0
 		 */
 		public static function instance() {
-			if ( is_null( self::$_instance ) ) {
+			if (is_null(self::$_instance)) {
 				self::$_instance = new self();
 			}
 			return self::$_instance;
@@ -64,7 +64,7 @@ 
 		 * @since 1.0.0
 		 */
 		public function __clone() {
-			_doing_it_wrong( __FUNCTION__, __( 'Foul!' ), '1.0.0' );
+			_doing_it_wrong(__FUNCTION__, __('Foul!'), '1.0.0');
 		}
 
 		/**
@@ -73,24 +73,24 @@ 
 		 * @since 1.0.0
 		 */
 		public function __wakeup() {
-			_doing_it_wrong( __FUNCTION__, __( 'Foul!' ), '1.0.0' );
+			_doing_it_wrong(__FUNCTION__, __('Foul!'), '1.0.0');
 		}
 
 		/**
 		 * Load the plugin.
 		 */
 		public function __construct() {
-			add_action( 'plugins_loaded', array( $this, 'load_plugin' ), 11 );
-			add_action( 'init', array( $this, 'init_plugin' ) );
-			add_filter( 'plugin_row_meta', array( $this, 'plugin_meta_links' ), 10, 4 );
+			add_action('plugins_loaded', array($this, 'load_plugin'), 11);
+			add_action('init', array($this, 'init_plugin'));
+			add_filter('plugin_row_meta', array($this, 'plugin_meta_links'), 10, 4);
 		}
 
 		public static function plugin_url() {
-			return plugins_url( basename( plugin_dir_path(__FILE__) ), basename( __FILE__ ) );
+			return plugins_url(basename(plugin_dir_path(__FILE__)), basename(__FILE__));
 		} // END plugin_url()
 
 		public static function plugin_path() {
-			return untrailingslashit( plugin_dir_path( __FILE__ ) );
+			return untrailingslashit(plugin_dir_path(__FILE__));
 		} // END plugin_path()
 
 		/*
@@ -102,23 +102,23 @@ 
 			global $woocommerce;
 
 			// Check that the required WooCommerce is running.
-			if ( version_compare( $woocommerce->version, self::REQ_WC_VERSION, '<' ) ) {
-				add_action( 'admin_notices', array( $this, 'wcsatt_stt_wc_admin_notice' ) );
+			if (version_compare($woocommerce->version, self::REQ_WC_VERSION, '<')) {
+				add_action('admin_notices', array($this, 'wcsatt_stt_wc_admin_notice'));
 				return false;
 			}
 
 			// Checks that WooCommerce Subscribe All the Things is running or is less than the required version.
-			if ( ! class_exists( 'WCS_ATT' ) || version_compare( WCS_ATT::VERSION, self::REQ_WCSATT_VERSION, '<' ) ) {
-				add_action( 'admin_notices', array( $this, 'wcsatt_stt_admin_notice' ) );
+			if (!class_exists('WCS_ATT') || version_compare(WCS_ATT::VERSION, self::REQ_WCSATT_VERSION, '<')) {
+				add_action('admin_notices', array($this, 'wcsatt_stt_admin_notice'));
 				return false;
 			}
 
-			require_once( 'includes/class-wcsatt-stt-cart.php' );
-			require_once( 'includes/class-wcsatt-stt-display.php' );
+			require_once('includes/class-wcsatt-stt-cart.php');
+			require_once('includes/class-wcsatt-stt-display.php');
 
 			// Admin includes
-			if ( is_admin() ) {
-				require_once( 'includes/admin/class-wcsatt-stt-admin.php' );
+			if (is_admin()) {
+				require_once('includes/admin/class-wcsatt-stt-admin.php');
 			}
 
 		} // END load_plugin()
@@ -129,7 +129,7 @@ 
 		 * @return void
 		 */
 		public function wcsatt_stt_wc_admin_notice() {
-			echo '<div class="error"><p>' . sprintf( __( '%1$s requires at least %2$s v%3$s in order to function. Please upgrade %2$s.', 'wc-satt-stt' ), 'WooCommerce Subscribe All the Things: Sign-up and Trial Add-on', 'WooCommerce', self::REQ_WC_VERSION ) . '</p></div>';
+			echo '<div class="error"><p>'.sprintf(__('%1$s requires at least %2$s v%3$s in order to function. Please upgrade %2$s.', 'wc-satt-stt'), 'WooCommerce Subscribe All the Things: Sign-up and Trial Add-on', 'WooCommerce', self::REQ_WC_VERSION).'</p></div>';
 		} // END wcsatt_stt_wc_admin_notice()
 
 		/**
@@ -138,7 +138,7 @@ 
 		 * @return void
 		 */
 		public function wcsatt_stt_admin_notice() {
-			echo '<div class="error"><p>' . sprintf( __( '%1$s requires at least %2$s v%3$s in order to function. Please upgrade %2$s.', 'wc-satt-stt' ), 'WooCommerce Subscribe All the Things: Sign-up and Trial Add-on', 'WooCommerce Subscribe All the Things', self::REQ_WCSATT_VERSION ) . '</p></div>';
+			echo '<div class="error"><p>'.sprintf(__('%1$s requires at least %2$s v%3$s in order to function. Please upgrade %2$s.', 'wc-satt-stt'), 'WooCommerce Subscribe All the Things: Sign-up and Trial Add-on', 'WooCommerce Subscribe All the Things', self::REQ_WCSATT_VERSION).'</p></div>';
 		} // END wcsatt_stt_admin_notice()
 
 		/**
@@ -148,7 +148,7 @@ 
 		 */
 		public function init_plugin() {
 			// Load text domain.
-			load_plugin_textdomain( 'wc-satt-stt', false, dirname( plugin_basename( __FILE__ ) ) . '/languages/' );
+			load_plugin_textdomain('wc-satt-stt', false, dirname(plugin_basename(__FILE__)).'/languages/');
 		} // END init_plugin()
 
 		/**
@@ -159,11 +159,11 @@ 
 		 * @param  array $data  Plugin Data
 		 * @return array
 		 */
-		public function plugin_meta_links( $links, $file, $data, $status ) {
-			if ( $file == plugin_basename( __FILE__ ) ) {
-				$author1 = '<a href="' . $data[ 'AuthorURI' ] . '">' . $data[ 'Author' ] . '</a>';
+		public function plugin_meta_links($links, $file, $data, $status) {
+			if ($file == plugin_basename(__FILE__)) {
+				$author1 = '<a href="'.$data['AuthorURI'].'">'.$data['Author'].'</a>';
 				$author2 = '<a href="http://www.subscriptiongroup.co.uk/">Subscription Group Limited</a>';
-				$links[ 1 ] = sprintf( __( 'By %s', self::TEXT_DOMAIN ), sprintf( __( '%s and %s', self::TEXT_DOMAIN ), $author1, $author2 ) );
+				$links[1] = sprintf(__('By %s', self::TEXT_DOMAIN), sprintf(__('%s and %s', self::TEXT_DOMAIN), $author1, $author2));
 			}
 
 			return $links;