WordImpress / Give

includes/api/class-give-api-v1.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 

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 within the donation form 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/admin/settings/class-settings-gateways.php

Indentation +21 added lines, -21 removed lines

@@ -86,13 +86,13 @@ 
 								'disabled' => __( 'Disabled', 'give' ),
 							)
 						),
-                        array(
-                            'name'  => __( 'PayPal Standard Gateway Settings Docs Link', 'give' ),
-                            'id'    => 'paypal_standard_gateway_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-gateway-paypal-standard' ),
-                            'title' => __( 'PayPal Standard Gateway Settings', 'give' ),
-                            'type'  => 'give_docs_link',
-                        ),
+						array(
+							'name'  => __( 'PayPal Standard Gateway Settings Docs Link', 'give' ),
+							'id'    => 'paypal_standard_gateway_settings_docs_link',
+							'url'   => esc_url( 'http://docs.givewp.com/settings-gateway-paypal-standard' ),
+							'title' => __( 'PayPal Standard Gateway Settings', 'give' ),
+							'type'  => 'give_docs_link',
+						),
 						array(
 							'type' => 'sectionend',
 							'id'   => 'give_title_gateway_settings_2',
@@ -145,13 +145,13 @@ 
 								'textarea_rows' => 6,
 							)
 						),
-                        array(
-                            'name'  => __( 'Offline Donations Settings Docs Link', 'give' ),
-                            'id'    => 'offline_gateway_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-gateway-offline-donations' ),
-                            'title' => __( 'Offline Gateway Settings', 'give' ),
-                            'type'  => 'give_docs_link',
-                        ),
+						array(
+							'name'  => __( 'Offline Donations Settings Docs Link', 'give' ),
+							'id'    => 'offline_gateway_settings_docs_link',
+							'url'   => esc_url( 'http://docs.givewp.com/settings-gateway-offline-donations' ),
+							'title' => __( 'Offline Gateway Settings', 'give' ),
+							'type'  => 'give_docs_link',
+						),
 						array(
 							'type' => 'sectionend',
 							'id'   => 'give_title_gateway_settings_3',
@@ -189,13 +189,13 @@ 
 							'id'   => 'default_gateway',
 							'type' => 'default_gateway'
 						),
-                        array(
-                            'name'  => __( 'Gateways Docs Link', 'give' ),
-                            'id'    => 'gateway_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-gateways' ),
-                            'title' => __( 'Gateway Settings', 'give' ),
-                            'type'  => 'give_docs_link',
-                        ),
+						array(
+							'name'  => __( 'Gateways Docs Link', 'give' ),
+							'id'    => 'gateway_settings_docs_link',
+							'url'   => esc_url( 'http://docs.givewp.com/settings-gateways' ),
+							'title' => __( 'Gateway Settings', 'give' ),
+							'type'  => 'give_docs_link',
+						),
 						array(
 							'id'   => 'give_title_gateway_settings_1',
 							'type' => 'sectionend'

Spacing +54 added lines, -54 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Gateways' ) ) :
+if ( ! class_exists('Give_Settings_Gateways')) :
 
 	/**
 	 * Give_Settings_Gateways.
@@ -27,7 +27,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'gateways';
-			$this->label = esc_html__( 'Payment Gateways', 'give' );
+			$this->label = esc_html__('Payment Gateways', 'give');
 
 			$this->default_tab = 'gateways-settings';
 
@@ -44,7 +44,7 @@ 
 			$settings        = array();
 			$current_section = give_get_current_setting_section();
 
-			switch ( $current_section ) {
+			switch ($current_section) {
 				case 'paypal-standard':
 					$settings = array(
 						// Section 2: PayPal Standard.
@@ -53,55 +53,55 @@ 
 							'id'   => 'give_title_gateway_settings_2',
 						),
 						array(
-							'name' => __( 'PayPal Email', 'give' ),
-							'desc' => __( 'Enter your PayPal account\'s email.', 'give' ),
+							'name' => __('PayPal Email', 'give'),
+							'desc' => __('Enter your PayPal account\'s email.', 'give'),
 							'id'   => 'paypal_email',
 							'type' => 'email',
 						),
 						array(
-							'name' => __( 'PayPal Page Style', 'give' ),
-							'desc' => __( 'Enter the name of the PayPal page style to use, or leave blank to use the default.', 'give' ),
+							'name' => __('PayPal Page Style', 'give'),
+							'desc' => __('Enter the name of the PayPal page style to use, or leave blank to use the default.', 'give'),
 							'id'   => 'paypal_page_style',
 							'type' => 'text',
 						),
 						array(
-							'name'    => __( 'PayPal Transaction Type', 'give' ),
-							'desc'    => __( 'Nonprofits must verify their status to withdraw donations they receive via PayPal. PayPal users that are not verified nonprofits must demonstrate how their donations will be used, once they raise more than $10,000. By default, Give transactions are sent to PayPal as donations. You may change the transaction type using this option if you feel you may not meet PayPal\'s donation requirements.', 'give' ),
+							'name'    => __('PayPal Transaction Type', 'give'),
+							'desc'    => __('Nonprofits must verify their status to withdraw donations they receive via PayPal. PayPal users that are not verified nonprofits must demonstrate how their donations will be used, once they raise more than $10,000. By default, Give transactions are sent to PayPal as donations. You may change the transaction type using this option if you feel you may not meet PayPal\'s donation requirements.', 'give'),
 							'id'      => 'paypal_button_type',
 							'type'    => 'radio_inline',
 							'options' => array(
-								'donation' => __( 'Donation', 'give' ),
-								'standard' => __( 'Standard Transaction', 'give' )
+								'donation' => __('Donation', 'give'),
+								'standard' => __('Standard Transaction', 'give')
 							),
 							'default' => 'donation',
 						),
 						array(
-							'name'    => __( 'Billing Details', 'give' ),
-							'desc'    => __( 'This option will enable the billing details section for PayPal Standard which requires the donor\'s address to complete the donation. These fields are not required by PayPal to process the transaction, but you may have a need to collect the data.', 'give' ),
+							'name'    => __('Billing Details', 'give'),
+							'desc'    => __('This option will enable the billing details section for PayPal Standard which requires the donor\'s address to complete the donation. These fields are not required by PayPal to process the transaction, but you may have a need to collect the data.', 'give'),
 							'id'      => 'paypal_standard_billing_details',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							)
 						),
 						array(
-							'name'    => __( 'PayPal IPN Verification', 'give' ),
-							'desc'    => __( 'If donations are not getting marked as complete, use a slightly less secure method of verifying donations.', 'give' ),
+							'name'    => __('PayPal IPN Verification', 'give'),
+							'desc'    => __('If donations are not getting marked as complete, use a slightly less secure method of verifying donations.', 'give'),
 							'id'      => 'paypal_verification',
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							)
 						),
                         array(
-                            'name'  => __( 'PayPal Standard Gateway Settings Docs Link', 'give' ),
+                            'name'  => __('PayPal Standard Gateway Settings Docs Link', 'give'),
                             'id'    => 'paypal_standard_gateway_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-gateway-paypal-standard' ),
-                            'title' => __( 'PayPal Standard Gateway Settings', 'give' ),
+                            'url'   => esc_url('http://docs.givewp.com/settings-gateway-paypal-standard'),
+                            'title' => __('PayPal Standard Gateway Settings', 'give'),
                             'type'  => 'give_docs_link',
                         ),
 						array(
@@ -119,19 +119,19 @@ 
 							'id'   => 'give_title_gateway_settings_3',
 						),
 						array(
-							'name'    => __( 'Collect Billing Details', 'give' ),
-							'desc'    => __( 'Enable to request billing details for offline donations. Will appear above offline donation instructions. Can be enabled/disabled per form.', 'give' ),
+							'name'    => __('Collect Billing Details', 'give'),
+							'desc'    => __('Enable to request billing details for offline donations. Will appear above offline donation instructions. Can be enabled/disabled per form.', 'give'),
 							'id'      => 'give_offline_donation_enable_billing_fields',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' )
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give')
 							)
 						),
 						array(
-							'name'    => __( 'Offline Donation Instructions', 'give' ),
-							'desc'    => __( 'The following content will appear for all forms when the user selects the offline donation payment option. Note: You may customize the content per form as needed.', 'give' ),
+							'name'    => __('Offline Donation Instructions', 'give'),
+							'desc'    => __('The following content will appear for all forms when the user selects the offline donation payment option. Note: You may customize the content per form as needed.', 'give'),
 							'id'      => 'global_offline_donation_content',
 							'default' => give_get_default_offline_donation_content(),
 							'type'    => 'wysiwyg',
@@ -140,15 +140,15 @@ 
 							)
 						),
 						array(
-							'name'    => __( 'Offline Donation Email Instructions Subject', 'give' ),
-							'desc'    => __( 'Enter the subject line for the donation receipt email.', 'give' ),
+							'name'    => __('Offline Donation Email Instructions Subject', 'give'),
+							'desc'    => __('Enter the subject line for the donation receipt email.', 'give'),
 							'id'      => 'offline_donation_subject',
-							'default' => esc_attr__( '{donation} - Offline Donation Instructions', 'give' ),
+							'default' => esc_attr__('{donation} - Offline Donation Instructions', 'give'),
 							'type'    => 'text'
 						),
 						array(
-							'name'    => __( 'Offline Donation Email Instructions', 'give' ),
-							'desc'    => __( 'Enter the instructions you want emailed to the donor after they have submitted the donation form. Most likely this would include important information like mailing address and who to make the check out to.', 'give' ) . ' ' . __( 'Available template tags:', 'give' ) . give_get_emails_tags_list(),
+							'name'    => __('Offline Donation Email Instructions', 'give'),
+							'desc'    => __('Enter the instructions you want emailed to the donor after they have submitted the donation form. Most likely this would include important information like mailing address and who to make the check out to.', 'give').' '.__('Available template tags:', 'give').give_get_emails_tags_list(),
 							'id'      => 'global_offline_donation_email',
 							'default' => give_get_default_offline_donation_email_content(),
 							'type'    => 'wysiwyg',
@@ -157,10 +157,10 @@ 
 							)
 						),
                         array(
-                            'name'  => __( 'Offline Donations Settings Docs Link', 'give' ),
+                            'name'  => __('Offline Donations Settings Docs Link', 'give'),
                             'id'    => 'offline_gateway_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-gateway-offline-donations' ),
-                            'title' => __( 'Offline Gateway Settings', 'give' ),
+                            'url'   => esc_url('http://docs.givewp.com/settings-gateway-offline-donations'),
+                            'title' => __('Offline Gateway Settings', 'give'),
                             'type'  => 'give_docs_link',
                         ),
 						array(
@@ -178,33 +178,33 @@ 
 							'type' => 'title'
 						),
 						array(
-							'name'    => __( 'Test Mode', 'give' ),
-							'desc'    => __( 'While in test mode no live donations are processed. To fully use test mode, you must have a sandbox (test) account for the payment gateway you are testing.', 'give' ),
+							'name'    => __('Test Mode', 'give'),
+							'desc'    => __('While in test mode no live donations are processed. To fully use test mode, you must have a sandbox (test) account for the payment gateway you are testing.', 'give'),
 							'id'      => 'test_mode',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							)
 						),
 						array(
-							'name' => __( 'Enabled Gateways', 'give' ),
-							'desc' => __( 'Enable your payment gateway. Can be ordered by dragging.', 'give' ),
+							'name' => __('Enabled Gateways', 'give'),
+							'desc' => __('Enable your payment gateway. Can be ordered by dragging.', 'give'),
 							'id'   => 'gateways',
 							'type' => 'enabled_gateways'
 						),
 						array(
-							'name' => __( 'Default Gateway', 'give' ),
-							'desc' => __( 'The gateway that will be selected by default.', 'give' ),
+							'name' => __('Default Gateway', 'give'),
+							'desc' => __('The gateway that will be selected by default.', 'give'),
 							'id'   => 'default_gateway',
 							'type' => 'default_gateway'
 						),
                         array(
-                            'name'  => __( 'Gateways Docs Link', 'give' ),
+                            'name'  => __('Gateways Docs Link', 'give'),
                             'id'    => 'gateway_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-gateways' ),
-                            'title' => __( 'Gateway Settings', 'give' ),
+                            'url'   => esc_url('http://docs.givewp.com/settings-gateways'),
+                            'title' => __('Gateway Settings', 'give'),
                             'type'  => 'give_docs_link',
                         ),
 						array(
@@ -219,7 +219,7 @@ 
 			 * Filter the payment gateways settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_gateways', $settings );
+			$settings = apply_filters('give_settings_gateways', $settings);
 
 			/**
 			 * Filter the settings.
@@ -228,7 +228,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -242,12 +242,12 @@ 
 		 */
 		public function get_sections() {
 			$sections = array(
-				'gateways-settings' => __( 'Gateways', 'give' ),
-				'paypal-standard'   => __( 'PayPal Standard', 'give' ),
-				'offline-donations' => __( 'Offline Donations', 'give' )
+				'gateways-settings' => __('Gateways', 'give'),
+				'paypal-standard'   => __('PayPal Standard', 'give'),
+				'offline-donations' => __('Offline Donations', 'give')
 			);
 
-			return apply_filters( 'give_get_sections_' . $this->id, $sections );
+			return apply_filters('give_get_sections_'.$this->id, $sections);
 		}
 	}
 

