WordImpress / Give

includes/admin/forms/metabox.php

Indentation +2 added lines, -2 removed lines

@@ -75,7 +75,7 @@ 
 					'row_classes'  => 'give-subfield',

 					'render_row_cb' 	=> 'give_cmb_amount_field_render_row_cb',

 					'sanitization_cb'   => 'give_sanitize_price_field_value',

-                    'attributes'   => array(

+					'attributes'   => array(

 						'placeholder' => give_format_decimal( '1.00' ),

 						'value'       => give_format_decimal( $price ),

 						'class'       => 'cmb-type-text-small give-money-field',

@@ -110,7 +110,7 @@ 
 							'before_field'      => give_get_option( 'currency_position' ) == 'before' ? '<span class="give-money-symbol  give-money-symbol-before">' . give_currency_symbol() . '</span>' : '',

 							'after_field'       => give_get_option( 'currency_position' ) == 'after' ? '<span class="give-money-symbol  give-money-symbol-after">' . give_currency_symbol() . '</span>' : '',

 							'sanitization_cb'   => 'give_sanitize_price_field_value',

-                            'attributes'        => array(

+							'attributes'        => array(

 								'placeholder' => give_format_decimal( '1.00' ),

 								'class'       => 'cmb-type-text-small give-money-field',

 							),

Spacing +184 added lines, -184 removed lines

@@ -10,11 +10,11 @@ 
  */

 

 // Exit if accessed directly.

-if ( ! defined( 'ABSPATH' ) ) {

+if ( ! defined('ABSPATH')) {

 	exit;

 }

 

-add_action( 'cmb2_init', 'give_single_forms_cmb2_metaboxes', 6 );

+add_action('cmb2_init', 'give_single_forms_cmb2_metaboxes', 6);

 

 /**

  * Define the metabox and field configurations.

@@ -22,20 +22,20 @@ 
 function give_single_forms_cmb2_metaboxes() {

 

 	$post_id               = give_get_admin_post_id();

-	$price                 = give_get_form_price( $post_id );

-	$custom_amount_minimum = give_get_form_minimum_price( $post_id );

-	$goal                  = give_get_form_goal( $post_id );

-	$variable_pricing      = give_has_variable_prices( $post_id );

-	$prices                = give_get_variable_prices( $post_id );

+	$price                 = give_get_form_price($post_id);

+	$custom_amount_minimum = give_get_form_minimum_price($post_id);

+	$goal                  = give_get_form_goal($post_id);

+	$variable_pricing      = give_has_variable_prices($post_id);

+	$prices                = give_get_variable_prices($post_id);

 

 	//No empty prices - min. 1.00 for new forms

-	if ( empty( $price ) && is_null( $post_id ) ) {

-		$price = esc_attr( give_format_amount( '1.00' ) );

+	if (empty($price) && is_null($post_id)) {

+		$price = esc_attr(give_format_amount('1.00'));

 	}

 

 	//Min. $1.00 for new forms

-	if ( empty( $custom_amount_minimum ) ) {

-		$custom_amount_minimum = esc_attr( give_format_amount( '1.00' ) );

+	if (empty($custom_amount_minimum)) {

+		$custom_amount_minimum = esc_attr(give_format_amount('1.00'));

 	}

 

 	// Start with an underscore to hide fields from custom fields list

@@ -47,327 +47,327 @@ 
 	/**

 	 * Repeatable Field Groups

 	 */

-	$meta_boxes['form_field_options'] = apply_filters( 'give_forms_field_options', array(

+	$meta_boxes['form_field_options'] = apply_filters('give_forms_field_options', array(

 		'id'           => 'form_field_options',

-		'title'        => esc_html__( 'Donation Options', 'give' ),

-		'object_types' => array( 'give_forms' ),

+		'title'        => esc_html__('Donation Options', 'give'),

+		'object_types' => array('give_forms'),

 		'context'      => 'normal',

 		'priority'     => 'high', //Show above Content WYSIWYG

-		'fields'       => apply_filters( 'give_forms_donation_form_metabox_fields', array(

+		'fields'       => apply_filters('give_forms_donation_form_metabox_fields', array(

 				//Donation Option

 				array(

-					'name'        => esc_html__( 'Donation Option', 'give' ),

-					'description' => esc_html__( 'Would you like this form to have one set donation price or multiple levels (for example, $10, $20, $50)?', 'give' ),

-					'id'          => $prefix . 'price_option',

+					'name'        => esc_html__('Donation Option', 'give'),

+					'description' => esc_html__('Would you like this form to have one set donation price or multiple levels (for example, $10, $20, $50)?', 'give'),

+					'id'          => $prefix.'price_option',

 					'type'        => 'radio_inline',

 					'default'     => 'set',

-					'options'     => apply_filters( 'give_forms_price_options', array(

-						'set'   => esc_html__( 'Set Donation', 'give' ),

-						'multi' => esc_html__( 'Multi-level Donation', 'give' ),

-					) ),

+					'options'     => apply_filters('give_forms_price_options', array(

+						'set'   => esc_html__('Set Donation', 'give'),

+						'multi' => esc_html__('Multi-level Donation', 'give'),

+					)),

 				),

 				array(

-					'name'         => esc_html__( 'Set Donation', 'give' ),

-					'description'  => esc_html__( 'This is the set donation amount for this form. If you have a "Custom Amount Minimum" set, make sure it is less than this amount.', 'give' ),

-					'id'           => $prefix . 'set_price',

+					'name'         => esc_html__('Set Donation', 'give'),

+					'description'  => esc_html__('This is the set donation amount for this form. If you have a "Custom Amount Minimum" set, make sure it is less than this amount.', 'give'),

+					'id'           => $prefix.'set_price',

 					'type'         => 'text_small',

 					'row_classes'  => 'give-subfield',

 					'render_row_cb' 	=> 'give_cmb_amount_field_render_row_cb',

 					'sanitization_cb'   => 'give_sanitize_price_field_value',

                     'attributes'   => array(

-						'placeholder' => give_format_decimal( '1.00' ),

-						'value'       => give_format_decimal( $price ),

+						'placeholder' => give_format_decimal('1.00'),

+						'value'       => give_format_decimal($price),

 						'class'       => 'cmb-type-text-small give-money-field',

 					),

 				),

 				//Donation levels: Header

 				array(

-					'id'   => $prefix . 'levels_header',

+					'id'   => $prefix.'levels_header',

 					'type' => 'levels_repeater_header',

 				),

 				//Donation Levels: Repeatable CMB2 Group

 				array(

-					'id'          => $prefix . 'donation_levels',

+					'id'          => $prefix.'donation_levels',

 					'type'        => 'group',

 					'row_classes' => 'give-subfield',

 					'options'     => array(

-						'add_button'    => esc_html__( 'Add Level', 'give' ),

+						'add_button'    => esc_html__('Add Level', 'give'),

 						'remove_button' => '<span class="dashicons dashicons-no"></span>',

 						'sortable'      => true, // beta

 					),

 					// Fields array works the same, except id's only need to be unique for this group. Prefix is not needed.

-					'fields'      => apply_filters( 'give_donation_levels_table_row', array(

+					'fields'      => apply_filters('give_donation_levels_table_row', array(

 						array(

-							'name' => esc_html__( 'ID', 'give' ),

-							'id'   => $prefix . 'id',

+							'name' => esc_html__('ID', 'give'),

+							'id'   => $prefix.'id',

 							'type' => 'levels_id',

 						),

 						array(

-							'name'              => esc_html__( 'Amount', 'give' ),

-							'id'                => $prefix . 'amount',

+							'name'              => esc_html__('Amount', 'give'),

+							'id'                => $prefix.'amount',

 							'type'              => 'text_small',

-							'before_field'      => give_get_option( 'currency_position' ) == 'before' ? '<span class="give-money-symbol  give-money-symbol-before">' . give_currency_symbol() . '</span>' : '',

-							'after_field'       => give_get_option( 'currency_position' ) == 'after' ? '<span class="give-money-symbol  give-money-symbol-after">' . give_currency_symbol() . '</span>' : '',

+							'before_field'      => give_get_option('currency_position') == 'before' ? '<span class="give-money-symbol  give-money-symbol-before">'.give_currency_symbol().'</span>' : '',

+							'after_field'       => give_get_option('currency_position') == 'after' ? '<span class="give-money-symbol  give-money-symbol-after">'.give_currency_symbol().'</span>' : '',

 							'sanitization_cb'   => 'give_sanitize_price_field_value',

                             'attributes'        => array(

-								'placeholder' => give_format_decimal( '1.00' ),

+								'placeholder' => give_format_decimal('1.00'),

 								'class'       => 'cmb-type-text-small give-money-field',

 							),

 							'before'       => 'give_format_admin_multilevel_amount',

 						),

 						array(

-							'name'       => esc_html__( 'Text', 'give' ),

-							'id'         => $prefix . 'text',

+							'name'       => esc_html__('Text', 'give'),

+							'id'         => $prefix.'text',

 							'type'       => 'text',

 							'attributes' => array(

-								'placeholder' => esc_html__( 'Donation Level', 'give' ),

+								'placeholder' => esc_html__('Donation Level', 'give'),

 								'class'       => 'give-multilevel-text-field',

 							),

 						),

 						array(

-							'name' => esc_html__( 'Default', 'give' ),

-							'id'   => $prefix . 'default',

+							'name' => esc_html__('Default', 'give'),

+							'id'   => $prefix.'default',

 							'type' => 'give_default_radio_inline'

 						),

-					) ),

+					)),

 				),

 				//Display Style

 				array(

-					'name'        => esc_html__( 'Display Style', 'give' ),

-					'description' => esc_html__( 'Set how the donations levels will display on the form.', 'give' ),

-					'id'          => $prefix . 'display_style',

+					'name'        => esc_html__('Display Style', 'give'),

+					'description' => esc_html__('Set how the donations levels will display on the form.', 'give'),

+					'id'          => $prefix.'display_style',

 					'type'        => 'radio_inline',

 					'default'     => 'buttons',

 					'options'     => array(

-						'buttons'  => esc_html__( 'Buttons', 'give' ),

-						'radios'   => esc_html__( 'Radios', 'give' ),

-						'dropdown' => esc_html__( 'Dropdown', 'give' ),

+						'buttons'  => esc_html__('Buttons', 'give'),

+						'radios'   => esc_html__('Radios', 'give'),

+						'dropdown' => esc_html__('Dropdown', 'give'),

 					),

 				),

 				//Custom Amount

 				array(

-					'name'        => esc_html__( 'Custom Amount', 'give' ),

-					'description' => esc_html__( 'Do you want the user to be able to input their own donation amount?', 'give' ),

-					'id'          => $prefix . 'custom_amount',

+					'name'        => esc_html__('Custom Amount', 'give'),

+					'description' => esc_html__('Do you want the user to be able to input their own donation amount?', 'give'),

+					'id'          => $prefix.'custom_amount',

 					'type'        => 'radio_inline',

 					'default'     => 'no',

 					'options'     => array(

-						'yes' => esc_html__( 'Yes', 'give' ),

-						'no'  => esc_html__( 'No', 'give' ),

+						'yes' => esc_html__('Yes', 'give'),

+						'no'  => esc_html__('No', 'give'),

 					),

 				),

 				array(

-					'name'              => esc_html__( 'Custom Amount Minimum', 'give' ),

-					'description'       => esc_html__( 'If you would like to set a minimum custom donation amount please enter it here.', 'give' ),

-					'id'                => $prefix . 'custom_amount_minimum',

+					'name'              => esc_html__('Custom Amount Minimum', 'give'),

+					'description'       => esc_html__('If you would like to set a minimum custom donation amount please enter it here.', 'give'),

+					'id'                => $prefix.'custom_amount_minimum',

 					'type'              => 'text_small',

 					'row_classes'       => 'give-subfield',

 					'render_row_cb'     => 'give_cmb_amount_field_render_row_cb',

 					'sanitization_cb'   => 'give_sanitize_price_field_value',

 					'attributes'   => array(

 						'placeholder' => give_format_decimal('1.00'),

-						'value'       => give_format_decimal( $custom_amount_minimum ),

+						'value'       => give_format_decimal($custom_amount_minimum),

 						'class'       => 'cmb-type-text-small give-money-field',

 					)

 				),

 				array(

-					'name'        => esc_html__( 'Custom Amount Text', 'give' ),

-					'description' => esc_html__( 'This text appears as a label below the custom amount field for set donation forms. For multi-level forms the text will appear as it\'s own level (ie button, radio, or select option).', 'give' ),

-					'id'          => $prefix . 'custom_amount_text',

+					'name'        => esc_html__('Custom Amount Text', 'give'),

+					'description' => esc_html__('This text appears as a label below the custom amount field for set donation forms. For multi-level forms the text will appear as it\'s own level (ie button, radio, or select option).', 'give'),

+					'id'          => $prefix.'custom_amount_text',

 					'type'        => 'text',

 					'row_classes' => 'give-subfield',

 					'attributes'  => array(

 						'rows'        => 3,

-						'placeholder' => esc_attr__( 'Give a Custom Amount', 'give' ),

+						'placeholder' => esc_attr__('Give a Custom Amount', 'give'),

 					),

 				),

 				//Goals

 				array(

-					'name'        => esc_html__( 'Goal', 'give' ),

-					'description' => esc_html__( 'Do you want to set a donation goal for this form?', 'give' ),

-					'id'          => $prefix . 'goal_option',

+					'name'        => esc_html__('Goal', 'give'),

+					'description' => esc_html__('Do you want to set a donation goal for this form?', 'give'),

+					'id'          => $prefix.'goal_option',

 					'type'        => 'radio_inline',

 					'default'     => 'no',

 					'options'     => array(

-						'yes' => esc_html__( 'Yes', 'give' ),

-						'no'  => esc_html__( 'No', 'give' ),

+						'yes' => esc_html__('Yes', 'give'),

+						'no'  => esc_html__('No', 'give'),

 					),

 				),

 				array(

-					'name'              => esc_html__( 'Goal Amount', 'give' ),

-					'description'       => esc_html__( 'This is the monetary goal amount you want to reach for this donation form.', 'give' ),

-					'id'                => $prefix . 'set_goal',

+					'name'              => esc_html__('Goal Amount', 'give'),

+					'description'       => esc_html__('This is the monetary goal amount you want to reach for this donation form.', 'give'),

+					'id'                => $prefix.'set_goal',

 					'type'              => 'text_small',

 					'row_classes'       => 'give-subfield',

 					'render_row_cb' 	=> 'give_cmb_amount_field_render_row_cb',

 					'sanitization_cb'   => 'give_sanitize_price_field_value',

 					'attributes'        => array(

-						'placeholder' => give_format_decimal( '0.00' ),

-						'value'       => give_format_decimal( $goal ),

+						'placeholder' => give_format_decimal('0.00'),

+						'value'       => give_format_decimal($goal),

 						'class'       => 'cmb-type-text-small give-money-field',

 					),

 				),

 

 				array(

-					'name'        => esc_html__( 'Goal Format', 'give' ),

-					'description' => esc_html__( 'Would you like to display the total amount raised based on your monetary goal or a percentage? For instance, "$500 of $1,000 raised" or "50% funded".', 'give' ),

-					'id'          => $prefix . 'goal_format',

+					'name'        => esc_html__('Goal Format', 'give'),

+					'description' => esc_html__('Would you like to display the total amount raised based on your monetary goal or a percentage? For instance, "$500 of $1,000 raised" or "50% funded".', 'give'),

+					'id'          => $prefix.'goal_format',

 					'type'        => 'radio_inline',

 					'default'     => 'amount',

 					'row_classes' => 'give-subfield',

 					'options'     => array(

-						'amount'     => esc_html__( 'Amount', 'give' ),

-						'percentage' => esc_html__( 'Percentage', 'give' ),

+						'amount'     => esc_html__('Amount', 'give'),

+						'percentage' => esc_html__('Percentage', 'give'),

 					),

 				),

 				array(

-					'name'        => esc_html__( 'Goal Progress Bar Color', 'give' ),

-					'id'          => $prefix . 'goal_color',

+					'name'        => esc_html__('Goal Progress Bar Color', 'give'),

+					'id'          => $prefix.'goal_color',

 					'type'        => 'colorpicker',

 					'row_classes' => 'give-subfield',

 					'default'     => '#2bc253',

 				),

 

 				array(

-					'name'        => esc_html__( 'Close Form when Goal Achieved', 'give' ),

-					'desc'        => esc_html__( 'Would you like to close the donation forms and stop accepting donations once this goal has been met?', 'give' ),

-					'id'          => $prefix . 'close_form_when_goal_achieved',

+					'name'        => esc_html__('Close Form when Goal Achieved', 'give'),

+					'desc'        => esc_html__('Would you like to close the donation forms and stop accepting donations once this goal has been met?', 'give'),

+					'id'          => $prefix.'close_form_when_goal_achieved',

 					'type'        => 'radio_inline',

 					'row_classes' => 'give-subfield',

 					'options'     => array(

-						'yes' => esc_html__( 'Yes', 'give' ),

-						'no'  => esc_html__( 'No', 'give' ),

+						'yes' => esc_html__('Yes', 'give'),

+						'no'  => esc_html__('No', 'give'),

 					),

 					'default'     => 'no',

 				),

 				array(

-					'name'        => esc_html__( 'Goal Achieved Message', 'give' ),

-					'desc'        => esc_html__( 'Would you like to display a custom message when the goal is closed? If none is provided the default message will be displayed', 'give' ),

-					'id'          => $prefix . 'form_goal_achieved_message',

+					'name'        => esc_html__('Goal Achieved Message', 'give'),

+					'desc'        => esc_html__('Would you like to display a custom message when the goal is closed? If none is provided the default message will be displayed', 'give'),

+					'id'          => $prefix.'form_goal_achieved_message',

 					'type'        => 'textarea',

 					'row_classes' => 'give-subfield',

 					'attributes'  => array(

-						'placeholder' => esc_attr__( 'Thank you to all our donors, we have met our fundraising goal.', 'give' ),

+						'placeholder' => esc_attr__('Thank you to all our donors, we have met our fundraising goal.', 'give'),

 					),

 				)

 			)

 		)

-	) );

+	));

 

 

 	/**

 	 * Content Field

 	 */

-	$meta_boxes['form_content_options'] = apply_filters( 'give_forms_content_options', array(

+	$meta_boxes['form_content_options'] = apply_filters('give_forms_content_options', array(

 		'id'           => 'form_content_options',

-		'title'        => esc_html__( 'Form Content', 'give' ),

-		'object_types' => array( 'give_forms' ),

+		'title'        => esc_html__('Form Content', 'give'),

+		'object_types' => array('give_forms'),

 		'context'      => 'normal',

 		'priority'     => 'high', //Show above Content WYSIWYG

-		'fields'       => apply_filters( 'give_forms_content_options_metabox_fields', array(

+		'fields'       => apply_filters('give_forms_content_options_metabox_fields', array(

 				//Donation Option

 				array(

-					'name'        => esc_html__( 'Display Content', 'give' ),

-					'description' => esc_html__( 'Do you want to display content? If you choose to display content, a WYSIWYG editor will appear which you will be able to enter content.', 'give' ),

-					'id'          => $prefix . 'content_option',

+					'name'        => esc_html__('Display Content', 'give'),

+					'description' => esc_html__('Do you want to display content? If you choose to display content, a WYSIWYG editor will appear which you will be able to enter content.', 'give'),

+					'id'          => $prefix.'content_option',

 					'type'        => 'select',

-					'options'     => apply_filters( 'give_forms_content_options_select', array(

-							'none'           => esc_html__( 'No Content', 'give' ),

-							'give_pre_form'  => esc_html__( 'Display content ABOVE the fields', 'give' ),

-							'give_post_form' => esc_html__( 'Display content BELOW the fields', 'give' ),

+					'options'     => apply_filters('give_forms_content_options_select', array(

+							'none'           => esc_html__('No Content', 'give'),

+							'give_pre_form'  => esc_html__('Display content ABOVE the fields', 'give'),

+							'give_post_form' => esc_html__('Display content BELOW the fields', 'give'),

 						)

 					),

 					'default'     => 'none',

 				),

 				array(

-					'name'        => esc_html__( 'Content', 'give' ),

-					'description' => esc_html__( 'This content will display on the single give form page.', 'give' ),

-					'id'          => $prefix . 'form_content',

+					'name'        => esc_html__('Content', 'give'),

+					'description' => esc_html__('This content will display on the single give form page.', 'give'),

+					'id'          => $prefix.'form_content',

 					'row_classes' => 'give-subfield',

 					'type'        => 'wysiwyg'

 				),

 			)

 		)

-	) );

+	));

 

 

 	/**

 	 * Display Options

 	 */

-	$meta_boxes['form_display_options'] = apply_filters( 'give_form_display_options', array(

+	$meta_boxes['form_display_options'] = apply_filters('give_form_display_options', array(

 			'id'           => 'form_display_options',

-			'title'        => esc_html__( 'Form Display Options', 'give' ),

-			'object_types' => array( 'give_forms' ),

+			'title'        => esc_html__('Form Display Options', 'give'),

+			'object_types' => array('give_forms'),

 			'context'      => 'normal', //  'normal', 'advanced', or 'side'

 			'priority'     => 'high', //Show above Content WYSIWYG

 			'show_names'   => true, // Show field names on the left

-			'fields'       => apply_filters( 'give_forms_display_options_metabox_fields', array(

+			'fields'       => apply_filters('give_forms_display_options_metabox_fields', array(

 					array(

-						'name'    => esc_html__( 'Payment Fields', 'give' ),

-						'desc'    => esc_html__( 'How would you like to display donation information for this form? The "Show on Page" option will display the entire form when the page loads. "Reveal Upon Click" places a button below the donation fields and upon click slides into view the rest of the fields. "Modal Window Upon Click" is a similar option, rather than sliding into view the fields they will open in a shadow box or "modal" window.', 'give' ),

-						'id'      => $prefix . 'payment_display',

+						'name'    => esc_html__('Payment Fields', 'give'),

+						'desc'    => esc_html__('How would you like to display donation information for this form? The "Show on Page" option will display the entire form when the page loads. "Reveal Upon Click" places a button below the donation fields and upon click slides into view the rest of the fields. "Modal Window Upon Click" is a similar option, rather than sliding into view the fields they will open in a shadow box or "modal" window.', 'give'),

+						'id'      => $prefix.'payment_display',

 						'type'    => 'select',

 						'options' => array(

-							'onpage' => esc_html__( 'Show on Page', 'give' ),

-							'reveal' => esc_html__( 'Reveal Upon Click', 'give' ),

-							'modal'  => esc_html__( 'Modal Window Upon Click', 'give' ),

+							'onpage' => esc_html__('Show on Page', 'give'),

+							'reveal' => esc_html__('Reveal Upon Click', 'give'),

+							'modal'  => esc_html__('Modal Window Upon Click', 'give'),

 						),

 						'default' => 'onpage',

 					),

 					array(

-						'id'          => $prefix . 'reveal_label',

-						'name'        => esc_html__( 'Reveal / Modal Open Text', 'give' ),

-						'desc'        => esc_html__( 'The button label for completing the donation.', 'give' ),

+						'id'          => $prefix.'reveal_label',

+						'name'        => esc_html__('Reveal / Modal Open Text', 'give'),

+						'desc'        => esc_html__('The button label for completing the donation.', 'give'),

 						'type'        => 'text_small',

 						'row_classes' => 'give-subfield',

 						'attributes'  => array(

-							'placeholder' => esc_attr__( 'Donate Now', 'give' ),

+							'placeholder' => esc_attr__('Donate Now', 'give'),

 						),

 					),

 					array(

-						'id'         => $prefix . 'checkout_label',

-						'name'       => esc_html__( 'Complete Donation Text', 'give' ),

-						'desc'       => esc_html__( 'The button label for completing the donation.', 'give' ),

+						'id'         => $prefix.'checkout_label',

+						'name'       => esc_html__('Complete Donation Text', 'give'),

+						'desc'       => esc_html__('The button label for completing the donation.', 'give'),

 						'type'       => 'text_small',

 						'attributes' => array(

-							'placeholder' => esc_html__( 'Donate Now', 'give' ),

+							'placeholder' => esc_html__('Donate Now', 'give'),

 						),

 					),

 					array(

-						'name' => esc_html__( 'Default Gateway', 'give' ),

-						'desc' => esc_html__( 'By default, the gateway for this form will inherit the global default gateway (set under Give > Settings > Payment Gateways). This option allows you to customize the default gateway for this form only.', 'give' ),

-						'id'   => $prefix . 'default_gateway',

+						'name' => esc_html__('Default Gateway', 'give'),

+						'desc' => esc_html__('By default, the gateway for this form will inherit the global default gateway (set under Give > Settings > Payment Gateways). This option allows you to customize the default gateway for this form only.', 'give'),

+						'id'   => $prefix.'default_gateway',

 						'type' => 'default_gateway'

 					),

 					array(

-						'name' => esc_html__( 'Disable Guest Donations', 'give' ),

-						'desc' => esc_html__( 'Do you want to require users be logged-in to make donations?', 'give' ),

-						'id'   => $prefix . 'logged_in_only',

+						'name' => esc_html__('Disable Guest Donations', 'give'),

+						'desc' => esc_html__('Do you want to require users be logged-in to make donations?', 'give'),

+						'id'   => $prefix.'logged_in_only',

 						'type' => 'checkbox'

 					),

 					array(

-						'name'    => esc_html__( 'Register / Login Form', 'give' ),

-						'desc'    => esc_html__( 'Display the registration and login forms in the payment section for non-logged-in users.', 'give' ),

-						'id'      => $prefix . 'show_register_form',

+						'name'    => esc_html__('Register / Login Form', 'give'),

+						'desc'    => esc_html__('Display the registration and login forms in the payment section for non-logged-in users.', 'give'),

+						'id'      => $prefix.'show_register_form',

 						'type'    => 'select',

 						'options' => array(

-							'both'         => esc_html__( 'Registration and Login Forms', 'give' ),

-							'registration' => esc_html__( 'Registration Form Only', 'give' ),

-							'login'        => esc_html__( 'Login Form Only', 'give' ),

-							'none'         => esc_html__( 'None', 'give' ),

+							'both'         => esc_html__('Registration and Login Forms', 'give'),

+							'registration' => esc_html__('Registration Form Only', 'give'),

+							'login'        => esc_html__('Login Form Only', 'give'),

+							'none'         => esc_html__('None', 'give'),

 						),

 						'default' => 'none',

 					),

 					array(

-						'name'    => esc_html__( 'Floating Labels', 'give' ),

+						'name'    => esc_html__('Floating Labels', 'give'),

 						/* translators: %s: forms https://givewp.com/documentation/core/give-forms/creating-give-forms/#floating-labels */

-						'desc'    => sprintf( __( 'Select the <a href="%s" target="_blank">floating labels</a> setting for this Give form.<br>Be aware that if you have the "Disable CSS" option enabled, you will need to style the floating labels yourself.', 'give' ), esc_url( 'https://givewp.com/documentation/core/give-forms/creating-give-forms/#floating-labels' ) ),

-						'id'      => $prefix . 'form_floating_labels',

+						'desc'    => sprintf(__('Select the <a href="%s" target="_blank">floating labels</a> setting for this Give form.<br>Be aware that if you have the "Disable CSS" option enabled, you will need to style the floating labels yourself.', 'give'), esc_url('https://givewp.com/documentation/core/give-forms/creating-give-forms/#floating-labels')),

+						'id'      => $prefix.'form_floating_labels',

 						'type'    => 'select',

 						'options' => array(

-							''         => esc_html__( 'Use the global setting', 'give' ),

-							'enabled'  => esc_html__( 'Enabled', 'give' ),

-							'disabled' => esc_html__( 'Disabled', 'give' ),

+							''         => esc_html__('Use the global setting', 'give'),

+							'enabled'  => esc_html__('Enabled', 'give'),

+							'disabled' => esc_html__('Disabled', 'give'),

 						),

 						'default' => 'none',

 					)

@@ -379,50 +379,50 @@ 
 	/**

 	 * Terms & Conditions

 	 */

-	$meta_boxes['form_terms_options'] = apply_filters( 'give_forms_terms_options', array(

+	$meta_boxes['form_terms_options'] = apply_filters('give_forms_terms_options', array(

 		'id'           => 'form_terms_options',

-		'title'        => esc_html__( 'Terms and Conditions', 'give' ),

-		'object_types' => array( 'give_forms' ),

+		'title'        => esc_html__('Terms and Conditions', 'give'),

+		'object_types' => array('give_forms'),

 		'context'      => 'normal',

 		'priority'     => 'high', //Show above Content WYSIWYG

-		'fields'       => apply_filters( 'give_forms_terms_options_metabox_fields', array(

+		'fields'       => apply_filters('give_forms_terms_options_metabox_fields', array(

 				//Donation Option

 				array(

-					'name'        => esc_html__( 'Terms and Conditions', 'give' ),

-					'description' => esc_html__( 'Do you want to require the user to agree to terms and conditions prior to being able to complete their donation?', 'give' ),

-					'id'          => $prefix . 'terms_option',

+					'name'        => esc_html__('Terms and Conditions', 'give'),

+					'description' => esc_html__('Do you want to require the user to agree to terms and conditions prior to being able to complete their donation?', 'give'),

+					'id'          => $prefix.'terms_option',

 					'type'        => 'select',

-					'options'     => apply_filters( 'give_forms_content_options_select', array(

-							'none' => esc_html__( 'No', 'give' ),

-							'yes'  => esc_html__( 'Yes', 'give' ),

+					'options'     => apply_filters('give_forms_content_options_select', array(

+							'none' => esc_html__('No', 'give'),

+							'yes'  => esc_html__('Yes', 'give'),

 						)

 					),

 					'default'     => 'none',

 				),

 				array(

-					'id'          => $prefix . 'agree_label',

-					'name'        => esc_html__( 'Agree to Terms Label', 'give' ),

-					'desc'        => esc_html__( 'The label shown next to the agree to terms check box. Add your own to customize or leave blank to use the default text placeholder.', 'give' ),

+					'id'          => $prefix.'agree_label',

+					'name'        => esc_html__('Agree to Terms Label', 'give'),

+					'desc'        => esc_html__('The label shown next to the agree to terms check box. Add your own to customize or leave blank to use the default text placeholder.', 'give'),

 					'type'        => 'text',

 					'row_classes' => 'give-subfield',

 					'size'        => 'regular',

 					'attributes'  => array(

-						'placeholder' => esc_attr__( 'Agree to Terms?', 'give' ),

+						'placeholder' => esc_attr__('Agree to Terms?', 'give'),

 					),

 				),

 				array(

-					'id'          => $prefix . 'agree_text',

+					'id'          => $prefix.'agree_text',

 					'row_classes' => 'give-subfield',

-					'name'        => esc_html__( 'Agreement Text', 'give' ),

-					'desc'        => esc_html__( 'This is the actual text which the user will have to agree to in order to make a donation.', 'give' ),

+					'name'        => esc_html__('Agreement Text', 'give'),

+					'desc'        => esc_html__('This is the actual text which the user will have to agree to in order to make a donation.', 'give'),

 					'type'        => 'wysiwyg'

 				),

 			)

 		)

-	) );

+	));

 

-	foreach ( $meta_boxes as $box ) {

-		$cmb = new_cmb2_box( $box );

+	foreach ($meta_boxes as $box) {

+		$cmb = new_cmb2_box($box);

 	}

 

 }

@@ -435,9 +435,9 @@ 
 

 	<div class="table-container">

 		<div class="table-row">

-			<div class="table-cell col-amount"><?php esc_html_e( 'Amount', 'give' ); ?></div>

-			<div class="table-cell col-text"><?php esc_html_e( 'Text', 'give' ); ?></div>

-			<div class="table-cell col-default"><?php esc_html_e( 'Default', 'give' ); ?></div>

+			<div class="table-cell col-amount"><?php esc_html_e('Amount', 'give'); ?></div>

+			<div class="table-cell col-text"><?php esc_html_e('Text', 'give'); ?></div>

+			<div class="table-cell col-default"><?php esc_html_e('Default', 'give'); ?></div>

 			<?php

 			/**

 			 * Fires in repeatable donation levels table head.

@@ -449,9 +449,9 @@ 
 			 *

 			 * @since 1.0

 			 */

-			do_action( 'give_donation_levels_table_head' );

+			do_action('give_donation_levels_table_head');

 			?>

-			<div class="table-cell col-sort"><?php esc_html_e( 'Sort', 'give' ); ?></div>

+			<div class="table-cell col-sort"><?php esc_html_e('Sort', 'give'); ?></div>

 

 		</div>

 	</div>

@@ -459,7 +459,7 @@ 
 	<?php

 }

 

-add_action( 'cmb2_render_levels_repeater_header', 'give_cmb_render_levels_repeater_header', 10 );

+add_action('cmb2_render_levels_repeater_header', 'give_cmb_render_levels_repeater_header', 10);

 

 

 /**

@@ -476,25 +476,25 @@ 
  * @param $object_type

  * @param $field_type_object

  */

-function give_cmb_render_levels_id( $field_object, $escaped_value, $object_id, $object_type, $field_type_object ) {

+function give_cmb_render_levels_id($field_object, $escaped_value, $object_id, $object_type, $field_type_object) {

 

-	$escaped_value = ( isset( $escaped_value['level_id'] ) ? $escaped_value['level_id'] : '' );

+	$escaped_value = (isset($escaped_value['level_id']) ? $escaped_value['level_id'] : '');

 

 	$field_options_array = array(

 		'class' => 'give-hidden give-level-id-input',

-		'name'  => $field_type_object->_name( '[level_id]' ),

-		'id'    => $field_type_object->_id( '_level_id' ),

+		'name'  => $field_type_object->_name('[level_id]'),

+		'id'    => $field_type_object->_id('_level_id'),

 		'value' => $escaped_value,

 		'type'  => 'number',

 		'desc'  => '',

 	);

 

-	echo '<p class="give-level-id">' . $escaped_value . '</p>';

-	echo $field_type_object->input( $field_options_array );

+	echo '<p class="give-level-id">'.$escaped_value.'</p>';

+	echo $field_type_object->input($field_options_array);

 

 }

 

-add_action( 'cmb2_render_levels_id', 'give_cmb_render_levels_id', 10, 5 );

+add_action('cmb2_render_levels_id', 'give_cmb_render_levels_id', 10, 5);

 

 

 /**

@@ -506,13 +506,13 @@ 
  * @param $object_type

  * @param $field_type_object

  */

-function give_cmb_give_default_radio_inline( $field_object, $escaped_value, $object_id, $object_type, $field_type_object ) {

-	echo '<input type="radio" class="cmb2-option donation-level-radio" name="' . $field_object->args['_name'] . '" id="' . $field_object->args['id'] . '" value="default" ' . checked( 'default', $escaped_value, false ) . '>';

-	echo '<label for="' . $field_object->args['id'] . '">' . esc_html__( 'Default', 'give' ) . '</label>';

+function give_cmb_give_default_radio_inline($field_object, $escaped_value, $object_id, $object_type, $field_type_object) {

+	echo '<input type="radio" class="cmb2-option donation-level-radio" name="'.$field_object->args['_name'].'" id="'.$field_object->args['id'].'" value="default" '.checked('default', $escaped_value, false).'>';

+	echo '<label for="'.$field_object->args['id'].'">'.esc_html__('Default', 'give').'</label>';

 

 }

 

-add_action( 'cmb2_render_give_default_radio_inline', 'give_cmb_give_default_radio_inline', 10, 5 );

+add_action('cmb2_render_give_default_radio_inline', 'give_cmb_give_default_radio_inline', 10, 5);

 

 

 /**

@@ -522,20 +522,20 @@ 
  */

 function give_add_shortcode_to_publish_metabox() {

 

-	if ( 'give_forms' !== get_post_type() ) {

+	if ('give_forms' !== get_post_type()) {

 		return false;

 	}

 

 	global $post;

 

 	//Only enqueue scripts for CPT on post type screen

-	if ( 'give_forms' === $post->post_type ) {

+	if ('give_forms' === $post->post_type) {

 		//Shortcode column with select all input

-		$shortcode = htmlentities( '[give_form id="' . $post->ID . '"]' );

-		echo '<div class="shortcode-wrap box-sizing"><label for="shortcode-input">' . esc_html__( 'Give Form Shortcode:', 'give' ) . '</label><input onClick="this.setSelectionRange(0, this.value.length)" type="text" name="shortcode-input" id="shortcode-input" class="shortcode-input" readonly value="' . $shortcode . '"></div>';

+		$shortcode = htmlentities('[give_form id="'.$post->ID.'"]');

+		echo '<div class="shortcode-wrap box-sizing"><label for="shortcode-input">'.esc_html__('Give Form Shortcode:', 'give').'</label><input onClick="this.setSelectionRange(0, this.value.length)" type="text" name="shortcode-input" id="shortcode-input" class="shortcode-input" readonly value="'.$shortcode.'"></div>';

 

 	}

 

 }

 

-add_action( 'post_submitbox_misc_actions', 'give_add_shortcode_to_publish_metabox' );

+add_action('post_submitbox_misc_actions', 'give_add_shortcode_to_publish_metabox');

includes/emails/class-give-emails.php

Spacing +58 added lines, -58 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -79,12 +79,12 @@ 
 	 */
 	public function __construct() {
 
-		if ( 'none' === $this->get_template() ) {
+		if ('none' === $this->get_template()) {
 			$this->html = false;
 		}
 
-		add_action( 'give_email_send_before', array( $this, 'send_before' ) );
-		add_action( 'give_email_send_after', array( $this, 'send_after' ) );
+		add_action('give_email_send_before', array($this, 'send_before'));
+		add_action('give_email_send_after', array($this, 'send_after'));
 
 	}
 
@@ -93,7 +93,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function __set( $key, $value ) {
+	public function __set($key, $value) {
 		$this->$key = $value;
 	}
 
@@ -103,11 +103,11 @@ 
 	 * @since 1.0
 	 */
 	public function get_from_name() {
-		if ( ! $this->from_name ) {
-			$this->from_name = give_get_option( 'from_name', get_bloginfo( 'name' ) );
+		if ( ! $this->from_name) {
+			$this->from_name = give_get_option('from_name', get_bloginfo('name'));
 		}
 
-		return apply_filters( 'give_email_from_name', wp_specialchars_decode( $this->from_name ), $this );
+		return apply_filters('give_email_from_name', wp_specialchars_decode($this->from_name), $this);
 	}
 
 	/**
@@ -116,11 +116,11 @@ 
 	 * @since 1.0
 	 */
 	public function get_from_address() {
-		if ( ! $this->from_address ) {
-			$this->from_address = give_get_option( 'from_email', get_option( 'admin_email' ) );
+		if ( ! $this->from_address) {
+			$this->from_address = give_get_option('from_email', get_option('admin_email'));
 		}
 
-		return apply_filters( 'give_email_from_address', $this->from_address, $this );
+		return apply_filters('give_email_from_address', $this->from_address, $this);
 	}
 
 	/**
@@ -129,13 +129,13 @@ 
 	 * @since 1.0
 	 */
 	public function get_content_type() {
-		if ( ! $this->content_type && $this->html ) {
-			$this->content_type = apply_filters( 'give_email_default_content_type', 'text/html', $this );
-		} else if ( ! $this->html ) {
+		if ( ! $this->content_type && $this->html) {
+			$this->content_type = apply_filters('give_email_default_content_type', 'text/html', $this);
+		} else if ( ! $this->html) {
 			$this->content_type = 'text/plain';
 		}
 
-		return apply_filters( 'give_email_content_type', $this->content_type, $this );
+		return apply_filters('give_email_content_type', $this->content_type, $this);
 	}
 
 	/**
@@ -144,13 +144,13 @@ 
 	 * @since 1.0
 	 */
 	public function get_headers() {
-		if ( ! $this->headers ) {
+		if ( ! $this->headers) {
 			$this->headers = "From: {$this->get_from_name()} <{$this->get_from_address()}>\r\n";
 			$this->headers .= "Reply-To: {$this->get_from_address()}\r\n";
 			$this->headers .= "Content-Type: {$this->get_content_type()}; charset=utf-8\r\n";
 		}
 
-		return apply_filters( 'give_email_headers', $this->headers, $this );
+		return apply_filters('give_email_headers', $this->headers, $this);
 	}
 
 	/**
@@ -160,11 +160,11 @@ 
 	 */
 	public function get_templates() {
 		$templates = array(
-			'default' => esc_html__( 'Default Template', 'give' ),
-			'none'    => esc_html__( 'No template, plain text only', 'give' )
+			'default' => esc_html__('Default Template', 'give'),
+			'none'    => esc_html__('No template, plain text only', 'give')
 		);
 
-		return apply_filters( 'give_email_templates', $templates );
+		return apply_filters('give_email_templates', $templates);
 	}
 
 	/**
@@ -173,11 +173,11 @@ 
 	 * @since 1.0
 	 */
 	public function get_template() {
-		if ( ! $this->template ) {
-			$this->template = give_get_option( 'email_template', 'default' );
+		if ( ! $this->template) {
+			$this->template = give_get_option('email_template', 'default');
 		}
 
-		return apply_filters( 'give_email_template', $this->template );
+		return apply_filters('give_email_template', $this->template);
 	}
 
 	/**
@@ -186,7 +186,7 @@ 
 	 * @since 1.0
 	 */
 	public function get_heading() {
-		return apply_filters( 'give_email_heading', $this->heading );
+		return apply_filters('give_email_heading', $this->heading);
 	}
 
 	/**
@@ -196,7 +196,7 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function parse_tags( $content ) {
+	public function parse_tags($content) {
 		return $content;
 	}
 
@@ -205,19 +205,19 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function build_email( $message ) {
+	public function build_email($message) {
 
-		if ( false === $this->html ) {
-			return apply_filters( 'give_email_message', wp_strip_all_tags( $message ), $this );
+		if (false === $this->html) {
+			return apply_filters('give_email_message', wp_strip_all_tags($message), $this);
 		}
 
-		$message = $this->text_to_html( $message );
+		$message = $this->text_to_html($message);
 
 		$template = $this->get_template();
 
 		ob_start();
 
-		give_get_template_part( 'emails/header', $template, true );
+		give_get_template_part('emails/header', $template, true);
 
 		/**
 		 * Fires in the email head.
@@ -226,17 +226,17 @@ 
 		 *
 		 * @param Give_Emails $this The email object.
 		 */
-		do_action( 'give_email_header', $this );
+		do_action('give_email_header', $this);
 
-		if ( has_action( 'give_email_template_' . $template ) ) {
+		if (has_action('give_email_template_'.$template)) {
 			/**
 			 * Fires in a specific email template.
 			 *
 			 * @since 1.0
 			 */
-			do_action( "give_email_template_{$template}" );
+			do_action("give_email_template_{$template}");
 		} else {
-			give_get_template_part( 'emails/body', $template, true );
+			give_get_template_part('emails/body', $template, true);
 		}
 
 		/**
@@ -246,9 +246,9 @@ 
 		 *
 		 * @param Give_Emails $this The email object.
 		 */
-		do_action( 'give_email_body', $this );
+		do_action('give_email_body', $this);
 
-		give_get_template_part( 'emails/footer', $template, true );
+		give_get_template_part('emails/footer', $template, true);
 
 		/**
 		 * Fires in the email footer.
@@ -257,12 +257,12 @@ 
 		 *
 		 * @param Give_Emails $this The email object.
 		 */
-		do_action( 'give_email_footer', $this );
+		do_action('give_email_footer', $this);
 
 		$body    = ob_get_clean();
-		$message = str_replace( '{email}', $message, $body );
+		$message = str_replace('{email}', $message, $body);
 
-		return apply_filters( 'give_email_message', $message, $this );
+		return apply_filters('give_email_message', $message, $this);
 	}
 
 	/**
@@ -275,10 +275,10 @@ 
 	 *
 	 * @return bool
 	 */
-	public function send( $to, $subject, $message, $attachments = '' ) {
+	public function send($to, $subject, $message, $attachments = '') {
 
-		if ( ! did_action( 'init' ) && ! did_action( 'admin_init' ) ) {
-			_doing_it_wrong( __FUNCTION__, esc_html__( 'You cannot send email with Give_Emails until init/admin_init has been reached.', 'give' ), null );
+		if ( ! did_action('init') && ! did_action('admin_init')) {
+			_doing_it_wrong(__FUNCTION__, esc_html__('You cannot send email with Give_Emails until init/admin_init has been reached.', 'give'), null);
 
 			return false;
 		}
@@ -290,16 +290,16 @@ 
 		 *
 		 * @param Give_Emails $this The email object.
 		 */
-		do_action( 'give_email_send_before', $this );
+		do_action('give_email_send_before', $this);
 
-		$subject = $this->parse_tags( $subject );
-		$message = $this->parse_tags( $message );
+		$subject = $this->parse_tags($subject);
+		$message = $this->parse_tags($message);
 
-		$message = $this->build_email( $message );
+		$message = $this->build_email($message);
 
-		$attachments = apply_filters( 'give_email_attachments', $attachments, $this );
+		$attachments = apply_filters('give_email_attachments', $attachments, $this);
 
-		$sent = wp_mail( $to, $subject, $message, $this->get_headers(), $attachments );
+		$sent = wp_mail($to, $subject, $message, $this->get_headers(), $attachments);
 
 		/**
 		 * Fires after sending an email.
@@ -308,7 +308,7 @@ 
 		 *
 		 * @param Give_Emails $this The email object.
 		 */
-		do_action( 'give_email_send_after', $this );
+		do_action('give_email_send_after', $this);
 
 		return $sent;
 
@@ -320,9 +320,9 @@ 
 	 * @since 1.0
 	 */
 	public function send_before() {
-		add_filter( 'wp_mail_from', array( $this, 'get_from_address' ) );
-		add_filter( 'wp_mail_from_name', array( $this, 'get_from_name' ) );
-		add_filter( 'wp_mail_content_type', array( $this, 'get_content_type' ) );
+		add_filter('wp_mail_from', array($this, 'get_from_address'));
+		add_filter('wp_mail_from_name', array($this, 'get_from_name'));
+		add_filter('wp_mail_content_type', array($this, 'get_content_type'));
 	}
 
 	/**
@@ -331,9 +331,9 @@ 
 	 * @since 1.0
 	 */
 	public function send_after() {
-		remove_filter( 'wp_mail_from', array( $this, 'get_from_address' ) );
-		remove_filter( 'wp_mail_from_name', array( $this, 'get_from_name' ) );
-		remove_filter( 'wp_mail_content_type', array( $this, 'get_content_type' ) );
+		remove_filter('wp_mail_from', array($this, 'get_from_address'));
+		remove_filter('wp_mail_from_name', array($this, 'get_from_name'));
+		remove_filter('wp_mail_content_type', array($this, 'get_content_type'));
 
 		// Reset heading to an empty string
 		$this->heading = '';
@@ -344,10 +344,10 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function text_to_html( $message ) {
+	public function text_to_html($message) {
 
-		if ( 'text/html' == $this->content_type || true === $this->html ) {
-			$message = wpautop( $message );
+		if ('text/html' == $this->content_type || true === $this->html) {
+			$message = wpautop($message);
 		}
 
 		return $message;

uninstall.php

Spacing +42 added lines, -42 removed lines

@@ -10,61 +10,61 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'WP_UNINSTALL_PLUGIN' ) ) {
+if ( ! defined('WP_UNINSTALL_PLUGIN')) {
 	exit;
 }
 
 // Load Give file.
-include_once( 'give.php' );
+include_once('give.php');
 
 global $wpdb, $wp_roles;
 
 
-if ( give_get_option( 'uninstall_on_delete' ) === 'on' ) {
+if (give_get_option('uninstall_on_delete') === 'on') {
 
 	// Delete All the Custom Post Types.
-	$give_taxonomies = array( 'form_category', 'form_tag', 'give_log_type' );
-	$give_post_types = array( 'give_forms', 'give_payment', 'give_log' );
-	foreach ( $give_post_types as $post_type ) {
+	$give_taxonomies = array('form_category', 'form_tag', 'give_log_type');
+	$give_post_types = array('give_forms', 'give_payment', 'give_log');
+	foreach ($give_post_types as $post_type) {
 
-		$give_taxonomies = array_merge( $give_taxonomies, get_object_taxonomies( $post_type ) );
-		$items           = get_posts( array(
+		$give_taxonomies = array_merge($give_taxonomies, get_object_taxonomies($post_type));
+		$items           = get_posts(array(
 			'post_type'   => $post_type,
 			'post_status' => 'any',
-			'numberposts' => - 1,
+			'numberposts' => -1,
 			'fields'      => 'ids',
-		) );
+		));
 
-		if ( $items ) {
-			foreach ( $items as $item ) {
-				wp_delete_post( $item, true );
+		if ($items) {
+			foreach ($items as $item) {
+				wp_delete_post($item, true);
 			}
 		}
 	}
 
 	// Delete All the Terms & Taxonomies.
-	foreach ( array_unique( array_filter( $give_taxonomies ) ) as $taxonomy ) {
+	foreach (array_unique(array_filter($give_taxonomies)) as $taxonomy) {
 
-		$terms = $wpdb->get_results( $wpdb->prepare( "SELECT t.*, tt.* FROM $wpdb->terms AS t INNER JOIN $wpdb->term_taxonomy AS tt ON t.term_id = tt.term_id WHERE tt.taxonomy IN ('%s') ORDER BY t.name ASC", $taxonomy ) );
+		$terms = $wpdb->get_results($wpdb->prepare("SELECT t.*, tt.* FROM $wpdb->terms AS t INNER JOIN $wpdb->term_taxonomy AS tt ON t.term_id = tt.term_id WHERE tt.taxonomy IN ('%s') ORDER BY t.name ASC", $taxonomy));
 
 		// Delete Terms.
-		if ( $terms ) {
-			foreach ( $terms as $term ) {
-				$wpdb->delete( $wpdb->term_taxonomy, array( 'term_taxonomy_id' => $term->term_taxonomy_id ) );
-				$wpdb->delete( $wpdb->terms, array( 'term_id' => $term->term_id ) );
+		if ($terms) {
+			foreach ($terms as $term) {
+				$wpdb->delete($wpdb->term_taxonomy, array('term_taxonomy_id' => $term->term_taxonomy_id));
+				$wpdb->delete($wpdb->terms, array('term_id' => $term->term_id));
 			}
 		}
 
 		// Delete Taxonomies.
-		$wpdb->delete( $wpdb->term_taxonomy, array( 'taxonomy' => $taxonomy ), array( '%s' ) );
+		$wpdb->delete($wpdb->term_taxonomy, array('taxonomy' => $taxonomy), array('%s'));
 	}
 
 	// Delete the Plugin Pages.
-	$give_created_pages = array( 'success_page', 'failure_page', 'history_page' );
-	foreach ( $give_created_pages as $p ) {
-		$page = give_get_option( $p, false );
-		if ( $page ) {
-			wp_delete_post( $page, true );
+	$give_created_pages = array('success_page', 'failure_page', 'history_page');
+	foreach ($give_created_pages as $p) {
+		$page = give_get_option($p, false);
+		if ($page) {
+			wp_delete_post($page, true);
 		}
 	}
 
@@ -72,20 +72,20 @@ 
 	Give()->roles->remove_caps();
 
 	// Delete the Roles.
-	$give_roles = array( 'give_manager', 'give_accountant', 'give_worker' );
-	foreach ( $give_roles as $role ) {
-		remove_role( $role );
+	$give_roles = array('give_manager', 'give_accountant', 'give_worker');
+	foreach ($give_roles as $role) {
+		remove_role($role);
 	}
 
 	// Remove all database tables.
-	$wpdb->query( 'DROP TABLE IF EXISTS ' . $wpdb->prefix . 'give_donors' );
-	$wpdb->query( 'DROP TABLE IF EXISTS ' . $wpdb->prefix . 'give_customers' );
-	$wpdb->query( 'DROP TABLE IF EXISTS ' . $wpdb->prefix . 'give_customermeta' );
+	$wpdb->query('DROP TABLE IF EXISTS '.$wpdb->prefix.'give_donors');
+	$wpdb->query('DROP TABLE IF EXISTS '.$wpdb->prefix.'give_customers');
+	$wpdb->query('DROP TABLE IF EXISTS '.$wpdb->prefix.'give_customermeta');
 
 	// Cleanup Cron Events.
-	wp_clear_scheduled_hook( 'give_daily_scheduled_events' );
-	wp_clear_scheduled_hook( 'give_daily_cron' );
-	wp_clear_scheduled_hook( 'give_weekly_cron' );
+	wp_clear_scheduled_hook('give_daily_scheduled_events');
+	wp_clear_scheduled_hook('give_daily_cron');
+	wp_clear_scheduled_hook('give_weekly_cron');
 
 	// Get all options.
 	$give_option_names = $wpdb->get_results(
@@ -96,24 +96,24 @@ 
 		ARRAY_A
 	);
 
-	if ( ! empty( $give_option_names ) ) {
+	if ( ! empty($give_option_names)) {
 		// Convert option name to transient or option name.
 		$new_give_option_names = array();
 
-		foreach ( $give_option_names as $option ) {
-			$new_give_option_names[] = ( false !== strpos( $option['option_name'], '_transient_' ) )
-				? str_replace( '_transient_', '', $option['option_name'] )
+		foreach ($give_option_names as $option) {
+			$new_give_option_names[] = (false !== strpos($option['option_name'], '_transient_'))
+				? str_replace('_transient_', '', $option['option_name'])
 				: $option['option_name'];
 		}
 
 		$give_option_names = $new_give_option_names;
 
 		// Delete all the Plugin Options.
-		foreach ( $give_option_names as $option ) {
-			if ( false !== strpos( $option, '_transient_' ) ) {
-				delete_transient( $option );
+		foreach ($give_option_names as $option) {
+			if (false !== strpos($option, '_transient_')) {
+				delete_transient($option);
 			} else {
-				delete_option( $option );
+				delete_option($option);
 			}
 		}
 	}

includes/formatting.php

Doc Comments +3 added lines, -3 removed lines

@@ -225,7 +225,7 @@ 
  * @param $field_args
  * @param $field
  *
- * @return bool
+ * @return false|null
  */
 function give_format_admin_multilevel_amount( $field_args, $field ) {
 
@@ -375,7 +375,7 @@ 
  * @param   array  $field_args
  * @param   object $field
  *
- * @return mixed
+ * @return string
  */
 function give_sanitize_thousand_separator( $value, $field_args, $field ){
     return stripslashes( $value );
@@ -408,7 +408,7 @@ 
  * @param   array  $field_args
  * @param   object $field
  *
- * @return  mixed
+ * @return  string
  */
 function give_sanitize_price_field_value( $value, $field_args, $field ){
     return give_sanitize_amount( $value );

Spacing +102 added lines, -102 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  * @return mixed
  */
 function give_get_price_decimals() {
-	return apply_filters( 'give_sanitize_amount_decimals', give_get_option( 'number_decimals', 0 ) );
+	return apply_filters('give_sanitize_amount_decimals', give_get_option('number_decimals', 0));
 }
 
 /**
@@ -34,7 +34,7 @@ 
  * @return mixed
  */
 function give_get_price_thousand_separator() {
-	return give_get_option( 'thousands_separator', ',' );
+	return give_get_option('thousands_separator', ',');
 }
 
 /**
@@ -45,7 +45,7 @@ 
  * @return mixed
  */
 function give_get_price_decimal_separator() {
-	return give_get_option( 'decimal_separator', '.' );
+	return give_get_option('decimal_separator', '.');
 }
 
 /**
@@ -62,67 +62,67 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_sanitize_amount( $number, $dp = false, $trim_zeros = false ) {
+function give_sanitize_amount($number, $dp = false, $trim_zeros = false) {
 
 	// Bailout.
-	if ( empty( $number ) ) {
+	if (empty($number)) {
 		return $number;
 	}
 
 	// Remove slash from amount.
 	// If thousand or decimal separator is set to ' then in $_POST or $_GET param we will get an escaped number.
 	// To prevent notices and warning remove slash from amount/number.
-	$number = wp_unslash( $number );
+	$number = wp_unslash($number);
 
 	$thousand_separator = give_get_price_thousand_separator();
 
 	$locale   = localeconv();
-	$decimals = array( give_get_price_decimal_separator(), $locale['decimal_point'], $locale['mon_decimal_point'] );
+	$decimals = array(give_get_price_decimal_separator(), $locale['decimal_point'], $locale['mon_decimal_point']);
 
 	// Remove locale from string
-	if ( ! is_float( $number ) ) {
-		$number = str_replace( $decimals, '.', $number );
+	if ( ! is_float($number)) {
+		$number = str_replace($decimals, '.', $number);
 	}
 
 	// Remove thousand amount formatting if amount has.
 	// This condition use to add backward compatibility to version before 1.6, because before version 1.6 we were saving formatted amount to db.
 	// Do not replace thousand separator from price if it is same as decimal separator, because it will be already replace by above code.
-	if ( ! in_array( $thousand_separator, $decimals ) && ( false !== strpos( $number, $thousand_separator ) ) ) {
-		$number = str_replace( $thousand_separator, '', $number );
-	} elseif ( in_array( $thousand_separator, $decimals ) ) {
-		$number = preg_replace( '/\.(?=.*\.)/', '', $number );
+	if ( ! in_array($thousand_separator, $decimals) && (false !== strpos($number, $thousand_separator))) {
+		$number = str_replace($thousand_separator, '', $number);
+	} elseif (in_array($thousand_separator, $decimals)) {
+		$number = preg_replace('/\.(?=.*\.)/', '', $number);
 	}
 
 	// Remove non numeric entity before decimal separator.
-	$number     = preg_replace( '/[^0-9\.]/', '', $number );
+	$number     = preg_replace('/[^0-9\.]/', '', $number);
 	$default_dp = give_get_price_decimals();
 
 	// Reset negative amount to zero.
-	if ( 0 > $number ) {
-		$number = number_format( 0, $default_dp, '.' );
+	if (0 > $number) {
+		$number = number_format(0, $default_dp, '.');
 	}
 
 	// If number does not have decimal then add number of decimals to it.
 	if (
-		false === strpos( $number, '.' )
-		|| ( $default_dp > strlen( substr( $number, strpos( $number, '.' ) + 1 ) ) )
+		false === strpos($number, '.')
+		|| ($default_dp > strlen(substr($number, strpos($number, '.') + 1)))
 	) {
-		$number = number_format( $number, $default_dp, '.', '' );
+		$number = number_format($number, $default_dp, '.', '');
 	}
 
 	// Format number by custom number of decimals.
-	if ( false !== $dp ) {
-		$dp     = intval( empty( $dp ) ? $default_dp : $dp );
-		$dp     = apply_filters( 'give_sanitize_amount_decimals', $dp, $number );
-		$number = number_format( floatval( $number ), $dp, '.', '' );
+	if (false !== $dp) {
+		$dp     = intval(empty($dp) ? $default_dp : $dp);
+		$dp     = apply_filters('give_sanitize_amount_decimals', $dp, $number);
+		$number = number_format(floatval($number), $dp, '.', '');
 	}
 
 	// Trim zeros.
-	if ( $trim_zeros && strstr( $number, '.' ) ) {
-		$number = rtrim( rtrim( $number, '0' ), '.' );
+	if ($trim_zeros && strstr($number, '.')) {
+		$number = rtrim(rtrim($number, '0'), '.');
 	}
 
-	return apply_filters( 'give_sanitize_amount', $number );
+	return apply_filters('give_sanitize_amount', $number);
 }
 
 /**
@@ -135,22 +135,22 @@ 
  *
  * @return string $amount   Newly formatted amount or Price Not Available
  */
-function give_format_amount( $amount, $decimals = true ) {
-	$thousands_sep = give_get_option( 'thousands_separator', ',' );
-	$decimal_sep   = give_get_option( 'decimal_separator', '.' );
+function give_format_amount($amount, $decimals = true) {
+	$thousands_sep = give_get_option('thousands_separator', ',');
+	$decimal_sep   = give_get_option('decimal_separator', '.');
 
-	if ( empty( $amount ) ) {
+	if (empty($amount)) {
 		$amount = 0;
 	} else {
 		// Sanitize amount before formatting.
-		$amount = give_sanitize_amount( $amount );
+		$amount = give_sanitize_amount($amount);
 	}
 
 	$decimals = $decimals ? give_get_price_decimals() : 0;
 
-	$formatted = number_format( $amount, $decimals, $decimal_sep, $thousands_sep );
+	$formatted = number_format($amount, $decimals, $decimal_sep, $thousands_sep);
 
-	return apply_filters( 'give_format_amount', $formatted, $amount, $decimals, $decimal_sep, $thousands_sep );
+	return apply_filters('give_format_amount', $formatted, $amount, $decimals, $decimal_sep, $thousands_sep);
 }
 
 
@@ -167,33 +167,33 @@ 
  *
  * @return float|string  formatted amount number with large number names.
  */
-function give_human_format_large_amount( $amount ) {
+function give_human_format_large_amount($amount) {
 
 	// Get thousand separator.
 	$thousands_sep = give_get_price_thousand_separator();
 
 	// Sanitize amount.
-	$sanitize_amount = give_sanitize_amount( $amount );
+	$sanitize_amount = give_sanitize_amount($amount);
 
 	// Explode amount to calculate name of large numbers.
-	$amount_array = explode( $thousands_sep, $amount );
+	$amount_array = explode($thousands_sep, $amount);
 
 	// Calculate amount parts count.
-	$amount_count_parts = count( $amount_array );
+	$amount_count_parts = count($amount_array);
 
 	// Human format amount (default).
 	$human_format_amount = $amount;
 
 	// Calculate large number formatted amount.
-	if ( 4 < $amount_count_parts ) {
-		$human_format_amount = sprintf( esc_html__( '%s trillion', 'give' ), round( ( $sanitize_amount / 1000000000000 ), 2 ) );
-	} elseif ( 3 < $amount_count_parts ) {
-		$human_format_amount = sprintf( esc_html__( '%s billion', 'give' ), round( ( $sanitize_amount / 1000000000 ), 2 ) );
-	} elseif ( 2 < $amount_count_parts ) {
-		$human_format_amount = sprintf( esc_html__( '%s million', 'give' ), round( ( $sanitize_amount / 1000000 ), 2 ) );
+	if (4 < $amount_count_parts) {
+		$human_format_amount = sprintf(esc_html__('%s trillion', 'give'), round(($sanitize_amount / 1000000000000), 2));
+	} elseif (3 < $amount_count_parts) {
+		$human_format_amount = sprintf(esc_html__('%s billion', 'give'), round(($sanitize_amount / 1000000000), 2));
+	} elseif (2 < $amount_count_parts) {
+		$human_format_amount = sprintf(esc_html__('%s million', 'give'), round(($sanitize_amount / 1000000), 2));
 	}
 
-	return apply_filters( 'give_human_format_large_amount', $human_format_amount, $amount, $sanitize_amount );
+	return apply_filters('give_human_format_large_amount', $human_format_amount, $amount, $sanitize_amount);
 }
 
 /**
@@ -206,15 +206,15 @@ 
  *
  * @return string $amount Newly formatted amount or Price Not Available
  */
-function give_format_decimal( $amount, $dp = false ) {
+function give_format_decimal($amount, $dp = false) {
 	$decimal_separator = give_get_price_decimal_separator();
-	$formatted_amount  = give_sanitize_amount( $amount, $dp );
+	$formatted_amount  = give_sanitize_amount($amount, $dp);
 
-	if ( false !== strpos( $formatted_amount, '.' ) ) {
-		$formatted_amount = str_replace( '.', $decimal_separator, $formatted_amount );
+	if (false !== strpos($formatted_amount, '.')) {
+		$formatted_amount = str_replace('.', $decimal_separator, $formatted_amount);
 	}
 
-	return apply_filters( 'give_format_decimal', $formatted_amount, $amount, $decimal_separator );
+	return apply_filters('give_format_decimal', $formatted_amount, $amount, $decimal_separator);
 }
 
 
@@ -228,13 +228,13 @@ 
  *
  * @return bool
  */
-function give_format_admin_multilevel_amount( $field_args, $field ) {
+function give_format_admin_multilevel_amount($field_args, $field) {
 
-	if ( empty( $field->value ) ) {
+	if (empty($field->value)) {
 		return false;
 	}
 
-	$field->value = give_format_decimal( $field->value );
+	$field->value = give_format_decimal($field->value);
 }
 
 /**
@@ -247,24 +247,24 @@ 
  *
  * @return mixed|string|void
  */
-function give_currency_filter( $price = '', $currency = '' ) {
+function give_currency_filter($price = '', $currency = '') {
 
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
 
-	$position = give_get_option( 'currency_position', 'before' );
+	$position = give_get_option('currency_position', 'before');
 
 	$negative = $price < 0;
 
-	if ( $negative ) {
+	if ($negative) {
 		// Remove proceeding "-".
-		$price = substr( $price, 1 );
+		$price = substr($price, 1);
 	}
 
-	$symbol = give_currency_symbol( $currency );
+	$symbol = give_currency_symbol($currency);
 
-	switch ( $currency ):
+	switch ($currency):
 		case 'GBP' :
 		case 'BRL' :
 		case 'EUR' :
@@ -293,13 +293,13 @@ 
 		case 'MAD' :
 		case 'KRW' :
 		case 'ZAR' :
-			$formatted = ( 'before' === $position ? $symbol . $price : $price . $symbol );
+			$formatted = ('before' === $position ? $symbol.$price : $price.$symbol);
 			break;
 		case 'NOK' :
-			$formatted = ( 'before' === $position ? $symbol . ' ' . $price : $price . ' ' . $symbol );
+			$formatted = ('before' === $position ? $symbol.' '.$price : $price.' '.$symbol);
 			break;
 		default :
-			$formatted = ( 'before' === $position ? $currency . ' ' . $price : $price . ' ' . $currency );
+			$formatted = ('before' === $position ? $currency.' '.$price : $price.' '.$currency);
 			break;
 	endswitch;
 
@@ -315,11 +315,11 @@ 
 	 *           filter name will be give_usd_currency_filter_after
 	 *
 	 */
-	$formatted = apply_filters( 'give_' . strtolower( $currency ) . "_currency_filter_{$position}", $formatted, $currency, $price );
+	$formatted = apply_filters('give_'.strtolower($currency)."_currency_filter_{$position}", $formatted, $currency, $price);
 
-	if ( $negative ) {
+	if ($negative) {
 		// Prepend the minus sign before the currency sign.
-		$formatted = '-' . $formatted;
+		$formatted = '-'.$formatted;
 	}
 
 	return $formatted;
@@ -335,22 +335,22 @@ 
  */
 function give_currency_decimal_filter() {
 
-	remove_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
+	remove_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
 
 	// Set default number of decimals.
 	$decimals = give_get_price_decimals();
 
-	add_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
+	add_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
 
 
 	// Get number of decimals with backward compatibility ( version < 1.6 )
-	if ( 1 <= func_num_args() ) {
-		$decimals = ( false === func_get_arg( 0 ) ? $decimals : absint( func_get_arg( 0 ) ) );
+	if (1 <= func_num_args()) {
+		$decimals = (false === func_get_arg(0) ? $decimals : absint(func_get_arg(0)));
 	}
 
 	$currency = give_get_currency();
 
-	switch ( $currency ) {
+	switch ($currency) {
 		case 'RIAL' :
 		case 'JPY' :
 		case 'TWD' :
@@ -360,11 +360,11 @@ 
 			break;
 	}
 
-	return apply_filters( 'give_currency_decimal_count', $decimals, $currency );
+	return apply_filters('give_currency_decimal_count', $decimals, $currency);
 }
 
-add_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
-add_filter( 'give_format_amount_decimals', 'give_currency_decimal_filter' );
+add_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
+add_filter('give_format_amount_decimals', 'give_currency_decimal_filter');
 
 /**
  * Sanitize thousand separator
@@ -378,8 +378,8 @@ 
  *
  * @return mixed
  */
-function give_sanitize_thousand_separator( $value, $field_args, $field ) {
-	return stripslashes( $value );
+function give_sanitize_thousand_separator($value, $field_args, $field) {
+	return stripslashes($value);
 }
 
 
@@ -395,8 +395,8 @@ 
  *
  * @return  mixed
  */
-function give_sanitize_number_decimals( $value, $field_args, $field ) {
-	return absint( $value );
+function give_sanitize_number_decimals($value, $field_args, $field) {
+	return absint($value);
 }
 
 /**
@@ -411,8 +411,8 @@ 
  *
  * @return  mixed
  */
-function give_sanitize_price_field_value( $value, $field_args, $field ) {
-	return give_sanitize_amount( $value );
+function give_sanitize_price_field_value($value, $field_args, $field) {
+	return give_sanitize_amount($value);
 }
 
 
@@ -426,29 +426,29 @@ 
  *
  * @return void
  */
-function give_cmb_amount_field_render_row_cb( $field_args, $field ) {
+function give_cmb_amount_field_render_row_cb($field_args, $field) {
 
 	// Get args.
-	$id                = $field->args( 'id' );
-	$label             = $field->args( 'name' );
-	$name              = $field->args( '_name' );
-	$description       = $field->args( 'description' );
-	$attributes        = $field->args( 'attributes' );
+	$id                = $field->args('id');
+	$label             = $field->args('name');
+	$name              = $field->args('_name');
+	$description       = $field->args('description');
+	$attributes        = $field->args('attributes');
 	$attributes_string = '';
 	$row_class         = $field->row_classes();
 
 	// Get attributes.
-	if ( ! empty( $attributes ) ) {
-		foreach ( $attributes as $attribute_name => $attribute_val ) {
+	if ( ! empty($attributes)) {
+		foreach ($attributes as $attribute_name => $attribute_val) {
 			$attributes_string[] = "$attribute_name=\"$attribute_val\"";
 		}
 
-		$attributes_string = implode( ' ', $attributes_string );
+		$attributes_string = implode(' ', $attributes_string);
 	}
 
 	// Get row class.
-	if ( ! empty( $row_class ) && is_array( $row_class ) ) {
-		$row_class = implode( ' ', $row_class );
+	if ( ! empty($row_class) && is_array($row_class)) {
+		$row_class = implode(' ', $row_class);
 	}
 	?>
 	<div class="cmb-row <?php echo $row_class; ?>">
@@ -456,9 +456,9 @@ 
 			<label for="<?php echo $id; ?>"><?php echo $label; ?></label>
 		</div>
 		<div class="cmb-td">
-			<?php echo( give_get_option( 'currency_position' ) == 'before' ? '<span class="give-money-symbol give-money-symbol-before">' . give_currency_symbol() . '</span>' : '' ); ?>
+			<?php echo(give_get_option('currency_position') == 'before' ? '<span class="give-money-symbol give-money-symbol-before">'.give_currency_symbol().'</span>' : ''); ?>
 			<input id="<?php echo $id; ?>" type="text" name="<?php echo $name; ?>" <?php echo $attributes_string ?>/>
-			<?php echo( give_get_option( 'currency_position' ) == 'after' ? '<span class="give-money-symbol give-money-symbol-after">' . give_currency_symbol() . '</span>' : '' ); ?>
+			<?php echo(give_get_option('currency_position') == 'after' ? '<span class="give-money-symbol give-money-symbol-after">'.give_currency_symbol().'</span>' : ''); ?>
 
 			<span class="cmb2-metabox-description"><?php echo $description; ?></span>
 		</div>
@@ -477,7 +477,7 @@ 
  *
  * @return string                  Date format string
  */
-function give_date_format ( $date_context = '' ) {
+function give_date_format($date_context = '') {
 	/**
 	 * Filter the date context
 	 *
@@ -498,20 +498,20 @@ 
 	 *
 	 *    }
 	 */
-	$date_format_contexts = apply_filters( 'give_date_format_contexts', array() );
+	$date_format_contexts = apply_filters('give_date_format_contexts', array());
 
 	// Set date format to default date format.
 	$date_format = get_option('date_format');
 
 
 	// Update date format if we have non empty date format context array and non empty date format string for that context.
-	if( $date_context &&  ! empty( $date_format_contexts ) && array_key_exists( $date_context, $date_format_contexts ) ) {
-		$date_format = ! empty( $date_format_contexts[ $date_context ] )
-			? $date_format_contexts[ $date_context ]
+	if ($date_context && ! empty($date_format_contexts) && array_key_exists($date_context, $date_format_contexts)) {
+		$date_format = ! empty($date_format_contexts[$date_context])
+			? $date_format_contexts[$date_context]
 			: $date_format;
 	}
 
-	return apply_filters( 'give_date_format', $date_format );
+	return apply_filters('give_date_format', $date_format);
 }
 
 /**
@@ -524,11 +524,11 @@ 
  *
  * @return string
  */
-function give_get_cache_key( $action, $query_args ) {
+function give_get_cache_key($action, $query_args) {
 	// Bailout.
-	if( ! is_array( $query_args ) || empty( $query_args ) ) {
+	if ( ! is_array($query_args) || empty($query_args)) {
 		return '';
 	}
 
-	return "give_cache_{$action}_" . substr( md5( serialize( $query_args ) ), 0, 15 );
+	return "give_cache_{$action}_".substr(md5(serialize($query_args)), 0, 15);
 }
\ No newline at end of file

includes/class-give-license-handler.php

Braces +1 added lines, -1 removed lines

@@ -689,7 +689,7 @@
 								"{$this->checkout_url}?edd_license_key={$subscription['license_key']}&utm_campaign=admin&utm_source=licenses&utm_medium=expired",
 								esc_url( add_query_arg( '_give_hide_license_notices_permanently', $subscription['id'], $_SERVER['REQUEST_URI'] ) )
 							);
-						}else{
+						} else{
 							$messages[$subscription['id']] = sprintf(
 								__( 'You Give addon license will expire in %s for payment <a href="%s" target="_blank">#%d</a>. <a href="%s" target="_blank">Click to renew an existing license</a> or <a href="%s">Click here if already renewed</a>.', 'give' ),
 								human_time_diff( current_time( 'timestamp', 1 ), strtotime( $subscription['expires'] ) ),

Doc Comments +5 added lines, -5 removed lines

@@ -219,7 +219,7 @@ 
 		 * @access private
 		 * @since  1.0
 		 *
-		 * @return bool
+		 * @return false|null
 		 */
 		public function auto_updater() {
 
@@ -475,7 +475,7 @@ 
 		 * @access public
 		 * @since  1.7
 		 *
-		 * @return bool|void
+		 * @return false|null
 		 */
 		public function weekly_license_check() {
 
@@ -527,7 +527,7 @@ 
 		 * @access public
 		 * @since  1.7
 		 *
-		 * @return bool|void
+		 * @return false|null
 		 */
 		public function weekly_subscription_check() {
 
@@ -607,7 +607,7 @@ 
 		 * @access private
 		 * @since  1.7
 		 *
-		 * @return bool|void
+		 * @return false|null
 		 */
 		private function __single_subscription_check() {
 			// Do not fire if license key is not set.
@@ -802,7 +802,7 @@ 
 		 * @access private
 		 * @since  1.7
 		 *
-		 * @return void|bool
+		 * @return false|null
 		 */
 		private function __remove_license_key_from_subscriptions(){
 			$subscriptions = get_option( 'give_subscriptions', array() );

Spacing +142 added lines, -142 removed lines

@@ -10,11 +10,11 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_License' ) ) :
+if ( ! class_exists('Give_License')) :
 
 	/**
 	 * Give_License Class
@@ -103,7 +103,7 @@ 
 		 *
 		 * @var    string
 		 */
-		private $api_url      = 'https://givewp.com/edd-sl-api/';
+		private $api_url = 'https://givewp.com/edd-sl-api/';
 
 		/**
 		 * Account URL
@@ -113,7 +113,7 @@ 
 		 *
 		 * @var null|string
 		 */
-		private $account_url  = 'https://givewp.com/my-account/';
+		private $account_url = 'https://givewp.com/my-account/';
 
 		/**
 		 * Ccheckout URL
@@ -142,19 +142,19 @@ 
 		 * @param string  $_checkout_url
 		 * @param string  $_account_url
 		 */
-		public function __construct( $_file, $_item_name, $_version, $_author, $_optname = null, $_api_url = null, $_checkout_url = null, $_account_url = null ) {
+		public function __construct($_file, $_item_name, $_version, $_author, $_optname = null, $_api_url = null, $_checkout_url = null, $_account_url = null) {
 			$give_options = give_get_settings();
 
 			$this->file           = $_file;
 			$this->item_name      = $_item_name;
-			$this->item_shortname = 'give_' . preg_replace( '/[^a-zA-Z0-9_\s]/', '', str_replace( ' ', '_', strtolower( $this->item_name ) ) );
+			$this->item_shortname = 'give_'.preg_replace('/[^a-zA-Z0-9_\s]/', '', str_replace(' ', '_', strtolower($this->item_name)));
 			$this->version        = $_version;
-			$this->license        = isset( $give_options[ $this->item_shortname . '_license_key' ] ) ? trim( $give_options[ $this->item_shortname . '_license_key' ] ) : '';
-			$this->license_data   = get_option( $this->item_shortname . '_license_active' );
+			$this->license        = isset($give_options[$this->item_shortname.'_license_key']) ? trim($give_options[$this->item_shortname.'_license_key']) : '';
+			$this->license_data   = get_option($this->item_shortname.'_license_active');
 			$this->author         = $_author;
-			$this->api_url        = is_null( $_api_url ) ? $this->api_url : $_api_url;
-			$this->checkout_url   = is_null( $_checkout_url ) ? $this->checkout_url : $_checkout_url;
-			$this->account_url    = is_null( $_account_url ) ? $this->account_url : $_account_url;
+			$this->api_url        = is_null($_api_url) ? $this->api_url : $_api_url;
+			$this->checkout_url   = is_null($_checkout_url) ? $this->checkout_url : $_checkout_url;
+			$this->account_url    = is_null($_account_url) ? $this->account_url : $_account_url;
 
 			// Setup hooks
 			$this->includes();
@@ -173,7 +173,7 @@ 
 		 * @return void
 		 */
 		private function includes() {
-			if ( ! class_exists( 'EDD_SL_Plugin_Updater' ) ) {
+			if ( ! class_exists('EDD_SL_Plugin_Updater')) {
 				require_once 'admin/EDD_SL_Plugin_Updater.php';
 			}
 		}
@@ -191,26 +191,26 @@ 
 		private function hooks() {
 
 			// Register settings
-			add_filter( 'give_settings_licenses', array( $this, 'settings' ), 1 );
+			add_filter('give_settings_licenses', array($this, 'settings'), 1);
 
 			// Activate license key on settings save
-			add_action( 'admin_init', array( $this, 'activate_license' ) );
+			add_action('admin_init', array($this, 'activate_license'));
 
 			// Deactivate license key
-			add_action( 'admin_init', array( $this, 'deactivate_license' ) );
+			add_action('admin_init', array($this, 'deactivate_license'));
 
 			// Updater
-			add_action( 'admin_init', array( $this, 'auto_updater' ), 0 );
+			add_action('admin_init', array($this, 'auto_updater'), 0);
 
-			add_action( 'admin_notices', array( $this, 'notices' ) );
+			add_action('admin_notices', array($this, 'notices'));
 
 			// Check license weekly.
-			add_action( 'give_weekly_scheduled_events', array( $this, 'weekly_license_check' ) );
-			add_action( 'give_validate_license_when_site_migrated', array( $this, 'weekly_license_check' ) );
+			add_action('give_weekly_scheduled_events', array($this, 'weekly_license_check'));
+			add_action('give_validate_license_when_site_migrated', array($this, 'weekly_license_check'));
 
 			// Check subscription weekly.
-			add_action( 'give_weekly_scheduled_events', array( $this, 'weekly_subscription_check' ) );
-			add_action( 'give_validate_license_when_site_migrated', array( $this, 'weekly_subscription_check' ) );
+			add_action('give_weekly_scheduled_events', array($this, 'weekly_subscription_check'));
+			add_action('give_validate_license_when_site_migrated', array($this, 'weekly_subscription_check'));
 		}
 
 		/**
@@ -223,7 +223,7 @@ 
 		 */
 		public function auto_updater() {
 
-			if ( ! $this->is_valid_license() ) {
+			if ( ! $this->is_valid_license()) {
 				return false;
 			}
 
@@ -252,16 +252,16 @@ 
 		 *
 		 * @return array           License settings.
 		 */
-		public function settings( $settings ) {
+		public function settings($settings) {
 
 			$give_license_settings = array(
 				array(
 					'name'    => $this->item_name,
-					'id'      => $this->item_shortname . '_license_key',
+					'id'      => $this->item_shortname.'_license_key',
 					'desc'    => '',
 					'type'    => 'license_key',
 					'options' => array(
-						'license'       => get_option( $this->item_shortname . '_license_active' ),
+						'license'       => get_option($this->item_shortname.'_license_active'),
 						'shortname'     => $this->item_shortname,
 						'item_name'     => $this->item_name,
 						'api_url'       => $this->api_url,
@@ -272,7 +272,7 @@ 
 				)
 			);
 
-			return array_merge( $settings, $give_license_settings );
+			return array_merge($settings, $give_license_settings);
 		}
 
 		/**
@@ -287,18 +287,18 @@ 
 		 *
 		 * @return array           License settings content.
 		 */
-		public function license_settings_content( $settings ) {
+		public function license_settings_content($settings) {
 
 			$give_license_settings = array(
 				array(
-					'name' => esc_html__( 'Add-on Licenses', 'give' ),
+					'name' => esc_html__('Add-on Licenses', 'give'),
 					'desc' => '<hr>',
 					'type' => 'give_title',
 					'id'   => 'give_title'
 				),
 			);
 
-			return array_merge( $settings, $give_license_settings );
+			return array_merge($settings, $give_license_settings);
 		}
 
 		/**
@@ -313,37 +313,37 @@ 
 		 */
 		public function activate_license() {
 			// Bailout: Check if license key set of not.
-			if ( ! isset( $_POST[ $this->item_shortname . '_license_key' ] ) ) {
+			if ( ! isset($_POST[$this->item_shortname.'_license_key'])) {
 				return;
 			}
 
 			// Security check.
-			if ( ! wp_verify_nonce( $_REQUEST[ $this->item_shortname . '_license_key-nonce' ], $this->item_shortname . '_license_key-nonce' ) ) {
+			if ( ! wp_verify_nonce($_REQUEST[$this->item_shortname.'_license_key-nonce'], $this->item_shortname.'_license_key-nonce')) {
 
-				wp_die( esc_html__( 'Nonce verification failed.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+				wp_die(esc_html__('Nonce verification failed.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 
 			}
 
 			// Check if user have correct permissions.
-			if ( ! current_user_can( 'manage_give_settings' ) ) {
+			if ( ! current_user_can('manage_give_settings')) {
 				return;
 			}
 
 			// Allow third party addon developers to handle license activation.
-			if( $this->__is_third_party_addon() ){
-				do_action( 'give_activate_license', $this );
+			if ($this->__is_third_party_addon()) {
+				do_action('give_activate_license', $this);
 				return;
 			}
 
 			// Delete previous license setting if a empty license key submitted.
-			if ( empty( $_POST[ $this->item_shortname . '_license_key' ] ) ) {
-				delete_option( $this->item_shortname . '_license_active' );
+			if (empty($_POST[$this->item_shortname.'_license_key'])) {
+				delete_option($this->item_shortname.'_license_active');
 				return;
 			}
 
 			// Do not simultaneously activate any addon if user want to deactivate any addon.
-			foreach ( $_POST as $key => $value ) {
-				if ( false !== strpos( $key, 'license_key_deactivate' ) ) {
+			foreach ($_POST as $key => $value) {
+				if (false !== strpos($key, 'license_key_deactivate')) {
 					// Don't activate a key when deactivating a different key
 					return;
 				}
@@ -351,15 +351,15 @@ 
 
 
 			// Check if plugin previously installed.
-			if ( $this->is_valid_license() ) {
+			if ($this->is_valid_license()) {
 				return;
 			}
 
 			// Get license key.
-			$license = sanitize_text_field( $_POST[ $this->item_shortname . '_license_key' ] );
+			$license = sanitize_text_field($_POST[$this->item_shortname.'_license_key']);
 
 			// Bailout.
-			if( empty( $license ) ) {
+			if (empty($license)) {
 				return;
 			}
 
@@ -370,7 +370,7 @@ 
 			$api_params = array(
 				'edd_action' => 'activate_license', //never change from "edd_" to "give_"!
 				'license'    => $license,
-				'item_name'  => urlencode( $this->item_name ),
+				'item_name'  => urlencode($this->item_name),
 				'url'        => home_url()
 			);
 
@@ -385,16 +385,16 @@ 
 			);
 
 			// Make sure there are no errors
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error($response)) {
 				return;
 			}
 
 			// Tell WordPress to look for updates
-			set_site_transient( 'update_plugins', null );
+			set_site_transient('update_plugins', null);
 
 			// Decode license data
-			$license_data = json_decode( wp_remote_retrieve_body( $response ) );
-			update_option( $this->item_shortname . '_license_active', $license_data );
+			$license_data = json_decode(wp_remote_retrieve_body($response));
+			update_option($this->item_shortname.'_license_active', $license_data);
 
 			// Check subscription for license key and store this to db (if any).
 			$this->__single_subscription_check();
@@ -412,34 +412,34 @@ 
 		 */
 		public function deactivate_license() {
 
-			if ( ! isset( $_POST[ $this->item_shortname . '_license_key' ] ) ) {
+			if ( ! isset($_POST[$this->item_shortname.'_license_key'])) {
 				return;
 			}
 
-			if ( ! wp_verify_nonce( $_REQUEST[ $this->item_shortname . '_license_key-nonce' ], $this->item_shortname . '_license_key-nonce' ) ) {
+			if ( ! wp_verify_nonce($_REQUEST[$this->item_shortname.'_license_key-nonce'], $this->item_shortname.'_license_key-nonce')) {
 
-				wp_die( esc_html__( 'Nonce verification failed.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+				wp_die(esc_html__('Nonce verification failed.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 
 			}
 
-			if ( ! current_user_can( 'manage_give_settings' ) ) {
+			if ( ! current_user_can('manage_give_settings')) {
 				return;
 			}
 
 			// Allow third party addon developers to handle license deactivation.
-			if( $this->__is_third_party_addon() ){
-				do_action( 'give_deactivate_license', $this );
+			if ($this->__is_third_party_addon()) {
+				do_action('give_deactivate_license', $this);
 				return;
 			}
 
 			// Run on deactivate button press
-			if ( isset( $_POST[ $this->item_shortname . '_license_key_deactivate' ] ) ) {
+			if (isset($_POST[$this->item_shortname.'_license_key_deactivate'])) {
 
 				// Data to send to the API
 				$api_params = array(
 					'edd_action' => 'deactivate_license', //never change from "edd_" to "give_"!
 					'license'    => $this->license,
-					'item_name'  => urlencode( $this->item_name ),
+					'item_name'  => urlencode($this->item_name),
 					'url'        => home_url()
 				);
 
@@ -454,15 +454,15 @@ 
 				);
 
 				// Make sure there are no errors
-				if ( is_wp_error( $response ) ) {
+				if (is_wp_error($response)) {
 					return;
 				}
 
 				// Decode the license data
-				$license_data = json_decode( wp_remote_retrieve_body( $response ) );
+				$license_data = json_decode(wp_remote_retrieve_body($response));
 
 				// Remove license data.
-				delete_option( $this->item_shortname . '_license_active' );
+				delete_option($this->item_shortname.'_license_active');
 
 				// Remove license key from subscriptions if exist.
 				$this->__remove_license_key_from_subscriptions();
@@ -479,18 +479,18 @@ 
 		 */
 		public function weekly_license_check() {
 
-			if( ! empty( $_POST['give_settings'] ) ) {
+			if ( ! empty($_POST['give_settings'])) {
 				// Don't fire when saving settings
 				return false;
 			}
 
-			if( empty( $this->license ) ) {
+			if (empty($this->license)) {
 				return false;
 			}
 
 			// Allow third party addon developers to handle their license check.
-			if( $this->__is_third_party_addon() ){
-				do_action( 'give_weekly_license_check', $this );
+			if ($this->__is_third_party_addon()) {
+				do_action('give_weekly_license_check', $this);
 				return false;
 			}
 
@@ -498,7 +498,7 @@ 
 			$api_params = array(
 				'edd_action'=> 'check_license',
 				'license' 	=> $this->license,
-				'item_name' => urlencode( $this->item_name ),
+				'item_name' => urlencode($this->item_name),
 				'url'       => home_url()
 			);
 
@@ -513,12 +513,12 @@ 
 			);
 
 			// Make sure the response came back okay.
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error($response)) {
 				return false;
 			}
 
-			$license_data = json_decode( wp_remote_retrieve_body( $response ) );
-			update_option( $this->item_shortname . '_license_active', $license_data );
+			$license_data = json_decode(wp_remote_retrieve_body($response));
+			update_option($this->item_shortname.'_license_active', $license_data);
 		}
 
 		/**
@@ -531,24 +531,24 @@ 
 		 */
 		public function weekly_subscription_check() {
 
-			if( ! empty( $_POST['give_settings'] ) ) {
+			if ( ! empty($_POST['give_settings'])) {
 				// Don't fire when saving settings
 				return false;
 			}
 
 			// Remove old subscription data.
-			if( absint( get_option( '_give_subscriptions_edit_last', true ) ) < current_time( 'timestamp' , 1 ) ){
-				delete_option( 'give_subscriptions' );
-				update_option( '_give_subscriptions_edit_last', strtotime( '+ 1 day', current_time( 'timestamp' , 1 ) ) );
+			if (absint(get_option('_give_subscriptions_edit_last', true)) < current_time('timestamp', 1)) {
+				delete_option('give_subscriptions');
+				update_option('_give_subscriptions_edit_last', strtotime('+ 1 day', current_time('timestamp', 1)));
 			}
 
-			if( empty( $this->license ) ) {
+			if (empty($this->license)) {
 				return false;
 			}
 
 			// Allow third party addon developers to handle there subscription check.
-			if( $this->__is_third_party_addon() ){
-				do_action( 'give_weekly_subscription_check', $this );
+			if ($this->__is_third_party_addon()) {
+				do_action('give_weekly_subscription_check', $this);
 				return false;
 			}
 
@@ -562,7 +562,7 @@ 
 				// This is custom feature to check subscriptions.
 				'edd_action'=> 'check_subscription',
 				'license' 	=> $this->license,
-				'item_name' => urlencode( $this->item_name ),
+				'item_name' => urlencode($this->item_name),
 				'url'       => home_url()
 			);
 
@@ -577,27 +577,27 @@ 
 			);
 
 			// Make sure the response came back okay.
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error($response)) {
 				return false;
 			}
 
-			$subscription_data = json_decode( wp_remote_retrieve_body( $response ), true );
+			$subscription_data = json_decode(wp_remote_retrieve_body($response), true);
 
-			if( ! empty( $subscription_data['success'] ) && absint( $subscription_data['success'] ) ) {
-				$subscriptions = get_option( 'give_subscriptions', array() );
+			if ( ! empty($subscription_data['success']) && absint($subscription_data['success'])) {
+				$subscriptions = get_option('give_subscriptions', array());
 
 				// Update subscription data only if subscription does not exist already.
-				if( ! array_key_exists( $subscription_data['id'], $subscriptions ) ) {
-					$subscriptions[ $subscription_data['id'] ] = $subscription_data;
-					$subscriptions[ $subscription_data['id'] ]['licenses'] = array();
+				if ( ! array_key_exists($subscription_data['id'], $subscriptions)) {
+					$subscriptions[$subscription_data['id']] = $subscription_data;
+					$subscriptions[$subscription_data['id']]['licenses'] = array();
 				}
 
 				// Store licenses for subscription.
-				if( ! in_array( $this->license, $subscriptions[ $subscription_data['id'] ]['licenses'] ) ) {
-					$subscriptions[ $subscription_data['id'] ]['licenses'][] = $this->license;
+				if ( ! in_array($this->license, $subscriptions[$subscription_data['id']]['licenses'])) {
+					$subscriptions[$subscription_data['id']]['licenses'][] = $this->license;
 				}
 
-				update_option( 'give_subscriptions', $subscriptions );
+				update_option('give_subscriptions', $subscriptions);
 			}
 		}
 
@@ -611,11 +611,11 @@ 
 		 */
 		private function __single_subscription_check() {
 			// Do not fire if license key is not set.
-			if ( ! isset( $_POST[ $this->item_shortname . '_license_key' ] ) ) {
+			if ( ! isset($_POST[$this->item_shortname.'_license_key'])) {
 				return false;
 			}
 
-			if( empty( $this->license ) ) {
+			if (empty($this->license)) {
 				return false;
 			}
 
@@ -626,7 +626,7 @@ 
 				// This is custom feature to check subscriptions.
 				'edd_action'=> 'check_subscription',
 				'license' 	=> $this->license,
-				'item_name' => urlencode( $this->item_name ),
+				'item_name' => urlencode($this->item_name),
 				'url'       => home_url()
 			);
 
@@ -641,27 +641,27 @@ 
 			);
 
 			// Make sure the response came back okay.
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error($response)) {
 				return false;
 			}
 
-			$subscription_data = json_decode( wp_remote_retrieve_body( $response ), true );
+			$subscription_data = json_decode(wp_remote_retrieve_body($response), true);
 
-			if( ! empty( $subscription_data['success'] ) && absint( $subscription_data['success'] ) ) {
-				$subscriptions = get_option( 'give_subscriptions', array() );
+			if ( ! empty($subscription_data['success']) && absint($subscription_data['success'])) {
+				$subscriptions = get_option('give_subscriptions', array());
 
 				// Update subscription data only if subscription does not exist already.
-				if( ! array_key_exists( $subscription_data['id'], $subscriptions ) ) {
-					$subscriptions[ $subscription_data['id'] ] = $subscription_data;
-					$subscriptions[ $subscription_data['id'] ]['licenses'] = array();
+				if ( ! array_key_exists($subscription_data['id'], $subscriptions)) {
+					$subscriptions[$subscription_data['id']] = $subscription_data;
+					$subscriptions[$subscription_data['id']]['licenses'] = array();
 				}
 
 				// Store licenses for subscription.
-				if( ! in_array( $this->license, $subscriptions[ $subscription_data['id'] ]['licenses'] ) ) {
-					$subscriptions[ $subscription_data['id'] ]['licenses'][] = $this->license;
+				if ( ! in_array($this->license, $subscriptions[$subscription_data['id']]['licenses'])) {
+					$subscriptions[$subscription_data['id']]['licenses'][] = $this->license;
 				}
 
-				update_option( 'give_subscriptions', $subscriptions );
+				update_option('give_subscriptions', $subscriptions);
 			}
 		}
 
@@ -679,87 +679,87 @@ 
 			static $addon_license_key_in_subscriptions;
 
 			// Set default value.
-			$addon_license_key_in_subscriptions = ! empty( $addon_license_key_in_subscriptions ) ? $addon_license_key_in_subscriptions : array();
+			$addon_license_key_in_subscriptions = ! empty($addon_license_key_in_subscriptions) ? $addon_license_key_in_subscriptions : array();
 
-			if( empty( $this->license ) ) {
+			if (empty($this->license)) {
 				return;
 			}
 
-			if( ! current_user_can( 'manage_shop_settings' ) ) {
+			if ( ! current_user_can('manage_shop_settings')) {
 				return;
 			}
 
 			// Do not show licenses notices on license tab.
-			if( ! empty( $_GET['tab'] ) && 'licenses' === $_GET['tab'] ) {
+			if ( ! empty($_GET['tab']) && 'licenses' === $_GET['tab']) {
 				return;
 			}
 
 			$messages = array();
 
 			// Get subscriptions.
-			$subscriptions = get_option( 'give_subscriptions' );
+			$subscriptions = get_option('give_subscriptions');
 
 			// Show subscription messages.
-			if( ! empty( $subscriptions ) && ! $showed_subscriptions_message ) {
+			if ( ! empty($subscriptions) && ! $showed_subscriptions_message) {
 
-				foreach ( $subscriptions as $subscription ) {
+				foreach ($subscriptions as $subscription) {
 					// Subscription expires timestamp.
-					$subscription_expires = strtotime( $subscription['expires'] );
+					$subscription_expires = strtotime($subscription['expires']);
 
 					// Start showing subscriptions message before one week of renewal date.
-					if( strtotime( '- 7 days', $subscription_expires ) > current_time( 'timestamp', 1 ) ) {
+					if (strtotime('- 7 days', $subscription_expires) > current_time('timestamp', 1)) {
 						continue;
 					}
 
 					// Check if subscription message already exist in messages.
-					if( array_key_exists( $subscription['id'], $messages ) ) {
+					if (array_key_exists($subscription['id'], $messages)) {
 						continue;
 					}
 
-					if( ( ! $this->__is_notice_dismissed( $subscription['id'] ) && 'active' !== $subscription['status'] ) ) {
+					if (( ! $this->__is_notice_dismissed($subscription['id']) && 'active' !== $subscription['status'])) {
 
-						if( strtotime( $subscription['expires'] ) < current_time( 'timestamp', 1 ) ) {// Check if license already expired.
+						if (strtotime($subscription['expires']) < current_time('timestamp', 1)) {// Check if license already expired.
 							$messages[$subscription['id']] = sprintf(
-								__( 'You Give addon license expired for payment <a href="%s" target="_blank">#%d</a>. <a href="%s" target="_blank">Click to renew an existing license</a> or <a href="%s">Click here if already renewed</a>.', 'give' ),
-								urldecode( $subscription['invoice_url'] ),
+								__('You Give addon license expired for payment <a href="%s" target="_blank">#%d</a>. <a href="%s" target="_blank">Click to renew an existing license</a> or <a href="%s">Click here if already renewed</a>.', 'give'),
+								urldecode($subscription['invoice_url']),
 								$subscription['payment_id'],
 								"{$this->checkout_url}?edd_license_key={$subscription['license_key']}&utm_campaign=admin&utm_source=licenses&utm_medium=expired",
-								esc_url( add_query_arg( '_give_hide_license_notices_permanently', $subscription['id'], $_SERVER['REQUEST_URI'] ) )
+								esc_url(add_query_arg('_give_hide_license_notices_permanently', $subscription['id'], $_SERVER['REQUEST_URI']))
 							);
-						}else{
+						} else {
 							$messages[$subscription['id']] = sprintf(
-								__( 'You Give addon license will expire in %s for payment <a href="%s" target="_blank">#%d</a>. <a href="%s" target="_blank">Click to renew an existing license</a> or <a href="%s">Click here if already renewed</a>.', 'give' ),
-								human_time_diff( current_time( 'timestamp', 1 ), strtotime( $subscription['expires'] ) ),
-								urldecode( $subscription['invoice_url'] ),
+								__('You Give addon license will expire in %s for payment <a href="%s" target="_blank">#%d</a>. <a href="%s" target="_blank">Click to renew an existing license</a> or <a href="%s">Click here if already renewed</a>.', 'give'),
+								human_time_diff(current_time('timestamp', 1), strtotime($subscription['expires'])),
+								urldecode($subscription['invoice_url']),
 								$subscription['payment_id'],
 								"{$this->checkout_url}?edd_license_key={$subscription['license_key']}&utm_campaign=admin&utm_source=licenses&utm_medium=expired",
-								esc_url( add_query_arg( '_give_hide_license_notices_permanently', $subscription['id'], $_SERVER['REQUEST_URI'] ) )
+								esc_url(add_query_arg('_give_hide_license_notices_permanently', $subscription['id'], $_SERVER['REQUEST_URI']))
 							);
 						}
 					}
 
 					// Stop validation for these licencse keys.
-					$addon_license_key_in_subscriptions = array_merge( $addon_license_key_in_subscriptions, $subscription['licenses'] );
+					$addon_license_key_in_subscriptions = array_merge($addon_license_key_in_subscriptions, $subscription['licenses']);
 				}
 				$showed_subscriptions_message = true;
 			}
 
 			// Show non subscription addon messages.
-			if( ! in_array( $this->license, $addon_license_key_in_subscriptions ) && ! $this->__is_notice_dismissed( 'general' ) && ! $this->is_valid_license() && empty( $showed_invalid_message ) ) {
+			if ( ! in_array($this->license, $addon_license_key_in_subscriptions) && ! $this->__is_notice_dismissed('general') && ! $this->is_valid_license() && empty($showed_invalid_message)) {
 
 				$messages['general'] = sprintf(
-					__( 'You have invalid or expired license keys for Give Addon. Please go to the <a href="%s">licenses page</a> to correct this issue.', 'give' ),
-					admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=licenses' )
+					__('You have invalid or expired license keys for Give Addon. Please go to the <a href="%s">licenses page</a> to correct this issue.', 'give'),
+					admin_url('edit.php?post_type=give_forms&page=give-settings&tab=licenses')
 				);
 				$showed_invalid_message = true;
 
 			}
 
 			// Print messages.
-			if( ! empty( $messages ) ) {
-				foreach( $messages as $notice_id => $message ) {
-					echo '<div class="notice notice-error is-dismissible give-license-notice" data-dismiss-notice-shortly="' . esc_url( add_query_arg( '_give_hide_license_notices_shortly', $notice_id, $_SERVER['REQUEST_URI'] ) ) . '">';
-					echo '<p>' . $message . '</p>';
+			if ( ! empty($messages)) {
+				foreach ($messages as $notice_id => $message) {
+					echo '<div class="notice notice-error is-dismissible give-license-notice" data-dismiss-notice-shortly="'.esc_url(add_query_arg('_give_hide_license_notices_shortly', $notice_id, $_SERVER['REQUEST_URI'])).'">';
+					echo '<p>'.$message.'</p>';
 					echo '</div>';
 				}
 			}
@@ -774,7 +774,7 @@ 
 		 * @return bool
 		 */
 		public function is_valid_license() {
-			if( apply_filters( 'give_is_valid_license' , ( is_object( $this->license_data ) && ! empty( $this->license_data ) &&  property_exists( $this->license_data, 'license' )&& 'valid' === $this->license_data->license ) ) ) {
+			if (apply_filters('give_is_valid_license', (is_object($this->license_data) && ! empty($this->license_data) && property_exists($this->license_data, 'license') && 'valid' === $this->license_data->license))) {
 				return true;
 			}
 
@@ -790,7 +790,7 @@ 
 		 * @return bool
 		 */
 		private function __is_third_party_addon() {
-			return ( false === strpos( $this->api_url, 'givewp.com/' ) );
+			return (false === strpos($this->api_url, 'givewp.com/'));
 		}
 
 		/**
@@ -804,26 +804,26 @@ 
 		 *
 		 * @return void|bool
 		 */
-		private function __remove_license_key_from_subscriptions(){
-			$subscriptions = get_option( 'give_subscriptions', array() );
+		private function __remove_license_key_from_subscriptions() {
+			$subscriptions = get_option('give_subscriptions', array());
 
 			// Bailout.
-			if( empty( $this->license ) ) {
+			if (empty($this->license)) {
 				return false;
 			}
 
-			if( ! empty( $subscriptions ) ) {
-				foreach ( $subscriptions as $subscription_id => $subscription ) {
-					$license_index = array_search( $this->license, $subscription['licenses'] );
-					if( false !== $license_index ) {
+			if ( ! empty($subscriptions)) {
+				foreach ($subscriptions as $subscription_id => $subscription) {
+					$license_index = array_search($this->license, $subscription['licenses']);
+					if (false !== $license_index) {
 						// Remove license key.
-						unset( $subscriptions[ $subscription_id ]['licenses'][$license_index] );
+						unset($subscriptions[$subscription_id]['licenses'][$license_index]);
 
 						// Rearrange license keys.
-						$subscriptions[ $subscription_id ]['licenses'] = array_values( $subscriptions[ $subscription_id ]['licenses'] );
+						$subscriptions[$subscription_id]['licenses'] = array_values($subscriptions[$subscription_id]['licenses']);
 
 						// Update subscription information.
-						update_option( 'give_subscriptions', $subscriptions );
+						update_option('give_subscriptions', $subscriptions);
 						break;
 					}
 				}
@@ -838,7 +838,7 @@ 
 		 *
 		 * @return void
 		 */
-		private function __remove_license_notices_show_blocker(){
+		private function __remove_license_notices_show_blocker() {
 			global $wpdb;
 
 			// Delete permanent notice blocker.
@@ -876,18 +876,18 @@ 
 		 *
 		 * @return bool
 		 */
-		private function __is_notice_dismissed( $notice_id ){
+		private function __is_notice_dismissed($notice_id) {
 			$current_user = wp_get_current_user();
 			$is_notice_dismissed = false;
 
 			// Ge is notice dismissed permanently.
-			$already_dismiss_notices = ( $already_dismiss_notices = get_user_meta( $current_user->ID, '_give_hide_license_notices_permanently', true ) )
+			$already_dismiss_notices = ($already_dismiss_notices = get_user_meta($current_user->ID, '_give_hide_license_notices_permanently', true))
 				? $already_dismiss_notices
 				: array();
 
 
-			if( in_array( $notice_id, $already_dismiss_notices ) || get_transient( "_give_hide_license_notices_shortly_{$current_user->ID}_{$notice_id}" ) ) {
-				$is_notice_dismissed =  true;
+			if (in_array($notice_id, $already_dismiss_notices) || get_transient("_give_hide_license_notices_shortly_{$current_user->ID}_{$notice_id}")) {
+				$is_notice_dismissed = true;
 			}
 
 			return $is_notice_dismissed;

includes/gateways/actions.php

Spacing +13 added lines, -13 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,14 +21,14 @@ 
  *
  * @param $data
  */
-function give_process_gateway_select( $data ) {
-	if ( isset( $_POST['gateway_submit'] ) ) {
-		wp_redirect( esc_url( add_query_arg( 'payment-mode', $_POST['payment-mode'] ) ) );
+function give_process_gateway_select($data) {
+	if (isset($_POST['gateway_submit'])) {
+		wp_redirect(esc_url(add_query_arg('payment-mode', $_POST['payment-mode'])));
 		exit;
 	}
 }
 
-add_action( 'give_gateway_select', 'give_process_gateway_select' );
+add_action('give_gateway_select', 'give_process_gateway_select');
 
 /**
  * Loads a payment gateway via AJAX
@@ -38,20 +38,20 @@ 
  * @return void
  */
 function give_load_ajax_gateway() {
-	if ( isset( $_POST['give_payment_mode'] ) ) {
+	if (isset($_POST['give_payment_mode'])) {
 		/**
 		 * Fire to render donation form.
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form', $_POST['give_form_id'] );
+		do_action('give_donation_form', $_POST['give_form_id']);
 
 		exit();
 	}
 }
 
-add_action( 'wp_ajax_give_load_gateway', 'give_load_ajax_gateway' );
-add_action( 'wp_ajax_nopriv_give_load_gateway', 'give_load_ajax_gateway' );
+add_action('wp_ajax_give_load_gateway', 'give_load_ajax_gateway');
+add_action('wp_ajax_nopriv_give_load_gateway', 'give_load_ajax_gateway');
 
 /**
  * Sets an error on checkout if no gateways are enabled
@@ -63,11 +63,11 @@ 
 function give_no_gateway_error() {
 	$gateways = give_get_enabled_payment_gateways();
 
-	if ( empty( $gateways ) ) {
-		give_set_error( 'no_gateways', esc_html__( 'You must enable a payment gateway to use Give.', 'give' ) );
+	if (empty($gateways)) {
+		give_set_error('no_gateways', esc_html__('You must enable a payment gateway to use Give.', 'give'));
 	} else {
-		give_unset_error( 'no_gateways' );
+		give_unset_error('no_gateways');
 	}
 }
 
-add_action( 'init', 'give_no_gateway_error' );
+add_action('init', 'give_no_gateway_error');

includes/payments/class-payment-stats.php

Spacing +59 added lines, -59 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -38,35 +38,35 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_sales( $form_id = 0, $start_date = false, $end_date = false, $status = 'publish' ) {
+	public function get_sales($form_id = 0, $start_date = false, $end_date = false, $status = 'publish') {
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
-		if ( empty( $form_id ) ) {
+		if (empty($form_id)) {
 
 			// Global sale stats
-			add_filter( 'give_count_payments_where', array( $this, 'count_where' ) );
+			add_filter('give_count_payments_where', array($this, 'count_where'));
 
-			if ( is_array( $status ) ) {
+			if (is_array($status)) {
 				$count = 0;
-				foreach ( $status as $payment_status ) {
+				foreach ($status as $payment_status) {
 					$count += give_count_payments()->$payment_status;
 				}
 			} else {
 				$count = give_count_payments()->$status;
 			}
 
-			remove_filter( 'give_count_payments_where', array( $this, 'count_where' ) );
+			remove_filter('give_count_payments_where', array($this, 'count_where'));
 
 		} else {
 
@@ -75,11 +75,11 @@ 
 			// Product specific stats
 			global $give_logs;
 
-			add_filter( 'posts_where', array( $this, 'payments_where' ) );
+			add_filter('posts_where', array($this, 'payments_where'));
 
-			$count = $give_logs->get_log_count( $form_id, 'sale' );
+			$count = $give_logs->get_log_count($form_id, 'sale');
 
-			remove_filter( 'posts_where', array( $this, 'payments_where' ) );
+			remove_filter('posts_where', array($this, 'payments_where'));
 
 		}
 
@@ -101,31 +101,31 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_earnings( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
+	public function get_earnings($form_id = 0, $start_date = false, $end_date = false, $gateway_id = false) {
 
 		global $wpdb;
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
-		add_filter( 'posts_where', array( $this, 'payments_where' ) );
+		add_filter('posts_where', array($this, 'payments_where'));
 
-		if ( empty( $form_id ) ) {
+		if (empty($form_id)) {
 
 			// Global earning stats
 			$args = array(
 				'post_type'              => 'give_payment',
 				'nopaging'               => true,
-				'post_status'            => array( 'publish' ),
+				'post_status'            => array('publish'),
 				'fields'                 => 'ids',
 				'update_post_term_cache' => false,
 				'suppress_filters'       => false,
@@ -137,24 +137,24 @@ 
 			);
 
 			//Filter by Gateway ID meta_key
-			if ( $gateway_id !== false ) {
+			if ($gateway_id !== false) {
 				$args['meta_key']   = '_give_payment_gateway';
 				$args['meta_value'] = $gateway_id;
 			}
 
-			$args = apply_filters( 'give_stats_earnings_args', $args );
-			$key  = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
-			$earnings = get_transient( $key );
+			$args = apply_filters('give_stats_earnings_args', $args);
+			$key  = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
+			$earnings = get_transient($key);
 			
-			if ( false === $earnings ) {
-				$sales    = get_posts( $args );
+			if (false === $earnings) {
+				$sales    = get_posts($args);
 				$earnings = 0;
-				if ( $sales ) {
-					$sales = implode( ',', array_map('intval', $sales ) );
-					$earnings += $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$sales})" );
+				if ($sales) {
+					$sales = implode(',', array_map('intval', $sales));
+					$earnings += $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$sales})");
 				}
 				// Cache the results for one hour
-				set_transient( $key, $earnings, HOUR_IN_SECONDS );
+				set_transient($key, $earnings, HOUR_IN_SECONDS);
 			}
 
 		} else {
@@ -174,37 +174,37 @@ 
 				'give_transient_type' => 'give_earnings',
 				// This is not a valid query arg, but is used for cache keying
 			);
-			$args = apply_filters( 'give_stats_earnings_args', $args );
-			$key  = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+			$args = apply_filters('give_stats_earnings_args', $args);
+			$key  = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 			//Set transient for faster stats
-			$earnings = get_transient( $key );
+			$earnings = get_transient($key);
 
-			if ( false === $earnings ) {
+			if (false === $earnings) {
 
 				$this->timestamp = false;
-				$log_ids  = $give_logs->get_connected_logs( $args, 'sale' );
+				$log_ids  = $give_logs->get_connected_logs($args, 'sale');
 				$earnings = 0;
 
-				if ( $log_ids ) {
-					$log_ids     = implode( ',', array_map('intval', $log_ids ) );
-					$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids);" );
+				if ($log_ids) {
+					$log_ids     = implode(',', array_map('intval', $log_ids));
+					$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids);");
 
-					foreach ( $payment_ids as $payment_id ) {
-						$earnings += give_get_payment_amount( $payment_id );
+					foreach ($payment_ids as $payment_id) {
+						$earnings += give_get_payment_amount($payment_id);
 					}
 					
 				}
 
 				// Cache the results for one hour
-				set_transient( $key, $earnings, 60 * 60 );
+				set_transient($key, $earnings, 60 * 60);
 			}
 		}
 
 		//remove our filter
-		remove_filter( 'posts_where', array( $this, 'payments_where' ) );
+		remove_filter('posts_where', array($this, 'payments_where'));
 
 		//return earnings
-		return round( $earnings, give_currency_decimal_filter() );
+		return round($earnings, give_currency_decimal_filter());
 
 	}
 
@@ -221,29 +221,29 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_earnings_cache_key( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
+	public function get_earnings_cache_key($form_id = 0, $start_date = false, $end_date = false, $gateway_id = false) {
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
-		add_filter( 'posts_where', array( $this, 'payments_where' ) );
+		add_filter('posts_where', array($this, 'payments_where'));
 
-		if ( empty( $form_id ) ) {
+		if (empty($form_id)) {
 
 			// Global earning stats
 			$args = array(
 				'post_type'              => 'give_payment',
 				'nopaging'               => true,
-				'post_status'            => array( 'publish' ),
+				'post_status'            => array('publish'),
 				'fields'                 => 'ids',
 				'update_post_term_cache' => false,
 				'suppress_filters'       => false,
@@ -255,13 +255,13 @@ 
 			);
 
 			//Filter by Gateway ID meta_key
-			if ( $gateway_id !== false ) {
+			if ($gateway_id !== false) {
 				$args['meta_key']   = '_give_payment_gateway';
 				$args['meta_value'] = $gateway_id;
 			}
 
-			$args = apply_filters( 'give_stats_earnings_args', $args );
-			$key  = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+			$args = apply_filters('give_stats_earnings_args', $args);
+			$key  = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 
 		} else {
 
@@ -280,12 +280,12 @@ 
 				'give_transient_type' => 'give_earnings',
 				// This is not a valid query arg, but is used for cache keying
 			);
-			$args = apply_filters( 'give_stats_earnings_args', $args );
-			$key  = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+			$args = apply_filters('give_stats_earnings_args', $args);
+			$key  = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 		}
 
 		//remove our filter
-		remove_filter( 'posts_where', array( $this, 'payments_where' ) );
+		remove_filter('posts_where', array($this, 'payments_where'));
 
 		//return earnings
 		return $key;
@@ -302,16 +302,16 @@ 
 	 *
 	 * @return array       Best selling forms
 	 */
-	public function get_best_selling( $number = 10 ) {
+	public function get_best_selling($number = 10) {
 
 		global $wpdb;
 
-		$give_forms = $wpdb->get_results( $wpdb->prepare(
+		$give_forms = $wpdb->get_results($wpdb->prepare(
 			"SELECT post_id as form_id, max(meta_value) as sales
 				FROM $wpdb->postmeta WHERE meta_key='_give_form_sales' AND meta_value > 0
 				GROUP BY meta_value+0
 				DESC LIMIT %d;", $number
-		) );
+		));
 
 		return $give_forms;
 	}

includes/admin/reporting/export/class-batch-export-forms.php

Spacing +53 added lines, -53 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -41,20 +41,20 @@ 
 	public function csv_cols() {
 
 		$cols = array(
-			'ID'                      => esc_html__( 'ID', 'give' ),
-			'post_name'               => esc_html__( 'Slug', 'give' ),
-			'post_title'              => esc_html__( 'Name', 'give' ),
-			'post_date'               => esc_html__( 'Date Created', 'give' ),
-			'post_author'             => esc_html__( 'Author', 'give' ),
-			'post_content'            => esc_html__( 'Description', 'give' ),
-			'post_excerpt'            => esc_html__( 'Excerpt', 'give' ),
-			'post_status'             => esc_html__( 'Status', 'give' ),
-			'categories'              => esc_html__( 'Categories', 'give' ),
-			'tags'                    => esc_html__( 'Tags', 'give' ),
-			'give_price'              => esc_html__( 'Price', 'give' ),
-			'_thumbnail_id'           => esc_html__( 'Featured Image', 'give' ),
-			'_give_form_sales'        => esc_html__( 'Donations', 'give' ),
-			'_give_download_earnings' => esc_html__( 'Income', 'give' ),
+			'ID'                      => esc_html__('ID', 'give'),
+			'post_name'               => esc_html__('Slug', 'give'),
+			'post_title'              => esc_html__('Name', 'give'),
+			'post_date'               => esc_html__('Date Created', 'give'),
+			'post_author'             => esc_html__('Author', 'give'),
+			'post_content'            => esc_html__('Description', 'give'),
+			'post_excerpt'            => esc_html__('Excerpt', 'give'),
+			'post_status'             => esc_html__('Status', 'give'),
+			'categories'              => esc_html__('Categories', 'give'),
+			'tags'                    => esc_html__('Tags', 'give'),
+			'give_price'              => esc_html__('Price', 'give'),
+			'_thumbnail_id'           => esc_html__('Featured Image', 'give'),
+			'_give_form_sales'        => esc_html__('Donations', 'give'),
+			'_give_download_earnings' => esc_html__('Income', 'give'),
 		);
 
 		return $cols;
@@ -86,43 +86,43 @@ 
 			'paged'          => $this->step
 		);
 
-		$downloads = new WP_Query( $args );
+		$downloads = new WP_Query($args);
 
-		if ( $downloads->posts ) {
-			foreach ( $downloads->posts as $download ) {
+		if ($downloads->posts) {
+			foreach ($downloads->posts as $download) {
 
 				$row = array();
 
-				foreach ( $this->csv_cols() as $key => $value ) {
+				foreach ($this->csv_cols() as $key => $value) {
 
 					// Setup default value
-					$row[ $key ] = '';
+					$row[$key] = '';
 
-					if ( in_array( $key, $meta ) ) {
+					if (in_array($key, $meta)) {
 
-						switch ( $key ) {
+						switch ($key) {
 
 							case '_thumbnail_id' :
 
-								$image_id    = get_post_thumbnail_id( $download->ID );
-								$row[ $key ] = wp_get_attachment_url( $image_id );
+								$image_id    = get_post_thumbnail_id($download->ID);
+								$row[$key] = wp_get_attachment_url($image_id);
 
 								break;
 
 							case 'give_price' :
 
-								if ( give_has_variable_prices( $download->ID ) ) {
+								if (give_has_variable_prices($download->ID)) {
 
 									$prices = array();
-									foreach ( give_get_variable_prices( $download->ID ) as $price ) {
-										$prices[] = $price['name'] . ': ' . $price['amount'];
+									foreach (give_get_variable_prices($download->ID) as $price) {
+										$prices[] = $price['name'].': '.$price['amount'];
 									}
 
-									$row[ $key ] = implode( ' | ', $prices );
+									$row[$key] = implode(' | ', $prices);
 
 								} else {
 
-									$row[ $key ] = give_get_download_price( $download->ID );
+									$row[$key] = give_get_download_price($download->ID);
 
 								}
 
@@ -132,54 +132,54 @@ 
 
 
 								$files = array();
-								foreach ( give_get_download_files( $download->ID ) as $file ) {
+								foreach (give_get_download_files($download->ID) as $file) {
 									$files[] = $file['file'];
 								}
 
-								$row[ $key ] = implode( ' | ', $files );
+								$row[$key] = implode(' | ', $files);
 
 								break;
 
 							default :
 
-								$row[ $key ] = get_post_meta( $download->ID, $key, true );
+								$row[$key] = get_post_meta($download->ID, $key, true);
 
 								break;
 
 						}
 
-					} elseif ( isset( $download->$key ) ) {
+					} elseif (isset($download->$key)) {
 
-						switch ( $key ) {
+						switch ($key) {
 
 							case 'post_author' :
 
-								$row[ $key ] = get_the_author_meta( 'user_login', $download->post_author );
+								$row[$key] = get_the_author_meta('user_login', $download->post_author);
 
 								break;
 
 							default :
 
-								$row[ $key ] = $download->$key;
+								$row[$key] = $download->$key;
 
 								break;
 						}
 
-					} elseif ( 'tags' == $key ) {
+					} elseif ('tags' == $key) {
 
-						$terms = get_the_terms( $download->ID, 'download_tag' );
-						if ( $terms ) {
-							$terms       = wp_list_pluck( $terms, 'name' );
-							$row[ $key ] = implode( ' | ', $terms );
+						$terms = get_the_terms($download->ID, 'download_tag');
+						if ($terms) {
+							$terms       = wp_list_pluck($terms, 'name');
+							$row[$key] = implode(' | ', $terms);
 						}
 
 
-					} elseif ( 'categories' == $key ) {
+					} elseif ('categories' == $key) {
 
-						$terms = get_the_terms( $download->ID, 'download_category' );
-						if ( $terms ) {
-							$terms       = wp_list_pluck( $terms, 'name' );
-							$row[ $key ] = implode( ' | ', $terms );
+						$terms = get_the_terms($download->ID, 'download_category');
+						if ($terms) {
+							$terms       = wp_list_pluck($terms, 'name');
+							$row[$key] = implode(' | ', $terms);
 						}
 
 					}
@@ -190,8 +190,8 @@ 
 
 			}
 
-			$data = apply_filters( 'give_export_get_data', $data );
-			$data = apply_filters( "give_export_get_data_{$this->export_type}", $data );
+			$data = apply_filters('give_export_get_data', $data);
+			$data = apply_filters("give_export_get_data_{$this->export_type}", $data);
 
 			return $data;
 		}
@@ -210,20 +210,20 @@ 
 
 		$args = array(
 			'post_type'      => 'give_forms',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'any',
 			'fields'         => 'ids',
 		);
 
-		$downloads  = new WP_Query( $args );
+		$downloads  = new WP_Query($args);
 		$total      = (int) $downloads->post_count;
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( 30 * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = ((30 * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 

includes/admin/reporting/export/class-batch-export-payments.php

Spacing +63 added lines, -63 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -39,30 +39,30 @@ 
 	 */
 	public function csv_cols() {
 		$cols = array(
-			'id'        => esc_html__( 'ID', 'give' ), // unaltered payment ID (use for querying).
-			'seq_id'    => esc_html__( 'Payment Number', 'give' ), // sequential payment ID.
-			'email'     => esc_html__( 'Email', 'give' ),
-			'first'     => esc_html__( 'First Name', 'give' ),
-			'last'      => esc_html__( 'Last Name', 'give' ),
-			'address1'  => esc_html__( 'Address 1', 'give' ),
-			'address2'  => esc_html__( 'Address 2', 'give' ),
-			'city'      => esc_html__( 'City', 'give' ),
-			'state'     => esc_html__( 'State', 'give' ),
-			'country'   => esc_html__( 'Country', 'give' ),
-			'zip'       => esc_html__( 'Zip / Postal Code', 'give' ),
-			'form_id'   => esc_html__( 'Form ID', 'give' ),
-			'form_name' => esc_html__( 'Form Name', 'give' ),
-			'amount'    => esc_html__( 'Amount', 'give' ) . ' (' . html_entity_decode( give_currency_filter( '' ) ) . ')',
-			'gateway'   => esc_html__( 'Payment Method', 'give' ),
-			'trans_id'  => esc_html__( 'Transaction ID', 'give' ),
-			'key'       => esc_html__( 'Key', 'give' ),
-			'date'      => esc_html__( 'Date', 'give' ),
-			'user'      => esc_html__( 'User', 'give' ),
-			'status'    => esc_html__( 'Status', 'give' )
+			'id'        => esc_html__('ID', 'give'), // unaltered payment ID (use for querying).
+			'seq_id'    => esc_html__('Payment Number', 'give'), // sequential payment ID.
+			'email'     => esc_html__('Email', 'give'),
+			'first'     => esc_html__('First Name', 'give'),
+			'last'      => esc_html__('Last Name', 'give'),
+			'address1'  => esc_html__('Address 1', 'give'),
+			'address2'  => esc_html__('Address 2', 'give'),
+			'city'      => esc_html__('City', 'give'),
+			'state'     => esc_html__('State', 'give'),
+			'country'   => esc_html__('Country', 'give'),
+			'zip'       => esc_html__('Zip / Postal Code', 'give'),
+			'form_id'   => esc_html__('Form ID', 'give'),
+			'form_name' => esc_html__('Form Name', 'give'),
+			'amount'    => esc_html__('Amount', 'give').' ('.html_entity_decode(give_currency_filter('')).')',
+			'gateway'   => esc_html__('Payment Method', 'give'),
+			'trans_id'  => esc_html__('Transaction ID', 'give'),
+			'key'       => esc_html__('Key', 'give'),
+			'date'      => esc_html__('Date', 'give'),
+			'user'      => esc_html__('User', 'give'),
+			'status'    => esc_html__('Status', 'give')
 		);
 
-		if ( ! give_get_option( 'enable_sequential' ) ) {
-			unset( $cols['seq_id'] );
+		if ( ! give_get_option('enable_sequential')) {
+			unset($cols['seq_id']);
 		}
 
 		return $cols;
@@ -87,12 +87,12 @@ 
 			'status' => $this->status
 		);
 
-		if ( ! empty( $this->start ) || ! empty( $this->end ) ) {
+		if ( ! empty($this->start) || ! empty($this->end)) {
 
 			$args['date_query'] = array(
 				array(
-					'after'     => date( 'Y-n-d 00:00:00', strtotime( $this->start ) ),
-					'before'    => date( 'Y-n-d 23:59:59', strtotime( $this->end ) ),
+					'after'     => date('Y-n-d 00:00:00', strtotime($this->start)),
+					'before'    => date('Y-n-d 23:59:59', strtotime($this->end)),
 					'inclusive' => true
 				)
 			);
@@ -101,52 +101,52 @@ 
 
 		//echo json_encode($args ); exit;
 
-		$payments = give_get_payments( $args );
+		$payments = give_get_payments($args);
 
-		if ( $payments ) {
+		if ($payments) {
 
-			foreach ( $payments as $payment ) {
-				$payment_meta = give_get_payment_meta( $payment->ID );
-				$user_info    = give_get_payment_meta_user_info( $payment->ID );
-				$total        = give_get_payment_amount( $payment->ID );
-				$user_id      = isset( $user_info['id'] ) && $user_info['id'] != - 1 ? $user_info['id'] : $user_info['email'];
+			foreach ($payments as $payment) {
+				$payment_meta = give_get_payment_meta($payment->ID);
+				$user_info    = give_get_payment_meta_user_info($payment->ID);
+				$total        = give_get_payment_amount($payment->ID);
+				$user_id      = isset($user_info['id']) && $user_info['id'] != - 1 ? $user_info['id'] : $user_info['email'];
 				$products     = '';
 				$skus         = '';
 
-				if ( is_numeric( $user_id ) ) {
-					$user = get_userdata( $user_id );
+				if (is_numeric($user_id)) {
+					$user = get_userdata($user_id);
 				} else {
 					$user = false;
 				}
 
 				$data[] = array(
 					'id'        => $payment->ID,
-					'seq_id'    => give_get_payment_number( $payment->ID ),
+					'seq_id'    => give_get_payment_number($payment->ID),
 					'email'     => $payment_meta['email'],
 					'first'     => $user_info['first_name'],
 					'last'      => $user_info['last_name'],
-					'address1'  => isset( $user_info['address']['line1'] ) ? $user_info['address']['line1'] : '',
-					'address2'  => isset( $user_info['address']['line2'] ) ? $user_info['address']['line2'] : '',
-					'city'      => isset( $user_info['address']['city'] ) ? $user_info['address']['city'] : '',
-					'state'     => isset( $user_info['address']['state'] ) ? $user_info['address']['state'] : '',
-					'country'   => isset( $user_info['address']['country'] ) ? $user_info['address']['country'] : '',
-					'zip'       => isset( $user_info['address']['zip'] ) ? $user_info['address']['zip'] : '',
-					'form_id'   => isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : '',
-					'form_name' => isset( $payment_meta['form_title'] ) ? $payment_meta['form_title'] : '',
+					'address1'  => isset($user_info['address']['line1']) ? $user_info['address']['line1'] : '',
+					'address2'  => isset($user_info['address']['line2']) ? $user_info['address']['line2'] : '',
+					'city'      => isset($user_info['address']['city']) ? $user_info['address']['city'] : '',
+					'state'     => isset($user_info['address']['state']) ? $user_info['address']['state'] : '',
+					'country'   => isset($user_info['address']['country']) ? $user_info['address']['country'] : '',
+					'zip'       => isset($user_info['address']['zip']) ? $user_info['address']['zip'] : '',
+					'form_id'   => isset($payment_meta['form_id']) ? $payment_meta['form_id'] : '',
+					'form_name' => isset($payment_meta['form_title']) ? $payment_meta['form_title'] : '',
 					'skus'      => $skus,
-					'amount'    => html_entity_decode( give_format_amount( $total ) ),
-					'gateway'   => give_get_gateway_admin_label( get_post_meta( $payment->ID, '_give_payment_gateway', true ) ),
-					'trans_id'  => give_get_payment_transaction_id( $payment->ID ),
+					'amount'    => html_entity_decode(give_format_amount($total)),
+					'gateway'   => give_get_gateway_admin_label(get_post_meta($payment->ID, '_give_payment_gateway', true)),
+					'trans_id'  => give_get_payment_transaction_id($payment->ID),
 					'key'       => $payment_meta['key'],
 					'date'      => $payment->post_date,
-					'user'      => $user ? $user->display_name : __( 'guest', 'give' ),
-					'status'    => give_get_payment_status( $payment, true )
+					'user'      => $user ? $user->display_name : __('guest', 'give'),
+					'status'    => give_get_payment_status($payment, true)
 				);
 
 			}
 
-			$data = apply_filters( 'give_export_get_data', $data );
-			$data = apply_filters( "give_export_get_data_{$this->export_type}", $data );
+			$data = apply_filters('give_export_get_data', $data);
+			$data = apply_filters("give_export_get_data_{$this->export_type}", $data);
 
 			return $data;
 
@@ -166,27 +166,27 @@ 
 
 		$status = $this->status;
 		$args   = array(
-			'start-date' => date( 'n/d/Y', strtotime( $this->start ) ),
-			'end-date'   => date( 'n/d/Y', strtotime( $this->end ) ),
+			'start-date' => date('n/d/Y', strtotime($this->start)),
+			'end-date'   => date('n/d/Y', strtotime($this->end)),
 		);
 
-		if ( 'any' == $status ) {
+		if ('any' == $status) {
 
-			$total = array_sum( (array) give_count_payments( $args ) );
+			$total = array_sum((array) give_count_payments($args));
 
 		} else {
 
-			$total = give_count_payments( $args )->$status;
+			$total = give_count_payments($args)->$status;
 
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( 30 * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = ((30 * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -200,9 +200,9 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing.
 	 */
-	public function set_properties( $request ) {
-		$this->start  = isset( $request['start'] ) ? sanitize_text_field( $request['start'] ) : '';
-		$this->end    = isset( $request['end'] ) ? sanitize_text_field( $request['end'] ) : '';
-		$this->status = isset( $request['status'] ) ? sanitize_text_field( $request['status'] ) : 'complete';
+	public function set_properties($request) {
+		$this->start  = isset($request['start']) ? sanitize_text_field($request['start']) : '';
+		$this->end    = isset($request['end']) ? sanitize_text_field($request['end']) : '';
+		$this->status = isset($request['status']) ? sanitize_text_field($request['status']) : 'complete';
 	}
 }