WordImpress / Give

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

Spacing +6 added lines, -6 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.3.0
  */
 
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 class Give_Shortcode_Login extends Give_Shortcode_Generator {
 
@@ -18,10 +18,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = esc_html( 'Login', 'give' );
-		$this->shortcode['label'] = esc_html( 'Login', 'give' );
+		$this->shortcode['title'] = esc_html('Login', 'give');
+		$this->shortcode['label'] = esc_html('Login', 'give');
 
-		parent::__construct( 'give_login' );
+		parent::__construct('give_login');
 	}
 
 	/**
@@ -34,13 +34,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 login.', 'give' ),
+				'tooltip'  => esc_attr('Enter an URL here to redirect to after login.', 'give'),
 			),
 		);
 	}

includes/admin/shortcodes/abstract-shortcode-generator.php

Indentation +2 added lines, -2 removed lines

@@ -248,8 +248,8 @@
 
 			// do not reindex array!
 			$field['options'] = array(
-				                    '' => ( $field['placeholder'] ? $field['placeholder'] : esc_html( '- Select -', 'give' ) ),
-			                    ) + $field['options'];
+									'' => ( $field['placeholder'] ? $field['placeholder'] : esc_html( '- Select -', 'give' ) ),
+								) + $field['options'];
 
 			foreach ( $field['options'] as $value => $text ) {
 				$new_listbox['values'][] = array(

Spacing +64 added lines, -64 removed lines

@@ -11,7 +11,7 @@ 
  * @since       1.3.0
  */
 
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 abstract class Give_Shortcode_Generator {
 
@@ -57,12 +57,12 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function __construct( $shortcode ) {
+	public function __construct($shortcode) {
 
 
 		$this->shortcode_tag = $shortcode;
 
-		add_action( 'admin_init', array( $this, 'init' ) );
+		add_action('admin_init', array($this, 'init'));
 
 	}
 
@@ -73,9 +73,9 @@ 
 	 */
 	public function init() {
 
-		if ( $this->shortcode_tag ) {
+		if ($this->shortcode_tag) {
 
-			$this->self = get_class( $this );
+			$this->self = get_class($this);
 
 			$this->errors   = array();
 			$this->required = array();
@@ -84,18 +84,18 @@ 
 			$fields = $this->get_fields();
 
 			$defaults = array(
-				'btn_close' => esc_html( 'Close', 'give' ),
-				'btn_okay'  => esc_html( 'Insert Shortcode', 'give' ),
+				'btn_close' => esc_html('Close', 'give'),
+				'btn_okay'  => esc_html('Insert Shortcode', 'give'),
 				'errors'    => $this->errors,
 				'fields'    => $fields,
-				'label'     => '[' . $this->shortcode_tag . ']',
+				'label'     => '['.$this->shortcode_tag.']',
 				'required'  => $this->required,
-				'title'     => esc_attr( 'Insert Shortcode', 'give' ),
+				'title'     => esc_attr('Insert Shortcode', 'give'),
 			);
 
-			if ( user_can_richedit() ) {
+			if (user_can_richedit()) {
 
-				Give_Shortcode_Button::$shortcodes[ $this->shortcode_tag ] = wp_parse_args( $this->shortcode, $defaults );
+				Give_Shortcode_Button::$shortcodes[$this->shortcode_tag] = wp_parse_args($this->shortcode, $defaults);
 
 			}
 		}
@@ -124,13 +124,13 @@ 
 	 *
 	 * @since 1.0
 	 */
-	protected function generate_fields( $defined_fields ) {
+	protected function generate_fields($defined_fields) {
 
 		$fields = array();
 
-		if ( is_array( $defined_fields ) ) {
+		if (is_array($defined_fields)) {
 
-			foreach ( $defined_fields as $field ) {
+			foreach ($defined_fields as $field) {
 
 				$defaults = array(
 					'label'       => false,
@@ -141,14 +141,14 @@ 
 					'type'        => '',
 				);
 
-				$field  = wp_parse_args( (array) $field, $defaults );
-				$method = 'generate_' . strtolower( $field['type'] );
+				$field  = wp_parse_args((array) $field, $defaults);
+				$method = 'generate_'.strtolower($field['type']);
 
-				if ( method_exists( $this, $method ) ) {
+				if (method_exists($this, $method)) {
 
-					$field = call_user_func( array( $this, $method ), $field );
+					$field = call_user_func(array($this, $method), $field);
 
-					if ( $field ) {
+					if ($field) {
 						$fields[] = $field;
 					}
 				}
@@ -168,22 +168,22 @@ 
 	protected function get_fields() {
 
 		$defined_fields   = $this->define_fields();
-		$generated_fields = $this->generate_fields( $defined_fields );
+		$generated_fields = $this->generate_fields($defined_fields);
 
 		$errors = array();
 
-		if ( ! empty( $this->errors ) ) {
-			foreach ( $this->required as $name => $alert ) {
-				if ( false === array_search( $name, array_column( $generated_fields, 'name' ) ) ) {
+		if ( ! empty($this->errors)) {
+			foreach ($this->required as $name => $alert) {
+				if (false === array_search($name, array_column($generated_fields, 'name'))) {
 
-					$errors[] = $this->errors[ $name ];
+					$errors[] = $this->errors[$name];
 				}
 			}
 
 			$this->errors = $errors;
 		}
 
-		if ( ! empty( $errors ) ) {
+		if ( ! empty($errors)) {
 
 			return $errors;
 		}
@@ -200,9 +200,9 @@ 
 	 *
 	 * @since 1.0
 	 */
-	protected function generate_container( $field ) {
+	protected function generate_container($field) {
 
-		if ( array_key_exists( 'html', $field ) ) {
+		if (array_key_exists('html', $field)) {
 
 			return array(
 				'type' => $field['type'],
@@ -222,36 +222,36 @@ 
 	 *
 	 * @since 1.0
 	 */
-	protected function generate_listbox( $field ) {
+	protected function generate_listbox($field) {
 
-		$listbox = shortcode_atts( array(
+		$listbox = shortcode_atts(array(
 			'label'    => '',
 			'minWidth' => '',
 			'name'     => false,
 			'tooltip'  => '',
 			'type'     => '',
 			'value'    => '',
-		), $field );
+		), $field);
 
-		if ( $this->validate( $field ) ) {
+		if ($this->validate($field)) {
 
 			$new_listbox = array();
 
-			foreach ( $listbox as $key => $value ) {
+			foreach ($listbox as $key => $value) {
 
-				if ( $key == 'value' && empty( $value ) ) {
-					$new_listbox[ $key ] = $listbox['name'];
-				} else if ( $value ) {
-					$new_listbox[ $key ] = $value;
+				if ($key == 'value' && empty($value)) {
+					$new_listbox[$key] = $listbox['name'];
+				} else if ($value) {
+					$new_listbox[$key] = $value;
 				}
 			}
 
 			// do not reindex array!
 			$field['options'] = array(
-				                    '' => ( $field['placeholder'] ? $field['placeholder'] : esc_html( '- Select -', 'give' ) ),
+				                    '' => ($field['placeholder'] ? $field['placeholder'] : esc_html('- Select -', 'give')),
 			                    ) + $field['options'];
 
-			foreach ( $field['options'] as $value => $text ) {
+			foreach ($field['options'] as $value => $text) {
 				$new_listbox['values'][] = array(
 					'text'  => $text,
 					'value' => $value,
@@ -273,7 +273,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	protected function generate_post( $field ) {
+	protected function generate_post($field) {
 
 		$args = array(
 			'post_type'      => 'post',
@@ -282,23 +282,23 @@ 
 			'posts_per_page' => 30,
 		);
 
-		$args    = wp_parse_args( (array) $field['query_args'], $args );
-		$posts   = get_posts( $args );
+		$args    = wp_parse_args((array) $field['query_args'], $args);
+		$posts   = get_posts($args);
 		$options = array();
 
-		if ( $posts ) {
-			foreach ( $posts as $post ) {
-				$options[ absint( $post->ID ) ] = esc_html( $post->post_title );
+		if ($posts) {
+			foreach ($posts as $post) {
+				$options[absint($post->ID)] = esc_html($post->post_title);
 			}
 
 			$field['type']    = 'listbox';
 			$field['options'] = $options;
 
-			return $this->generate_listbox( $field );
+			return $this->generate_listbox($field);
 		}
 
 		// perform validation here before returning false
-		$this->validate( $field );
+		$this->validate($field);
 
 		return false;
 	}
@@ -312,9 +312,9 @@ 
 	 *
 	 * @since 1.0
 	 */
-	protected function generate_textbox( $field ) {
+	protected function generate_textbox($field) {
 
-		$textbox = shortcode_atts( array(
+		$textbox = shortcode_atts(array(
 			'label'     => '',
 			'maxLength' => '',
 			'minHeight' => '',
@@ -324,10 +324,10 @@ 
 			'tooltip'   => '',
 			'type'      => '',
 			'value'     => '',
-		), $field );
+		), $field);
 
-		if ( $this->validate( $field ) ) {
-			return array_filter( $textbox, array( $this, 'return_textbox_value' ) );
+		if ($this->validate($field)) {
+			return array_filter($textbox, array($this, 'return_textbox_value'));
 		}
 
 		return false;
@@ -340,7 +340,7 @@ 
 	 *
 	 * @return bool
 	 */
-	function return_textbox_value( $value ) {
+	function return_textbox_value($value) {
 		return $value !== '';
 	}
 
@@ -356,9 +356,9 @@ 
 	 *
 	 * @since 1.0
 	 */
-	protected function validate( $field ) {
+	protected function validate($field) {
 
-		extract( shortcode_atts(
+		extract(shortcode_atts(
 				array(
 					'name'     => false,
 					'required' => false,
@@ -366,36 +366,36 @@ 
 				), $field )
 		);
 
-		if ( $name ) {
+		if ($name) {
 
-			if ( isset( $required['error'] ) ) {
+			if (isset($required['error'])) {
 
 				$error = array(
 					'type' => 'container',
 					'html' => $required['error'],
 				);
 
-				$this->errors[ $name ] = $this->generate_container( $error );
+				$this->errors[$name] = $this->generate_container($error);
 			}
 
-			if ( ! ! $required || is_array( $required ) ) {
+			if ( ! ! $required || is_array($required)) {
 
-				$alert = esc_html( 'Some of the Shortcode options are required.', 'give' );
+				$alert = esc_html('Some of the Shortcode options are required.', 'give');
 
-				if ( isset( $required['alert'] ) ) {
+				if (isset($required['alert'])) {
 
 					$alert = $required['alert'];
 
-				} else if ( ! empty( $label ) ) {
+				} else if ( ! empty($label)) {
 
 					$alert = sprintf(
 						/* translators: %s: option lable */
-						esc_html( 'The "%s" option is required.', 'give' ),
-						str_replace( ':', '', $label )
+						esc_html('The "%s" option is required.', 'give'),
+						str_replace(':', '', $label)
 					);
 				}
 
-				$this->required[ $name ] = $alert;
+				$this->required[$name] = $alert;
 			}
 
 			return true;

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

Spacing +33 added lines, -33 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.3.0
  */
 
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 /**
  * Class Give_Shortcode_Donation_Form
@@ -21,10 +21,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');
 	}
 
 	/**
@@ -36,8 +36,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(
@@ -47,68 +47,68 @@ 
 					'post_type' => 'give_forms',
 				),
 				'name'        => 'id',
-				'tooltip'     => esc_attr( 'Select a Donation Form', 'give' ),
-				'placeholder' => esc_attr( '- Select a Form -', 'give' ),
+				'tooltip'     => esc_attr('Select a Donation Form', 'give'),
+				'placeholder' => esc_attr('- Select a 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 donation forms were 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 donation forms were found!', 'give'), $create_form_link),
 				),
 			),
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="strong margin-top">%s</p>', esc_html( 'Optional form settings', 'give' ) ),
+				'html' => sprintf('<p class="strong margin-top">%s</p>', esc_html('Optional form settings', 'give')),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'show_title',
-				'label'   => esc_html( 'Show Title:', 'give' ),
-				'tooltip' => esc_attr( 'Do you want to display the form title?', 'give' ),
+				'label'   => esc_html('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_html( 'Show Goal:', 'give' ),
-				'tooltip' => esc_attr( 'Do you want to display the donation goal?', 'give' ),
+				'label'   => esc_html('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_html( 'Display Content:', 'give' ),
-				'tooltip' => esc_attr( 'Do you want to display the form content?', 'give' ),
+				'label'   => esc_html('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 above the form fields', 'give' ),
-					'below' => esc_html( 'Display below the form fields', 'give' ),
+					'none'  => esc_html('No Content', 'give'),
+					'above' => esc_html('Display above the form fields', 'give'),
+					'below' => esc_html('Display below the form fields', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'display_style',
-				'label'   => esc_html( 'Payment Fields:', 'give' ),
-				'tooltip' => esc_attr( 'How would you like to display payment information?', 'give' ),
+				'label'   => esc_html('Payment Fields:', 'give'),
+				'tooltip' => esc_attr('How would you like to display payment information?', 'give'),
 				'options' => array(
-					'onpage' => esc_html( 'Show on Page', 'give' ),
-					'reveal' => esc_html( 'Reveal Upon Click', 'give' ),
-					'modal'  => esc_html( 'Modal Window Upon Click', 'give' ),
+					'onpage' => esc_html('Show on Page', 'give'),
+					'reveal' => esc_html('Reveal Upon Click', 'give'),
+					'modal'  => esc_html('Modal Window Upon Click', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'float_labels',
-				'label'   => esc_html( 'Floating Labels:', 'give' ),
-				'tooltip' => esc_attr( 'Override the default floating labels setting for this form?', 'give' ),
+				'label'   => esc_html('Floating Labels:', 'give'),
+				'tooltip' => esc_attr('Override the default floating labels setting for this form?', 'give'),
 				'options' => array(
-					'enabled'  => esc_html( 'Enabled', 'give' ),
-					'disabled' => esc_html( 'Disabled', 'give' ),
+					'enabled'  => esc_html('Enabled', 'give'),
+					'disabled' => esc_html('Disabled', 'give'),
 				),
 			),
 		);

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

Spacing +3 added lines, -3 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.3.0
  */
 
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 class Give_Shortcode_Donation_History extends Give_Shortcode_Generator {
 
@@ -18,9 +18,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/admin/shortcodes/shortcode-give-receipt.php

Spacing +23 added lines, -23 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.3.0
  */
 
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 class Give_Shortcode_Donation_Receipt extends Give_Shortcode_Generator {
 
@@ -18,10 +18,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = esc_html( 'Donation Receipt', 'give' );
-		$this->shortcode['label'] = esc_html( 'Donation Receipt', 'give' );
+		$this->shortcode['title'] = esc_html('Donation Receipt', 'give');
+		$this->shortcode['label'] = esc_html('Donation Receipt', 'give');
 
-		parent::__construct( 'give_receipt' );
+		parent::__construct('give_receipt');
 	}
 
 	/**
@@ -34,60 +34,60 @@ 
 		return array(
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="strong">%s</p>', esc_html( 'Optional settings', 'give' ) ),
+				'html' => sprintf('<p class="strong">%s</p>', esc_html('Optional settings', 'give')),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'price',
-				'label'   => esc_html( 'Show Donation Amount:', 'give' ),
+				'label'   => esc_html('Show Donation Amount:', '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'    => 'donor',
-				'label'   => esc_html( 'Show Donor Name:', 'give' ),
+				'label'   => esc_html('Show Donor Name:', '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'    => 'date',
-				'label'   => esc_html( 'Show Date:', 'give' ),
+				'label'   => esc_html('Show Date:', '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'    => 'payment_key',
-				'label'   => esc_html( 'Show Payment Key:', 'give' ),
+				'label'   => esc_html('Show Payment Key:', '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'    => 'payment_method',
-				'label'   => esc_html( 'Show Payment Method:', 'give' ),
+				'label'   => esc_html('Show Payment Method:', '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'    => 'payment_id',
-				'label'   => esc_html( 'Show Payment ID:', 'give' ),
+				'label'   => esc_html('Show Payment ID:', 'give'),
 				'options' => array(
-					'true'  => esc_html( 'Show', 'give' ),
-					'false' => esc_html( 'Hide', 'give' ),
+					'true'  => esc_html('Show', 'give'),
+					'false' => esc_html('Hide', 'give'),
 				),
 			),
 		);

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

Spacing +19 added lines, -19 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.3.0
  */
 
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 class Give_Shortcode_Donation_Form_Goal extends Give_Shortcode_Generator {
 
@@ -18,10 +18,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = esc_html( 'Donation Form Goal', 'give' );
-		$this->shortcode['label'] = esc_html( 'Donation Form Goal', 'give' );
+		$this->shortcode['title'] = esc_html('Donation Form Goal', 'give');
+		$this->shortcode['label'] = esc_html('Donation Form Goal', 'give');
 
-		parent::__construct( 'give_goal' );
+		parent::__construct('give_goal');
 	}
 
 	/**
@@ -33,8 +33,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(
@@ -44,35 +44,35 @@ 
 					'post_type' => 'give_forms',
 				),
 				'name'        => 'id',
-				'tooltip'     => esc_attr( 'Select a Donation Form', 'give' ),
-				'placeholder' => esc_attr( '- Select a Form -', 'give' ),
+				'tooltip'     => esc_attr('Select a Donation Form', 'give'),
+				'placeholder' => esc_attr('- Select a 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 donation forms were 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 donation forms were 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_text',
-				'label'   => esc_html( 'Show Text:', 'give' ),
-				'tooltip' => esc_attr( 'This text displays the amount of income raised compared to the goal.', 'give' ),
+				'label'   => esc_html('Show Text:', 'give'),
+				'tooltip' => esc_attr('This text displays the amount of income raised compared to the 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_bar',
-				'label'   => esc_html( 'Show Progress Bar:', 'give' ),
-				'tooltip' => esc_attr( 'Do you want to display the goal\'s progress bar?', 'give' ),
+				'label'   => esc_html('Show Progress Bar:', 'give'),
+				'tooltip' => esc_attr('Do you want to display the goal\'s progress bar?', 'give'),
 				'options' => array(
-					'true'  => esc_html( 'Show', 'give' ),
-					'false' => esc_html( 'Hide', 'give' ),
+					'true'  => esc_html('Show', 'give'),
+					'false' => esc_html('Hide', 'give'),
 				),
 			),
 		);

includes/admin/dashboard-widgets.php

Spacing +33 added lines, -33 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,12 +21,12 @@ 
  * @return void
  */
 function give_register_dashboard_widgets() {
-	if ( current_user_can( apply_filters( 'give_dashboard_stats_cap', 'view_give_reports' ) ) ) {
-		wp_add_dashboard_widget( 'give_dashboard_sales', esc_html( 'Give: Donation Statistics', 'give' ), 'give_dashboard_sales_widget' );
+	if (current_user_can(apply_filters('give_dashboard_stats_cap', 'view_give_reports'))) {
+		wp_add_dashboard_widget('give_dashboard_sales', esc_html('Give: Donation Statistics', 'give'), 'give_dashboard_sales_widget');
 	}
 }
 
-add_action( 'wp_dashboard_setup', 'give_register_dashboard_widgets', 10 );
+add_action('wp_dashboard_setup', 'give_register_dashboard_widgets', 10);
 
 /**
  * Sales Summary Dashboard Widget
@@ -38,7 +38,7 @@ 
  */
 function give_dashboard_sales_widget() {
 
-	if ( ! current_user_can( apply_filters( 'give_dashboard_stats_cap', 'view_give_reports' ) ) ) {
+	if ( ! current_user_can(apply_filters('give_dashboard_stats_cap', 'view_give_reports'))) {
 		return;
 	}
 	$stats = new Give_Payment_Stats; ?>
@@ -46,25 +46,25 @@ 
 	<div class="give-dashboard-widget">
 
 		<div class="give-dashboard-today give-clearfix">
-			<h3 class="give-dashboard-date-today"><?php echo date( 'F j, Y' ); ?></h3>
+			<h3 class="give-dashboard-date-today"><?php echo date('F j, Y'); ?></h3>
 
 			<p class="give-dashboard-happy-day"><?php
 				printf(
 					/* translators: %s: day of the week */
-					esc_html( 'Happy %s!', 'give' ),
-					date( 'l', current_time( 'timestamp' ) )
+					esc_html('Happy %s!', 'give'),
+					date('l', current_time('timestamp'))
 				);
 			?></p>
 
-			<?php $earnings_today = $stats->get_earnings( 0, 'today', false ); ?>
+			<?php $earnings_today = $stats->get_earnings(0, 'today', false); ?>
 
-			<p class="give-dashboard-today-earnings"><?php echo give_currency_filter( give_format_amount( $earnings_today ) ); ?></p>
+			<p class="give-dashboard-today-earnings"><?php echo give_currency_filter(give_format_amount($earnings_today)); ?></p>
 
-			<p class="give-orders-today"><?php $donations_today = $stats->get_sales( 0, 'today', false, array(
+			<p class="give-orders-today"><?php $donations_today = $stats->get_sales(0, 'today', false, array(
 					'publish',
 					'revoked'
-				) ); ?><?php echo give_format_amount( $donations_today, false ); ?>
-				<span><?php echo _x( 'donations today', 'Displays in WP admin dashboard widget after the day\'s total donations', 'give' ); ?></span>
+				)); ?><?php echo give_format_amount($donations_today, false); ?>
+				<span><?php echo _x('donations today', 'Displays in WP admin dashboard widget after the day\'s total donations', 'give'); ?></span>
 			</p>
 
 
@@ -74,34 +74,34 @@ 
 		<table class="give-table-stats">
 			<thead style="display: none;">
 			<tr>
-				<th><?php esc_html_e( 'This Week', 'give' ); ?></th>
-				<th><?php esc_html_e( 'This Month', 'give' ); ?></th>
-				<th><?php esc_html_e( 'Past 30 Days', 'give' ); ?></th>
+				<th><?php esc_html_e('This Week', 'give'); ?></th>
+				<th><?php esc_html_e('This Month', 'give'); ?></th>
+				<th><?php esc_html_e('Past 30 Days', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
 			<tr id="give-table-stats-tr-1">
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_week' ) ) ); ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_week'))); ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php esc_html_e( 'this week', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php esc_html_e('this week', 'give'); ?></p>
 				</td>
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_month' ) ) ); ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_month'))); ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php esc_html_e( 'this month', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php esc_html_e('this month', 'give'); ?></p>
 				</td>
 			</tr>
 			<tr id="give-table-stats-tr-2">
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'last_month' ) ) ) ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'last_month'))) ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php esc_html_e( 'last month', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php esc_html_e('last month', 'give'); ?></p>
 				</td>
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_year', false ) ) ) ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_year', false))) ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php esc_html_e( 'this year', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php esc_html_e('this year', 'give'); ?></p>
 				</td>
 			</tr>
 			</tbody>
@@ -121,25 +121,25 @@ 
  *
  * @return array
  */
-function give_dashboard_at_a_glance_widget( $items ) {
-	$num_posts = wp_count_posts( 'give_forms' );
+function give_dashboard_at_a_glance_widget($items) {
+	$num_posts = wp_count_posts('give_forms');
 
-	if ( $num_posts && $num_posts->publish ) {
+	if ($num_posts && $num_posts->publish) {
         
 		$text = sprintf(
 			/* translators: 1: number of posts published 2: forms singular label 3: forms plural label */
-			_n( '%1$s Give %2$s', '%1$s Give %3$s', $num_posts->publish, 'give' ),
+			_n('%1$s Give %2$s', '%1$s Give %3$s', $num_posts->publish, 'give'),
 			$num_posts->publish,
 			give_get_forms_label_singular(),
 			give_get_forms_label_plural()
 		);
 
-		$text = sprintf( $text, number_format_i18n( $num_posts->publish ) );
+		$text = sprintf($text, number_format_i18n($num_posts->publish));
 
-		if ( current_user_can( 'edit_give_forms', get_current_user_id() ) ) {
-			$text = sprintf( '<a class="give-forms-count" href="edit.php?post_type=give_forms">%1$s</a>', $text );
+		if (current_user_can('edit_give_forms', get_current_user_id())) {
+			$text = sprintf('<a class="give-forms-count" href="edit.php?post_type=give_forms">%1$s</a>', $text);
 		} else {
-			$text = sprintf( '<span class="give-forms-count">%1$s</span>', $text );
+			$text = sprintf('<span class="give-forms-count">%1$s</span>', $text);
 		}
 
 		$items[] = $text;
@@ -148,4 +148,4 @@ 
 	return $items;
 }
 
-add_filter( 'dashboard_glance_items', 'give_dashboard_at_a_glance_widget', 1, 1);
+add_filter('dashboard_glance_items', 'give_dashboard_at_a_glance_widget', 1, 1);

includes/admin/admin-pages.php

Spacing +65 added lines, -65 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -29,31 +29,31 @@ 
 	global $give_settings_page, $give_payments_page, $give_reports_page, $give_add_ons_page, $give_upgrades_screen, $give_donors_page;
 
 	//Payments
-	$give_payment       = get_post_type_object( 'give_payment' );
-	$give_payments_page = add_submenu_page( 'edit.php?post_type=give_forms', $give_payment->labels->name, $give_payment->labels->menu_name, 'edit_give_payments', 'give-payment-history', 'give_payment_history_page' );
+	$give_payment       = get_post_type_object('give_payment');
+	$give_payments_page = add_submenu_page('edit.php?post_type=give_forms', $give_payment->labels->name, $give_payment->labels->menu_name, 'edit_give_payments', 'give-payment-history', 'give_payment_history_page');
 
 	//Donors
-	$give_donors_page = add_submenu_page( 'edit.php?post_type=give_forms', esc_html( 'Donors', 'give' ), esc_html( 'Donors', 'give' ), 'view_give_reports', 'give-donors', 'give_customers_page' );
+	$give_donors_page = add_submenu_page('edit.php?post_type=give_forms', esc_html('Donors', 'give'), esc_html('Donors', 'give'), 'view_give_reports', 'give-donors', 'give_customers_page');
 
 	//Reports`
-	$give_reports_page = add_submenu_page( 'edit.php?post_type=give_forms', esc_html( 'Donation Reports', 'give' ), esc_html( 'Reports', 'give' ), 'view_give_reports', 'give-reports', 'give_reports_page' );
+	$give_reports_page = add_submenu_page('edit.php?post_type=give_forms', esc_html('Donation Reports', 'give'), esc_html('Reports', 'give'), 'view_give_reports', 'give-reports', 'give_reports_page');
 
 	//Settings
-	$give_settings_page = add_submenu_page( 'edit.php?post_type=give_forms', esc_html( 'Give Settings', 'give' ), esc_html( 'Settings', 'give' ), 'manage_give_settings', 'give-settings', array(
+	$give_settings_page = add_submenu_page('edit.php?post_type=give_forms', esc_html('Give Settings', 'give'), esc_html('Settings', 'give'), 'manage_give_settings', 'give-settings', array(
 		Give()->give_settings,
 		'admin_page_display'
-	) );
+	));
 
 	//Add-ons
-	$give_add_ons_page = add_submenu_page( 'edit.php?post_type=give_forms', esc_html( 'Give Add-ons', 'give' ), esc_html( 'Add-ons', 'give' ), 'install_plugins', 'give-addons', 'give_add_ons_page' );
+	$give_add_ons_page = add_submenu_page('edit.php?post_type=give_forms', esc_html('Give Add-ons', 'give'), esc_html('Add-ons', 'give'), 'install_plugins', 'give-addons', 'give_add_ons_page');
 
 	//Upgrades
-	$give_upgrades_screen = add_submenu_page( null, esc_html( 'Give Upgrades', 'give' ), esc_html( 'Give Upgrades', 'give' ), 'manage_give_settings', 'give-upgrades', 'give_upgrades_screen' );
+	$give_upgrades_screen = add_submenu_page(null, esc_html('Give Upgrades', 'give'), esc_html('Give Upgrades', 'give'), 'manage_give_settings', 'give-upgrades', 'give_upgrades_screen');
 
 
 }
 
-add_action( 'admin_menu', 'give_add_options_links', 10 );
+add_action('admin_menu', 'give_add_options_links', 10);
 
 /**
  *  Determines whether the current admin page is a Give admin page.
@@ -68,224 +68,224 @@ 
  *
  * @return bool True if Give admin page.
  */
-function give_is_admin_page( $passed_page = '', $passed_view = '' ) {
+function give_is_admin_page($passed_page = '', $passed_view = '') {
 
 	global $pagenow, $typenow;
 
 	$found     = false;
-	$post_type = isset( $_GET['post_type'] ) ? strtolower( $_GET['post_type'] ) : false;
-	$action    = isset( $_GET['action'] ) ? strtolower( $_GET['action'] ) : false;
-	$taxonomy  = isset( $_GET['taxonomy'] ) ? strtolower( $_GET['taxonomy'] ) : false;
-	$page      = isset( $_GET['page'] ) ? strtolower( $_GET['page'] ) : false;
-	$view      = isset( $_GET['view'] ) ? strtolower( $_GET['view'] ) : false;
-	$tab       = isset( $_GET['tab'] ) ? strtolower( $_GET['tab'] ) : false;
+	$post_type = isset($_GET['post_type']) ? strtolower($_GET['post_type']) : false;
+	$action    = isset($_GET['action']) ? strtolower($_GET['action']) : false;
+	$taxonomy  = isset($_GET['taxonomy']) ? strtolower($_GET['taxonomy']) : false;
+	$page      = isset($_GET['page']) ? strtolower($_GET['page']) : false;
+	$view      = isset($_GET['view']) ? strtolower($_GET['view']) : false;
+	$tab       = isset($_GET['tab']) ? strtolower($_GET['tab']) : false;
 
-	switch ( $passed_page ) {
+	switch ($passed_page) {
 		case 'give_forms':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php') {
 						$found = true;
 					}
 					break;
 				case 'edit':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'post.php' ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'post.php') {
 						$found = true;
 					}
 					break;
 				case 'new':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'post-new.php' ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'post-new.php') {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) || 'give_forms' === $post_type || ( 'post-new.php' == $pagenow && 'give_forms' === $post_type ) ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) || 'give_forms' === $post_type || ('post-new.php' == $pagenow && 'give_forms' === $post_type)) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'categories':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
 				case 'new':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'edit' !== $action && 'give_forms_category' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'edit' !== $action && 'give_forms_category' === $taxonomy) {
 						$found = true;
 					}
 					break;
 				case 'edit':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'edit' === $action && 'give_forms_category' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'edit' === $action && 'give_forms_category' === $taxonomy) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'give_forms_category' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'give_forms_category' === $taxonomy) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'tags':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
 				case 'new':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'edit' !== $action && 'give_forms_tag' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'edit' !== $action && 'give_forms_tag' === $taxonomy) {
 						$found = true;
 					}
 					break;
 				case 'edit':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'edit' === $action && 'give_forms_tag' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'edit' === $action && 'give_forms_tag' === $taxonomy) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'give_forms_tag' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'give_forms_tag' === $taxonomy) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'payments':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-payment-history' === $page && false === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-payment-history' === $page && false === $view) {
 						$found = true;
 					}
 					break;
 				case 'edit':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-payment-history' === $page && 'view-order-details' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-payment-history' === $page && 'view-order-details' === $view) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-payment-history' === $page ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-payment-history' === $page) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'reports':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				// If you want to do something like enqueue a script on a particular report's duration, look at $_GET[ 'range' ]
 				case 'earnings':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && ( 'earnings' === $view || '-1' === $view || false === $view ) ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && ('earnings' === $view || '-1' === $view || false === $view)) {
 						$found = true;
 					}
 					break;
 				case 'donors':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && 'customers' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && 'customers' === $view) {
 						$found = true;
 					}
 					break;
 				case 'gateways':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && 'gateways' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && 'gateways' === $view) {
 						$found = true;
 					}
 					break;
 				case 'export':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && 'export' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && 'export' === $view) {
 						$found = true;
 					}
 					break;
 				case 'logs':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && 'logs' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && 'logs' === $view) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'settings':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'general':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && ( 'general' === $tab || false === $tab ) ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && ('general' === $tab || false === $tab)) {
 						$found = true;
 					}
 					break;
 				case 'gateways':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'gateways' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'gateways' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'emails':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'emails' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'emails' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'display':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'display' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'display' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'licenses':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'licenses' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'licenses' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'api':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'api' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'api' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'advanced':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'advanced' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'advanced' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'system_info':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'system_info' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'system_info' === $tab) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'addons':
-			if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-addons' === $page ) {
+			if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-addons' === $page) {
 				$found = true;
 			}
 			break;
 		case 'donors':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-donors' === $page && false === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-donors' === $page && false === $view) {
 						$found = true;
 					}
 					break;
 				case 'overview':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-donors' === $page && 'overview' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-donors' === $page && 'overview' === $view) {
 						$found = true;
 					}
 					break;
 				case 'notes':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-donors' === $page && 'notes' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-donors' === $page && 'notes' === $view) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-donors' === $page ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-donors' === $page) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'reports':
-			if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page ) {
+			if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page) {
 				$found = true;
 			}
 			break;
 		default:
 			global $give_payments_page, $give_settings_page, $give_reports_page, $give_system_info_page, $give_add_ons_page, $give_settings_export, $give_upgrades_screen, $give_customers_page;
 
-			$admin_pages = apply_filters( 'give_admin_pages', array(
+			$admin_pages = apply_filters('give_admin_pages', array(
 				$give_payments_page,
 				$give_settings_page,
 				$give_reports_page,
@@ -294,18 +294,18 @@ 
 				$give_upgrades_screen,
 				$give_settings_export,
 				$give_customers_page
-			) );
-			if ( 'give_forms' == $typenow || 'index.php' == $pagenow || 'post-new.php' == $pagenow || 'post.php' == $pagenow ) {
+			));
+			if ('give_forms' == $typenow || 'index.php' == $pagenow || 'post-new.php' == $pagenow || 'post.php' == $pagenow) {
 				$found = true;
-				if ( 'give-upgrades' === $page ) {
+				if ('give-upgrades' === $page) {
 					$found = false;
 				}
-			} elseif ( in_array( $pagenow, $admin_pages ) ) {
+			} elseif (in_array($pagenow, $admin_pages)) {
 				$found = true;
 			}
 			break;
 	}
 
-	return (bool) apply_filters( 'give_is_admin_page', $found, $page, $view, $passed_page, $passed_view );
+	return (bool) apply_filters('give_is_admin_page', $found, $page, $view, $passed_page, $passed_view);
 
 }
\ No newline at end of file

includes/admin/class-addon-activation-banner.php

Spacing +22 added lines, -22 removed lines

@@ -8,7 +8,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,12 +26,12 @@ 
 	 *
 	 * @param $_banner_details
 	 */
-	function __construct( $_banner_details ) {
+	function __construct($_banner_details) {
 
 		global $current_user;
 		$this->banner_details = $_banner_details;
-		$this->test_mode      = ( $this->banner_details['testing'] == 'true' ) ? true : false;
-		$this->nag_meta_key   = 'give_addon_activation_ignore_' . sanitize_title( $this->banner_details['name'] );
+		$this->test_mode      = ($this->banner_details['testing'] == 'true') ? true : false;
+		$this->nag_meta_key   = 'give_addon_activation_ignore_'.sanitize_title($this->banner_details['name']);
 
 		//Get current user
 		$this->user_id = $current_user->ID;
@@ -50,13 +50,13 @@ 
 	public function init() {
 
 		//Testing?
-		if ( $this->test_mode ) {
-			delete_user_meta( $this->user_id, $this->nag_meta_key );
+		if ($this->test_mode) {
+			delete_user_meta($this->user_id, $this->nag_meta_key);
 		}
 
 		//Get the current page to add the notice to
-		add_action( 'current_screen', array( $this, 'give_addon_notice_ignore' ) );
-		add_action( 'admin_notices', array( $this, 'give_addon_activation_admin_notice' ) );
+		add_action('current_screen', array($this, 'give_addon_notice_ignore'));
+		add_action('admin_notices', array($this, 'give_addon_activation_admin_notice'));
 
 
 	}
@@ -71,13 +71,13 @@ 
 		global $pagenow;
 
 		//Make sure we're on the plugins page.
-		if ( $pagenow !== 'plugins.php' ) {
+		if ($pagenow !== 'plugins.php') {
 			return false;
 		}
 
 		// If the user hasn't already dismissed our alert,
 		// Output the activation banner
-		if ( ! get_user_meta( $this->user_id, $this->nag_meta_key ) ) { ?>
+		if ( ! get_user_meta($this->user_id, $this->nag_meta_key)) { ?>
 
 			<!-- * I output inline styles here
 				 * because there's no reason to keep these
@@ -157,15 +157,15 @@ 
 				<h3><?php
 					printf(
 						/* translators: %s: Add-on name */
-						esc_html( "Thank you for installing Give's %s Add-on!", 'give' ),
-						'<span>' . $this->banner_details['name'] . '</span>'
+						esc_html("Thank you for installing Give's %s Add-on!", 'give'),
+						'<span>'.$this->banner_details['name'].'</span>'
 					);
 				?></h3>
 
 				<a href="<?php
 				//The Dismiss Button
-				$nag_admin_dismiss_url = 'plugins.php?' . $this->nag_meta_key . '=0';
-				echo admin_url( $nag_admin_dismiss_url ); ?>" class="dismiss"><span class="dashicons dashicons-dismiss"></span></a>
+				$nag_admin_dismiss_url = 'plugins.php?'.$this->nag_meta_key.'=0';
+				echo admin_url($nag_admin_dismiss_url); ?>" class="dismiss"><span class="dashicons dashicons-dismiss"></span></a>
 
 				<!-- * Now we output a few "actions"
 					 * that the user can take from here -->
@@ -173,21 +173,21 @@ 
 				<div class="alert-actions">
 
 					<?php //Point them to your settings page
-					if ( isset( $this->banner_details['settings_url'] ) ) { ?>
+					if (isset($this->banner_details['settings_url'])) { ?>
 						<a href="<?php echo $this->banner_details['settings_url']; ?>">
-							<span class="dashicons dashicons-admin-settings"></span><?php esc_html_e( 'Go to Settings', 'give' ); ?>
+							<span class="dashicons dashicons-admin-settings"></span><?php esc_html_e('Go to Settings', 'give'); ?>
 						</a>
 					<?php } ?>
 
 					<?php
 					// Show them how to configure the Addon
-					if ( isset( $this->banner_details['documentation_url'] ) ) { ?>
+					if (isset($this->banner_details['documentation_url'])) { ?>
 						<a href="<?php echo $this->banner_details['documentation_url'] ?>" target="_blank">
 							<span class="dashicons dashicons-media-text"></span>
 							<?php
 								printf(
 									/* translators: %s: Add-on name */
-									esc_html( 'Documentation: %s Add-on', 'give' ),
+									esc_html('Documentation: %s Add-on', 'give'),
 									$this->banner_details['name']
 								);
 							?>
@@ -195,10 +195,10 @@ 
 					<?php } ?>
 					<?php
 					//Let them signup for plugin updates
-					if ( isset( $this->banner_details['support_url'] ) ) { ?>
+					if (isset($this->banner_details['support_url'])) { ?>
 
 						<a href="<?php echo $this->banner_details['support_url'] ?>" target="_blank">
-							<span class="dashicons dashicons-sos"></span><?php esc_html_e( 'Get Support', 'give' ); ?>
+							<span class="dashicons dashicons-sos"></span><?php esc_html_e('Get Support', 'give'); ?>
 						</a>
 
 					<?php } ?>
@@ -220,13 +220,13 @@ 
 		/* If user clicks to ignore the notice, add that to their user meta the banner then checks whether this tag exists already or not.
 		 * See here: http://codex.wordpress.org/Function_Reference/add_user_meta
 		 */
-		if ( isset( $_GET[ $this->nag_meta_key ] ) && '0' == $_GET[ $this->nag_meta_key ] ) {
+		if (isset($_GET[$this->nag_meta_key]) && '0' == $_GET[$this->nag_meta_key]) {
 
 			//Get the global user
 			global $current_user;
 			$user_id = $current_user->ID;
 
-			add_user_meta( $user_id, $this->nag_meta_key, 'true', true );
+			add_user_meta($user_id, $this->nag_meta_key, 'true', true);
 		}
 	}