oldtownmedia / test-content-suite

abstracts/type.php

Spacing +4 added lines, -4 removed lines

@@ -10,7 +10,7 @@ 
  * @subpackage Test Content
  * @author     Old Town Media
  */
-abstract class Type{
+abstract class Type {
 
 	/**
 	 * type
@@ -22,13 +22,13 @@ 
 	protected $type;
 
 
-	public function register_type(){
+	public function register_type() {
 
-		add_action( 'tc_types', 'set_type' );
+		add_action('tc_types', 'set_type');
 
 	}
 
-	public function set_type( $types ){
+	public function set_type($types) {
 
 		$types[] = $this->type;
 

includes/class-admin-page.php

Spacing +47 added lines, -47 removed lines

@@ -8,7 +8,7 @@ 
  * @subpackage Evans
  * @author     Old Town Media
  */
-class AdminPage{
+class AdminPage {
 
 	/**
 	 * plugin
@@ -45,16 +45,16 @@ 
 	 *
 	 * @see admin_menu, wp_ajax actions
 	 */
-	public function hooks(){
+	public function hooks() {
 
 		$connection = new ConnectionTest;
-		$this->definitions	= $this->plugin->get_definitions();
-		$this->connected	= $connection->test();
+		$this->definitions = $this->plugin->get_definitions();
+		$this->connected = $connection->test();
 
-		add_action( 'admin_menu' , array( $this, 'add_menu_item' ) );
-		add_action( 'wp_ajax_handle_test_data', array( $this, 'handle_test_data_callback' ) );
-		add_filter( 'plugin_action_links_' . $this->definitions->basename , array( $this, 'add_settings_link' ) );
-		add_action( 'admin_notices', array( $this, 'internet_connected_admin_notice' ) );
+		add_action('admin_menu', array($this, 'add_menu_item'));
+		add_action('wp_ajax_handle_test_data', array($this, 'handle_test_data_callback'));
+		add_filter('plugin_action_links_'.$this->definitions->basename, array($this, 'add_settings_link'));
+		add_action('admin_notices', array($this, 'internet_connected_admin_notice'));
 
 	}
 
@@ -64,7 +64,7 @@ 
 	 *
 	 * @param Plugin $plugin Main plugin instance.
 	 */
-	public function set_plugin( $plugin ) {
+	public function set_plugin($plugin) {
 
 		$this->plugin = $plugin;
 		return $this;
@@ -81,14 +81,14 @@ 
 
 		$page = add_submenu_page(
 			'tools.php',
-			__( 'Create Test Content', 'otm-test-content' ),
-			__( 'Test Content', 'otm-test-content' ),
+			__('Create Test Content', 'otm-test-content'),
+			__('Test Content', 'otm-test-content'),
 			'manage_options',
 			'create-test-data',
-			array( $this, 'admin_page' )
+			array($this, 'admin_page')
 		);
 
-		add_action( 'admin_print_styles-' . $page, array( $this, 'load_scripts' ) );
+		add_action('admin_print_styles-'.$page, array($this, 'load_scripts'));
 
 	}
 
@@ -98,10 +98,10 @@ 
 	 * @param  array $links Existing links
 	 * @return array 		Modified links
 	 */
-	public function add_settings_link( $links ) {
+	public function add_settings_link($links) {
 
-		$settings_link = '<a href="tools.php?page=create-test-data">' . __( 'Build Test Content', 'otm-test-content' ) . '</a>';
-  		array_push( $links, $settings_link );
+		$settings_link = '<a href="tools.php?page=create-test-data">'.__('Build Test Content', 'otm-test-content').'</a>';
+  		array_push($links, $settings_link);
   		return $links;
 
 	}
@@ -114,24 +114,24 @@ 
 	 * Internet, and the test fails displays a notice informing the user that
 	 * images will not pull into test data.
 	 */
-	public function internet_connected_admin_notice(){
+	public function internet_connected_admin_notice() {
 
 		// Get the current admin screen & verify that we're on the right one
 		// before continuing.
 		$screen = get_current_screen();
 
-		if ( $screen->base != 'tools_page_create-test-data' ){
+		if ($screen->base != 'tools_page_create-test-data') {
 			return;
 		}
 
 		// Check the response
-		if ( $this->connected ) {
+		if ($this->connected) {
 			// We got a response so early return
 			return;
 		} else {
 			// We didn't get a reponse so print the notice out
 			echo '<div class="notice notice-error">';
-		        echo '<p>'.__( 'WordPress could not connect to Splashbase and therefore images will not pull into metaboxes/thumbnails. Turn Airplane Mode off or reconnect to the Internet to get images when creating test data.', 'otm-test-content' ).'</p>';
+		        echo '<p>'.__('WordPress could not connect to Splashbase and therefore images will not pull into metaboxes/thumbnails. Turn Airplane Mode off or reconnect to the Internet to get images when creating test data.', 'otm-test-content').'</p>';
 		    echo '</div>';
 		}
 
@@ -141,19 +141,19 @@ 
 	/**
 	 * Load our script in the admin section and serve in data.
 	 */
-	public function load_scripts(){
+	public function load_scripts() {
 
-		wp_enqueue_script( 'test-content-js', plugins_url( 'assets/admin.js' , dirname( __FILE__ ) ) );
-		wp_enqueue_style( 'test-content-css', plugins_url( 'assets/admin.css' , dirname( __FILE__ ) ) );
+		wp_enqueue_script('test-content-js', plugins_url('assets/admin.js', dirname(__FILE__)));
+		wp_enqueue_style('test-content-css', plugins_url('assets/admin.css', dirname(__FILE__)));
 
 		$data = array(
-			'nonce'			=> wp_create_nonce( 'handle-test-data' ),
-			'createdStr'	=> __( 'Created', 'otm-test-content' ),
-			'deletedStr'	=> __( 'Deleting', 'otm-test-content' ),
-			'creatingStr'	=> __( 'Creating', 'otm-test-content' ),
+			'nonce'			=> wp_create_nonce('handle-test-data'),
+			'createdStr'	=> __('Created', 'otm-test-content'),
+			'deletedStr'	=> __('Deleting', 'otm-test-content'),
+			'creatingStr'	=> __('Creating', 'otm-test-content'),
 		);
 
-		wp_localize_script( 'test-content-js', 'test_content', $data );
+		wp_localize_script('test-content-js', 'test_content', $data);
 
 	}
 
@@ -165,21 +165,21 @@ 
 	 */
 	public function handle_test_data_callback() {
 
-		$action		= $_REQUEST['todo'];
-		$nonce		= $_REQUEST['nonce'];
+		$action = $_REQUEST['todo'];
+		$nonce = $_REQUEST['nonce'];
 
 		// Verify that we have a proper logged in user and it's the right person
-		if ( empty( $nonce ) || ! wp_verify_nonce( $nonce, 'handle-test-data' ) ){
+		if (empty($nonce) || !wp_verify_nonce($nonce, 'handle-test-data')) {
 			return;
 		}
 
-		if ( $action == 'delete' ){
+		if ($action == 'delete') {
 
-			$this->deletion_routing( $_REQUEST );
+			$this->deletion_routing($_REQUEST);
 
-		} elseif ( $action == 'create' ){
+		} elseif ($action == 'create') {
 
-			$this->creation_routing( $_REQUEST );
+			$this->creation_routing($_REQUEST);
 
 		}
 
@@ -192,11 +192,11 @@ 
 	 * Choose which type of creation needs to be accomplished and route through
 	 * the correct class.
 	 */
-	private function creation_routing( $data ){
+	private function creation_routing($data) {
 
-		$type = 'testContent\Types\\' . ucwords( $data['type'] );
+		$type = 'testContent\Types\\'.ucwords($data['type']);
 		$object = new $type();
-		$object->create_objects( $data['slug'], $data['connection'], true, 1 );
+		$object->create_objects($data['slug'], $data['connection'], true, 1);
 
 	}
 
@@ -205,17 +205,17 @@ 
 	 * Choose which type of deletion needs to be accomplished and route through
 	 * the correct method of Delete.
 	 */
-	private function deletion_routing( $data ){
+	private function deletion_routing($data) {
 
 		$delete_content = new Delete;
 
-		if ( $data['type'] == 'all' ){
+		if ($data['type'] == 'all') {
 
-			$delete_content->delete_all_test_data( true );
+			$delete_content->delete_all_test_data(true);
 
 		} else {
 
-			$delete_content->delete_objects( true, $data );
+			$delete_content->delete_objects(true, $data);
 
 		}
 
@@ -225,20 +225,20 @@ 
 	/**
 	 * Print out our admin page to control test data.
 	 */
-	public function admin_page(){
-		echo '<div class="wrap" id="options_editor">' . "\n";
+	public function admin_page() {
+		echo '<div class="wrap" id="options_editor">'."\n";
 
-			echo '<h2>' . __( 'Create Test Data' , 'otm-test-content' ) . '</h2>' . "\n";
+			echo '<h2>'.__('Create Test Data', 'otm-test-content').'</h2>'."\n";
 
 			echo "<div class='nav-tab-wrapper'>";
 
-				do_action( 'tc-admin-tabs', '' );
+				do_action('tc-admin-tabs', '');
 
 			echo "</div>";
 
 			echo "";
 
-				do_action( 'tc-admin-sections', '' );
+				do_action('tc-admin-sections', '');
 
 			echo "";
 

includes/class-delete.php

Spacing +16 added lines, -16 removed lines

@@ -8,32 +8,32 @@ 
  * @subpackage Evans
  * @author     Old Town Media
  */
-class Delete{
+class Delete {
 
 	/**
 	 * Delete all test content created ever.
 	 *
 	 * @access private
 	 */
-	public function delete_all_test_data( $echo = false ){
+	public function delete_all_test_data($echo = false) {
 
-		if ( !$this->user_can_delete() ){
+		if (!$this->user_can_delete()) {
 			return;
 		}
 
 		// Loop through all post types and remove any test data
-		$post_types = get_post_types( array( 'public' => true ), 'objects' );
-		foreach ( $post_types as $post_type ) :
+		$post_types = get_post_types(array('public' => true), 'objects');
+		foreach ($post_types as $post_type) :
 
-		    $this->delete_posts( $post_type->name, $echo );
+		    $this->delete_posts($post_type->name, $echo);
 
 		endforeach;
 
 		// Loop through all taxonomies and remove any data
 		$taxonomies = get_taxonomies();
-		foreach ( $taxonomies as $tax ) :
+		foreach ($taxonomies as $tax) :
 
-		    $this->delete_terms( $tax, $echo );
+		    $this->delete_terms($tax, $echo);
 
 		endforeach;
 
@@ -51,22 +51,22 @@ 
 	 * @param string $slug a custom post type ID.
 	 * @param boolean $echo Whether or not to echo the result
 	 */
-	public function delete_objects( $echo = false, $data ){
+	public function delete_objects($echo = false, $data) {
 
 		// Make sure that the current user is logged in & has full permissions.
-		if ( !$this->user_can_delete() ){
+		if (!$this->user_can_delete()) {
 			return;
 		}
 
-		if ( empty( $data ) ){
+		if (empty($data)) {
 			return;
 		}
 
-		$type = 'testContent\Types\\' . ucwords( $data['type'] );
+		$type = 'testContent\Types\\'.ucwords($data['type']);
 		$slug = $data['slug'];
 
 		$object = new $type();
-		$object->delete( $slug, $echo );
+		$object->delete($slug, $echo);
 
 	}
 
@@ -76,15 +76,15 @@ 
 	 *
 	 * @see is_user_logged_in, current_user_can
 	 */
-	public function user_can_delete(){
+	public function user_can_delete() {
 
 		// User must be logged in
-		if ( !is_user_logged_in() ){
+		if (!is_user_logged_in()) {
 			return false;
 		}
 
 		// User must have editor priveledges, at a minimum
-		if ( !current_user_can( 'delete_others_posts' ) ){
+		if (!current_user_can('delete_others_posts')) {
 			return false;
 		}
 

includes/class-metaboxes.php

Spacing +80 added lines, -80 removed lines

@@ -8,7 +8,7 @@ 
  * @subpackage Evans
  * @author     Old Town Media
  */
-class Metaboxes{
+class Metaboxes {
 
 	/**
 	 * Decide which cmb library to try and loop to get our metaboxes.
@@ -22,31 +22,31 @@ 
 	 * @param string $slug Post Type slug ID.
 	 * @return array Fields to fill in for our post object.
 	 */
-	public function get_metaboxes( $slug ){
+	public function get_metaboxes($slug) {
 		$cmb2_fields = $cmb_fields = $acf_fields = array();
 
 		// CMB2
-		if ( class_exists( 'CMB2', false ) ) {
-			$cmb2_fields = $this->get_cmb2_metaboxes( $slug );
+		if (class_exists('CMB2', false)) {
+			$cmb2_fields = $this->get_cmb2_metaboxes($slug);
 		}
 
 		// Custom Metaboxes and Fields (CMB1)
-		if ( class_exists( 'cmb_Meta_Box', false ) ) {
-			$cmb_fields = $this->get_cmb1_metaboxes( $slug );
+		if (class_exists('cmb_Meta_Box', false)) {
+			$cmb_fields = $this->get_cmb1_metaboxes($slug);
 		}
 
 		// Advanced Custom Fields (ACF Free)
-		if ( class_exists( 'acf', false ) ) {
-			$acf_fields = $this->get_acf_free_metaboxes( $slug );
+		if (class_exists('acf', false)) {
+			$acf_fields = $this->get_acf_free_metaboxes($slug);
 		}
 
 		// Return our array
-		return array_merge( $cmb2_fields, $cmb_fields, $acf_fields );
+		return array_merge($cmb2_fields, $cmb_fields, $acf_fields);
 
 	}
 
 
-	private function get_acf_free_metaboxes( $slug ){
+	private function get_acf_free_metaboxes($slug) {
 
 		$fields = array();
 
@@ -54,21 +54,21 @@ 
 		$fieldsets = $this->get_all_acf_field_groups();
 
 		// Return empty array if there are no fieldsets at all
-		if ( empty( $fieldsets ) ){
+		if (empty($fieldsets)) {
 			return $fields;
 		}
 
 		// Loop through each fieldset for possible matches
-		foreach ( $fieldsets as $fieldset ){
+		foreach ($fieldsets as $fieldset) {
 
-			if ( $this->is_acf_field_in_post_type( $slug, $fieldset ) ){
+			if ($this->is_acf_field_in_post_type($slug, $fieldset)) {
 
 				// If this is the first group of fields, simply set the value
 				// Else, merge this group with the previous one
-				if ( empty( $fields ) ){
+				if (empty($fields)) {
 					$fields = $fieldset->fields;
 				} else {
-					$fields = array_merge( $fields, $fieldset->fields );
+					$fields = array_merge($fields, $fieldset->fields);
 				}
 
 			}
@@ -80,16 +80,16 @@ 
 	}
 
 
-	private function is_acf_field_in_post_type( $slug, $fieldset ){
+	private function is_acf_field_in_post_type($slug, $fieldset) {
 
 		// Make sure we have something to parse
-		if ( empty( $fieldset ) ){
+		if (empty($fieldset)) {
 			return false;
 		}
 
 		// Loop through the rules to check for post type matches
-		foreach ( $fieldset->rules as $rule ){
-			if ( $rule['param'] === 'post_type' && $rule['value'] === $slug ){
+		foreach ($fieldset->rules as $rule) {
+			if ($rule['param'] === 'post_type' && $rule['value'] === $slug) {
 				return true;
 			}
 		}
@@ -100,7 +100,7 @@ 
 	}
 
 
-	private function get_all_acf_field_groups(){
+	private function get_all_acf_field_groups() {
 		$info = $rules = $fields = array();
 
 		$args = array(
@@ -108,20 +108,20 @@ 
 			'posts_per_page'=> 500
 		);
 
-		$objects = new \WP_Query( $args );
+		$objects = new \WP_Query($args);
 
-		if ( $objects->have_posts() ) :
-			while ( $objects->have_posts() ) : $objects->the_post();
+		if ($objects->have_posts()) :
+			while ($objects->have_posts()) : $objects->the_post();
 
-				$data = get_metadata( 'post', get_the_id() );
+				$data = get_metadata('post', get_the_id());
 
-				foreach ( $data['rule'] as $rule ){
-					$rules[] = unserialize( $rule );
+				foreach ($data['rule'] as $rule) {
+					$rules[] = unserialize($rule);
 				}
 
-				foreach ( $data as $key => $value ){
-					if ( substr( $key, 0, 6 ) == 'field_' ) :
-						$field_detail = unserialize( $value[0] );
+				foreach ($data as $key => $value) {
+					if (substr($key, 0, 6) == 'field_') :
+						$field_detail = unserialize($value[0]);
 						$fields[] = array(
 							'key'	 => $field_detail['key'],
 							'type'	 => $field_detail['type'],
@@ -159,55 +159,55 @@ 
 	 * @param string $slug a custom post type ID.
 	 * @return array Array of fields.
 	 */
-	private function get_cmb2_metaboxes( $slug ){
+	private function get_cmb2_metaboxes($slug) {
 
 		$fields = array();
 
 		// Get all metaboxes from CMB2 library
-		$all_metaboxes = apply_filters( 'cmb2_meta_boxes', array() );
+		$all_metaboxes = apply_filters('cmb2_meta_boxes', array());
 
 		// Loop through all possible sets of metaboxes added the old way
-		foreach ( $all_metaboxes as $metabox_array ){
+		foreach ($all_metaboxes as $metabox_array) {
 
 			// If the custom post type ID matches this set of fields, set & stop
-			if ( in_array( $slug, $metabox_array['object_types'] ) ) {
+			if (in_array($slug, $metabox_array['object_types'])) {
 
 				// If this is the first group of fields, simply set the value
 				// Else, merge this group with the previous one
-				if ( empty( $fields ) ){
+				if (empty($fields)) {
 					$fields = $metabox_array['fields'];
 				} else {
-					$fields = array_merge( $fields, $metabox_array['fields'] );
+					$fields = array_merge($fields, $metabox_array['fields']);
 				}
 			}
 
 		}
 
 		// Loop through all metaboxes added the new way
-		foreach ( \CMB2_Boxes::get_all() as $cmb ) {
+		foreach (\CMB2_Boxes::get_all() as $cmb) {
 
 			// Create the default
 			$match = false;
 
 			// Establish correct cmb types
-			if ( is_string( $cmb->meta_box['object_types'] ) ){
-				if ( $cmb->meta_box['object_types'] == $slug ){
+			if (is_string($cmb->meta_box['object_types'])) {
+				if ($cmb->meta_box['object_types'] == $slug) {
 					$match = true;
 				}
 			} else {
-				if ( in_array( $slug, $cmb->meta_box['object_types'] ) ){
+				if (in_array($slug, $cmb->meta_box['object_types'])) {
 					$match = true;
 				}
 			}
 
-			if ( $match !== true ){
+			if ($match !== true) {
 				continue;
 			}
 
-			if ( empty( $fields ) ){
+			if (empty($fields)) {
 				$fields = $cmb->meta_box['fields'];
 			} else {
-				$fields = array_merge( $fields, $cmb->meta_box['fields'] );
+				$fields = array_merge($fields, $cmb->meta_box['fields']);
 			}
 
 		}
@@ -231,25 +231,25 @@ 
 	 * @param string $slug a custom post type ID.
 	 * @return array Array of fields.
 	 */
-	private function get_cmb1_metaboxes( $slug ){
+	private function get_cmb1_metaboxes($slug) {
 
 		$fields = array();
 
 		// Get all metaboxes from CMB2 library
-		$all_metaboxes = apply_filters( 'cmb_meta_boxes', array() );
+		$all_metaboxes = apply_filters('cmb_meta_boxes', array());
 
 		// Loop through all possible sets of metaboxes
-		foreach ( $all_metaboxes as $metabox_array ){
+		foreach ($all_metaboxes as $metabox_array) {
 
 			// If the custom post type ID matches this set of fields, set & stop
-			if ( in_array( $slug, $metabox_array['pages'] ) ) {
+			if (in_array($slug, $metabox_array['pages'])) {
 
 				// If this is the first group of fields, simply set the value
 				// Else, merge this group with the previous one
-				if ( empty( $fields ) ){
+				if (empty($fields)) {
 					$fields = $metabox_array['fields'];
 				} else {
-					$fields = array_merge( $fields, $metabox_array['fields'] );
+					$fields = array_merge($fields, $metabox_array['fields']);
 				}
 			}
 
@@ -272,36 +272,36 @@ 
 	 * @param int $post_id Single post ID.
 	 * @param array $cmb custom metabox array from CMB2.
 	 */
-	public function random_metabox_content( $post_id, $cmb, $connected ){
+	public function random_metabox_content($post_id, $cmb, $connected) {
 		$value = '';
 
 		// First check that our post ID & cmb array aren't empty
-		if ( empty( $cmb ) || empty( $post_id ) ){
+		if (empty($cmb) || empty($post_id)) {
 			return;
 		}
 
 		// Fetch the appropriate type of data and return
-		switch( $cmb['type'] ){
+		switch ($cmb['type']) {
 
 			case 'text':
 			case 'text_small':
 			case 'text_medium':
 
 				// If phone is in the id, fetch a phone #
-				if ( stripos( $cmb['id'], 'phone' ) ){
+				if (stripos($cmb['id'], 'phone')) {
 					$value = TestContent::phone();
 
 				// If email is in the id, fetch an email address
-				} elseif ( stripos( $cmb['id'], 'email' ) ){
+				} elseif (stripos($cmb['id'], 'email')) {
 					$value = TestContent::email();
 
 				// If time is in the id, fetch a time string
-				} elseif ( stripos( $cmb['id'], 'time' ) ){
+				} elseif (stripos($cmb['id'], 'time')) {
 					$value = TestContent::time();
 
 				// Otherwise, just a random text string
 				} else {
-					$value = TestContent::title( rand( 10, 50 ) );
+					$value = TestContent::title(rand(10, 50));
 				}
 
 				break;
@@ -332,14 +332,14 @@ 
 
 			case 'text_date':
 
-				$value = TestContent::date( 'm/d/Y' );
+				$value = TestContent::date('m/d/Y');
 
 				break;
 
 			case 'text_date_timestamp':
 			case 'text_datetime_timestamp':
 
-				$value = TestContent::date( 'U' );
+				$value = TestContent::date('U');
 
 				break;
 
@@ -347,13 +347,13 @@ 
 
 			case 'text_money':
 
-				$value = rand( 0, 100000 );
+				$value = rand(0, 100000);
 
 				break;
 
 			case 'test_colorpicker':
 
-				$value = '#' . str_pad( dechex( mt_rand( 0, 0xFFFFFF ) ), 6, '0', STR_PAD_LEFT );
+				$value = '#'.str_pad(dechex(mt_rand(0, 0xFFFFFF)), 6, '0', STR_PAD_LEFT);
 
 				break;
 
@@ -370,8 +370,8 @@ 
 			case 'radio':
 
 				// Grab a random item out of the array and return the key
-				$new_val = array_slice( $cmb['options'], rand( 0, count( $cmb['options'] ) ), 1 );
-				$value = key( $new_val );
+				$new_val = array_slice($cmb['options'], rand(0, count($cmb['options'])), 1);
+				$value = key($new_val);
 
 				break;
 
@@ -382,7 +382,7 @@ 
 			case 'checkbox':
 
 				// 50/50 odds of being turned on
-				if ( rand( 0, 1 ) == 1 ){
+				if (rand(0, 1) == 1) {
 					$value = 'on';
 				}
 
@@ -393,10 +393,10 @@ 
 				$new_option = array();
 
 				// Loop through each of our options
-				foreach ( $cmb['options'] as $key => $value ){
+				foreach ($cmb['options'] as $key => $value) {
 
 					// 50/50 chance of being included
-					if ( rand( 0, 1 ) ){
+					if (rand(0, 1)) {
 						$new_option[] = $key;
 					}
 
@@ -414,8 +414,8 @@ 
 
 			case 'file':
 
-				if ( true == $connected ){
-					$value = TestContent::image( $post_id );
+				if (true == $connected) {
+					$value = TestContent::image($post_id);
 				}
 
 				break;
@@ -431,38 +431,38 @@ 
 		}
 
 		// Value must exist to attempt to insert
-		if ( ! empty( $value ) && ! is_wp_error( $value ) ){
+		if (!empty($value) && !is_wp_error($value)) {
 
-			$this->update_meta( $post_id, $value, $cmb );
+			$this->update_meta($post_id, $value, $cmb);
 
 		// If we're dealing with a WP Error object, just return the message for debugging
-		} elseif ( is_wp_error( $value ) ){
-			error_log( $value->get_error_message() );
+		} elseif (is_wp_error($value)) {
+			error_log($value->get_error_message());
 			return $value->get_error_message();
 		}
 
 	} // end random_metabox_content
 
 
-	private function update_meta( $post_id, $value, $cmb ){
+	private function update_meta($post_id, $value, $cmb) {
 
 		$type 	= $cmb['type'];
-		$id		= $cmb['id'];
-		$value = apply_filters( "tc_{$type}_metabox", $value );	// Filter by metabox type
-		$value = apply_filters( "tc_{$id}_metabox", $value ); // Filter by metabox ID
+		$id = $cmb['id'];
+		$value = apply_filters("tc_{$type}_metabox", $value); // Filter by metabox type
+		$value = apply_filters("tc_{$id}_metabox", $value); // Filter by metabox ID
 
 		// Files must be treated separately - they use the attachment ID
 		// & url of media for separate cmb values.
-		if ( $cmb['type'] != 'file' ){
-			add_post_meta( $post_id, $cmb['id'], $value, true );
+		if ($cmb['type'] != 'file') {
+			add_post_meta($post_id, $cmb['id'], $value, true);
 		} else {
-			add_post_meta( $post_id, $cmb['id'].'_id', $value, true );
-			add_post_meta( $post_id, $cmb['id'], wp_get_attachment_url( $value ), true );
+			add_post_meta($post_id, $cmb['id'].'_id', $value, true);
+			add_post_meta($post_id, $cmb['id'], wp_get_attachment_url($value), true);
 		}
 
 		// Add extra, redundant meta. Because, why not have rows for the price of one?
-		if ( $cmb['source'] === 'acf' ){
-			add_post_meta( $post_id, '_' . $cmb['id'], $cmb['key'], true );
+		if ($cmb['source'] === 'acf') {
+			add_post_meta($post_id, '_'.$cmb['id'], $cmb['key'], true);
 		}
 
 	}

abstracts/view.php

Spacing +19 added lines, -19 removed lines

@@ -10,7 +10,7 @@ 
  * @subpackage Test Content
  * @author     Old Town Media
  */
-abstract class View{
+abstract class View {
 
 	/**
 	 * title
@@ -44,10 +44,10 @@ 
 	 *
 	 * @see tab, view
 	 */
-	public function register_view(){
+	public function register_view() {
 
-		add_action( 'tc-admin-tabs', array( $this, 'tab' ), $this->priority );
-		add_action( 'tc-admin-sections', array( $this, 'view' ), $this->priority );
+		add_action('tc-admin-tabs', array($this, 'tab'), $this->priority);
+		add_action('tc-admin-sections', array($this, 'view'), $this->priority);
 
 	}
 
@@ -58,10 +58,10 @@ 
 	 * Each view has a tab and tab navigation - this function compiles our
 	 * navigation tab. Rarely extended.
 	 */
-	public function tab(){
+	public function tab() {
 		$html = "";
 
-		$html .= "<a class='nav-tab' data-type='".sanitize_title( $this->title )."' href='javascript:void(0)'>";
+		$html .= "<a class='nav-tab' data-type='".sanitize_title($this->title)."' href='javascript:void(0)'>";
 			$html .= $this->title;
 		$html .= "</a>";
 
@@ -78,10 +78,10 @@ 
 	 *
 	 * @see actions_section, options_section
 	 */
-	public function view(){
+	public function view() {
 		$html = '';
 
-		$html .= "<section class='test-content-tab' data-type='".sanitize_title( $this->title )."'>";
+		$html .= "<section class='test-content-tab' data-type='".sanitize_title($this->title)."'>";
 			$html .= $this->actions_section();
 			$html .= $this->options_section();
 		$html .= "</section>";
@@ -98,7 +98,7 @@ 
 	 *
 	 * @return string HTML content.
 	 */
-	protected function actions_section(){
+	protected function actions_section() {
 		$html = '';
 		return $html;
 	}
@@ -115,13 +115,13 @@ 
 	 * @param string $html Existing HTML content.
 	 * @return string HTML section content.
 	 */
-	protected function options_section( $html = '' ){
+	protected function options_section($html = '') {
 		$html .= "<hr>";
 
 		$html .= "<div class='test-data-cpt'>";
 			$html .= "<h3>";
-				$html .= "<span class='label'>".__( 'Quantity', 'otm-test-content' )."</span>";
-				$html .= "<input type='number' value='0' class='quantity-adjustment' for='".$this->type."' placeholder='".__( '', 'otm-test-content' )."'> ";
+				$html .= "<span class='label'>".__('Quantity', 'otm-test-content')."</span>";
+				$html .= "<input type='number' value='0' class='quantity-adjustment' for='".$this->type."' placeholder='".__('', 'otm-test-content')."'> ";
 			$html .= "</h3>";
 		$html .= "</div>";
 
@@ -139,22 +139,22 @@ 
 	 * @param string $text Text to display in the button.
 	 * @return string HTML.
 	 */
-	protected function build_button( $action, $slug, $text ){
+	protected function build_button($action, $slug, $text) {
 		$html = $dashicon = '';
 
-		if ( $action == 'create' ){
+		if ($action == 'create') {
 			$dashicon = 'dashicons-plus';
-		} elseif ( $action == 'delete' ){
+		} elseif ($action == 'delete') {
 			$dashicon = 'dashicons-trash';
 		}
 
 		$html .= "<a href='javascript:void(0);' ";
-			$html .= " data-type='" . $this->type . "'";
-			$html .= " data-slug='" . $slug . "'";
-			$html .= " data-todo='" . $action . "'";
+			$html .= " data-type='".$this->type."'";
+			$html .= " data-slug='".$slug."'";
+			$html .= " data-todo='".$action."'";
 			$html .= " class='button-primary handle-test-data'";
 		$html .= "/>";
-			$html .= "<span class='dashicons " . $dashicon . "'></span>";
+			$html .= "<span class='dashicons ".$dashicon."'></span>";
 			$html .= $text;
 		$html .= "</a>";