WordPoints / hooks-api

src/includes/classes/hook/arg/current/site.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
 	 */
 	public function get_value() {
 
-		if ( wordpoints_is_network_context() ) {
+		if (wordpoints_is_network_context()) {
 			return false;
 		}
 
@@ -35,7 +35,7 @@ 
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Site', 'wordpoints' );
+		return __('Site', 'wordpoints');
 	}
 }
 

src/includes/classes/entity/change.php

Doc Comments +15 added lines

@@ -58,8 +58,20 @@ 
 
 // this is basically a reactor
 interface WordPoints_Entity_Change_ListenerI {
+
+	/**
+	 * @return void
+	 */
 	public function created( WordPoints_Entity $entity );
+
+	/**
+	 * @return void
+	 */
 	public function updated( WordPoints_Entity $before, WordPoints_Entity $after );
+
+	/**
+	 * @return void
+	 */
 	public function deleted( WordPoints_Entity $entity );
 }
 
@@ -129,6 +141,9 @@ 
 		return true;
 	}
 
+	/**
+	 * @param string $type
+	 */
 	protected function fire_event( $type, WordPoints_Entity $entity ) {
 
 		$args = new WordPoints_Hook_Event_Args( array() );

Spacing +63 added lines, -63 removed lines

@@ -10,19 +10,19 @@ 
 // this is basicaly a router.
 class WordPoints_Entity_chang_trigger implements WordPoints_Entity_Change_ListenerI {
 
-	public function created( WordPoints_Entity $entity ) {
+	public function created(WordPoints_Entity $entity) {
 		// can multiple entities be created at once? not using insert() but via query()
 		// maybe? and how shoudl we handle that, all at once or one by one?
 
 		/** @var WordPoints_Entity_Change_ListenerI[] $listeners */
-		$listeners = wordpoints_apps()->get_sub_app( 'entity_change_listeners' );
-		foreach ( $listeners as $listener ) {
-			$listener->created( $entity );
+		$listeners = wordpoints_apps()->get_sub_app('entity_change_listeners');
+		foreach ($listeners as $listener) {
+			$listener->created($entity);
 		}
 	}
 
 
-	public function updated( WordPoints_Entity $before, WordPoints_Entity $after ) {
+	public function updated(WordPoints_Entity $before, WordPoints_Entity $after) {
 
 		// what if multiple entities are updated at once? do we run each one separatately?
 		// mayb ethat should be left up to the listenter.
@@ -30,7 +30,7 @@ 
 	}
 
 
-	public function deleted( WordPoints_Entity $entity ) {
+	public function deleted(WordPoints_Entity $entity) {
 
 		// what if we need information about this entitie's relationships, etc., that
 		// isn't included inthe entity object?
@@ -58,41 +58,41 @@ 
 
 // this is basically a reactor
 interface WordPoints_Entity_Change_ListenerI {
-	public function created( WordPoints_Entity $entity );
-	public function updated( WordPoints_Entity $before, WordPoints_Entity $after );
-	public function deleted( WordPoints_Entity $entity );
+	public function created(WordPoints_Entity $entity);
+	public function updated(WordPoints_Entity $before, WordPoints_Entity $after);
+	public function deleted(WordPoints_Entity $entity);
 }
 
 class WordPoints_Entity_Change_Listener_Hooks implements WordPoints_Entity_Change_ListenerI {
 
-	public function created( WordPoints_Entity $entity ) {
+	public function created(WordPoints_Entity $entity) {
 
 		// maybe we would have multuiple events with requirements for a single entity?
 		// like comment author and post commentn author targets for the same hook.
 		// but I guess taht is just one event.
 		// what about user register vs user create on MS?
-		if ( $this->matches_requirements( $entity ) ) {
-			$this->fire_event( 'add', $entity );
+		if ($this->matches_requirements($entity)) {
+			$this->fire_event('add', $entity);
 		}
 	}
 
-	public function updated( WordPoints_Entity $before, WordPoints_Entity $after ) {
+	public function updated(WordPoints_Entity $before, WordPoints_Entity $after) {
 
-		if ( $this->matches_requirements( $after ) ) {
-			if ( ! $this->matches_requirements( $before ) ) {
-				$this->fire_event( 'add', $after );
+		if ($this->matches_requirements($after)) {
+			if ( ! $this->matches_requirements($before)) {
+				$this->fire_event('add', $after);
 			}
 		} else {
-			if ( $this->matches_requirements( $before ) ) {
-				$this->fire_event( 'remove', $after );
+			if ($this->matches_requirements($before)) {
+				$this->fire_event('remove', $after);
 			}
 		}
 	}
 
-	public function deleted( WordPoints_Entity $entity ) {
+	public function deleted(WordPoints_Entity $entity) {
 
-		if ( $this->matches_requirements( $entity ) ) {
-			$this->fire_event( 'remove', $entity );
+		if ($this->matches_requirements($entity)) {
+			$this->fire_event('remove', $entity);
 		}
 	}
 
@@ -108,20 +108,20 @@ 
 	// called into play when the entity itself is deleted is moot, because the
 	// reltionships, etc., aren't even taken into account when toggle-off is called,
 	// in the points reactor. other reators might, i guess.
-	protected function matches_requirements( WordPoints_Entity $entity ) {
+	protected function matches_requirements(WordPoints_Entity $entity) {
 
 		/** @var WordPoints_Class_Registry $defaults */
-		$defaults = wordpoints_apps()->get_sub_app( 'entity_possession_defaults' );
+		$defaults = wordpoints_apps()->get_sub_app('entity_possession_defaults');
 
-		$defaults = $defaults->get( $entity->get_slug() );
+		$defaults = $defaults->get($entity->get_slug());
 
-		if ( ! $defaults ) {
+		if ( ! $defaults) {
 			return false;
 		}
 
 		// use conditions here?
-		foreach ( $defaults as $child => $value ) {
-			if ( $entity->get_child( $child ) !== $value ) {
+		foreach ($defaults as $child => $value) {
+			if ($entity->get_child($child) !== $value) {
 				return false;
 			}
 		}
@@ -129,13 +129,13 @@ 
 		return true;
 	}
 
-	protected function fire_event( $type, WordPoints_Entity $entity ) {
+	protected function fire_event($type, WordPoints_Entity $entity) {
 
-		$args = new WordPoints_Hook_Event_Args( array() );
-		$args->add_entity( $entity );
+		$args = new WordPoints_Hook_Event_Args(array());
+		$args->add_entity($entity);
 
 		wordpoints_hooks()->fire(
-			$type . '_entity_' . $entity->get_slug(),
+			$type.'_entity_'.$entity->get_slug(),
 			$args,
 			'toggle_on'
 		);
@@ -144,31 +144,31 @@ 
 
 class WordPoints_Entity_Change_Listener_Points implements WordPoints_Entity_Change_ListenerI {
 
-	public function created( WordPoints_Entity $entity ) {
+	public function created(WordPoints_Entity $entity) {
 
 		$this->process_entityish(
 			$entity
-			, $this->get_settings_for_entity( $entity )
+			, $this->get_settings_for_entity($entity)
 		);
 	}
 
-	protected function process_entityish( WordPoints_EntityishI $entity, $settings ) {
+	protected function process_entityish(WordPoints_EntityishI $entity, $settings) {
 
 		// possibly make this more like extension handling
-		if ( ! $this->meets_conditions( $entity, $settings['conditions'] ) ) {
+		if ( ! $this->meets_conditions($entity, $settings['conditions'])) {
 			return;
 		}
 
 		// possibly make this more like reactor handling.
-		$this->award_points( $settings );
+		$this->award_points($settings);
 
 		// only proces the attributes taht have changed.
 		// acutally, in this case, the entity was just created.
-		if ( $entity instanceof WordPoints_Entity_ParentI ) {
+		if ($entity instanceof WordPoints_Entity_ParentI) {
 
 			// this check runs on attributes only.
-			foreach ( $settings['children'] as $child_slug => $child_settings ) {
-				$this->process_entityish( $entity->get_child( $child_slug ), $child_settings );
+			foreach ($settings['children'] as $child_slug => $child_settings) {
+				$this->process_entityish($entity->get_child($child_slug), $child_settings);
 			}
 
 			// we also need to check for any children, like relationships, that have
@@ -176,7 +176,7 @@ 
 			// for this we need a list of relationships.
 			// if we do two-way relationships, we may need to have infinite-loop
 			// halding here, depending.
-			foreach ( $this->get_related_entities( $entity ) as $child_entity ) {
+			foreach ($this->get_related_entities($entity) as $child_entity) {
 				// If a comment was just created, for example, and we have conditions
 				// on the post entity, that affect the comment author, this allows
 				// us to handle those by pulling up the settings for the post entity
@@ -195,7 +195,7 @@ 
 				// as we've proposed...
 				$this->process_entityish(
 					$child_entity
-					, $this->get_entity_settings( $child_entity )
+					, $this->get_entity_settings($child_entity)
 				);
 			}
 		}
@@ -209,15 +209,15 @@ 
 	// doesn't really make sense.
 	// I think relationships can be created and deleted, but not really updated.
 	// so maybe what we need is a separate api?
-	public function updated( WordPoints_Entity $before, WordPoints_Entity $after ) {
+	public function updated(WordPoints_Entity $before, WordPoints_Entity $after) {
 
-		$settings = $this->get_settings_for_entity( $before );
+		$settings = $this->get_settings_for_entity($before);
 
 		/** @var WordPoints_Class_Registry_ChildrenI $children */
-		$children = wordpoints_entities()->get_sub_app( 'children' );
-		foreach ( $children->get_children_slugs( $before->get_slug() ) as $child_slug ) {
+		$children = wordpoints_entities()->get_sub_app('children');
+		foreach ($children->get_children_slugs($before->get_slug()) as $child_slug) {
 			
-			if ( ! isset( $settings['children'][ $child_slug ] ) ) {
+			if ( ! isset($settings['children'][$child_slug])) {
 				continue;
 			}
 			
@@ -232,23 +232,23 @@ 
 			// but then we'll not be checking even the relationships taht are defined
 			// on the atts. so we have to decide whether those should be handled by
 			// a separate api or not.
-			if ( $before->get_the_attr_value( $child_slug ) === $after->get_the_attr_value( $child_slug ) ) {
+			if ($before->get_the_attr_value($child_slug) === $after->get_the_attr_value($child_slug)) {
 				continue;
 			}
 
 			$this->process_modified_entityish(
-				$before->get_child( $child_slug )
-				, $after->get_child( $child_slug )
-				, $settings['children'][ $child_slug ]
+				$before->get_child($child_slug)
+				, $after->get_child($child_slug)
+				, $settings['children'][$child_slug]
 			);
 		}
 	}
 
-	protected function process_modified_entityish( WordPoints_EntityishI $before, WordPoints_EntityishI $after, $settings ) {
+	protected function process_modified_entityish(WordPoints_EntityishI $before, WordPoints_EntityishI $after, $settings) {
 
-		if ( ! $this->meets_conditions( $before, $settings['conditions'] ) ) {
-			if ( $this->meets_conditions( $after, $settings['conditions'] ) ) {
-				$this->award_points( $settings );
+		if ( ! $this->meets_conditions($before, $settings['conditions'])) {
+			if ($this->meets_conditions($after, $settings['conditions'])) {
+				$this->award_points($settings);
 
 				// also need to process other eneities that are affected by this
 				// change, which may have conditions on this, i.e., that could be
@@ -265,38 +265,38 @@ 
 				// then we do indeed need to run through the children/parents.
 			}
 		} else {
-			if ( ! $this->meets_conditions( $after, $settings['conditions'] ) ) {
-				$this->remove_points( $settings );
+			if ( ! $this->meets_conditions($after, $settings['conditions'])) {
+				$this->remove_points($settings);
 			}
 		}
 	}
 
-	public function deleted( WordPoints_Entity $entity ) {
+	public function deleted(WordPoints_Entity $entity) {
 		// basically the opposite of created().
-		$this->process_entityish_reverse( $entity );
+		$this->process_entityish_reverse($entity);
 	}
 
-	protected function meets_conditions( WordPoints_EntityishI $entityish, $conditions ) {
+	protected function meets_conditions(WordPoints_EntityishI $entityish, $conditions) {
 		// use conditions api
 
 		return false;
 	}
 
-	private function award_points( $settings ) {
+	private function award_points($settings) {
 
-		$hierarchy = new WordPoints_Entity_Hierarchy( $this->eitnty );
+		$hierarchy = new WordPoints_Entity_Hierarchy($this->eitnty);
 		// todo we'll need to introudce entity array targets, possibly.
 		// how will we know how to reverse the target reltinoship chains in the UI
 		// before saving? i guess we'll need to either have a dedicated index fo taht
 		// or else juust look it up by looping through the relationships (though we'd
 		// have to remove the entity array {} part from any one-to-many relationships
 		// and maybe we'd have to add it to others?)
-		$targets = $hierarchy->get_from_hierarchy( $settings['target'] );
+		$targets = $hierarchy->get_from_hierarchy($settings['target']);
 
 		// maybe just one target, maybe several, depending.
 	}
 
-	private function get_related_entities( WordPoints_Entity $entity ) {
+	private function get_related_entities(WordPoints_Entity $entity) {
 
 		// what we need is parental conditions. Conditions that go up the chain,
 		// and look back at parent entities. This way we can handle the info about

src/includes/classes/entity/user/role/name.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return _x( 'Name', 'user role', 'wordpoints' );
+		return _x('Name', 'user role', 'wordpoints');
 	}
 }
 

src/includes/classes/entity/term/id.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return _x( 'ID', 'taxonomy term', 'wordpoints' );
+		return _x('ID', 'taxonomy term', 'wordpoints');
 	}
 }
 // EOF

src/includes/classes/wpdb/wrapper.php

Spacing +51 added lines, -51 removed lines

@@ -11,60 +11,60 @@ 
 
 	protected $wpdb;
 
-	public function __construct( $wpdb ) {
+	public function __construct($wpdb) {
 		$this->wpdb = $wpdb;
 	}
 
-	public function __get( $name ) {
+	public function __get($name) {
 		return $this->wpdb->$name;
 	}
 
-	public function __set( $name, $value ) {
+	public function __set($name, $value) {
 		$this->wpdb->$name = $value;
 	}
 
-	public function __isset( $name ) {
-		return isset( $this->wpdb->$name );
+	public function __isset($name) {
+		return isset($this->wpdb->$name);
 	}
 
-	public function __unset( $name ) {
-		unset( $this->wpdb->$name );
+	public function __unset($name) {
+		unset($this->wpdb->$name);
 	}
 
-	public function __call( $name, $arguments ) {
+	public function __call($name, $arguments) {
 
-		$wrappers = wordpoints_apps()->get_sub_app( 'wpdb_wrappers' );
-		$slugs = $wrappers->get_child_slugs( $name );
+		$wrappers = wordpoints_apps()->get_sub_app('wpdb_wrappers');
+		$slugs = $wrappers->get_child_slugs($name);
 		$should_listen = $query = false;
 
 
-		switch ( $name ) {
+		switch ($name) {
 
 			case 'insert':
 //			case 'replace':
-				$query = new WordPoints_WPDB_Query_Data_Array( $this->args[0] );
+				$query = new WordPoints_WPDB_Query_Data_Array($this->args[0]);
 			break;
 
 			case 'update':
-				$query =  new WordPoints_WPDB_Query_Data_Array( $this->args[0], $this->args[2] );
+				$query = new WordPoints_WPDB_Query_Data_Array($this->args[0], $this->args[2]);
 			break;
 
 			case 'delete':
-				$query =  new WordPoints_WPDB_Query_Data_Array( $this->args[0], $this->args[1] );
+				$query = new WordPoints_WPDB_Query_Data_Array($this->args[0], $this->args[1]);
 			break;
 
 			case 'query':
-				$query =  new WordPoints_WPDB_Query_Data_SQL( $this->args[0] );
+				$query = new WordPoints_WPDB_Query_Data_SQL($this->args[0]);
 			break;
 		}
 
-		if ( $query ) {
+		if ($query) {
 			/** @var WordPoints_WPDB_Query_WrapperI $wrapper */
-			$wrapper = $wrappers->get( $name, $query );
+			$wrapper = $wrappers->get($name, $query);
 
 			$result = $wrapper->execute();
 		} else {
-			$result = call_user_func_array( array( $this->wpdb, $name ), $arguments );
+			$result = call_user_func_array(array($this->wpdb, $name), $arguments);
 		}
 
 		return $result;
@@ -84,7 +84,7 @@ 
 
 class WordPoints_MySQL_Query_Parser_Simple_Insert implements WordPoints_MySQL_Query_Parser_SimpleI {
 
-	public function __construct( $sql ) {
+	public function __construct($sql) {
 		$this->sql = $sql;
 	}
 
@@ -99,7 +99,7 @@ 
 		// columns will not be needed. then the other code can decide what to do.
 		// although I guess sense we have the tablel name we could reun the query
 		// ourselves if we need/want to.
-		if ( stripos( $this->sql, 'ON DUPLICATE KEY UPDATE' ) ) {
+		if (stripos($this->sql, 'ON DUPLICATE KEY UPDATE')) {
 
 		}
 
@@ -119,7 +119,7 @@ 
 	 */
 	protected $parser;
 
-	public function __construct( $sql ) {
+	public function __construct($sql) {
 		$this->sql = $sql;
 		$this->parser = $this->get_parser();
 	}
@@ -130,7 +130,7 @@ 
 		// of query it is.
 		// maybe the parser will actually be passed in, and the query type deterimed
 		// by teh method wrapper.
-		$parser = new WordPoints_MySQL_Query_Parser_Simple_Insert( $this->sql );
+		$parser = new WordPoints_MySQL_Query_Parser_Simple_Insert($this->sql);
 		// maybe we don't even need to parse.
 		return $parser;
 	}
@@ -149,7 +149,7 @@ 
 	protected $table_name;
 	protected $where_data;
 
-	public function __construct( $table_name, $where_data = array() ) {
+	public function __construct($table_name, $where_data = array()) {
 		$this->table_name = $table_name;
 		$this->where_data = $where_data;
 	}
@@ -164,7 +164,7 @@ 
 }
 
 interface WordPoints_WPDB_Query_WrapperI {
-	public function __construct( $slug, $args, $wpdb );
+	public function __construct($slug, $args, $wpdb);
 	public function execute();
 
 }
@@ -185,7 +185,7 @@ 
 	protected $result;
 	protected $backed_up_wpdb_vars;
 
-	public function __construct( $slug, $args, $wpdb ) {
+	public function __construct($slug, $args, $wpdb) {
 		$this->slug = $slug;
 		$this->args = $args;
 		$this->wpdb = $wpdb;
@@ -194,20 +194,20 @@ 
 
 	protected function get_query_data() {
 
-		switch ( $this->slug ) {
+		switch ($this->slug) {
 
 			case 'insert':
 				//			case 'replace':
-				return new WordPoints_WPDB_Query_Data_Array( $this->args[0] );
+				return new WordPoints_WPDB_Query_Data_Array($this->args[0]);
 
 			case 'update':
-				return new WordPoints_WPDB_Query_Data_Array( $this->args[0], $this->args[2] );
+				return new WordPoints_WPDB_Query_Data_Array($this->args[0], $this->args[2]);
 
 			case 'delete':
-				return new WordPoints_WPDB_Query_Data_Array( $this->args[0], $this->args[1] );
+				return new WordPoints_WPDB_Query_Data_Array($this->args[0], $this->args[1]);
 
 			case 'query':
-				return new WordPoints_WPDB_Query_Data_SQL( $this->args[0] );
+				return new WordPoints_WPDB_Query_Data_SQL($this->args[0]);
 		}
 
 		return false;
@@ -217,13 +217,13 @@ 
 
 		$should_listen = $this->should_listen();
 
-		if ( $should_listen ) {
+		if ($should_listen) {
 			$this->before_query();
 		}
 
-		$this->result = call_user_func_array( array( $this->wpdb, $this->slug ), $this->args );
+		$this->result = call_user_func_array(array($this->wpdb, $this->slug), $this->args);
 
-		if ( $should_listen && $this->was_successful_query() ) {
+		if ($should_listen && $this->was_successful_query()) {
 			$this->backup_wpdb_vars();
 			$this->after_query();
 			$this->restore_wpdb_vars();
@@ -252,8 +252,8 @@ 
 
 		// we could check for the availableility of eahc valu here in case of dropins
 		// or they become not available in the future.
-		foreach ( $to_backup as $property ) {
-			$this->backed_up_wpdb_vars[ $property ] = $this->wpdb->$property;
+		foreach ($to_backup as $property) {
+			$this->backed_up_wpdb_vars[$property] = $this->wpdb->$property;
 		}
 	}
 
@@ -263,7 +263,7 @@ 
 		// with the backed up result.
 		$this->wpdb->flush();
 
-		foreach ( $this->backed_up_wpdb_vars as $property => $value ) {
+		foreach ($this->backed_up_wpdb_vars as $property => $value) {
 			$this->wpdb->$property = $value;
 		}
 	}
@@ -301,18 +301,18 @@ 
 	 */
 	protected function is_entity_table() {
 
-		$this->entity = 'new entity object' . $this->data->get_table_name();
+		$this->entity = 'new entity object'.$this->data->get_table_name();
 
 		return true;
 	}
 
 	public function should_listen() {
 
-		if ( ! $this->data ) {
+		if ( ! $this->data) {
 			return false;
 		}
 
-		if ( $this->is_entity_table() ) {
+		if ($this->is_entity_table()) {
 			return false;
 		}
 
@@ -320,7 +320,7 @@ 
 	}
 
 	protected function was_successful_query() {
-		return (bool) wordpoints_posint( $not_by_ref = $this->wpdb->num_rows );
+		return (bool) wordpoints_posint($not_by_ref = $this->wpdb->num_rows);
 	}
 }
 
@@ -330,7 +330,7 @@ 
 	protected $args;
 	protected $wpdb;
 
-	public function __construct( $slug, $args, $wpdb ) {
+	public function __construct($slug, $args, $wpdb) {
 		$this->slug = $slug;
 		$this->args = $args;
 		$this->wpdb = $wpdb;
@@ -341,10 +341,10 @@ 
 		// check if this is a query that we want to listen to.
 		// if so, call execute on a particular child.
 		// otherwise, just call execute.
-		if ( $this->get_query_type() ) {
+		if ($this->get_query_type()) {
 
 		} else {
-			$result = call_user_func_array( array( $this->wpdb, $this->slug ), $this->args );
+			$result = call_user_func_array(array($this->wpdb, $this->slug), $this->args);
 		}
 
 		return $result;
@@ -369,15 +369,15 @@ 
 
 		$ids = $this->get_entity_ids();
 
-		$this->entities = new WordPoints_Entity_Array( $this->entity->get_slug() );
-		$this->entities->set_the_value( $ids );
+		$this->entities = new WordPoints_Entity_Array($this->entity->get_slug());
+		$this->entities->set_the_value($ids);
 	}
 
 	protected function get_entity_ids() {
 		$where = $this->data->get_where_clause();
-		$id_field = wordpoints_escape_mysql_identifier( $this->entity->get_id_field() );
-		$table_name = wordpoints_escape_mysql_identifier( $this->data->get_table_name() );
-		return $this->wpdb->get_col( "SELECT {$id_field} FROM {$table_name} {$where}" );
+		$id_field = wordpoints_escape_mysql_identifier($this->entity->get_id_field());
+		$table_name = wordpoints_escape_mysql_identifier($this->data->get_table_name());
+		return $this->wpdb->get_col("SELECT {$id_field} FROM {$table_name} {$where}");
 	}
 
 	protected function after_query() {
@@ -396,13 +396,13 @@ 
 		$where_clause = $this->data->get_where_clause();
 
 		// get a before snapshot, if any matching are found.
-		if ( $where_clause ) {
+		if ($where_clause) {
 
 		}
 	}
 
 	protected function after_query() {
-		$this->entity->set_the_value( $this->wpdb->insert_id );
+		$this->entity->set_the_value($this->wpdb->insert_id);
 		// I guess we invoke the api to run the action now?
 		$this->trigger_entity_create_actions();
 	}
@@ -413,7 +413,7 @@ 
 
 	protected function after_query() {
 		// I guess we invoke the api to run the action now?
-		$this->trigger_entity_delete_actions( $this->entities );
+		$this->trigger_entity_delete_actions($this->entities);
 	}
 }
 

Doc Comments +16 added lines

@@ -74,11 +74,23 @@ 
 
 interface WordPoints_WPDB_Query_Data {
 	public function get_table_name();
+
+	/**
+	 * @return void
+	 */
 	public function get_where_clause();
 }
 
 interface WordPoints_MySQL_Query_Parser_SimpleI {
+
+	/**
+	 * @return void
+	 */
 	public function get_table_name();
+
+	/**
+	 * @return string
+	 */
 	public function get_where_clause();
 }
 
@@ -164,6 +176,10 @@ 
 }
 
 interface WordPoints_WPDB_Query_WrapperI {
+
+	/**
+	 * @return void
+	 */
 	public function __construct( $slug, $args, $wpdb );
 	public function execute();
 

src/includes/classes/hook/settings.php

Spacing +31 added lines, -31 removed lines

@@ -8,13 +8,13 @@ 
  */
 
 interface WordPoints_Hook_Reaction_SettingI {
-	public function __construct( $slug );
+	public function __construct($slug);
 	public function get_type();
 	public function get_label();
 	public function is_required();
-	public function get( WordPoints_Hook_ReactionI $reaction );
-	public function validate( $value, WordPoints_Hook_Reaction_Validator $validator );
-	public function save( $value, WordPoints_Hook_ReactionI $reaction, $is_new );
+	public function get(WordPoints_Hook_ReactionI $reaction);
+	public function validate($value, WordPoints_Hook_Reaction_Validator $validator);
+	public function save($value, WordPoints_Hook_ReactionI $reaction, $is_new);
 }
 
 class WordPoints_Hook_Reaction_Setting implements WordPoints_Hook_Reaction_SettingI {
@@ -23,7 +23,7 @@ 
 	protected $type = 'hidden';
 	protected $required = true;
 
-	public function __construct( $slug ) {
+	public function __construct($slug) {
 		$this->slug = $slug;
 	}
 
@@ -39,17 +39,17 @@ 
 		return $this->required;
 	}
 
-	public function get( WordPoints_Hook_ReactionI $reaction ) {
-		return $reaction->get_meta( $this->slug );
+	public function get(WordPoints_Hook_ReactionI $reaction) {
+		return $reaction->get_meta($this->slug);
 	}
 
-	public function validate( $value, WordPoints_Hook_Reaction_Validator $validator ) {
+	public function validate($value, WordPoints_Hook_Reaction_Validator $validator) {
 
-		if ( $this->is_required() && '' === trim( $value ) ) {
+		if ($this->is_required() && '' === trim($value)) {
 
 			$validator->add_error(
 				sprintf(
-					__( '%s cannot be empty.', 'wordpoints' )
+					__('%s cannot be empty.', 'wordpoints')
 					, $this->get_label()
 				)
 				, $this->slug
@@ -59,8 +59,8 @@ 
 		return $value;
 	}
 
-	public function save( $value, WordPoints_Hook_ReactionI $reaction, $is_new ) {
-		$reaction->update_meta( $this->slug, $value );
+	public function save($value, WordPoints_Hook_ReactionI $reaction, $is_new) {
+		$reaction->update_meta($this->slug, $value);
 	}
 }
 //
@@ -82,12 +82,12 @@ 
 
 		$fields = array();
 
-		foreach ( $this->settings as $slug => $class ) {
+		foreach ($this->settings as $slug => $class) {
 
 			/** @var WordPoints_Hook_Reaction_SettingI $setting */
-			$setting = new $class( $slug );
+			$setting = new $class($slug);
 
-			$fields[ $slug ] = array(
+			$fields[$slug] = array(
 				'type' => $setting->get_type(),
 				'label' => $setting->get_label(),
 				'required' => $setting->is_required(),
@@ -97,29 +97,29 @@ 
 		return $fields;
 	}
 
-	public function save_settings( $reaction, $settings, $is_new ) {
+	public function save_settings($reaction, $settings, $is_new) {
 
-		foreach ( $this->settings as $slug => $class ) {
+		foreach ($this->settings as $slug => $class) {
 
 			/** @var WordPoints_Hook_Reaction_SettingI $setting */
-			$setting = new $class( $slug );
+			$setting = new $class($slug);
 
-			$value = isset( $settings[ $slug ] ) ? $settings[ $slug ] : null;
+			$value = isset($settings[$slug]) ? $settings[$slug] : null;
 
-			$setting->save( $value, $reaction, $is_new );
+			$setting->save($value, $reaction, $is_new);
 		}
 	}
 
-	public function validate_settings( $settings, $validator ) {
+	public function validate_settings($settings, $validator) {
 
-		foreach ( $this->settings as $slug => $class ) {
+		foreach ($this->settings as $slug => $class) {
 
 			/** @var WordPoints_Hook_Reaction_SettingI $setting */
-			$setting = new $class( $slug );
+			$setting = new $class($slug);
 
-			$value = isset( $settings[ $slug ] ) ? $settings[ $slug ] : null;
+			$value = isset($settings[$slug]) ? $settings[$slug] : null;
 
-			$settings[ $slug ] = $setting->validate( $value, $validator );
+			$settings[$slug] = $setting->validate($value, $validator);
 		}
 
 		return $settings;
@@ -133,8 +133,8 @@ 
  *
  * @param WordPoints_Hooks $hooks
  */
-function wordpoints_hook_settings_app( $hooks ) {
-	$hooks->sub_apps()->register( 'settings', 'WordPoints_Hook_Settings' );
+function wordpoints_hook_settings_app($hooks) {
+	$hooks->sub_apps()->register('settings', 'WordPoints_Hook_Settings');
 }
 //add_action( 'wordpoints_hooks_init', 'wordpoints_hook_settings_app' );
 
@@ -145,12 +145,12 @@ 
  *
  * @param WordPoints_Hook_Settings $settings
  */
-function wordpoints_register_hook_settings( $settings ) {
+function wordpoints_register_hook_settings($settings) {
 
-	add_action( 'wordpoints_hook_reaction_save', array( $settings, 'save_settings' ), 10, 3 );
-	add_filter( 'wordpoints_hook_reaction_validate', array( $settings, 'validate_settings' ), 10, 2 );
+	add_action('wordpoints_hook_reaction_save', array($settings, 'save_settings'), 10, 3);
+	add_filter('wordpoints_hook_reaction_validate', array($settings, 'validate_settings'), 10, 2);
 
-	$settings->register( 'description', 'WordPoints_Hook_Reaction_Setting_Description' );
+	$settings->register('description', 'WordPoints_Hook_Reaction_Setting_Description');
 }
 //add_action( 'wordpoints_hook_settings_init', 'wordpoints_register_hook_settings' );
 

src/hooks-api.php

Spacing +3 added lines, -3 removed lines

@@ -44,18 +44,18 @@
  *
  * @since 1.0.0
  */
-require_once( dirname( __FILE__ ) . '/includes/functions.php' );
+require_once(dirname(__FILE__).'/includes/functions.php');
 
 /**
  * The module's actions and filters.
  *
  * @since 1.0.0
  */
-require_once( dirname( __FILE__ ) . '/includes/actions.php' );
+require_once(dirname(__FILE__).'/includes/actions.php');
 
 // Register the classes to autoload.
 WordPoints_Class_Autoloader::register_dir(
-	dirname( __FILE__ ) . '/includes/classes'
+	dirname(__FILE__).'/includes/classes'
 	, 'WordPoints_'
 );
 

src/includes/classes/index.php

Spacing +23 added lines, -23 removed lines

@@ -10,29 +10,29 @@
  */
 
 // auto-generated {
-$dir = dirname( __FILE__ );
-require_once( $dir . '/specedi.php' );
-require_once( $dir . '/hook/retroactive/queryi.php' );
-require_once( $dir . '/hook/retroactive/query/modifieri.php' );
-require_once( $dir . '/spec.php' );
-require_once( $dir . '/entity/change.php' );
-require_once( $dir . '/entity/post/terms.php' );
-require_once( $dir . '/entity/post/type.php' );
-require_once( $dir . '/entity/post/type/name.php' );
-require_once( $dir . '/entity/post/type/relationship.php' );
-require_once( $dir . '/entity/site.php' );
-require_once( $dir . '/entity/term.php' );
-require_once( $dir . '/entity/term/id.php' );
-require_once( $dir . '/entity/user/role/name.php' );
-require_once( $dir . '/hierarchy.php' );
-require_once( $dir . '/hook/arg/current/post.php' );
-require_once( $dir . '/hook/arg/current/site.php' );
-require_once( $dir . '/hook/retroactive/conditions.php' );
-require_once( $dir . '/hook/retroactive/query.php' );
-require_once( $dir . '/hook/retroactive/queryable.php' );
-require_once( $dir . '/hook/settings.php' );
-require_once( $dir . '/query/builder/db/mysql.php' );
-require_once( $dir . '/wpdb/wrapper.php' );
+$dir = dirname(__FILE__);
+require_once($dir.'/specedi.php');
+require_once($dir.'/hook/retroactive/queryi.php');
+require_once($dir.'/hook/retroactive/query/modifieri.php');
+require_once($dir.'/spec.php');
+require_once($dir.'/entity/change.php');
+require_once($dir.'/entity/post/terms.php');
+require_once($dir.'/entity/post/type.php');
+require_once($dir.'/entity/post/type/name.php');
+require_once($dir.'/entity/post/type/relationship.php');
+require_once($dir.'/entity/site.php');
+require_once($dir.'/entity/term.php');
+require_once($dir.'/entity/term/id.php');
+require_once($dir.'/entity/user/role/name.php');
+require_once($dir.'/hierarchy.php');
+require_once($dir.'/hook/arg/current/post.php');
+require_once($dir.'/hook/arg/current/site.php');
+require_once($dir.'/hook/retroactive/conditions.php');
+require_once($dir.'/hook/retroactive/query.php');
+require_once($dir.'/hook/retroactive/queryable.php');
+require_once($dir.'/hook/settings.php');
+require_once($dir.'/query/builder/db/mysql.php');
+require_once($dir.'/wpdb/wrapper.php');
 // }
 
 // EOF

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

Spacing +2 added lines, -2 removed lines

@@ -33,14 +33,14 @@
 	 * @since 1.0.0
 	 */
 	public function get_title() {
-		return __( 'Content Type', 'wordpoints' );
+		return __('Content Type', 'wordpoints');
 	}
 
 	/**
 	 * @since 1.0.0
 	 */
 	public function get_storage_array() {
-		return get_post_types( array( 'public' => true ), 'labels' );
+		return get_post_types(array('public' => true), 'labels');
 	}
 }