WordImpress / Give

includes/admin/settings/class-settings-cmb2-backward-compatibility.php

Doc Comments +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
 		 *
 		 * @since  1.8
 		 *
-		 * @param  $setting_tab
+		 * @param  string $setting_tab
 		 *
 		 * @return string
 		 */

Spacing +119 added lines, -119 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.8
  */
 
-if ( ! class_exists( 'Give_CMB2_Settings_Loader' ) ) :
+if ( ! class_exists('Give_CMB2_Settings_Loader')) :
 
 	/**
 	 * This class loads the cmb2 settings.
@@ -52,30 +52,30 @@ 
 
 			// Get current tab.
 			$this->current_tab     = give_get_current_setting_tab();
-			$this->current_section = empty( $_REQUEST['section'] ) ? ( current( array_keys( $this->get_sections() ) ) ) : sanitize_title( $_REQUEST['section'] );
+			$this->current_section = empty($_REQUEST['section']) ? (current(array_keys($this->get_sections()))) : sanitize_title($_REQUEST['section']);
 
 			// Tab ID.
 			$this->id = $this->current_tab;
 
 			// Add addon tabs.
-			add_filter( 'give-settings_tabs_array', array( $this, 'add_addon_settings_page' ), 999999 );
+			add_filter('give-settings_tabs_array', array($this, 'add_addon_settings_page'), 999999);
 
 			// Add save hook to addons.
-			add_action( 'give-settings_get_settings_pages', array( $this, 'setup_addon_save_hook' ), 999999 );
+			add_action('give-settings_get_settings_pages', array($this, 'setup_addon_save_hook'), 999999);
 
 			// Add backward compatibility filters plugin settings.
-			$setting_tabs = array( 'general', 'gateways', 'display', 'emails', 'addons', 'licenses' );
+			$setting_tabs = array('general', 'gateways', 'display', 'emails', 'addons', 'licenses');
 
 			// Filter Payment Gateways settings.
-			if ( in_array( $this->current_tab, $setting_tabs ) ) {
-				add_filter( "give_get_settings_{$this->current_tab}", array(
+			if (in_array($this->current_tab, $setting_tabs)) {
+				add_filter("give_get_settings_{$this->current_tab}", array(
 					$this,
 					'get_filtered_addon_settings',
-				), 999999, 1 );
-				add_filter( "give_get_sections_{$this->current_tab}", array(
+				), 999999, 1);
+				add_filter("give_get_sections_{$this->current_tab}", array(
 					$this,
 					'get_filtered_addon_sections',
-				), 999999, 1 );
+				), 999999, 1);
 			}
 		}
 
@@ -88,12 +88,12 @@ 
 		 *
 		 * @return string
 		 */
