WordPoints / hooks-api

src/includes/classes/hook/action/comment/new.php

Spacing +2 added lines, -2 removed lines

@@ -19,11 +19,11 @@
 	 */
 	public function should_fire() {
 
-		if ( ! isset( $this->args[1]->comment_approved ) ) {
+		if ( ! isset($this->args[1]->comment_approved)) {
 			return false;
 		}
 
-		if ( 1 !== (int) $this->args[1]->comment_approved ) {
+		if (1 !== (int) $this->args[1]->comment_approved) {
 			return false;
 		}
 

src/admin/admin.php

Spacing +3 added lines, -3 removed lines

@@ -12,17 +12,17 @@
  *
  * @since 1.0.0
  */
-require_once( dirname( __FILE__ ) . '/includes/functions.php' );
+require_once(dirname(__FILE__).'/includes/functions.php');
 
 /**
  * The admin-side actions and filters.
  *
  * @since 1.0.0
  */
-require_once( dirname( __FILE__ ) . '/includes/actions.php' );
+require_once(dirname(__FILE__).'/includes/actions.php');
 
 WordPoints_Class_Autoloader::register_dir(
-	dirname( __FILE__ ) . '/includes/classes'
+	dirname(__FILE__).'/includes/classes'
 	, 'WordPoints_Admin_'
 );
 

src/admin/includes/classes/ajax/hooks.php

Spacing +42 added lines, -42 removed lines

@@ -52,7 +52,7 @@ 
 	 *
 	 * @return array The hook reaction data extracted into an array.
 	 */
-	public static function prepare_hook_reaction( $reaction ) {
+	public static function prepare_hook_reaction($reaction) {
 
 		$reactor = $reaction->get_reactor_slug();
 
@@ -95,17 +95,17 @@ 
 
 		add_action(
 			'wp_ajax_wordpoints_admin_create_hook_reaction'
-			, array( $this, 'create_hook_reaction' )
+			, array($this, 'create_hook_reaction')
 		);
 
 		add_action(
 			'wp_ajax_wordpoints_admin_update_hook_reaction'
-			, array( $this, 'update_hook_reaction' )
+			, array($this, 'update_hook_reaction')
 		);
 
 		add_action(
 			'wp_ajax_wordpoints_admin_delete_hook_reaction'
-			, array( $this, 'delete_hook_reaction' )
+			, array($this, 'delete_hook_reaction')
 		);
 	}
 
@@ -124,9 +124,9 @@ 
 			"wordpoints_create_hook_reaction|{$this->reactor_slug}"
 		);
 
-		$reaction = $reactor->reactions->create_reaction( $this->get_data() );
+		$reaction = $reactor->reactions->create_reaction($this->get_data());
 
-		$this->send_json_result( $reaction, 'create' );
+		$this->send_json_result($reaction, 'create');
 	}
 
 	/**
@@ -150,7 +150,7 @@ 
 			, $this->get_data()
 		);
 
-		$this->send_json_result( $reaction, 'update' );
+		$this->send_json_result($reaction, 'update');
 	}
 
 	/**
@@ -169,10 +169,10 @@ 
 			"wordpoints_delete_hook_reaction|{$this->reactor_slug}|{$reaction->ID}"
 		);
 
-		$result = $reactor->reactions->delete_reaction( $reaction->ID );
+		$result = $reactor->reactions->delete_reaction($reaction->ID);
 
-		if ( ! $result ) {
-			wp_send_json_error( array( 'message' => __( 'There was an error deleting the reaction. Please try again.', 'wordpoints' ) ) );
+		if ( ! $result) {
+			wp_send_json_error(array('message' => __('There was an error deleting the reaction. Please try again.', 'wordpoints')));
 		}
 
 		wp_send_json_success();
@@ -193,11 +193,11 @@ 
 	 *
 	 * @param string $debug_context Context sent with the message (for debugging).
 	 */
-	private function unexpected_error( $debug_context ) {
+	private function unexpected_error($debug_context) {
 
 		wp_send_json_error(
 			array(
-				'message' => __( 'There was an unexpected error. Try reloading the page.', 'wordpoints' ),
+				'message' => __('There was an unexpected error. Try reloading the page.', 'wordpoints'),
 				'debug'   => $debug_context,
 			)
 		);
@@ -213,8 +213,8 @@ 
 	 */
 	private function verify_user_can() {
 
-		if ( ! current_user_can( 'manage_options' ) ) {
-			wp_send_json_error( array( 'message' => __( 'You do not have permission to perform this action. Maybe you have been logged out?', 'wordpoints' ) ) );
+		if ( ! current_user_can('manage_options')) {
+			wp_send_json_error(array('message' => __('You do not have permission to perform this action. Maybe you have been logged out?', 'wordpoints')));
 		}
 	}
 
@@ -227,14 +227,14 @@ 
 	 *
 	 * @param string $action The action the nonce should be for.
 	 */
-	private function verify_request( $action ) {
+	private function verify_request($action) {
 
 		if (
-			empty( $_POST['nonce'] )
-			|| ! wordpoints_verify_nonce( 'nonce', $action, null, 'post' )
+			empty($_POST['nonce'])
+			|| ! wordpoints_verify_nonce('nonce', $action, null, 'post')
 		) {
 			wp_send_json_error(
-				array( 'message' => __( 'Your security token for this action has expired. Refresh the page and try again.', 'wordpoints' ) )
+				array('message' => __('Your security token for this action has expired. Refresh the page and try again.', 'wordpoints'))
 			);
 		}
 	}
@@ -248,16 +248,16 @@ 
 	 */
 	protected function get_reactor() {
 
-		if ( ! isset( $_POST['reactor'] ) ) { // WPCS: CSRF OK.
-			$this->unexpected_error( 'reactor' );
+		if ( ! isset($_POST['reactor'])) { // WPCS: CSRF OK.
+			$this->unexpected_error('reactor');
 		}
 
-		$reactor_slug = sanitize_key( $_POST['reactor'] ); // WPCS: CSRF OK.
+		$reactor_slug = sanitize_key($_POST['reactor']); // WPCS: CSRF OK.
 
-		$reactor = wordpoints_hooks()->reactors->get( $reactor_slug );
+		$reactor = wordpoints_hooks()->reactors->get($reactor_slug);
 
-		if ( ! $reactor instanceof WordPoints_Hook_Reactor ) {
-			$this->unexpected_error( 'reactor_invalid' );
+		if ( ! $reactor instanceof WordPoints_Hook_Reactor) {
+			$this->unexpected_error('reactor_invalid');
 		}
 
 		$this->reactor_slug = $reactor_slug;
@@ -275,16 +275,16 @@ 
 	 */
 	protected function get_reaction() {
 
-		if ( ! isset( $_POST['id'] ) ) { // WPCS: CSRF OK.
-			$this->unexpected_error( 'id' );
+		if ( ! isset($_POST['id'])) { // WPCS: CSRF OK.
+			$this->unexpected_error('id');
 		}
 
 		$reaction = $this->reactor->reactions->get_reaction(
-			wordpoints_int( $_POST['id'] ) // WPCS: CSRF OK.
+			wordpoints_int($_POST['id']) // WPCS: CSRF OK.
 		);
 
-		if ( ! $reaction ) {
-			wp_send_json_error( array( 'message' => __( 'The reaction ID passed to the server is invalid. Perhaps it has been deleted. Try reloading the page.', 'wordpoints' ) ) );
+		if ( ! $reaction) {
+			wp_send_json_error(array('message' => __('The reaction ID passed to the server is invalid. Perhaps it has been deleted. Try reloading the page.', 'wordpoints')));
 		}
 
 		return $reaction;
@@ -299,9 +299,9 @@ 
 	 */
 	protected function get_data() {
 
-		$data = wp_unslash( $_POST ); // WPCS: CSRF OK.
+		$data = wp_unslash($_POST); // WPCS: CSRF OK.
 
-		unset( $data['id'], $data['action'], $data['nonce'], $data['reactor'] );
+		unset($data['id'], $data['action'], $data['nonce'], $data['reactor']);
 
 		return $data;
 	}
@@ -314,30 +314,30 @@ 
 	 * @param mixed  $result The result of the action.
 	 * @param string $action The action being performed: 'create' or 'update'.
 	 */
-	private function send_json_result( $result, $action ) {
+	private function send_json_result($result, $action) {
 
-		if ( ! $result ) {
+		if ( ! $result) {
 
-			if ( 'create' === $action ) {
-				$message = __( 'There was an error adding the reaction. Please try again.', 'wordpoints' );
+			if ('create' === $action) {
+				$message = __('There was an error adding the reaction. Please try again.', 'wordpoints');
 			} else {
-				$message = __( 'There was an error updating the reaction. Please try again.', 'wordpoints' );
+				$message = __('There was an error updating the reaction. Please try again.', 'wordpoints');
 			}
 
-			wp_send_json_error( array( 'message' => $message ) );
+			wp_send_json_error(array('message' => $message));
 
-		} elseif ( $result instanceof WordPoints_Hook_Reaction_Validator ) {
+		} elseif ($result instanceof WordPoints_Hook_Reaction_Validator) {
 
-			wp_send_json_error( array( 'errors' => $result->get_errors() ) );
+			wp_send_json_error(array('errors' => $result->get_errors()));
 		}
 
 		$data = null;
 
-		if ( 'create' === $action ) {
-			$data = self::prepare_hook_reaction( $result );
+		if ('create' === $action) {
+			$data = self::prepare_hook_reaction($result);
 		}
 
-		wp_send_json_success( $data );
+		wp_send_json_success($data);
 	}
 }
 

src/includes/classes/hook/condition/entity/array/contains.php

Spacing +34 added lines, -34 removed lines

@@ -56,7 +56,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Contains', 'wordpoints' );
+		return __('Contains', 'wordpoints');
 	}
 
 	/**
@@ -66,13 +66,13 @@ 
 		return array(
 			'min' => array(
 				'slug'    => 'min',
-				'label'   => __( 'Minimum number of items', 'wordpoints' ),
+				'label'   => __('Minimum number of items', 'wordpoints'),
 				'type'    => 'number',
 				'default' => 1,
 			),
 			'max' => array(
 				'slug'  => 'max',
-				'label' => __( 'Maximum number of items', 'wordpoints' ),
+				'label' => __('Maximum number of items', 'wordpoints'),
 				'type'  => 'number',
 			),
 		);
@@ -92,8 +92,8 @@ 
 
 		$this->validate_count();
 
-		if ( isset( $settings['conditions'] ) ) {
-			$this->validate_conditions( $arg );
+		if (isset($settings['conditions'])) {
+			$this->validate_conditions($arg);
 		}
 
 		return $this->settings;
@@ -107,31 +107,31 @@ 
 	protected function validate_count() {
 
 		if (
-			! empty( $this->settings['max'] )
-			&& ! wordpoints_posint( $this->settings['max'] )
+			! empty($this->settings['max'])
+			&& ! wordpoints_posint($this->settings['max'])
 		) {
 			$this->validator->add_error(
-				__( 'The maximum must be a positive integer.', 'wordpoints' )
+				__('The maximum must be a positive integer.', 'wordpoints')
 				, 'max'
 			);
 		}
 
-		if ( ! empty( $this->settings['min'] ) ) {
+		if ( ! empty($this->settings['min'])) {
 
-			if ( ! wordpoints_posint( $this->settings['min'] ) ) {
+			if ( ! wordpoints_posint($this->settings['min'])) {
 
 				$this->validator->add_error(
-					__( 'The minimum must be a positive integer.', 'wordpoints' )
+					__('The minimum must be a positive integer.', 'wordpoints')
 					, 'min'
 				);
 
 			} elseif (
-				! empty( $this->settings['max'] )
+				! empty($this->settings['max'])
 				&& $this->settings['max'] < $this->settings['min']
 			) {
 
 				$this->validator->add_error(
-					__( 'The minimum must be less than the maximum.', 'wordpoints' )
+					__('The minimum must be less than the maximum.', 'wordpoints')
 					, 'min'
 				);
 			}
@@ -145,25 +145,25 @@ 
 	 *
 	 * @param WordPoints_EntityishI $current_arg The current arg.
 	 */
-	protected function validate_conditions( $current_arg ) {
+	protected function validate_conditions($current_arg) {
 
-		$args = new WordPoints_Hook_Event_Args( array() );
+		$args = new WordPoints_Hook_Event_Args(array());
 
-		if ( $current_arg instanceof WordPoints_Entity_Array ) {
+		if ($current_arg instanceof WordPoints_Entity_Array) {
 
 			$entity = wordpoints_entities()->get(
 				$current_arg->get_entity_slug()
 			);
 
-			if ( $entity instanceof WordPoints_Entity ) {
-				$args->add_entity( $entity );
+			if ($entity instanceof WordPoints_Entity) {
+				$args->add_entity($entity);
 			}
 		}
 
-		$args->set_validator( $this->validator );
+		$args->set_validator($this->validator);
 
 		$conditions = $this->conditions_extension->validate_settings(
-			array( 'conditions' => $this->settings['conditions'] )
+			array('conditions' => $this->settings['conditions'])
 			, $this->validator
 			, $args
 		);
@@ -174,7 +174,7 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function is_met( array $settings, WordPoints_Hook_Event_Args $args ) {
+	public function is_met(array $settings, WordPoints_Hook_Event_Args $args) {
 
 		$this->settings = $settings;
 
@@ -182,16 +182,16 @@ 
 
 		$entities = array();
 
-		if ( $arg instanceof WordPoints_Entity_Array ) {
+		if ($arg instanceof WordPoints_Entity_Array) {
 
 			$entities = $arg->get_the_entities();
 
-			if ( isset( $this->settings['conditions'] ) ) {
-				$entities = $this->filter_entities( $entities );
+			if (isset($this->settings['conditions'])) {
+				$entities = $this->filter_entities($entities);
 			}
 		}
 
-		return $this->check_count( count( $entities ) );
+		return $this->check_count(count($entities));
 	}
 
 	/**
@@ -203,20 +203,20 @@ 
 	 *
 	 * @return WordPoints_Entity[] The entities that matched the sub-conditions.
 	 */
-	protected function filter_entities( $entities ) {
+	protected function filter_entities($entities) {
 
-		foreach ( $entities as $index => $entity ) {
+		foreach ($entities as $index => $entity) {
 
-			$event_args = new WordPoints_Hook_Event_Args( array() );
-			$event_args->add_entity( $entity );
+			$event_args = new WordPoints_Hook_Event_Args(array());
+			$event_args->add_entity($entity);
 
 			$matches = $this->conditions_extension->conditions_are_met(
 				$this->settings['conditions']
 				, $event_args
 			);
 
-			if ( ! $matches ) {
-				unset( $entities[ $index ] );
+			if ( ! $matches) {
+				unset($entities[$index]);
 			}
 		}
 
@@ -232,13 +232,13 @@ 
 	 *
 	 * @return bool Whether the count met the requirements.
 	 */
-	protected function check_count( $count ) {
+	protected function check_count($count) {
 
-		if ( isset( $this->settings['max'] ) && $count > $this->settings['max'] ) {
+		if (isset($this->settings['max']) && $count > $this->settings['max']) {
 			return false;
 		}
 
-		if ( isset( $this->settings['min'] ) && $count < $this->settings['min'] ) {
+		if (isset($this->settings['min']) && $count < $this->settings['min']) {
 			return false;
 		}
 

src/includes/classes/hook/event/media/upload.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
 	 */
 	public function get_title() {
 
-		return __( 'Upload Media', 'wordpoints' );
+		return __('Upload Media', 'wordpoints');
 	}
 
 	/**
@@ -27,7 +27,7 @@ 
 	 */
 	public function get_description() {
 
-		return __( 'When a file is uploaded to the Media Library.', 'wordpoints' );
+		return __('When a file is uploaded to the Media Library.', 'wordpoints');
 	}
 }
 

src/includes/classes/entity/post.php

Spacing +4 added lines, -4 removed lines

@@ -36,9 +36,9 @@ 
 	 */
 	public function get_title() {
 
-		$post_type = get_post_type_object( substr( $this->slug, 5 /* post\ */ ) );
+		$post_type = get_post_type_object(substr($this->slug, 5 /* post\ */));
 
-		if ( $post_type ) {
+		if ($post_type) {
 			return $post_type->labels->singular_name;
 		} else {
 			return $this->slug;
@@ -48,8 +48,8 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function user_can_view( $user_id, $id ) {
-		return user_can( $user_id, 'read_post', $id );
+	public function user_can_view($user_id, $id) {
+		return user_can($user_id, 'read_post', $id);
 	}
 }
 

src/includes/classes/entity/term.php

Spacing +4 added lines, -4 removed lines

@@ -29,9 +29,9 @@ 
 	 */
 	public function get_title() {
 
-		$taxonomy = get_taxonomy( substr( $this->slug, 5 /* term\ */ ) );
+		$taxonomy = get_taxonomy(substr($this->slug, 5 /* term\ */));
 
-		if ( $taxonomy ) {
+		if ($taxonomy) {
 			return $taxonomy->labels->singular_name;
 		} else {
 			return $this->slug;
@@ -41,8 +41,8 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function get_entity( $id ) {
-		return get_term( $id, 'taxonomy' );
+	public function get_entity($id) {
+		return get_term($id, 'taxonomy');
 	}
 }
 

src/includes/classes/entity/post/terms.php

Spacing +5 added lines, -5 removed lines

@@ -29,9 +29,9 @@ 
 	 */
 	public function get_title() {
 
-		$taxonomy = get_taxonomy( substr( $this->slug, 6 /* terms\ */ ) );
+		$taxonomy = get_taxonomy(substr($this->slug, 6 /* terms\ */));
 
-		if ( $taxonomy ) {
+		if ($taxonomy) {
 			return $taxonomy->labels->name;
 		} else {
 			return parent::get_title();
@@ -41,13 +41,13 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	protected function get_related_entity_ids( WordPoints_Entity $entity ) {
+	protected function get_related_entity_ids(WordPoints_Entity $entity) {
 
 		$id = $entity->get_the_id();
 
-		$taxonomies = get_object_taxonomies( get_post( $id ) );
+		$taxonomies = get_object_taxonomies(get_post($id));
 
-		return wp_get_object_terms( $id, $taxonomies, array( 'fields' => 'ids' ) );
+		return wp_get_object_terms($id, $taxonomies, array('fields' => 'ids'));
 	}
 }
 

src/includes/classes/app.php

Spacing +27 added lines, -27 removed lines

@@ -97,16 +97,16 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $slug, $parent = null ) {
+	public function __construct($slug, $parent = null) {
 
 		$this->slug = $slug;
 		$this->full_slug = $slug;
 
-		if ( $parent instanceof WordPoints_App ) {
+		if ($parent instanceof WordPoints_App) {
 			$this->parent = $parent;
 
-			if ( 'apps' !== $this->parent->full_slug ) {
-				$this->full_slug = $this->parent->full_slug . '-' . $this->full_slug;
+			if ('apps' !== $this->parent->full_slug) {
+				$this->full_slug = $this->parent->full_slug.'-'.$this->full_slug;
 			}
 		}
 
@@ -118,32 +118,32 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __isset( $var ) {
-		return $this->sub_apps->is_registered( $var );
+	public function __isset($var) {
+		return $this->sub_apps->is_registered($var);
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __get( $var ) {
+	public function __get($var) {
 
-		if ( 'sub_apps' === $var ) {
+		if ('sub_apps' === $var) {
 			return $this->$var;
 		}
 
-		$sub = $this->sub_apps->get( $var, array( $this ) );
+		$sub = $this->sub_apps->get($var, array($this));
 
-		if ( ! $sub ) {
+		if ( ! $sub) {
 			return null;
 		}
 
 		if (
-			empty( $this->did_init[ $var ] )
+			empty($this->did_init[$var])
 			&& ! self::$main->silent
-			&& $this->should_do_registry_init( $sub )
+			&& $this->should_do_registry_init($sub)
 		) {
 
-			$is_protected_property = array_key_exists( $var, get_object_vars( $this ) );
+			$is_protected_property = array_key_exists($var, get_object_vars($this));
 
 			// When the below action is called it is possible that some hooked code
 			// may attempt to access the property that is being initialized. However,
@@ -152,7 +152,7 @@ 
 			// temporarily set this as a public property. However, we can't do this
 			// for properties that the class has declared, because otherwise it would
 			// be possible to modify and then unset protected class properties.
-			if ( ! $is_protected_property ) {
+			if ( ! $is_protected_property) {
 				$this->allow_set = true;
 				$this->$var = $sub;
 				$this->allow_set = false;
@@ -168,25 +168,25 @@ 
 			 * @param WordPoints_Class_RegistryI|WordPoints_Class_Registry_ChildrenI
 			 *        $registry The registry object.
 			 */
-			do_action( "wordpoints_init_app_registry-{$this->full_slug}-{$var}", $sub );
+			do_action("wordpoints_init_app_registry-{$this->full_slug}-{$var}", $sub);
 
 			// Because this property was public, it's possible that it might have
 			// been modified inadvertently. If this happens we give a warning.
-			if ( $sub !== $this->$var ) {
+			if ($sub !== $this->$var) {
 				_doing_it_wrong(
 					__METHOD__
-					, esc_html( "Do not modify the {$var} property directly." )
+					, esc_html("Do not modify the {$var} property directly.")
 					, '1.0.0'
 				);
 			}
 
-			if ( ! $is_protected_property ) {
+			if ( ! $is_protected_property) {
 				$this->allow_set = true;
-				unset( $this->$var );
+				unset($this->$var);
 				$this->allow_set = false;
 			}
 
-			$this->did_init[ $var ] = true;
+			$this->did_init[$var] = true;
 		}
 
 		return $sub;
@@ -195,9 +195,9 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __set( $var, $value ) {
+	public function __set($var, $value) {
 
-		if ( $this->allow_set ) {
+		if ($this->allow_set) {
 			$this->$var = $value;
 		} else {
 			_doing_it_wrong(
@@ -211,10 +211,10 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __unset( $var ) {
+	public function __unset($var) {
 
-		if ( $this->allow_set ) {
-			unset( $this->$var );
+		if ($this->allow_set) {
+			unset($this->$var);
 		} else {
 			_doing_it_wrong(
 				__METHOD__
@@ -233,7 +233,7 @@ 
 	 *
 	 * @return bool Whether to call the init action or not.
 	 */
-	protected function should_do_registry_init( $registry ) {
+	protected function should_do_registry_init($registry) {
 		return (
 		   $registry instanceof WordPoints_Class_RegistryI
 		   || $registry instanceof WordPoints_Class_Registry_ChildrenI
@@ -257,7 +257,7 @@ 
 		 *
 		 * @param WordPoints_App $app The app object.
 		 */
-		do_action( "wordpoints_init_app-{$this->full_slug}", $this );
+		do_action("wordpoints_init_app-{$this->full_slug}", $this);
 	}
 }