eventespresso / event-espresso-core

modules/ticket_selector/TicketDatetimeAvailabilityTracker.php

Spacing +13 added lines, -13 removed lines

@@ -66,24 +66,24 @@ 
     public function ticketDatetimeAvailability(EE_Ticket $ticket, $get_original_ticket_spaces = false)
     {
         // if the $_available_spaces array has not been set up yet...
-        if (! isset($this->available_spaces['tickets'][ $ticket->ID() ])) {
+        if ( ! isset($this->available_spaces['tickets'][$ticket->ID()])) {
             $this->setInitialTicketDatetimeAvailability($ticket);
         }
         $available_spaces = $ticket->qty() - $ticket->sold();
-        if (isset($this->available_spaces['tickets'][ $ticket->ID() ])) {
+        if (isset($this->available_spaces['tickets'][$ticket->ID()])) {
             // loop thru tickets, which will ALSO include individual ticket records AND a total
-            foreach ($this->available_spaces['tickets'][ $ticket->ID() ] as $DTD_ID => $spaces) {
+            foreach ($this->available_spaces['tickets'][$ticket->ID()] as $DTD_ID => $spaces) {
                 // if we want the original datetime availability BEFORE we started subtracting tickets ?
                 if ($get_original_ticket_spaces) {
                     // then grab the available spaces from the "tickets" array
                     // and compare with the above to get the lowest number
                     $available_spaces = min(
                         $available_spaces,
-                        $this->available_spaces['tickets'][ $ticket->ID() ][ $DTD_ID ]
+                        $this->available_spaces['tickets'][$ticket->ID()][$DTD_ID]
                     );
                 } else {
                     // we want the updated ticket availability as stored in the "datetimes" array
-                    $available_spaces = min($available_spaces, $this->available_spaces['datetimes'][ $DTD_ID ]);
+                    $available_spaces = min($available_spaces, $this->available_spaces['datetimes'][$DTD_ID]);
                 }
             }
         }
@@ -114,7 +114,7 @@ 
                 'order_by' => array('DTT_EVT_start' => 'ASC'),
             )
         );
-        if (! empty($datetimes)) {
+        if ( ! empty($datetimes)) {
             // now loop thru all of the datetimes
             foreach ($datetimes as $datetime) {
                 if ($datetime instanceof EE_Datetime) {
@@ -122,17 +122,17 @@ 
                     $spaces_remaining = $datetime->spaces_remaining();
                     // save the total available spaces ( the lesser of the ticket qty minus the number of tickets sold
                     // or the datetime spaces remaining) to this ticket using the datetime ID as the key
-                    $this->available_spaces['tickets'][ $ticket->ID() ][ $datetime->ID() ] = min(
+                    $this->available_spaces['tickets'][$ticket->ID()][$datetime->ID()] = min(
                         $ticket->qty() - $ticket->sold(),
                         $spaces_remaining
                     );
                     // if the remaining spaces for this datetime is already set,
                     // then compare that against the datetime spaces remaining, and take the lowest number,
                     // else just take the datetime spaces remaining, and assign to the datetimes array
-                    $this->available_spaces['datetimes'][ $datetime->ID() ] = isset(
-                        $this->available_spaces['datetimes'][ $datetime->ID() ]
+                    $this->available_spaces['datetimes'][$datetime->ID()] = isset(
+                        $this->available_spaces['datetimes'][$datetime->ID()]
                     )
-                        ? min($this->available_spaces['datetimes'][ $datetime->ID() ], $spaces_remaining)
+                        ? min($this->available_spaces['datetimes'][$datetime->ID()], $spaces_remaining)
                         : $spaces_remaining;
                 }
             }
@@ -148,11 +148,11 @@ 
      */
     public function recalculateTicketDatetimeAvailability(EE_Ticket $ticket, $qty = 0)
     {
-        if (isset($this->available_spaces['tickets'][ $ticket->ID() ])) {
+        if (isset($this->available_spaces['tickets'][$ticket->ID()])) {
             // loop thru tickets, which will ALSO include individual ticket records AND a total
-            foreach ($this->available_spaces['tickets'][ $ticket->ID() ] as $DTD_ID => $spaces) {
+            foreach ($this->available_spaces['tickets'][$ticket->ID()] as $DTD_ID => $spaces) {
                 // subtract the qty of selected tickets from each datetime's available spaces this ticket has access to,
-                $this->available_spaces['datetimes'][ $DTD_ID ] -= $qty;
+                $this->available_spaces['datetimes'][$DTD_ID] -= $qty;
             }
         }
     }

Indentation +213 added lines, -213 removed lines

@@ -21,217 +21,217 @@
  */
 class TicketDatetimeAvailabilityTracker
 {
-    /**
-     * array of datetimes and the spaces available for them
-     *
-     * @var array[][]
-     */
-    private $available_spaces = array();
-
-    /**
-     * @var EEM_Datetime $datetime_model
-     */
-    private $datetime_model;
-
-
-    /**
-     * TicketDatetimeAvailabilityTracker constructor.
-     *
-     * @param EEM_Datetime $datetime_model
-     */
-    public function __construct(EEM_Datetime $datetime_model)
-    {
-        $this->datetime_model = $datetime_model;
-    }
-
-
-    /**
-     * ticketDatetimeAvailability
-     * creates an array of tickets plus all of the datetimes available to each ticket
-     * and tracks the spaces remaining for each of those datetimes
-     *
-     * @param EE_Ticket $ticket - selected ticket
-     * @param bool      $get_original_ticket_spaces
-     * @return int
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    public function ticketDatetimeAvailability(EE_Ticket $ticket, $get_original_ticket_spaces = false)
-    {
-        // if the $_available_spaces array has not been set up yet...
-        if (! isset($this->available_spaces['tickets'][ $ticket->ID() ])) {
-            $this->setInitialTicketDatetimeAvailability($ticket);
-        }
-        $available_spaces = $ticket->qty() - $ticket->sold();
-        if (isset($this->available_spaces['tickets'][ $ticket->ID() ])) {
-            // loop thru tickets, which will ALSO include individual ticket records AND a total
-            foreach ($this->available_spaces['tickets'][ $ticket->ID() ] as $DTD_ID => $spaces) {
-                // if we want the original datetime availability BEFORE we started subtracting tickets ?
-                if ($get_original_ticket_spaces) {
-                    // then grab the available spaces from the "tickets" array
-                    // and compare with the above to get the lowest number
-                    $available_spaces = min(
-                        $available_spaces,
-                        $this->available_spaces['tickets'][ $ticket->ID() ][ $DTD_ID ]
-                    );
-                } else {
-                    // we want the updated ticket availability as stored in the "datetimes" array
-                    $available_spaces = min($available_spaces, $this->available_spaces['datetimes'][ $DTD_ID ]);
-                }
-            }
-        }
-        return $available_spaces;
-    }
-
-
-    /**
-     * @param EE_Ticket $ticket
-     * @return void
-     * @throws InvalidArgumentException
-     * @throws InvalidInterfaceException
-     * @throws InvalidDataTypeException
-     * @throws EE_Error
-     */
-    private function setInitialTicketDatetimeAvailability(EE_Ticket $ticket)
-    {
-        // first, get all of the datetimes that are available to this ticket
-        $datetimes = $ticket->get_many_related(
-            'Datetime',
-            array(
-                array(
-                    'DTT_EVT_end' => array(
-                        '>=',
-                        $this->datetime_model->current_time_for_query('DTT_EVT_end'),
-                    ),
-                ),
-                'order_by' => array('DTT_EVT_start' => 'ASC'),
-            )
-        );
-        if (! empty($datetimes)) {
-            // now loop thru all of the datetimes
-            foreach ($datetimes as $datetime) {
-                if ($datetime instanceof EE_Datetime) {
-                    // the number of spaces available for the datetime without considering individual ticket quantities
-                    $spaces_remaining = $datetime->spaces_remaining();
-                    // save the total available spaces ( the lesser of the ticket qty minus the number of tickets sold
-                    // or the datetime spaces remaining) to this ticket using the datetime ID as the key
-                    $this->available_spaces['tickets'][ $ticket->ID() ][ $datetime->ID() ] = min(
-                        $ticket->qty() - $ticket->sold(),
-                        $spaces_remaining
-                    );
-                    // if the remaining spaces for this datetime is already set,
-                    // then compare that against the datetime spaces remaining, and take the lowest number,
-                    // else just take the datetime spaces remaining, and assign to the datetimes array
-                    $this->available_spaces['datetimes'][ $datetime->ID() ] = isset(
-                        $this->available_spaces['datetimes'][ $datetime->ID() ]
-                    )
-                        ? min($this->available_spaces['datetimes'][ $datetime->ID() ], $spaces_remaining)
-                        : $spaces_remaining;
-                }
-            }
-        }
-    }
-
-
-    /**
-     * @param    EE_Ticket $ticket
-     * @param    int       $qty
-     * @return    void
-     * @throws EE_Error
-     */
-    public function recalculateTicketDatetimeAvailability(EE_Ticket $ticket, $qty = 0)
-    {
-        if (isset($this->available_spaces['tickets'][ $ticket->ID() ])) {
-            // loop thru tickets, which will ALSO include individual ticket records AND a total
-            foreach ($this->available_spaces['tickets'][ $ticket->ID() ] as $DTD_ID => $spaces) {
-                // subtract the qty of selected tickets from each datetime's available spaces this ticket has access to,
-                $this->available_spaces['datetimes'][ $DTD_ID ] -= $qty;
-            }
-        }
-    }
-
-
-    /**
-     * @param EE_Ticket $ticket
-     * @param           $qty
-     * @param int       $total_ticket_count
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    public function processAvailabilityError(EE_Ticket $ticket, $qty, $total_ticket_count = 1)
-    {
-        // tickets can not be purchased but let's find the exact number left
-        // for the last ticket selected PRIOR to subtracting tickets
-        $available_spaces = $this->ticketDatetimeAvailability($ticket, true);
-        // greedy greedy greedy eh?
-        if ($available_spaces > 0) {
-            if (
-                apply_filters(
-                    'FHEE__EE_Ticket_Selector___add_ticket_to_cart__allow_display_availability_error',
-                    true,
-                    $ticket,
-                    $qty,
-                    $available_spaces
-                )
-            ) {
-                $this->availabilityError(
-                    $available_spaces,
-                    $total_ticket_count
-                );
-            }
-        } else {
-            EE_Error::add_error(
-                esc_html__(
-                    'We\'re sorry, but there are no available spaces left for this event at this particular date and time.',
-                    'event_espresso'
-                ),
-                __FILE__,
-                __FUNCTION__,
-                __LINE__
-            );
-        }
-    }
-
-
-    /**
-     * @param int $available_spaces
-     * @param int $total_ticket_count
-     */
-    private function availabilityError($available_spaces = 1, $total_ticket_count = 1)
-    {
-        // add error messaging - we're using the _n function that will generate
-        // the appropriate singular or plural message based on the number of $available_spaces
-        if ($total_ticket_count) {
-            $msg = sprintf(
-                esc_html(
-                    _n(
-                        'We\'re sorry, but there is only %1$s available space left for this event at this particular date and time. Please select a different number (or different combination) of tickets by cancelling the current selection and choosing again, or proceed to registration.',
-                        'We\'re sorry, but there are only %1$s available spaces left for this event at this particular date and time. Please select a different number (or different combination) of tickets by cancelling the current selection and choosing again, or proceed to registration.',
-                        $available_spaces,
-                        'event_espresso'
-                    )
-                ),
-                $available_spaces,
-                '<br />'
-            );
-        } else {
-            $msg = sprintf(
-                esc_html(
-                    _n(
-                        'We\'re sorry, but there is only %1$s available space left for this event at this particular date and time. Please select a different number (or different combination) of tickets.',
-                        'We\'re sorry, but there are only %1$s available spaces left for this event at this particular date and time. Please select a different number (or different combination) of tickets.',
-                        $available_spaces,
-                        'event_espresso'
-                    )
-                ),
-                $available_spaces,
-                '<br />'
-            );
-        }
-        EE_Error::add_error($msg, __FILE__, __FUNCTION__, __LINE__);
-    }
+	/**
+	 * array of datetimes and the spaces available for them
+	 *
+	 * @var array[][]
+	 */
+	private $available_spaces = array();
+
+	/**
+	 * @var EEM_Datetime $datetime_model
+	 */
+	private $datetime_model;
+
+
+	/**
+	 * TicketDatetimeAvailabilityTracker constructor.
+	 *
+	 * @param EEM_Datetime $datetime_model
+	 */
+	public function __construct(EEM_Datetime $datetime_model)
+	{
+		$this->datetime_model = $datetime_model;
+	}
+
+
+	/**
+	 * ticketDatetimeAvailability
+	 * creates an array of tickets plus all of the datetimes available to each ticket
+	 * and tracks the spaces remaining for each of those datetimes
+	 *
+	 * @param EE_Ticket $ticket - selected ticket
+	 * @param bool      $get_original_ticket_spaces
+	 * @return int
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	public function ticketDatetimeAvailability(EE_Ticket $ticket, $get_original_ticket_spaces = false)
+	{
+		// if the $_available_spaces array has not been set up yet...
+		if (! isset($this->available_spaces['tickets'][ $ticket->ID() ])) {
+			$this->setInitialTicketDatetimeAvailability($ticket);
+		}
+		$available_spaces = $ticket->qty() - $ticket->sold();
+		if (isset($this->available_spaces['tickets'][ $ticket->ID() ])) {
+			// loop thru tickets, which will ALSO include individual ticket records AND a total
+			foreach ($this->available_spaces['tickets'][ $ticket->ID() ] as $DTD_ID => $spaces) {
+				// if we want the original datetime availability BEFORE we started subtracting tickets ?
+				if ($get_original_ticket_spaces) {
+					// then grab the available spaces from the "tickets" array
+					// and compare with the above to get the lowest number
+					$available_spaces = min(
+						$available_spaces,
+						$this->available_spaces['tickets'][ $ticket->ID() ][ $DTD_ID ]
+					);
+				} else {
+					// we want the updated ticket availability as stored in the "datetimes" array
+					$available_spaces = min($available_spaces, $this->available_spaces['datetimes'][ $DTD_ID ]);
+				}
+			}
+		}
+		return $available_spaces;
+	}
+
+
+	/**
+	 * @param EE_Ticket $ticket
+	 * @return void
+	 * @throws InvalidArgumentException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidDataTypeException
+	 * @throws EE_Error
+	 */
+	private function setInitialTicketDatetimeAvailability(EE_Ticket $ticket)
+	{
+		// first, get all of the datetimes that are available to this ticket
+		$datetimes = $ticket->get_many_related(
+			'Datetime',
+			array(
+				array(
+					'DTT_EVT_end' => array(
+						'>=',
+						$this->datetime_model->current_time_for_query('DTT_EVT_end'),
+					),
+				),
+				'order_by' => array('DTT_EVT_start' => 'ASC'),
+			)
+		);
+		if (! empty($datetimes)) {
+			// now loop thru all of the datetimes
+			foreach ($datetimes as $datetime) {
+				if ($datetime instanceof EE_Datetime) {
+					// the number of spaces available for the datetime without considering individual ticket quantities
+					$spaces_remaining = $datetime->spaces_remaining();
+					// save the total available spaces ( the lesser of the ticket qty minus the number of tickets sold
+					// or the datetime spaces remaining) to this ticket using the datetime ID as the key
+					$this->available_spaces['tickets'][ $ticket->ID() ][ $datetime->ID() ] = min(
+						$ticket->qty() - $ticket->sold(),
+						$spaces_remaining
+					);
+					// if the remaining spaces for this datetime is already set,
+					// then compare that against the datetime spaces remaining, and take the lowest number,
+					// else just take the datetime spaces remaining, and assign to the datetimes array
+					$this->available_spaces['datetimes'][ $datetime->ID() ] = isset(
+						$this->available_spaces['datetimes'][ $datetime->ID() ]
+					)
+						? min($this->available_spaces['datetimes'][ $datetime->ID() ], $spaces_remaining)
+						: $spaces_remaining;
+				}
+			}
+		}
+	}
+
+
+	/**
+	 * @param    EE_Ticket $ticket
+	 * @param    int       $qty
+	 * @return    void
+	 * @throws EE_Error
+	 */
+	public function recalculateTicketDatetimeAvailability(EE_Ticket $ticket, $qty = 0)
+	{
+		if (isset($this->available_spaces['tickets'][ $ticket->ID() ])) {
+			// loop thru tickets, which will ALSO include individual ticket records AND a total
+			foreach ($this->available_spaces['tickets'][ $ticket->ID() ] as $DTD_ID => $spaces) {
+				// subtract the qty of selected tickets from each datetime's available spaces this ticket has access to,
+				$this->available_spaces['datetimes'][ $DTD_ID ] -= $qty;
+			}
+		}
+	}
+
+
+	/**
+	 * @param EE_Ticket $ticket
+	 * @param           $qty
+	 * @param int       $total_ticket_count
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	public function processAvailabilityError(EE_Ticket $ticket, $qty, $total_ticket_count = 1)
+	{
+		// tickets can not be purchased but let's find the exact number left
+		// for the last ticket selected PRIOR to subtracting tickets
+		$available_spaces = $this->ticketDatetimeAvailability($ticket, true);
+		// greedy greedy greedy eh?
+		if ($available_spaces > 0) {
+			if (
+				apply_filters(
+					'FHEE__EE_Ticket_Selector___add_ticket_to_cart__allow_display_availability_error',
+					true,
+					$ticket,
+					$qty,
+					$available_spaces
+				)
+			) {
+				$this->availabilityError(
+					$available_spaces,
+					$total_ticket_count
+				);
+			}
+		} else {
+			EE_Error::add_error(
+				esc_html__(
+					'We\'re sorry, but there are no available spaces left for this event at this particular date and time.',
+					'event_espresso'
+				),
+				__FILE__,
+				__FUNCTION__,
+				__LINE__
+			);
+		}
+	}
+
+
+	/**
+	 * @param int $available_spaces
+	 * @param int $total_ticket_count
+	 */
+	private function availabilityError($available_spaces = 1, $total_ticket_count = 1)
+	{
+		// add error messaging - we're using the _n function that will generate
+		// the appropriate singular or plural message based on the number of $available_spaces
+		if ($total_ticket_count) {
+			$msg = sprintf(
+				esc_html(
+					_n(
+						'We\'re sorry, but there is only %1$s available space left for this event at this particular date and time. Please select a different number (or different combination) of tickets by cancelling the current selection and choosing again, or proceed to registration.',
+						'We\'re sorry, but there are only %1$s available spaces left for this event at this particular date and time. Please select a different number (or different combination) of tickets by cancelling the current selection and choosing again, or proceed to registration.',
+						$available_spaces,
+						'event_espresso'
+					)
+				),
+				$available_spaces,
+				'<br />'
+			);
+		} else {
+			$msg = sprintf(
+				esc_html(
+					_n(
+						'We\'re sorry, but there is only %1$s available space left for this event at this particular date and time. Please select a different number (or different combination) of tickets.',
+						'We\'re sorry, but there are only %1$s available spaces left for this event at this particular date and time. Please select a different number (or different combination) of tickets.',
+						$available_spaces,
+						'event_espresso'
+					)
+				),
+				$available_spaces,
+				'<br />'
+			);
+		}
+		EE_Error::add_error($msg, __FILE__, __FUNCTION__, __LINE__);
+	}
 }

core/services/bootstrap/BootstrapDependencyInjectionContainer.php

Spacing +2 added lines, -2 removed lines

@@ -78,13 +78,13 @@
         // EE_Dependency_Map: info about how to load classes required by other classes
         espresso_load_required(
             'EE_Dependency_Map',
-            EE_CORE . 'EE_Dependency_Map.core.php'
+            EE_CORE.'EE_Dependency_Map.core.php'
         );
         $this->dependency_map = EE_Dependency_Map::instance($this->class_cache);
         // EE_Registry: central repository for classes (legacy)
         espresso_load_required(
             'EE_Registry',
-            EE_CORE . 'EE_Registry.core.php'
+            EE_CORE.'EE_Registry.core.php'
         );
         $this->registry = EE_Registry::instance(
             $this->dependency_map,

Indentation +119 added lines, -119 removed lines

@@ -24,123 +24,123 @@
  */
 class BootstrapDependencyInjectionContainer
 {
-    /**
-     * @var EE_Dependency_Map $dependency_map
-     */
-    protected $dependency_map;
-
-    /**
-     * @type LoaderInterface $loader
-     */
-    protected $loader;
-
-    /**
-     * @var EE_Registry $registry
-     */
-    protected $registry;
-
-    /**
-     * @var ClassInterfaceCache $class_cache
-     */
-    private $class_cache;
-
-    /**
-     * @var Mirror
-     */
-    private $mirror;
-
-    /**
-     * @var ObjectIdentifier
-     */
-    private $object_identifier;
-
-
-    /**
-     * Can't use this just yet until we exorcise some more of our singleton usage from core
-     */
-    public function buildDependencyInjectionContainer()
-    {
-        // build DI container
-        // $OpenCoffeeShop = new EventEspresso\core\services\container\OpenCoffeeShop();
-        // $OpenCoffeeShop->addRecipes();
-        // $CoffeeShop = $OpenCoffeeShop->CoffeeShop();
-    }
-
-
-    /**
-     * Setups  EE_Registry and EE_Dependency_Map
-     *
-     * @throws EE_Error
-     */
-    public function buildLegacyDependencyInjectionContainer()
-    {
-        $this->class_cache = new ClassInterfaceCache();
-        $this->object_identifier = new ObjectIdentifier($this->class_cache);
-        $this->mirror = new Mirror();
-        // EE_Dependency_Map: info about how to load classes required by other classes
-        espresso_load_required(
-            'EE_Dependency_Map',
-            EE_CORE . 'EE_Dependency_Map.core.php'
-        );
-        $this->dependency_map = EE_Dependency_Map::instance($this->class_cache);
-        // EE_Registry: central repository for classes (legacy)
-        espresso_load_required(
-            'EE_Registry',
-            EE_CORE . 'EE_Registry.core.php'
-        );
-        $this->registry = EE_Registry::instance(
-            $this->dependency_map,
-            $this->mirror,
-            $this->class_cache,
-            $this->object_identifier
-        );
-    }
-
-
-    /**
-     * Performs initial setup for the generic Loader
-     *
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     * @throws InvalidArgumentException
-     */
-    public function buildLoader()
-    {
-        $this->loader = LoaderFactory::getLoader(
-            $this->registry,
-            $this->class_cache,
-            $this->object_identifier
-        );
-        $this->loader->share('EventEspresso\core\services\loaders\ClassInterfaceCache', $this->class_cache);
-        $this->loader->share('EventEspresso\core\services\loaders\ObjectIdentifier', $this->object_identifier);
-        $this->loader->share('EventEspresso\core\services\container\Mirror', $this->mirror);
-        $this->dependency_map->setLoader($this->loader);
-    }
-
-
-    /**
-     * @return EE_Dependency_Map
-     */
-    public function getDependencyMap()
-    {
-        return $this->dependency_map;
-    }
-
-
-    /**
-     * @return EE_Registry
-     */
-    public function getRegistry()
-    {
-        return $this->registry;
-    }
-
-
-    /**
-     * @return LoaderInterface
-     */
-    public function getLoader()
-    {
-        return $this->loader;
-    }
+	/**
+	 * @var EE_Dependency_Map $dependency_map
+	 */
+	protected $dependency_map;
+
+	/**
+	 * @type LoaderInterface $loader
+	 */
+	protected $loader;
+
+	/**
+	 * @var EE_Registry $registry
+	 */
+	protected $registry;
+
+	/**
+	 * @var ClassInterfaceCache $class_cache
+	 */
+	private $class_cache;
+
+	/**
+	 * @var Mirror
+	 */
+	private $mirror;
+
+	/**
+	 * @var ObjectIdentifier
+	 */
+	private $object_identifier;
+
+
+	/**
+	 * Can't use this just yet until we exorcise some more of our singleton usage from core
+	 */
+	public function buildDependencyInjectionContainer()
+	{
+		// build DI container
+		// $OpenCoffeeShop = new EventEspresso\core\services\container\OpenCoffeeShop();
+		// $OpenCoffeeShop->addRecipes();
+		// $CoffeeShop = $OpenCoffeeShop->CoffeeShop();
+	}
+
+
+	/**
+	 * Setups  EE_Registry and EE_Dependency_Map
+	 *
+	 * @throws EE_Error
+	 */
+	public function buildLegacyDependencyInjectionContainer()
+	{
+		$this->class_cache = new ClassInterfaceCache();
+		$this->object_identifier = new ObjectIdentifier($this->class_cache);
+		$this->mirror = new Mirror();
+		// EE_Dependency_Map: info about how to load classes required by other classes
+		espresso_load_required(
+			'EE_Dependency_Map',
+			EE_CORE . 'EE_Dependency_Map.core.php'
+		);
+		$this->dependency_map = EE_Dependency_Map::instance($this->class_cache);
+		// EE_Registry: central repository for classes (legacy)
+		espresso_load_required(
+			'EE_Registry',
+			EE_CORE . 'EE_Registry.core.php'
+		);
+		$this->registry = EE_Registry::instance(
+			$this->dependency_map,
+			$this->mirror,
+			$this->class_cache,
+			$this->object_identifier
+		);
+	}
+
+
+	/**
+	 * Performs initial setup for the generic Loader
+	 *
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidArgumentException
+	 */
+	public function buildLoader()
+	{
+		$this->loader = LoaderFactory::getLoader(
+			$this->registry,
+			$this->class_cache,
+			$this->object_identifier
+		);
+		$this->loader->share('EventEspresso\core\services\loaders\ClassInterfaceCache', $this->class_cache);
+		$this->loader->share('EventEspresso\core\services\loaders\ObjectIdentifier', $this->object_identifier);
+		$this->loader->share('EventEspresso\core\services\container\Mirror', $this->mirror);
+		$this->dependency_map->setLoader($this->loader);
+	}
+
+
+	/**
+	 * @return EE_Dependency_Map
+	 */
+	public function getDependencyMap()
+	{
+		return $this->dependency_map;
+	}
+
+
+	/**
+	 * @return EE_Registry
+	 */
+	public function getRegistry()
+	{
+		return $this->registry;
+	}
+
+
+	/**
+	 * @return LoaderInterface
+	 */
+	public function getLoader()
+	{
+		return $this->loader;
+	}
 }

core/services/loaders/CoreLoader.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
      */
     public function __construct($generator)
     {
-        if (! ($generator instanceof EE_Registry || $generator instanceof CoffeeShop)) {
+        if ( ! ($generator instanceof EE_Registry || $generator instanceof CoffeeShop)) {
             throw new InvalidArgumentException(
                 esc_html__(
                     'The CoreLoader class must receive an instance of EE_Registry or the CoffeeShop DI container.',

Indentation +101 added lines, -101 removed lines

@@ -28,111 +28,111 @@
  */
 class CoreLoader implements LoaderDecoratorInterface
 {
-    /**
-     * @var EE_Registry|CoffeeShop $generator
-     */
-    private $generator;
+	/**
+	 * @var EE_Registry|CoffeeShop $generator
+	 */
+	private $generator;
 
 
-    /**
-     * CoreLoader constructor.
-     *
-     * @param EE_Registry|CoffeeShop $generator
-     * @throws InvalidArgumentException
-     */
-    public function __construct($generator)
-    {
-        if (! ($generator instanceof EE_Registry || $generator instanceof CoffeeShop)) {
-            throw new InvalidArgumentException(
-                esc_html__(
-                    'The CoreLoader class must receive an instance of EE_Registry or the CoffeeShop DI container.',
-                    'event_espresso'
-                )
-            );
-        }
-        $this->generator = $generator;
-    }
+	/**
+	 * CoreLoader constructor.
+	 *
+	 * @param EE_Registry|CoffeeShop $generator
+	 * @throws InvalidArgumentException
+	 */
+	public function __construct($generator)
+	{
+		if (! ($generator instanceof EE_Registry || $generator instanceof CoffeeShop)) {
+			throw new InvalidArgumentException(
+				esc_html__(
+					'The CoreLoader class must receive an instance of EE_Registry or the CoffeeShop DI container.',
+					'event_espresso'
+				)
+			);
+		}
+		$this->generator = $generator;
+	}
 
 
-    /**
-     * Calls the appropriate loading method from the installed generator;
-     * If EE_Registry is being used, then the additional parameters for the EE_Registry::create() method
-     * can be added to the $arguments array and they will be extracted and passed to EE_Registry::create(),
-     * but NOT to the class being instantiated.
-     * This is done by adding the parameters to the $arguments array as follows:
-     *  array(
-     *      'EE_Registry::create(from_db)'   => true, // boolean value, default = false
-     *      'EE_Registry::create(load_only)' => true, // boolean value, default = false
-     *      'EE_Registry::create(addon)'     => true, // boolean value, default = false
-     *  )
-     *
-     * @param string $fqcn
-     * @param array  $arguments
-     * @param bool   $shared
-     * @return mixed
-     * @throws OutOfBoundsException
-     * @throws ServiceExistsException
-     * @throws InstantiationException
-     * @throws InvalidIdentifierException
-     * @throws InvalidDataTypeException
-     * @throws InvalidClassException
-     * @throws EE_Error
-     * @throws ServiceNotFoundException
-     * @throws ReflectionException
-     * @throws InvalidInterfaceException
-     * @throws InvalidArgumentException
-     */
-    public function load($fqcn, $arguments = array(), $shared = true)
-    {
-        $shared = filter_var($shared, FILTER_VALIDATE_BOOLEAN);
-        if ($this->generator instanceof EE_Registry) {
-            // check if additional EE_Registry::create() arguments have been passed
-            // from_db
-            $from_db = isset($arguments['EE_Registry::create(from_db)'])
-                ? filter_var($arguments['EE_Registry::create(from_db)'], FILTER_VALIDATE_BOOLEAN)
-                : false;
-            // load_only
-            $load_only = isset($arguments['EE_Registry::create(load_only)'])
-                ? filter_var($arguments['EE_Registry::create(load_only)'], FILTER_VALIDATE_BOOLEAN)
-                : false;
-            // addon
-            $addon = isset($arguments['EE_Registry::create(addon)'])
-                ? filter_var($arguments['EE_Registry::create(addon)'], FILTER_VALIDATE_BOOLEAN)
-                : false;
-            unset(
-                $arguments['EE_Registry::create(from_db)'],
-                $arguments['EE_Registry::create(load_only)'],
-                $arguments['EE_Registry::create(addon)']
-            );
-            // addons need to be cached on EE_Registry
-            $shared = $addon ? true : $shared;
-            return $this->generator->create(
-                $fqcn,
-                $arguments,
-                $shared,
-                $from_db,
-                $load_only,
-                $addon
-            );
-        }
-        return $this->generator->brew(
-            $fqcn,
-            $arguments,
-            $shared ? CoffeeMaker::BREW_SHARED : CoffeeMaker::BREW_NEW
-        );
-    }
+	/**
+	 * Calls the appropriate loading method from the installed generator;
+	 * If EE_Registry is being used, then the additional parameters for the EE_Registry::create() method
+	 * can be added to the $arguments array and they will be extracted and passed to EE_Registry::create(),
+	 * but NOT to the class being instantiated.
+	 * This is done by adding the parameters to the $arguments array as follows:
+	 *  array(
+	 *      'EE_Registry::create(from_db)'   => true, // boolean value, default = false
+	 *      'EE_Registry::create(load_only)' => true, // boolean value, default = false
+	 *      'EE_Registry::create(addon)'     => true, // boolean value, default = false
+	 *  )
+	 *
+	 * @param string $fqcn
+	 * @param array  $arguments
+	 * @param bool   $shared
+	 * @return mixed
+	 * @throws OutOfBoundsException
+	 * @throws ServiceExistsException
+	 * @throws InstantiationException
+	 * @throws InvalidIdentifierException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidClassException
+	 * @throws EE_Error
+	 * @throws ServiceNotFoundException
+	 * @throws ReflectionException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidArgumentException
+	 */
+	public function load($fqcn, $arguments = array(), $shared = true)
+	{
+		$shared = filter_var($shared, FILTER_VALIDATE_BOOLEAN);
+		if ($this->generator instanceof EE_Registry) {
+			// check if additional EE_Registry::create() arguments have been passed
+			// from_db
+			$from_db = isset($arguments['EE_Registry::create(from_db)'])
+				? filter_var($arguments['EE_Registry::create(from_db)'], FILTER_VALIDATE_BOOLEAN)
+				: false;
+			// load_only
+			$load_only = isset($arguments['EE_Registry::create(load_only)'])
+				? filter_var($arguments['EE_Registry::create(load_only)'], FILTER_VALIDATE_BOOLEAN)
+				: false;
+			// addon
+			$addon = isset($arguments['EE_Registry::create(addon)'])
+				? filter_var($arguments['EE_Registry::create(addon)'], FILTER_VALIDATE_BOOLEAN)
+				: false;
+			unset(
+				$arguments['EE_Registry::create(from_db)'],
+				$arguments['EE_Registry::create(load_only)'],
+				$arguments['EE_Registry::create(addon)']
+			);
+			// addons need to be cached on EE_Registry
+			$shared = $addon ? true : $shared;
+			return $this->generator->create(
+				$fqcn,
+				$arguments,
+				$shared,
+				$from_db,
+				$load_only,
+				$addon
+			);
+		}
+		return $this->generator->brew(
+			$fqcn,
+			$arguments,
+			$shared ? CoffeeMaker::BREW_SHARED : CoffeeMaker::BREW_NEW
+		);
+	}
 
 
-    /**
-     * calls reset() on generator if method exists
-     *
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function reset()
-    {
-        if ($this->generator instanceof ResettableInterface) {
-            $this->generator->reset();
-        }
-    }
+	/**
+	 * calls reset() on generator if method exists
+	 *
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function reset()
+	{
+		if ($this->generator instanceof ResettableInterface) {
+			$this->generator->reset();
+		}
+	}
 }

core/domain/services/custom_post_types/RegisterCustomPostTypes.php

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
     {
         $custom_post_types = $this->custom_post_types->getDefinitions();
         foreach ($custom_post_types as $custom_post_type => $CPT) {
-            $this->wp_post_types[ $custom_post_type ] = $this->registerCustomPostType(
+            $this->wp_post_types[$custom_post_type] = $this->registerCustomPostType(
                 $custom_post_type,
                 $CPT['singular_name'],
                 $CPT['plural_name'],

Indentation +295 added lines, -295 removed lines

@@ -17,314 +17,314 @@
  */
 class RegisterCustomPostTypes
 {
-    /**
-     * @var CustomPostTypeDefinitions $custom_post_types
-     */
-    public $custom_post_types;
+	/**
+	 * @var CustomPostTypeDefinitions $custom_post_types
+	 */
+	public $custom_post_types;
 
-    /**
-     * @var WP_Post_Type[] $wp_post_types
-     */
-    public $wp_post_types = array();
+	/**
+	 * @var WP_Post_Type[] $wp_post_types
+	 */
+	public $wp_post_types = array();
 
 
-    /**
-     * RegisterCustomPostTypes constructor.
-     *
-     * @param CustomPostTypeDefinitions $custom_post_types
-     */
-    public function __construct(CustomPostTypeDefinitions $custom_post_types)
-    {
-        $this->custom_post_types = $custom_post_types;
-    }
+	/**
+	 * RegisterCustomPostTypes constructor.
+	 *
+	 * @param CustomPostTypeDefinitions $custom_post_types
+	 */
+	public function __construct(CustomPostTypeDefinitions $custom_post_types)
+	{
+		$this->custom_post_types = $custom_post_types;
+	}
 
 
-    /**
-     * @return WP_Post_Type[]
-     */
-    public function getRegisteredCustomPostTypes()
-    {
-        return $this->wp_post_types;
-    }
+	/**
+	 * @return WP_Post_Type[]
+	 */
+	public function getRegisteredCustomPostTypes()
+	{
+		return $this->wp_post_types;
+	}
 
 
-    /**
-     * @return void
-     * @throws DomainException
-     */
-    public function registerCustomPostTypes()
-    {
-        $custom_post_types = $this->custom_post_types->getDefinitions();
-        foreach ($custom_post_types as $custom_post_type => $CPT) {
-            $this->wp_post_types[ $custom_post_type ] = $this->registerCustomPostType(
-                $custom_post_type,
-                $CPT['singular_name'],
-                $CPT['plural_name'],
-                $CPT['singular_slug'],
-                $CPT['plural_slug'],
-                $CPT['args']
-            );
-        }
-    }
+	/**
+	 * @return void
+	 * @throws DomainException
+	 */
+	public function registerCustomPostTypes()
+	{
+		$custom_post_types = $this->custom_post_types->getDefinitions();
+		foreach ($custom_post_types as $custom_post_type => $CPT) {
+			$this->wp_post_types[ $custom_post_type ] = $this->registerCustomPostType(
+				$custom_post_type,
+				$CPT['singular_name'],
+				$CPT['plural_name'],
+				$CPT['singular_slug'],
+				$CPT['plural_slug'],
+				$CPT['args']
+			);
+		}
+	}
 
 
-    /**
-     * Registers a new custom post type. Sets default settings given only the following params.
-     * Returns the registered post type object, or an error object.
-     *
-     * @param string $post_type          the actual post type name
-     *                                   IMPORTANT:
-     *                                   this must match what the slug is for admin pages related to this CPT
-     *                                   Also any models must use this slug as well
-     * @param string $singular_name      a pre-internationalized string for the singular name of the objects
-     * @param string $plural_name        a pre-internationalized string for the plural name of the objects
-     * @param string $singular_slug
-     * @param string $plural_slug
-     * @param array  $override_arguments exactly like $args as described in
-     *                                   http://codex.wordpress.org/Function_Reference/register_post_type
-     * @return WP_Post_Type|WP_Error
-     * @throws DomainException
-     */
-    public function registerCustomPostType(
-        $post_type,
-        $singular_name,
-        $plural_name,
-        $singular_slug = '',
-        $plural_slug = '',
-        array $override_arguments = array()
-    ) {
-        $wp_post_type = register_post_type(
-            $post_type,
-            $this->prepareArguments(
-                $post_type,
-                $singular_name,
-                $plural_name,
-                $singular_slug,
-                $plural_slug,
-                $override_arguments
-            )
-        );
-        if ($wp_post_type instanceof WP_Error) {
-            throw new DomainException($wp_post_type->get_error_message());
-        }
-        return $wp_post_type;
-    }
+	/**
+	 * Registers a new custom post type. Sets default settings given only the following params.
+	 * Returns the registered post type object, or an error object.
+	 *
+	 * @param string $post_type          the actual post type name
+	 *                                   IMPORTANT:
+	 *                                   this must match what the slug is for admin pages related to this CPT
+	 *                                   Also any models must use this slug as well
+	 * @param string $singular_name      a pre-internationalized string for the singular name of the objects
+	 * @param string $plural_name        a pre-internationalized string for the plural name of the objects
+	 * @param string $singular_slug
+	 * @param string $plural_slug
+	 * @param array  $override_arguments exactly like $args as described in
+	 *                                   http://codex.wordpress.org/Function_Reference/register_post_type
+	 * @return WP_Post_Type|WP_Error
+	 * @throws DomainException
+	 */
+	public function registerCustomPostType(
+		$post_type,
+		$singular_name,
+		$plural_name,
+		$singular_slug = '',
+		$plural_slug = '',
+		array $override_arguments = array()
+	) {
+		$wp_post_type = register_post_type(
+			$post_type,
+			$this->prepareArguments(
+				$post_type,
+				$singular_name,
+				$plural_name,
+				$singular_slug,
+				$plural_slug,
+				$override_arguments
+			)
+		);
+		if ($wp_post_type instanceof WP_Error) {
+			throw new DomainException($wp_post_type->get_error_message());
+		}
+		return $wp_post_type;
+	}
 
 
-    /**
-     * @param string $post_type          the actual post type name
-     * @param string $singular_name      a pre-internationalized string for the singular name of the objects
-     * @param string $plural_name        a pre-internationalized string for the plural name of the objects
-     * @param string $singular_slug
-     * @param string $plural_slug
-     * @param array  $override_arguments The default values set in this function will be overridden
-     *                                   by whatever you set in $override_arguments
-     * @return array
-     */
-    protected function prepareArguments(
-        $post_type,
-        $singular_name,
-        $plural_name,
-        $singular_slug,
-        $plural_slug,
-        array $override_arguments = array()
-    ) {
-        // verify plural slug and singular slug, if they aren't we'll use $singular_name and $plural_name
-        $singular_slug = ! empty($singular_slug) ? $singular_slug : $singular_name;
-        $plural_slug = ! empty($plural_slug) ? $plural_slug : $plural_name;
-        $labels = $this->getLabels(
-            $singular_name,
-            $plural_name,
-            $singular_slug,
-            $plural_slug
-        );
-        // note the page_templates arg in the supports index is something specific to EE.
-        // WordPress doesn't actually have that in their register_post_type api.
-        $arguments = $this->getDefaultArguments($labels, $post_type, $plural_slug);
-        if ($override_arguments) {
-            if (isset($override_arguments['labels'])) {
-                $labels = array_merge($arguments['labels'], $override_arguments['labels']);
-            }
-            $arguments = array_merge($arguments, $override_arguments);
-            $arguments['labels'] = $labels;
-        }
-        return $arguments;
-    }
+	/**
+	 * @param string $post_type          the actual post type name
+	 * @param string $singular_name      a pre-internationalized string for the singular name of the objects
+	 * @param string $plural_name        a pre-internationalized string for the plural name of the objects
+	 * @param string $singular_slug
+	 * @param string $plural_slug
+	 * @param array  $override_arguments The default values set in this function will be overridden
+	 *                                   by whatever you set in $override_arguments
+	 * @return array
+	 */
+	protected function prepareArguments(
+		$post_type,
+		$singular_name,
+		$plural_name,
+		$singular_slug,
+		$plural_slug,
+		array $override_arguments = array()
+	) {
+		// verify plural slug and singular slug, if they aren't we'll use $singular_name and $plural_name
+		$singular_slug = ! empty($singular_slug) ? $singular_slug : $singular_name;
+		$plural_slug = ! empty($plural_slug) ? $plural_slug : $plural_name;
+		$labels = $this->getLabels(
+			$singular_name,
+			$plural_name,
+			$singular_slug,
+			$plural_slug
+		);
+		// note the page_templates arg in the supports index is something specific to EE.
+		// WordPress doesn't actually have that in their register_post_type api.
+		$arguments = $this->getDefaultArguments($labels, $post_type, $plural_slug);
+		if ($override_arguments) {
+			if (isset($override_arguments['labels'])) {
+				$labels = array_merge($arguments['labels'], $override_arguments['labels']);
+			}
+			$arguments = array_merge($arguments, $override_arguments);
+			$arguments['labels'] = $labels;
+		}
+		return $arguments;
+	}
 
 
-    /**
-     * @param string $singular_name
-     * @param string $plural_name
-     * @param string $singular_slug
-     * @param string $plural_slug
-     * @return array
-     */
-    private function getLabels($singular_name, $plural_name, $singular_slug, $plural_slug)
-    {
-        return array(
-            'name'                     => $plural_name,
-            'singular_name'            => $singular_name,
-            'singular_slug'            => $singular_slug,
-            'plural_slug'              => $plural_slug,
-            'add_new'                  => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('Add New %s', 'Add New Event', 'event_espresso'),
-                $singular_name
-            ),
-            'add_new_item'             => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('Add New %s', 'Add New Event', 'event_espresso'),
-                $singular_name
-            ),
-            'edit_item'                => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('Edit %s', 'Edit Event', 'event_espresso'),
-                $singular_name
-            ),
-            'new_item'                 => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('New %s', 'New Event', 'event_espresso'),
-                $singular_name
-            ),
-            'all_items'                => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('All %s', 'All Events', 'event_espresso'),
-                $plural_name
-            ),
-            'view_item'                => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('View %s', 'View Event', 'event_espresso'),
-                $singular_name
-            ),
-            'view_items'               => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('View %s', 'View Events', 'event_espresso'),
-                $plural_name
-            ),
-            'archives'                 => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('%s Archives', 'Event Archives', 'event_espresso'),
-                $singular_name
-            ),
-            'attributes'               => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('%s Attributes', 'Event Attributes', 'event_espresso'),
-                $singular_name
-            ),
-            'insert_into_item'         => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('Insert into this %s', 'Insert into this Event', 'event_espresso'),
-                $singular_name
-            ),
-            'uploaded_to_this_item'    => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('Uploaded to this %s', 'Uploaded to this Event', 'event_espresso'),
-                $singular_name
-            ),
-            'filter_items_list'        => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('Filter %s list', 'Filter Events list', 'event_espresso'),
-                $plural_name
-            ),
-            'items_list_navigation'    => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('%s list navigation', 'Events list navigation', 'event_espresso'),
-                $plural_name
-            ),
-            'items_list'               => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('%s list', 'Events list', 'event_espresso'),
-                $plural_name
-            ),
-            'item_published'           => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('%s published', 'Event published', 'event_espresso'),
-                $singular_name
-            ),
-            'item_published_privately' => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('%s published privately', 'Event published privately', 'event_espresso'),
-                $singular_name
-            ),
-            'item_reverted_to_draft'   => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('%s reverted to draft', 'Event reverted to draft', 'event_espresso'),
-                $singular_name
-            ),
-            'item_scheduled'           => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('%s scheduled', 'Event scheduled', 'event_espresso'),
-                $singular_name
-            ),
-            'item_updated'             => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('%s updated', 'Event updated', 'event_espresso'),
-                $singular_name
-            ),
-            'search_items'             => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('Search %s', 'Search Events', 'event_espresso'),
-                $plural_name
-            ),
-            'not_found'                => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('No %s found', 'No Events found', 'event_espresso'),
-                $plural_name
-            ),
-            'not_found_in_trash'       => sprintf(
-                /* Translators: Post Type Label */
-                esc_html_x('No %s found in Trash', 'No Events found in Trash', 'event_espresso'),
-                $plural_name
-            ),
-            'parent_item_colon'        => '',
-            'menu_name'                => $plural_name,
-        );
-    }
+	/**
+	 * @param string $singular_name
+	 * @param string $plural_name
+	 * @param string $singular_slug
+	 * @param string $plural_slug
+	 * @return array
+	 */
+	private function getLabels($singular_name, $plural_name, $singular_slug, $plural_slug)
+	{
+		return array(
+			'name'                     => $plural_name,
+			'singular_name'            => $singular_name,
+			'singular_slug'            => $singular_slug,
+			'plural_slug'              => $plural_slug,
+			'add_new'                  => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('Add New %s', 'Add New Event', 'event_espresso'),
+				$singular_name
+			),
+			'add_new_item'             => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('Add New %s', 'Add New Event', 'event_espresso'),
+				$singular_name
+			),
+			'edit_item'                => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('Edit %s', 'Edit Event', 'event_espresso'),
+				$singular_name
+			),
+			'new_item'                 => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('New %s', 'New Event', 'event_espresso'),
+				$singular_name
+			),
+			'all_items'                => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('All %s', 'All Events', 'event_espresso'),
+				$plural_name
+			),
+			'view_item'                => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('View %s', 'View Event', 'event_espresso'),
+				$singular_name
+			),
+			'view_items'               => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('View %s', 'View Events', 'event_espresso'),
+				$plural_name
+			),
+			'archives'                 => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('%s Archives', 'Event Archives', 'event_espresso'),
+				$singular_name
+			),
+			'attributes'               => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('%s Attributes', 'Event Attributes', 'event_espresso'),
+				$singular_name
+			),
+			'insert_into_item'         => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('Insert into this %s', 'Insert into this Event', 'event_espresso'),
+				$singular_name
+			),
+			'uploaded_to_this_item'    => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('Uploaded to this %s', 'Uploaded to this Event', 'event_espresso'),
+				$singular_name
+			),
+			'filter_items_list'        => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('Filter %s list', 'Filter Events list', 'event_espresso'),
+				$plural_name
+			),
+			'items_list_navigation'    => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('%s list navigation', 'Events list navigation', 'event_espresso'),
+				$plural_name
+			),
+			'items_list'               => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('%s list', 'Events list', 'event_espresso'),
+				$plural_name
+			),
+			'item_published'           => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('%s published', 'Event published', 'event_espresso'),
+				$singular_name
+			),
+			'item_published_privately' => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('%s published privately', 'Event published privately', 'event_espresso'),
+				$singular_name
+			),
+			'item_reverted_to_draft'   => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('%s reverted to draft', 'Event reverted to draft', 'event_espresso'),
+				$singular_name
+			),
+			'item_scheduled'           => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('%s scheduled', 'Event scheduled', 'event_espresso'),
+				$singular_name
+			),
+			'item_updated'             => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('%s updated', 'Event updated', 'event_espresso'),
+				$singular_name
+			),
+			'search_items'             => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('Search %s', 'Search Events', 'event_espresso'),
+				$plural_name
+			),
+			'not_found'                => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('No %s found', 'No Events found', 'event_espresso'),
+				$plural_name
+			),
+			'not_found_in_trash'       => sprintf(
+				/* Translators: Post Type Label */
+				esc_html_x('No %s found in Trash', 'No Events found in Trash', 'event_espresso'),
+				$plural_name
+			),
+			'parent_item_colon'        => '',
+			'menu_name'                => $plural_name,
+		);
+	}
 
 
-    /**
-     * @param array  $labels
-     * @param string $post_type
-     * @param string $plural_slug
-     * @return array
-     */
-    private function getDefaultArguments(array $labels, $post_type, $plural_slug)
-    {
-        return array(
-            'labels'             => $labels,
-            'public'             => true,
-            'publicly_queryable' => true,
-            'show_ui'            => false,
-            'show_ee_ui'         => true,
-            'show_in_menu'       => false,
-            'show_in_nav_menus'  => false,
-            'query_var'          => true,
-            'rewrite'            => apply_filters(
-                'FHEE__EventEspresso_core_domain_entities_custom_post_types_RegisterCustomPostTypes__getDefaultArguments__rewrite',
-                // legacy filter applied for now,
-                // later on we'll run a has_filter($tag) check and throw a doing_it_wrong() notice
-                apply_filters(
-                    'FHEE__EE_Register_CPTs__register_CPT__rewrite',
-                    array('slug' => $plural_slug),
-                    $post_type
-                ),
-                $post_type,
-                $plural_slug
-            ),
-            'capability_type'    => 'post',
-            'map_meta_cap'       => true,
-            'has_archive'        => true,
-            'hierarchical'       => false,
-            'menu_position'      => null,
-            'supports'           => array(
-                'title',
-                'editor',
-                'author',
-                'thumbnail',
-                'excerpt',
-                'custom-fields',
-                'comments',
-            ),
-        );
-    }
+	/**
+	 * @param array  $labels
+	 * @param string $post_type
+	 * @param string $plural_slug
+	 * @return array
+	 */
+	private function getDefaultArguments(array $labels, $post_type, $plural_slug)
+	{
+		return array(
+			'labels'             => $labels,
+			'public'             => true,
+			'publicly_queryable' => true,
+			'show_ui'            => false,
+			'show_ee_ui'         => true,
+			'show_in_menu'       => false,
+			'show_in_nav_menus'  => false,
+			'query_var'          => true,
+			'rewrite'            => apply_filters(
+				'FHEE__EventEspresso_core_domain_entities_custom_post_types_RegisterCustomPostTypes__getDefaultArguments__rewrite',
+				// legacy filter applied for now,
+				// later on we'll run a has_filter($tag) check and throw a doing_it_wrong() notice
+				apply_filters(
+					'FHEE__EE_Register_CPTs__register_CPT__rewrite',
+					array('slug' => $plural_slug),
+					$post_type
+				),
+				$post_type,
+				$plural_slug
+			),
+			'capability_type'    => 'post',
+			'map_meta_cap'       => true,
+			'has_archive'        => true,
+			'hierarchical'       => false,
+			'menu_position'      => null,
+			'supports'           => array(
+				'title',
+				'editor',
+				'author',
+				'thumbnail',
+				'excerpt',
+				'custom-fields',
+				'comments',
+			),
+		);
+	}
 }