-		function set_default_setting_tab( $setting_tab ) {
+		function set_default_setting_tab($setting_tab) {
 			$default_tab = '';
 
 			// Set default tab to first setting tab.
-			if ( $sections = array_keys( $this->get_sections() ) ) {
-				$default_tab = current( $sections );
+			if ($sections = array_keys($this->get_sections())) {
+				$default_tab = current($sections);
 			}
 
 			return $default_tab;
@@ -108,29 +108,29 @@ 
 		 *
 		 * @return mixed
 		 */
-		function add_addon_settings_page( $pages ) {
+		function add_addon_settings_page($pages) {
 			// Previous setting page.
 			$previous_pages = $this->prev_settings->give_get_settings_tabs();
 
 			// API and System Info setting tab merge to Tools setting tab, so remove them from tabs.
-			unset( $previous_pages['api'] );
-			unset( $previous_pages['system_info'] );
+			unset($previous_pages['api']);
+			unset($previous_pages['system_info']);
 
 			// Tab is not register.
-			$pages_diff = array_keys( array_diff( $previous_pages, $pages ) );
+			$pages_diff = array_keys(array_diff($previous_pages, $pages));
 
 			// Merge old settings with new settings.
-			$pages = array_merge( $pages, $previous_pages );
+			$pages = array_merge($pages, $previous_pages);
 
-			if ( in_array( $this->current_tab, $pages_diff ) ) {
+			if (in_array($this->current_tab, $pages_diff)) {
 				// Filter & actions.
-				add_filter( "give_default_setting_tab_section_{$this->current_tab}", array(
+				add_filter("give_default_setting_tab_section_{$this->current_tab}", array(
 					$this,
 					'set_default_setting_tab',
-				), 10 );
-				add_action( "give-settings_sections_{$this->current_tab}_page", array( $this, 'output_sections' ) );
-				add_action( "give-settings_settings_{$this->current_tab}_page", array( $this, 'output' ), 10 );
-				add_action( "give-settings_save_{$this->current_tab}", array( $this, 'save' ) );
+				), 10);
+				add_action("give-settings_sections_{$this->current_tab}_page", array($this, 'output_sections'));
+				add_action("give-settings_settings_{$this->current_tab}_page", array($this, 'output'), 10);
+				add_action("give-settings_save_{$this->current_tab}", array($this, 'save'));
 			}
 
 			return $pages;
@@ -146,29 +146,29 @@ 
 		 *
 		 * @return mixed
 		 */
-		function setup_addon_save_hook( $pages ) {
+		function setup_addon_save_hook($pages) {
 			$page_ids = array();
 
-			foreach ( $pages as $page ) {
-				$page_ids = $page->add_settings_page( $page_ids );
+			foreach ($pages as $page) {
+				$page_ids = $page->add_settings_page($page_ids);
 			}
 
 			// Previous setting page.
 			$previous_pages = $this->prev_settings->give_get_settings_tabs();
 
 			// API and System Info setting tab merge to Tools setting tab, so remove them from tabs.
-			unset( $previous_pages['api'] );
-			unset( $previous_pages['system_info'] );
+			unset($previous_pages['api']);
+			unset($previous_pages['system_info']);
 
 			// Tab is not register.
-			$pages_diff = array_keys( array_diff( $previous_pages, $page_ids ) );
+			$pages_diff = array_keys(array_diff($previous_pages, $page_ids));
 
 			// Merge old settings with new settings.
-			$pages = array_merge( $page_ids, $previous_pages );
+			$pages = array_merge($page_ids, $previous_pages);
 
-			if ( in_array( $this->current_tab, $pages_diff ) ) {
+			if (in_array($this->current_tab, $pages_diff)) {
 				// Filter & actions.
-				add_action( "give-settings_save_{$this->current_tab}", array( $this, 'save' ) );
+				add_action("give-settings_save_{$this->current_tab}", array($this, 'save'));
 			}
 
 			return $pages;
@@ -183,16 +183,16 @@ 
 		 *
 		 * @return string
 		 */
-		function get_section_name( $field_name ) {
+		function get_section_name($field_name) {
 			// Bailout.
-			if ( empty( $field_name ) ) {
+			if (empty($field_name)) {
 				return $field_name;
 			}
 
-			$section_name = explode( ' ', $field_name );
+			$section_name = explode(' ', $field_name);
 
 			// Output.
-			return strip_tags( implode( ' ', $section_name ) );
+			return strip_tags(implode(' ', $section_name));
 		}
 
 
@@ -205,39 +205,39 @@ 
 		 *
 		 * @return mixed
 		 */
-		function get_filtered_addon_sections( $sections = array() ) {
+		function get_filtered_addon_sections($sections = array()) {
 			// New sections.
 			$new_sections = array();
-			$sections_ID  = array_keys( $sections );
+			$sections_ID  = array_keys($sections);
 
-			if ( ( $setting_fields = $this->prev_settings->give_settings( $this->current_tab ) ) && ! empty( $setting_fields['fields'] ) ) {
+			if (($setting_fields = $this->prev_settings->give_settings($this->current_tab)) && ! empty($setting_fields['fields'])) {
 
-				foreach ( $setting_fields['fields'] as $field ) {
+				foreach ($setting_fields['fields'] as $field) {
 					// Section name.
-					$field['name'] = isset( $field['name'] ) ? $field['name'] : '';
-					$section_name  = $this->get_section_name( $field['name'] );
+					$field['name'] = isset($field['name']) ? $field['name'] : '';
+					$section_name  = $this->get_section_name($field['name']);
 
 					// Check if section name exit and section title array is not empty.
-					if ( ! empty( $sections ) && ! empty( $field['name'] ) ) {
+					if ( ! empty($sections) && ! empty($field['name'])) {
 
 						// Bailout: Do not load section if it is already exist.
 						if (
-							in_array( sanitize_title( $field['name'] ), $sections_ID ) // Check section id.
-							|| in_array( $section_name, $sections )                    // Check section name.
+							in_array(sanitize_title($field['name']), $sections_ID) // Check section id.
+							|| in_array($section_name, $sections)                    // Check section name.
 						) {
 							continue;
 						}
 					}
 
 					// Collect new sections from addons.
-					if ( 'give_title' == $field['type'] ) {
-						$new_sections[ sanitize_title( $field['name'] ) ] = $section_name;
+					if ('give_title' == $field['type']) {
+						$new_sections[sanitize_title($field['name'])] = $section_name;
 					}
 				}
 			}
 
 			// Add new section.
-			$sections = array_merge( $sections, $new_sections );
+			$sections = array_merge($sections, $new_sections);
 
 			// Output.
 			return $sections;
@@ -254,19 +254,19 @@ 
 		 *
 		 * @return array
 		 */
-		function get_filtered_addon_settings( $settings, $setting_fields = array() ) {
+		function get_filtered_addon_settings($settings, $setting_fields = array()) {
 			global $wp_filter;
 
 			$new_setting_fields = array();
 
-			if ( ! empty( $settings ) ) {
+			if ( ! empty($settings)) {
 				// Bailout: If setting array contain first element of type title then it means it is already created with new setting api (skip this section ).
-				if ( isset( $settings[0]['type'] ) && 'title' == $settings[0]['type'] ) {
-					foreach ( $settings as $setting ) {
+				if (isset($settings[0]['type']) && 'title' == $settings[0]['type']) {
+					foreach ($settings as $setting) {
 						$new_setting_fields[] = $setting;
 
 						// We need setting only till first section end.
-						if ( 'sectionend' === $setting['type'] ) {
+						if ('sectionend' === $setting['type']) {
 							break;
 						}
 					}
@@ -278,24 +278,24 @@ 
 				$prev_title_field_id = '';
 
 				// Create new setting fields.
-				foreach ( $settings as $index => $field ) {
+				foreach ($settings as $index => $field) {
 
 					// Bailout: Must need field type to process.
-					if ( ! isset( $field['type'] ) ) {
+					if ( ! isset($field['type'])) {
 						continue;
 					}
 
 					// Set wrapper class if any.
-					if ( ! empty( $field['row_classes'] ) ) {
+					if ( ! empty($field['row_classes'])) {
 						$field['wrapper_class'] = $field['row_classes'];
-						unset( $field['row_classes'] );
+						unset($field['row_classes']);
 					}
 
-					$field['name'] = ! isset( $field['name'] ) ? '' : $field['name'];
-					$field['desc'] = ! isset( $field['desc'] ) ? '' : $field['desc'];
+					$field['name'] = ! isset($field['name']) ? '' : $field['name'];
+					$field['desc'] = ! isset($field['desc']) ? '' : $field['desc'];
 
 					// Modify cmb2 setting fields.
-					switch ( $field['type'] ) {
+					switch ($field['type']) {
 						case 'text' :
 						case 'file' :
 							$field['css'] = 'width:25em;';
@@ -319,18 +319,18 @@ 
 							$field['type'] = 'title';
 
 							// Since we are showing sections, so there now ned to show horizontal rules.
-							if ( '<hr>' === $field['desc'] ) {
+							if ('<hr>' === $field['desc']) {
 								$field['desc'] = '';
 							}
 
 							break;
 					}
 
-					if ( 'title' === $field['type'] ) {
+					if ('title' === $field['type']) {
 
 						// If we do not have first element as title then these field will be skip from frontend
 						// because there are not belong to any section, so put all abandon fields under first section.
-						if ( $index && empty( $prev_title_field_id ) ) {
+						if ($index && empty($prev_title_field_id)) {
 							array_unshift(
 								$new_setting_fields,
 								array(
@@ -344,7 +344,7 @@ 
 							$prev_title_field_id = $field['id'];
 
 							continue;
-						} elseif ( $index ) {
+						} elseif ($index) {
 							// Section end.
 							$new_setting_fields[] = array(
 								'type' => 'sectionend',
@@ -376,52 +376,52 @@ 
 
 				// Check if setting page has title section or not.
 				// If setting page does not have title section  then add title section to it and fix section end array id.
-				if ( 'title' !== $new_setting_fields[0]['type'] ) {
+				if ('title' !== $new_setting_fields[0]['type']) {
 					array_unshift(
 						$new_setting_fields,
 						array(
-							'title' => ( isset( $settings['give_title'] ) ? $settings['give_title'] : '' ),
+							'title' => (isset($settings['give_title']) ? $settings['give_title'] : ''),
 							'type'  => 'title',
-							'desc'  => ! empty( $setting_fields['desc'] ) ? $setting_fields['desc'] : '',
-							'id'    => ( isset( $settings['id'] ) ? $settings['id'] : '' ),
+							'desc'  => ! empty($setting_fields['desc']) ? $setting_fields['desc'] : '',
+							'id'    => (isset($settings['id']) ? $settings['id'] : ''),
 						)
 					);
 
 					// Update id in section end array if does not contain.
-					if ( empty( $new_setting_fields[ count( $new_setting_fields ) - 1 ]['id'] ) ) {
-						$new_setting_fields[ count( $new_setting_fields ) - 1 ]['id'] = ( isset( $settings['id'] ) ? $settings['id'] : '' );
+					if (empty($new_setting_fields[count($new_setting_fields) - 1]['id'])) {
+						$new_setting_fields[count($new_setting_fields) - 1]['id'] = (isset($settings['id']) ? $settings['id'] : '');
 					}
 				}
 
 				// Return only section related settings.
-				if ( $sections = $this->get_filtered_addon_sections() ) {
-					$new_setting_fields = $this->get_section_settings( $new_setting_fields );
+				if ($sections = $this->get_filtered_addon_sections()) {
+					$new_setting_fields = $this->get_section_settings($new_setting_fields);
 				}
 
 				// Third party plugin backward compatibility.
-				$wp_filter_keys = array_keys( $wp_filter );
-				foreach ( $new_setting_fields as $index => $field ) {
-					if ( ! isset( $field['type'] ) || in_array( $field['type'], array( 'title', 'sectionend' ) ) ) {
+				$wp_filter_keys = array_keys($wp_filter);
+				foreach ($new_setting_fields as $index => $field) {
+					if ( ! isset($field['type']) || in_array($field['type'], array('title', 'sectionend'))) {
 						continue;
 					}
 
 					$cmb2_filter_name = "cmb2_render_{$field['type']}";
 
-					if ( in_array( $cmb2_filter_name, $wp_filter_keys ) ) {
+					if (in_array($cmb2_filter_name, $wp_filter_keys)) {
 
-						if ( 0 >= version_compare( 4.7, get_bloginfo( 'version' ) ) && ! empty( $wp_filter[ $cmb2_filter_name ]->callbacks ) ) {
-							$cmb2_filter_arr = current( $wp_filter[ $cmb2_filter_name ]->callbacks );
+						if (0 >= version_compare(4.7, get_bloginfo('version')) && ! empty($wp_filter[$cmb2_filter_name]->callbacks)) {
+							$cmb2_filter_arr = current($wp_filter[$cmb2_filter_name]->callbacks);
 						} else {
-							$cmb2_filter_arr = current( $wp_filter[ $cmb2_filter_name ] );
+							$cmb2_filter_arr = current($wp_filter[$cmb2_filter_name]);
 						}
 
-						if ( ! empty( $cmb2_filter_arr ) ) {
+						if ( ! empty($cmb2_filter_arr)) {
 							// Note: function can be called either globally or with class object, it depends on how developer invoke it.
-							$new_setting_fields[ $index ]['func'] = current( $cmb2_filter_arr );
-							add_action( "give_admin_field_{$field['type']}", array(
+							$new_setting_fields[$index]['func'] = current($cmb2_filter_arr);
+							add_action("give_admin_field_{$field['type']}", array(
 								$this,
 								'addon_setting_field',
-							), 10, 2 );
+							), 10, 2);
 						}
 					}
 				}
@@ -442,31 +442,31 @@ 
 		 *
 		 * @return array
 		 */
-		function get_section_settings( $tab_settings ) {
+		function get_section_settings($tab_settings) {
 			$current_section = give_get_current_setting_section();
 
 			// Note: If we are opening default tabe for addon setting then it is possible that we will get empty string as current section
 			// because default section filter added after save hook fire, so we will always get problem to save first section [default] or if there are only on section
 			// This is hack to fix this.
-			if ( empty( $current_section ) ) {
-				$current_section = $this->set_default_setting_tab( $current_section );
+			if (empty($current_section)) {
+				$current_section = $this->set_default_setting_tab($current_section);
 			}
 
 			$section_start               = false;
 			$section_end                 = false;
 			$section_only_setting_fields = array();
 
-			foreach ( $tab_settings as $field ) {
-				if ( 'title' == $field['type'] && $current_section == sanitize_title( $field['title'] ) ) {
+			foreach ($tab_settings as $field) {
+				if ('title' == $field['type'] && $current_section == sanitize_title($field['title'])) {
 					$section_start = true;
 				}
 
-				if ( ! $section_start || $section_end ) {
+				if ( ! $section_start || $section_end) {
 					continue;
 				}
 
-				if ( $section_start && ! $section_end ) {
-					if ( 'sectionend' == $field['type'] ) {
+				if ($section_start && ! $section_end) {
+					if ('sectionend' == $field['type']) {
 						$section_end = true;
 					}
 					$section_only_setting_fields[] = $field;
@@ -476,7 +476,7 @@ 
 			// Remove title from setting, pevent it from render in setting tab.
 			$section_only_setting_fields[0]['title'] = '';
 
-			return apply_filters( "give_get_settings_{$this->current_tab}_{$current_section}", $section_only_setting_fields, $tab_settings );
+			return apply_filters("give_get_settings_{$this->current_tab}_{$current_section}", $section_only_setting_fields, $tab_settings);
 		}
 
 
@@ -490,17 +490,17 @@ 
 		 *
 		 * @return void
 		 */
-		function addon_setting_field( $field, $saved_value ) {
+		function addon_setting_field($field, $saved_value) {
 			// Create object for cmb2  function callback backward compatibility.
 			// Note: Do not call any cmb2 function on these objects
-			$field_obj      = (object) array( 'args' => $field );
-			$field_type_obj = (object) array( 'field' => $field_obj );
+			$field_obj      = (object) array('args' => $field);
+			$field_type_obj = (object) array('field' => $field_obj);
 
-			switch ( $this->current_tab ) :
+			switch ($this->current_tab) :
 				case 'licenses':
 					?>
 					<div class="give-settings-wrap give-settings-wrap-<?php echo $this->current_tab; ?>">
-						<?php $field['func']['function']( $field_obj, $saved_value, '', '', $field_type_obj ); ?>
+						<?php $field['func']['function']($field_obj, $saved_value, '', '', $field_type_obj); ?>
 					</div>
 					<?php break;
 
@@ -508,19 +508,19 @@ 
 					$colspan = 'colspan="2"';
 					?>
 					<tr valign="top">
-						<?php if ( ! empty( $field['name'] ) && ! in_array( $field['name'], array( '&nbsp;' ) ) ) : ?>
+						<?php if ( ! empty($field['name']) && ! in_array($field['name'], array('&nbsp;'))) : ?>
 							<th scope="row" class="titledesc">
 								<label
-										for="<?php echo esc_attr( $field['name'] ); ?>"><?php echo $field['title']; ?></label>
+										for="<?php echo esc_attr($field['name']); ?>"><?php echo $field['title']; ?></label>
 							</th>
 							<?php $colspan = ''; ?>
 						<?php endif; ?>
 						<td class="give-forminp" <?php echo $colspan; ?>>
 							<?php
-							if ( is_array( $field['func']['function'] ) ) {
-								$field['func']['function'][0]->$field['func']['function'][1]( $field_obj, $saved_value, '', '', $field_type_obj );
+							if (is_array($field['func']['function'])) {
+								$field['func']['function'][0]->$field['func']['function'][1]($field_obj, $saved_value, '', '', $field_type_obj);
 							} else {
-								$field['func']['function']( $field_obj, $saved_value, '', '', $field_type_obj );
+								$field['func']['function']($field_obj, $saved_value, '', '', $field_type_obj);
 							}
 							?>
 						</td>
@@ -538,10 +538,10 @@ 
 		public function get_sections() {
 			$sections = array();
 
-			if ( ( $setting_fields = $this->prev_settings->give_settings( $this->current_tab ) ) && ! empty( $setting_fields['fields'] ) ) {
-				foreach ( $setting_fields['fields'] as $field ) {
-					if ( 'give_title' == $field['type'] ) {
-						$sections[ sanitize_title( $field['name'] ) ] = $this->get_section_name( $field['name'] );
+			if (($setting_fields = $this->prev_settings->give_settings($this->current_tab)) && ! empty($setting_fields['fields'])) {
+				foreach ($setting_fields['fields'] as $field) {
+					if ('give_title' == $field['type']) {
+						$sections[sanitize_title($field['name'])] = $this->get_section_name($field['name']);
 					}
 				}
 			}
@@ -561,16 +561,16 @@ 
 
 			$new_setting_fields = array();
 
-			if ( $setting_fields = $this->prev_settings->give_settings( $this->current_tab ) ) {
-				if ( isset( $setting_fields['fields'] ) ) {
+			if ($setting_fields = $this->prev_settings->give_settings($this->current_tab)) {
+				if (isset($setting_fields['fields'])) {
 
 					$tab_data = array(
 						'id'         => $setting_fields['id'],
 						'give_title' => $setting_fields['give_title'],
-						'desc'       => ( isset( $setting_fields['desc'] ) ? $setting_fields['desc'] : '' ),
+						'desc'       => (isset($setting_fields['desc']) ? $setting_fields['desc'] : ''),
 					);
 
-					$new_setting_fields = $this->get_filtered_addon_settings( $setting_fields['fields'], $tab_data );
+					$new_setting_fields = $this->get_filtered_addon_settings($setting_fields['fields'], $tab_data);
 				}
 			}
 
@@ -587,24 +587,24 @@ 
 			$sections = $this->get_sections();
 
 			// Show section settings only if setting section exist.
-			if ( $this->current_section && ! in_array( $this->current_section, array_keys( $sections ) ) ) {
-				echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>';
+			if ($this->current_section && ! in_array($this->current_section, array_keys($sections))) {
+				echo '<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>';
 				$GLOBALS['give_hide_save_button'] = true;
 
 				return;
 			}
 
 			// Bailout.
-			if ( empty( $sections ) ) {
+			if (empty($sections)) {
 				return;
 			}
 
 			echo '<ul class="subsubsub">';
 
-			$array_keys = array_keys( $sections );
+			$array_keys = array_keys($sections);
 
-			foreach ( $sections as $id => $label ) {
-				echo '<li><a href="' . admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=' . $this->current_tab . '&section=' . sanitize_title( $id ) ) . '" class="' . ( $this->current_section == $id ? 'current' : '' ) . '">' . strip_tags( $label ) . '</a> ' . ( end( $array_keys ) == $id ? '' : '|' ) . ' </li>';
+			foreach ($sections as $id => $label) {
+				echo '<li><a href="'.admin_url('edit.php?post_type=give_forms&page=give-settings&tab='.$this->current_tab.'&section='.sanitize_title($id)).'" class="'.($this->current_section == $id ? 'current' : '').'">'.strip_tags($label).'</a> '.(end($array_keys) == $id ? '' : '|').' </li>';
 			}
 
 			echo '</ul><br class="clear" /><hr>';
@@ -619,7 +619,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -631,7 +631,7 @@ 
 		public function save() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::save_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::save_fields($settings, 'give_settings');
 		}
 	}
 endif;

includes/class-give-template-loader.php

Spacing +23 added lines, -23 removed lines

@@ -10,7 +10,7 @@ 
  */

 

 // Exit if accessed directly.

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

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

 	exit;

 }

 

@@ -36,29 +36,29 @@ 
 		/**

 		 * Templates

 		 */

-		add_filter( 'template_include', array( __CLASS__, 'template_loader' ) );

+		add_filter('template_include', array(__CLASS__, 'template_loader'));

 

 		/**

 		 * Content Wrappers

 		 */

-		add_action( 'give_before_main_content', 'give_output_content_wrapper', 10 );

-		add_action( 'give_after_main_content', 'give_output_content_wrapper_end', 10 );

+		add_action('give_before_main_content', 'give_output_content_wrapper', 10);

+		add_action('give_after_main_content', 'give_output_content_wrapper_end', 10);

 

 		/**

 		 * Entry Summary Classes

 		 */

-		add_filter( 'give_forms_single_summary_classes', array( $this, 'give_set_single_summary_classes' ) );

+		add_filter('give_forms_single_summary_classes', array($this, 'give_set_single_summary_classes'));

 

 		/**

 		 * Sidebar

 		 */

-		add_action( 'give_before_single_form_summary', array( $this, 'give_output_sidebar_option' ), 1 );

+		add_action('give_before_single_form_summary', array($this, 'give_output_sidebar_option'), 1);

 

 		/**

 		 * Single Forms Summary Box

 		 */

-		add_action( 'give_single_form_summary', 'give_template_single_title', 5 );

-		add_action( 'give_single_form_summary', 'give_get_donation_form', 10 );

+		add_action('give_single_form_summary', 'give_template_single_title', 5);

+		add_action('give_single_form_summary', 'give_get_donation_form', 10);

 

 	}

 

@@ -73,10 +73,10 @@ 
 	 *

 	 * @return string $classes List of space separated class names.

 	 */

-	public function give_set_single_summary_classes( $classes ) {

+	public function give_set_single_summary_classes($classes) {

 

 		//Add full width class when feature image is disabled AND no widgets are present

-		if ( ! give_is_setting_enabled( give_get_option( 'form_sidebar' ) ) ) {

+		if ( ! give_is_setting_enabled(give_get_option('form_sidebar'))) {

 			$classes .= ' give-full-width';

 		}

 

@@ -97,11 +97,11 @@ 
 	public function give_output_sidebar_option() {

 

 		//Add full width class when feature image is disabled AND no widgets are present

-		if ( give_is_setting_enabled( give_get_option( 'form_sidebar' ) ) ) {

-			add_action( 'give_before_single_form_summary', 'give_left_sidebar_pre_wrap', 5 );

-			add_action( 'give_before_single_form_summary', 'give_show_form_images', 10 );

-			add_action( 'give_before_single_form_summary', 'give_get_forms_sidebar', 20 );

-			add_action( 'give_before_single_form_summary', 'give_left_sidebar_post_wrap', 30 );

+		if (give_is_setting_enabled(give_get_option('form_sidebar'))) {

+			add_action('give_before_single_form_summary', 'give_left_sidebar_pre_wrap', 5);

+			add_action('give_before_single_form_summary', 'give_show_form_images', 10);

+			add_action('give_before_single_form_summary', 'give_get_forms_sidebar', 20);

+			add_action('give_before_single_form_summary', 'give_left_sidebar_post_wrap', 30);

 		}

 

 	}

@@ -123,20 +123,20 @@ 
 	 *

 	 * @return string $template

 	 */

-	public static function template_loader( $template ) {

-		$find = array( 'give.php' );

+	public static function template_loader($template) {

+		$find = array('give.php');

 		$file = '';

 

-		if ( is_single() && get_post_type() == 'give_forms' ) {

+		if (is_single() && get_post_type() == 'give_forms') {

 			$file   = 'single-give-form.php';

 			$find[] = $file;

-			$find[] = apply_filters( 'give_template_path', 'give/' ) . $file;

+			$find[] = apply_filters('give_template_path', 'give/').$file;

 		}

 

-		if ( $file ) {

-			$template = locate_template( array_unique( $find ) );

-			if ( ! $template ) {

-				$template = GIVE_PLUGIN_DIR . '/templates/' . $file;

+		if ($file) {

+			$template = locate_template(array_unique($find));

+			if ( ! $template) {

+				$template = GIVE_PLUGIN_DIR.'/templates/'.$file;

 			}

 		}

 

includes/admin/reporting/tools.php

Spacing +26 added lines, -26 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -22,7 +22,7 @@ 
  */
 function give_tools_recount_stats_display() {
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
+	if ( ! current_user_can('manage_give_settings')) {
 		return;
 	}
 
@@ -31,27 +31,27 @@ 
 	 *
 	 * @since 1.5
 	 */
-	do_action( 'give_tools_recount_stats_before' );
+	do_action('give_tools_recount_stats_before');
 	?>
 	<div id="poststuff">
 		<div class="postbox">
 
-			<h2 class="hndle ui-sortable-handle"><span><?php esc_html_e( 'Recount Stats', 'give' ); ?></span></h2>
+			<h2 class="hndle ui-sortable-handle"><span><?php esc_html_e('Recount Stats', 'give'); ?></span></h2>
 
 			<div class="inside recount-stats-controls">
-				<p><?php esc_html_e( 'Use these tools to recount stats, delete test transactions, or reset stats.', 'give' ); ?></p>
+				<p><?php esc_html_e('Use these tools to recount stats, delete test transactions, or reset stats.', 'give'); ?></p>
 				<form method="post" id="give-tools-recount-form" class="give-export-form">
 
-					<?php wp_nonce_field( 'give_ajax_export', 'give_ajax_export' ); ?>
+					<?php wp_nonce_field('give_ajax_export', 'give_ajax_export'); ?>
 
 					<select name="give-export-class" id="recount-stats-type">
-						<option value="0" selected="selected" disabled="disabled"><?php esc_html_e( 'Please select an option', 'give' ); ?></option>
-						<option data-type="recount-stats" value="Give_Tools_Recount_Income"><?php esc_html_e( 'Recalculate Total Donation Income Amount', 'give' ); ?></option>
-						<option data-type="recount-form" value="Give_Tools_Recount_Form_Stats"><?php esc_html_e( 'Recalculate Income Amount and Donation Counts for a Form', 'give' ); ?></option>
-						<option data-type="recount-all" value="Give_Tools_Recount_All_Stats"><?php esc_html_e( 'Recalculate Income Amount and Donation Counts for All Forms', 'give' ); ?></option>
-						<option data-type="recount-customer-stats" value="Give_Tools_Recount_Customer_Stats"><?php esc_html_e( 'Recalculate Donor Statistics', 'give' ); ?></option>
-						<option data-type="delete-test-transactions" value="Give_Tools_Delete_Test_Transactions"><?php esc_html_e( 'Delete Test Transactions', 'give' ); ?></option>
-						<option data-type="reset-stats" value="Give_Tools_Reset_Stats"><?php esc_html_e( 'Delete All Data', 'give' ); ?></option>
+						<option value="0" selected="selected" disabled="disabled"><?php esc_html_e('Please select an option', 'give'); ?></option>
+						<option data-type="recount-stats" value="Give_Tools_Recount_Income"><?php esc_html_e('Recalculate Total Donation Income Amount', 'give'); ?></option>
+						<option data-type="recount-form" value="Give_Tools_Recount_Form_Stats"><?php esc_html_e('Recalculate Income Amount and Donation Counts for a Form', 'give'); ?></option>
+						<option data-type="recount-all" value="Give_Tools_Recount_All_Stats"><?php esc_html_e('Recalculate Income Amount and Donation Counts for All Forms', 'give'); ?></option>
+						<option data-type="recount-customer-stats" value="Give_Tools_Recount_Customer_Stats"><?php esc_html_e('Recalculate Donor Statistics', 'give'); ?></option>
+						<option data-type="delete-test-transactions" value="Give_Tools_Delete_Test_Transactions"><?php esc_html_e('Delete Test Transactions', 'give'); ?></option>
+						<option data-type="reset-stats" value="Give_Tools_Reset_Stats"><?php esc_html_e('Delete All Data', 'give'); ?></option>
 						<?php
 						/**
 						 * Fires in the recount stats selectbox.
@@ -60,7 +60,7 @@ 
 						 *
 						 * @since 1.5
 						 */
-						do_action( 'give_recount_tool_options' );
+						do_action('give_recount_tool_options');
 						?>
 					</select>
 
@@ -68,22 +68,22 @@ 
 						<?php
 						$args = array(
 							'name'   => 'form_id',
-							'number' => - 1,
+							'number' => -1,
 							'chosen' => true,
 						);
-						echo Give()->html->forms_dropdown( $args );
+						echo Give()->html->forms_dropdown($args);
 						?>
 					</span>
 
-					<input type="submit" id="recount-stats-submit" value="<?php esc_attr_e( 'Submit', 'give' ); ?>" class="button-secondary"/>
+					<input type="submit" id="recount-stats-submit" value="<?php esc_attr_e('Submit', 'give'); ?>" class="button-secondary"/>
 
 					<br/>
 
 					<span class="give-recount-stats-descriptions">
-						<span id="recount-stats"><?php esc_html_e( 'Recalculates the overall donation income amount.', 'give' ); ?></span>
-						<span id="recount-form"><?php esc_html_e( 'Recalculates the donation and income stats for a specific form.', 'give' ); ?></span>
-						<span id="recount-all"><?php esc_html_e( 'Recalculates the earnings and sales stats for all forms.', 'give' ); ?></span>
-						<span id="recount-customer-stats"><?php esc_html_e( 'Recalculates the lifetime value and donation counts for all donors.', 'give' ); ?></span>
+						<span id="recount-stats"><?php esc_html_e('Recalculates the overall donation income amount.', 'give'); ?></span>
+						<span id="recount-form"><?php esc_html_e('Recalculates the donation and income stats for a specific form.', 'give'); ?></span>
+						<span id="recount-all"><?php esc_html_e('Recalculates the earnings and sales stats for all forms.', 'give'); ?></span>
+						<span id="recount-customer-stats"><?php esc_html_e('Recalculates the lifetime value and donation counts for all donors.', 'give'); ?></span>
 						<?php
 						/**
 						 * Fires in the recount stats description area.
@@ -92,10 +92,10 @@ 
 						 *
 						 * @since 1.5
 						 */
-						do_action( 'give_recount_tool_descriptions' );
+						do_action('give_recount_tool_descriptions');
 						?>
-						<span id="delete-test-transactions"><?php _e( '<strong>Deletes</strong> all TEST donations, donors, and related log entries.', 'give' ); ?></span>
-						<span id="reset-stats"><?php _e( '<strong>Deletes</strong> ALL donations, donors, and related log entries regardless of test or live mode.', 'give' ); ?></span>
+						<span id="delete-test-transactions"><?php _e('<strong>Deletes</strong> all TEST donations, donors, and related log entries.', 'give'); ?></span>
+						<span id="reset-stats"><?php _e('<strong>Deletes</strong> ALL donations, donors, and related log entries regardless of test or live mode.', 'give'); ?></span>
 					</span>
 
 					<span class="spinner"></span>
@@ -109,7 +109,7 @@ 
 				 *
 				 * @since 1.5
 				 */
-				do_action( 'give_tools_recount_forms' );
+				do_action('give_tools_recount_forms');
 				?>
 			</div><!-- .inside -->
 		</div><!-- .postbox -->
@@ -120,5 +120,5 @@ 
 	 *
 	 * @since 1.5
 	 */
-	do_action( 'give_tools_recount_stats_after' );
+	do_action('give_tools_recount_stats_after');
 }
\ No newline at end of file

includes/admin/tools/class-settings-api.php

Spacing +11 added lines, -11 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_API' ) ) :
+if ( ! class_exists('Give_Settings_API')) :
 
 	/**
 	 * Give_Settings_API.
@@ -43,10 +43,10 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'api';
-			$this->label = esc_html__( 'API', 'give' );
+			$this->label = esc_html__('API', 'give');
 
-			add_filter( 'give-tools_tabs_array', array( $this, 'add_settings_page' ), 20 );
-			add_action( "give-tools_settings_{$this->id}_page", array( $this, 'output' ) );
+			add_filter('give-tools_tabs_array', array($this, 'add_settings_page'), 20);
+			add_action("give-tools_settings_{$this->id}_page", array($this, 'output'));
 		}
 
 		/**
@@ -56,8 +56,8 @@ 
 		 * @param  array $pages Lst of pages.
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
-			$pages[ $this->id ] = $this->label;
+		public function add_settings_page($pages) {
+			$pages[$this->id] = $this->label;
 
 			return $pages;
 		}
@@ -73,7 +73,7 @@ 
 			$GLOBALS['give_hide_save_button'] = true;
 
 			// Get settings.
-			$settings = apply_filters( 'give_settings_system', array(
+			$settings = apply_filters('give_settings_system', array(
 				array(
 					'id'   => 'give_tools_api',
 					'type' => 'title',
@@ -81,7 +81,7 @@ 
 				),
 				array(
 					'id'   => 'api',
-					'name' => esc_html__( 'API', 'give' ),
+					'name' => esc_html__('API', 'give'),
 					'type' => 'api',
 				),
 				array(
@@ -97,7 +97,7 @@ 
 			 * @since  1.8
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -112,7 +112,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 	}
 

includes/admin/tools/class-settings-data.php

Spacing +14 added lines, -14 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Data' ) ) :
+if ( ! class_exists('Give_Settings_Data')) :
 
 	/**
 	 * Give_Settings_Data.
@@ -43,15 +43,15 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'data';
-			$this->label = esc_html__( 'Data', 'give' );
+			$this->label = esc_html__('Data', 'give');
 
-			add_filter( 'give-tools_tabs_array', array( $this, 'add_settings_page' ), 20 );
-			add_action( "give-tools_settings_{$this->id}_page", array( $this, 'output' ) );
+			add_filter('give-tools_tabs_array', array($this, 'add_settings_page'), 20);
+			add_action("give-tools_settings_{$this->id}_page", array($this, 'output'));
 
 			// Do not use main form for this tab.
-			if( give_get_current_setting_tab() === $this->id ) {
-				add_action( "give-tools_open_form", '__return_empty_string' );
-				add_action( "give-tools_close_form", '__return_empty_string' );
+			if (give_get_current_setting_tab() === $this->id) {
+				add_action("give-tools_open_form", '__return_empty_string');
+				add_action("give-tools_close_form", '__return_empty_string');
 			}
 		}
 
@@ -62,8 +62,8 @@ 
 		 * @param  array $pages Lst of pages.
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
-			$pages[ $this->id ] = $this->label;
+		public function add_settings_page($pages) {
+			$pages[$this->id] = $this->label;
 
 			return $pages;
 		}
@@ -79,7 +79,7 @@ 
 			$GLOBALS['give_hide_save_button'] = true;
 
 			// Get settings.
-			$settings = apply_filters( 'give_settings_system', array(
+			$settings = apply_filters('give_settings_system', array(
 				array(
 					'id'   => 'give_tools_tools',
 					'type' => 'title',
@@ -87,7 +87,7 @@ 
 				),
 				array(
 					'id'   => 'api',
-					'name' => esc_html__( 'Tools', 'give' ),
+					'name' => esc_html__('Tools', 'give'),
 					'type' => 'data',
 				),
 				array(
@@ -103,7 +103,7 @@ 
 			 * @since  1.8
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -118,7 +118,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 	}
 

includes/class-give-logging.php

Spacing +68 added lines, -68 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -43,15 +43,15 @@ 
 	 */
 	public function __setup_hooks() {
 		// Create the log post type
-		add_action( 'init', array( $this, 'register_post_type' ), 1 );
+		add_action('init', array($this, 'register_post_type'), 1);
 
 		// Create types taxonomy and default types
-		add_action( 'init', array( $this, 'register_taxonomy' ), 1 );
+		add_action('init', array($this, 'register_taxonomy'), 1);
 
-		add_action( 'save_post_give_payment', array( $this, 'background_process_delete_cache') );
-		add_action( 'save_post_give_forms', array( $this, 'background_process_delete_cache') );
-		add_action( 'save_post_give_log', array( $this, 'background_process_delete_cache') );
-		add_action( 'give_delete_log_cache', array( $this, 'delete_cache') );
+		add_action('save_post_give_payment', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_forms', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_log', array($this, 'background_process_delete_cache'));
+		add_action('give_delete_log_cache', array($this, 'delete_cache'));
 	}
 
 	/**
@@ -67,7 +67,7 @@ 
 	public function register_post_type() {
 		/* Logs post type */
 		$log_args = array(
-			'labels'              => array( 'name' => esc_html__( 'Logs', 'give' ) ),
+			'labels'              => array('name' => esc_html__('Logs', 'give')),
 			'public'              => false,
 			'exclude_from_search' => true,
 			'publicly_queryable'  => false,
@@ -75,11 +75,11 @@ 
 			'query_var'           => false,
 			'rewrite'             => false,
 			'capability_type'     => 'post',
-			'supports'            => array( 'title', 'editor' ),
+			'supports'            => array('title', 'editor'),
 			'can_export'          => true,
 		);
 
-		register_post_type( 'give_log', $log_args );
+		register_post_type('give_log', $log_args);
 	}
 
 	/**
@@ -93,7 +93,7 @@ 
 	 * @return void
 	 */
 	public function register_taxonomy() {
-		register_taxonomy( 'give_log_type', 'give_log', array( 'public' => false ) );
+		register_taxonomy('give_log_type', 'give_log', array('public' => false));
 	}
 
 	/**
@@ -113,7 +113,7 @@ 
 			'api_request',
 		);
 
-		return apply_filters( 'give_log_types', $terms );
+		return apply_filters('give_log_types', $terms);
 	}
 
 	/**
@@ -128,8 +128,8 @@ 
 	 *
 	 * @return bool         Whether log type is valid.
 	 */
-	public function valid_type( $type ) {
-		return in_array( $type, $this->log_types() );
+	public function valid_type($type) {
+		return in_array($type, $this->log_types());
 	}
 
 	/**
@@ -148,7 +148,7 @@ 
 	 *
 	 * @return int             Log ID.
 	 */
-	public function add( $title = '', $message = '', $parent = 0, $type = null ) {
+	public function add($title = '', $message = '', $parent = 0, $type = null) {
 		$log_data = array(
 			'post_title'   => $title,
 			'post_content' => $message,
@@ -156,7 +156,7 @@ 
 			'log_type'     => $type,
 		);
 
-		return $this->insert_log( $log_data );
+		return $this->insert_log($log_data);
 	}
 
 	/**
@@ -173,12 +173,12 @@ 
 	 *
 	 * @return array             An array of the connected logs.
 	 */
-	public function get_logs( $object_id = 0, $type = null, $paged = null ) {
-		return $this->get_connected_logs( array(
+	public function get_logs($object_id = 0, $type = null, $paged = null) {
+		return $this->get_connected_logs(array(
 			'post_parent' => $object_id,
 			'paged'       => $paged,
 			'log_type'    => $type,
-		) );
+		));
 	}
 
 	/**
@@ -192,7 +192,7 @@ 
 	 *
 	 * @return int             The ID of the newly created log item.
 	 */
-	public function insert_log( $log_data = array(), $log_meta = array() ) {
+	public function insert_log($log_data = array(), $log_meta = array()) {
 		$defaults = array(
 			'post_type'    => 'give_log',
 			'post_status'  => 'publish',
@@ -201,7 +201,7 @@ 
 			'log_type'     => false,
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
 		/**
 		 * Fires before inserting log entry.
@@ -211,20 +211,20 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_pre_insert_log', $log_data, $log_meta );
+		do_action('give_pre_insert_log', $log_data, $log_meta);
 
 		// Store the log entry
-		$log_id = wp_insert_post( $args );
+		$log_id = wp_insert_post($args);
 
 		// Set the log type, if any
-		if ( $log_data['log_type'] && $this->valid_type( $log_data['log_type'] ) ) {
-			wp_set_object_terms( $log_id, $log_data['log_type'], 'give_log_type', false );
+		if ($log_data['log_type'] && $this->valid_type($log_data['log_type'])) {
+			wp_set_object_terms($log_id, $log_data['log_type'], 'give_log_type', false);
 		}
 
 		// Set log meta, if any
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				update_post_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				update_post_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 			}
 		}
 
@@ -237,7 +237,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_post_insert_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_insert_log', $log_id, $log_data, $log_meta);
 
 		return $log_id;
 	}
@@ -253,7 +253,7 @@ 
 	 *
 	 * @return bool|null       True if successful, false otherwise.
 	 */
-	public function update_log( $log_data = array(), $log_meta = array() ) {
+	public function update_log($log_data = array(), $log_meta = array()) {
 
 		/**
 		 * Fires before updating log entry.
@@ -263,7 +263,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_pre_update_log', $log_data, $log_meta );
+		do_action('give_pre_update_log', $log_data, $log_meta);
 
 		$defaults = array(
 			'post_type'   => 'give_log',
@@ -271,15 +271,15 @@ 
 			'post_parent' => 0,
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
 		// Store the log entry
-		$log_id = wp_update_post( $args );
+		$log_id = wp_update_post($args);
 
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				if ( ! empty( $meta ) ) {
-					update_post_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				if ( ! empty($meta)) {
+					update_post_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 				}
 			}
 		}
@@ -293,7 +293,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_post_update_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_update_log', $log_id, $log_data, $log_meta);
 	}
 
 	/**
@@ -308,19 +308,19 @@ 
 	 *
 	 * @return array|false Array if logs were found, false otherwise.
 	 */
-	public function get_connected_logs( $args = array() ) {
+	public function get_connected_logs($args = array()) {
 
 		$defaults = array(
 			'post_type'      => 'give_log',
 			'posts_per_page' => 20,
 			'post_status'    => 'publish',
-			'paged'          => get_query_var( 'paged' ),
+			'paged'          => get_query_var('paged'),
 			'log_type'       => false,
 		);
 
-		$query_args = wp_parse_args( $args, $defaults );
+		$query_args = wp_parse_args($args, $defaults);
 
-		if ( $query_args['log_type'] && $this->valid_type( $query_args['log_type'] ) ) {
+		if ($query_args['log_type'] && $this->valid_type($query_args['log_type'])) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -330,9 +330,9 @@ 
 			);
 		}
 
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 
-		if ( $logs ) {
+		if ($logs) {
 			return $logs;
 		}
 
@@ -355,17 +355,17 @@ 
 	 *
 	 * @return int                Log count.
 	 */
-	public function get_log_count( $object_id = 0, $type = null, $meta_query = null, $date_query = null ) {
+	public function get_log_count($object_id = 0, $type = null, $meta_query = null, $date_query = null) {
 
 		$query_args = array(
 			'post_parent'    => $object_id,
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids',
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -375,25 +375,25 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		if ( ! empty( $date_query ) ) {
+		if ( ! empty($date_query)) {
 			$query_args['date_query'] = $date_query;
 		}
 
 
 		// Get cache key for current query.
-		$cache_key =  give_get_cache_key( 'get_log_count', $query_args );
+		$cache_key = give_get_cache_key('get_log_count', $query_args);
 
 		// check if cache already exist or not.
-		if( ! ( $logs_count = get_option( $cache_key ) ) ) {
-			$logs = new WP_Query( $query_args );
+		if ( ! ($logs_count = get_option($cache_key))) {
+			$logs = new WP_Query($query_args);
 			$logs_count = (int) $logs->post_count;
 
 			// Cache results.
-			add_option( $cache_key, $logs_count, '', 'no' );
+			add_option($cache_key, $logs_count, '', 'no');
 		}
 
 		return $logs_count;
@@ -413,16 +413,16 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_logs( $object_id = 0, $type = null, $meta_query = null ) {
+	public function delete_logs($object_id = 0, $type = null, $meta_query = null) {
 		$query_args = array(
 			'post_parent'    => $object_id,
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids',
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -432,15 +432,15 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
-				wp_delete_post( $log, true );
+		if ($logs) {
+			foreach ($logs as $log) {
+				wp_delete_post($log, true);
 			}
 		}
 	}
@@ -453,8 +453,8 @@ 
 	 *
 	 * @param int $post_id
 	 */
-	public function background_process_delete_cache( $post_id ) {
-		wp_schedule_single_event( time(), 'give_delete_log_cache' );
+	public function background_process_delete_cache($post_id) {
+		wp_schedule_single_event(time(), 'give_delete_log_cache');
 	}
 
 	/**
@@ -476,14 +476,14 @@ 
 		);
 
 		// Bailout.
-		if( empty( $cache_option_names ) ) {
+		if (empty($cache_option_names)) {
 			return false;
 		}
 
 
 		// Delete log cache.
-		foreach ( $cache_option_names as $option_name ) {
-			delete_option( $option_name['option_name'] );
+		foreach ($cache_option_names as $option_name) {
+			delete_option($option_name['option_name']);
 		}
 	}
 }
@@ -506,10 +506,10 @@ 
  *
  * @return int             ID of the new log entry.
  */
-function give_record_log( $title = '', $message = '', $parent = 0, $type = null ) {
+function give_record_log($title = '', $message = '', $parent = 0, $type = null) {
 	/* @var Give_Logging $give_logs */
 	global $give_logs;
-	$log = $give_logs->add( $title, $message, $parent, $type );
+	$log = $give_logs->add($title, $message, $parent, $type);
 
 	return $log;
 }

Doc Comments +1 added lines, -1 removed lines

@@ -463,7 +463,7 @@
 	 * @since  1.7
 	 * @access public
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function delete_cache() {
 		global $wpdb;

includes/class-give-email-access.php

Spacing +62 added lines, -62 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -96,7 +96,7 @@ 
 	public function __construct() {
 
 		// get it started
-		add_action( 'init', array( $this, 'init' ) );
+		add_action('init', array($this, 'init'));
 	}
 
 	/**
@@ -111,30 +111,30 @@ 
 	 */
 	public function init() {
 
-		$is_enabled = give_get_option( 'email_access' );
+		$is_enabled = give_get_option('email_access');
 
 		//Non-logged in users only
-		if ( is_user_logged_in() || $is_enabled !== 'on' || is_admin() ) {
+		if (is_user_logged_in() || $is_enabled !== 'on' || is_admin()) {
 			return;
 		}
 
 		//Are db columns setup?
-		$is_setup = give_get_option( 'email_access_installed' );
-		if ( empty( $is_setup ) ) {
+		$is_setup = give_get_option('email_access_installed');
+		if (empty($is_setup)) {
 			$this->create_columns();
 		}
 
 		// Timeouts
-		$this->verify_throttle  = apply_filters( 'give_nl_verify_throttle', 300 );
-		$this->token_expiration = apply_filters( 'give_nl_token_expiration', 7200 );
+		$this->verify_throttle  = apply_filters('give_nl_verify_throttle', 300);
+		$this->token_expiration = apply_filters('give_nl_token_expiration', 7200);
 
 		// Setup login
 		$this->check_for_token();
 
-		if ( $this->token_exists ) {
-			add_filter( 'give_can_view_receipt', '__return_true' );
-			add_filter( 'give_user_pending_verification', '__return_false' );
-			add_filter( 'give_get_users_donations_args', array( $this, 'users_donations_args' ) );
+		if ($this->token_exists) {
+			add_filter('give_can_view_receipt', '__return_true');
+			add_filter('give_user_pending_verification', '__return_false');
+			add_filter('give_get_users_donations_args', array($this, 'users_donations_args'));
 		}
 	}
 
@@ -148,25 +148,25 @@ 
 	 *
 	 * @return bool
 	 */
-	public function can_send_email( $customer_id ) {
+	public function can_send_email($customer_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Prevent multiple emails within X minutes
-		$throttle = date( 'Y-m-d H:i:s', time() - $this->verify_throttle );
+		$throttle = date('Y-m-d H:i:s', time() - $this->verify_throttle);
 
 		// Does a user row exist?
 		$exists = (int) $wpdb->get_var(
-			$wpdb->prepare( "SELECT COUNT(*) FROM {$wpdb->prefix}give_customers WHERE id = %d", $customer_id )
+			$wpdb->prepare("SELECT COUNT(*) FROM {$wpdb->prefix}give_customers WHERE id = %d", $customer_id)
 		);
 
-		if ( 0 < $exists ) {
+		if (0 < $exists) {
 			$row_id = (int) $wpdb->get_var(
-				$wpdb->prepare( "SELECT id FROM {$wpdb->prefix}give_customers WHERE id = %d AND (verify_throttle < %s OR verify_key = '') LIMIT 1", $customer_id, $throttle )
+				$wpdb->prepare("SELECT id FROM {$wpdb->prefix}give_customers WHERE id = %d AND (verify_throttle < %s OR verify_key = '') LIMIT 1", $customer_id, $throttle)
 			);
 
-			if ( $row_id < 1 ) {
-				give_set_error( 'give_email_access_attempts_exhausted', esc_html__( 'Please wait a few minutes before requesting a new email access link.', 'give' ) );
+			if ($row_id < 1) {
+				give_set_error('give_email_access_attempts_exhausted', esc_html__('Please wait a few minutes before requesting a new email access link.', 'give'));
 
 				return false;
 			}
@@ -186,34 +186,34 @@ 
 	 *
 	 * @return void
 	 */
-	public function send_email( $customer_id, $email ) {
+	public function send_email($customer_id, $email) {
 
-		$verify_key = wp_generate_password( 20, false );
+		$verify_key = wp_generate_password(20, false);
 
 		// Generate a new verify key
-		$this->set_verify_key( $customer_id, $email, $verify_key );
+		$this->set_verify_key($customer_id, $email, $verify_key);
 
 		// Get the donation history page
-		$page_id = give_get_option( 'history_page' );
+		$page_id = give_get_option('history_page');
 
-		$access_url = add_query_arg( array(
+		$access_url = add_query_arg(array(
 			'give_nl' => $verify_key,
-		), get_permalink( $page_id ) );
+		), get_permalink($page_id));
 
 		//Nice subject and message
-		$subject = apply_filters( 'give_email_access_token_subject', sprintf( esc_html__( 'Your Access Link to %s', 'give' ), get_bloginfo( 'name' ) ) );
+		$subject = apply_filters('give_email_access_token_subject', sprintf(esc_html__('Your Access Link to %s', 'give'), get_bloginfo('name')));
 
-		$message = esc_html__( 'You or someone in your organization requested an access link be sent to this email address. This is a temporary access link for you to view your donation information. Click on the link below to view:', 'give' ) . "\n\n";
-		$message .= '<a href="' . esc_url( $access_url ) . '" target="_blank">' . esc_html__( 'Access My Donation Details &raquo;', 'give' ) . '</a>' . "\n\n";
+		$message = esc_html__('You or someone in your organization requested an access link be sent to this email address. This is a temporary access link for you to view your donation information. Click on the link below to view:', 'give')."\n\n";
+		$message .= '<a href="'.esc_url($access_url).'" target="_blank">'.esc_html__('Access My Donation Details &raquo;', 'give').'</a>'."\n\n";
 		$message .= "\n\n";
-		$message .= esc_html__( 'Sincerely,', 'give' ) . "\n";
-		$message .= get_bloginfo( 'name' ) . "\n";
+		$message .= esc_html__('Sincerely,', 'give')."\n";
+		$message .= get_bloginfo('name')."\n";
 
-		$message = apply_filters( 'give_email_access_token_message', $message );
+		$message = apply_filters('give_email_access_token_message', $message);
 
 		// Send the email
-		Give()->emails->__set( 'heading', apply_filters( 'give_email_access_token_heading', esc_html__( 'Your Access Link', 'give' ) ) );
-		Give()->emails->send( $email, $subject, $message );
+		Give()->emails->__set('heading', apply_filters('give_email_access_token_heading', esc_html__('Your Access Link', 'give')));
+		Give()->emails->send($email, $subject, $message);
 
 	}
 
@@ -227,24 +227,24 @@ 
 	 */
 	public function check_for_token() {
 
-		$token = isset( $_GET['give_nl'] ) ? $_GET['give_nl'] : '';
+		$token = isset($_GET['give_nl']) ? $_GET['give_nl'] : '';
 
 		// Check for cookie
-		if ( empty( $token ) ) {
-			$token = isset( $_COOKIE['give_nl'] ) ? $_COOKIE['give_nl'] : '';
+		if (empty($token)) {
+			$token = isset($_COOKIE['give_nl']) ? $_COOKIE['give_nl'] : '';
 		}
 
-		if ( ! empty( $token ) ) {
-			if ( ! $this->is_valid_token( $token ) ) {
-				if ( ! $this->is_valid_verify_key( $token ) ) {
+		if ( ! empty($token)) {
+			if ( ! $this->is_valid_token($token)) {
+				if ( ! $this->is_valid_verify_key($token)) {
 					return;
 				}
 			}
 
 			$this->token_exists = true;
 			// Set cookie
-			$lifetime = current_time( 'timestamp' ) + Give()->session->set_expiration_time();
-			@setcookie( 'give_nl', $token, $lifetime, COOKIEPATH, COOKIE_DOMAIN, false );
+			$lifetime = current_time('timestamp') + Give()->session->set_expiration_time();
+			@setcookie('give_nl', $token, $lifetime, COOKIEPATH, COOKIE_DOMAIN, false);
 		}
 	}
 
@@ -258,18 +258,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function is_valid_token( $token ) {
+	public function is_valid_token($token) {
 
 		global $wpdb;
 
 		// Make sure token isn't expired
-		$expires = date( 'Y-m-d H:i:s', time() - $this->token_expiration );
+		$expires = date('Y-m-d H:i:s', time() - $this->token_expiration);
 
 		$email = $wpdb->get_var(
-			$wpdb->prepare( "SELECT email FROM {$wpdb->prefix}give_customers WHERE token = %s AND verify_throttle >= %s LIMIT 1", $token, $expires )
+			$wpdb->prepare("SELECT email FROM {$wpdb->prefix}give_customers WHERE token = %s AND verify_throttle >= %s LIMIT 1", $token, $expires)
 		);
 
-		if ( ! empty( $email ) ) {
+		if ( ! empty($email)) {
 			$this->token_email = $email;
 			$this->token       = $token;
 
@@ -277,8 +277,8 @@ 
 		}
 
 		//Set error only if email access form isn't being submitted
-		if ( ! isset( $_POST['give_email'] ) && ! isset( $_POST['_wpnonce'] ) ) {
-			give_set_error( 'give_email_token_expired', apply_filters( 'give_email_token_expired_message', esc_html__( 'Your access token has expired. Please request a new one below:', 'give' ) ) );
+		if ( ! isset($_POST['give_email']) && ! isset($_POST['_wpnonce'])) {
+			give_set_error('give_email_token_expired', apply_filters('give_email_token_expired_message', esc_html__('Your access token has expired. Please request a new one below:', 'give')));
 		}
 
 		return false;
@@ -297,25 +297,25 @@ 
 	 *
 	 * @return void
 	 */
-	public function set_verify_key( $customer_id, $email, $verify_key ) {
+	public function set_verify_key($customer_id, $email, $verify_key) {
 		global $wpdb;
 
-		$now = date( 'Y-m-d H:i:s' );
+		$now = date('Y-m-d H:i:s');
 
 		// Insert or update?
 		$row_id = (int) $wpdb->get_var(
-			$wpdb->prepare( "SELECT id FROM {$wpdb->prefix}give_customers WHERE id = %d LIMIT 1", $customer_id )
+			$wpdb->prepare("SELECT id FROM {$wpdb->prefix}give_customers WHERE id = %d LIMIT 1", $customer_id)
 		);
 
 		// Update
-		if ( ! empty( $row_id ) ) {
+		if ( ! empty($row_id)) {
 			$wpdb->query(
-				$wpdb->prepare( "UPDATE {$wpdb->prefix}give_customers SET verify_key = %s, verify_throttle = %s WHERE id = %d LIMIT 1", $verify_key, $now, $row_id )
+				$wpdb->prepare("UPDATE {$wpdb->prefix}give_customers SET verify_key = %s, verify_throttle = %s WHERE id = %d LIMIT 1", $verify_key, $now, $row_id)
 			);
 		} // Insert
 		else {
 			$wpdb->query(
-				$wpdb->prepare( "INSERT INTO {$wpdb->prefix}give_customers ( verify_key, verify_throttle) VALUES (%s, %s)", $verify_key, $now )
+				$wpdb->prepare("INSERT INTO {$wpdb->prefix}give_customers ( verify_key, verify_throttle) VALUES (%s, %s)", $verify_key, $now)
 			);
 		}
 	}
@@ -330,21 +330,21 @@ 
 	 *
 	 * @return bool
 	 */
-	public function is_valid_verify_key( $token ) {
+	public function is_valid_verify_key($token) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// See if the verify_key exists
 		$row = $wpdb->get_row(
-			$wpdb->prepare( "SELECT id, email FROM {$wpdb->prefix}give_customers WHERE verify_key = %s LIMIT 1", $token )
+			$wpdb->prepare("SELECT id, email FROM {$wpdb->prefix}give_customers WHERE verify_key = %s LIMIT 1", $token)
 		);
 
-		$now = date( 'Y-m-d H:i:s' );
+		$now = date('Y-m-d H:i:s');
 
 		// Set token
-		if ( ! empty( $row ) ) {
+		if ( ! empty($row)) {
 			$wpdb->query(
-				$wpdb->prepare( "UPDATE {$wpdb->prefix}give_customers SET verify_key = '', token = %s, verify_throttle = %s WHERE id = %d LIMIT 1", $token, $now, $row->id )
+				$wpdb->prepare("UPDATE {$wpdb->prefix}give_customers SET verify_key = '', token = %s, verify_throttle = %s WHERE id = %d LIMIT 1", $token, $now, $row->id)
 			);
 
 			$this->token_email = $row->email;
@@ -368,7 +368,7 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function users_donations_args( $args ) {
+	public function users_donations_args($args) {
 		$args['user'] = $this->token_email;
 
 		return $args;
@@ -389,11 +389,11 @@ 
 		global $wpdb;
 
 		//Create columns in customers table
-		$query = $wpdb->query( "ALTER TABLE {$wpdb->prefix}give_customers ADD `token` VARCHAR(255) CHARACTER SET utf8 NOT NULL, ADD `verify_key` VARCHAR(255) CHARACTER SET utf8 NOT NULL AFTER `token`, ADD `verify_throttle` DATETIME NOT NULL AFTER `verify_key`" );
+		$query = $wpdb->query("ALTER TABLE {$wpdb->prefix}give_customers ADD `token` VARCHAR(255) CHARACTER SET utf8 NOT NULL, ADD `verify_key` VARCHAR(255) CHARACTER SET utf8 NOT NULL AFTER `token`, ADD `verify_throttle` DATETIME NOT NULL AFTER `verify_key`");
 
 		//Columns added properly
-		if ( $query ) {
-			give_update_option( 'email_access_installed', 1 );
+		if ($query) {
+			give_update_option('email_access_installed', 1);
 		}
 
 	}

includes/price-functions.php

Spacing +19 added lines, -19 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,13 +23,13 @@ 
  *
  * @return bool true if has variable prices, false otherwise
  */
-function give_has_variable_prices( $form_id = 0 ) {
+function give_has_variable_prices($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->has_variable_prices();
 }
@@ -44,13 +44,13 @@ 
  *
  * @return array Variable prices
  */
-function give_get_variable_prices( $form_id = 0 ) {
+function give_get_variable_prices($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->prices;
 
@@ -67,13 +67,13 @@ 
  *
  * @return string $default_price
  */
-function give_get_default_multilevel_amount( $form_id ) {
+function give_get_default_multilevel_amount($form_id) {
 	$default_price = '1.00';
-	$prices        = apply_filters( 'give_form_variable_prices', give_get_variable_prices( $form_id ), $form_id );
+	$prices        = apply_filters('give_form_variable_prices', give_get_variable_prices($form_id), $form_id);
 
-	foreach ( $prices as $price ) {
+	foreach ($prices as $price) {
 
-		if ( isset( $price['_give_default'] ) && $price['_give_default'] === 'default' ) {
+		if (isset($price['_give_default']) && $price['_give_default'] === 'default') {
 			$default_price = $price['_give_amount'];
 		}
 
@@ -94,19 +94,19 @@ 
  * @return string $default_price
  * @since      1.0
  */
-function give_get_default_form_amount( $form_id ) {
+function give_get_default_form_amount($form_id) {
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$default_amount = give_get_default_multilevel_amount( $form_id );
+		$default_amount = give_get_default_multilevel_amount($form_id);
 
 	} else {
 
-		$default_amount = get_post_meta( $form_id, '_give_set_price', true );
+		$default_amount = get_post_meta($form_id, '_give_set_price', true);
 
 	}
 
-	return apply_filters( 'give_default_form_amount', $default_amount );
+	return apply_filters('give_default_form_amount', $default_amount);
 
 }
 
@@ -124,13 +124,13 @@ 
  *
  * @return bool
  */
-function give_is_custom_price_mode( $form_id = 0 ) {
+function give_is_custom_price_mode($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->is_custom_price_mode();
 }
\ No newline at end of file

includes/class-give-roles.php

Spacing +58 added lines, -58 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,7 +35,7 @@ 
 	 * @access public
 	 */
 	public function __construct() {
-		add_filter( 'give_map_meta_cap', array( $this, 'meta_caps' ), 10, 4 );
+		add_filter('give_map_meta_cap', array($this, 'meta_caps'), 10, 4);
 	}
 
 	/**
@@ -49,7 +49,7 @@ 
 	 * @return void
 	 */
 	public function add_roles() {
-		add_role( 'give_manager', esc_html__( 'Give Manager', 'give' ), array(
+		add_role('give_manager', esc_html__('Give Manager', 'give'), array(
 			'read'                   => true,
 			'edit_posts'             => true,
 			'delete_posts'           => true,
@@ -78,20 +78,20 @@ 
 			'publish_posts'          => true,
 			'read_private_pages'     => true,
 			'read_private_posts'     => true
-		) );
+		));
 
-		add_role( 'give_accountant', esc_html__( 'Give Accountant', 'give' ), array(
+		add_role('give_accountant', esc_html__('Give Accountant', 'give'), array(
 			'read'         => true,
 			'edit_posts'   => false,
 			'delete_posts' => false
-		) );
+		));
 
-		add_role( 'give_worker', esc_html__( 'Give Worker', 'give' ), array(
+		add_role('give_worker', esc_html__('Give Worker', 'give'), array(
 			'read'         => true,
 			'edit_posts'   => false,
 			'upload_files' => true,
 			'delete_posts' => false
-		) );
+		));
 
 	}
 
@@ -110,38 +110,38 @@ 
 	public function add_caps() {
 		global $wp_roles;
 
-		if ( class_exists( 'WP_Roles' ) ) {
-			if ( ! isset( $wp_roles ) ) {
+		if (class_exists('WP_Roles')) {
+			if ( ! isset($wp_roles)) {
 				$wp_roles = new WP_Roles();
 			}
 		}
 
-		if ( is_object( $wp_roles ) ) {
-			$wp_roles->add_cap( 'give_manager', 'view_give_reports' );
-			$wp_roles->add_cap( 'give_manager', 'view_give_sensitive_data' );
-			$wp_roles->add_cap( 'give_manager', 'export_give_reports' );
-			$wp_roles->add_cap( 'give_manager', 'manage_give_settings' );
+		if (is_object($wp_roles)) {
+			$wp_roles->add_cap('give_manager', 'view_give_reports');
+			$wp_roles->add_cap('give_manager', 'view_give_sensitive_data');
+			$wp_roles->add_cap('give_manager', 'export_give_reports');
+			$wp_roles->add_cap('give_manager', 'manage_give_settings');
 
-			$wp_roles->add_cap( 'administrator', 'view_give_reports' );
-			$wp_roles->add_cap( 'administrator', 'view_give_sensitive_data' );
-			$wp_roles->add_cap( 'administrator', 'export_give_reports' );
-			$wp_roles->add_cap( 'administrator', 'manage_give_settings' );
+			$wp_roles->add_cap('administrator', 'view_give_reports');
+			$wp_roles->add_cap('administrator', 'view_give_sensitive_data');
+			$wp_roles->add_cap('administrator', 'export_give_reports');
+			$wp_roles->add_cap('administrator', 'manage_give_settings');
 
 			// Add the main post type capabilities.
 			$capabilities = $this->get_core_caps();
-			foreach ( $capabilities as $cap_group ) {
-				foreach ( $cap_group as $cap ) {
-					$wp_roles->add_cap( 'administrator', $cap );
-					$wp_roles->add_cap( 'give_manager', $cap );
-					$wp_roles->add_cap( 'give_worker', $cap );
+			foreach ($capabilities as $cap_group) {
+				foreach ($cap_group as $cap) {
+					$wp_roles->add_cap('administrator', $cap);
+					$wp_roles->add_cap('give_manager', $cap);
+					$wp_roles->add_cap('give_worker', $cap);
 				}
 			}
 
-			$wp_roles->add_cap( 'give_accountant', 'edit_give_forms' );
-			$wp_roles->add_cap( 'give_accountant', 'read_private_give_forms' );
-			$wp_roles->add_cap( 'give_accountant', 'view_give_reports' );
-			$wp_roles->add_cap( 'give_accountant', 'export_give_reports' );
-			$wp_roles->add_cap( 'give_accountant', 'edit_give_payments' );
+			$wp_roles->add_cap('give_accountant', 'edit_give_forms');
+			$wp_roles->add_cap('give_accountant', 'read_private_give_forms');
+			$wp_roles->add_cap('give_accountant', 'view_give_reports');
+			$wp_roles->add_cap('give_accountant', 'export_give_reports');
+			$wp_roles->add_cap('give_accountant', 'edit_give_payments');
 
 		}
 	}
@@ -159,10 +159,10 @@ 
 	public function get_core_caps() {
 		$capabilities = array();
 
-		$capability_types = array( 'give_form', 'give_payment' );
+		$capability_types = array('give_form', 'give_payment');
 
-		foreach ( $capability_types as $capability_type ) {
-			$capabilities[ $capability_type ] = array(
+		foreach ($capability_types as $capability_type) {
+			$capabilities[$capability_type] = array(
 				// Post type.
 				"edit_{$capability_type}",
 				"read_{$capability_type}",
@@ -208,22 +208,22 @@ 
 	 *
 	 * @return array  $caps    Meta capabilities.
 	 */
-	public function meta_caps( $caps, $cap, $user_id, $args ) {
+	public function meta_caps($caps, $cap, $user_id, $args) {
 
-		switch ( $cap ) {
+		switch ($cap) {
 
 			case 'view_give_form_stats' :
 
-				if ( empty( $args[0] ) ) {
+				if (empty($args[0])) {
 					break;
 				}
 
-				$form = get_post( $args[0] );
-				if ( empty( $form ) ) {
+				$form = get_post($args[0]);
+				if (empty($form)) {
 					break;
 				}
 
-				if ( user_can( $user_id, 'view_give_reports' ) || $user_id == $form->post_author ) {
+				if (user_can($user_id, 'view_give_reports') || $user_id == $form->post_author) {
 					$caps = array();
 				}
 
@@ -250,41 +250,41 @@ 
 
 		global $wp_roles;
 
-		if ( class_exists( 'WP_Roles' ) ) {
-			if ( ! isset( $wp_roles ) ) {
+		if (class_exists('WP_Roles')) {
+			if ( ! isset($wp_roles)) {
 				$wp_roles = new WP_Roles();
 			}
 		}
 
-		if ( is_object( $wp_roles ) ) {
+		if (is_object($wp_roles)) {
 			// Give Manager Capabilities.
-			$wp_roles->remove_cap( 'give_manager', 'view_give_reports' );
-			$wp_roles->remove_cap( 'give_manager', 'view_give_sensitive_data' );
-			$wp_roles->remove_cap( 'give_manager', 'export_give_reports' );
-			$wp_roles->remove_cap( 'give_manager', 'manage_give_settings' );
+			$wp_roles->remove_cap('give_manager', 'view_give_reports');
+			$wp_roles->remove_cap('give_manager', 'view_give_sensitive_data');
+			$wp_roles->remove_cap('give_manager', 'export_give_reports');
+			$wp_roles->remove_cap('give_manager', 'manage_give_settings');
 
 			// Site Administrator Capabilities.
-			$wp_roles->remove_cap( 'administrator', 'view_give_reports' );
-			$wp_roles->remove_cap( 'administrator', 'view_give_sensitive_data' );
-			$wp_roles->remove_cap( 'administrator', 'export_give_reports' );
-			$wp_roles->remove_cap( 'administrator', 'manage_give_settings' );
+			$wp_roles->remove_cap('administrator', 'view_give_reports');
+			$wp_roles->remove_cap('administrator', 'view_give_sensitive_data');
+			$wp_roles->remove_cap('administrator', 'export_give_reports');
+			$wp_roles->remove_cap('administrator', 'manage_give_settings');
 
 			// Remove the Main Post Type Capabilities.
 			$capabilities = $this->get_core_caps();
 
-			foreach ( $capabilities as $cap_group ) {
-				foreach ( $cap_group as $cap ) {
-					$wp_roles->remove_cap( 'give_manager', $cap );
-					$wp_roles->remove_cap( 'administrator', $cap );
-					$wp_roles->remove_cap( 'give_worker', $cap );
+			foreach ($capabilities as $cap_group) {
+				foreach ($cap_group as $cap) {
+					$wp_roles->remove_cap('give_manager', $cap);
+					$wp_roles->remove_cap('administrator', $cap);
+					$wp_roles->remove_cap('give_worker', $cap);
 				}
 			}
 
 			/** Give Accountant Capabilities */
-			$wp_roles->remove_cap( 'give_accountant', 'edit_give_forms' );
-			$wp_roles->remove_cap( 'give_accountant', 'read_private_give_forms' );
-			$wp_roles->remove_cap( 'give_accountant', 'view_give_reports' );
-			$wp_roles->remove_cap( 'give_accountant', 'export_give_reports' );
+			$wp_roles->remove_cap('give_accountant', 'edit_give_forms');
+			$wp_roles->remove_cap('give_accountant', 'read_private_give_forms');
+			$wp_roles->remove_cap('give_accountant', 'view_give_reports');
+			$wp_roles->remove_cap('give_accountant', 'export_give_reports');
 
 		}
 	}