eventespresso / event-espresso-core

core/db_models/strategies/EE_Restriction_Generator_Meta.strategy.php

Spacing +13 added lines, -13 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 /**
@@ -14,7 +14,7 @@ 
  * @author				Mike Nelson
  *
  */
-class EE_Restriction_Generator_Meta extends EE_Restriction_Generator_Base{
+class EE_Restriction_Generator_Meta extends EE_Restriction_Generator_Base {
 
 	/**
 	 *
@@ -33,7 +33,7 @@ 
 	 * @param string $key_field_name
 	 * @param string $value_field_name
 	 */
-	public function __construct( $key_field_name, $value_field_name ) {
+	public function __construct($key_field_name, $value_field_name) {
 		$this->_key_field_name = $key_field_name;
 		$this->_value_field_name = $value_field_name;
 	}
@@ -45,21 +45,21 @@ 
 	 * @throws EE_Error
 	 */
 	protected function _generate_restrictions() {
-		$whitelisted_meta_keys = apply_filters( 'FHEE__EE_Restriction_Generator_Meta___generate_restrictions__whitelisted_meta_keys', array() );
-		$blacklisted_meta_keys = apply_filters( 'FHEE__EE_Restriction_Generator_Meta___generate_restrictions__blacklisted_meta_keys', array() );
+		$whitelisted_meta_keys = apply_filters('FHEE__EE_Restriction_Generator_Meta___generate_restrictions__whitelisted_meta_keys', array());
+		$blacklisted_meta_keys = apply_filters('FHEE__EE_Restriction_Generator_Meta___generate_restrictions__blacklisted_meta_keys', array());
 		$conditions = array(
-			$this->_key_field_name => array( 'NOT_LIKE', "\\\\_%" ),//each slash is escaped because we are using double quotes, and 
+			$this->_key_field_name => array('NOT_LIKE', "\\\\_%"), //each slash is escaped because we are using double quotes, and 
 			//stripslashes will be called on this because the models assume this is from user input
-			$this->_value_field_name => array( 'NOT_REGEXP', '^[aOs]:[\d]:.*$')					
+			$this->_value_field_name => array('NOT_REGEXP', '^[aOs]:[\d]:.*$')					
 		);
-		if( ! empty( $blacklisted_meta_keys ) ) {
-			$conditions[ $this->_key_field_name . '*blacklisted' ] = array( 'NOT_IN', $blacklisted_meta_keys );
+		if ( ! empty($blacklisted_meta_keys)) {
+			$conditions[$this->_key_field_name.'*blacklisted'] = array('NOT_IN', $blacklisted_meta_keys);
 		}
-		if( ! empty( $whitelisted_meta_keys ) ) {
+		if ( ! empty($whitelisted_meta_keys)) {
 			$conditions = array(
 				'OR*whitelisted-or-normal' => array(
 					'AND' => $conditions,
-					$this->_key_field_name . '*whitelisted' => array( 'IN', $whitelisted_meta_keys ) 
+					$this->_key_field_name.'*whitelisted' => array('IN', $whitelisted_meta_keys) 
 				)
 			);
 		}
@@ -67,7 +67,7 @@ 
 			//only allow access to non-protected metas if they're an admin
 			EE_Restriction_Generator_Base::get_default_restrictions_cap() => new EE_Return_None_Where_Conditions(),
 			//don't allow access to protected metas to anyone. If they want that, don't apply caps to the query
-			'apply-to-all-queries-using-caps' => new EE_Default_Where_Conditions( $conditions ),
+			'apply-to-all-queries-using-caps' => new EE_Default_Where_Conditions($conditions),
 		);
 	}
 }

core/db_models/strategies/EE_CPT_Minimum_Where_Conditions.strategy.php

Doc Comments +4 added lines, -1 removed lines

@@ -7,6 +7,10 @@ 
 
 	protected $_post_type;
 	protected $_meta_field;
+
+	/**
+	 * @param string $post_type
+	 */
 	function __construct($post_type, $meta_field_to_chk = ''){
 		$this->_post_type = $post_type;
 		$this->_meta_field = $meta_field_to_chk;
@@ -27,7 +31,6 @@ 
 	}
 	/**
 	 * Gets the where default where conditions for a custom post type model
-	 * @param string $model_relation_path. Eg, from Event to Payment, this should be "Registration.Transaction.Payment"
 	 * @return array like EEM_Base::get_all's $query_params's index [0] (where conditions)
 	 */
 	protected function _get_default_where_conditions() {

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 /**
  *
@@ -15,11 +15,11 @@ 
  * @since		 	   4.8.29.rc.010
  *
  */
-class EE_CPT_Minimum_Where_Conditions extends EE_Default_Where_Conditions{
+class EE_CPT_Minimum_Where_Conditions extends EE_Default_Where_Conditions {
 
 	protected $_post_type;
 	protected $_meta_field;
-	function __construct($post_type, $meta_field_to_chk = ''){
+	function __construct($post_type, $meta_field_to_chk = '') {
 		$this->_post_type = $post_type;
 		$this->_meta_field = $meta_field_to_chk;
 	}
@@ -29,10 +29,10 @@ 
 	 * @param string $column column name
 	 * @return EE_Model_Field_Base
 	 */
-	protected function _get_field_on_column($column){
+	protected function _get_field_on_column($column) {
 		$all_fields = $this->_model->field_settings(true);
-		foreach($all_fields as $field_name => $field_obj){
-			if($column == $field_obj->get_table_column()){
+		foreach ($all_fields as $field_name => $field_obj) {
+			if ($column == $field_obj->get_table_column()) {
 				return $field_obj;
 			}
 		}

Indentation +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
 /**
  *
  * Class EE_CPT_Minimum_Where_Conditions
-  * 
+ * 
  * Strategy specifically for adding where conditions specific to CPT models.
  * But only sets the minimum, so any row of the right type will get used
  *

core/db_models/strategies/EE_CPT_Where_Conditions.strategy.php

Doc Comments -1 removed lines

@@ -17,7 +17,6 @@
 class EE_CPT_Where_Conditions extends EE_CPT_Minimum_Where_Conditions{
 	/**
 	 * Gets the where default where conditions for a custom post type model
-	 * @param string $model_relation_path. Eg, from Event to Payment, this should be "Registration.Transaction.Payment"
 	 * @return array like EEM_Base::get_all's $query_params's index [0] (where conditions)
 	 */
 	protected function _get_default_where_conditions() {

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 /**
  *
@@ -14,7 +14,7 @@ 
  * * @since		 	   4.6.0
  *
  */
-class EE_CPT_Where_Conditions extends EE_CPT_Minimum_Where_Conditions{
+class EE_CPT_Where_Conditions extends EE_CPT_Minimum_Where_Conditions {
 	/**
 	 * Gets the where default where conditions for a custom post type model
 	 * @param string $model_relation_path. Eg, from Event to Payment, this should be "Registration.Transaction.Payment"
@@ -25,7 +25,7 @@ 
 		return array_merge( 
 			parent::_get_default_where_conditions(),
 			array( 
-				$status_field->get_name() => array('NOT IN',array('auto-draft','trash') )
+				$status_field->get_name() => array('NOT IN', array('auto-draft', 'trash'))
 			)
 		);
 	}

Indentation +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
 /**
  *
  * Class EE_Default_Where_Conditions
-  * 
+ * 
  * Strategy specifically for adding where conditions specific to CPT models.
  *
  * @package         Event Espresso

caffeinated/modules/recaptcha/ReCaptcha/RequestMethod.php

Indentation +7 added lines, -7 removed lines

@@ -32,11 +32,11 @@
 interface RequestMethod
 {
 
-    /**
-     * Submit the request with the specified parameters.
-     *
-     * @param RequestParameters $params Request parameters
-     * @return string Body of the reCAPTCHA response
-     */
-    public function submit(RequestParameters $params);
+	/**
+	 * Submit the request with the specified parameters.
+	 *
+	 * @param RequestParameters $params Request parameters
+	 * @return string Body of the reCAPTCHA response
+	 */
+	public function submit(RequestParameters $params);
 }

core/libraries/repositories/ObjectInfoArrayKeyStrategy.php

Doc Comments +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 	* @access protected
 	* @param object $object
 	* @param mixed  $info
-	* @return bool
+	* @return boolean|null
 	*/
 	public function setObjectInfo( $object, $info = null ) {
 		$info = is_array( $info ) ? $info : array( 'ID' => spl_object_hash( $object ) );

Spacing +18 added lines, -18 removed lines

@@ -38,7 +38,7 @@ 
 	/**
 	* @param EEI_Collection $object_collection
 	*/
-	public function setCollection( EEI_Collection $object_collection ) {
+	public function setCollection(EEI_Collection $object_collection) {
 		$this->object_collection = $object_collection;
 	}
 
@@ -55,12 +55,12 @@ 
 	* @param mixed  $info
 	* @return bool
 	*/
-	public function setObjectInfo( $object, $info = null ) {
-		$info = is_array( $info ) ? $info : array( 'ID' => spl_object_hash( $object ) );
+	public function setObjectInfo($object, $info = null) {
+		$info = is_array($info) ? $info : array('ID' => spl_object_hash($object));
 		$this->object_collection->rewind();
-		while ( $this->object_collection->valid() ) {
-			if ( $object == $this->object_collection->current() ) {
-				$this->object_collection->setInfo( $info );
+		while ($this->object_collection->valid()) {
+			if ($object == $this->object_collection->current()) {
+				$this->object_collection->setInfo($info);
 				$this->object_collection->rewind();
 				return;
 			}
@@ -79,22 +79,22 @@ 
 	* @param array $array_of_key_value_pairings
 	* @return null|object
 	*/
-	public function getObjectByInfo( $array_of_key_value_pairings ) {
+	public function getObjectByInfo($array_of_key_value_pairings) {
 		$this->object_collection->rewind();
-		while ( $this->object_collection->valid() ) {
+		while ($this->object_collection->valid()) {
 			$currentInfo = $this->object_collection->getInfo();
-			if ( ! is_array( $currentInfo ) ) {
+			if ( ! is_array($currentInfo)) {
 				$this->object_collection->next();
 				continue;
 			}
 			$found = true;
-			foreach ( $array_of_key_value_pairings as $key => $value ) {
-				if ( ! isset( $currentInfo[ $key ] ) || $currentInfo[ $key ] !== $value ) {
+			foreach ($array_of_key_value_pairings as $key => $value) {
+				if ( ! isset($currentInfo[$key]) || $currentInfo[$key] !== $value) {
 					$found = false;
 					break;
 				}
 			}
-			if ( ! $found ) {
+			if ( ! $found) {
 				continue;
 			}
 			$object = $this->object_collection->current();
@@ -115,22 +115,22 @@ 
 	 * @param $array_of_key_value_pairings
 	 * @return void
 	 */
-	public function setCurrentByInfo( $array_of_key_value_pairings ) {
+	public function setCurrentByInfo($array_of_key_value_pairings) {
 		$this->object_collection->rewind();
-		while ( $this->object_collection->valid() ) {
+		while ($this->object_collection->valid()) {
 			$currentInfo = $this->object_collection->getInfo();
-			if ( ! is_array( $currentInfo ) ) {
+			if ( ! is_array($currentInfo)) {
 				$this->object_collection->next();
 				continue;
 			}
 			$found = true;
-			foreach ( $array_of_key_value_pairings as $key => $value ) {
-				if ( ! isset( $currentInfo[ $key ] ) || $currentInfo[ $key ] !== $value ) {
+			foreach ($array_of_key_value_pairings as $key => $value) {
+				if ( ! isset($currentInfo[$key]) || $currentInfo[$key] !== $value) {
 					$found = false;
 					break;
 				}
 			}
-			if ( ! $found ) {
+			if ( ! $found) {
 				$this->object_collection->next();
 			}
 			break;

Indentation +22 added lines, -22 removed lines

@@ -29,15 +29,15 @@ 
 
 
 	/**
-	* @type ObjectRepository $object_collection
-	*/
+	 * @type ObjectRepository $object_collection
+	 */
 	protected $object_collection;
 
 
 
 	/**
-	* @param EEI_Collection $object_collection
-	*/
+	 * @param EEI_Collection $object_collection
+	 */
 	public function setCollection( EEI_Collection $object_collection ) {
 		$this->object_collection = $object_collection;
 	}
@@ -45,16 +45,16 @@ 
 
 
 	/**
-	* setObjectInfo
-	*
-	* Sets the data associated with an object in the SplObjectStorage
-	* if no $info is supplied, then the spl_object_hash() is used
-	*
-	* @access protected
-	* @param object $object
-	* @param mixed  $info
-	* @return bool
-	*/
+	 * setObjectInfo
+	 *
+	 * Sets the data associated with an object in the SplObjectStorage
+	 * if no $info is supplied, then the spl_object_hash() is used
+	 *
+	 * @access protected
+	 * @param object $object
+	 * @param mixed  $info
+	 * @return bool
+	 */
 	public function setObjectInfo( $object, $info = null ) {
 		$info = is_array( $info ) ? $info : array( 'ID' => spl_object_hash( $object ) );
 		$this->object_collection->rewind();
@@ -71,14 +71,14 @@ 
 
 
 	/**
-	* getObjectByInfo
-	*
-	* finds and returns an object in the repository based on the info that was set using addObject()
-	*
-	* @access protected
-	* @param array $array_of_key_value_pairings
-	* @return null|object
-	*/
+	 * getObjectByInfo
+	 *
+	 * finds and returns an object in the repository based on the info that was set using addObject()
+	 *
+	 * @access protected
+	 * @param array $array_of_key_value_pairings
+	 * @return null|object
+	 */
 	public function getObjectByInfo( $array_of_key_value_pairings ) {
 		$this->object_collection->rewind();
 		while ( $this->object_collection->valid() ) {

core/services/cart/Cart.php

Doc Comments +3 added lines

@@ -68,6 +68,9 @@
 
 
 
+	 /**
+	  * @param string $ID
+	  */
 	 function __construct(
 		 $ID,
 		 CartItemRepository $cartItemRepository,

Spacing +49 added lines, -49 removed lines

@@ -75,11 +75,11 @@ 
 		 CartTotal $cartTotal,
 		 \DateTime $created = null
 	 ) {
-		 $this->ID 		 		= $ID;
-		 $this->items  			= $cartItemRepository;
+		 $this->ID = $ID;
+		 $this->items = $cartItemRepository;
 		 $this->cartCalculators = $cartCalculatorRepository;
-		 $this->cartTotal 		= $cartTotal;
-		 $this->setCreated( $created );
+		 $this->cartTotal = $cartTotal;
+		 $this->setCreated($created);
 	 }
 
 
@@ -102,8 +102,8 @@ 
 	 /**
 	  * @param boolean $open
 	  */
-	 public function setOpen( $open = true ) {
-		 $this->open = filter_var( $open, FILTER_VALIDATE_BOOLEAN );
+	 public function setOpen($open = true) {
+		 $this->open = filter_var($open, FILTER_VALIDATE_BOOLEAN);
 	 }
 
 
@@ -112,7 +112,7 @@ 
 	  * sets the cart open status to false
 	  */
 	 public function closeCart() {
-		 $this->setOpen( false );
+		 $this->setOpen(false);
 	 }
 
 
@@ -121,9 +121,9 @@ 
 	  * @access protected
 	  * @param \DateTime $created
 	  */
-	 protected function setCreated( \DateTime $created = null ) {
-		 if ( ! $created instanceof \DateTime ) {
-			 $created = new \DateTime( 'now', new \DateTimeZone( 'UTC' ) );
+	 protected function setCreated(\DateTime $created = null) {
+		 if ( ! $created instanceof \DateTime) {
+			 $created = new \DateTime('now', new \DateTimeZone('UTC'));
 		 }
 		 $this->created = $created;
 	 }
@@ -143,9 +143,9 @@ 
 	  * @access protected
 	  * @param \DateTime $updated
 	  */
-	 protected function setUpdated( \DateTime $updated = null ) {
-		 if ( ! $updated instanceof \DateTime ) {
-			 $updated = new \DateTime( 'now', new \DateTimeZone( 'UTC' ) );
+	 protected function setUpdated(\DateTime $updated = null) {
+		 if ( ! $updated instanceof \DateTime) {
+			 $updated = new \DateTime('now', new \DateTimeZone('UTC'));
 		 }
 		 $this->updated = $updated;
 	 }
@@ -167,8 +167,8 @@ 
 	  * @param CartItemOption[] $options
 	  * @return bool
 	  */
-	 public function addTicket( EEI_Ticket $ticket, $quantity = 1, $options = array() ) {
-		 return $this->addItem( $ticket, $quantity, $options, 'Ticket' );
+	 public function addTicket(EEI_Ticket $ticket, $quantity = 1, $options = array()) {
+		 return $this->addItem($ticket, $quantity, $options, 'Ticket');
 	 }
 
 
@@ -177,7 +177,7 @@ 
 	  * @return EEI_Ticket[]
 	  */
 	 public function getTickets() {
-		 return $this->getItems( 'Ticket' );
+		 return $this->getItems('Ticket');
 	 }
 
 
@@ -186,7 +186,7 @@ 
 	  * @return TicketCartItem[]
 	  */
 	 public function getTicketCartItems() {
-		 return $this->getCartItems( 'Ticket' );
+		 return $this->getCartItems('Ticket');
 	 }
 
 
@@ -197,8 +197,8 @@ 
 	  * @param CartItemOption[]  $options
 	  * @return bool
 	  */
-	 public function addDiscount( DiscountInterface $discount, $quantity = 1, $options = array() ) {
-		 return $this->addItem( $discount, $quantity, $options, 'Discount' );
+	 public function addDiscount(DiscountInterface $discount, $quantity = 1, $options = array()) {
+		 return $this->addItem($discount, $quantity, $options, 'Discount');
 	 }
 
 
@@ -207,7 +207,7 @@ 
 	  * @return DiscountInterface[]
 	  */
 	 public function getDiscounts() {
-		 return $this->getItems( 'Discount' );
+		 return $this->getItems('Discount');
 	 }
 
 
@@ -216,7 +216,7 @@ 
 	  * @return DiscountCartItem[]
 	  */
 	 public function getDiscountCartItems() {
-		 return $this->getCartItems( 'Discount' );
+		 return $this->getCartItems('Discount');
 	 }
 
 
@@ -227,8 +227,8 @@ 
 	  * @param CartItemOption[]   $options
 	  * @return bool
 	  */
-	 public function addSurcharge( SurchargeInterface $surcharge, $quantity = 1, $options = array() ) {
-		 return $this->addItem( $surcharge, $quantity, $options, 'Surcharge' );
+	 public function addSurcharge(SurchargeInterface $surcharge, $quantity = 1, $options = array()) {
+		 return $this->addItem($surcharge, $quantity, $options, 'Surcharge');
  	}
 
 
@@ -237,7 +237,7 @@ 
 	  * @return SurchargeInterface[]
 	  */
 	 public function getSurcharges() {
-		 return $this->getItems( 'Surcharge' );
+		 return $this->getItems('Surcharge');
 	 }
 
 
@@ -246,7 +246,7 @@ 
 	  * @return SurchargeCartItem[]
 	  */
 	 public function getSurchargeCartItems() {
-		 return $this->getCartItems( 'Surcharge' );
+		 return $this->getCartItems('Surcharge');
 	 }
 
 
@@ -260,23 +260,23 @@ 
 	  * @return bool
 	  * @throws \EE_Error
 	  */
-	 protected function addItem( $item, $quantity, $options, $type = 'Ticket' ) {
-		 $cartItemClass = $this->getCartItemClass( $type );
-		 $cartItem = new $cartItemClass( $item, $this );
-		 if ( ! $cartItem instanceof CartItem ) {
+	 protected function addItem($item, $quantity, $options, $type = 'Ticket') {
+		 $cartItemClass = $this->getCartItemClass($type);
+		 $cartItem = new $cartItemClass($item, $this);
+		 if ( ! $cartItem instanceof CartItem) {
 			 throw new \EE_Error(
 				 sprintf(
-					 __( '"%1$s" is not a valid CartItem class.', 'event_espresso' ),
+					 __('"%1$s" is not a valid CartItem class.', 'event_espresso'),
 					 $cartItemClass
 				 )
 			 );
 		 }
-		 foreach ( $options as $option ) {
-			 if ( $option instanceof CartItemOption ) {
-				 $cartItem->addCartItemOption( $option );
+		 foreach ($options as $option) {
+			 if ($option instanceof CartItemOption) {
+				 $cartItem->addCartItemOption($option);
 			 }
 		 }
-		 return $this->items->addItem( $cartItem, $quantity );
+		 return $this->items->addItem($cartItem, $quantity);
 	 }
 
 
@@ -287,11 +287,11 @@ 
 	  * @return array
 	  * @throws \EE_Error
 	  */
-	 protected function getItems( $type = 'Ticket' ) {
-		 $cartItemClass = $this->getCartItemClass( $type );
+	 protected function getItems($type = 'Ticket') {
+		 $cartItemClass = $this->getCartItemClass($type);
 		 $items = array();
-		 foreach ( $this->items as $item ) {
-			 if ( $item instanceof $cartItemClass ) {
+		 foreach ($this->items as $item) {
+			 if ($item instanceof $cartItemClass) {
 				 $items[] = $item->getItem();
 			 }
 		 }
@@ -306,11 +306,11 @@ 
 	  * @return CartItem[]
 	  * @throws \EE_Error
 	  */
-	 protected function getCartItems( $type = 'Ticket' ) {
-		 $cartItemClass = $this->getCartItemClass( $type );
+	 protected function getCartItems($type = 'Ticket') {
+		 $cartItemClass = $this->getCartItemClass($type);
 		 $cartItems = array();
-		 foreach ( $this->items as $item ) {
-			 if ( $item instanceof $cartItemClass ) {
+		 foreach ($this->items as $item) {
+			 if ($item instanceof $cartItemClass) {
 				 $cartItems[] = $item;
 			 }
 		 }
@@ -324,12 +324,12 @@ 
 	  * @return string
 	  * @throws \EE_Error
 	  */
-	 public function getCartItemClass( $type = 'Ticket' ) {
-		 $itemClass = $type . 'CartItem';
-		 if ( ! class_exists( $itemClass ) || ! is_subclass_of( $itemClass, 'CartItem' ) ) {
+	 public function getCartItemClass($type = 'Ticket') {
+		 $itemClass = $type.'CartItem';
+		 if ( ! class_exists($itemClass) || ! is_subclass_of($itemClass, 'CartItem')) {
 			 throw new \EE_Error(
 				 sprintf(
-					 __( 'The "%1$s" class is either missing or not a valid CartItem class.', 'event_espresso' ),
+					 __('The "%1$s" class is either missing or not a valid CartItem class.', 'event_espresso'),
 					 $itemClass
 				 )
 			 );
@@ -348,8 +348,8 @@ 
 	 protected function calculateCartTotal() {
 		 // allow each cart calculator to modify the subtotals
 		 $this->cartCalculators->rewind();
-		 while ( $this->cartCalculators->valid() ) {
-			 $this->cartCalculators->current()->calculateTotal( $this, $this->cartTotal );
+		 while ($this->cartCalculators->valid()) {
+			 $this->cartCalculators->current()->calculateTotal($this, $this->cartTotal);
 			 $this->cartCalculators->next();
 		 }
 		 // ya gotta ADD IT UP! ADD IT UP!
@@ -359,7 +359,7 @@ 
 			 - $this->cartTotal->discountTotal
 			 + $this->cartTotal->taxSubtotal;
 		 // no negative values please
-		 $this->cartTotal->grandTotal = max( 0, $this->cartTotal->grandTotal );
+		 $this->cartTotal->grandTotal = max(0, $this->cartTotal->grandTotal);
 		 return $this->cartTotal;
 	 }
 

core/services/cart/CartItemRepository.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace EventEspresso\core\services\cart;
 
-use EventEspresso\Core;
 use EventEspresso\Core\Libraries\Repositories\EE_Object_Collection;
 
 if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {

Spacing +24 added lines, -24 removed lines

@@ -4,8 +4,8 @@ 
 use EventEspresso\Core;
 use EventEspresso\Core\Libraries\Repositories\EE_Object_Collection;
 
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 
@@ -30,13 +30,13 @@ 
 	 * @param int      $quantity
 	 * @return bool
 	 */
-	public function addItem( CartItem $item, $quantity = 1 ) {
-		if ( $this->has( $item ) ) {
-			$item->setQuantity( $item->quantity() + $quantity );
+	public function addItem(CartItem $item, $quantity = 1) {
+		if ($this->has($item)) {
+			$item->setQuantity($item->quantity() + $quantity);
 			return true;
 		}
-		$item->setQuantity( $quantity );
-		return $this->add( $item, $item->SKU() );
+		$item->setQuantity($quantity);
+		return $this->add($item, $item->SKU());
 	}
 
 
@@ -46,12 +46,12 @@ 
 	 * @return CartItem
 	 * @throws \EE_Error
 	 */
-	public function getItem( $SKU ) {
-		$item = $this->get_by_info( $SKU );
-		if ( ! $item instanceof CartItem ) {
-			throw new \EE_Error( 'Invalid or missing Item in cart.' );
+	public function getItem($SKU) {
+		$item = $this->get_by_info($SKU);
+		if ( ! $item instanceof CartItem) {
+			throw new \EE_Error('Invalid or missing Item in cart.');
 		}
-		return $item->validateItem( $item );
+		return $item->validateItem($item);
 	}
 
 
@@ -60,8 +60,8 @@ 
 	 * @param mixed $SKU
 	 * @return CartItem
 	 */
-	public function getItemBySKU( $SKU ) {
-		return $this->getItem( $SKU );
+	public function getItemBySKU($SKU) {
+		return $this->getItem($SKU);
 	}
 
 
@@ -70,8 +70,8 @@ 
 	 * @param CartItem $item
 	 * @return bool
 	 */
-	public function hasItem( CartItem $item ) {
-		return $this->has( $item );
+	public function hasItem(CartItem $item) {
+		return $this->has($item);
 	}
 
 
@@ -80,9 +80,9 @@ 
 	 * @param mixed $SKU
 	 * @return bool
 	 */
-	public function hasItemBySKU( $SKU ) {
-		$item = $this->get_by_info( $SKU );
-		return $this->has( $item );
+	public function hasItemBySKU($SKU) {
+		$item = $this->get_by_info($SKU);
+		return $this->has($item);
 	}
 
 
@@ -91,8 +91,8 @@ 
 	 * @param CartItem $item
 	 * @return void
 	 */
-	public function removeItem( CartItem $item ) {
-		$this->remove( $item );
+	public function removeItem(CartItem $item) {
+		$this->remove($item);
 	}
 
 
@@ -101,9 +101,9 @@ 
 	 * @param mixed $SKU
 	 * @return void
 	 */
-	public function removeItemBySKU( $SKU ) {
-		$item = $this->get_by_info( $SKU );
-		$this->remove( $item );
+	public function removeItemBySKU($SKU) {
+		$item = $this->get_by_info($SKU);
+		$this->remove($item);
 	}
 
 

core/services/cart/SurchargeCartItem.php

Doc Comments +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 
 	/**
 	 * @param $item
-	 * @return object
+	 * @return SurchargeInterface
 	 * @throws \EE_Error
 	 */
 	public function validateItem( $item ) {

Spacing +9 added lines, -9 removed lines

@@ -3,8 +3,8 @@ 
 
 use EventEspresso\core\interfaces\cart\SurchargeInterface;
 
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 
@@ -29,9 +29,9 @@ 
 	 * @param SurchargeInterface $surcharge
 	 * @param Cart 			$cart
 	 */
-	public function __construct( SurchargeInterface $surcharge, Cart $cart ) {
+	public function __construct(SurchargeInterface $surcharge, Cart $cart) {
 		$this->item = $surcharge;
-		parent::__construct( $cart );
+		parent::__construct($cart);
 	}
 
 
@@ -40,7 +40,7 @@ 
 	 * @return string
 	 */
 	public function generateSKU() {
-		return md5( $this->item->ID() . $this->item->description() );
+		return md5($this->item->ID().$this->item->description());
 	}
 
 
@@ -50,9 +50,9 @@ 
 	 * @return object
 	 * @throws \EE_Error
 	 */
-	public function validateItem( $item ) {
-		if ( ! $item instanceof SurchargeInterface ) {
-			throw new \EE_Error( 'Invalid or missing surcharge in cart.' );
+	public function validateItem($item) {
+		if ( ! $item instanceof SurchargeInterface) {
+			throw new \EE_Error('Invalid or missing surcharge in cart.');
 		}
 		return $item;
 	}
@@ -63,7 +63,7 @@ 
 	 * @return float
 	 */
 	public function calculatePrice() {
-		return (float)( $this->getItem()->amount() * $this->quantity );
+		return (float) ($this->getItem()->amount() * $this->quantity);
 	}
 
 

core/services/cart/TicketCartItem.php

Spacing +9 added lines, -9 removed lines

@@ -3,8 +3,8 @@ 
 
 use EventEspresso\core\interfaces\EEI_Ticket;
 
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 
@@ -29,9 +29,9 @@ 
 	 * @param EEI_Ticket 	$ticket
 	 * @param Cart 			$cart
 	 */
-	public function __construct( EEI_Ticket $ticket, Cart $cart ) {
+	public function __construct(EEI_Ticket $ticket, Cart $cart) {
 		$this->item = $ticket;
-		parent::__construct( $cart );
+		parent::__construct($cart);
 	}
 
 
@@ -40,7 +40,7 @@ 
 	 * @return string
 	 */
 	public function generateSKU() {
-		return md5( $this->item->ID() . $this->item->name_and_info() );
+		return md5($this->item->ID().$this->item->name_and_info());
 	}
 
 
@@ -50,9 +50,9 @@ 
 	 * @return object
 	 * @throws \EE_Error
 	 */
-	public function validateItem( $item ) {
-		if ( ! $item instanceof \EE_Ticket ) {
-			throw new \EE_Error( 'Invalid or missing ticket in cart.' );
+	public function validateItem($item) {
+		if ( ! $item instanceof \EE_Ticket) {
+			throw new \EE_Error('Invalid or missing ticket in cart.');
 		}
 		return $item;
 	}
@@ -63,7 +63,7 @@ 
 	 * @return float
 	 */
 	public function calculatePrice() {
-		return (float)( $this->getItem()->ticket_price() * $this->quantity );
+		return (float) ($this->getItem()->ticket_price() * $this->quantity);
 	}