WordImpress / Give

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

Spacing +6 added lines, -6 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,10 +21,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = esc_html__( 'Register', 'give' );
-		$this->shortcode['label'] = esc_html__( 'Register', 'give' );
+		$this->shortcode['title'] = esc_html__('Register', 'give');
+		$this->shortcode['label'] = esc_html__('Register', 'give');
 
-		parent::__construct( 'give_register' );
+		parent::__construct('give_register');
 	}
 
 	/**
@@ -37,13 +37,13 @@ 
 		return array(
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="no-margin">%s</p>', esc_html__( 'Redirect URL (optional):', 'give' ) ),
+				'html' => sprintf('<p class="no-margin">%s</p>', esc_html__('Redirect URL (optional):', 'give')),
 			),
 			array(
 				'type'     => 'textbox',
 				'name'     => 'redirect',
 				'minWidth' => 320,
-				'tooltip'  => esc_attr__( 'Enter an URL here to redirect to after registering.', 'give' ),
+				'tooltip'  => esc_attr__('Enter an URL here to redirect to after registering.', 'give'),
 			),
 		);
 	}

includes/admin/shortcodes/shortcode-give-donation-history.php

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,9 +21,9 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['label'] = esc_html__( 'Donation History', 'give' );
+		$this->shortcode['label'] = esc_html__('Donation History', 'give');
 
-		parent::__construct( 'donation_history' );
+		parent::__construct('donation_history');
 	}
 }
 

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/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/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/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 +78 added lines, -78 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,69 +64,69 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
 		/* @var Give_Payment $payment */
-		$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 'amount' :
-				$value = give_currency_filter( give_format_amount( $item['amount'], array( 'sanitize' => false, 'donation_id' => $item['payment_id'] ) ), array( 'currency_code' => give_get_payment_currency_code( $item['payment_id'] ) ) );
-				$value .= sprintf( '<br><small>%1$s %2$s</small>', __( 'via', 'give' ), give_get_gateway_admin_label( $payment->gateway ) );
+				$value = give_currency_filter(give_format_amount($item['amount'], array('sanitize' => false, 'donation_id' => $item['payment_id'])), array('currency_code' => give_get_payment_currency_code($item['payment_id'])));
+				$value .= sprintf('<br><small>%1$s %2$s</small>', __('via', 'give'), give_get_gateway_admin_label($payment->gateway));
 
 				return $value;
 
 			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 .= Give()->tooltips->render_span( array(
-						'label'       => __( 'This donation was made in test mode.', 'give' ),
-						'tag_content' => __( 'Test', 'give' ),
+				if ($payment->mode == 'test') {
+					$value .= Give()->tooltips->render_span(array(
+						'label'       => __('This donation was made in test mode.', 'give'),
+						'tag_content' => __('Test', 'give'),
 						'attributes'  => array(
 							'class' => 'give-item-label give-item-label-orange give-test-mode-transactions-label',
 						),
-					) );
+					));
 				}
 
 				return $value;
 
 			case 'donation' :
-				$serial_code = Give()->seq_donation_number->get_serial_code( $payment );
-				$value = Give()->tooltips->render_link( array(
+				$serial_code = Give()->seq_donation_number->get_serial_code($payment);
+				$value = Give()->tooltips->render_link(array(
 					/* translators: %s Sequential Donation ID. */
-					'label'       => sprintf( __( 'View Donation %s', 'give' ), $serial_code ),
+					'label'       => sprintf(__('View Donation %s', 'give'), $serial_code),
 					'tag_content' => $serial_code,
-					'link'        => esc_url( add_query_arg( 'id', $payment->ID, admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details' ) ) ),
-				) );
+					'link'        => esc_url(add_query_arg('id', $payment->ID, admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details'))),
+				));
 
-				if ( ! empty( $item['donor_id'] ) ) {
-					$title_prefix = Give()->donor_meta->get_meta( $item['donor_id'], '_give_donor_title_prefix', true );
+				if ( ! empty($item['donor_id'])) {
+					$title_prefix = Give()->donor_meta->get_meta($item['donor_id'], '_give_donor_title_prefix', true);
 
 					$value .= sprintf(
 						'&nbsp;%1$s&nbsp;<a href="%2$s">%3$s</a><br>',
-						esc_html__( 'by', 'give' ),
-						admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . $item['donor_id'] ),
-						give_get_donor_name_with_title_prefixes( $title_prefix, $item['donor_name'] )
+						esc_html__('by', 'give'),
+						admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.$item['donor_id']),
+						give_get_donor_name_with_title_prefixes($title_prefix, $item['donor_name'])
 					);
 				} else {
 					$value .= sprintf(
 						'&nbsp;%1$s&nbsp;%2$s<br>',
-						esc_html__( 'by', 'give' ),
-						__( 'No donor attached', 'give' )
+						esc_html__('by', 'give'),
+						__('No donor attached', 'give')
 					);
 				}
 
 				return $value;
 
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		} // End switch().
 	}
 
@@ -139,12 +139,12 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'       => __( 'Log ID', 'give' ),
-			'donation' => __( 'Donation', 'give' ),
-			'form'     => __( 'Form', 'give' ),
-			'status'   => __( 'Status', 'give' ),
-			'amount'   => __( 'Donation Amount', 'give' ),
-			'date'     => __( 'Date', 'give' ),
+			'ID'       => __('Log ID', 'give'),
+			'donation' => __('Donation', 'give'),
+			'form'     => __('Form', 'give'),
+			'status'   => __('Status', 'give'),
+			'amount'   => __('Donation Amount', 'give'),
+			'date'     => __('Date', 'give'),
 		);
 
 		return $columns;
@@ -158,7 +158,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;
 	}
 
 	/**
@@ -169,7 +169,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;
 	}
 
 	/**
@@ -180,7 +180,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;
 	}
 
 	/**
@@ -191,7 +191,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;
 	}
 
 
@@ -207,19 +207,19 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
-			<?php if ( 'top' === $which ) : ?>
+			<?php if ('top' === $which) : ?>
 				<div class="alignleft actions bulkactions">
-					<?php $this->bulk_actions( $which ); ?>
+					<?php $this->bulk_actions($which); ?>
 				</div>
 			<?php endif; ?>
 
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -244,7 +244,7 @@ 
 
 		$meta_query = array();
 
-		if ( $user ) {
+		if ($user) {
 			// Show only logs from a specific user.
 			$meta_query[] = array(
 				'key'   => '_give_log_user_id',
@@ -252,7 +252,7 @@ 
 			);
 		}
 
-		if ( $give_form ) {
+		if ($give_form) {
 			$meta_query[] = array(
 				'key'   => '_give_log_form_id',
 				'value' => $give_form,
@@ -260,8 +260,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';
@@ -270,32 +270,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,
@@ -317,7 +317,7 @@ 
 	 * @param string $which
 	 * @return void
 	 */
-	function bulk_actions( $which = '' ) {
+	function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -329,12 +329,12 @@ 
 	 * @return void
 	 */
 	public function give_forms_filter() {
-		echo Give()->html->forms_dropdown( array(
+		echo Give()->html->forms_dropdown(array(
 			'selected' => $this->get_filtered_give_form(),
 			'name'   => 'form',
 			'id'     => 'give-log-form-filter',
 			'chosen' => true,
-		) );
+		));
 	}
 
 	/**
@@ -348,22 +348,22 @@ 
 	public function get_logs() {
 		$logs_data = array();
 		$log_query = $this->get_query_params();
-		$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) {
 				/* @var Give_payment $payment */
-				$payment = new Give_Payment( $log->log_parent );
+				$payment = new Give_Payment($log->log_parent);
 
 				// Make sure this payment hasn't been deleted
-				if ( get_post( $payment->ID ) ) :
+				if (get_post($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'       => $payment->form_id,
 						'amount'     => $payment->total,
 						'donor_id'    => $payment->customer_id,
-						'donor_name' => trim( "{$payment->first_name} $payment->last_name" ),
+						'donor_name' => trim("{$payment->first_name} $payment->last_name"),
 						'date'       => $payment->date,
 					);
 
@@ -391,15 +391,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( 0, 'sale', $this->get_meta_query() );
+		$total_items           = Give()->logs->get_log_count(0, '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),
 			)
 		);
 	}

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

Spacing +51 added lines, -51 removed lines

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

includes/class-give-license-handler.php

Doc Comments +3 added lines, -3 removed lines

@@ -703,7 +703,7 @@ 
 		 * @access private
 		 * @since  1.7
 		 *
