oldtownmedia / test-content-suite

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>";
 

test-content.php

Indentation +10 added lines, -10 removed lines

@@ -28,17 +28,17 @@ 
  		return;
  	}
 
-    $class = str_replace( __NAMESPACE__ . '\\', '', $class );
+	$class = str_replace( __NAMESPACE__ . '\\', '', $class );
 
 
 	if ( 'Views' === $namespace[1] ){
-        $class = strtolower( preg_replace( '/(?<!^)([A-Z])/', '/\1', $class ) );
-        $class = str_replace( '\\', '', $class );
-     	$file  = dirname( __FILE__ ) . '/' . $class . '.php';
-    } else {
-        $class = strtolower( preg_replace( '/(?<!^)([A-Z])/', '-\\1', $class ) );
-     	$file  = dirname( __FILE__ ) . '/includes/class-' . $class . '.php';
-    }
+		$class = strtolower( preg_replace( '/(?<!^)([A-Z])/', '/\1', $class ) );
+		$class = str_replace( '\\', '', $class );
+	 	$file  = dirname( __FILE__ ) . '/' . $class . '.php';
+	} else {
+		$class = strtolower( preg_replace( '/(?<!^)([A-Z])/', '-\\1', $class ) );
+	 	$file  = dirname( __FILE__ ) . '/includes/class-' . $class . '.php';
+	}
 
  	if ( file_exists( $file ) ) {
  		require_once( $file );
@@ -76,8 +76,8 @@ 
 
  // Register hook providers and views.
  plugin()->register_hooks( new AdminPage() )
-         ->register_views( new Views\Posts() )
-         ->register_views( new Views\Terms() )
+		 ->register_views( new Views\Posts() )
+		 ->register_views( new Views\Terms() )
 		 ->register_views( new Views\Various() );
 
 // Load textdomain hook

Spacing +22 added lines, -22 removed lines

@@ -21,30 +21,30 @@ 
  *
  * @param string $class Class name.
  */
-function test_content_autoloader( $class ) {
-	$namespace = explode( '\\', $class );
+function test_content_autoloader($class) {
+	$namespace = explode('\\', $class);
 
- 	if ( __NAMESPACE__ !== $namespace[0] ){
+ 	if (__NAMESPACE__ !== $namespace[0]) {
  		return;
  	}
 
-    $class = str_replace( __NAMESPACE__ . '\\', '', $class );
+    $class = str_replace(__NAMESPACE__.'\\', '', $class);
 
 
-	if ( 'Views' === $namespace[1] ){
-        $class = strtolower( preg_replace( '/(?<!^)([A-Z])/', '/\1', $class ) );
-        $class = str_replace( '\\', '', $class );
-     	$file  = dirname( __FILE__ ) . '/' . $class . '.php';
+	if ('Views' === $namespace[1]) {
+        $class = strtolower(preg_replace('/(?<!^)([A-Z])/', '/\1', $class));
+        $class = str_replace('\\', '', $class);
+     	$file  = dirname(__FILE__).'/'.$class.'.php';
     } else {
-        $class = strtolower( preg_replace( '/(?<!^)([A-Z])/', '-\\1', $class ) );
-     	$file  = dirname( __FILE__ ) . '/includes/class-' . $class . '.php';
+        $class = strtolower(preg_replace('/(?<!^)([A-Z])/', '-\\1', $class));
+     	$file  = dirname(__FILE__).'/includes/class-'.$class.'.php';
     }
 
- 	if ( file_exists( $file ) ) {
- 		require_once( $file );
+ 	if (file_exists($file)) {
+ 		require_once($file);
  	}
  }
- spl_autoload_register( __NAMESPACE__ . '\test_content_autoloader' );
+ spl_autoload_register(__NAMESPACE__.'\test_content_autoloader');
 
 
 
@@ -56,7 +56,7 @@ 
  function plugin() {
  	static $instance;
 
- 	if ( null === $instance ) {
+ 	if (null === $instance) {
  		$instance = new Plugin();
  	}
 
@@ -66,19 +66,19 @@ 
 // Set our definitions for later use
  plugin()->set_definitions(
 	(object) array(
-		'basename'	=> plugin_basename( __FILE__ ),
-		'directory'	=> plugin_dir_path( __FILE__ ),
+		'basename'	=> plugin_basename(__FILE__),
+		'directory'	=> plugin_dir_path(__FILE__),
 		'file'		=> __FILE__,
 		'slug' 		=> 'structure',
-		'url'		=> plugin_dir_url( __FILE__ )
+		'url'		=> plugin_dir_url(__FILE__)
 	)
 );
 
  // Register hook providers and views.
- plugin()->register_hooks( new AdminPage() )
-         ->register_views( new Views\Posts() )
-         ->register_views( new Views\Terms() )
-		 ->register_views( new Views\Various() );
+ plugin()->register_hooks(new AdminPage())
+         ->register_views(new Views\Posts())
+         ->register_views(new Views\Terms())
+		 ->register_views(new Views\Various());
 
 // Load textdomain hook
-add_action( 'plugins_loaded', array( plugin(), 'load_textdomain' ) );
+add_action('plugins_loaded', array(plugin(), 'load_textdomain'));

includes/class-plugin.php

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-admin-page.php

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-delete.php

Indentation +10 added lines, -10 removed lines

@@ -25,7 +25,7 @@ 
 		$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;
 
@@ -33,7 +33,7 @@ 
 		$taxonomies = get_taxonomies();
 		foreach ( $taxonomies as $tax ) :
 
-		    $this->delete_terms( $tax, $echo );
+			$this->delete_terms( $tax, $echo );
 
 		endforeach;
 
@@ -182,14 +182,14 @@ 
 
 		// Query for our terms
 		$args = array(
-		    'hide_empty' => false,
-		    'meta_query' => array(
-		        array(
-		           'key'       => 'evans_test_content',
-		           'value'     => '__test__',
-		           'compare'   => '='
-		        )
-		    )
+			'hide_empty' => false,
+			'meta_query' => array(
+				array(
+				   'key'       => 'evans_test_content',
+				   'value'     => '__test__',
+				   'compare'   => '='
+				)
+			)
 		);
 
 		$terms = get_terms( $slug, $args );

Spacing +42 added lines, -42 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;
 
@@ -52,15 +52,15 @@ 
 	 * @param string $slug a custom post type ID.
 	 * @param boolean $echo Whether or not to echo the result
 	 */
-	public function delete_posts( $slug, $echo = false ){
+	public function delete_posts($slug, $echo = false) {
 
 		// Make sure that the current user is logged in & has full permissions.
-		if ( !$this->user_can_delete() ){
+		if (!$this->user_can_delete()) {
 			return;
 		}
 
 		// Check that $cptslg has a string.
-		if ( empty( $slug ) ){
+		if (empty($slug)) {
 			return;
 		}
 
@@ -77,39 +77,39 @@ 
 			),
 		);
 
-		$objects = new \WP_Query( $query );
+		$objects = new \WP_Query($query);
 
-		if ( $objects->have_posts() ){
+		if ($objects->have_posts()) {
 
 			$events = array();
 
-			while ( $objects->have_posts() ) : $objects->the_post();
+			while ($objects->have_posts()) : $objects->the_post();
 
 				// Find any media associated with the test post and delete it as well
-				$this->delete_associated_media( get_the_id() );
+				$this->delete_associated_media(get_the_id());
 
-				if ( $echo === true ){
+				if ($echo === true) {
 					$events[] = array(
 						'type'		=> 'deleted',
 						'pid'		=> get_the_id(),
-						'post_type'	=> get_post_type( get_the_id() ),
+						'post_type'	=> get_post_type(get_the_id()),
 						'link'		=> ''
 					);
 				}
 
 				// Force delete the post
-				wp_delete_post( get_the_id(), true );
+				wp_delete_post(get_the_id(), true);
 
 			endwhile;
 
-			$obj = get_post_type_object( $slug );
+			$obj = get_post_type_object($slug);
 
 			$events[] = array(
 				'type'		=> 'general',
-				'message'	=> __( 'Deleted', 'otm-test-content' ) . ' ' . $obj->labels->all_items
+				'message'	=> __('Deleted', 'otm-test-content').' '.$obj->labels->all_items
 			);
 
-			echo \json_encode( $events );
+			echo \json_encode($events);
 
 		}
 
@@ -129,26 +129,26 @@ 
 	 *
 	 * @param int $pid a custom post type ID.
 	 */
-	private function delete_associated_media( $pid ){
+	private function delete_associated_media($pid) {
 
 		// Make sure that the current user is logged in & has full permissions.
-		if ( !$this->user_can_delete() ){
+		if (!$this->user_can_delete()) {
 			return;
 		}
 
 		// Make sure $pid is, in fact, an ID
-		if ( !is_int( $pid ) ){
+		if (!is_int($pid)) {
 			return;
 		}
 
 		// Get our images
-		$media = get_attached_media( 'image', $pid );
+		$media = get_attached_media('image', $pid);
 
-		if ( !empty( $media ) ){
+		if (!empty($media)) {
 
 			// Loop through the media & delete each one
-			foreach ( $media as $attachment ){
-				wp_delete_attachment( $attachment->ID, true );
+			foreach ($media as $attachment) {
+				wp_delete_attachment($attachment->ID, true);
 			}
 
 		}
@@ -168,15 +168,15 @@ 
 	 * @param string $slug a custom post type ID.
 	 * @param boolean $echo Whether or not to echo the result
 	 */
-	public function delete_terms( $slug, $echo = false ){
+	public function delete_terms($slug, $echo = false) {
 
 		// Make sure that the current user is logged in & has full permissions.
-		if ( !$this->user_can_delete() ){
+		if (!$this->user_can_delete()) {
 			return;
 		}
 
 		// Check that $cptslg has a string.
-		if ( empty( $slug ) ){
+		if (empty($slug)) {
 			return;
 		}
 
@@ -192,15 +192,15 @@ 
 		    )
 		);
 
-		$terms = get_terms( $slug, $args );
+		$terms = get_terms($slug, $args);
 
-		if ( !empty( $terms ) ){
+		if (!empty($terms)) {
 
 			$events = array();
 
-			foreach ( $terms as $term ){
+			foreach ($terms as $term) {
 
-				if ( $echo === true ){
+				if ($echo === true) {
 					$events[] = array(
 						'type'		=> 'deleted',
 						'pid'		=> $term->term_id,
@@ -210,18 +210,18 @@ 
 				}
 
 				// Delete our term
-				wp_delete_term( $term->term_id, $slug );
+				wp_delete_term($term->term_id, $slug);
 
 			}
 
-			$taxonomy = get_taxonomy( $slug );
+			$taxonomy = get_taxonomy($slug);
 
 			$events[] = array(
 				'type'		=> 'general',
-				'message'	=> __( 'Deleted', 'otm-test-content' ) . ' ' . $taxonomy->labels->name
+				'message'	=> __('Deleted', 'otm-test-content').' '.$taxonomy->labels->name
 			);
 
-			echo \json_encode( $events );
+			echo \json_encode($events);
 
 		}
 
@@ -233,15 +233,15 @@ 
 	 *
 	 * @see is_user_logged_in, current_user_can
 	 */
-	private function user_can_delete(){
+	private 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;
 		}