core/domain/values/assets/Asset.php

Spacing +2 added lines, -2 removed lines

@@ -82,7 +82,7 @@ 
      */
     private function setType($type)
     {
-        if (! in_array($type, $this->validAssetTypes(), true)) {
+        if ( ! in_array($type, $this->validAssetTypes(), true)) {
             throw new InvalidDataTypeException(
                 'Asset::$type',
                 $type,
@@ -99,7 +99,7 @@ 
      */
     private function setHandle($handle)
     {
-        if (! is_string($handle)) {
+        if ( ! is_string($handle)) {
             throw new InvalidDataTypeException(
                 '$handle',
                 $handle,

Indentation +202 added lines, -202 removed lines

@@ -15,206 +15,206 @@
  */
 abstract class Asset
 {
-    /**
-     * indicates the file extension for a CSS file
-     */
-    const EXT_CSS = '.css';
-
-    /**
-     * indicates the file extension for a JS file
-     */
-    const EXT_JS = '.js';
-
-    /**
-     * indicates the file extension for a JS file
-     */
-    const EXT_PHP = '.php';
-
-    /**
-     * indicates the file extension for a build distribution CSS file
-     */
-    const FILE_EXTENSION_DISTRIBUTION_CSS = '.dist.css';
-
-    /**
-     * indicates the file extension for a build distribution JS file
-     */
-    const FILE_EXTENSION_DISTRIBUTION_JS = '.dist.js';
-
-    /**
-     * Indicates the file extension for a build distribution dependencies json file.
-     */
-    const FILE_EXTENSION_DISTRIBUTION_DEPS = '.dist.deps.php';
-
-    /**
-     * indicates a Cascading Style Sheet asset
-     */
-    const TYPE_CSS = 'css';
-
-    /**
-     * indicates a Javascript asset
-     */
-    const TYPE_JS = 'js';
-
-    /**
-     * indicates a JSON asset
-     */
-    CONST TYPE_JSON = 'json';
-    /**
-     * indicates a PHP asset
-     */
-    CONST TYPE_PHP = 'php';
-
-    /**
-     * indicates a Javascript manifest file
-     */
-    const TYPE_MANIFEST = 'manifest';
-
-    /**
-     * @var DomainInterface $domain
-     */
-    protected $domain;
-
-    /**
-     * @var string $type
-     */
-    private $type;
-
-    /**
-     * @var string $handle
-     */
-    private $handle;
-
-    /**
-     * @var bool $registered
-     */
-    private $registered = false;
-
-    /**
-     * @var bool $enqueue_immediately
-     */
-    private $enqueue_immediately = false;
-
-
-    /**
-     * Asset constructor.
-     *
-     * @param                 $type
-     * @param string          $handle
-     * @param DomainInterface $domain
-     * @throws InvalidDataTypeException
-     */
-    public function __construct($type, $handle, DomainInterface $domain)
-    {
-        $this->domain = $domain;
-        $this->setType($type);
-        $this->setHandle($handle);
-    }
-
-
-    /**
-     * @return array
-     */
-    public function validAssetTypes()
-    {
-        return array(
-            Asset::TYPE_CSS,
-            Asset::TYPE_JS,
-            Asset::TYPE_MANIFEST,
-        );
-    }
-
-
-    /**
-     * @param string $type
-     * @throws InvalidDataTypeException
-     */
-    private function setType($type)
-    {
-        if (! in_array($type, $this->validAssetTypes(), true)) {
-            throw new InvalidDataTypeException(
-                'Asset::$type',
-                $type,
-                'one of the TYPE_* class constants on \EventEspresso\core\domain\values\Asset is required'
-            );
-        }
-        $this->type = $type;
-    }
-
-
-    /**
-     * @param string $handle
-     * @throws InvalidDataTypeException
-     */
-    private function setHandle($handle)
-    {
-        if (! is_string($handle)) {
-            throw new InvalidDataTypeException(
-                '$handle',
-                $handle,
-                'string'
-            );
-        }
-        $this->handle = $handle;
-    }
-
-
-    /**
-     * @return string
-     */
-    public function assetNamespace()
-    {
-        return $this->domain->assetNamespace();
-    }
-
-
-    /**
-     * @return string
-     */
-    public function type()
-    {
-        return $this->type;
-    }
-
-
-    /**
-     * @return string
-     */
-    public function handle()
-    {
-        return $this->handle;
-    }
-
-    /**
-     * @return bool
-     */
-    public function isRegistered()
-    {
-        return $this->registered;
-    }
-
-    /**
-     * @param bool $registered
-     */
-    public function setRegistered($registered = true)
-    {
-        $this->registered = filter_var($registered, FILTER_VALIDATE_BOOLEAN);
-    }
-
-
-    /**
-     * @return bool
-     */
-    public function enqueueImmediately()
-    {
-        return $this->enqueue_immediately;
-    }
-
-
-    /**
-     * @param bool $enqueue_immediately
-     */
-    public function setEnqueueImmediately($enqueue_immediately = true)
-    {
-        $this->enqueue_immediately = filter_var($enqueue_immediately, FILTER_VALIDATE_BOOLEAN);
-    }
+	/**
+	 * indicates the file extension for a CSS file
+	 */
+	const EXT_CSS = '.css';
+
+	/**
+	 * indicates the file extension for a JS file
+	 */
+	const EXT_JS = '.js';
+
+	/**
+	 * indicates the file extension for a JS file
+	 */
+	const EXT_PHP = '.php';
+
+	/**
+	 * indicates the file extension for a build distribution CSS file
+	 */
+	const FILE_EXTENSION_DISTRIBUTION_CSS = '.dist.css';
+
+	/**
+	 * indicates the file extension for a build distribution JS file
+	 */
+	const FILE_EXTENSION_DISTRIBUTION_JS = '.dist.js';
+
+	/**
+	 * Indicates the file extension for a build distribution dependencies json file.
+	 */
+	const FILE_EXTENSION_DISTRIBUTION_DEPS = '.dist.deps.php';
+
+	/**
+	 * indicates a Cascading Style Sheet asset
+	 */
+	const TYPE_CSS = 'css';
+
+	/**
+	 * indicates a Javascript asset
+	 */
+	const TYPE_JS = 'js';
+
+	/**
+	 * indicates a JSON asset
+	 */
+	CONST TYPE_JSON = 'json';
+	/**
+	 * indicates a PHP asset
+	 */
+	CONST TYPE_PHP = 'php';
+
+	/**
+	 * indicates a Javascript manifest file
+	 */
+	const TYPE_MANIFEST = 'manifest';
+
+	/**
+	 * @var DomainInterface $domain
+	 */
+	protected $domain;
+
+	/**
+	 * @var string $type
+	 */
+	private $type;
+
+	/**
+	 * @var string $handle
+	 */
+	private $handle;
+
+	/**
+	 * @var bool $registered
+	 */
+	private $registered = false;
+
+	/**
+	 * @var bool $enqueue_immediately
+	 */
+	private $enqueue_immediately = false;
+
+
+	/**
+	 * Asset constructor.
+	 *
+	 * @param                 $type
+	 * @param string          $handle
+	 * @param DomainInterface $domain
+	 * @throws InvalidDataTypeException
+	 */
+	public function __construct($type, $handle, DomainInterface $domain)
+	{
+		$this->domain = $domain;
+		$this->setType($type);
+		$this->setHandle($handle);
+	}
+
+
+	/**
+	 * @return array
+	 */
+	public function validAssetTypes()
+	{
+		return array(
+			Asset::TYPE_CSS,
+			Asset::TYPE_JS,
+			Asset::TYPE_MANIFEST,
+		);
+	}
+
+
+	/**
+	 * @param string $type
+	 * @throws InvalidDataTypeException
+	 */
+	private function setType($type)
+	{
+		if (! in_array($type, $this->validAssetTypes(), true)) {
+			throw new InvalidDataTypeException(
+				'Asset::$type',
+				$type,
+				'one of the TYPE_* class constants on \EventEspresso\core\domain\values\Asset is required'
+			);
+		}
+		$this->type = $type;
+	}
+
+
+	/**
+	 * @param string $handle
+	 * @throws InvalidDataTypeException
+	 */
+	private function setHandle($handle)
+	{
+		if (! is_string($handle)) {
+			throw new InvalidDataTypeException(
+				'$handle',
+				$handle,
+				'string'
+			);
+		}
+		$this->handle = $handle;
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function assetNamespace()
+	{
+		return $this->domain->assetNamespace();
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function type()
+	{
+		return $this->type;
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function handle()
+	{
+		return $this->handle;
+	}
+
+	/**
+	 * @return bool
+	 */
+	public function isRegistered()
+	{
+		return $this->registered;
+	}
+
+	/**
+	 * @param bool $registered
+	 */
+	public function setRegistered($registered = true)
+	{
+		$this->registered = filter_var($registered, FILTER_VALIDATE_BOOLEAN);
+	}
+
+
+	/**
+	 * @return bool
+	 */
+	public function enqueueImmediately()
+	{
+		return $this->enqueue_immediately;
+	}
+
+
+	/**
+	 * @param bool $enqueue_immediately
+	 */
+	public function setEnqueueImmediately($enqueue_immediately = true)
+	{
+		$this->enqueue_immediately = filter_var($enqueue_immediately, FILTER_VALIDATE_BOOLEAN);
+	}
 }

core/domain/values/assets/StylesheetAsset.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
      */
     private function setMedia($media)
     {
-        if (! is_string($media)) {
+        if ( ! is_string($media)) {
             throw new InvalidDataTypeException(
                 '$media',
                 $media,

Indentation +59 added lines, -59 removed lines

@@ -17,71 +17,71 @@
 class StylesheetAsset extends BrowserAsset
 {
 
-    /**
-     * @var string $media
-     */
-    private $media;
+	/**
+	 * @var string $media
+	 */
+	private $media;
 
 
-    /**
-     * CssFile constructor.
-     *
-     * @param                 $handle
-     * @param string          $source
-     * @param array           $dependencies
-     * @param DomainInterface $domain
-     * @param string          $media
-     * @param string          $version
-     * @throws InvalidDataTypeException
-     * @throws DomainException
-     */
-    public function __construct(
-        $handle,
-        $source,
-        array $dependencies,
-        DomainInterface $domain,
-        $media = 'all',
-        $version = ''
-    ) {
-        parent::__construct(Asset::TYPE_CSS, $handle, $source, $dependencies, $domain, $version);
-        $this->setMedia($media);
-    }
+	/**
+	 * CssFile constructor.
+	 *
+	 * @param                 $handle
+	 * @param string          $source
+	 * @param array           $dependencies
+	 * @param DomainInterface $domain
+	 * @param string          $media
+	 * @param string          $version
+	 * @throws InvalidDataTypeException
+	 * @throws DomainException
+	 */
+	public function __construct(
+		$handle,
+		$source,
+		array $dependencies,
+		DomainInterface $domain,
+		$media = 'all',
+		$version = ''
+	) {
+		parent::__construct(Asset::TYPE_CSS, $handle, $source, $dependencies, $domain, $version);
+		$this->setMedia($media);
+	}
 
 
-    /**
-     * @return string
-     */
-    public function media()
-    {
-        return $this->media;
-    }
+	/**
+	 * @return string
+	 */
+	public function media()
+	{
+		return $this->media;
+	}
 
 
-    /**
-     * @param string $media
-     * @throws InvalidDataTypeException
-     */
-    private function setMedia($media)
-    {
-        if (! is_string($media)) {
-            throw new InvalidDataTypeException(
-                '$media',
-                $media,
-                'string'
-            );
-        }
-        $this->media = $media;
-    }
+	/**
+	 * @param string $media
+	 * @throws InvalidDataTypeException
+	 */
+	private function setMedia($media)
+	{
+		if (! is_string($media)) {
+			throw new InvalidDataTypeException(
+				'$media',
+				$media,
+				'string'
+			);
+		}
+		$this->media = $media;
+	}
 
 
-    /**
-     * @since 4.9.62.p
-     */
-    public function enqueueAsset()
-    {
-        if ($this->source() === '') {
-            return;
-        }
-        wp_enqueue_style($this->handle());
-    }
+	/**
+	 * @since 4.9.62.p
+	 */
+	public function enqueueAsset()
+	{
+		if ($this->source() === '') {
+			return;
+		}
+		wp_enqueue_style($this->handle());
+	}
 }

widgets/EspressoWidget.php

Spacing +1 added lines, -1 removed lines

@@ -35,6 +35,6 @@
      */
     public static function getIdBase($widget_class)
     {
-        return sanitize_title(str_replace(array('EEW_', '_'), array('EE_', '-'), $widget_class)) . '-widget';
+        return sanitize_title(str_replace(array('EEW_', '_'), array('EE_', '-'), $widget_class)).'-widget';
     }
 }

Indentation +22 added lines, -22 removed lines

@@ -11,28 +11,28 @@
  */
 class EspressoWidget extends \WP_Widget
 {
-    /**
-     * @param string $name
-     * @param array  $widget_options
-     * @param array  $control_options
-     */
-    public function __construct($name = '', array $widget_options = array(), array $control_options = array())
-    {
-        $id_base = EspressoWidget::getIdBase(get_class($this));
-        $control_options['id_base'] = $id_base;
-        $control_options['height'] = isset($control_options['height']) ? $control_options['height'] : 300;
-        $control_options['width'] = isset($control_options['width']) ? $control_options['width'] : 350;
-        // Register widget with WordPress
-        parent::__construct($id_base, $name, $widget_options, $control_options);
-    }
+	/**
+	 * @param string $name
+	 * @param array  $widget_options
+	 * @param array  $control_options
+	 */
+	public function __construct($name = '', array $widget_options = array(), array $control_options = array())
+	{
+		$id_base = EspressoWidget::getIdBase(get_class($this));
+		$control_options['id_base'] = $id_base;
+		$control_options['height'] = isset($control_options['height']) ? $control_options['height'] : 300;
+		$control_options['width'] = isset($control_options['width']) ? $control_options['width'] : 350;
+		// Register widget with WordPress
+		parent::__construct($id_base, $name, $widget_options, $control_options);
+	}
 
 
-    /**
-     * @param string $widget_class
-     * @return string
-     */
-    public static function getIdBase($widget_class)
-    {
-        return sanitize_title(str_replace(array('EEW_', '_'), array('EE_', '-'), $widget_class)) . '-widget';
-    }
+	/**
+	 * @param string $widget_class
+	 * @return string
+	 */
+	public static function getIdBase($widget_class)
+	{
+		return sanitize_title(str_replace(array('EEW_', '_'), array('EE_', '-'), $widget_class)) . '-widget';
+	}
 }

modules/ticket_selector/TicketSelectorIframe.php

Spacing +2 added lines, -2 removed lines

@@ -60,7 +60,7 @@ 
                 $this
             )
         );
-        if (! apply_filters('FHEE__EED_Ticket_Selector__ticket_selector_iframe__load_theme_css', false, $this)) {
+        if ( ! apply_filters('FHEE__EED_Ticket_Selector__ticket_selector_iframe__load_theme_css', false, $this)) {
             $this->addStylesheets(array('site_theme' => ''));
         }
         $this->addScripts(
@@ -79,7 +79,7 @@ 
             array(),
             $this
         );
-        if (! empty($js_attributes)) {
+        if ( ! empty($js_attributes)) {
             $this->addScriptAttributes($js_attributes);
         }
         $this->addLocalizedVars(

Indentation +73 added lines, -73 removed lines

@@ -19,77 +19,77 @@
  */
 class TicketSelectorIframe extends Iframe
 {
-    /**
-     * TicketSelectorIframe constructor.
-     *
-     * @param EEM_Event        $event_model
-     * @param CurrentPage      $current_page
-     * @param RequestInterface $request
-     * @throws EE_Error
-     */
-    public function __construct(EEM_Event $event_model, CurrentPage $current_page, RequestInterface $request)
-    {
-        $current_page->setEspressoPage(true);
-        $ticket_selector = LoaderFactory::getLoader()->getNew(DisplayTicketSelector::class);
-        $ticket_selector->setIframe();
-        $event = $event_model->get_one_by_ID($request->getRequestParam('event', 0, 'int'));
-        parent::__construct(
-            esc_html__('Ticket Selector', 'event_espresso'),
-            $ticket_selector->display($event)
-        );
-        $this->addStylesheets(
-            apply_filters(
-                'FHEE__EED_Ticket_Selector__ticket_selector_iframe__css',
-                array(
-                    'ticket_selector_embed' => TICKET_SELECTOR_ASSETS_URL
-                                               . 'ticket_selector_embed.css?ver='
-                                               . EVENT_ESPRESSO_VERSION,
-                    'ticket_selector'       => TICKET_SELECTOR_ASSETS_URL
-                                               . 'ticket_selector.css?ver='
-                                               . EVENT_ESPRESSO_VERSION,
-                ),
-                $this
-            )
-        );
-        if (! apply_filters('FHEE__EED_Ticket_Selector__ticket_selector_iframe__load_theme_css', false, $this)) {
-            $this->addStylesheets(array('site_theme' => ''));
-        }
-        $this->addScripts(
-            apply_filters(
-                'FHEE__EED_Ticket_Selector__ticket_selector_iframe__js',
-                array(
-                    'ticket_selector_iframe_embed' => TICKET_SELECTOR_ASSETS_URL
-                                                      . 'ticket_selector_iframe_embed.js?ver='
-                                                      . EVENT_ESPRESSO_VERSION,
-                ),
-                $this
-            )
-        );
-        $js_attributes = apply_filters(
-            'FHEE__EventEspresso_modules_ticket_selector_TicketSelectorIframe__construct__js_attributes',
-            array(),
-            $this
-        );
-        if (! empty($js_attributes)) {
-            $this->addScriptAttributes($js_attributes);
-        }
-        $this->addLocalizedVars(
-            apply_filters(
-                'FHEE__EventEspresso_modules_ticket_selector_TicketSelectorIframe__construct__localized_vars',
-                array(
-                    'ticket_selector_iframe' => true,
-                    'EEDTicketSelectorMsg'   => wp_strip_all_tags(
-                        __(
-                            'Please choose at least one ticket before continuing.',
-                            'event_espresso'
-                        )
-                    ),
-                )
-            )
-        );
-        do_action(
-            'AHEE__EventEspresso_modules_ticket_selector_TicketSelectorIframe__construct__complete',
-            $this
-        );
-    }
+	/**
+	 * TicketSelectorIframe constructor.
+	 *
+	 * @param EEM_Event        $event_model
+	 * @param CurrentPage      $current_page
+	 * @param RequestInterface $request
+	 * @throws EE_Error
+	 */
+	public function __construct(EEM_Event $event_model, CurrentPage $current_page, RequestInterface $request)
+	{
+		$current_page->setEspressoPage(true);
+		$ticket_selector = LoaderFactory::getLoader()->getNew(DisplayTicketSelector::class);
+		$ticket_selector->setIframe();
+		$event = $event_model->get_one_by_ID($request->getRequestParam('event', 0, 'int'));
+		parent::__construct(
+			esc_html__('Ticket Selector', 'event_espresso'),
+			$ticket_selector->display($event)
+		);
+		$this->addStylesheets(
+			apply_filters(
+				'FHEE__EED_Ticket_Selector__ticket_selector_iframe__css',
+				array(
+					'ticket_selector_embed' => TICKET_SELECTOR_ASSETS_URL
+											   . 'ticket_selector_embed.css?ver='
+											   . EVENT_ESPRESSO_VERSION,
+					'ticket_selector'       => TICKET_SELECTOR_ASSETS_URL
+											   . 'ticket_selector.css?ver='
+											   . EVENT_ESPRESSO_VERSION,
+				),
+				$this
+			)
+		);
+		if (! apply_filters('FHEE__EED_Ticket_Selector__ticket_selector_iframe__load_theme_css', false, $this)) {
+			$this->addStylesheets(array('site_theme' => ''));
+		}
+		$this->addScripts(
+			apply_filters(
+				'FHEE__EED_Ticket_Selector__ticket_selector_iframe__js',
+				array(
+					'ticket_selector_iframe_embed' => TICKET_SELECTOR_ASSETS_URL
+													  . 'ticket_selector_iframe_embed.js?ver='
+													  . EVENT_ESPRESSO_VERSION,
+				),
+				$this
+			)
+		);
+		$js_attributes = apply_filters(
+			'FHEE__EventEspresso_modules_ticket_selector_TicketSelectorIframe__construct__js_attributes',
+			array(),
+			$this
+		);
+		if (! empty($js_attributes)) {
+			$this->addScriptAttributes($js_attributes);
+		}
+		$this->addLocalizedVars(
+			apply_filters(
+				'FHEE__EventEspresso_modules_ticket_selector_TicketSelectorIframe__construct__localized_vars',
+				array(
+					'ticket_selector_iframe' => true,
+					'EEDTicketSelectorMsg'   => wp_strip_all_tags(
+						__(
+							'Please choose at least one ticket before continuing.',
+							'event_espresso'
+						)
+					),
+				)
+			)
+		);
+		do_action(
+			'AHEE__EventEspresso_modules_ticket_selector_TicketSelectorIframe__construct__complete',
+			$this
+		);
+	}
 }

caffeinated/modules/recaptcha/ReCaptcha/RequestMethod/SocketPost.php

Indentation +83 added lines, -83 removed lines

@@ -33,87 +33,87 @@
  */
 class SocketPost implements RequestMethod
 {
-    /**
-     * reCAPTCHA service host.
-     *
-     * @const string
-     */
-    const RECAPTCHA_HOST = 'www.google.com';
-
-    /**
-     * @const string reCAPTCHA service path
-     */
-    const SITE_VERIFY_PATH = '/recaptcha/api/siteverify';
-
-    /**
-     * @const string Bad request error
-     */
-    const BAD_REQUEST = '{"success": false, "error-codes": ["invalid-request"]}';
-
-    /**
-     * @const string Bad response error
-     */
-    const BAD_RESPONSE = '{"success": false, "error-codes": ["invalid-response"]}';
-
-    /**
-     * Socket to the reCAPTCHA service
-     *
-     * @var Socket
-     */
-    private $socket;
-
-    /**
-     * Constructor
-     *
-     * @param \ReCaptcha\RequestMethod\Socket $socket optional socket, injectable for testing
-     */
-    public function __construct(Socket $socket = null)
-    {
-        if (! is_null($socket)) {
-            $this->socket = $socket;
-        } else {
-            $this->socket = new Socket();
-        }
-    }
-
-    /**
-     * Submit the POST request with the specified parameters.
-     *
-     * @param RequestParameters $params Request parameters
-     * @return string Body of the reCAPTCHA response
-     */
-    public function submit(RequestParameters $params)
-    {
-        $errno = 0;
-        $errstr = '';
-
-        if ($this->socket->fsockopen('ssl://' . self::RECAPTCHA_HOST, 443, $errno, $errstr, 30) !== false) {
-            $content = $params->toQueryString();
-
-            $request = "POST " . self::SITE_VERIFY_PATH . " HTTP/1.1\r\n";
-            $request .= "Host: " . self::RECAPTCHA_HOST . "\r\n";
-            $request .= "Content-Type: application/x-www-form-urlencoded\r\n";
-            $request .= "Content-length: " . strlen($content) . "\r\n";
-            $request .= "Connection: close\r\n\r\n";
-            $request .= $content . "\r\n\r\n";
-
-            $this->socket->fwrite($request);
-            $response = '';
-
-            while (! $this->socket->feof()) {
-                $response .= $this->socket->fgets(4096);
-            }
-
-            $this->socket->fclose();
-
-            if (0 === strpos($response, 'HTTP/1.1 200 OK')) {
-                $parts = preg_split("#\n\s*\n#Uis", $response);
-                return $parts[1];
-            }
-
-            return self::BAD_RESPONSE;
-        }
-
-        return self::BAD_REQUEST;
-    }
+	/**
+	 * reCAPTCHA service host.
+	 *
+	 * @const string
+	 */
+	const RECAPTCHA_HOST = 'www.google.com';
+
+	/**
+	 * @const string reCAPTCHA service path
+	 */
+	const SITE_VERIFY_PATH = '/recaptcha/api/siteverify';
+
+	/**
+	 * @const string Bad request error
+	 */
+	const BAD_REQUEST = '{"success": false, "error-codes": ["invalid-request"]}';
+
+	/**
+	 * @const string Bad response error
+	 */
+	const BAD_RESPONSE = '{"success": false, "error-codes": ["invalid-response"]}';
+
+	/**
+	 * Socket to the reCAPTCHA service
+	 *
+	 * @var Socket
+	 */
+	private $socket;
+
+	/**
+	 * Constructor
+	 *
+	 * @param \ReCaptcha\RequestMethod\Socket $socket optional socket, injectable for testing
+	 */
+	public function __construct(Socket $socket = null)
+	{
+		if (! is_null($socket)) {
+			$this->socket = $socket;
+		} else {
+			$this->socket = new Socket();
+		}
+	}
+
+	/**
+	 * Submit the POST request with the specified parameters.
+	 *
+	 * @param RequestParameters $params Request parameters
+	 * @return string Body of the reCAPTCHA response
+	 */
+	public function submit(RequestParameters $params)
+	{
+		$errno = 0;
+		$errstr = '';
+
+		if ($this->socket->fsockopen('ssl://' . self::RECAPTCHA_HOST, 443, $errno, $errstr, 30) !== false) {
+			$content = $params->toQueryString();
+
+			$request = "POST " . self::SITE_VERIFY_PATH . " HTTP/1.1\r\n";
+			$request .= "Host: " . self::RECAPTCHA_HOST . "\r\n";
+			$request .= "Content-Type: application/x-www-form-urlencoded\r\n";
+			$request .= "Content-length: " . strlen($content) . "\r\n";
+			$request .= "Connection: close\r\n\r\n";
+			$request .= $content . "\r\n\r\n";
+
+			$this->socket->fwrite($request);
+			$response = '';
+
+			while (! $this->socket->feof()) {
+				$response .= $this->socket->fgets(4096);
+			}
+
+			$this->socket->fclose();
+
+			if (0 === strpos($response, 'HTTP/1.1 200 OK')) {
+				$parts = preg_split("#\n\s*\n#Uis", $response);
+				return $parts[1];
+			}
+
+			return self::BAD_RESPONSE;
+		}
+
+		return self::BAD_REQUEST;
+	}
 }

Spacing +7 added lines, -7 removed lines

@@ -69,7 +69,7 @@ 
      */
     public function __construct(Socket $socket = null)
     {
-        if (! is_null($socket)) {
+        if ( ! is_null($socket)) {
             $this->socket = $socket;
         } else {
             $this->socket = new Socket();
@@ -87,20 +87,20 @@ 
         $errno = 0;
         $errstr = '';
 
-        if ($this->socket->fsockopen('ssl://' . self::RECAPTCHA_HOST, 443, $errno, $errstr, 30) !== false) {
+        if ($this->socket->fsockopen('ssl://'.self::RECAPTCHA_HOST, 443, $errno, $errstr, 30) !== false) {
             $content = $params->toQueryString();
 
-            $request = "POST " . self::SITE_VERIFY_PATH . " HTTP/1.1\r\n";
-            $request .= "Host: " . self::RECAPTCHA_HOST . "\r\n";
+            $request = "POST ".self::SITE_VERIFY_PATH." HTTP/1.1\r\n";
+            $request .= "Host: ".self::RECAPTCHA_HOST."\r\n";
             $request .= "Content-Type: application/x-www-form-urlencoded\r\n";
-            $request .= "Content-length: " . strlen($content) . "\r\n";
+            $request .= "Content-length: ".strlen($content)."\r\n";
             $request .= "Connection: close\r\n\r\n";
-            $request .= $content . "\r\n\r\n";
+            $request .= $content."\r\n\r\n";
 
             $this->socket->fwrite($request);
             $response = '';
 
-            while (! $this->socket->feof()) {
+            while ( ! $this->socket->feof()) {
                 $response .= $this->socket->fgets(4096);
             }