oldtownmedia / test-content-suite

includes/class-admin-page.php

Indentation +2 added lines, -2 removed lines

@@ -89,8 +89,8 @@
 		} else {
 			// We didn't get a reponse so print the notice out
 			echo '<div class="notice notice-error is-dismissible">';
-		        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>';
+				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>';
 		}
 
 	}

Spacing +52 added lines, -52 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,17 +192,17 @@ 
 	 * 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) {
 
-		if ( $data['type'] == 'post' ){
+		if ($data['type'] == 'post') {
 
 			$create_content = new CreatePost;
-			$create_content->create_post_type_content( $data['slug'], $data['connection'], true, 1 );
+			$create_content->create_post_type_content($data['slug'], $data['connection'], true, 1);
 
-		} elseif( $data['type'] == 'term' ){
+		} elseif ($data['type'] == 'term') {
 
 			$create_content = new CreateTerm;
-			$create_content->create_terms( $data['slug'], true, 1 );
+			$create_content->create_terms($data['slug'], true, 1);
 
 		}
 
@@ -213,21 +213,21 @@ 
 	 * 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'] == 'post' ){
+		if ($data['type'] == 'post') {
 
-			$delete_content->delete_posts( $data['slug'], true );
+			$delete_content->delete_posts($data['slug'], true);
 
-		} elseif ( $data['type'] == 'term' ){
+		} elseif ($data['type'] == 'term') {
 
-			$delete_content->delete_terms( $data['slug'], true );
+			$delete_content->delete_terms($data['slug'], true);
 
-		} elseif ( $data['type'] == 'all' ){
+		} elseif ($data['type'] == 'all') {
 
-			$delete_content->delete_all_test_data( true );
+			$delete_content->delete_all_test_data(true);
 
 		}
 
@@ -237,20 +237,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-plugin.php

Indentation +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
 	 * @see load_plugin_textdomain
 	 */
 	public function load_textdomain() {
-	    load_plugin_textdomain( 'otm-test-content', FALSE, basename( dirname( $this->definitions->file ) ) . '/languages/' );
+		load_plugin_textdomain( 'otm-test-content', FALSE, basename( dirname( $this->definitions->file ) ) . '/languages/' );
 	}
 
 }

Spacing +7 added lines, -7 removed lines

@@ -8,7 +8,7 @@ 
  * @subpackage Evans
  * @author     Old Town Media
  */
-class Plugin{
+class Plugin {
 
 	/**
 	 * Plugin definitions.
@@ -34,7 +34,7 @@ 
 	 * @param  object $definitions Information about the plugin
 	 * @return object $this
 	 */
-	public function set_definitions( $definitions ) {
+	public function set_definitions($definitions) {
 
 		$this->definitions = $definitions;
 		return $this;
@@ -48,10 +48,10 @@ 
 	 * @param  object $provider Hook provider.
 	 * @return object $this
 	 */
-	public function register_hooks( $provider ) {
+	public function register_hooks($provider) {
 
-		if ( method_exists( $provider, 'set_plugin' ) ) {
-			$provider->set_plugin( $this );
+		if (method_exists($provider, 'set_plugin')) {
+			$provider->set_plugin($this);
 		}
 
 		$provider->hooks();
@@ -66,7 +66,7 @@ 
 	 * @param  object $provider Hook provider.
 	 * @return object $this
 	 */
-	public function register_views( $provider ) {
+	public function register_views($provider) {
 
 		$provider->register_view();
 		return $this;
@@ -80,7 +80,7 @@ 
 	 * @see load_plugin_textdomain
 	 */
 	public function load_textdomain() {
-	    load_plugin_textdomain( 'otm-test-content', FALSE, basename( dirname( $this->definitions->file ) ) . '/languages/' );
+	    load_plugin_textdomain('otm-test-content', FALSE, basename(dirname($this->definitions->file)).'/languages/');
 	}
 
 }

includes/class-connection-test.php

Spacing +23 added lines, -23 removed lines

@@ -17,7 +17,7 @@ 
  * @subpackage Evans
  * @author     Old Town Media
  */
-class ConnectionTest{
+class ConnectionTest {
 
 	/**
 	 * Run all of our connection tests.
@@ -26,33 +26,33 @@ 
 	 *
 	 * @return boolean Status of connection to Internet/Splashbase.
 	 */
-	public function test(){
+	public function test() {
 
 		/*
 		 * Make sure that we're looking at the correct admin page
 		 */
-		if ( ! $this->check_admin_page() ){
+		if (!$this->check_admin_page()) {
 			return;
 		}
 
 		/*
 		 * Test #1 - Check for Airplane Mode plugin status
 		 */
-		if ( ! $this->check_airplane_mode() ){
+		if (!$this->check_airplane_mode()) {
 			return false;
 		}
 
 		/*
 		 * Test #2 - Check Internet connection in general
 		 */
-		if ( ! $this->check_internet() ){
+		if (!$this->check_internet()) {
 			return false;
 		}
 
 		/*
 		 * Test #3 - Check External URL itself (Splashbase here)
 		 */
-		if ( ! $this->check_external_url( 'http://www.splashbase.co/api/v1/images/' ) ){
+		if (!$this->check_external_url('http://www.splashbase.co/api/v1/images/')) {
 			return false;
 		}
 
@@ -75,22 +75,22 @@ 
 	 *
 	 * @return boolean Whether or not we're in the right place.
 	 */
-	private function check_admin_page(){
+	private function check_admin_page() {
 		global $current_screen;
 
 		// Only run if we're in the admin page
-		if ( !is_admin() ){
+		if (!is_admin()) {
 			return false;
 		}
 
 		// Get the current admin screen & verify that we're on the right one
 		// before continuing.
-		if ( isset ( $current_screen ) && 'tools_page_create-test-data' != $current_screen->base ){
+		if (isset ($current_screen) && 'tools_page_create-test-data' != $current_screen->base) {
 			return false;
 		}
 
-		$last_uri_bit = explode( '=', $_SERVER['REQUEST_URI'] );
-		if ( 'create-test-data' != end( $last_uri_bit ) ){
+		$last_uri_bit = explode('=', $_SERVER['REQUEST_URI']);
+		if ('create-test-data' != end($last_uri_bit)) {
 			return false;
 		}
 
@@ -108,13 +108,13 @@ 
 	 *
 	 * @return boolean Connected or not.
 	 */
-	private function check_airplane_mode(){
+	private function check_airplane_mode() {
 
-		if ( class_exists( 'Airplane_Mode_Core' ) ){
+		if (class_exists('Airplane_Mode_Core')) {
 			// Is airplane mode active?
-			$airplane_mode = get_site_option( 'airplane-mode' );
+			$airplane_mode = get_site_option('airplane-mode');
 
-			if ( $airplane_mode === 'on' ){
+			if ($airplane_mode === 'on') {
 				return false;
 			}
 		}
@@ -133,17 +133,17 @@ 
 	 *
 	 * @return boolean Connected or not.
 	 */
-	private function check_internet(){
+	private function check_internet() {
 
 		// Attempt to open a socket connection to Google
-		$connected = @fsockopen( "www.google.com", 80 );
+		$connected = @fsockopen("www.google.com", 80);
 
-		if ( !$connected ){
+		if (!$connected) {
 			return false;
 		}
 
 		// Close out our 1st test
-		fclose( $connected );
+		fclose($connected);
 
 		return true;
 
@@ -160,12 +160,12 @@ 
 	 * @param string $url External URL to attempt to reach.
 	 * @return boolean Connected or not.
 	 */
-	private function check_external_url( $url ){
+	private function check_external_url($url) {
 
-		$test_url = esc_url( $url );
-		$response = wp_remote_get( $test_url );
+		$test_url = esc_url($url);
+		$response = wp_remote_get($test_url);
 
-		if ( !is_array( $response ) ){
+		if (!is_array($response)) {
 			return false;
 		}
 

uninstall.php

Indentation +2 added lines, -2 removed lines

@@ -4,12 +4,12 @@
 
 // If uninstall is not called from WordPress, exit
 if ( !defined( 'WP_UNINSTALL_PLUGIN' ) ) {
-    exit();
+	exit();
 }
 
 // Check if the current user has priveledges to run this method
 if ( ! current_user_can( 'activate_plugins' ) ){
-    return;
+	return;
 }
 
 require dirname( __FILE__ ) . '/includes/class-delete.php';

Spacing +3 added lines, -3 removed lines

@@ -3,16 +3,16 @@
 use testContent as test;
 
 // If uninstall is not called from WordPress, exit
-if ( !defined( 'WP_UNINSTALL_PLUGIN' ) ) {
+if (!defined('WP_UNINSTALL_PLUGIN')) {
     exit();
 }
 
 // Check if the current user has priveledges to run this method
-if ( ! current_user_can( 'activate_plugins' ) ){
+if (!current_user_can('activate_plugins')) {
     return;
 }
 
-require dirname( __FILE__ ) . '/includes/class-delete.php';
+require dirname(__FILE__).'/includes/class-delete.php';
 $delete = new test\Delete;
 
 // Delete all the things

includes/class-metaboxes.php

Doc Comments +3 added lines

@@ -47,6 +47,9 @@
 	}
 
 
+	/**
+	 * @param string $slug
+	 */
 	private function get_acf_free_metaboxes( $slug ){
 
 		// This damn plugin. Is. A. Freaking. Nightmare.

Spacing +77 added lines, -77 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,23 +22,23 @@ 
 	 * @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) {
 
 		$fields = array();
 
 		// CMB2
-		if ( class_exists( 'CMB2', false ) ) {
-			$fields = $this->get_cmb2_metaboxes( $slug );
+		if (class_exists('CMB2', false)) {
+			$fields = $this->get_cmb2_metaboxes($slug);
 		}
 
 		// Custom Metaboxes and Fields (CMB1)
-		if ( class_exists( 'cmb_Meta_Box', false ) ) {
-			$fields = $this->get_cmb1_metaboxes( $slug );
+		if (class_exists('cmb_Meta_Box', false)) {
+			$fields = $this->get_cmb1_metaboxes($slug);
 		}
 
 		// Advanced Custom Fields (ACF Free)
-		if ( class_exists( 'acf', false ) ) {
-			$fields = $this->get_acf_free_metaboxes( $slug );
+		if (class_exists('acf', false)) {
+			$fields = $this->get_acf_free_metaboxes($slug);
 		}
 
 		// Return our array
@@ -47,25 +47,25 @@ 
 	}
 
 
-	private function get_acf_free_metaboxes( $slug ){
+	private function get_acf_free_metaboxes($slug) {
 
 		// This damn plugin. Is. A. Freaking. Nightmare.
 		$fieldsets = $this->get_all_acf_field_groups();
 
-		if ( empty( $fieldsets ) ){
+		if (empty($fieldsets)) {
 			return true;
 		}
 
-		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);
 				}
 
 			}
@@ -77,14 +77,14 @@ 
 	}
 
 
-	private function is_acf_field_in_post_type( $slug, $fieldset ){
+	private function is_acf_field_in_post_type($slug, $fieldset) {
 
-		if ( empty( $fieldset ) ){
+		if (empty($fieldset)) {
 			return;
 		}
 
-		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;
 			}
 		}
@@ -94,7 +94,7 @@ 
 	}
 
 
-	private function get_all_acf_field_groups(){
+	private function get_all_acf_field_groups() {
 		$info = $rules = $fields = array();
 
 		$args = array(
@@ -102,20 +102,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'],
@@ -153,55 +153,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']);
 			}
 
 		}
@@ -225,25 +225,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']);
 				}
 			}
 
@@ -266,36 +266,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;
@@ -326,14 +326,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;
 
@@ -341,13 +341,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;
 
@@ -364,8 +364,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;
 
@@ -376,7 +376,7 @@ 
 			case 'checkbox':
 
 				// 50/50 odds of being turned on
-				if ( rand( 0, 1 ) == 1 ){
+				if (rand(0, 1) == 1) {
 					$value = 'on';
 				}
 
@@ -387,10 +387,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;
 					}
 
@@ -408,8 +408,8 @@ 
 
 			case 'file':
 
-				if ( true == $connected ){
-					$value = TestContent::image( $post_id );
+				if (true == $connected) {
+					$value = TestContent::image($post_id);
 				}
 
 				break;
@@ -425,29 +425,29 @@ 
 		}
 
 		// Value must exist to attempt to insert
-		if ( !empty( $value ) && !is_wp_error( $value ) ){
+		if (!empty($value) && !is_wp_error($value)) {
 
 			$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);
 			}
 
-			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);
 			}
 
 		// 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();
 		}
 

views/posts.php

Spacing +9 added lines, -9 removed lines

@@ -9,11 +9,11 @@ 
  * @subpackage Test Content
  * @author     Old Town Media
  */
-class Posts extends View{
+class Posts extends View {
 
 	protected $title	= 'Posts';
 	protected $type		= 'post';
-	protected $priority	= 1;
+	protected $priority = 1;
 
 	/**
 	 * Our sections action block - button to create and delete.
@@ -22,20 +22,20 @@ 
 	 *
 	 * @return string HTML content.
 	 */
-	protected function actions_section(){
+	protected function actions_section() {
 		$html = '';
 
 		// Loop through every post type available on the site
-		$post_types = get_post_types( array( 'public' => true ), 'objects' );
+		$post_types = get_post_types(array('public' => true), 'objects');
 
-		foreach ( $post_types as $post_type ) :
+		foreach ($post_types as $post_type) :
 
 			$skipped_cpts = array(
 				'attachment'
 			);
 
 			// Skip banned cpts
-			if ( in_array( $post_type->name, $skipped_cpts ) ){
+			if (in_array($post_type->name, $skipped_cpts)) {
 				continue;
 			}
 
@@ -43,9 +43,9 @@ 
 
 				$html .= "<h3>";
 
-					$html .= "<span class='label'>" . $post_type->labels->name . "</span>";
-					$html .= $this->build_button( 'create', $post_type->name, __( 'Create Test Data', 'otm-test-content' ) );
-					$html .= $this->build_button( 'delete', $post_type->name, __( 'Delete Test Data', 'otm-test-content' ) );
+					$html .= "<span class='label'>".$post_type->labels->name."</span>";
+					$html .= $this->build_button('create', $post_type->name, __('Create Test Data', 'otm-test-content'));
+					$html .= $this->build_button('delete', $post_type->name, __('Delete Test Data', 'otm-test-content'));
 
 				$html .= "</h3>";
 

views/various.php

Spacing +6 added lines, -6 removed lines

@@ -10,11 +10,11 @@ 
  * @subpackage Test Content
  * @author     Old Town Media
  */
-class Various extends View{
+class Various extends View {
 
 	protected $title	= 'Various';
 	protected $type		= 'all';
-	protected $priority	= 3;
+	protected $priority = 3;
 
 
 	/**
@@ -24,15 +24,15 @@ 
 	 *
 	 * @return string HTML content.
 	 */
-	protected function actions_section(){
+	protected function actions_section() {
 		$html = '';
 
 		$html .= "<div class='test-data-cpt'>";
 
 			$html .= "<h3>";
 
-				$html .= "<span class='label'>" . __( 'Clean Site', 'otm-test-content' ) . "</span>";
-				$html .= $this->build_button( 'delete', 'all', __( 'Delete All Test Data', 'otm-test-content' ) );
+				$html .= "<span class='label'>".__('Clean Site', 'otm-test-content')."</span>";
+				$html .= $this->build_button('delete', 'all', __('Delete All Test Data', 'otm-test-content'));
 
 			$html .= "</h3>";
 
@@ -50,7 +50,7 @@ 
 	 * @param string $html Existing HTML content.
 	 * @return string HTML section content.
 	 */
-	protected function options_section( $html = '' ){
+	protected function options_section($html = '') {
 		return $html;
 	}
 

views/terms.php

Spacing +11 added lines, -11 removed lines

@@ -10,11 +10,11 @@ 
  * @subpackage Test Content
  * @author     Old Town Media
  */
-class Terms extends View{
+class Terms extends View {
 
 	protected $title	= 'Terms';
 	protected $type		= 'term';
-	protected $priority	= 2;
+	protected $priority = 2;
 
 
 	/**
@@ -24,25 +24,25 @@ 
 	 *
 	 * @return string HTML content.
 	 */
-	protected function actions_section(){
+	protected function actions_section() {
 		$html = '';
 
 		$taxonomies = get_taxonomies();
 
-		foreach ( $taxonomies as $tax ) :
+		foreach ($taxonomies as $tax) :
 
 			$skipped_taxonomies = array(
-				'post_format',				// We shouldn't be making random post format classes
-				'product_shipping_class',	// These aren't used visually and are therefore skipped
-				'nav_menu',					// Menus are handled seperately of taxonomies
+				'post_format', // We shouldn't be making random post format classes
+				'product_shipping_class', // These aren't used visually and are therefore skipped
+				'nav_menu', // Menus are handled seperately of taxonomies
 			);
 
 			// Skip banned taxonomies
-			if ( in_array( $tax, $skipped_taxonomies ) ){
+			if (in_array($tax, $skipped_taxonomies)) {
 				continue;
 			}
 
-			$taxonomy = get_taxonomy( $tax );
+			$taxonomy = get_taxonomy($tax);
 
 			$html .= "<div class='test-data-cpt'>";
 
@@ -50,8 +50,8 @@ 
 
 				$html .= "<span class='label'>".$taxonomy->labels->name."</span>";
 
-				$html .= $this->build_button( 'create', $tax, __( 'Create', 'otm-test-content' )." ".$taxonomy->labels->name );
-				$html .= $this->build_button( 'delete', $tax, __( 'Delete', 'otm-test-content' )." ".$taxonomy->labels->name );
+				$html .= $this->build_button('create', $tax, __('Create', 'otm-test-content')." ".$taxonomy->labels->name);
+				$html .= $this->build_button('delete', $tax, __('Delete', 'otm-test-content')." ".$taxonomy->labels->name);
 
 				$html .= "</h3>";
 

views/view.php

Spacing +17 added lines, -17 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,7 +58,7 @@ 
 	 * 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' href='javascript:void(0)'>";
@@ -78,7 +78,7 @@ 
 	 *
 	 * @see actions_section, options_section
 	 */
-	public function view(){
+	public function view() {
 		$html = '';
 
 		$html .= "<section class='test-content-tab'>";
@@ -98,7 +98,7 @@ 
 	 *
 	 * @return string HTML content.
 	 */
-	protected function actions_section(){
+	protected function actions_section() {
 		$html = '';
 		return $html;
 	}
@@ -115,11 +115,11 @@ 
 	 * @param string $html Existing HTML content.
 	 * @return string HTML section content.
 	 */
-	protected function options_section( $html = '' ){
+	protected function options_section($html = '') {
 		$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>";
 
@@ -137,22 +137,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>";