-		 * @return bool
+		 * @return false|null
 		 */
 		private function __remove_license_key_from_subscriptions() {
 			$subscriptions = get_option( 'give_subscriptions', array() );
@@ -736,7 +736,7 @@ 
 		 * @param $plugin_data
 		 * @param $status
 		 *
-		 * @return bool
+		 * @return false|null
 		 */
 		public function plugin_page_notices( $plugin_file, $plugin_data, $status ) {
 			// Bailout.
@@ -758,7 +758,7 @@ 
 		 *
 		 * @since  1.8.7
 		 * @access public
-		 * @return array
+		 * @return string
 		 */
 		public function license_state_message() {
 			$message_data = array();

Spacing +153 added lines, -153 removed lines

@@ -10,11 +10,11 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_License' ) ) :
+if ( ! class_exists('Give_License')) :
 
 	/**
 	 * Give_License Class
@@ -153,31 +153,31 @@ 
 		 * @param string $_checkout_url
 		 * @param string $_account_url
 		 */
-		public function __construct( $_file, $_item_name, $_version, $_author, $_optname = null, $_api_url = null, $_checkout_url = null, $_account_url = null ) {
+		public function __construct($_file, $_item_name, $_version, $_author, $_optname = null, $_api_url = null, $_checkout_url = null, $_account_url = null) {
 
 			$give_options = give_get_settings();
 
 			$this->file             = $_file;
 			$this->item_name        = $_item_name;
-			$this->item_shortname   = self::get_short_name( $this->item_name );
+			$this->item_shortname   = self::get_short_name($this->item_name);
 			$this->version          = $_version;
-			$this->license          = isset( $give_options[ $this->item_shortname . '_license_key' ] ) ? trim( $give_options[ $this->item_shortname . '_license_key' ] ) : '';
-			$this->license_data     = __give_get_active_license_info( $this->item_shortname );
+			$this->license          = isset($give_options[$this->item_shortname.'_license_key']) ? trim($give_options[$this->item_shortname.'_license_key']) : '';
+			$this->license_data     = __give_get_active_license_info($this->item_shortname);
 			$this->author           = $_author;
-			$this->api_url          = is_null( $_api_url ) ? $this->api_url : $_api_url;
-			$this->checkout_url     = is_null( $_checkout_url ) ? $this->checkout_url : $_checkout_url;
-			$this->account_url      = is_null( $_account_url ) ? $this->account_url : $_account_url;
+			$this->api_url          = is_null($_api_url) ? $this->api_url : $_api_url;
+			$this->checkout_url     = is_null($_checkout_url) ? $this->checkout_url : $_checkout_url;
+			$this->account_url      = is_null($_account_url) ? $this->account_url : $_account_url;
 			$this->auto_updater_obj = null;
 
 			// Add Setting for Give Add-on activation status.
-			$is_addon_activated = get_option( 'give_is_addon_activated' );
-			if ( ! $is_addon_activated && is_object( $this ) ) {
-				update_option( 'give_is_addon_activated', true, false );
-				Give_Cache::set( 'give_cache_hide_license_notice_after_activation', true, DAY_IN_SECONDS );
+			$is_addon_activated = get_option('give_is_addon_activated');
+			if ( ! $is_addon_activated && is_object($this)) {
+				update_option('give_is_addon_activated', true, false);
+				Give_Cache::set('give_cache_hide_license_notice_after_activation', true, DAY_IN_SECONDS);
 			}
 
 			// Add plugin to registered licenses list.
-			array_push( self::$licensed_addons, plugin_basename( $this->file ) );
+			array_push(self::$licensed_addons, plugin_basename($this->file));
 
 			// Setup hooks
 			$this->includes();
@@ -196,9 +196,9 @@ 
 		 *
 		 * @return string
 		 */
-		public static function get_short_name( $plugin_name ) {
-			$plugin_name = trim( str_replace( 'Give - ', '', $plugin_name ) );
-			$plugin_name = 'give_' . preg_replace( '/[^a-zA-Z0-9_\s]/', '', str_replace( ' ', '_', strtolower( $plugin_name ) ) );
+		public static function get_short_name($plugin_name) {
+			$plugin_name = trim(str_replace('Give - ', '', $plugin_name));
+			$plugin_name = 'give_'.preg_replace('/[^a-zA-Z0-9_\s]/', '', str_replace(' ', '_', strtolower($plugin_name)));
 
 			return $plugin_name;
 		}
@@ -215,7 +215,7 @@ 
 		 */
 		private function includes() {
 
-			if ( ! class_exists( 'EDD_SL_Plugin_Updater' ) ) {
+			if ( ! class_exists('EDD_SL_Plugin_Updater')) {
 				require_once 'admin/EDD_SL_Plugin_Updater.php';
 			}
 		}
@@ -233,28 +233,28 @@ 
 		private function hooks() {
 
 			// Register settings.
-			add_filter( 'give_settings_licenses', array( $this, 'settings' ), 1 );
+			add_filter('give_settings_licenses', array($this, 'settings'), 1);
 
 			// Activate license key on settings save.
-			add_action( 'admin_init', array( $this, 'activate_license' ), 10 );
+			add_action('admin_init', array($this, 'activate_license'), 10);
 
 			// Deactivate license key.
-			add_action( 'admin_init', array( $this, 'deactivate_license' ), 11 );
+			add_action('admin_init', array($this, 'deactivate_license'), 11);
 
 			// Updater.
-			add_action( 'admin_init', array( $this, 'auto_updater' ), 0 );
-			add_action( 'admin_notices', array( $this, 'notices' ) );
+			add_action('admin_init', array($this, 'auto_updater'), 0);
+			add_action('admin_notices', array($this, 'notices'));
 
 			// Check license weekly.
-			Give_Cron::add_weekly_event( array( $this, 'weekly_license_check' ) );
+			Give_Cron::add_weekly_event(array($this, 'weekly_license_check'));
 
 			// Check subscription weekly.
-			Give_Cron::add_weekly_event( array( $this, 'weekly_subscription_check' ) );
+			Give_Cron::add_weekly_event(array($this, 'weekly_subscription_check'));
 
 			// Show addon notice on plugin page.
-			$plugin_name = explode( 'plugins/', $this->file );
-			$plugin_name = end( $plugin_name );
-			add_action( "after_plugin_row_{$plugin_name}", array( $this, 'plugin_page_notices' ), 10, 3 );
+			$plugin_name = explode('plugins/', $this->file);
+			$plugin_name = end($plugin_name);
+			add_action("after_plugin_row_{$plugin_name}", array($this, 'plugin_page_notices'), 10, 3);
 
 		}
 
@@ -294,16 +294,16 @@ 
 		 *
 		 * @return array           License settings.
 		 */
-		public function settings( $settings ) {
+		public function settings($settings) {
 
 			$give_license_settings = array(
 				array(
 					'name'    => $this->item_name,
-					'id'      => $this->item_shortname . '_license_key',
+					'id'      => $this->item_shortname.'_license_key',
 					'desc'    => '',
 					'type'    => 'license_key',
 					'options' => array(
-						'license'      => get_option( $this->item_shortname . '_license_active' ),
+						'license'      => get_option($this->item_shortname.'_license_active'),
 						'shortname'    => $this->item_shortname,
 						'item_name'    => $this->item_name,
 						'api_url'      => $this->api_url,
@@ -314,7 +314,7 @@ 
 				),
 			);
 
-			return array_merge( $settings, $give_license_settings );
+			return array_merge($settings, $give_license_settings);
 		}
 
 		/**
@@ -329,18 +329,18 @@ 
 		 *
 		 * @return array           License settings content.
 		 */
-		public function license_settings_content( $settings ) {
+		public function license_settings_content($settings) {
 
 			$give_license_settings = array(
 				array(
-					'name' => __( 'Add-on Licenses', 'give' ),
+					'name' => __('Add-on Licenses', 'give'),
 					'desc' => '<hr>',
 					'type' => 'give_title',
 					'id'   => 'give_title',
 				),
 			);
 
-			return array_merge( $settings, $give_license_settings );
+			return array_merge($settings, $give_license_settings);
 		}
 
 		/**
@@ -355,62 +355,62 @@ 
 		 */
 		public function activate_license() {
 			// Bailout.
-			if ( ! $this->__is_user_can_edit_license() ) {
+			if ( ! $this->__is_user_can_edit_license()) {
 				return;
 			}
 
 			// Allow third party addon developers to handle license activation.
-			if ( $this->__is_third_party_addon() ) {
-				do_action( 'give_activate_license', $this );
+			if ($this->__is_third_party_addon()) {
+				do_action('give_activate_license', $this);
 
 				return;
 			}
 
 			// Delete previous license setting if a empty license key submitted.
-			if ( empty( $_POST["{$this->item_shortname}_license_key"] ) ) {
+			if (empty($_POST["{$this->item_shortname}_license_key"])) {
 				$this->unset_license();
 
 				return;
 			}
 
 			// Do not simultaneously activate add-ons if the user want to deactivate a specific add-on.
-			if( $this->is_deactivating_license() ) {
+			if ($this->is_deactivating_license()) {
 				return;
 			}
 
 			// Check if plugin previously installed.
-			if ( $this->is_valid_license() ) {
+			if ($this->is_valid_license()) {
 				return;
 			}
 
 			// Get license key.
-			$this->license = sanitize_text_field( $_POST[ $this->item_shortname . '_license_key' ] );
+			$this->license = sanitize_text_field($_POST[$this->item_shortname.'_license_key']);
 
 			// Delete previous license key from subscription if previously added.
 			$this->__remove_license_key_from_subscriptions();
 
 			// Make sure there are no api errors.
-			if ( ! ( $license_data = $this->get_license_info( 'activate_license' ) ) ) {
+			if ( ! ($license_data = $this->get_license_info('activate_license'))) {
 				return;
 			}
 
 			// Make sure license is valid.
 			// return because admin will want to activate license again.
-			if ( ! $this->is_license( $license_data ) ) {
+			if ( ! $this->is_license($license_data)) {
 				// Add license key.
-				give_update_option( "{$this->item_shortname}_license_key", $this->license );
+				give_update_option("{$this->item_shortname}_license_key", $this->license);
 
 				return;
 			}
 
 			// Tell WordPress to look for updates.
-			set_site_transient( 'update_plugins', null );
+			set_site_transient('update_plugins', null);
 
 			// Add license data.
-			update_option( "{$this->item_shortname}_license_active", $license_data, false );
+			update_option("{$this->item_shortname}_license_active", $license_data, false);
 
 			// Add license key.
-			give_update_option( "{$this->item_shortname}_license_key", $this->license );
+			give_update_option("{$this->item_shortname}_license_key", $this->license);
 
 			// Check subscription for license key and store this to db (if any).
 			$this->__single_subscription_check();
@@ -428,19 +428,19 @@ 
 		 */
 		public function deactivate_license() {
 			// Bailout.
-			if ( ! $this->__is_user_can_edit_license() ) {
+			if ( ! $this->__is_user_can_edit_license()) {
 				return;
 			}
 
 			// Allow third party add-on developers to handle license deactivation.
-			if ( $this->__is_third_party_addon() ) {
-				do_action( 'give_deactivate_license', $this );
+			if ($this->__is_third_party_addon()) {
+				do_action('give_deactivate_license', $this);
 
 				return;
 			}
 
 			// Run on deactivate button press.
-			if ( isset( $_POST[ $this->item_shortname . '_license_key_deactivate' ] ) ) {
+			if (isset($_POST[$this->item_shortname.'_license_key_deactivate'])) {
 				$this->unset_license();
 			}
 		}
@@ -456,30 +456,30 @@ 
 		public function weekly_license_check() {
 
 			if (
-				! empty( $_POST['give_settings'] ) ||
-				empty( $this->license )
+				! empty($_POST['give_settings']) ||
+				empty($this->license)
 			) {
 				return;
 			}
 
 			// Allow third party add-on developers to handle their license check.
-			if ( $this->__is_third_party_addon() ) {
-				do_action( 'give_weekly_license_check', $this );
+			if ($this->__is_third_party_addon()) {
+				do_action('give_weekly_license_check', $this);
 
 				return;
 			}
 
 			// Make sure there are no api errors.
-			if ( ! ( $license_data = $this->get_license_info( 'check_license' ) ) ) {
+			if ( ! ($license_data = $this->get_license_info('check_license'))) {
 				return;
 			}
 
 			// Bailout.
-			if ( ! $this->is_license( $license_data ) ) {
+			if ( ! $this->is_license($license_data)) {
 				return;
 			}
 
-			update_option( $this->item_shortname . '_license_active', $license_data, false );
+			update_option($this->item_shortname.'_license_active', $license_data, false);
 
 			return;
 		}
@@ -495,21 +495,21 @@ 
 		public function weekly_subscription_check() {
 			// Bailout.
 			if (
-				! empty( $_POST['give_settings'] ) ||
-				empty( $this->license )
+				! empty($_POST['give_settings']) ||
+				empty($this->license)
 			) {
 				return;
 			}
 
 			// Remove old subscription data.
-			if ( absint( get_option( '_give_subscriptions_edit_last', true ) ) < current_time( 'timestamp', 1 ) ) {
-				delete_option( 'give_subscriptions' );
-				update_option( '_give_subscriptions_edit_last', strtotime( '+ 1 day', current_time( 'timestamp', 1 ) ), false );
+			if (absint(get_option('_give_subscriptions_edit_last', true)) < current_time('timestamp', 1)) {
+				delete_option('give_subscriptions');
+				update_option('_give_subscriptions_edit_last', strtotime('+ 1 day', current_time('timestamp', 1)), false);
 			}
 
 			// Allow third party add-on developers to handle their subscription check.
-			if ( $this->__is_third_party_addon() ) {
-				do_action( 'give_weekly_subscription_check', $this );
+			if ($this->__is_third_party_addon()) {
+				do_action('give_weekly_subscription_check', $this);
 
 				return;
 			}
@@ -526,7 +526,7 @@ 
 		 * @return void
 		 */
 		private function __single_subscription_check() {
-			if ( empty( $this->license ) ) {
+			if (empty($this->license)) {
 				return;
 			}
 
@@ -537,26 +537,26 @@ 
 			 * By default edd software licensing api does not have api to check subscription.
 			 * This is a custom feature to check subscriptions.
 			 */
-			$subscription_data = $this->get_license_info( 'check_subscription', true );
+			$subscription_data = $this->get_license_info('check_subscription', true);
 
-			if ( ! empty( $subscription_data['success'] ) && absint( $subscription_data['success'] ) ) {
+			if ( ! empty($subscription_data['success']) && absint($subscription_data['success'])) {
 
-				$subscriptions = get_option( 'give_subscriptions', array() );
+				$subscriptions = get_option('give_subscriptions', array());
 
 				// Update subscription data only if subscription does not exist already.
-				$subscriptions[ $subscription_data['id'] ] = $subscription_data;
+				$subscriptions[$subscription_data['id']] = $subscription_data;
 
 				// Initiate default set of license for subscription.
-				if ( ! isset( $subscriptions[ $subscription_data['id'] ]['licenses'] ) ) {
-					$subscriptions[ $subscription_data['id'] ]['licenses'] = array();
+				if ( ! isset($subscriptions[$subscription_data['id']]['licenses'])) {
+					$subscriptions[$subscription_data['id']]['licenses'] = array();
 				}
 
 				// Store licenses for subscription.
-				if ( ! in_array( $this->license, $subscriptions[ $subscription_data['id'] ]['licenses'] ) ) {
-					$subscriptions[ $subscription_data['id'] ]['licenses'][] = $this->license;
+				if ( ! in_array($this->license, $subscriptions[$subscription_data['id']]['licenses'])) {
+					$subscriptions[$subscription_data['id']]['licenses'][] = $this->license;
 				}
 
-				update_option( 'give_subscriptions', $subscriptions, false );
+				update_option('give_subscriptions', $subscriptions, false);
 			}
 		}
 
@@ -570,12 +570,12 @@ 
 		 */
 		public function notices() {
 
-			if ( ! current_user_can( 'manage_give_settings' ) ) {
+			if ( ! current_user_can('manage_give_settings')) {
 				return;
 			}
 
 			// Do not show licenses notices on license tab.
-			if ( 'licenses' === give_get_current_setting_tab() ) {
+			if ('licenses' === give_get_current_setting_tab()) {
 				return;
 			}
 
@@ -584,118 +584,118 @@ 
 			static $addon_license_key_in_subscriptions;
 
 			// Set default value.
-			$addon_license_key_in_subscriptions = ! empty( $addon_license_key_in_subscriptions ) ? $addon_license_key_in_subscriptions : array();
+			$addon_license_key_in_subscriptions = ! empty($addon_license_key_in_subscriptions) ? $addon_license_key_in_subscriptions : array();
 			$messages                           = array();
 
 			// Check whether admin has Give Add-on activated since 24 hours?
-			$is_license_notice_hidden = Give_Cache::get( 'give_cache_hide_license_notice_after_activation' );
+			$is_license_notice_hidden = Give_Cache::get('give_cache_hide_license_notice_after_activation');
 
 			// Display Invalid License notice, if its more than 24 hours since first Give Add-on activation.
 			if (
-				empty( $this->license )
-				&& empty( $showed_invalid_message )
-				&& ( false === $is_license_notice_hidden )
+				empty($this->license)
+				&& empty($showed_invalid_message)
+				&& (false === $is_license_notice_hidden)
 			) {
 
-				Give()->notices->register_notice( array(
+				Give()->notices->register_notice(array(
 					'id'               => 'give-invalid-license',
 					'type'             => 'error',
 					'description'      => sprintf(
-						__( 'You have invalid or expired license keys for one or more Give Add-ons. Please go to the <a href="%s">licenses page</a> to correct this issue.', 'give' ),
-						admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=licenses' )
+						__('You have invalid or expired license keys for one or more Give Add-ons. Please go to the <a href="%s">licenses page</a> to correct this issue.', 'give'),
+						admin_url('edit.php?post_type=give_forms&page=give-settings&tab=licenses')
 					),
 					'dismissible_type' => 'user',
 					'dismiss_interval' => 'shortly',
-				) );
+				));
 
 				$showed_invalid_message = true;
 
 			}
 
 			// Get subscriptions.
-			$subscriptions = get_option( 'give_subscriptions' );
+			$subscriptions = get_option('give_subscriptions');
 
 			// Show subscription messages.
-			if ( ! empty( $subscriptions ) && ! $showed_subscriptions_message ) {
+			if ( ! empty($subscriptions) && ! $showed_subscriptions_message) {
 
-				foreach ( $subscriptions as $subscription ) {
+				foreach ($subscriptions as $subscription) {
 					// Subscription expires timestamp.
-					$subscription_expires = strtotime( $subscription['expires'] );
+					$subscription_expires = strtotime($subscription['expires']);
 
 					// Start showing subscriptions message before one week of renewal date.
-					if ( strtotime( '- 7 days', $subscription_expires ) > current_time( 'timestamp', 1 ) ) {
+					if (strtotime('- 7 days', $subscription_expires) > current_time('timestamp', 1)) {
 						continue;
 					}
 
 					// Check if subscription message already exist in messages.
-					if ( array_key_exists( $subscription['id'], $messages ) ) {
+					if (array_key_exists($subscription['id'], $messages)) {
 						continue;
 					}
 
 					// Check if license already expired.
-					if ( strtotime( $subscription['expires'] ) < current_time( 'timestamp', 1 ) ) {
-						Give()->notices->register_notice( array(
+					if (strtotime($subscription['expires']) < current_time('timestamp', 1)) {
+						Give()->notices->register_notice(array(
 							'id'               => "give-expired-subscription-{$subscription['id']}",
 							'type'             => 'error',
 							'description'      => sprintf(
-								__( 'Your Give add-on license expired for payment <a href="%1$s" target="_blank">#%2$d</a>. <a href="%3$s" target="_blank">Click to renew an existing license</a> or %4$s.', 'give' ),
-								urldecode( $subscription['invoice_url'] ),
+								__('Your Give add-on license expired for payment <a href="%1$s" target="_blank">#%2$d</a>. <a href="%3$s" target="_blank">Click to renew an existing license</a> or %4$s.', 'give'),
+								urldecode($subscription['invoice_url']),
 								$subscription['payment_id'],
 								"{$this->checkout_url}?edd_license_key={$subscription['license_key']}&utm_campaign=admin&utm_source=licenses&utm_medium=expired",
-								Give()->notices->get_dismiss_link( array(
-									'title'            => __( 'Click here if already renewed', 'give' ),
+								Give()->notices->get_dismiss_link(array(
+									'title'            => __('Click here if already renewed', 'give'),
 									'dismissible_type' => 'user',
 									'dismiss_interval' => 'permanent',
-								) )
+								))
 							),
 							'dismissible_type' => 'user',
 							'dismiss_interval' => 'shortly',
-						) );
+						));
 					} else {
-						Give()->notices->register_notice( array(
+						Give()->notices->register_notice(array(
 							'id'               => "give-expires-subscription-{$subscription['id']}",
 							'type'             => 'error',
 							'description'      => sprintf(
-								__( 'Your Give add-on license will expire in %1$s for payment <a href="%2$s" target="_blank">#%3$d</a>. <a href="%4$s" target="_blank">Click to renew an existing license</a> or %5$s.', 'give' ),
-								human_time_diff( current_time( 'timestamp', 1 ), strtotime( $subscription['expires'] ) ),
-								urldecode( $subscription['invoice_url'] ),
+								__('Your Give add-on license will expire in %1$s for payment <a href="%2$s" target="_blank">#%3$d</a>. <a href="%4$s" target="_blank">Click to renew an existing license</a> or %5$s.', 'give'),
+								human_time_diff(current_time('timestamp', 1), strtotime($subscription['expires'])),
+								urldecode($subscription['invoice_url']),
 								$subscription['payment_id'],
 								"{$this->checkout_url}?edd_license_key={$subscription['license_key']}&utm_campaign=admin&utm_source=licenses&utm_medium=expired",
-								Give()->notices->get_dismiss_link( array(
-									'title'            => __( 'Click here if already renewed', 'give' ),
+								Give()->notices->get_dismiss_link(array(
+									'title'            => __('Click here if already renewed', 'give'),
 									'dismissible_type' => 'user',
 									'dismiss_interval' => 'permanent',
-								) )
+								))
 							),
 							'dismissible_type' => 'user',
 							'dismiss_interval' => 'shortly',
-						) );
+						));
 					}
 
 					// Stop validation for these license keys.
-					$addon_license_key_in_subscriptions = array_merge( $addon_license_key_in_subscriptions, $subscription['licenses'] );
+					$addon_license_key_in_subscriptions = array_merge($addon_license_key_in_subscriptions, $subscription['licenses']);
 				}// End foreach().
 				$showed_subscriptions_message = true;
 			}// End if().
 
 			// Show Non Subscription Give Add-on messages.
 			if (
-				! in_array( $this->license, $addon_license_key_in_subscriptions )
-				&& ! empty( $this->license )
-				&& empty( $showed_invalid_message )
+				! in_array($this->license, $addon_license_key_in_subscriptions)
+				&& ! empty($this->license)
+				&& empty($showed_invalid_message)
 				&& ! $this->is_valid_license()
 			) {
 
-				Give()->notices->register_notice( array(
+				Give()->notices->register_notice(array(
 					'id'               => 'give-invalid-license',
 					'type'             => 'error',
 					'description'      => sprintf(
-						__( 'You have invalid or expired license keys for one or more Give Add-ons. Please go to the <a href="%s">licenses page</a> to correct this issue.', 'give' ),
-						admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=licenses' )
+						__('You have invalid or expired license keys for one or more Give Add-ons. Please go to the <a href="%s">licenses page</a> to correct this issue.', 'give'),
+						admin_url('edit.php?post_type=give_forms&page=give-settings&tab=licenses')
 					),
 					'dismissible_type' => 'user',
 					'dismiss_interval' => 'shortly',
-				) );
+				));
 
 				$showed_invalid_message = true;
 
@@ -712,10 +712,10 @@ 
 		 *
 		 * @return bool
 		 */
-		public function is_valid_license( $licence_data = null ) {
-			$license_data = empty( $licence_data ) ? $this->license_data : $licence_data;
+		public function is_valid_license($licence_data = null) {
+			$license_data = empty($licence_data) ? $this->license_data : $licence_data;
 
-			if ( apply_filters( 'give_is_valid_license', ( $this->is_license( $license_data ) && 'valid' === $license_data->license ) ) ) {
+			if (apply_filters('give_is_valid_license', ($this->is_license($license_data) && 'valid' === $license_data->license))) {
 				return true;
 			}
 
@@ -733,10 +733,10 @@ 
 		 *
 		 * @return bool
 		 */
-		public function is_license( $licence_data = null ) {
-			$license_data = empty( $licence_data ) ? $this->license_data : $licence_data;
+		public function is_license($licence_data = null) {
+			$license_data = empty($licence_data) ? $this->license_data : $licence_data;
 
-			if ( apply_filters( 'give_is_license', ( is_object( $license_data ) && ! empty( $license_data ) && property_exists( $license_data, 'license' ) ) ) ) {
+			if (apply_filters('give_is_license', (is_object($license_data) && ! empty($license_data) && property_exists($license_data, 'license')))) {
 				return true;
 			}
 
@@ -752,7 +752,7 @@ 
 		 * @return bool
 		 */
 		private function __is_third_party_addon() {
-			return ( false === strpos( $this->api_url, 'givewp.com/' ) );
+			return (false === strpos($this->api_url, 'givewp.com/'));
 		}
 
 		/**
@@ -767,25 +767,25 @@ 
 		 * @return bool
 		 */
 		private function __remove_license_key_from_subscriptions() {
-			$subscriptions = get_option( 'give_subscriptions', array() );
+			$subscriptions = get_option('give_subscriptions', array());
 
 			// Bailout.
-			if ( empty( $this->license ) ) {
+			if (empty($this->license)) {
 				return false;
 			}
 
-			if ( ! empty( $subscriptions ) ) {
-				foreach ( $subscriptions as $subscription_id => $subscription ) {
-					$license_index = array_search( $this->license, $subscription['licenses'] );
-					if ( false !== $license_index ) {
+			if ( ! empty($subscriptions)) {
+				foreach ($subscriptions as $subscription_id => $subscription) {
+					$license_index = array_search($this->license, $subscription['licenses']);
+					if (false !== $license_index) {
 						// Remove license key.
-						unset( $subscriptions[ $subscription_id ]['licenses'][ $license_index ] );
+						unset($subscriptions[$subscription_id]['licenses'][$license_index]);
 
 						// Rearrange license keys.
-						$subscriptions[ $subscription_id ]['licenses'] = array_values( $subscriptions[ $subscription_id ]['licenses'] );
+						$subscriptions[$subscription_id]['licenses'] = array_values($subscriptions[$subscription_id]['licenses']);
 
 						// Update subscription information.
-						update_option( 'give_subscriptions', $subscriptions, false );
+						update_option('give_subscriptions', $subscriptions, false);
 						break;
 					}
 				}
@@ -799,17 +799,17 @@ 
 		 *
 		 * @return bool
 		 */
-		public function plugin_page_notices( $plugin_file, $plugin_data, $status ) {
+		public function plugin_page_notices($plugin_file, $plugin_data, $status) {
 			// Bailout.
-			if ( $this->is_valid_license() ) {
+			if ($this->is_valid_license()) {
 				return false;
 			}
 
 			$update_notice_wrap = '<tr class="give-addon-notice-tr active"><td colspan="3" class="colspanchange"><div class="notice inline notice-warning notice-alt give-invalid-license"><p><span class="dashicons dashicons-info"></span> %s</p></div></td></tr>';
 			$message            = $this->license_state_message();
 
-			if ( ! empty( $message['message'] ) ) {
-				echo sprintf( $update_notice_wrap, $message['message'] );
+			if ( ! empty($message['message'])) {
+				echo sprintf($update_notice_wrap, $message['message']);
 			}
 		}
 
@@ -824,11 +824,11 @@ 
 		public function license_state_message() {
 			$message_data = array();
 
-			if ( ! $this->is_valid_license() ) {
+			if ( ! $this->is_valid_license()) {
 
 				$message_data['message'] = sprintf(
 					'Please <a href="%1$s">activate your license</a> to receive updates and support for the %2$s add-on.',
-					esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=licenses' ) ),
+					esc_url(admin_url('edit.php?post_type=give_forms&page=give-settings&tab=licenses')),
 					$this->item_name
 				);
 			}
@@ -847,7 +847,7 @@ 
 			// Bailout.
 			if (
 				! Give_Admin_Settings::verify_nonce() ||
-				! current_user_can( 'manage_give_settings' ) ||
+				! current_user_can('manage_give_settings') ||
 				'licenses' !== give_get_current_setting_tab()
 			) {
 				return false;
@@ -855,10 +855,10 @@ 
 
 			// Security check.
 			if (
-				isset( $_POST[ $this->item_shortname . '_license_key-nonce' ] ) &&
-				! wp_verify_nonce( $_REQUEST[ $this->item_shortname . '_license_key-nonce' ], $this->item_shortname . '_license_key-nonce' )
+				isset($_POST[$this->item_shortname.'_license_key-nonce']) &&
+				! wp_verify_nonce($_REQUEST[$this->item_shortname.'_license_key-nonce'], $this->item_shortname.'_license_key-nonce')
 			) {
-				wp_die( __( 'Nonce verification failed.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+				wp_die(__('Nonce verification failed.', 'give'), __('Error', 'give'), array('response' => 403));
 			}
 
 			return true;
@@ -876,9 +876,9 @@ 
 		 *
 		 * @return mixed
 		 */
-		public function get_license_info( $edd_action = '', $response_in_array = false ) {
+		public function get_license_info($edd_action = '', $response_in_array = false) {
 
-			if ( empty( $edd_action ) ) {
+			if (empty($edd_action)) {
 				return false;
 			}
 
@@ -886,7 +886,7 @@ 
 			$api_params = array(
 				'edd_action' => $edd_action, // never change from "edd_" to "give_"!
 				'license'    => $this->license,
-				'item_name'  => urlencode( $this->item_name ),
+				'item_name'  => urlencode($this->item_name),
 				'url'        => home_url(),
 			);
 
@@ -901,11 +901,11 @@ 
 			);
 
 			// Make sure there are no errors.
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error($response)) {
 				return false;
 			}
 
-			return json_decode( wp_remote_retrieve_body( $response ), $response_in_array );
+			return json_decode(wp_remote_retrieve_body($response), $response_in_array);
 		}
 
 
@@ -921,9 +921,9 @@ 
 			$this->__remove_license_key_from_subscriptions();
 
 			// Remove license from database.
-			delete_option( "{$this->item_shortname}_license_active" );
-			give_delete_option( "{$this->item_shortname}_license_key" );
-			unset( $_POST["{$this->item_shortname}_license_key"] );
+			delete_option("{$this->item_shortname}_license_active");
+			give_delete_option("{$this->item_shortname}_license_key");
+			unset($_POST["{$this->item_shortname}_license_key"]);
 
 			// Unset license param.
 			$this->license = '';
@@ -941,8 +941,8 @@ 
 		private function is_deactivating_license() {
 			$status = false;
 
-			foreach ( $_POST as $key => $value ) {
-				if ( false !== strpos( $key, 'license_key_deactivate' ) ) {
+			foreach ($_POST as $key => $value) {
+				if (false !== strpos($key, 'license_key_deactivate')) {
 					$status = true;
 					break;
 				}

includes/gateways/offline-donations.php

Indentation +4 added lines, -4 removed lines

@@ -249,10 +249,10 @@
 	$admin_message .= '<strong>' . __( 'Amount:', 'give' ) . '</strong> {amount}' . "\n\n";
 
 	$admin_message .= sprintf(
-		                  '<a href="%1$s">%2$s</a>',
-		                  admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment_id ),
-		                  __( 'View Donation Details &raquo;', 'give' )
-	                  ) . "\n\n";
+						  '<a href="%1$s">%2$s</a>',
+						  admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment_id ),
+						  __( 'View Donation Details &raquo;', 'give' )
+					  ) . "\n\n";
 
 	$admin_message = apply_filters( 'give_offline_admin_donation_notification', $admin_message, $payment_id );
 	$admin_message = give_do_email_tags( $admin_message, $payment_id );

Spacing +131 added lines, -131 removed lines

@@ -18,17 +18,17 @@ 
  *
  * @return array
  */
-function give_offline_register_gateway( $gateways ) {
+function give_offline_register_gateway($gateways) {
 	// Format: ID => Name
 	$gateways['offline'] = array(
-		'admin_label'    => esc_attr__( 'Offline Donation', 'give' ),
-		'checkout_label' => esc_attr__( 'Offline Donation', 'give' ),
+		'admin_label'    => esc_attr__('Offline Donation', 'give'),
+		'checkout_label' => esc_attr__('Offline Donation', 'give'),
 	);
 
 	return $gateways;
 }
 
-add_filter( 'give_payment_gateways', 'give_offline_register_gateway', 1 );
+add_filter('give_payment_gateways', 'give_offline_register_gateway', 1);
 
 /**
  * Add our payment instructions to the checkout
@@ -39,9 +39,9 @@ 
  *
  * @return void
  */
-function give_offline_payment_cc_form( $form_id ) {
+function give_offline_payment_cc_form($form_id) {
 	// Get offline payment instruction.
-	$offline_instructions = give_get_offline_payment_instruction( $form_id, true );
+	$offline_instructions = give_get_offline_payment_instruction($form_id, true);
 
 	ob_start();
 
@@ -52,10 +52,10 @@ 
 	 *
 	 * @param int $form_id Give form id.
 	 */
-	do_action( 'give_before_offline_info_fields', $form_id );
+	do_action('give_before_offline_info_fields', $form_id);
 	?>
     <fieldset id="give_offline_payment_info">
-		<?php echo stripslashes( $offline_instructions ); ?>
+		<?php echo stripslashes($offline_instructions); ?>
     </fieldset>
 	<?php
 	/**
@@ -65,35 +65,35 @@ 
 	 *
 	 * @param int $form_id Give form id.
 	 */
-	do_action( 'give_after_offline_info_fields', $form_id );
+	do_action('give_after_offline_info_fields', $form_id);
 
 	echo ob_get_clean();
 }
 
-add_action( 'give_offline_cc_form', 'give_offline_payment_cc_form' );
+add_action('give_offline_cc_form', 'give_offline_payment_cc_form');
 
 /**
  * Give Offline Billing Field
  *
  * @param $form_id
  */
-function give_offline_billing_fields( $form_id ) {
+function give_offline_billing_fields($form_id) {
 	//Enable Default CC fields (billing info)
-	$post_offline_cc_fields        = give_get_meta( $form_id, '_give_offline_donation_enable_billing_fields_single', true );
-	$post_offline_customize_option = give_get_meta( $form_id, '_give_customize_offline_donations', true );
+	$post_offline_cc_fields        = give_get_meta($form_id, '_give_offline_donation_enable_billing_fields_single', true);
+	$post_offline_customize_option = give_get_meta($form_id, '_give_customize_offline_donations', true);
 
-	$global_offline_cc_fields = give_get_option( 'give_offline_donation_enable_billing_fields' );
+	$global_offline_cc_fields = give_get_option('give_offline_donation_enable_billing_fields');
 
 	//Output CC Address fields if global option is on and user hasn't elected to customize this form's offline donation options
 	if (
-		( give_is_setting_enabled( $post_offline_customize_option, 'global' ) && give_is_setting_enabled( $global_offline_cc_fields ) )
-		|| ( give_is_setting_enabled( $post_offline_customize_option, 'enabled' ) && give_is_setting_enabled( $post_offline_cc_fields ) )
+		(give_is_setting_enabled($post_offline_customize_option, 'global') && give_is_setting_enabled($global_offline_cc_fields))
+		|| (give_is_setting_enabled($post_offline_customize_option, 'enabled') && give_is_setting_enabled($post_offline_cc_fields))
 	) {
-		give_default_cc_address_fields( $form_id );
+		give_default_cc_address_fields($form_id);
 	}
 }
 
-add_action( 'give_before_offline_info_fields', 'give_offline_billing_fields', 10, 1 );
+add_action('give_before_offline_info_fields', 'give_offline_billing_fields', 10, 1);
 
 /**
  * Process the payment
@@ -104,18 +104,18 @@ 
  *
  * @return void
  */
-function give_offline_process_payment( $purchase_data ) {
+function give_offline_process_payment($purchase_data) {
 
 	// Setup the payment details.
 	$payment_data = array(
 		'price'           => $purchase_data['price'],
 		'give_form_title' => $purchase_data['post_data']['give-form-title'],
-		'give_form_id'    => intval( $purchase_data['post_data']['give-form-id'] ),
-		'give_price_id'   => isset( $purchase_data['post_data']['give-price-id'] ) ? $purchase_data['post_data']['give-price-id'] : '',
+		'give_form_id'    => intval($purchase_data['post_data']['give-form-id']),
+		'give_price_id'   => isset($purchase_data['post_data']['give-price-id']) ? $purchase_data['post_data']['give-price-id'] : '',
 		'date'            => $purchase_data['date'],
 		'user_email'      => $purchase_data['user_email'],
 		'purchase_key'    => $purchase_data['purchase_key'],
-		'currency'        => give_get_currency( $purchase_data['post_data']['give-form-id'], $purchase_data ),
+		'currency'        => give_get_currency($purchase_data['post_data']['give-form-id'], $purchase_data),
 		'user_info'       => $purchase_data['user_info'],
 		'status'          => 'pending',
 		'gateway'         => 'offline',
@@ -123,18 +123,18 @@ 
 
 
 	// record the pending payment
-	$payment = give_insert_payment( $payment_data );
+	$payment = give_insert_payment($payment_data);
 
-	if ( $payment ) {
+	if ($payment) {
 		give_send_to_success_page();
 	} else {
 		// if errors are present, send the user back to the donation form so they can be corrected
-		give_send_back_to_checkout( '?payment-mode=' . $purchase_data['post_data']['give-gateway'] );
+		give_send_back_to_checkout('?payment-mode='.$purchase_data['post_data']['give-gateway']);
 	}
 
 }
 
-add_action( 'give_gateway_offline', 'give_offline_process_payment' );
+add_action('give_gateway_offline', 'give_offline_process_payment');
 
 
 /**
@@ -147,59 +147,59 @@ 
  * @since       1.0
  * @return void
  */
-function give_offline_send_donor_instructions( $payment_id = 0 ) {
+function give_offline_send_donor_instructions($payment_id = 0) {
 
-	$payment_data                      = give_get_payment_meta( $payment_id );
-	$post_offline_customization_option = give_get_meta( $payment_data['form_id'], '_give_customize_offline_donations', true );
+	$payment_data                      = give_get_payment_meta($payment_id);
+	$post_offline_customization_option = give_get_meta($payment_data['form_id'], '_give_customize_offline_donations', true);
 
 	//Customize email content depending on whether the single form has been customized
-	$email_content = give_get_option( 'global_offline_donation_email' );
+	$email_content = give_get_option('global_offline_donation_email');
 
-	if ( give_is_setting_enabled( $post_offline_customization_option, 'enabled' ) ) {
-		$email_content = give_get_meta( $payment_data['form_id'], '_give_offline_donation_email', true );
+	if (give_is_setting_enabled($post_offline_customization_option, 'enabled')) {
+		$email_content = give_get_meta($payment_data['form_id'], '_give_offline_donation_email', true);
 	}
 
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
 
 	/**
 	 * Filters the from name.
 	 *
 	 * @since 1.7
 	 */
-	$from_name = apply_filters( 'give_donation_from_name', $from_name, $payment_id, $payment_data );
+	$from_name = apply_filters('give_donation_from_name', $from_name, $payment_id, $payment_data);
 
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
 
 	/**
 	 * Filters the from email.
 	 *
 	 * @since 1.7
 	 */
-	$from_email = apply_filters( 'give_donation_from_address', $from_email, $payment_id, $payment_data );
+	$from_email = apply_filters('give_donation_from_address', $from_email, $payment_id, $payment_data);
 
-	$to_email = give_get_payment_user_email( $payment_id );
+	$to_email = give_get_payment_user_email($payment_id);
 
-	$subject = give_get_option( 'offline_donation_subject', __( 'Offline Donation Instructions', 'give' ) );
-	if ( give_is_setting_enabled( $post_offline_customization_option, 'enabled' ) ) {
-		$subject = give_get_meta( $payment_data['form_id'], '_give_offline_donation_subject', true );
+	$subject = give_get_option('offline_donation_subject', __('Offline Donation Instructions', 'give'));
+	if (give_is_setting_enabled($post_offline_customization_option, 'enabled')) {
+		$subject = give_get_meta($payment_data['form_id'], '_give_offline_donation_subject', true);
 	}
 
-	$subject = apply_filters( 'give_offline_donation_subject', wp_strip_all_tags( $subject ), $payment_id );
-	$subject = give_do_email_tags( $subject, $payment_id );
+	$subject = apply_filters('give_offline_donation_subject', wp_strip_all_tags($subject), $payment_id);
+	$subject = give_do_email_tags($subject, $payment_id);
 
-	$attachments = apply_filters( 'give_offline_donation_attachments', array(), $payment_id, $payment_data );
-	$message     = give_do_email_tags( $email_content, $payment_id );
+	$attachments = apply_filters('give_offline_donation_attachments', array(), $payment_id, $payment_data);
+	$message     = give_do_email_tags($email_content, $payment_id);
 
 	$emails = Give()->emails;
 
-	$emails->__set( 'from_name', $from_name );
-	$emails->__set( 'from_email', $from_email );
-	$emails->__set( 'heading', __( 'Offline Donation Instructions', 'give' ) );
+	$emails->__set('from_name', $from_name);
+	$emails->__set('from_email', $from_email);
+	$emails->__set('heading', __('Offline Donation Instructions', 'give'));
 
-	$headers = apply_filters( 'give_receipt_headers', $emails->get_headers(), $payment_id, $payment_data );
-	$emails->__set( 'headers', $headers );
+	$headers = apply_filters('give_receipt_headers', $emails->get_headers(), $payment_id, $payment_data);
+	$emails->__set('headers', $headers);
 
-	$emails->send( $to_email, $subject, $message, $attachments );
+	$emails->send($to_email, $subject, $message, $attachments);
 
 }
 
@@ -216,54 +216,54 @@ 
  * @return void
  *
  */
-function give_offline_send_admin_notice( $payment_id = 0 ) {
+function give_offline_send_admin_notice($payment_id = 0) {
 
 	/* Send an email notification to the admin */
 	$admin_email = give_get_admin_notice_emails();
-	$user_info   = give_get_payment_meta_user_info( $payment_id );
+	$user_info   = give_get_payment_meta_user_info($payment_id);
 
-	if ( isset( $user_info['id'] ) && $user_info['id'] > 0 ) {
-		$user_data = get_userdata( $user_info['id'] );
+	if (isset($user_info['id']) && $user_info['id'] > 0) {
+		$user_data = get_userdata($user_info['id']);
 		$name      = $user_data->display_name;
-	} elseif ( isset( $user_info['first_name'] ) && isset( $user_info['last_name'] ) ) {
-		$name = $user_info['first_name'] . ' ' . $user_info['last_name'];
+	} elseif (isset($user_info['first_name']) && isset($user_info['last_name'])) {
+		$name = $user_info['first_name'].' '.$user_info['last_name'];
 	} else {
 		$name = $user_info['email'];
 	}
 
-	$amount = give_donation_amount( $payment_id );
+	$amount = give_donation_amount($payment_id);
 
-	$admin_subject = apply_filters( 'give_offline_admin_donation_notification_subject', __( 'New Pending Donation', 'give' ), $payment_id );
+	$admin_subject = apply_filters('give_offline_admin_donation_notification_subject', __('New Pending Donation', 'give'), $payment_id);
 
-	$admin_message = __( 'Dear Admin,', 'give' ) . "\n\n";
-	$admin_message .= sprintf(__( 'A new offline donation has been made on your website for %s.', 'give' ), $amount) . "\n\n";
-	$admin_message .= __( 'The donation is in a pending status and is awaiting payment. Donation instructions have been emailed to the donor. Once you receive payment, be sure to mark the donation as complete using the link below.', 'give' ) . "\n\n";
+	$admin_message = __('Dear Admin,', 'give')."\n\n";
+	$admin_message .= sprintf(__('A new offline donation has been made on your website for %s.', 'give'), $amount)."\n\n";
+	$admin_message .= __('The donation is in a pending status and is awaiting payment. Donation instructions have been emailed to the donor. Once you receive payment, be sure to mark the donation as complete using the link below.', 'give')."\n\n";
 
 
-	$admin_message .= '<strong>' . __( 'Donor:', 'give' ) . '</strong> {fullname}' . "\n";
-	$admin_message .= '<strong>' . __( 'Amount:', 'give' ) . '</strong> {amount}' . "\n\n";
+	$admin_message .= '<strong>'.__('Donor:', 'give').'</strong> {fullname}'."\n";
+	$admin_message .= '<strong>'.__('Amount:', 'give').'</strong> {amount}'."\n\n";
 
 	$admin_message .= sprintf(
 		                  '<a href="%1$s">%2$s</a>',
-		                  admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment_id ),
-		                  __( 'View Donation Details &raquo;', 'give' )
-	                  ) . "\n\n";
+		                  admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$payment_id),
+		                  __('View Donation Details &raquo;', 'give')
+	                  )."\n\n";
 
-	$admin_message = apply_filters( 'give_offline_admin_donation_notification', $admin_message, $payment_id );
-	$admin_message = give_do_email_tags( $admin_message, $payment_id );
+	$admin_message = apply_filters('give_offline_admin_donation_notification', $admin_message, $payment_id);
+	$admin_message = give_do_email_tags($admin_message, $payment_id);
 
-	$attachments   = apply_filters( 'give_offline_admin_donation_notification_attachments', array(), $payment_id );
-	$admin_headers = apply_filters( 'give_offline_admin_donation_notification_headers', array(), $payment_id );
+	$attachments   = apply_filters('give_offline_admin_donation_notification_attachments', array(), $payment_id);
+	$admin_headers = apply_filters('give_offline_admin_donation_notification_headers', array(), $payment_id);
 
 	//Send Email
 	$emails = Give()->emails;
-	$emails->__set( 'heading', __( 'New Offline Donation', 'give' ) );
+	$emails->__set('heading', __('New Offline Donation', 'give'));
 
-	if ( ! empty( $admin_headers ) ) {
-		$emails->__set( 'headers', $admin_headers );
+	if ( ! empty($admin_headers)) {
+		$emails->__set('headers', $admin_headers);
 	}
 
-	$emails->send( $admin_email, $admin_subject, $admin_message, $attachments );
+	$emails->send($admin_email, $admin_subject, $admin_message, $attachments);
 
 }
 
@@ -275,20 +275,20 @@ 
  *
  * @return array
  */
-function give_offline_add_settings( $settings ) {
+function give_offline_add_settings($settings) {
 
 	// Bailout: Do not show offline gateways setting in to metabox if its disabled globally.
-	if ( in_array( 'offline', (array) give_get_option( 'gateways' ) ) ) {
+	if (in_array('offline', (array) give_get_option('gateways'))) {
 		return $settings;
 	}
 
 	//Vars
 	$prefix = '_give_';
 
-	$is_gateway_active = give_is_gateway_active( 'offline' );
+	$is_gateway_active = give_is_gateway_active('offline');
 
 	//this gateway isn't active
-	if ( ! $is_gateway_active ) {
+	if ( ! $is_gateway_active) {
 		//return settings and bounce
 		return $settings;
 	}
@@ -297,34 +297,34 @@ 
 	$check_settings = array(
 
 		array(
-			'name'    => __( 'Offline Donations', 'give' ),
-			'desc'    => __( 'Do you want to customize the donation instructions for this form?', 'give' ),
-			'id'      => $prefix . 'customize_offline_donations',
+			'name'    => __('Offline Donations', 'give'),
+			'desc'    => __('Do you want to customize the donation instructions for this form?', 'give'),
+			'id'      => $prefix.'customize_offline_donations',
 			'type'    => 'radio_inline',
 			'default' => 'global',
-			'options' => apply_filters( 'give_forms_content_options_select', array(
-					'global'   => __( 'Global Option', 'give' ),
-					'enabled'  => __( 'Customize', 'give' ),
-					'disabled' => __( 'Disable', 'give' ),
+			'options' => apply_filters('give_forms_content_options_select', array(
+					'global'   => __('Global Option', 'give'),
+					'enabled'  => __('Customize', 'give'),
+					'disabled' => __('Disable', 'give'),
 				)
 			),
 		),
 		array(
-			'name'        => __( 'Billing Fields', 'give' ),
-			'desc'        => __( 'This option will enable the billing details section for this form\'s offline donation payment gateway. The fieldset will appear above the offline donation instructions.', 'give' ),
-			'id'          => $prefix . 'offline_donation_enable_billing_fields_single',
+			'name'        => __('Billing Fields', 'give'),
+			'desc'        => __('This option will enable the billing details section for this form\'s offline donation payment gateway. The fieldset will appear above the offline donation instructions.', 'give'),
+			'id'          => $prefix.'offline_donation_enable_billing_fields_single',
 			'row_classes' => 'give-subfield give-hidden',
 			'type'        => 'radio_inline',
 			'default'     => 'disabled',
 			'options'     => array(
-				'enabled'  => __( 'Enabled', 'give' ),
-				'disabled' => __( 'Disabled', 'give' ),
+				'enabled'  => __('Enabled', 'give'),
+				'disabled' => __('Disabled', 'give'),
 			),
 		),
 		array(
-			'id'          => $prefix . 'offline_checkout_notes',
-			'name'        => __( 'Donation Instructions', 'give' ),
-			'desc'        => __( 'Enter the instructions you want to display to the donor during the donation process. Most likely this would include important information like mailing address and who to make the check out to.', 'give' ),
+			'id'          => $prefix.'offline_checkout_notes',
+			'name'        => __('Donation Instructions', 'give'),
+			'desc'        => __('Enter the instructions you want to display to the donor during the donation process. Most likely this would include important information like mailing address and who to make the check out to.', 'give'),
 			'default'     => give_get_default_offline_donation_content(),
 			'type'        => 'wysiwyg',
 			'row_classes' => 'give-subfield give-hidden',
@@ -336,14 +336,14 @@ 
 			'name'  => 'offline_docs',
 			'type'  => 'docs_link',
 			'url'   => 'http://docs.givewp.com/settings-gateway-offline-donations',
-			'title' => __( 'Offline Donations', 'give' ),
+			'title' => __('Offline Donations', 'give'),
 		),
 	);
 
-	return array_merge( $settings, $check_settings );
+	return array_merge($settings, $check_settings);
 }
 
-add_filter( 'give_forms_offline_donations_metabox_fields', 'give_offline_add_settings' );
+add_filter('give_forms_offline_donations_metabox_fields', 'give_offline_add_settings');
 
 
 /**
@@ -354,24 +354,24 @@ 
  * @return string
  */
 function give_get_default_offline_donation_content() {
-	$default_text = '<p>' . __( 'In order to make an offline donation we ask that you please follow these instructions', 'give' ) . ': </p>';
+	$default_text = '<p>'.__('In order to make an offline donation we ask that you please follow these instructions', 'give').': </p>';
 	$default_text .= '<ol>';
 	$default_text .= '<li>';
 	$default_text .= sprintf(
 	/* translators: %s: site name */
-		__( 'Make a check payable to "{sitename}"', 'give' ) );
+		__('Make a check payable to "{sitename}"', 'give') );
 	$default_text .= '</li>';
 	$default_text .= '<li>';
 	$default_text .= sprintf(
 	/* translators: %s: site name */
-		__( 'On the memo line of the check, please indicate that the donation is for "{sitename}"', 'give' ) );
+		__('On the memo line of the check, please indicate that the donation is for "{sitename}"', 'give') );
 	$default_text .= '</li>';
-	$default_text .= '<li>' . __( 'Please mail your check to:', 'give' ) . '</li>';
+	$default_text .= '<li>'.__('Please mail your check to:', 'give').'</li>';
 	$default_text .= '</ol>';
 	$default_text .= '{offline_mailing_address}<br>';
-	$default_text .= '<p>' . __( 'All contributions will be gratefully acknowledged and are tax deductible.', 'give' ) . '</p>';
+	$default_text .= '<p>'.__('All contributions will be gratefully acknowledged and are tax deductible.', 'give').'</p>';
 
-	return apply_filters( 'give_default_offline_donation_content', $default_text );
+	return apply_filters('give_default_offline_donation_content', $default_text);
 
 }
 
@@ -383,28 +383,28 @@ 
  * @return string
  */
 function give_get_default_offline_donation_email_content() {
-	$default_text = '<p>' . __( 'Dear {name},', 'give' ) . '</p>';
-	$default_text .= '<p>' . __( 'Thank you for your offline donation request! Your generosity is greatly appreciated. In order to make an offline donation we ask that you please follow these instructions:', 'give' ) . '</p>';
+	$default_text = '<p>'.__('Dear {name},', 'give').'</p>';
+	$default_text .= '<p>'.__('Thank you for your offline donation request! Your generosity is greatly appreciated. In order to make an offline donation we ask that you please follow these instructions:', 'give').'</p>';
 	$default_text .= '<ol>';
 	$default_text .= '<li>';
 	$default_text .= sprintf(
 	/* translators: %s: site name */
-		__( 'Make a check payable to "{sitename}"', 'give' )
+		__('Make a check payable to "{sitename}"', 'give')
 	);
 	$default_text .= '</li>';
 	$default_text .= '<li>';
 	$default_text .= sprintf(
-		__( 'On the memo line of the check, please indicate that the donation is for "{sitename}"', 'give' )
+		__('On the memo line of the check, please indicate that the donation is for "{sitename}"', 'give')
 	);
 	$default_text .= '</li>';
-	$default_text .= '<li>' . __( 'Please mail your check to:', 'give' ) . '</li>';
+	$default_text .= '<li>'.__('Please mail your check to:', 'give').'</li>';
 	$default_text .= '</ol>';
 	$default_text .= '{offline_mailing_address}<br>';
-	$default_text .= '<p>' . __( 'Once your donation has been received we will mark it as complete and you will receive an email receipt for your records. Please contact us with any questions you may have!', 'give' ) . '</p>';
-	$default_text .= '<p>' . __( 'Sincerely,', 'give' ) . '</p>';
+	$default_text .= '<p>'.__('Once your donation has been received we will mark it as complete and you will receive an email receipt for your records. Please contact us with any questions you may have!', 'give').'</p>';
+	$default_text .= '<p>'.__('Sincerely,', 'give').'</p>';
 	$default_text .= '<p>{sitename}</p>';
 
-	return apply_filters( 'give_default_offline_donation_content', $default_text );
+	return apply_filters('give_default_offline_donation_content', $default_text);
 
 }
 
@@ -418,35 +418,35 @@ 
  *
  * @return string
  */
-function give_get_offline_payment_instruction( $form_id, $wpautop = false ) {
+function give_get_offline_payment_instruction($form_id, $wpautop = false) {
 	// Bailout.
-	if ( ! $form_id ) {
+	if ( ! $form_id) {
 		return '';
 	}
 
-	$post_offline_customization_option = give_get_meta( $form_id, '_give_customize_offline_donations', true );
-	$post_offline_instructions         = give_get_meta( $form_id, '_give_offline_checkout_notes', true );
-	$global_offline_instruction        = give_get_option( 'global_offline_donation_content' );
+	$post_offline_customization_option = give_get_meta($form_id, '_give_customize_offline_donations', true);
+	$post_offline_instructions         = give_get_meta($form_id, '_give_offline_checkout_notes', true);
+	$global_offline_instruction        = give_get_option('global_offline_donation_content');
 	$offline_instructions              = $global_offline_instruction;
 
-	if ( give_is_setting_enabled( $post_offline_customization_option ) ) {
+	if (give_is_setting_enabled($post_offline_customization_option)) {
 		$offline_instructions = $post_offline_instructions;
 	}
 
-	$settings_url = admin_url( 'post.php?post=' . $form_id . '&action=edit&message=1' );
+	$settings_url = admin_url('post.php?post='.$form_id.'&action=edit&message=1');
 
 	/* translators: %s: form settings url */
-	$offline_instructions = ! empty( $offline_instructions )
+	$offline_instructions = ! empty($offline_instructions)
 		? $offline_instructions
 		: sprintf(
-			__( 'Please enter offline donation instructions in <a href="%s">this form\'s settings</a>.', 'give' ),
+			__('Please enter offline donation instructions in <a href="%s">this form\'s settings</a>.', 'give'),
 			$settings_url
 		);
 
-	$offline_instructions = give_do_email_tags( $offline_instructions, null );
+	$offline_instructions = give_do_email_tags($offline_instructions, null);
 
 	$formmated_offline_instructions = $wpautop
-		? wpautop( do_shortcode( $offline_instructions ) )
+		? wpautop(do_shortcode($offline_instructions))
 		: $offline_instructions;
 
 	/**
@@ -477,21 +477,21 @@ 
  *
  * @return array
  */
-function give_filter_offline_gateway( $gateway_list, $form_id ) {
+function give_filter_offline_gateway($gateway_list, $form_id) {
 	if (
 		// Show offline payment gateway if enable for new donation form.
-		( false === strpos( $_SERVER['REQUEST_URI'], '/wp-admin/post-new.php?post_type=give_forms' ) )
+		(false === strpos($_SERVER['REQUEST_URI'], '/wp-admin/post-new.php?post_type=give_forms'))
 		&& $form_id
-		&& ! give_is_setting_enabled( give_get_meta( $form_id, '_give_customize_offline_donations', true ), array( 'enabled', 'global' ) )
+		&& ! give_is_setting_enabled(give_get_meta($form_id, '_give_customize_offline_donations', true), array('enabled', 'global'))
 	) {
-		unset( $gateway_list['offline'] );
+		unset($gateway_list['offline']);
 	}
 
 	// Output.
 	return $gateway_list;
 }
 
-add_filter( 'give_enabled_payment_gateways', 'give_filter_offline_gateway', 10, 2 );
+add_filter('give_enabled_payment_gateways', 'give_filter_offline_gateway', 10, 2);
 
 /**
  * Set default gateway to global default payment gateway
@@ -505,13 +505,13 @@ 
  *
  * @return void
  */
-function _give_customize_offline_donations_on_save_callback( $meta_key, $meta_value, $postid ) {
+function _give_customize_offline_donations_on_save_callback($meta_key, $meta_value, $postid) {
 	if (
-		! give_is_setting_enabled( $meta_value, array( 'global', 'enabled' ) )
-		&& ( 'offline' === give_get_meta( $postid, '_give_default_gateway', true ) )
+		! give_is_setting_enabled($meta_value, array('global', 'enabled'))
+		&& ('offline' === give_get_meta($postid, '_give_default_gateway', true))
 	) {
-		give_update_meta( $postid, '_give_default_gateway', 'global' );
+		give_update_meta($postid, '_give_default_gateway', 'global');
 	}
 }
 
-add_filter( 'give_save__give_customize_offline_donations', '_give_customize_offline_donations_on_save_callback', 10, 3 );
+add_filter('give_save__give_customize_offline_donations', '_give_customize_offline_donations_on_save_callback', 10, 3);