includes/admin/EDD_SL_Plugin_Updater.php

Doc Comments +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
 	 * @uses api_request()
 	 *
 	 * @param array   $_transient_data Update array build by WordPress.
-	 * @return array Modified update array with custom plugin data.
+	 * @return stdClass Modified update array with custom plugin data.
 	 */
 	public function check_update( $_transient_data ) {
 

Braces +3 added lines, -1 removed lines

@@ -1,7 +1,9 @@
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
 
 /**
  * Allows plugins to use their own update API.

Spacing +129 added lines, -129 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined('ABSPATH')) exit;
 
 /**
  * Allows plugins to use their own update API.
@@ -29,20 +29,20 @@ 
 	 * @param string  $_plugin_file Path to the plugin file.
 	 * @param array   $_api_data    Optional data to send with API calls.
 	 */
-	public function __construct( $_api_url, $_plugin_file, $_api_data = null ) {
+	public function __construct($_api_url, $_plugin_file, $_api_data = null) {
 
 		global $edd_plugin_data;
 
-		$this->api_url     = trailingslashit( $_api_url );
+		$this->api_url     = trailingslashit($_api_url);
 		$this->api_data    = $_api_data;
-		$this->name        = plugin_basename( $_plugin_file );
-		$this->slug        = basename( $_plugin_file, '.php' );
+		$this->name        = plugin_basename($_plugin_file);
+		$this->slug        = basename($_plugin_file, '.php');
 		$this->version     = $_api_data['version'];
-		$this->wp_override = isset( $_api_data['wp_override'] ) ? (bool) $_api_data['wp_override'] : false;
-		$this->beta        = ! empty( $this->api_data['beta'] ) ? true : false;
-		$this->cache_key   = md5( serialize( $this->slug . $this->api_data['license'] . $this->beta ) );
+		$this->wp_override = isset($_api_data['wp_override']) ? (bool) $_api_data['wp_override'] : false;
+		$this->beta        = ! empty($this->api_data['beta']) ? true : false;
+		$this->cache_key   = md5(serialize($this->slug.$this->api_data['license'].$this->beta));
 
-		$edd_plugin_data[ $this->slug ] = $this->api_data;
+		$edd_plugin_data[$this->slug] = $this->api_data;
 
 		// Set up hooks.
 		$this->init();
@@ -58,11 +58,11 @@ 
 	 */
 	public function init() {
 
-		add_filter( 'pre_set_site_transient_update_plugins', array( $this, 'check_update' ) );
-		add_filter( 'plugins_api', array( $this, 'plugins_api_filter' ), 10, 3 );
-		remove_action( 'after_plugin_row_' . $this->name, 'wp_plugin_update_row', 10 );
-		add_action( 'after_plugin_row_' . $this->name, array( $this, 'show_update_notification' ), 10, 2 );
-		add_action( 'admin_init', array( $this, 'show_changelog' ) );
+		add_filter('pre_set_site_transient_update_plugins', array($this, 'check_update'));
+		add_filter('plugins_api', array($this, 'plugins_api_filter'), 10, 3);
+		remove_action('after_plugin_row_'.$this->name, 'wp_plugin_update_row', 10);
+		add_action('after_plugin_row_'.$this->name, array($this, 'show_update_notification'), 10, 2);
+		add_action('admin_init', array($this, 'show_changelog'));
 
 	}
 
@@ -79,41 +79,41 @@ 
 	 * @param array   $_transient_data Update array build by WordPress.
 	 * @return array Modified update array with custom plugin data.
 	 */
-	public function check_update( $_transient_data ) {
+	public function check_update($_transient_data) {
 
 		global $pagenow;
 
-		if ( ! is_object( $_transient_data ) ) {
+		if ( ! is_object($_transient_data)) {
 			$_transient_data = new stdClass;
 		}
 
-		if ( 'plugins.php' == $pagenow && is_multisite() ) {
+		if ('plugins.php' == $pagenow && is_multisite()) {
 			return $_transient_data;
 		}
 
-		if ( ! empty( $_transient_data->response ) && ! empty( $_transient_data->response[ $this->name ] ) && false === $this->wp_override ) {
+		if ( ! empty($_transient_data->response) && ! empty($_transient_data->response[$this->name]) && false === $this->wp_override) {
 			return $_transient_data;
 		}
 
 		$version_info = $this->get_cached_version_info();
 
-		if ( false === $version_info ) {
-			$version_info = $this->api_request( 'plugin_latest_version', array( 'slug' => $this->slug, 'beta' => $this->beta ) );
+		if (false === $version_info) {
+			$version_info = $this->api_request('plugin_latest_version', array('slug' => $this->slug, 'beta' => $this->beta));
 
-			$this->set_version_info_cache( $version_info );
+			$this->set_version_info_cache($version_info);
 
 		}
 
-		if ( false !== $version_info && is_object( $version_info ) && isset( $version_info->new_version ) ) {
+		if (false !== $version_info && is_object($version_info) && isset($version_info->new_version)) {
 
-			if ( version_compare( $this->version, $version_info->new_version, '<' ) ) {
+			if (version_compare($this->version, $version_info->new_version, '<')) {
 
-				$_transient_data->response[ $this->name ] = $version_info;
+				$_transient_data->response[$this->name] = $version_info;
 
 			}
 
-			$_transient_data->last_checked           = current_time( 'timestamp' );
-			$_transient_data->checked[ $this->name ] = $this->version;
+			$_transient_data->last_checked           = current_time('timestamp');
+			$_transient_data->checked[$this->name] = $this->version;
 
 		}
 
@@ -126,97 +126,97 @@ 
 	 * @param string  $file
 	 * @param array   $plugin
 	 */
-	public function show_update_notification( $file, $plugin ) {
+	public function show_update_notification($file, $plugin) {
 
-		if ( is_network_admin() ) {
+		if (is_network_admin()) {
 			return;
 		}
 
-		if( ! current_user_can( 'update_plugins' ) ) {
+		if ( ! current_user_can('update_plugins')) {
 			return;
 		}
 
-		if( ! is_multisite() ) {
+		if ( ! is_multisite()) {
 			return;
 		}
 
-		if ( $this->name != $file ) {
+		if ($this->name != $file) {
 			return;
 		}
 
 		// Remove our filter on the site transient
-		remove_filter( 'pre_set_site_transient_update_plugins', array( $this, 'check_update' ), 10 );
+		remove_filter('pre_set_site_transient_update_plugins', array($this, 'check_update'), 10);
 
-		$update_cache = get_site_transient( 'update_plugins' );
+		$update_cache = get_site_transient('update_plugins');
 
-		$update_cache = is_object( $update_cache ) ? $update_cache : new stdClass();
+		$update_cache = is_object($update_cache) ? $update_cache : new stdClass();
 
-		if ( empty( $update_cache->response ) || empty( $update_cache->response[ $this->name ] ) ) {
+		if (empty($update_cache->response) || empty($update_cache->response[$this->name])) {
 
 			$version_info = $this->get_cached_version_info();
 
-			if ( false === $version_info ) {
-				$version_info = $this->api_request( 'plugin_latest_version', array( 'slug' => $this->slug, 'beta' => $this->beta ) );
+			if (false === $version_info) {
+				$version_info = $this->api_request('plugin_latest_version', array('slug' => $this->slug, 'beta' => $this->beta));
 
-				$this->set_version_info_cache( $version_info );
+				$this->set_version_info_cache($version_info);
 			}
 
-			if ( ! is_object( $version_info ) ) {
+			if ( ! is_object($version_info)) {
 				return;
 			}
 
-			if ( version_compare( $this->version, $version_info->new_version, '<' ) ) {
+			if (version_compare($this->version, $version_info->new_version, '<')) {
 
-				$update_cache->response[ $this->name ] = $version_info;
+				$update_cache->response[$this->name] = $version_info;
 
 			}
 
-			$update_cache->last_checked = current_time( 'timestamp' );
-			$update_cache->checked[ $this->name ] = $this->version;
+			$update_cache->last_checked = current_time('timestamp');
+			$update_cache->checked[$this->name] = $this->version;
 
-			set_site_transient( 'update_plugins', $update_cache );
+			set_site_transient('update_plugins', $update_cache);
 
 		} else {
 
-			$version_info = $update_cache->response[ $this->name ];
+			$version_info = $update_cache->response[$this->name];
 
 		}
 
 		// Restore our filter
-		add_filter( 'pre_set_site_transient_update_plugins', array( $this, 'check_update' ) );
+		add_filter('pre_set_site_transient_update_plugins', array($this, 'check_update'));
 
-		if ( ! empty( $update_cache->response[ $this->name ] ) && version_compare( $this->version, $version_info->new_version, '<' ) ) {
+		if ( ! empty($update_cache->response[$this->name]) && version_compare($this->version, $version_info->new_version, '<')) {
 
 			// build a plugin list row, with update notification
-			$wp_list_table = _get_list_table( 'WP_Plugins_List_Table' );
+			$wp_list_table = _get_list_table('WP_Plugins_List_Table');
 			# <tr class="plugin-update-tr"><td colspan="' . $wp_list_table->get_column_count() . '" class="plugin-update colspanchange">
-			echo '<tr class="plugin-update-tr" id="' . $this->slug . '-update" data-slug="' . $this->slug . '" data-plugin="' . $this->slug . '/' . $file . '">';
+			echo '<tr class="plugin-update-tr" id="'.$this->slug.'-update" data-slug="'.$this->slug.'" data-plugin="'.$this->slug.'/'.$file.'">';
 			echo '<td colspan="3" class="plugin-update colspanchange">';
 			echo '<div class="update-message notice inline notice-warning notice-alt">';
 
-			$changelog_link = self_admin_url( 'index.php?edd_sl_action=view_plugin_changelog&plugin=' . $this->name . '&slug=' . $this->slug . '&TB_iframe=true&width=772&height=911' );
+			$changelog_link = self_admin_url('index.php?edd_sl_action=view_plugin_changelog&plugin='.$this->name.'&slug='.$this->slug.'&TB_iframe=true&width=772&height=911');
 
-			if ( empty( $version_info->download_link ) ) {
+			if (empty($version_info->download_link)) {
 				printf(
-					__( 'There is a new version of %1$s available. %2$sView version %3$s details%4$s.', 'give' ),
-					esc_html( $version_info->name ),
-					'<a target="_blank" class="thickbox" href="' . esc_url( $changelog_link ) . '">',
-					esc_html( $version_info->new_version ),
+					__('There is a new version of %1$s available. %2$sView version %3$s details%4$s.', 'give'),
+					esc_html($version_info->name),
+					'<a target="_blank" class="thickbox" href="'.esc_url($changelog_link).'">',
+					esc_html($version_info->new_version),
 					'</a>'
 				);
 			} else {
 				printf(
-					__( 'There is a new version of %1$s available. %2$sView version %3$s details%4$s or %5$supdate now%6$s.', 'give' ),
-					esc_html( $version_info->name ),
-					'<a target="_blank" class="thickbox" href="' . esc_url( $changelog_link ) . '">',
-					esc_html( $version_info->new_version ),
+					__('There is a new version of %1$s available. %2$sView version %3$s details%4$s or %5$supdate now%6$s.', 'give'),
+					esc_html($version_info->name),
+					'<a target="_blank" class="thickbox" href="'.esc_url($changelog_link).'">',
+					esc_html($version_info->new_version),
 					'</a>',
-					'<a href="' . esc_url( wp_nonce_url( self_admin_url( 'update.php?action=upgrade-plugin&plugin=' ) . $this->name, 'upgrade-plugin_' . $this->name ) ) .'">',
+					'<a href="'.esc_url(wp_nonce_url(self_admin_url('update.php?action=upgrade-plugin&plugin=').$this->name, 'upgrade-plugin_'.$this->name)).'">',
 					'</a>'
 				);
 			}
 
-			do_action( "in_plugin_update_message-{$file}", $plugin, $version_info );
+			do_action("in_plugin_update_message-{$file}", $plugin, $version_info);
 
 			echo '</div></td></tr>';
 		}
@@ -232,15 +232,15 @@ 
 	 * @param object  $_args
 	 * @return object $_data
 	 */
-	public function plugins_api_filter( $_data, $_action = '', $_args = null ) {
+	public function plugins_api_filter($_data, $_action = '', $_args = null) {
 
-		if ( $_action != 'plugin_information' ) {
+		if ($_action != 'plugin_information') {
 
 			return $_data;
 
 		}
 
-		if ( ! isset( $_args->slug ) || ( $_args->slug != $this->slug ) ) {
+		if ( ! isset($_args->slug) || ($_args->slug != $this->slug)) {
 
 			return $_data;
 
@@ -255,20 +255,20 @@ 
 			)
 		);
 
-		$cache_key = 'edd_api_request_' . md5( serialize( $this->slug . $this->api_data['license'] . $this->beta ) );
+		$cache_key = 'edd_api_request_'.md5(serialize($this->slug.$this->api_data['license'].$this->beta));
 
 		// Get the transient where we store the api request for this plugin for 24 hours
-		$edd_api_request_transient = $this->get_cached_version_info( $cache_key );
+		$edd_api_request_transient = $this->get_cached_version_info($cache_key);
 
 		//If we have no transient-saved value, run the API, set a fresh transient with the API value, and return that value too right now.
-		if ( empty( $edd_api_request_transient ) ) {
+		if (empty($edd_api_request_transient)) {
 
-			$api_response = $this->api_request( 'plugin_information', $to_send );
+			$api_response = $this->api_request('plugin_information', $to_send);
 
 			// Expires in 3 hours
-			$this->set_version_info_cache( $api_response, $cache_key );
+			$this->set_version_info_cache($api_response, $cache_key);
 
-			if ( false !== $api_response ) {
+			if (false !== $api_response) {
 				$_data = $api_response;
 			}
 
@@ -277,20 +277,20 @@ 
 		}
 
 		// Convert sections into an associative array, since we're getting an object, but Core expects an array.
-		if ( isset( $_data->sections ) && ! is_array( $_data->sections ) ) {
+		if (isset($_data->sections) && ! is_array($_data->sections)) {
 			$new_sections = array();
-			foreach ( $_data->sections as $key => $value ) {
-				$new_sections[ $key ] = $value;
+			foreach ($_data->sections as $key => $value) {
+				$new_sections[$key] = $value;
 			}
 
 			$_data->sections = $new_sections;
 		}
 
 		// Convert banners into an associative array, since we're getting an object, but Core expects an array.
-		if ( isset( $_data->banners ) && ! is_array( $_data->banners ) ) {
+		if (isset($_data->banners) && ! is_array($_data->banners)) {
 			$new_banners = array();
-			foreach ( $_data->banners as $key => $value ) {
-				$new_banners[ $key ] = $value;
+			foreach ($_data->banners as $key => $value) {
+				$new_banners[$key] = $value;
 			}
 
 			$_data->banners = $new_banners;
@@ -306,9 +306,9 @@ 
 	 * @param string  $url
 	 * @return object $array
 	 */
-	public function http_request_args( $args, $url ) {
+	public function http_request_args($args, $url) {
 		// If it is an https request and we are performing a package download, disable ssl verification
-		if ( strpos( $url, 'https://' ) !== false && strpos( $url, 'edd_action=package_download' ) ) {
+		if (strpos($url, 'https://') !== false && strpos($url, 'edd_action=package_download')) {
 			$args['sslverify'] = false;
 		}
 		return $args;
@@ -325,50 +325,50 @@ 
 	 * @param array   $_data   Parameters for the API action.
 	 * @return false|object
 	 */
-	private function api_request( $_action, $_data ) {
+	private function api_request($_action, $_data) {
 
 		global $wp_version;
 
-		$data = array_merge( $this->api_data, $_data );
+		$data = array_merge($this->api_data, $_data);
 
-		if ( $data['slug'] != $this->slug ) {
+		if ($data['slug'] != $this->slug) {
 			return;
 		}
 
-		if( $this->api_url == trailingslashit (home_url() ) ) {
+		if ($this->api_url == trailingslashit(home_url())) {
 			return false; // Don't allow a plugin to ping itself
 		}
 
 		$api_params = array(
 			'edd_action' => 'get_version',
-			'license'    => ! empty( $data['license'] ) ? $data['license'] : '',
-			'item_name'  => isset( $data['item_name'] ) ? $data['item_name'] : false,
-			'item_id'    => isset( $data['item_id'] ) ? $data['item_id'] : false,
-			'version'    => isset( $data['version'] ) ? $data['version'] : false,
+			'license'    => ! empty($data['license']) ? $data['license'] : '',
+			'item_name'  => isset($data['item_name']) ? $data['item_name'] : false,
+			'item_id'    => isset($data['item_id']) ? $data['item_id'] : false,
+			'version'    => isset($data['version']) ? $data['version'] : false,
 			'slug'       => $data['slug'],
 			'author'     => $data['author'],
 			'url'        => home_url(),
-			'beta'       => ! empty( $data['beta'] ),
+			'beta'       => ! empty($data['beta']),
 		);
 
-		$request = wp_remote_post( $this->api_url, array( 'timeout' => 15, 'sslverify' => false, 'body' => $api_params ) );
+		$request = wp_remote_post($this->api_url, array('timeout' => 15, 'sslverify' => false, 'body' => $api_params));
 
-		if ( ! is_wp_error( $request ) ) {
-			$request = json_decode( wp_remote_retrieve_body( $request ) );
+		if ( ! is_wp_error($request)) {
+			$request = json_decode(wp_remote_retrieve_body($request));
 		}
 
-		if ( $request && isset( $request->sections ) ) {
-			$request->sections = maybe_unserialize( $request->sections );
+		if ($request && isset($request->sections)) {
+			$request->sections = maybe_unserialize($request->sections);
 		} else {
 			$request = false;
 		}
 
-		if ( $request && isset( $request->banners ) ) {
-			$request->banners = maybe_unserialize( $request->banners );
+		if ($request && isset($request->banners)) {
+			$request->banners = maybe_unserialize($request->banners);
 		}
 
-		if( ! empty( $request->sections ) ) {
-			foreach( $request->sections as $key => $section ) {
+		if ( ! empty($request->sections)) {
+			foreach ($request->sections as $key => $section) {
 				$request->$key = (array) $section;
 			}
 		}
@@ -380,97 +380,97 @@ 
 
 		global $edd_plugin_data;
 
-		if( empty( $_REQUEST['edd_sl_action'] ) || 'view_plugin_changelog' != $_REQUEST['edd_sl_action'] ) {
+		if (empty($_REQUEST['edd_sl_action']) || 'view_plugin_changelog' != $_REQUEST['edd_sl_action']) {
 			return;
 		}
 
-		if( empty( $_REQUEST['plugin'] ) ) {
+		if (empty($_REQUEST['plugin'])) {
 			return;
 		}
 
-		if( empty( $_REQUEST['slug'] ) ) {
+		if (empty($_REQUEST['slug'])) {
 			return;
 		}
 
-		if( ! current_user_can( 'update_plugins' ) ) {
-			wp_die( __( 'You do not have permission to install plugin updates', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! current_user_can('update_plugins')) {
+			wp_die(__('You do not have permission to install plugin updates', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
-		$data         = $edd_plugin_data[ $_REQUEST['slug'] ];
-		$beta         = ! empty( $data['beta'] ) ? true : false;
-		$cache_key    = md5( 'edd_plugin_' . sanitize_key( $_REQUEST['plugin'] ) . '_' . $beta . '_version_info' );
-		$version_info = $this->get_cached_version_info( $cache_key );
+		$data         = $edd_plugin_data[$_REQUEST['slug']];
+		$beta         = ! empty($data['beta']) ? true : false;
+		$cache_key    = md5('edd_plugin_'.sanitize_key($_REQUEST['plugin']).'_'.$beta.'_version_info');
+		$version_info = $this->get_cached_version_info($cache_key);
 
-		if( false === $version_info ) {
+		if (false === $version_info) {
 
 			$api_params = array(
 				'edd_action' => 'get_version',
-				'item_name'  => isset( $data['item_name'] ) ? $data['item_name'] : false,
-				'item_id'    => isset( $data['item_id'] ) ? $data['item_id'] : false,
+				'item_name'  => isset($data['item_name']) ? $data['item_name'] : false,
+				'item_id'    => isset($data['item_id']) ? $data['item_id'] : false,
 				'slug'       => $_REQUEST['slug'],
 				'author'     => $data['author'],
 				'url'        => home_url(),
-				'beta'       => ! empty( $data['beta'] )
+				'beta'       => ! empty($data['beta'])
 			);
 
-			$request = wp_remote_post( $this->api_url, array( 'timeout' => 15, 'sslverify' => false, 'body' => $api_params ) );
+			$request = wp_remote_post($this->api_url, array('timeout' => 15, 'sslverify' => false, 'body' => $api_params));
 
-			if ( ! is_wp_error( $request ) ) {
-				$version_info = json_decode( wp_remote_retrieve_body( $request ) );
+			if ( ! is_wp_error($request)) {
+				$version_info = json_decode(wp_remote_retrieve_body($request));
 			}
 
 
-			if ( ! empty( $version_info ) && isset( $version_info->sections ) ) {
-				$version_info->sections = maybe_unserialize( $version_info->sections );
+			if ( ! empty($version_info) && isset($version_info->sections)) {
+				$version_info->sections = maybe_unserialize($version_info->sections);
 			} else {
 				$version_info = false;
 			}
 
-			if( ! empty( $version_info ) ) {
-				foreach( $version_info->sections as $key => $section ) {
+			if ( ! empty($version_info)) {
+				foreach ($version_info->sections as $key => $section) {
 					$version_info->$key = (array) $section;
 				}
 			}
 
-			$this->set_version_info_cache( $version_info, $cache_key );
+			$this->set_version_info_cache($version_info, $cache_key);
 
 		}
 
-		if( ! empty( $version_info ) && isset( $version_info->sections['changelog'] ) ) {
-			echo '<div style="background:#fff;padding:10px;">' . $version_info->sections['changelog'] . '</div>';
+		if ( ! empty($version_info) && isset($version_info->sections['changelog'])) {
+			echo '<div style="background:#fff;padding:10px;">'.$version_info->sections['changelog'].'</div>';
 		}
 
 		exit;
 	}
 
-	public function get_cached_version_info( $cache_key = '' ) {
+	public function get_cached_version_info($cache_key = '') {
 
-		if( empty( $cache_key ) ) {
+		if (empty($cache_key)) {
 			$cache_key = $this->cache_key;
 		}
 
-		$cache = get_option( $cache_key );
+		$cache = get_option($cache_key);
 
-		if( empty( $cache['timeout'] ) || current_time( 'timestamp' ) > $cache['timeout'] ) {
+		if (empty($cache['timeout']) || current_time('timestamp') > $cache['timeout']) {
 			return false; // Cache is expired
 		}
 
-		return json_decode( $cache['value'] );
+		return json_decode($cache['value']);
 
 	}
 
-	public function set_version_info_cache( $value = '', $cache_key = '' ) {
+	public function set_version_info_cache($value = '', $cache_key = '') {
 
-		if( empty( $cache_key ) ) {
+		if (empty($cache_key)) {
 			$cache_key = $this->cache_key;
 		}
 
 		$data = array(
-			'timeout' => strtotime( '+3 hours', current_time( 'timestamp' ) ),
-			'value'   => json_encode( $value )
+			'timeout' => strtotime('+3 hours', current_time('timestamp')),
+			'value'   => json_encode($value)
 		);
 
-		update_option( $cache_key, $data );
+		update_option($cache_key, $data);
 
 	}
 

includes/forms/widget.php

Spacing +65 added lines, -65 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -19,7 +19,7 @@ 
  *
  * @since 1.0
  */
-class Give_Forms_Widget extends WP_Widget{
+class Give_Forms_Widget extends WP_Widget {
 	/**
 	 * The widget class name
 	 *
@@ -30,19 +30,19 @@ 
 	/**
 	 * Instantiate the class
 	 */
-	public function __construct(){
-		$this->self = get_class( $this );
+	public function __construct() {
+		$this->self = get_class($this);
 
 		parent::__construct(
-			strtolower( $this->self ),
-			esc_html__( 'Give - Donation Form', 'give' ),
+			strtolower($this->self),
+			esc_html__('Give - Donation Form', 'give'),
 			array(
-				'description' => esc_html__( 'Display a Give Donation Form in your theme\'s widget powered sidebar.', 'give' )
+				'description' => esc_html__('Display a Give Donation Form in your theme\'s widget powered sidebar.', 'give')
 			)
 		);
 
-		add_action( 'widgets_init',          array( $this, 'widget_init' ) );
-		add_action( 'admin_enqueue_scripts', array( $this, 'admin_widget_scripts' ) );
+		add_action('widgets_init', array($this, 'widget_init'));
+		add_action('admin_enqueue_scripts', array($this, 'admin_widget_scripts'));
 	}
 
 	/**
@@ -52,23 +52,23 @@ 
 	 *
 	 * @return void
 	 */
-	public function admin_widget_scripts( $hook ){
+	public function admin_widget_scripts($hook) {
 		// Directories of assets
-		$js_dir     = GIVE_PLUGIN_URL . 'assets/js/admin/';
-		$js_plugins = GIVE_PLUGIN_URL . 'assets/js/plugins/';
-		$css_dir    = GIVE_PLUGIN_URL . 'assets/css/';
+		$js_dir     = GIVE_PLUGIN_URL.'assets/js/admin/';
+		$js_plugins = GIVE_PLUGIN_URL.'assets/js/plugins/';
+		$css_dir    = GIVE_PLUGIN_URL.'assets/css/';
 
 		// Use minified libraries if SCRIPT_DEBUG is turned off
-		$suffix = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';
+		$suffix = (defined('SCRIPT_DEBUG') && SCRIPT_DEBUG) ? '' : '.min';
 
 		// Widget Script
-		if ( $hook == 'widgets.php' ) {
+		if ($hook == 'widgets.php') {
 
-			wp_enqueue_style( 'give-qtip-css', $css_dir . 'jquery.qtip' . $suffix . '.css' );
+			wp_enqueue_style('give-qtip-css', $css_dir.'jquery.qtip'.$suffix.'.css');
 
-			wp_enqueue_script( 'give-qtip', $js_plugins . 'jquery.qtip' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION );
+			wp_enqueue_script('give-qtip', $js_plugins.'jquery.qtip'.$suffix.'.js', array('jquery'), GIVE_VERSION);
 
-			wp_enqueue_script( 'give-admin-widgets-scripts', $js_dir . 'admin-widgets' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, false );
+			wp_enqueue_script('give-admin-widgets-scripts', $js_dir.'admin-widgets'.$suffix.'.js', array('jquery'), GIVE_VERSION, false);
 		}
 	}
 
@@ -79,9 +79,9 @@ 
 	 *                        before_widget, and after_widget.
 	 * @param array $instance The settings for the particular instance of the widget.
 	 */
-	public function widget( $args, $instance ){
-		$title = !empty( $instance['title'] ) ? $instance['title'] : '';
-		$title = apply_filters( 'widget_title', $title, $instance, $this->id_base );
+	public function widget($args, $instance) {
+		$title = ! empty($instance['title']) ? $instance['title'] : '';
+		$title = apply_filters('widget_title', $title, $instance, $this->id_base);
 
 		echo $args['before_widget'];
 
@@ -90,11 +90,11 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_before_forms_widget' );
+		do_action('give_before_forms_widget');
 
-		echo $title ? $args['before_title'] . $title . $args['after_title'] : '';
+		echo $title ? $args['before_title'].$title.$args['after_title'] : '';
 
-		give_get_donation_form( $instance );
+		give_get_donation_form($instance);
 
 		echo $args['after_widget'];
 
@@ -103,7 +103,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_after_forms_widget' );
+		do_action('give_after_forms_widget');
 	}
 
 	/**
@@ -113,7 +113,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function form( $instance ){
+	public function form($instance) {
 		$defaults = array(
 			'title'         => '',
 			'id'            => '',
@@ -123,84 +123,84 @@ 
 			'continue_button_title' => '',
 		);
 
-		$instance = wp_parse_args( (array) $instance, $defaults );
+		$instance = wp_parse_args((array) $instance, $defaults);
 
 		// Backward compatibility: Set float labels as default if, it was set as empty previous.
-		$instance['float_labels'] = empty( $instance['float_labels'] ) ? 'global' : $instance['float_labels'];
+		$instance['float_labels'] = empty($instance['float_labels']) ? 'global' : $instance['float_labels'];
 
 		// Query Give Forms
 		$args = array(
 			'post_type'      => 'give_forms',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 		);
 
-		$give_forms = get_posts( $args );
+		$give_forms = get_posts($args);
 		?>
 		<div class="give_forms_widget_container">
 
 			<?php // Widget: widget Title. ?>
 			<p>
-				<label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php esc_html_e( 'Title:', 'give' ); ?></label>
-				<input type="text" class="widefat" id="<?php echo $this->get_field_id( 'title' ); ?>" name="<?php echo $this->get_field_name( 'title' ); ?>" value="<?php esc_attr_e( $instance['title'] ); ?>" /><br>
-				<small class="give-field-description"><?php esc_html_e( 'Leave blank to hide the widget title.', 'give' ); ?></small>
+				<label for="<?php echo $this->get_field_id('title'); ?>"><?php esc_html_e('Title:', 'give'); ?></label>
+				<input type="text" class="widefat" id="<?php echo $this->get_field_id('title'); ?>" name="<?php echo $this->get_field_name('title'); ?>" value="<?php esc_attr_e($instance['title']); ?>" /><br>
+				<small class="give-field-description"><?php esc_html_e('Leave blank to hide the widget title.', 'give'); ?></small>
 			</p>
 
 			<?php // Widget: Give Form?>
 			<p>
-				<label for="<?php echo esc_attr( $this->get_field_id( 'id' ) ); ?>"><?php esc_html_e( 'Give Form:', 'give' ); ?></label>
-				<select class="widefat" name="<?php echo esc_attr( $this->get_field_name( 'id' ) ); ?>" id="<?php echo esc_attr( $this->get_field_id( 'id' ) ); ?>">
-					<option value="current"><?php esc_html_e( '- Select -', 'give' ); ?></option>
-					<?php foreach ( $give_forms as $give_form ) { ?>
-						<?php $form_title = empty( $give_form->post_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $give_form->ID ) : $give_form->post_title; ?>
-						<option <?php selected( absint( $instance['id'] ), $give_form->ID ); ?> value="<?php echo esc_attr( $give_form->ID ); ?>"><?php echo $form_title; ?></option>
+				<label for="<?php echo esc_attr($this->get_field_id('id')); ?>"><?php esc_html_e('Give Form:', 'give'); ?></label>
+				<select class="widefat" name="<?php echo esc_attr($this->get_field_name('id')); ?>" id="<?php echo esc_attr($this->get_field_id('id')); ?>">
+					<option value="current"><?php esc_html_e('- Select -', 'give'); ?></option>
+					<?php foreach ($give_forms as $give_form) { ?>
+						<?php $form_title = empty($give_form->post_title) ? sprintf(__('Untitled (#%s)', 'give'), $give_form->ID) : $give_form->post_title; ?>
+						<option <?php selected(absint($instance['id']), $give_form->ID); ?> value="<?php echo esc_attr($give_form->ID); ?>"><?php echo $form_title; ?></option>
 					<?php } ?>
 				</select><br>
-				<small class="give-field-description"><?php esc_html_e( 'Select a Give Form to embed in this widget.', 'give' ); ?></small>
+				<small class="give-field-description"><?php esc_html_e('Select a Give Form to embed in this widget.', 'give'); ?></small>
 			</p>
 
 			<?php // Widget: Display Style ?>
 			<p class="give_forms_display_style_setting_row">
-				<label for="<?php echo esc_attr( $this->get_field_id( 'display_style' ) ); ?>"><?php esc_html_e( 'Display Style:', 'give' ); ?></label><br>
-				<label for="<?php echo $this->get_field_id( 'display_style' ); ?>-onpage"><input type="radio" class="widefat" id="<?php echo $this->get_field_id( 'display_style' ); ?>-onpage" name="<?php echo $this->get_field_name( 'display_style' ); ?>" value="onpage" <?php checked( $instance['display_style'], 'onpage' ); ?>> <?php echo esc_html__( 'All Fields', 'give' ); ?></label>
-				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id( 'display_style' ); ?>-reveal"><input type="radio" class="widefat" id="<?php echo $this->get_field_id( 'display_style' ); ?>-reveal" name="<?php echo $this->get_field_name( 'display_style' ); ?>" value="reveal" <?php checked( $instance['display_style'], 'reveal' ); ?>> <?php echo esc_html__( 'Reveal', 'give' ); ?></label>
-				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id( 'display_style' ); ?>-modal"><input type="radio" class="widefat" id="<?php echo $this->get_field_id( 'display_style' ); ?>-modal" name="<?php echo $this->get_field_name( 'display_style' ); ?>" value="modal" <?php checked( $instance['display_style'], 'modal' ); ?>> <?php echo esc_html__( 'Modal', 'give' ); ?></label>
-				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id( 'display_style' ); ?>-button"><input type="radio" class="widefat" id="<?php echo $this->get_field_id( 'display_style' ); ?>-button" name="<?php echo $this->get_field_name( 'display_style' ); ?>" value="button" <?php checked( $instance['display_style'], 'button' ); ?>> <?php echo esc_html__( 'Button', 'give' ); ?></label><br>
+				<label for="<?php echo esc_attr($this->get_field_id('display_style')); ?>"><?php esc_html_e('Display Style:', 'give'); ?></label><br>
+				<label for="<?php echo $this->get_field_id('display_style'); ?>-onpage"><input type="radio" class="widefat" id="<?php echo $this->get_field_id('display_style'); ?>-onpage" name="<?php echo $this->get_field_name('display_style'); ?>" value="onpage" <?php checked($instance['display_style'], 'onpage'); ?>> <?php echo esc_html__('All Fields', 'give'); ?></label>
+				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id('display_style'); ?>-reveal"><input type="radio" class="widefat" id="<?php echo $this->get_field_id('display_style'); ?>-reveal" name="<?php echo $this->get_field_name('display_style'); ?>" value="reveal" <?php checked($instance['display_style'], 'reveal'); ?>> <?php echo esc_html__('Reveal', 'give'); ?></label>
+				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id('display_style'); ?>-modal"><input type="radio" class="widefat" id="<?php echo $this->get_field_id('display_style'); ?>-modal" name="<?php echo $this->get_field_name('display_style'); ?>" value="modal" <?php checked($instance['display_style'], 'modal'); ?>> <?php echo esc_html__('Modal', 'give'); ?></label>
+				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id('display_style'); ?>-button"><input type="radio" class="widefat" id="<?php echo $this->get_field_id('display_style'); ?>-button" name="<?php echo $this->get_field_name('display_style'); ?>" value="button" <?php checked($instance['display_style'], 'button'); ?>> <?php echo esc_html__('Button', 'give'); ?></label><br>
 				<small class="give-field-description">
-					<?php echo esc_html__( 'Select a Give Form style.', 'give' ); ?>
+					<?php echo esc_html__('Select a Give Form style.', 'give'); ?>
 				</small>
 			</p>
 
 			<?php // Widget: Continue Button Title. ?>
 			<p class="give_forms_continue_button_title_setting_row">
-				<label for="<?php echo $this->get_field_id( 'continue_button_title' ); ?>"><?php esc_html_e( 'Button Text:', 'give' ); ?></label>
-				<input type="text" class="widefat" id="<?php echo $this->get_field_id( 'continue_button_title' ); ?>" name="<?php echo $this->get_field_name( 'continue_button_title' ); ?>" value="<?php esc_attr_e( $instance['continue_button_title'] ); ?>" /><br>
-				<small class="give-field-description"><?php esc_html_e( 'The button label for displaying the additional payment fields.', 'give' ); ?></small>
+				<label for="<?php echo $this->get_field_id('continue_button_title'); ?>"><?php esc_html_e('Button Text:', 'give'); ?></label>
+				<input type="text" class="widefat" id="<?php echo $this->get_field_id('continue_button_title'); ?>" name="<?php echo $this->get_field_name('continue_button_title'); ?>" value="<?php esc_attr_e($instance['continue_button_title']); ?>" /><br>
+				<small class="give-field-description"><?php esc_html_e('The button label for displaying the additional payment fields.', 'give'); ?></small>
 			</p>
 
 			<?php // Widget: Floating Labels ?>
 			<p>
-				<label for="<?php echo esc_attr( $this->get_field_id( 'float_labels' ) ); ?>"><?php esc_html_e( 'Floating Labels (optional):', 'give' ); ?></label><br>
-				<label for="<?php echo $this->get_field_id( 'float_labels' ); ?>-global"><input type="radio" class="widefat" id="<?php echo $this->get_field_id( 'float_labels' ); ?>-global" name="<?php echo $this->get_field_name( 'float_labels' ); ?>" value="global" <?php checked( $instance['float_labels'], 'global' ); ?>> <?php echo esc_html__( 'Global Option', 'give' ); ?></label>
-				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id( 'float_labels' ); ?>-enabled"><input type="radio" class="widefat" id="<?php echo $this->get_field_id( 'float_labels' ); ?>-enabled" name="<?php echo $this->get_field_name( 'float_labels' ); ?>" value="enabled" <?php checked( $instance['float_labels'], 'enabled' ); ?>> <?php echo esc_html__( 'Yes', 'give' ); ?></label>
-				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id( 'float_labels' ); ?>-disabled"><input type="radio" class="widefat" id="<?php echo $this->get_field_id( 'float_labels' ); ?>-disabled" name="<?php echo $this->get_field_name( 'float_labels' ); ?>" value="disabled" <?php checked( $instance['float_labels'], 'disabled' ); ?>> <?php echo esc_html__( 'No', 'give' ); ?></label><br>
+				<label for="<?php echo esc_attr($this->get_field_id('float_labels')); ?>"><?php esc_html_e('Floating Labels (optional):', 'give'); ?></label><br>
+				<label for="<?php echo $this->get_field_id('float_labels'); ?>-global"><input type="radio" class="widefat" id="<?php echo $this->get_field_id('float_labels'); ?>-global" name="<?php echo $this->get_field_name('float_labels'); ?>" value="global" <?php checked($instance['float_labels'], 'global'); ?>> <?php echo esc_html__('Global Option', 'give'); ?></label>
+				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id('float_labels'); ?>-enabled"><input type="radio" class="widefat" id="<?php echo $this->get_field_id('float_labels'); ?>-enabled" name="<?php echo $this->get_field_name('float_labels'); ?>" value="enabled" <?php checked($instance['float_labels'], 'enabled'); ?>> <?php echo esc_html__('Yes', 'give'); ?></label>
+				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id('float_labels'); ?>-disabled"><input type="radio" class="widefat" id="<?php echo $this->get_field_id('float_labels'); ?>-disabled" name="<?php echo $this->get_field_name('float_labels'); ?>" value="disabled" <?php checked($instance['float_labels'], 'disabled'); ?>> <?php echo esc_html__('No', 'give'); ?></label><br>
 				<small class="give-field-description">
 					<?php
 					printf(
 					/* translators: %s: http://docs.givewp.com/form-floating-labels */
-						__( 'Override the <a href="%s" target="_blank">floating labels</a> setting for this Give form.', 'give' ),
-						esc_url( 'http://docs.givewp.com/form-floating-labels' )
+						__('Override the <a href="%s" target="_blank">floating labels</a> setting for this Give form.', 'give'),
+						esc_url('http://docs.givewp.com/form-floating-labels')
 					);
 					?></small>
 			</p>
 
 			<?php // Widget: Display Content ?>
 			<p>
-				<label for="<?php echo esc_attr( $this->get_field_id( 'show_content' ) ); ?>"><?php esc_html_e( 'Display Content (optional):', 'give' ); ?></label><br>
-				<label for="<?php echo $this->get_field_id( 'show_content' ); ?>-none"><input type="radio" class="widefat" id="<?php echo $this->get_field_id( 'show_content' ); ?>-none" name="<?php echo $this->get_field_name( 'show_content' ); ?>" value="none" <?php checked( $instance['show_content'], 'none' ); ?>> <?php echo esc_html__( 'None', 'give' ); ?></label>
-				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id( 'show_content' ); ?>-above"><input type="radio" class="widefat" id="<?php echo $this->get_field_id( 'show_content' ); ?>-above" name="<?php echo $this->get_field_name( 'show_content' ); ?>" value="above" <?php checked( $instance['show_content'], 'above' ); ?>> <?php echo esc_html__( 'Above', 'give' ); ?></label>
-				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id( 'show_content' ); ?>-below"><input type="radio" class="widefat" id="<?php echo $this->get_field_id( 'show_content' ); ?>-below" name="<?php echo $this->get_field_name( 'show_content' ); ?>" value="below" <?php checked( $instance['show_content'], 'below' ); ?>> <?php echo esc_html__( 'Below', 'give' ); ?></label><br>
-				<small class="give-field-description"><?php esc_html_e( 'Override the display content setting for this Give form.', 'give' ); ?></small>
+				<label for="<?php echo esc_attr($this->get_field_id('show_content')); ?>"><?php esc_html_e('Display Content (optional):', 'give'); ?></label><br>
+				<label for="<?php echo $this->get_field_id('show_content'); ?>-none"><input type="radio" class="widefat" id="<?php echo $this->get_field_id('show_content'); ?>-none" name="<?php echo $this->get_field_name('show_content'); ?>" value="none" <?php checked($instance['show_content'], 'none'); ?>> <?php echo esc_html__('None', 'give'); ?></label>
+				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id('show_content'); ?>-above"><input type="radio" class="widefat" id="<?php echo $this->get_field_id('show_content'); ?>-above" name="<?php echo $this->get_field_name('show_content'); ?>" value="above" <?php checked($instance['show_content'], 'above'); ?>> <?php echo esc_html__('Above', 'give'); ?></label>
+				&nbsp;&nbsp;<label for="<?php echo $this->get_field_id('show_content'); ?>-below"><input type="radio" class="widefat" id="<?php echo $this->get_field_id('show_content'); ?>-below" name="<?php echo $this->get_field_name('show_content'); ?>" value="below" <?php checked($instance['show_content'], 'below'); ?>> <?php echo esc_html__('Below', 'give'); ?></label><br>
+				<small class="give-field-description"><?php esc_html_e('Override the display content setting for this Give form.', 'give'); ?></small>
 		</div>
 		<?php
 	}
@@ -210,8 +210,8 @@ 
 	 *
 	 * @return void
 	 */
-	function widget_init(){
-		register_widget( $this->self );
+	function widget_init() {
+		register_widget($this->self);
 	}
 
 	/**
@@ -222,7 +222,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function update( $new_instance, $old_instance ){
+	public function update($new_instance, $old_instance) {
 		$this->flush_widget_cache();
 
 		return $new_instance;
@@ -233,8 +233,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function flush_widget_cache(){
-		wp_cache_delete( $this->self, 'widget' );
+	public function flush_widget_cache() {
+		wp_cache_delete($this->self, 'widget');
 	}
 }
 

includes/admin/shortcodes/shortcode-give-form.php

Spacing +32 added lines, -32 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,10 +24,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = esc_html__( 'Donation Form', 'give' );
-		$this->shortcode['label'] = esc_html__( 'Donation Form', 'give' );
+		$this->shortcode['title'] = esc_html__('Donation Form', 'give');
+		$this->shortcode['label'] = esc_html__('Donation Form', 'give');
 
-		parent::__construct( 'give_form' );
+		parent::__construct('give_form');
 	}
 
 	/**
@@ -39,8 +39,8 @@ 
 
 		$create_form_link = sprintf(
 		/* translators: %s: create new form URL */
-			__( '<a href="%s">Create</a> a new Donation Form.', 'give' ),
-			admin_url( 'post-new.php?post_type=give_forms' )
+			__('<a href="%s">Create</a> a new Donation Form.', 'give'),
+			admin_url('post-new.php?post_type=give_forms')
 		);
 
 		return array(
@@ -50,68 +50,68 @@ 
 					'post_type' => 'give_forms',
 				),
 				'name'        => 'id',
-				'tooltip'     => esc_attr__( 'Select a Donation Form', 'give' ),
-				'placeholder' => '- ' . esc_attr__( 'Select a Donation Form', 'give' ) . ' -',
+				'tooltip'     => esc_attr__('Select a Donation Form', 'give'),
+				'placeholder' => '- '.esc_attr__('Select a Donation Form', 'give').' -',
 				'required'    => array(
-					'alert' => esc_html__( 'You must first select a Form!', 'give' ),
-					'error' => sprintf( '<p class="strong">%s</p><p class="no-margin">%s</p>', esc_html__( 'No forms found.', 'give' ), $create_form_link ),
+					'alert' => esc_html__('You must first select a Form!', 'give'),
+					'error' => sprintf('<p class="strong">%s</p><p class="no-margin">%s</p>', esc_html__('No forms found.', 'give'), $create_form_link),
 				),
 			),
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="strong margin-top">%s</p>', esc_html__( 'Optional settings', 'give' ) ),
+				'html' => sprintf('<p class="strong margin-top">%s</p>', esc_html__('Optional settings', 'give')),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'show_title',
-				'label'   => esc_attr__( 'Show Title', 'give' ),
-				'tooltip' => esc_attr__( 'Do you want to display the form title?', 'give' ),
+				'label'   => esc_attr__('Show Title', 'give'),
+				'tooltip' => esc_attr__('Do you want to display the form title?', 'give'),
 				'options' => array(
-					'true'  => esc_html__( 'Show', 'give' ),
-					'false' => esc_html__( 'Hide', 'give' ),
+					'true'  => esc_html__('Show', 'give'),
+					'false' => esc_html__('Hide', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'show_goal',
-				'label'   => esc_attr__( 'Show Goal', 'give' ),
-				'tooltip' => esc_attr__( 'Do you want to display the donation goal?', 'give' ),
+				'label'   => esc_attr__('Show Goal', 'give'),
+				'tooltip' => esc_attr__('Do you want to display the donation goal?', 'give'),
 				'options' => array(
-					'true'  => esc_html__( 'Show', 'give' ),
-					'false' => esc_html__( 'Hide', 'give' ),
+					'true'  => esc_html__('Show', 'give'),
+					'false' => esc_html__('Hide', 'give'),
 				),
 			),
 			array(
 				'type'     => 'listbox',
 				'name'     => 'show_content',
 				'minWidth' => 240,
-				'label'    => esc_attr__( 'Display Content', 'give' ),
-				'tooltip'  => esc_attr__( 'Do you want to display the form content?', 'give' ),
+				'label'    => esc_attr__('Display Content', 'give'),
+				'tooltip'  => esc_attr__('Do you want to display the form content?', 'give'),
 				'options'  => array(
-					'none'  => esc_html__( 'No Content', 'give' ),
-					'above' => esc_html__( 'Display content ABOVE the fields', 'give' ),
-					'below' => esc_html__( 'Display content BELOW the fields', 'give' ),
+					'none'  => esc_html__('No Content', 'give'),
+					'above' => esc_html__('Display content ABOVE the fields', 'give'),
+					'below' => esc_html__('Display content BELOW the fields', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'display_style',
 				'classes' => 'give-display-style',
-				'label'   => esc_attr__( 'Display Options', 'give' ),
-				'tooltip' => esc_attr__( 'How would you like to display donation information?', 'give' ),
+				'label'   => esc_attr__('Display Options', 'give'),
+				'tooltip' => esc_attr__('How would you like to display donation information?', 'give'),
 				'options' => array(
-					'onpage' => esc_html__( 'All Fields', 'give' ),
-					'modal'  => esc_html__( 'Modal', 'give' ),
-					'reveal' => esc_html__( 'Reveal', 'give' ),
-					'button' => esc_html__( 'Button', 'give' ),
+					'onpage' => esc_html__('All Fields', 'give'),
+					'modal'  => esc_html__('Modal', 'give'),
+					'reveal' => esc_html__('Reveal', 'give'),
+					'button' => esc_html__('Button', 'give'),
 				),
 			),
 			array(
 				'type'    => 'textbox',
 				'classes' => 'give-hidden give-continue-button-title',
 				'name'    => 'continue_button_title',
-				'label'   => esc_attr__( 'Button Text', 'give' ),
-				'tooltip' => esc_attr__( 'The button label for displaying the additional payment fields.', 'give' ),
+				'label'   => esc_attr__('Button Text', 'give'),
+				'tooltip' => esc_attr__('The button label for displaying the additional payment fields.', 'give'),
 			),
 		);
 	}

includes/class-give-cache.php

Doc Comments +1 added lines, -1 removed lines

@@ -205,7 +205,7 @@
 	 *
 	 * @param bool $force If set to true then all cached values will be delete instead of only expired
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public static function delete_all_expired( $force = false ) {
 		global $wpdb;

Spacing +51 added lines, -51 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -43,7 +43,7 @@ 
 	 * @return static
 	 */
 	public static function get_instance() {
-		if ( ! isset( self::$instance ) && ! ( self::$instance instanceof Give_Cache ) ) {
+		if ( ! isset(self::$instance) && ! (self::$instance instanceof Give_Cache)) {
 			self::$instance = new Give_Cache();
 		}
 
@@ -58,7 +58,7 @@ 
 	 */
 	public function setup_hooks() {
 		// weekly delete all expired cache.
-		Give_Cron::add_weekly_event( array( $this, 'delete_all_expired' ) );
+		Give_Cron::add_weekly_event(array($this, 'delete_all_expired'));
 	}
 
 	/**
@@ -72,12 +72,12 @@ 
 	 * @return string
 	 */
 
-	public static function get_key( $action, $query_args = null ) {
+	public static function get_key($action, $query_args = null) {
 		$cache_key = "give_cache_{$action}";
 
 		// Bailout.
-		if ( ! empty( $query_args ) ) {
-			$cache_key = "{$cache_key}_" . substr( md5( serialize( $query_args ) ), 0, 15 );
+		if ( ! empty($query_args)) {
+			$cache_key = "{$cache_key}_".substr(md5(serialize($query_args)), 0, 15);
 		}
 
 		return $cache_key;
@@ -95,26 +95,26 @@ 
 	 * @return mixed
 	 */
 
-	public static function get( $cache_key, $custom_key = false, $query_args = array() ) {
-		if ( ! self::is_valid_cache_key( $cache_key ) ) {
-			if ( ! $custom_key ) {
-				return new WP_Error( 'give_invalid_cache_key', __( 'Cache key format should be give_cache_*', 'give' ) );
+	public static function get($cache_key, $custom_key = false, $query_args = array()) {
+		if ( ! self::is_valid_cache_key($cache_key)) {
+			if ( ! $custom_key) {
+				return new WP_Error('give_invalid_cache_key', __('Cache key format should be give_cache_*', 'give'));
 			}
 
-			$cache_key = self::get_key( $cache_key, $query_args );
+			$cache_key = self::get_key($cache_key, $query_args);
 		}
 
-		$option = get_option( $cache_key );
+		$option = get_option($cache_key);
 
 		// Backward compatibility (<1.8.7).
-		if ( ! is_array( $option ) || empty( $option ) || ! array_key_exists( 'expiration', $option ) ) {
+		if ( ! is_array($option) || empty($option) || ! array_key_exists('expiration', $option)) {
 			return $option;
 		}
 
 		// Get current time.
-		$current_time = current_time( 'timestamp', 1 );
+		$current_time = current_time('timestamp', 1);
 
-		if ( empty( $option['expiration'] ) || ( $current_time < $option['expiration'] ) ) {
+		if (empty($option['expiration']) || ($current_time < $option['expiration'])) {
 			$option = $option['data'];
 		} else {
 			$option = false;
@@ -137,23 +137,23 @@ 
 	 * @return mixed
 	 */
 
-	public static function set( $cache_key, $data, $expiration = null, $custom_key = false, $query_args = array() ) {
-		if ( ! self::is_valid_cache_key( $cache_key ) ) {
-			if ( ! $custom_key ) {
-				return new WP_Error( 'give_invalid_cache_key', __( 'Cache key format should be give_cache_*', 'give' ) );
+	public static function set($cache_key, $data, $expiration = null, $custom_key = false, $query_args = array()) {
+		if ( ! self::is_valid_cache_key($cache_key)) {
+			if ( ! $custom_key) {
+				return new WP_Error('give_invalid_cache_key', __('Cache key format should be give_cache_*', 'give'));
 			}
 
-			$cache_key = self::get_key( $cache_key, $query_args );
+			$cache_key = self::get_key($cache_key, $query_args);
 		}
 
 		$option_value = array(
 			'data'       => $data,
-			'expiration' => ! is_null( $expiration )
-				? ( $expiration + current_time( 'timestamp', 1 ) )
+			'expiration' => ! is_null($expiration)
+				? ($expiration + current_time('timestamp', 1))
 				: null,
 		);
 
-		$result = update_option( $cache_key, $option_value, 'no' );
+		$result = update_option($cache_key, $option_value, 'no');
 
 		return $result;
 	}
@@ -168,27 +168,27 @@ 
 	 * @return bool|WP_Error
 	 */
 
-	public static function delete( $cache_keys ) {
+	public static function delete($cache_keys) {
 		$result = true;
 		$invalid_keys = array();
 
-		if ( ! empty( $cache_keys ) ) {
-			$cache_keys = is_array( $cache_keys ) ? $cache_keys : array( $cache_keys );
+		if ( ! empty($cache_keys)) {
+			$cache_keys = is_array($cache_keys) ? $cache_keys : array($cache_keys);
 
-			foreach ( $cache_keys as $cache_key ) {
-				if ( ! self::is_valid_cache_key( $cache_key ) ) {
+			foreach ($cache_keys as $cache_key) {
+				if ( ! self::is_valid_cache_key($cache_key)) {
 					$invalid_keys[] = $cache_key;
 					$result = false;
 				}
 
-				delete_option( $cache_key );
+				delete_option($cache_key);
 			}
 		}
 
-		if( ! $result ) {
+		if ( ! $result) {
 			$result = new WP_Error(
 				'give_invalid_cache_key',
-					__( 'Cache key format should be give_cache_*', 'give' ),
+					__('Cache key format should be give_cache_*', 'give'),
 					$invalid_keys
 			);
 		}
@@ -207,7 +207,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function delete_all_expired( $force = false ) {
+	public static function delete_all_expired($force = false) {
 		global $wpdb;
 		$options = $wpdb->get_results(
 			$wpdb->prepare(
@@ -221,30 +221,30 @@ 
 		);
 
 		// Bailout.
-		if ( empty( $options ) ) {
+		if (empty($options)) {
 			return false;
 		}
 
-		$current_time = current_time( 'timestamp', 1 );
+		$current_time = current_time('timestamp', 1);
 
 		// Delete log cache.
-		foreach ( $options as $option ) {
-			$option['option_value'] = maybe_unserialize( $option['option_value'] );
+		foreach ($options as $option) {
+			$option['option_value'] = maybe_unserialize($option['option_value']);
 
 			if (
 				(
-					! self::is_valid_cache_key( $option['option_name'] )
-					|| ! is_array( $option['option_value'] ) // Backward compatibility (<1.8.7).
-					|| ! array_key_exists( 'expiration', $option['option_value'] ) // Backward compatibility (<1.8.7).
-					|| empty( $option['option_value']['expiration'] )
-					|| ( $current_time < $option['option_value']['expiration'] )
+					! self::is_valid_cache_key($option['option_name'])
+					|| ! is_array($option['option_value']) // Backward compatibility (<1.8.7).
+					|| ! array_key_exists('expiration', $option['option_value']) // Backward compatibility (<1.8.7).
+					|| empty($option['option_value']['expiration'])
+					|| ($current_time < $option['option_value']['expiration'])
 				)
 				&& ! $force
 			) {
 				continue;
 			}
 
-			self::delete( $option['option_name'] );
+			self::delete($option['option_name']);
 		}
 	}
 
@@ -260,16 +260,16 @@ 
 	 *
 	 * @return array
 	 */
-	public static function get_options_like( $option_name, $fields = false ) {
+	public static function get_options_like($option_name, $fields = false) {
 		global $wpdb;
 
-		if ( empty( $option_name ) ) {
+		if (empty($option_name)) {
 			return array();
 		}
 
 		$field_names = $fields ? 'option_name, option_value' : 'option_name';
 
-		if ( $fields ) {
+		if ($fields) {
 			$options = $wpdb->get_results(
 				$wpdb->prepare(
 					"SELECT {$field_names }
@@ -293,10 +293,10 @@ 
 			);
 		}
 
-		if ( ! empty( $options ) && $fields ) {
-			foreach ( $options as $index => $option ) {
-				$option['option_value'] = maybe_unserialize( $option['option_value'] );
-				$options[ $index ]      = $option;
+		if ( ! empty($options) && $fields) {
+			foreach ($options as $index => $option) {
+				$option['option_value'] = maybe_unserialize($option['option_value']);
+				$options[$index]      = $option;
 			}
 		}
 
@@ -313,8 +313,8 @@ 
 	 *
 	 * @return bool|int
 	 */
-	public static function is_valid_cache_key( $cache_key ) {
-		return ( false !== strpos( $cache_key, 'give_cache_' ) );
+	public static function is_valid_cache_key($cache_key) {
+		return (false !== strpos($cache_key, 'give_cache_'));
 	}
 }
 

includes/admin/tools/logs/class-api-requests-logs-list-table.php

Doc Comments +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
 	 *
 	 * @access public
 	 * @since  1.0
-	 * @return string|bool String if search is present, false otherwise
+	 * @return string|false String if search is present, false otherwise
 	 */
 	public function get_search() {
 		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;

Spacing +65 added lines, -65 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -45,11 +45,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 	}
 
 	/**
@@ -63,20 +63,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
-			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?>
 		</p>
 		<?php
 	}
@@ -90,10 +90,10 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'      => esc_html__( 'Log ID', 'give' ),
-			'details' => esc_html__( 'Request Details', 'give' ),
-			'ip'      => esc_html__( 'Request IP', 'give' ),
-			'date'    => esc_html__( 'Date', 'give' ),
+			'ID'      => esc_html__('Log ID', 'give'),
+			'details' => esc_html__('Request Details', 'give'),
+			'ip'      => esc_html__('Request IP', 'give'),
+			'date'    => esc_html__('Date', 'give'),
 		);
 
 		return $columns;
@@ -110,10 +110,10 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($item, $column_name) {
+		switch ($column_name) {
 			default:
-				return esc_attr( $item[ $column_name ] );
+				return esc_attr($item[$column_name]);
 		}
 	}
 
@@ -127,26 +127,26 @@ 
 	 *
 	 * @return void
 	 */
-	public function column_details( $item ) {
+	public function column_details($item) {
 		?>
-		<a href="#TB_inline?width=640&amp;inlineId=log-details-<?php echo $item['ID']; ?>" class="thickbox"><?php esc_html_e( 'View Request', 'give' ); ?></a>
+		<a href="#TB_inline?width=640&amp;inlineId=log-details-<?php echo $item['ID']; ?>" class="thickbox"><?php esc_html_e('View Request', 'give'); ?></a>
 		<div id="log-details-<?php echo $item['ID']; ?>" style="display:none;">
 			<?php
 
-			$request = get_post_field( 'post_excerpt', $item['ID'] );
-			$error   = get_post_field( 'post_content', $item['ID'] );
-			echo '<p><strong>' . esc_html__( 'API Request:', 'give' ) . '</strong></p>';
-			echo '<div>' . $request . '</div>';
-			if ( ! empty( $error ) ) {
-				echo '<p><strong>' . esc_html__( 'Error', 'give' ) . '</strong></p>';
-				echo '<div>' . esc_html( $error ) . '</div>';
+			$request = get_post_field('post_excerpt', $item['ID']);
+			$error   = get_post_field('post_content', $item['ID']);
+			echo '<p><strong>'.esc_html__('API Request:', 'give').'</strong></p>';
+			echo '<div>'.$request.'</div>';
+			if ( ! empty($error)) {
+				echo '<p><strong>'.esc_html__('Error', 'give').'</strong></p>';
+				echo '<div>'.esc_html($error).'</div>';
 			}
-			echo '<p><strong>' . esc_html__( 'API User:', 'give' ) . '</strong></p>';
-			echo '<div>' . give_get_meta( $item['ID'], '_give_log_user', true ) . '</div>';
-			echo '<p><strong>' . esc_html__( 'API Key:', 'give' ) . '</strong></p>';
-			echo '<div>' . give_get_meta( $item['ID'], '_give_log_key', true ) . '</div>';
-			echo '<p><strong>' . esc_html__( 'Request Date:', 'give' ) . '</strong></p>';
-			echo '<div>' . get_post_field( 'post_date', $item['ID'] ) . '</div>';
+			echo '<p><strong>'.esc_html__('API User:', 'give').'</strong></p>';
+			echo '<div>'.give_get_meta($item['ID'], '_give_log_user', true).'</div>';
+			echo '<p><strong>'.esc_html__('API Key:', 'give').'</strong></p>';
+			echo '<div>'.give_get_meta($item['ID'], '_give_log_key', true).'</div>';
+			echo '<p><strong>'.esc_html__('Request Date:', 'give').'</strong></p>';
+			echo '<div>'.get_post_field('post_date', $item['ID']).'</div>';
 			?>
 		</div>
 		<?php
@@ -160,7 +160,7 @@ 
 	 * @return string|bool String if search is present, false otherwise
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 
@@ -176,19 +176,19 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -210,31 +210,31 @@ 
 
 		$search = $this->get_search();
 
-		if ( $search ) {
-			if ( filter_var( $search, FILTER_VALIDATE_IP ) ) {
+		if ($search) {
+			if (filter_var($search, FILTER_VALIDATE_IP)) {
 				// This is an IP address search
 				$key = '_give_log_request_ip';
-			} elseif ( is_email( $search ) ) {
+			} elseif (is_email($search)) {
 				// This is an email search
-				$userdata = get_user_by( 'email', $search );
+				$userdata = get_user_by('email', $search);
 
-				if ( $userdata ) {
+				if ($userdata) {
 					$search = $userdata->ID;
 				}
 
 				$key = '_give_log_user';
-			} elseif ( strlen( $search ) == 32 ) {
+			} elseif (strlen($search) == 32) {
 				// Look for an API key
 				$key = '_give_log_key';
-			} elseif ( stristr( $search, 'token:' ) ) {
+			} elseif (stristr($search, 'token:')) {
 				// Look for an API token
-				$search = str_ireplace( 'token:', '', $search );
+				$search = str_ireplace('token:', '', $search);
 				$key    = '_give_log_token';
 			} else {
 				// This is (probably) a user ID search
-				$userdata = get_userdata( $search );
+				$userdata = get_userdata($search);
 
-				if ( $userdata ) {
+				if ($userdata) {
 					$search = $userdata->ID;
 				}
 
@@ -260,7 +260,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -270,7 +270,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	function bulk_actions( $which = '' ) {
+	function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -294,14 +294,14 @@ 
 			'posts_per_page' => $this->per_page,
 		);
 
-		$logs = $give_logs->get_connected_logs( $log_query );
+		$logs = $give_logs->get_connected_logs($log_query);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
+		if ($logs) {
+			foreach ($logs as $log) {
 
 				$logs_data[] = array(
 					'ID'   => $log->ID,
-					'ip'   => give_get_meta( $log->ID, '_give_log_request_ip', true ),
+					'ip'   => give_get_meta($log->ID, '_give_log_request_ip', true),
 					'date' => $log->post_date,
 				);
 			}
@@ -329,14 +329,14 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array(); // No hidden columns
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$this->items           = $this->get_logs();
-		$total_items           = $give_logs->get_log_count( 0, 'api_request' );
+		$total_items           = $give_logs->get_log_count(0, 'api_request');
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 			)
 		);
 	}

includes/admin/tools/logs/class-sales-logs-list-table.php

Doc Comments +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
 	 *
 	 * @access public
 	 * @since  1.0
-	 * @return string|bool string If search is present, false otherwise
+	 * @return string|false string If search is present, false otherwise
 	 */
 	public function get_search() {
 		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;

Spacing +81 added lines, -81 removed lines

@@ -9,13 +9,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded.
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -44,13 +44,13 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 
-		add_action( 'give_log_view_actions', array( $this, 'give_forms_filter' ) );
+		add_action('give_log_view_actions', array($this, 'give_forms_filter'));
 	}
 
 	/**
@@ -64,41 +64,41 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		$payment = give_get_payment_by( 'id', $item['payment_id'] );
+		$payment = give_get_payment_by('id', $item['payment_id']);
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'form' :
-				$form_title = get_the_title( $item[ $column_name ] );
-				$form_title = empty( $form_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $item[ $column_name ] ) : $form_title;
-				return '<a href="' . esc_url( add_query_arg( 'form', $item[ $column_name ] ) ) . '" >' . esc_html( $form_title ). '</a>';
+				$form_title = get_the_title($item[$column_name]);
+				$form_title = empty($form_title) ? sprintf(__('Untitled (#%s)', 'give'), $item[$column_name]) : $form_title;
+				return '<a href="'.esc_url(add_query_arg('form', $item[$column_name])).'" >'.esc_html($form_title).'</a>';
 
 			case 'donor_id' :
 				return sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $item['donor_id'] ) ),
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($item['donor_id'])),
 					$item['donor_name']
 				);
 
 			case 'amount' :
-				return give_currency_filter( give_format_amount( $item['amount'], array( 'sanitize' => false, 'donation_id' => $item['payment_id'] ) ), give_get_payment_currency_code( $item['payment_id'] ) );
+				return give_currency_filter(give_format_amount($item['amount'], array('sanitize' => false, 'donation_id' => $item['payment_id'])), give_get_payment_currency_code($item['payment_id']));
 
 			case 'status' :
 
-				$value = '<div class="give-donation-status status-' . sanitize_title( give_get_payment_status( $payment, true ) ) . '"><span class="give-donation-status-icon"></span> ' . give_get_payment_status( $payment, true ) . '</div>';
+				$value = '<div class="give-donation-status status-'.sanitize_title(give_get_payment_status($payment, true)).'"><span class="give-donation-status-icon"></span> '.give_get_payment_status($payment, true).'</div>';
 
-				if ( $payment->mode == 'test' ) {
-					$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="' . esc_attr__( 'This donation was made in test mode.', 'give' ) . '">' . __( 'Test', 'give' ) . '</span>';
+				if ($payment->mode == 'test') {
+					$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="'.esc_attr__('This donation was made in test mode.', 'give').'">'.__('Test', 'give').'</span>';
 				}
 
 				return $value;
 
 			case 'payment_id' :
-				return '<a href="' . admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $item['payment_id'] ) . '">' . give_get_payment_number( $item['payment_id'] ) . '</a>';
+				return '<a href="'.admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$item['payment_id']).'">'.give_get_payment_number($item['payment_id']).'</a>';
 
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -111,13 +111,13 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'         => __( 'Log ID', 'give' ),
-			'donor_id'   => __( 'Donor', 'give' ),
-			'form'       => __( 'Form', 'give' ),
-			'amount'     => __( 'Donation Amount', 'give' ),
-			'status'     => __( 'Status', 'give' ),
-			'payment_id' => __( 'Donation ID', 'give' ),
-			'date'       => __( 'Date', 'give' ),
+			'ID'         => __('Log ID', 'give'),
+			'donor_id'   => __('Donor', 'give'),
+			'form'       => __('Form', 'give'),
+			'amount'     => __('Donation Amount', 'give'),
+			'status'     => __('Status', 'give'),
+			'payment_id' => __('Donation ID', 'give'),
+			'date'       => __('Date', 'give'),
 		);
 
 		return $columns;
@@ -131,7 +131,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -142,7 +142,7 @@ 
 	 * @return mixed int If User ID, string If Email/Login
 	 */
 	public function get_filtered_user() {
-		return isset( $_GET['user'] ) ? absint( $_GET['user'] ) : false;
+		return isset($_GET['user']) ? absint($_GET['user']) : false;
 	}
 
 	/**
@@ -153,7 +153,7 @@ 
 	 * @return int Download ID
 	 */
 	public function get_filtered_give_form() {
-		return ! empty( $_GET['form'] ) ? absint( $_GET['form'] ) : false;
+		return ! empty($_GET['form']) ? absint($_GET['form']) : false;
 	}
 
 	/**
@@ -164,7 +164,7 @@ 
 	 * @return string|bool string If search is present, false otherwise
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 
@@ -180,20 +180,20 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -217,7 +217,7 @@ 
 
 		$meta_query = array();
 
-		if ( $user ) {
+		if ($user) {
 			// Show only logs from a specific user.
 			$meta_query[] = array(
 				'key'   => '_give_log_user_id',
@@ -226,8 +226,8 @@ 
 		}
 
 		$search = $this->get_search();
-		if ( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				// This is an email search. We use this to ensure it works for guest users and logged-in users.
 				$key     = '_give_log_user_info';
 				$compare = 'LIKE';
@@ -236,32 +236,32 @@ 
 				$key     = '_give_log_user_id';
 				$compare = 'LIKE';
 
-				if ( ! is_numeric( $search ) ) {
+				if ( ! is_numeric($search)) {
 					// Searching for user by username
-					$user = get_user_by( 'login', $search );
+					$user = get_user_by('login', $search);
 
-					if ( $user ) {
+					if ($user) {
 						// Found one, set meta value to user's ID.
 						$search = $user->ID;
 					} else {
 						// No user found so let's do a real search query.
-						$users = new WP_User_Query( array(
+						$users = new WP_User_Query(array(
 							'search'         => $search,
-							'search_columns' => array( 'user_url', 'user_nicename' ),
+							'search_columns' => array('user_url', 'user_nicename'),
 							'number'         => 1,
 							'fields'         => 'ids',
-						) );
+						));
 
 						$found_user = $users->get_results();
 
-						if ( $found_user ) {
+						if ($found_user) {
 							$search = $found_user[0];
 						}
 					}
 				}
 			}
 
-			if ( ! $this->file_search ) {
+			if ( ! $this->file_search) {
 				// Meta query only works for non file name search.
 				$meta_query[] = array(
 					'key'     => $key,
@@ -282,7 +282,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	function bulk_actions( $which = '' ) {
+	function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -294,24 +294,24 @@ 
 	 * @return void
 	 */
 	public function give_forms_filter() {
-		$give_forms = get_posts( array(
+		$give_forms = get_posts(array(
 			'post_type'              => 'give_forms',
 			'post_status'            => 'any',
-			'posts_per_page'         => - 1,
+			'posts_per_page'         => -1,
 			'orderby'                => 'title',
 			'order'                  => 'ASC',
 			'fields'                 => 'ids',
 			'update_post_meta_cache' => false,
 			'update_post_term_cache' => false,
-		) );
+		));
 
-		if ( $give_forms ) {
+		if ($give_forms) {
 			echo '<select name="form" id="give-log-form-filter">';
-			echo '<option value="0">' . __( 'All', 'give' ) . '</option>';
-			foreach ( $give_forms as $form ) {
-				$form_title = get_the_title( $form );
-				$form_title = empty( $form_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $form ) : $form_title;
-				echo '<option value="' . $form . '"' . selected( $form, $this->get_filtered_give_form() ) . '>' . esc_html( $form_title ) . '</option>';
+			echo '<option value="0">'.__('All', 'give').'</option>';
+			foreach ($give_forms as $form) {
+				$form_title = get_the_title($form);
+				$form_title = empty($form_title) ? sprintf(__('Untitled (#%s)', 'give'), $form) : $form_title;
+				echo '<option value="'.$form.'"'.selected($form, $this->get_filtered_give_form()).'>'.esc_html($form_title).'</option>';
 			}
 			echo '</select>';
 		}
@@ -331,7 +331,7 @@ 
 
 		$logs_data = array();
 		$paged     = $this->get_paged();
-		$give_form = empty( $_GET['s'] ) ? $this->get_filtered_give_form() : null;
+		$give_form = empty($_GET['s']) ? $this->get_filtered_give_form() : null;
 		$user      = $this->get_filtered_user();
 
 		$log_query = array(
@@ -342,37 +342,37 @@ 
 			'posts_per_page' => $this->per_page,
 		);
 
-		$cache_key = Give_Cache::get_key( 'get_logs', $log_query );
+		$cache_key = Give_Cache::get_key('get_logs', $log_query);
 
 		// Return result from cache if exist.
-		if ( ! ( $logs_data = Give_Cache::get( $cache_key ) ) ) {
-			$logs = $give_logs->get_connected_logs( $log_query );
+		if ( ! ($logs_data = Give_Cache::get($cache_key))) {
+			$logs = $give_logs->get_connected_logs($log_query);
 
-			if ( $logs ) {
-				foreach ( $logs as $log ) {
-					$payment_id = give_get_meta( $log->ID, '_give_log_payment_id', true );
+			if ($logs) {
+				foreach ($logs as $log) {
+					$payment_id = give_get_meta($log->ID, '_give_log_payment_id', true);
 
 					// Make sure this payment hasn't been deleted.
-					if ( get_post( $payment_id ) ) :
-						$user_info      = give_get_payment_meta_user_info( $payment_id );
-						$payment_amount = give_donation_amount( $payment_id );
+					if (get_post($payment_id)) :
+						$user_info      = give_get_payment_meta_user_info($payment_id);
+						$payment_amount = give_donation_amount($payment_id);
 
 						$logs_data[] = array(
-							'ID'         => '<span class="give-item-label give-item-label-gray">' . $log->ID . '</span>',
+							'ID'         => '<span class="give-item-label give-item-label-gray">'.$log->ID.'</span>',
 							'payment_id' => $payment_id,
 							'form'       => $log->post_parent,
 							'amount'     => $payment_amount,
-							'donor_id'   => give_get_payment_donor_id( $payment_id ),
-							'donor_name' => trim( "{$user_info['first_name']} {$user_info['last_name']}" ),
-							'date'       => get_post_field( 'post_date', $payment_id ),
+							'donor_id'   => give_get_payment_donor_id($payment_id),
+							'donor_name' => trim("{$user_info['first_name']} {$user_info['last_name']}"),
+							'date'       => get_post_field('post_date', $payment_id),
 						);
 
 					endif;
 				}
 
 				// Cache results.
-				if ( ! empty( $logs_data ) ) {
-					Give_Cache::set( $cache_key, $logs_data );
+				if ( ! empty($logs_data)) {
+					Give_Cache::set($cache_key, $logs_data);
 				}
 			}
 		}
@@ -400,15 +400,15 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array();
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$current_page          = $this->get_pagenum();
 		$this->items           = $this->get_logs();
-		$total_items           = $give_logs->get_log_count( $this->get_filtered_give_form(), 'sale', $this->get_meta_query() );
+		$total_items           = $give_logs->get_log_count($this->get_filtered_give_form(), 'sale', $this->get_meta_query());
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 			)
 		);
 	}