WordPoints / hooks-api

src/includes/classes/hook/condition.php

Spacing +10 added lines, -10 removed lines

@@ -23,33 +23,33 @@ 
 		WordPoints_Hook_Reaction_Validator $validator
 	) {
 
-		if ( ! isset( $settings['value'] ) || '' === $settings['value'] ) {
+		if ( ! isset($settings['value']) || '' === $settings['value']) {
 
 			$settings_fields = $this->get_settings_fields();
 
 			$validator->add_error(
 				sprintf(
-					__( '%s is required.', 'wordpoints' )
+					__('%s is required.', 'wordpoints')
 					, $settings_fields['value']['label']
 				)
 				, 'value'
 			);
 
-		} elseif ( $arg instanceof WordPoints_Entity_Attr ) {
+		} elseif ($arg instanceof WordPoints_Entity_Attr) {
 
 			$data_types = wordpoints_apps()->data_types;
 
-			$data_type = $data_types->get( $arg->get_data_type() );
+			$data_type = $data_types->get($arg->get_data_type());
 
 			// If this data type isn't recognized, that's probably OK. Validation is
 			// just to help the user know that they've made a mistake anyway.
-			if ( ! ( $data_type instanceof WordPoints_Data_TypeI ) ) {
+			if ( ! ($data_type instanceof WordPoints_Data_TypeI)) {
 				return $settings;
 			}
 
-			$validated_value = $data_type->validate_value( $settings['value'] );
+			$validated_value = $data_type->validate_value($settings['value']);
 
-			if ( is_wp_error( $validated_value ) ) {
+			if (is_wp_error($validated_value)) {
 
 				$settings_fields = $this->get_settings_fields();
 
@@ -66,13 +66,13 @@ 
 
 			$settings['value'] = $validated_value;
 
-		} elseif ( $arg instanceof WordPoints_Entity ) {
+		} elseif ($arg instanceof WordPoints_Entity) {
 
-			if ( ! $arg->exists( $settings['value'] ) ) {
+			if ( ! $arg->exists($settings['value'])) {
 				$validator->add_error(
 					sprintf(
 						// translators: 1. item type, 2. item ID/slug.
-						__( '%1$s “%2$s” not found.', 'wordpoints' )
+						__('%1$s “%2$s” not found.', 'wordpoints')
 						, $arg->get_title()
 						, $settings['value']
 					)

src/includes/classes/hook/condition/equals.php

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Equals', 'wordpoints' );
+		return __('Equals', 'wordpoints');
 	}
 
 	/**
@@ -29,7 +29,7 @@ 
 		return array(
 			'value' => array(
 				'type' => 'text',
-				'label' => __( 'Equals', 'wordpoints' ), // TODO
+				'label' => __('Equals', 'wordpoints'), // TODO
 			),
 		);
 	}
@@ -37,7 +37,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) {
 
 		return $settings['value'] === $args->get_current()->get_the_value();
 	}

src/includes/classes/hook/condition/string/contains.php

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Contains', 'wordpoints' );
+		return __('Contains', 'wordpoints');
 	}
 
 	/**
@@ -28,7 +28,7 @@ 
 		return array(
 			'value' => array(
 				'type'     => 'text',
-				'label'    => __( 'Contains', 'wordpoints' ), // TODO
+				'label'    => __('Contains', 'wordpoints'), // TODO
 				'required' => true,
 			),
 		);
@@ -37,7 +37,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) {
 
 		return false !== strpos(
 			$args->get_current()->get_the_value()

src/includes/classes/hook/conditioni.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
 	 *
 	 * @return bool Whether this condition is met.
 	 */
-	public function is_met( array $settings, WordPoints_Hook_Event_Args $args );
+	public function is_met(array $settings, WordPoints_Hook_Event_Args $args);
 }
 
 // EOF

src/includes/classes/hook/event.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $slug ) {
+	public function __construct($slug) {
 
 		$this->slug = $slug;
 	}

src/includes/classes/hook/event/user/register.php

Spacing +2 added lines, -2 removed lines

@@ -18,14 +18,14 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Register', 'wordpoints' );
+		return __('Register', 'wordpoints');
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_description() {
-		return __( 'Registering.', 'wordpoints' );
+		return __('Registering.', 'wordpoints');
 	}
 }
 

src/includes/classes/hook/event/user/visit.php

Spacing +2 added lines, -2 removed lines

@@ -18,14 +18,14 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Visit', 'wordpoints' );
+		return __('Visit', 'wordpoints');
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_description() {
-		return __( 'When a logged-in user or guest visits the site.', 'wordpoints' );
+		return __('When a logged-in user or guest visits the site.', 'wordpoints');
 	}
 }
 

src/includes/classes/hook/eventi.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 	/**
 	 * @param string $slug The event slug.
 	 */
-	public function __construct( $slug );
+	public function __construct($slug);
 
 	/**
 	 * Get the event slug.

src/includes/classes/hook/events.php

Spacing +21 added lines, -21 removed lines

@@ -37,13 +37,13 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function __construct( $slug ) {
+	public function __construct($slug) {
 
 		$hooks = wordpoints_hooks();
 
 		$this->router = $hooks->router;
 
-		parent::__construct( $slug );
+		parent::__construct($slug);
 	}
 
 	/**
@@ -51,7 +51,7 @@ 
 	 */
 	public function init() {
 
-		$this->sub_apps->register( 'args', 'WordPoints_Class_Registry_Children' );
+		$this->sub_apps->register('args', 'WordPoints_Class_Registry_Children');
 
 		parent::init();
 	}
@@ -75,29 +75,29 @@ 
 	 *
 	 * @return bool Whether the event was registered.
 	 */
-	public function register( $slug, $class, array $args = array() ) {
+	public function register($slug, $class, array $args = array()) {
 
 		// TODO shoudl this be required?
-		if ( ! isset( $args['actions'] ) ) {
+		if ( ! isset($args['actions'])) {
 			return false;
 		}
 
-		parent::register( $slug, $class, $args );
+		parent::register($slug, $class, $args);
 
-		foreach ( $args['actions'] as $type => $actions ) {
-			foreach ( (array) $actions as $action_slug ) {
-				$this->router->add_event_to_action( $slug, $action_slug, $type );
+		foreach ($args['actions'] as $type => $actions) {
+			foreach ((array) $actions as $action_slug) {
+				$this->router->add_event_to_action($slug, $action_slug, $type);
 			}
 		}
 
 		// TODO should this be required?
-		if ( isset( $args['args'] ) ) {
-			foreach ( $args['args'] as $arg_slug => $class ) {
-				$this->args->register( $slug, $arg_slug, $class );
+		if (isset($args['args'])) {
+			foreach ($args['args'] as $arg_slug => $class) {
+				$this->args->register($slug, $arg_slug, $class);
 			}
 		}
 
-		$this->event_data[ $slug ] = $args;
+		$this->event_data[$slug] = $args;
 
 		return true;
 	}
@@ -105,23 +105,23 @@ 
 	/**
 	 * @since 1.0.0
 	 */
-	public function deregister( $slug ) {
+	public function deregister($slug) {
 
-		if ( ! $this->is_registered( $slug ) ) {
+		if ( ! $this->is_registered($slug)) {
 			return;
 		}
 
-		parent::deregister( $slug );
+		parent::deregister($slug);
 
-		foreach ( (array) $this->event_data[ $slug ]['actions'] as $type => $actions ) {
-			foreach ( (array) $actions as $action_slug ) {
-				$this->router->remove_event_from_action( $slug, $action_slug, $type );
+		foreach ((array) $this->event_data[$slug]['actions'] as $type => $actions) {
+			foreach ((array) $actions as $action_slug) {
+				$this->router->remove_event_from_action($slug, $action_slug, $type);
 			}
 		}
 
-		$this->args->deregister_children( $slug );
+		$this->args->deregister_children($slug);
 
-		unset( $this->event_data[ $slug ] );
+		unset($this->event_data[$slug]);
 	}
 }