eventespresso / event-espresso-core

core/domain/services/cron/jobs/ExpiredTransactionCheck.php

Indentation +248 added lines, -248 removed lines

@@ -17,252 +17,252 @@
 
 class ExpiredTransactionCheck extends CronJob
 {
-    private ?EE_Payment_Processor $payment_processor = null;
-
-    private ?EE_Transaction_Processor $transaction_processor = null;
-
-    /**
-     * array of TXN IDs
-     *
-     * @var array
-     */
-    protected array $expired_transactions = [];
-
-
-    private function loadTransactionProcessor()
-    {
-        $this->transaction_processor = $this->loader->getShared(EE_Transaction_Processor::class);
-    }
-
-
-    private function loadPaymentProcessor()
-    {
-        $this->payment_processor = $this->loader->getShared(EE_Payment_Processor::class);
-    }
-
-
-    public function setHooks(): void
-    {
-        add_action(
-            'AHEE__EE_Cron_Tasks__expired_transaction_check',
-            [$this, 'expiredTransactionCheck']
-        );
-    }
-
-
-    /**
-     * schedule_expired_transaction_check
-     * sets a wp_schedule_single_event() for following up on TXNs after their session has expired
-     *
-     * @param int $timestamp
-     * @param int $TXN_ID
-     */
-    public static function scheduleExpiredTransactionCheck(
-        int $timestamp,
-        int $TXN_ID
-    ): void {
-        // validate $TXN_ID and $timestamp
-        $TXN_ID    = absint($TXN_ID);
-        $timestamp = absint($timestamp);
-        if ($TXN_ID && $timestamp) {
-            wp_schedule_single_event(
-                $timestamp,
-                'AHEE__EE_Cron_Tasks__expired_transaction_check',
-                [$TXN_ID]
-            );
-        }
-    }
-
-
-    /**
-     * this is the callback for the action hook:
-     * 'AHEE__EE_Cron_Tasks__transaction_session_expiration_check'
-     * which is utilized by wp_schedule_single_event()
-     * in \EED_Single_Page_Checkout::_initialize_transaction().
-     * The passed TXN_ID gets added to an array, and then the
-     * process_expired_transactions() function is hooked into
-     * 'AHEE__EE_System__core_loaded_and_ready' which will actually handle the
-     * processing of any failed transactions, because doing so now would be
-     * too early and the required resources may not be available
-     *
-     * @param int $TXN_ID
-     */
-    public function expiredTransactionCheck(int $TXN_ID = 0): void
-    {
-        if (absint($TXN_ID)) {
-            $this->expired_transactions[ $TXN_ID ] = $TXN_ID;
-            add_action(
-                'shutdown',
-                [$this, 'processExpiredTransactions'],
-                5
-            );
-        }
-    }
-
-
-    /**
-     * loops through the $this->expired_transactions array and processes any failed TXNs
-     *
-     * @throws EE_Error
-     * @throws ReflectionException
-     * @throws DomainException
-     * @throws RuntimeException
-     */
-    public function processExpiredTransactions(): void
-    {
-        if (
-            // are there any TXNs that need cleaning up ?
-            empty($this->expired_transactions)
-            // reschedule the cron if we can't hit the db right now
-            || CronUtilities::rescheduleCronForTransactions(
-                [ExpiredTransactionCheck::class, 'scheduleExpiredTransactionCheck'],
-                $this->expired_transactions
-            )
-        ) {
-            return;
-        }
-        $this->loadTransactionProcessor();
-        // set revisit flag for txn processor
-        $this->transaction_processor->set_revisit();
-        foreach ($this->expired_transactions as $TXN_ID) {
-            $transaction = EEM_Transaction::instance()->get_one_by_ID($TXN_ID);
-            // verify transaction and whether it is failed or not
-            if ($transaction instanceof EE_Transaction) {
-                switch ($transaction->status_ID()) {
-
-                    case EEM_Transaction::complete_status_code:
-                        $this->processCompletedTransaction($transaction);
-                        break;
-
-                    case EEM_Transaction::overpaid_status_code:
-                        $this->processOverpaidTransaction($transaction);
-                        break;
-
-                    case EEM_Transaction::incomplete_status_code:
-                        $this->processIncompletedTransaction($transaction);
-                        break;
-
-                    case EEM_Transaction::abandoned_status_code:
-                        $this->processAbandonedTransaction($transaction);
-                        break;
-
-                    case EEM_Transaction::failed_status_code:
-                        $this->processFailedTransaction($transaction);
-                        break;
-                }
-            }
-            unset($this->expired_transactions[ $TXN_ID ]);
-        }
-    }
-
-
-    /**
-     * @param EE_Transaction $transaction
-     * @return void
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    private function processCompletedTransaction(EE_Transaction $transaction)
-    {
-        // Don't update the transaction/registrations if the Primary Registration is Not Approved.
-        $primary_registration = $transaction->primary_registration();
-        if (
-            $primary_registration instanceof EE_Registration
-            && $primary_registration->status_ID() !== RegStatus::AWAITING_REVIEW
-        ) {
-            $this->transaction_processor->update_transaction_and_registrations_after_checkout_or_payment(
-                $transaction,
-                $transaction->last_payment()
-            );
-            do_action(
-                'AHEE__EE_Cron_Tasks__process_expired_transactions__completed_transaction',
-                $transaction
-            );
-        }
-    }
-
-
-    /**
-     * @param EE_Transaction $transaction
-     * @return void
-     */
-    private function processOverpaidTransaction(EE_Transaction $transaction)
-    {
-        do_action(
-            'AHEE__EE_Cron_Tasks__process_expired_transactions__overpaid_transaction',
-            $transaction
-        );
-    }
-
-
-    /**
-     * @param EE_Transaction $transaction
-     * @return void
-     */
-    private function processIncompletedTransaction(EE_Transaction $transaction)
-    {
-        do_action(
-            'AHEE__EE_Cron_Tasks__process_expired_transactions__incomplete_transaction',
-            $transaction
-        );
-        // todo : move business logic into EE_Transaction_Processor for finalizing abandoned transactions
-    }
-
-
-    /**
-     * @param EE_Transaction $transaction
-     * @return void
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    private function processAbandonedTransaction(EE_Transaction $transaction)
-    {
-        // run hook before updating transaction, primarily so
-        // EED_Ticket_Sales_Monitor::process_abandoned_transactions() can release reserved tickets
-        do_action(
-            'AHEE__EE_Cron_Tasks__process_expired_transactions__abandoned_transaction',
-            $transaction
-        );
-        // don't finalize the TXN if it has already been completed
-        if ($transaction->all_reg_steps_completed() !== true) {
-            $this->loadPaymentProcessor();
-            // let's simulate an IPN here which will trigger any notifications that need to go out
-            $this->payment_processor->update_txn_based_on_payment(
-                $transaction,
-                $transaction->last_payment(),
-                true,
-                true
-            );
-        }
-    }
-
-
-    /**
-     * @param EE_Transaction $transaction
-     * @return void
-     */
-    private function processFailedTransaction(EE_Transaction $transaction)
-    {
-        do_action(
-            'AHEE__EE_Cron_Tasks__process_expired_transactions__failed_transaction',
-            $transaction
-        );
-        // todo :
-        // perform garbage collection here and remove clean_out_junk_transactions()
-        // $registrations = $transaction->registrations();
-        // if (! empty($registrations)) {
-        //     foreach ($registrations as $registration) {
-        //         if ($registration instanceof EE_Registration) {
-        //             $delete_registration = true;
-        //             if ($registration->attendee() instanceof EE_Attendee) {
-        //                 $delete_registration = false;
-        //             }
-        //             if ($delete_registration) {
-        //                 $registration->delete_permanently();
-        //                 $registration->delete_related_permanently();
-        //             }
-        //         }
-        //     }
-        // }
-    }
+	private ?EE_Payment_Processor $payment_processor = null;
+
+	private ?EE_Transaction_Processor $transaction_processor = null;
+
+	/**
+	 * array of TXN IDs
+	 *
+	 * @var array
+	 */
+	protected array $expired_transactions = [];
+
+
+	private function loadTransactionProcessor()
+	{
+		$this->transaction_processor = $this->loader->getShared(EE_Transaction_Processor::class);
+	}
+
+
+	private function loadPaymentProcessor()
+	{
+		$this->payment_processor = $this->loader->getShared(EE_Payment_Processor::class);
+	}
+
+
+	public function setHooks(): void
+	{
+		add_action(
+			'AHEE__EE_Cron_Tasks__expired_transaction_check',
+			[$this, 'expiredTransactionCheck']
+		);
+	}
+
+
+	/**
+	 * schedule_expired_transaction_check
+	 * sets a wp_schedule_single_event() for following up on TXNs after their session has expired
+	 *
+	 * @param int $timestamp
+	 * @param int $TXN_ID
+	 */
+	public static function scheduleExpiredTransactionCheck(
+		int $timestamp,
+		int $TXN_ID
+	): void {
+		// validate $TXN_ID and $timestamp
+		$TXN_ID    = absint($TXN_ID);
+		$timestamp = absint($timestamp);
+		if ($TXN_ID && $timestamp) {
+			wp_schedule_single_event(
+				$timestamp,
+				'AHEE__EE_Cron_Tasks__expired_transaction_check',
+				[$TXN_ID]
+			);
+		}
+	}
+
+
+	/**
+	 * this is the callback for the action hook:
+	 * 'AHEE__EE_Cron_Tasks__transaction_session_expiration_check'
+	 * which is utilized by wp_schedule_single_event()
+	 * in \EED_Single_Page_Checkout::_initialize_transaction().
+	 * The passed TXN_ID gets added to an array, and then the
+	 * process_expired_transactions() function is hooked into
+	 * 'AHEE__EE_System__core_loaded_and_ready' which will actually handle the
+	 * processing of any failed transactions, because doing so now would be
+	 * too early and the required resources may not be available
+	 *
+	 * @param int $TXN_ID
+	 */
+	public function expiredTransactionCheck(int $TXN_ID = 0): void
+	{
+		if (absint($TXN_ID)) {
+			$this->expired_transactions[ $TXN_ID ] = $TXN_ID;
+			add_action(
+				'shutdown',
+				[$this, 'processExpiredTransactions'],
+				5
+			);
+		}
+	}
+
+
+	/**
+	 * loops through the $this->expired_transactions array and processes any failed TXNs
+	 *
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 * @throws DomainException
+	 * @throws RuntimeException
+	 */
+	public function processExpiredTransactions(): void
+	{
+		if (
+			// are there any TXNs that need cleaning up ?
+			empty($this->expired_transactions)
+			// reschedule the cron if we can't hit the db right now
+			|| CronUtilities::rescheduleCronForTransactions(
+				[ExpiredTransactionCheck::class, 'scheduleExpiredTransactionCheck'],
+				$this->expired_transactions
+			)
+		) {
+			return;
+		}
+		$this->loadTransactionProcessor();
+		// set revisit flag for txn processor
+		$this->transaction_processor->set_revisit();
+		foreach ($this->expired_transactions as $TXN_ID) {
+			$transaction = EEM_Transaction::instance()->get_one_by_ID($TXN_ID);
+			// verify transaction and whether it is failed or not
+			if ($transaction instanceof EE_Transaction) {
+				switch ($transaction->status_ID()) {
+
+					case EEM_Transaction::complete_status_code:
+						$this->processCompletedTransaction($transaction);
+						break;
+
+					case EEM_Transaction::overpaid_status_code:
+						$this->processOverpaidTransaction($transaction);
+						break;
+
+					case EEM_Transaction::incomplete_status_code:
+						$this->processIncompletedTransaction($transaction);
+						break;
+
+					case EEM_Transaction::abandoned_status_code:
+						$this->processAbandonedTransaction($transaction);
+						break;
+
+					case EEM_Transaction::failed_status_code:
+						$this->processFailedTransaction($transaction);
+						break;
+				}
+			}
+			unset($this->expired_transactions[ $TXN_ID ]);
+		}
+	}
+
+
+	/**
+	 * @param EE_Transaction $transaction
+	 * @return void
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	private function processCompletedTransaction(EE_Transaction $transaction)
+	{
+		// Don't update the transaction/registrations if the Primary Registration is Not Approved.
+		$primary_registration = $transaction->primary_registration();
+		if (
+			$primary_registration instanceof EE_Registration
+			&& $primary_registration->status_ID() !== RegStatus::AWAITING_REVIEW
+		) {
+			$this->transaction_processor->update_transaction_and_registrations_after_checkout_or_payment(
+				$transaction,
+				$transaction->last_payment()
+			);
+			do_action(
+				'AHEE__EE_Cron_Tasks__process_expired_transactions__completed_transaction',
+				$transaction
+			);
+		}
+	}
+
+
+	/**
+	 * @param EE_Transaction $transaction
+	 * @return void
+	 */
+	private function processOverpaidTransaction(EE_Transaction $transaction)
+	{
+		do_action(
+			'AHEE__EE_Cron_Tasks__process_expired_transactions__overpaid_transaction',
+			$transaction
+		);
+	}
+
+
+	/**
+	 * @param EE_Transaction $transaction
+	 * @return void
+	 */
+	private function processIncompletedTransaction(EE_Transaction $transaction)
+	{
+		do_action(
+			'AHEE__EE_Cron_Tasks__process_expired_transactions__incomplete_transaction',
+			$transaction
+		);
+		// todo : move business logic into EE_Transaction_Processor for finalizing abandoned transactions
+	}
+
+
+	/**
+	 * @param EE_Transaction $transaction
+	 * @return void
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	private function processAbandonedTransaction(EE_Transaction $transaction)
+	{
+		// run hook before updating transaction, primarily so
+		// EED_Ticket_Sales_Monitor::process_abandoned_transactions() can release reserved tickets
+		do_action(
+			'AHEE__EE_Cron_Tasks__process_expired_transactions__abandoned_transaction',
+			$transaction
+		);
+		// don't finalize the TXN if it has already been completed
+		if ($transaction->all_reg_steps_completed() !== true) {
+			$this->loadPaymentProcessor();
+			// let's simulate an IPN here which will trigger any notifications that need to go out
+			$this->payment_processor->update_txn_based_on_payment(
+				$transaction,
+				$transaction->last_payment(),
+				true,
+				true
+			);
+		}
+	}
+
+
+	/**
+	 * @param EE_Transaction $transaction
+	 * @return void
+	 */
+	private function processFailedTransaction(EE_Transaction $transaction)
+	{
+		do_action(
+			'AHEE__EE_Cron_Tasks__process_expired_transactions__failed_transaction',
+			$transaction
+		);
+		// todo :
+		// perform garbage collection here and remove clean_out_junk_transactions()
+		// $registrations = $transaction->registrations();
+		// if (! empty($registrations)) {
+		//     foreach ($registrations as $registration) {
+		//         if ($registration instanceof EE_Registration) {
+		//             $delete_registration = true;
+		//             if ($registration->attendee() instanceof EE_Attendee) {
+		//                 $delete_registration = false;
+		//             }
+		//             if ($delete_registration) {
+		//                 $registration->delete_permanently();
+		//                 $registration->delete_related_permanently();
+		//             }
+		//         }
+		//     }
+		// }
+	}
 }

core/domain/services/registration/CancelRegistrationService.php

Indentation +45 added lines, -45 removed lines

@@ -19,55 +19,55 @@
  */
 class CancelRegistrationService
 {
-    /**
-     * @var CancelTicketLineItemService $cancel_ticket_line_item_service
-     */
-    private $cancel_ticket_line_item_service;
+	/**
+	 * @var CancelTicketLineItemService $cancel_ticket_line_item_service
+	 */
+	private $cancel_ticket_line_item_service;
 
 
-    /**
-     * Command constructor
-     *
-     * @param CancelTicketLineItemService $cancel_ticket_line_item_service
-     */
-    public function __construct(CancelTicketLineItemService $cancel_ticket_line_item_service)
-    {
-        $this->cancel_ticket_line_item_service = $cancel_ticket_line_item_service;
-    }
+	/**
+	 * Command constructor
+	 *
+	 * @param CancelTicketLineItemService $cancel_ticket_line_item_service
+	 */
+	public function __construct(CancelTicketLineItemService $cancel_ticket_line_item_service)
+	{
+		$this->cancel_ticket_line_item_service = $cancel_ticket_line_item_service;
+	}
 
 
-    /**
-     * @param EE_Registration $registration
-     * @param bool            $cancel_ticket_line_item
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function cancelRegistrationAndTicketLineItem(EE_Registration $registration, $cancel_ticket_line_item = true)
-    {
-        // first cancel the original line item for the registration's ticket
-        if ($cancel_ticket_line_item) {
-            $this->cancel_ticket_line_item_service->forRegistration($registration);
-        }
-        $this->cancelRegistrationOnly($registration);
-    }
+	/**
+	 * @param EE_Registration $registration
+	 * @param bool            $cancel_ticket_line_item
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function cancelRegistrationAndTicketLineItem(EE_Registration $registration, $cancel_ticket_line_item = true)
+	{
+		// first cancel the original line item for the registration's ticket
+		if ($cancel_ticket_line_item) {
+			$this->cancel_ticket_line_item_service->forRegistration($registration);
+		}
+		$this->cancelRegistrationOnly($registration);
+	}
 
 
-    /**
-     * @param EE_Registration $registration
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function cancelRegistrationOnly(EE_Registration $registration)
-    {
-        // now cancel the registration itself
-        $registration->set_status(
-            RegStatus::CANCELLED,
-            false,
-            new Context(
-                __METHOD__,
-                esc_html__('Executed when a registration is cancelled.', 'event_espresso')
-            )
-        );
-        $registration->save();
-    }
+	/**
+	 * @param EE_Registration $registration
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function cancelRegistrationOnly(EE_Registration $registration)
+	{
+		// now cancel the registration itself
+		$registration->set_status(
+			RegStatus::CANCELLED,
+			false,
+			new Context(
+				__METHOD__,
+				esc_html__('Executed when a registration is cancelled.', 'event_espresso')
+			)
+		);
+		$registration->save();
+	}
 }

core/domain/services/registration/form/v1/RegForm.php

Indentation +202 added lines, -202 removed lines

@@ -30,233 +30,233 @@
  */
 class RegForm extends EE_Form_Section_Proper
 {
-    public EE_Registration_Config $reg_config;
+	public EE_Registration_Config $reg_config;
 
-    public EE_SPCO_Reg_Step_Attendee_Information $reg_step;
+	public EE_SPCO_Reg_Step_Attendee_Information $reg_step;
 
-    private bool $print_copy_info = false;
+	private bool $print_copy_info = false;
 
-    protected int $reg_form_count = 0;
+	protected int $reg_form_count = 0;
 
-    private array $required_questions = [];
+	private array $required_questions = [];
 
-    private array $template_args = [];
+	private array $template_args = [];
 
 
-    /**
-     * RegForm constructor.
-     *
-     * @param EE_SPCO_Reg_Step_Attendee_Information $reg_step
-     * @param EE_Registration_Config                $reg_config
-     * @throws ReflectionException
-     * @throws EE_Error
-     */
-    public function __construct(
-        EE_SPCO_Reg_Step_Attendee_Information $reg_step,
-        EE_Registration_Config $reg_config
-    ) {
-        $this->reg_step   = $reg_step;
-        $this->reg_config = $reg_config;
-        // setup some classes so that they are ready for loading during construction of other classes
-        LoaderFactory::getShared(CountryOptions::class, [$this->reg_step->checkout->action]);
-        LoaderFactory::getShared(StateOptions::class, [$this->reg_step->checkout->action]);
-        LoaderFactory::getShared(RegFormQuestionFactory::class, [[$this, 'addRequiredQuestion']]);
-        parent::__construct(
-            [
-                'name'            => $this->reg_step->reg_form_name(),
-                'html_id'         => $this->reg_step->reg_form_name(),
-                'subsections'     => $this->generateSubsections(),
-                'layout_strategy' => new EE_Template_Layout(
-                    [
-                        'layout_template_file' => $this->reg_step->template(), // layout_template
-                        'template_args'        => $this->template_args,
-                    ]
-                ),
-            ]
-        );
-    }
+	/**
+	 * RegForm constructor.
+	 *
+	 * @param EE_SPCO_Reg_Step_Attendee_Information $reg_step
+	 * @param EE_Registration_Config                $reg_config
+	 * @throws ReflectionException
+	 * @throws EE_Error
+	 */
+	public function __construct(
+		EE_SPCO_Reg_Step_Attendee_Information $reg_step,
+		EE_Registration_Config $reg_config
+	) {
+		$this->reg_step   = $reg_step;
+		$this->reg_config = $reg_config;
+		// setup some classes so that they are ready for loading during construction of other classes
+		LoaderFactory::getShared(CountryOptions::class, [$this->reg_step->checkout->action]);
+		LoaderFactory::getShared(StateOptions::class, [$this->reg_step->checkout->action]);
+		LoaderFactory::getShared(RegFormQuestionFactory::class, [[$this, 'addRequiredQuestion']]);
+		parent::__construct(
+			[
+				'name'            => $this->reg_step->reg_form_name(),
+				'html_id'         => $this->reg_step->reg_form_name(),
+				'subsections'     => $this->generateSubsections(),
+				'layout_strategy' => new EE_Template_Layout(
+					[
+						'layout_template_file' => $this->reg_step->template(), // layout_template
+						'template_args'        => $this->template_args,
+					]
+				),
+			]
+		);
+	}
 
 
-    /**
-     * @return void
-     */
-    public function enablePrintCopyInfo(): void
-    {
-        $this->print_copy_info = true;
-    }
+	/**
+	 * @return void
+	 */
+	public function enablePrintCopyInfo(): void
+	{
+		$this->print_copy_info = true;
+	}
 
 
-    /**
-     * @return bool
-     */
-    public function printCopyInfo(): bool
-    {
-        return $this->print_copy_info;
-    }
+	/**
+	 * @return bool
+	 */
+	public function printCopyInfo(): bool
+	{
+		return $this->print_copy_info;
+	}
 
 
-    /**
-     * @return int
-     */
-    public function regFormCount(): int
-    {
-        return $this->reg_form_count;
-    }
+	/**
+	 * @return int
+	 */
+	public function regFormCount(): int
+	{
+		return $this->reg_form_count;
+	}
 
 
-    /**
-     * @return array
-     */
-    public function requiredQuestions(): array
-    {
-        return $this->required_questions;
-    }
+	/**
+	 * @return array
+	 */
+	public function requiredQuestions(): array
+	{
+		return $this->required_questions;
+	}
 
 
-    /**
-     * @param string $identifier
-     * @param string $required_question
-     */
-    public function addRequiredQuestion(string $identifier, string $required_question): void
-    {
-        $this->required_questions[ $identifier ] = $required_question;
-    }
+	/**
+	 * @param string $identifier
+	 * @param string $required_question
+	 */
+	public function addRequiredQuestion(string $identifier, string $required_question): void
+	{
+		$this->required_questions[ $identifier ] = $required_question;
+	}
 
 
-    /**
-     * @return EE_Form_Section_Proper[]
-     * @throws DomainException
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws ReflectionException
-     * @throws EntityNotFoundException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    private function generateSubsections(): array
-    {
-        // Init reg forms count.
-        $this->reg_form_count = 0;
+	/**
+	 * @return EE_Form_Section_Proper[]
+	 * @throws DomainException
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws ReflectionException
+	 * @throws EntityNotFoundException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	private function generateSubsections(): array
+	{
+		// Init reg forms count.
+		$this->reg_form_count = 0;
 
-        $primary_registrant = null;
-        // autoload Line_Item_Display classes
-        EEH_Autoloader::register_line_item_display_autoloaders();
-        $Line_Item_Display = new EE_Line_Item_Display();
-        // calculate taxes
-        $Line_Item_Display->display_line_item(
-            $this->reg_step->checkout->cart->get_grand_total(),
-            ['set_tax_rate' => true]
-        );
-        $extra_inputs_section = $this->reg_step->reg_step_hidden_inputs();
-        $this->addPrivacyConsentCheckbox($extra_inputs_section);
-        $subsections = [
-            'default_hidden_inputs' => $extra_inputs_section,
-        ];
+		$primary_registrant = null;
+		// autoload Line_Item_Display classes
+		EEH_Autoloader::register_line_item_display_autoloaders();
+		$Line_Item_Display = new EE_Line_Item_Display();
+		// calculate taxes
+		$Line_Item_Display->display_line_item(
+			$this->reg_step->checkout->cart->get_grand_total(),
+			['set_tax_rate' => true]
+		);
+		$extra_inputs_section = $this->reg_step->reg_step_hidden_inputs();
+		$this->addPrivacyConsentCheckbox($extra_inputs_section);
+		$subsections = [
+			'default_hidden_inputs' => $extra_inputs_section,
+		];
 
-        $this->template_args = [
-            'revisit'       => $this->reg_step->checkout->revisit,
-            'registrations' => [],
-            'ticket_count'  => [],
-        ];
-        // grab the saved registrations from the transaction
-        $registrations = $this->reg_step->checkout->transaction->registrations(
-            $this->reg_step->checkout->reg_cache_where_params
-        );
-        if ($registrations) {
-            foreach ($registrations as $registration) {
-                // can this registration be processed during this visit ?
-                if (
-                    $registration instanceof EE_Registration
-                    && $this->reg_step->checkout->visit_allows_processing_of_this_registration($registration)
-                ) {
-                    $reg_url_link = $registration->reg_url_link();
-                    /** @var RegistrantForm $registrant_form */
-                    $registrant_form = LoaderFactory::getNew(
-                        RegistrantForm::class,
-                        [
-                            $registration,
-                            $this->reg_config->copyAttendeeInfo(),
-                            [$this, 'enablePrintCopyInfo'],
-                            $this->reg_step,
-                        ]
-                    );
-                    // Increment the reg forms number if form is valid.
-                    if ($registrant_form->hasQuestions()) {
-                        $this->reg_form_count++;
-                        $subsections[ $reg_url_link ] = $registrant_form;
-                    } else {
-                        // or just add a blank section if there are no questions
-                        $subsections[ $reg_url_link ] = new EE_Form_Section_HTML();
-                    }
+		$this->template_args = [
+			'revisit'       => $this->reg_step->checkout->revisit,
+			'registrations' => [],
+			'ticket_count'  => [],
+		];
+		// grab the saved registrations from the transaction
+		$registrations = $this->reg_step->checkout->transaction->registrations(
+			$this->reg_step->checkout->reg_cache_where_params
+		);
+		if ($registrations) {
+			foreach ($registrations as $registration) {
+				// can this registration be processed during this visit ?
+				if (
+					$registration instanceof EE_Registration
+					&& $this->reg_step->checkout->visit_allows_processing_of_this_registration($registration)
+				) {
+					$reg_url_link = $registration->reg_url_link();
+					/** @var RegistrantForm $registrant_form */
+					$registrant_form = LoaderFactory::getNew(
+						RegistrantForm::class,
+						[
+							$registration,
+							$this->reg_config->copyAttendeeInfo(),
+							[$this, 'enablePrintCopyInfo'],
+							$this->reg_step,
+						]
+					);
+					// Increment the reg forms number if form is valid.
+					if ($registrant_form->hasQuestions()) {
+						$this->reg_form_count++;
+						$subsections[ $reg_url_link ] = $registrant_form;
+					} else {
+						// or just add a blank section if there are no questions
+						$subsections[ $reg_url_link ] = new EE_Form_Section_HTML();
+					}
 
-                    $this->template_args['registrations'][ $reg_url_link ]                = $registration;
-                    $this->template_args['ticket_count'][ $registration->ticket()->ID() ] = isset(
-                        $this->template_args['ticket_count'][ $registration->ticket()->ID() ]
-                    )
-                        ? $this->template_args['ticket_count'][ $registration->ticket()->ID() ] + 1
-                        : 1;
-                    $ticket_line_item = EEH_Line_Item::get_line_items_by_object_type_and_IDs(
-                        $this->reg_step->checkout->cart->get_grand_total(),
-                        'Ticket',
-                        [$registration->ticket()->ID()]
-                    );
-                    $ticket_line_item = reset($ticket_line_item);
-                    $this->template_args['ticket_line_item'][ $registration->ticket()->ID() ] =
-                        $ticket_line_item instanceof EE_Line_Item
-                            ? $Line_Item_Display->display_line_item($ticket_line_item)
-                            : '';
-                    if ($registration->is_primary_registrant()) {
-                        $primary_registrant = $reg_url_link;
-                    }
-                }
-            }
+					$this->template_args['registrations'][ $reg_url_link ]                = $registration;
+					$this->template_args['ticket_count'][ $registration->ticket()->ID() ] = isset(
+						$this->template_args['ticket_count'][ $registration->ticket()->ID() ]
+					)
+						? $this->template_args['ticket_count'][ $registration->ticket()->ID() ] + 1
+						: 1;
+					$ticket_line_item = EEH_Line_Item::get_line_items_by_object_type_and_IDs(
+						$this->reg_step->checkout->cart->get_grand_total(),
+						'Ticket',
+						[$registration->ticket()->ID()]
+					);
+					$ticket_line_item = reset($ticket_line_item);
+					$this->template_args['ticket_line_item'][ $registration->ticket()->ID() ] =
+						$ticket_line_item instanceof EE_Line_Item
+							? $Line_Item_Display->display_line_item($ticket_line_item)
+							: '';
+					if ($registration->is_primary_registrant()) {
+						$primary_registrant = $reg_url_link;
+					}
+				}
+			}
 
-            if ($primary_registrant && count($registrations) > 1) {
-                if (
-                    isset($subsections[ $primary_registrant ])
-                    && $subsections[ $primary_registrant ] instanceof EE_Form_Section_Proper
-                ) {
-                    $copy_options['spco_copy_attendee_chk'] = $this->print_copy_info
-                        ? new CopyAttendeeInfoForm($registrations, $this->reg_step->slug())
-                        : new AutoCopyAttendeeInfoForm($this->reg_step->slug());
-                    $subsections[ $primary_registrant ]->add_subsections(
-                        $copy_options,
-                        'primary_registrant',
-                        false
-                    );
-                }
-            }
-        }
+			if ($primary_registrant && count($registrations) > 1) {
+				if (
+					isset($subsections[ $primary_registrant ])
+					&& $subsections[ $primary_registrant ] instanceof EE_Form_Section_Proper
+				) {
+					$copy_options['spco_copy_attendee_chk'] = $this->print_copy_info
+						? new CopyAttendeeInfoForm($registrations, $this->reg_step->slug())
+						: new AutoCopyAttendeeInfoForm($this->reg_step->slug());
+					$subsections[ $primary_registrant ]->add_subsections(
+						$copy_options,
+						'primary_registrant',
+						false
+					);
+				}
+			}
+		}
 
-        // Set the registration form template (default: one form per ticket details table).
-        // We decide the template to used based on the number of forms.
-        $template = $this->reg_form_count > 1
-            ? SPCO_REG_STEPS_PATH . $this->reg_step->slug() . '/attendee_info_main.template.php'
-            : SPCO_REG_STEPS_PATH . $this->reg_step->slug() . '/attendee_info_single.template.php';
-        $this->reg_step->setTemplate($template);
+		// Set the registration form template (default: one form per ticket details table).
+		// We decide the template to used based on the number of forms.
+		$template = $this->reg_form_count > 1
+			? SPCO_REG_STEPS_PATH . $this->reg_step->slug() . '/attendee_info_main.template.php'
+			: SPCO_REG_STEPS_PATH . $this->reg_step->slug() . '/attendee_info_single.template.php';
+		$this->reg_step->setTemplate($template);
 
-        return $subsections;
-    }
+		return $subsections;
+	}
 
 
-    /**
-     * @param EE_Form_Section_Proper $extra_inputs_section
-     * @throws EE_Error
-     */
-    private function addPrivacyConsentCheckbox(EE_Form_Section_Proper $extra_inputs_section)
-    {
-        // if this isn't a revisit, and they have the privacy consent box enabled, add it
-        if (! $this->reg_step->checkout->revisit && $this->reg_config->isConsentCheckboxEnabled()) {
-            $extra_inputs_section->add_subsections(
-                [
-                    'consent_box' => new PrivacyConsentCheckboxForm(
-                        $this->reg_step->slug(),
-                        $this->reg_config->getConsentCheckboxLabelText()
-                    )
-                ],
-                null,
-                false
-            );
-        }
-    }
+	/**
+	 * @param EE_Form_Section_Proper $extra_inputs_section
+	 * @throws EE_Error
+	 */
+	private function addPrivacyConsentCheckbox(EE_Form_Section_Proper $extra_inputs_section)
+	{
+		// if this isn't a revisit, and they have the privacy consent box enabled, add it
+		if (! $this->reg_step->checkout->revisit && $this->reg_config->isConsentCheckboxEnabled()) {
+			$extra_inputs_section->add_subsections(
+				[
+					'consent_box' => new PrivacyConsentCheckboxForm(
+						$this->reg_step->slug(),
+						$this->reg_config->getConsentCheckboxLabelText()
+					)
+				],
+				null,
+				false
+			);
+		}
+	}
 }

Spacing +14 added lines, -14 removed lines

@@ -119,7 +119,7 @@ 
      */
     public function addRequiredQuestion(string $identifier, string $required_question): void
     {
-        $this->required_questions[ $identifier ] = $required_question;
+        $this->required_questions[$identifier] = $required_question;
     }
 
 
@@ -183,17 +183,17 @@ 
                     // Increment the reg forms number if form is valid.
                     if ($registrant_form->hasQuestions()) {
                         $this->reg_form_count++;
-                        $subsections[ $reg_url_link ] = $registrant_form;
+                        $subsections[$reg_url_link] = $registrant_form;
                     } else {
                         // or just add a blank section if there are no questions
-                        $subsections[ $reg_url_link ] = new EE_Form_Section_HTML();
+                        $subsections[$reg_url_link] = new EE_Form_Section_HTML();
                     }
 
-                    $this->template_args['registrations'][ $reg_url_link ]                = $registration;
-                    $this->template_args['ticket_count'][ $registration->ticket()->ID() ] = isset(
-                        $this->template_args['ticket_count'][ $registration->ticket()->ID() ]
+                    $this->template_args['registrations'][$reg_url_link]                = $registration;
+                    $this->template_args['ticket_count'][$registration->ticket()->ID()] = isset(
+                        $this->template_args['ticket_count'][$registration->ticket()->ID()]
                     )
-                        ? $this->template_args['ticket_count'][ $registration->ticket()->ID() ] + 1
+                        ? $this->template_args['ticket_count'][$registration->ticket()->ID()] + 1
                         : 1;
                     $ticket_line_item = EEH_Line_Item::get_line_items_by_object_type_and_IDs(
                         $this->reg_step->checkout->cart->get_grand_total(),
@@ -201,7 +201,7 @@ 
                         [$registration->ticket()->ID()]
                     );
                     $ticket_line_item = reset($ticket_line_item);
-                    $this->template_args['ticket_line_item'][ $registration->ticket()->ID() ] =
+                    $this->template_args['ticket_line_item'][$registration->ticket()->ID()] =
                         $ticket_line_item instanceof EE_Line_Item
                             ? $Line_Item_Display->display_line_item($ticket_line_item)
                             : '';
@@ -213,13 +213,13 @@ 
 
             if ($primary_registrant && count($registrations) > 1) {
                 if (
-                    isset($subsections[ $primary_registrant ])
-                    && $subsections[ $primary_registrant ] instanceof EE_Form_Section_Proper
+                    isset($subsections[$primary_registrant])
+                    && $subsections[$primary_registrant] instanceof EE_Form_Section_Proper
                 ) {
                     $copy_options['spco_copy_attendee_chk'] = $this->print_copy_info
                         ? new CopyAttendeeInfoForm($registrations, $this->reg_step->slug())
                         : new AutoCopyAttendeeInfoForm($this->reg_step->slug());
-                    $subsections[ $primary_registrant ]->add_subsections(
+                    $subsections[$primary_registrant]->add_subsections(
                         $copy_options,
                         'primary_registrant',
                         false
@@ -231,8 +231,8 @@ 
         // Set the registration form template (default: one form per ticket details table).
         // We decide the template to used based on the number of forms.
         $template = $this->reg_form_count > 1
-            ? SPCO_REG_STEPS_PATH . $this->reg_step->slug() . '/attendee_info_main.template.php'
-            : SPCO_REG_STEPS_PATH . $this->reg_step->slug() . '/attendee_info_single.template.php';
+            ? SPCO_REG_STEPS_PATH . $this->reg_step->slug().'/attendee_info_main.template.php'
+            : SPCO_REG_STEPS_PATH.$this->reg_step->slug().'/attendee_info_single.template.php';
         $this->reg_step->setTemplate($template);
 
         return $subsections;
@@ -246,7 +246,7 @@ 
     private function addPrivacyConsentCheckbox(EE_Form_Section_Proper $extra_inputs_section)
     {
         // if this isn't a revisit, and they have the privacy consent box enabled, add it
-        if (! $this->reg_step->checkout->revisit && $this->reg_config->isConsentCheckboxEnabled()) {
+        if ( ! $this->reg_step->checkout->revisit && $this->reg_config->isConsentCheckboxEnabled()) {
             $extra_inputs_section->add_subsections(
                 [
                     'consent_box' => new PrivacyConsentCheckboxForm(

core/domain/services/registration/CreateRegistrationService.php

Indentation +96 added lines, -96 removed lines

@@ -27,104 +27,104 @@
  */
 class CreateRegistrationService extends DomainService
 {
-    /**
-     * @param EE_Event       $event
-     * @param EE_Transaction $transaction
-     * @param EE_Ticket      $ticket
-     * @param EE_Line_Item   $ticket_line_item
-     * @param int            $reg_count
-     * @param int            $reg_group_size
-     * @param string         $reg_status
-     * @return EE_Registration
-     * @throws OutOfRangeException
-     * @throws EE_Error
-     * @throws UnexpectedEntityException
-     * @throws ReflectionException
-     */
-    public function create(
-        EE_Event $event,
-        EE_Transaction $transaction,
-        EE_Ticket $ticket,
-        EE_Line_Item $ticket_line_item,
-        int $reg_count,
-        int $reg_group_size,
-        string $reg_status = RegStatus::INCOMPLETE
-    ): EE_Registration {
-        $registrations = $transaction->registrations();
-        $reg_count = $reg_count ?: count($registrations) + 1;
-        $reg_url_link = new RegUrlLink($reg_count, $ticket_line_item);
-        $reg_code = new RegCode($reg_url_link, $transaction, $ticket);
-        // generate new EE_Registration
-        $registration = EE_Registration::new_instance(
-            array(
-                'EVT_ID'          => $event->ID(),
-                'TXN_ID'          => $transaction->ID(),
-                'TKT_ID'          => $ticket->ID(),
-                'STS_ID'          => $reg_status,
-                'REG_final_price' => $this->resolveFinalPrice($transaction, $ticket, $ticket_line_item),
-                'REG_session'     => EE_Registry::instance()->SSN->id(),
-                'REG_count'       => $reg_count,
-                'REG_group_size'  => $reg_group_size ?: $this->incrementRegGroupSize($registrations),
-                'REG_url_link'    => $reg_url_link,
-                'REG_code'        => $reg_code,
-            )
-        );
-        if (! $registration instanceof EE_Registration) {
-            throw new UnexpectedEntityException($registration, 'EE_Registration');
-        }
-        // save registration so that we have an ID
-        $registration->save();
-        // track reservation on reg but don't adjust ticket and datetime reserved counts
-        // because that is done as soon as the tickets are added/removed from the cart
-        $registration->reserve_ticket(false, 'CreateRegistrationService:' . __LINE__);
-        $registration->_add_relation_to($event, 'Event', array(), $event->ID());
-        $registration->_add_relation_to($ticket, 'Ticket', array(), $ticket->ID());
-        $transaction->_add_relation_to($registration, 'Registration', array(), $registration->ID());
-        $registration->save();
-        return $registration;
-    }
+	/**
+	 * @param EE_Event       $event
+	 * @param EE_Transaction $transaction
+	 * @param EE_Ticket      $ticket
+	 * @param EE_Line_Item   $ticket_line_item
+	 * @param int            $reg_count
+	 * @param int            $reg_group_size
+	 * @param string         $reg_status
+	 * @return EE_Registration
+	 * @throws OutOfRangeException
+	 * @throws EE_Error
+	 * @throws UnexpectedEntityException
+	 * @throws ReflectionException
+	 */
+	public function create(
+		EE_Event $event,
+		EE_Transaction $transaction,
+		EE_Ticket $ticket,
+		EE_Line_Item $ticket_line_item,
+		int $reg_count,
+		int $reg_group_size,
+		string $reg_status = RegStatus::INCOMPLETE
+	): EE_Registration {
+		$registrations = $transaction->registrations();
+		$reg_count = $reg_count ?: count($registrations) + 1;
+		$reg_url_link = new RegUrlLink($reg_count, $ticket_line_item);
+		$reg_code = new RegCode($reg_url_link, $transaction, $ticket);
+		// generate new EE_Registration
+		$registration = EE_Registration::new_instance(
+			array(
+				'EVT_ID'          => $event->ID(),
+				'TXN_ID'          => $transaction->ID(),
+				'TKT_ID'          => $ticket->ID(),
+				'STS_ID'          => $reg_status,
+				'REG_final_price' => $this->resolveFinalPrice($transaction, $ticket, $ticket_line_item),
+				'REG_session'     => EE_Registry::instance()->SSN->id(),
+				'REG_count'       => $reg_count,
+				'REG_group_size'  => $reg_group_size ?: $this->incrementRegGroupSize($registrations),
+				'REG_url_link'    => $reg_url_link,
+				'REG_code'        => $reg_code,
+			)
+		);
+		if (! $registration instanceof EE_Registration) {
+			throw new UnexpectedEntityException($registration, 'EE_Registration');
+		}
+		// save registration so that we have an ID
+		$registration->save();
+		// track reservation on reg but don't adjust ticket and datetime reserved counts
+		// because that is done as soon as the tickets are added/removed from the cart
+		$registration->reserve_ticket(false, 'CreateRegistrationService:' . __LINE__);
+		$registration->_add_relation_to($event, 'Event', array(), $event->ID());
+		$registration->_add_relation_to($ticket, 'Ticket', array(), $ticket->ID());
+		$transaction->_add_relation_to($registration, 'Registration', array(), $registration->ID());
+		$registration->save();
+		return $registration;
+	}
 
 
-    /**
-     * @param EE_Transaction $transaction
-     * @param EE_Ticket      $ticket
-     * @param EE_Line_Item   $ticket_line_item
-     * @return float
-     * @throws EE_Error
-     * @throws OutOfRangeException
-     * @throws ReflectionException
-     */
-    protected function resolveFinalPrice(
-        EE_Transaction $transaction,
-        EE_Ticket $ticket,
-        EE_Line_Item $ticket_line_item
-    ): float {
-        $final_price = EEH_Line_Item::calculate_final_price_for_ticket_line_item(
-            $transaction->total_line_item(),
-            $ticket_line_item
-        );
-        return $final_price !== null ? $final_price : $ticket->get_ticket_total_with_taxes();
-    }
+	/**
+	 * @param EE_Transaction $transaction
+	 * @param EE_Ticket      $ticket
+	 * @param EE_Line_Item   $ticket_line_item
+	 * @return float
+	 * @throws EE_Error
+	 * @throws OutOfRangeException
+	 * @throws ReflectionException
+	 */
+	protected function resolveFinalPrice(
+		EE_Transaction $transaction,
+		EE_Ticket $ticket,
+		EE_Line_Item $ticket_line_item
+	): float {
+		$final_price = EEH_Line_Item::calculate_final_price_for_ticket_line_item(
+			$transaction->total_line_item(),
+			$ticket_line_item
+		);
+		return $final_price !== null ? $final_price : $ticket->get_ticket_total_with_taxes();
+	}
 
 
-    /**
-     * @param EE_Registration[] $registrations
-     * @param boolean           $update_existing_registrations
-     * @return int
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    protected function incrementRegGroupSize(array $registrations, bool $update_existing_registrations = true): int
-    {
-        $new_reg_group_size = count($registrations) + 1;
-        if ($update_existing_registrations) {
-            foreach ($registrations as $registration) {
-                if ($registration instanceof EE_Registration) {
-                    $registration->set_group_size($new_reg_group_size);
-                    $registration->save();
-                }
-            }
-        }
-        return $new_reg_group_size;
-    }
+	/**
+	 * @param EE_Registration[] $registrations
+	 * @param boolean           $update_existing_registrations
+	 * @return int
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	protected function incrementRegGroupSize(array $registrations, bool $update_existing_registrations = true): int
+	{
+		$new_reg_group_size = count($registrations) + 1;
+		if ($update_existing_registrations) {
+			foreach ($registrations as $registration) {
+				if ($registration instanceof EE_Registration) {
+					$registration->set_group_size($new_reg_group_size);
+					$registration->save();
+				}
+			}
+		}
+		return $new_reg_group_size;
+	}
 }

core/domain/services/registration/RegStatus.php

Indentation +88 added lines, -88 removed lines

@@ -11,102 +11,102 @@
  */
 class RegStatus
 {
-    /**
-     * Status ID (STS_ID on esp_status table) to indicate an APPROVED registration.
-     * the TXN may or may not be completed ( paid in full )
-     * Payments are allowed.
-     * A space IS reserved.
-     * Registration is active
-     */
-    public const APPROVED = 'RAP';
+	/**
+	 * Status ID (STS_ID on esp_status table) to indicate an APPROVED registration.
+	 * the TXN may or may not be completed ( paid in full )
+	 * Payments are allowed.
+	 * A space IS reserved.
+	 * Registration is active
+	 */
+	public const APPROVED = 'RAP';
 
-    /**
-     * Status ID (STS_ID on esp_status table) to indicate an UNAPPROVED registration.
-     * Payments are NOT allowed.
-     * Event Admin must manually toggle STS_ID for it to change
-     * No space reserved.
-     * Registration is active
-     */
-    public const AWAITING_REVIEW = 'RNA';
+	/**
+	 * Status ID (STS_ID on esp_status table) to indicate an UNAPPROVED registration.
+	 * Payments are NOT allowed.
+	 * Event Admin must manually toggle STS_ID for it to change
+	 * No space reserved.
+	 * Registration is active
+	 */
+	public const AWAITING_REVIEW = 'RNA';
 
-    /**
-     * Status ID (STS_ID on esp_status table) to indicate a registration was CANCELLED by the attendee.
-     * Payments are NOT allowed.
-     * NO space reserved.
-     * Registration is NOT active
-     */
-    public const CANCELLED = 'RCN';
+	/**
+	 * Status ID (STS_ID on esp_status table) to indicate a registration was CANCELLED by the attendee.
+	 * Payments are NOT allowed.
+	 * NO space reserved.
+	 * Registration is NOT active
+	 */
+	public const CANCELLED = 'RCN';
 
-    /**
-     * Status ID (STS_ID on esp_status table) to indicate a registration was DECLINED by the Event Admin
-     * Payments are NOT allowed.
-     * No space reserved.
-     * Registration is NOT active
-     */
-    public const DECLINED = 'RDC';
+	/**
+	 * Status ID (STS_ID on esp_status table) to indicate a registration was DECLINED by the Event Admin
+	 * Payments are NOT allowed.
+	 * No space reserved.
+	 * Registration is NOT active
+	 */
+	public const DECLINED = 'RDC';
 
-    /**
-     * Status ID (STS_ID on esp_status table) to indicate an INCOMPLETE registration.
-     * Initial status for registrations when they are first created
-     * Payments are NOT allowed.
-     * Automatically toggled to whatever the default Event registration status is upon completion of the attendee
-     * information reg step NO space reserved. Registration is NOT active
-     */
-    public const INCOMPLETE = 'RIC';
+	/**
+	 * Status ID (STS_ID on esp_status table) to indicate an INCOMPLETE registration.
+	 * Initial status for registrations when they are first created
+	 * Payments are NOT allowed.
+	 * Automatically toggled to whatever the default Event registration status is upon completion of the attendee
+	 * information reg step NO space reserved. Registration is NOT active
+	 */
+	public const INCOMPLETE = 'RIC';
 
-    /**
-     * Status ID (STS_ID on esp_status table) to indicate registration is PENDING_PAYMENT .
-     * Payments are allowed.
-     * STS_ID will automatically be toggled to RAP if payment is made in full by the attendee
-     * No space reserved.
-     * Registration is active
-     */
-    public const PENDING_PAYMENT = 'RPP';
+	/**
+	 * Status ID (STS_ID on esp_status table) to indicate registration is PENDING_PAYMENT .
+	 * Payments are allowed.
+	 * STS_ID will automatically be toggled to RAP if payment is made in full by the attendee
+	 * No space reserved.
+	 * Registration is active
+	 */
+	public const PENDING_PAYMENT = 'RPP';
 
-    /**
-     * Status ID (STS_ID on esp_status table) to indicate registration is on the WAIT_LIST .
-     * Payments are allowed.
-     * STS_ID will automatically be toggled to RAP if payment is made in full by the attendee
-     * No space reserved.
-     * Registration is active
-     */
-    public const WAIT_LIST = 'RWL';
+	/**
+	 * Status ID (STS_ID on esp_status table) to indicate registration is on the WAIT_LIST .
+	 * Payments are allowed.
+	 * STS_ID will automatically be toggled to RAP if payment is made in full by the attendee
+	 * No space reserved.
+	 * Registration is active
+	 */
+	public const WAIT_LIST = 'RWL';
 
 
-    /**
-     * returns an array of ALL Reg statuses for use in comparisons
-     *
-     * @return string[]
-     */
-    public static function validRegStatuses(): array
-    {
-        return [
-            RegStatus::APPROVED,
-            RegStatus::AWAITING_REVIEW,
-            RegStatus::CANCELLED,
-            RegStatus::DECLINED,
-            RegStatus::INCOMPLETE,
-            RegStatus::PENDING_PAYMENT,
-            RegStatus::WAIT_LIST,
-        ];
-    }
+	/**
+	 * returns an array of ALL Reg statuses for use in comparisons
+	 *
+	 * @return string[]
+	 */
+	public static function validRegStatuses(): array
+	{
+		return [
+			RegStatus::APPROVED,
+			RegStatus::AWAITING_REVIEW,
+			RegStatus::CANCELLED,
+			RegStatus::DECLINED,
+			RegStatus::INCOMPLETE,
+			RegStatus::PENDING_PAYMENT,
+			RegStatus::WAIT_LIST,
+		];
+	}
 
 
-    public static function isValidStatus(string $status, bool $throw_exception = true): bool
-    {
-        $valid = in_array($status, RegStatus::validRegStatuses(), true);
-        if (! $valid && $throw_exception) {
-            throw new InvalidArgumentException(
-                sprintf(
-                    esc_html__(
-                        'Invalid registration status "%1$s" provided. Valid statuses are: %2$s',
-                        'event_espresso'
-                    ),
-                    $status,
-                    implode(', ', RegStatus::validRegStatuses())
-                )
-            );
-        }
-        return $valid;
-    }
+	public static function isValidStatus(string $status, bool $throw_exception = true): bool
+	{
+		$valid = in_array($status, RegStatus::validRegStatuses(), true);
+		if (! $valid && $throw_exception) {
+			throw new InvalidArgumentException(
+				sprintf(
+					esc_html__(
+						'Invalid registration status "%1$s" provided. Valid statuses are: %2$s',
+						'event_espresso'
+					),
+					$status,
+					implode(', ', RegStatus::validRegStatuses())
+				)
+			);
+		}
+		return $valid;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
     public static function isValidStatus(string $status, bool $throw_exception = true): bool
     {
         $valid = in_array($status, RegStatus::validRegStatuses(), true);
-        if (! $valid && $throw_exception) {
+        if ( ! $valid && $throw_exception) {
             throw new InvalidArgumentException(
                 sprintf(
                     esc_html__(

core/domain/services/commands/registration/CreateRegistrationCommand.php

Indentation +154 added lines, -154 removed lines

@@ -26,158 +26,158 @@
  */
 class CreateRegistrationCommand extends Command implements CommandRequiresCapCheckInterface
 {
-    /**
-     * @var EE_Transaction $transaction
-     */
-    private $transaction;
-
-    /**
-     * @var EE_Ticket $ticket
-     */
-    private $ticket;
-
-    /**
-     * @var EE_Line_Item $ticket_line_item
-     */
-    private $ticket_line_item;
-
-    /**
-     * @var int $reg_count
-     */
-    private $reg_count;
-
-    /**
-     * @var int $reg_group_size
-     */
-    private $reg_group_size;
-
-    /**
-     * @var string $reg_status
-     */
-    private $reg_status;
-
-    /**
-     * @var EE_Registration $registration
-     */
-    protected $registration;
-
-
-    /**
-     * CreateRegistrationCommand constructor.
-     *
-     * @param EE_Transaction $transaction
-     * @param EE_Line_Item   $ticket_line_item
-     * @param int            $reg_count
-     * @param int            $reg_group_size
-     * @param string         $reg_status
-     * @param EE_Ticket|null $ticket
-     * @throws InvalidEntityException
-     * @throws EE_Error
-     */
-    public function __construct(
-        EE_Transaction $transaction,
-        EE_Line_Item $ticket_line_item,
-        $reg_count = 1,
-        $reg_group_size = 0,
-        $reg_status = RegStatus::INCOMPLETE,
-        EE_Ticket $ticket = null
-    ) {
-        defined('EVENT_ESPRESSO_VERSION') || exit;
-        $this->transaction      = $transaction;
-        $this->ticket_line_item = $ticket_line_item;
-        $this->reg_count        = absint($reg_count);
-        $this->reg_group_size   = absint($reg_group_size);
-        $this->reg_status       = $reg_status;
-        // grab the related ticket object for this line_item if one wasn't already supplied
-        $this->ticket = $ticket instanceof EE_Ticket ? $ticket : $this->ticket_line_item->ticket();
-        if (! $this->ticket instanceof EE_Ticket) {
-            throw new InvalidEntityException(
-                is_object($this->ticket) ? get_class($this->ticket) : gettype($this->ticket),
-                'EE_Ticket',
-                sprintf(
-                    esc_html__('Line item %s did not contain a valid ticket', 'event_espresso'),
-                    $ticket_line_item->ID()
-                )
-            );
-        }
-    }
-
-
-    /**
-     * @return CapCheckInterface
-     * @throws InvalidDataTypeException
-     */
-    public function getCapCheck()
-    {
-        if (! $this->cap_check instanceof CapCheckInterface) {
-            // need cap for non-AJAX admin requests
-            $this->cap_check = ! (defined('DOING_AJAX') && DOING_AJAX) && is_admin()
-                ? new CapCheck('ee_edit_registrations', 'create_new_registration')
-                : new PublicCapabilities('', 'create_new_registration');
-        }
-        return $this->cap_check;
-    }
-
-
-    /**
-     * @return EE_Transaction
-     */
-    public function transaction()
-    {
-        return $this->transaction;
-    }
-
-
-    /**
-     * @return EE_Ticket
-     */
-    public function ticket()
-    {
-        return $this->ticket;
-    }
-
-
-    /**
-     * @return EE_Line_Item
-     */
-    public function ticketLineItem()
-    {
-        return $this->ticket_line_item;
-    }
-
-
-    /**
-     * @return int
-     */
-    public function regCount()
-    {
-        return $this->reg_count;
-    }
-
-
-    /**
-     * @return int
-     */
-    public function regGroupSize()
-    {
-        return $this->reg_group_size;
-    }
-
-
-    /**
-     * @return string
-     */
-    public function regStatus()
-    {
-        return $this->reg_status;
-    }
-
-
-    /**
-     * @return EE_Registration
-     */
-    public function registration()
-    {
-        return $this->registration;
-    }
+	/**
+	 * @var EE_Transaction $transaction
+	 */
+	private $transaction;
+
+	/**
+	 * @var EE_Ticket $ticket
+	 */
+	private $ticket;
+
+	/**
+	 * @var EE_Line_Item $ticket_line_item
+	 */
+	private $ticket_line_item;
+
+	/**
+	 * @var int $reg_count
+	 */
+	private $reg_count;
+
+	/**
+	 * @var int $reg_group_size
+	 */
+	private $reg_group_size;
+
+	/**
+	 * @var string $reg_status
+	 */
+	private $reg_status;
+
+	/**
+	 * @var EE_Registration $registration
+	 */
+	protected $registration;
+
+
+	/**
+	 * CreateRegistrationCommand constructor.
+	 *
+	 * @param EE_Transaction $transaction
+	 * @param EE_Line_Item   $ticket_line_item
+	 * @param int            $reg_count
+	 * @param int            $reg_group_size
+	 * @param string         $reg_status
+	 * @param EE_Ticket|null $ticket
+	 * @throws InvalidEntityException
+	 * @throws EE_Error
+	 */
+	public function __construct(
+		EE_Transaction $transaction,
+		EE_Line_Item $ticket_line_item,
+		$reg_count = 1,
+		$reg_group_size = 0,
+		$reg_status = RegStatus::INCOMPLETE,
+		EE_Ticket $ticket = null
+	) {
+		defined('EVENT_ESPRESSO_VERSION') || exit;
+		$this->transaction      = $transaction;
+		$this->ticket_line_item = $ticket_line_item;
+		$this->reg_count        = absint($reg_count);
+		$this->reg_group_size   = absint($reg_group_size);
+		$this->reg_status       = $reg_status;
+		// grab the related ticket object for this line_item if one wasn't already supplied
+		$this->ticket = $ticket instanceof EE_Ticket ? $ticket : $this->ticket_line_item->ticket();
+		if (! $this->ticket instanceof EE_Ticket) {
+			throw new InvalidEntityException(
+				is_object($this->ticket) ? get_class($this->ticket) : gettype($this->ticket),
+				'EE_Ticket',
+				sprintf(
+					esc_html__('Line item %s did not contain a valid ticket', 'event_espresso'),
+					$ticket_line_item->ID()
+				)
+			);
+		}
+	}
+
+
+	/**
+	 * @return CapCheckInterface
+	 * @throws InvalidDataTypeException
+	 */
+	public function getCapCheck()
+	{
+		if (! $this->cap_check instanceof CapCheckInterface) {
+			// need cap for non-AJAX admin requests
+			$this->cap_check = ! (defined('DOING_AJAX') && DOING_AJAX) && is_admin()
+				? new CapCheck('ee_edit_registrations', 'create_new_registration')
+				: new PublicCapabilities('', 'create_new_registration');
+		}
+		return $this->cap_check;
+	}
+
+
+	/**
+	 * @return EE_Transaction
+	 */
+	public function transaction()
+	{
+		return $this->transaction;
+	}
+
+
+	/**
+	 * @return EE_Ticket
+	 */
+	public function ticket()
+	{
+		return $this->ticket;
+	}
+
+
+	/**
+	 * @return EE_Line_Item
+	 */
+	public function ticketLineItem()
+	{
+		return $this->ticket_line_item;
+	}
+
+
+	/**
+	 * @return int
+	 */
+	public function regCount()
+	{
+		return $this->reg_count;
+	}
+
+
+	/**
+	 * @return int
+	 */
+	public function regGroupSize()
+	{
+		return $this->reg_group_size;
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function regStatus()
+	{
+		return $this->reg_status;
+	}
+
+
+	/**
+	 * @return EE_Registration
+	 */
+	public function registration()
+	{
+		return $this->registration;
+	}
 }

core/domain/services/commands/registration/CancelRegistrationAndTicketLineItemCommandHandler.php

Indentation +50 added lines, -50 removed lines

@@ -27,58 +27,58 @@
  */
 class CancelRegistrationAndTicketLineItemCommandHandler extends CommandHandler
 {
-    /**
-     * @var CancelTicketLineItemService $cancel_ticket_line_item_service
-     */
-    private $cancel_ticket_line_item_service;
+	/**
+	 * @var CancelTicketLineItemService $cancel_ticket_line_item_service
+	 */
+	private $cancel_ticket_line_item_service;
 
 
-    /**
-     * Command constructor
-     *
-     * @param CancelTicketLineItemService $cancel_ticket_line_item_service
-     */
-    public function __construct(CancelTicketLineItemService $cancel_ticket_line_item_service)
-    {
-        defined('EVENT_ESPRESSO_VERSION') || exit;
-        $this->cancel_ticket_line_item_service = $cancel_ticket_line_item_service;
-    }
+	/**
+	 * Command constructor
+	 *
+	 * @param CancelTicketLineItemService $cancel_ticket_line_item_service
+	 */
+	public function __construct(CancelTicketLineItemService $cancel_ticket_line_item_service)
+	{
+		defined('EVENT_ESPRESSO_VERSION') || exit;
+		$this->cancel_ticket_line_item_service = $cancel_ticket_line_item_service;
+	}
 
 
-    /**
-     * @param CommandInterface|CancelRegistrationAndTicketLineItemCommand $command
-     * @return boolean
-     * @throws DomainException
-     * @throws EE_Error
-     * @throws EntityNotFoundException
-     * @throws InvalidDataTypeException
-     * @throws InvalidEntityException
-     * @throws InvalidInterfaceException
-     * @throws InvalidArgumentException
-     * @throws ReflectionException
-     * @throws RuntimeException
-     */
-    public function handle(CommandInterface $command)
-    {
-        /** @var CancelRegistrationAndTicketLineItemCommand $command */
-        if (! $command instanceof CancelRegistrationAndTicketLineItemCommand) {
-            throw new InvalidEntityException(get_class($command), 'CancelRegistrationAndTicketLineItemCommand');
-        }
-        $registration = $command->registration();
-        $this->cancel_ticket_line_item_service->forRegistration($registration);
-        // cancel original registration
-        $registration->set_status(
-            RegStatus::CANCELLED,
-            false,
-            new Context(
-                'cancel-registration-and-ticket-line-item-command-handler',
-                esc_html__(
-                    'Executed when the registration status is updated via CancelRegistrationAndTicketLineItemCommandHandler.',
-                    'event_espresso'
-                )
-            )
-        );
-        $registration->save();
-        return true;
-    }
+	/**
+	 * @param CommandInterface|CancelRegistrationAndTicketLineItemCommand $command
+	 * @return boolean
+	 * @throws DomainException
+	 * @throws EE_Error
+	 * @throws EntityNotFoundException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidEntityException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidArgumentException
+	 * @throws ReflectionException
+	 * @throws RuntimeException
+	 */
+	public function handle(CommandInterface $command)
+	{
+		/** @var CancelRegistrationAndTicketLineItemCommand $command */
+		if (! $command instanceof CancelRegistrationAndTicketLineItemCommand) {
+			throw new InvalidEntityException(get_class($command), 'CancelRegistrationAndTicketLineItemCommand');
+		}
+		$registration = $command->registration();
+		$this->cancel_ticket_line_item_service->forRegistration($registration);
+		// cancel original registration
+		$registration->set_status(
+			RegStatus::CANCELLED,
+			false,
+			new Context(
+				'cancel-registration-and-ticket-line-item-command-handler',
+				esc_html__(
+					'Executed when the registration status is updated via CancelRegistrationAndTicketLineItemCommandHandler.',
+					'event_espresso'
+				)
+			)
+		);
+		$registration->save();
+		return true;
+	}
 }

core/domain/services/contexts/RequestTypeContextChecker.php

Indentation +232 added lines, -232 removed lines

@@ -15,236 +15,236 @@
  */
 class RequestTypeContextChecker extends ContextChecker implements RequestTypeContextCheckerInterface
 {
-    private RequestTypeContext $request_type;
-
-
-    /**
-     * RequestTypeContextChecker constructor.
-     *
-     * @param RequestTypeContext $request_type
-     */
-    public function __construct(RequestTypeContext $request_type)
-    {
-        $this->request_type = $request_type;
-        parent::__construct(
-            'RequestTypeContextChecker',
-            $this->request_type->validRequestTypes()
-        );
-    }
-
-
-    /**
-     * true if the current request involves some form of activation
-     *
-     * @return bool
-     */
-    public function isActivation(): bool
-    {
-        return ($this->request_type->slug() === RequestTypeContext::ACTIVATION || $this->request_type->isActivation())
-            && ! $this->request_type->isUnitTesting();
-    }
-
-
-    /**
-     * @param $is_activation
-     */
-    public function setIsActivation($is_activation)
-    {
-        $this->request_type->setIsActivation($is_activation);
-    }
-
-
-    /**
-     * true if the current request is for the admin and is not being made via AJAX
-     *
-     * @return bool
-     */
-    public function isAdmin(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::ADMIN && ! $this->isActivation();
-    }
-
-
-    /**
-     * true if the current request is for the admin AND is being made via AJAX
-     *
-     * @return bool
-     */
-    public function isAdminAjax(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::AJAX_ADMIN && ! $this->isActivation();
-    }
-
-
-    /**
-     * true if the current request is being made via AJAX... any AJAX
-     *
-     * @return bool
-     */
-    public function isAjax(): bool
-    {
-        return $this->isEeAjax() || $this->isOtherAjax();
-    }
-
-
-    /**
-     * true if the current request is for either the EE admin or EE frontend AND is being made via AJAX
-     *
-     * @return bool
-     */
-    public function isEeAjax(): bool
-    {
-        return $this->isAdminAjax() || $this->isFrontAjax();
-    }
-
-
-    /**
-     * true if the current request is being made via AJAX but is NOT for EE related logic
-     *
-     * @return bool
-     */
-    public function isOtherAjax(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::AJAX_OTHER && ! $this->isActivation();
-    }
-
-    /**
-     * true if the current request is for the EE REST API
-     *
-     * @return bool
-     */
-    public function isApi(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::API && ! $this->isActivation();
-    }
-
-
-    /**
-     * true if the current request is from the command line
-     *
-     * @return bool
-     */
-    public function isCli(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::CLI && ! $this->isActivation();
-    }
-
-
-    /**
-     * true if the current request is for a WP_Cron
-     *
-     * @return bool
-     */
-    public function isCron(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::CRON && ! $this->isActivation();
-    }
-
-
-    /**
-     * true if the current request is for a feed (ie: RSS)
-     *
-     * @return bool
-     */
-    public function isFeed(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::FEED && ! $this->isActivation();
-    }
-
-
-    /**
-     * true if the current request is for the frontend and is not being made via AJAX
-     *
-     * @return bool
-     */
-    public function isFrontend(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::FRONTEND && ! $this->isActivation();
-    }
-
-
-    /**
-     * true if the current request is for the frontend AND is being made via AJAX
-     *
-     * @return bool
-     */
-    public function isFrontAjax(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::AJAX_FRONT && ! $this->isActivation();
-    }
-
-
-    /**
-     * true if the current request is for the EE GraphQL manager
-     *
-     * @return bool
-     */
-    public function isGQL(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::GQL && ! $this->isActivation();
-    }
-
-
-    /**
-     * true if the current request is for content that is to be displayed within an iframe
-     *
-     * @return bool
-     */
-    public function isIframe(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::IFRAME && ! $this->isActivation();
-    }
-
-
-    /**
-     * true if the current request is occurring while unit testing
-     *
-     * @return bool
-     */
-    public function isUnitTesting(): bool
-    {
-        return $this->request_type->isUnitTesting() && ! $this->request_type->isActivation();
-    }
-
-
-    /**
-     * true if the current request is for the WP REST API
-     *
-     * @return bool
-     */
-    public function isWordPressApi(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::WP_API && ! $this->isActivation();
-    }
-
-
-    /**
-     * true if the current request is being made via AJAX for the WP Heartbeat
-     *
-     * @return bool
-     */
-    public function isWordPressHeartbeat(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::AJAX_HEARTBEAT && ! $this->isActivation();
-    }
-
-
-    /**
-     * true if the current request is a loopback sent from WP core to test for errors
-     *
-     * @return bool
-     */
-    public function isWordPressScrape(): bool
-    {
-        return $this->request_type->slug() === RequestTypeContext::WP_SCRAPE;
-    }
-
-
-    /**
-     * @return string
-     */
-    public function slug(): string
-    {
-        return $this->request_type->slug();
-    }
+	private RequestTypeContext $request_type;
+
+
+	/**
+	 * RequestTypeContextChecker constructor.
+	 *
+	 * @param RequestTypeContext $request_type
+	 */
+	public function __construct(RequestTypeContext $request_type)
+	{
+		$this->request_type = $request_type;
+		parent::__construct(
+			'RequestTypeContextChecker',
+			$this->request_type->validRequestTypes()
+		);
+	}
+
+
+	/**
+	 * true if the current request involves some form of activation
+	 *
+	 * @return bool
+	 */
+	public function isActivation(): bool
+	{
+		return ($this->request_type->slug() === RequestTypeContext::ACTIVATION || $this->request_type->isActivation())
+			&& ! $this->request_type->isUnitTesting();
+	}
+
+
+	/**
+	 * @param $is_activation
+	 */
+	public function setIsActivation($is_activation)
+	{
+		$this->request_type->setIsActivation($is_activation);
+	}
+
+
+	/**
+	 * true if the current request is for the admin and is not being made via AJAX
+	 *
+	 * @return bool
+	 */
+	public function isAdmin(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::ADMIN && ! $this->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is for the admin AND is being made via AJAX
+	 *
+	 * @return bool
+	 */
+	public function isAdminAjax(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::AJAX_ADMIN && ! $this->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is being made via AJAX... any AJAX
+	 *
+	 * @return bool
+	 */
+	public function isAjax(): bool
+	{
+		return $this->isEeAjax() || $this->isOtherAjax();
+	}
+
+
+	/**
+	 * true if the current request is for either the EE admin or EE frontend AND is being made via AJAX
+	 *
+	 * @return bool
+	 */
+	public function isEeAjax(): bool
+	{
+		return $this->isAdminAjax() || $this->isFrontAjax();
+	}
+
+
+	/**
+	 * true if the current request is being made via AJAX but is NOT for EE related logic
+	 *
+	 * @return bool
+	 */
+	public function isOtherAjax(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::AJAX_OTHER && ! $this->isActivation();
+	}
+
+	/**
+	 * true if the current request is for the EE REST API
+	 *
+	 * @return bool
+	 */
+	public function isApi(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::API && ! $this->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is from the command line
+	 *
+	 * @return bool
+	 */
+	public function isCli(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::CLI && ! $this->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is for a WP_Cron
+	 *
+	 * @return bool
+	 */
+	public function isCron(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::CRON && ! $this->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is for a feed (ie: RSS)
+	 *
+	 * @return bool
+	 */
+	public function isFeed(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::FEED && ! $this->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is for the frontend and is not being made via AJAX
+	 *
+	 * @return bool
+	 */
+	public function isFrontend(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::FRONTEND && ! $this->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is for the frontend AND is being made via AJAX
+	 *
+	 * @return bool
+	 */
+	public function isFrontAjax(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::AJAX_FRONT && ! $this->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is for the EE GraphQL manager
+	 *
+	 * @return bool
+	 */
+	public function isGQL(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::GQL && ! $this->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is for content that is to be displayed within an iframe
+	 *
+	 * @return bool
+	 */
+	public function isIframe(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::IFRAME && ! $this->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is occurring while unit testing
+	 *
+	 * @return bool
+	 */
+	public function isUnitTesting(): bool
+	{
+		return $this->request_type->isUnitTesting() && ! $this->request_type->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is for the WP REST API
+	 *
+	 * @return bool
+	 */
+	public function isWordPressApi(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::WP_API && ! $this->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is being made via AJAX for the WP Heartbeat
+	 *
+	 * @return bool
+	 */
+	public function isWordPressHeartbeat(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::AJAX_HEARTBEAT && ! $this->isActivation();
+	}
+
+
+	/**
+	 * true if the current request is a loopback sent from WP core to test for errors
+	 *
+	 * @return bool
+	 */
+	public function isWordPressScrape(): bool
+	{
+		return $this->request_type->slug() === RequestTypeContext::WP_SCRAPE;
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function slug(): string
+	{
+		return $this->request_type->slug();
+	}
 }

core/domain/services/licensing/LicenseStatus.php

Indentation +74 added lines, -74 removed lines

@@ -4,90 +4,90 @@
 
 class LicenseStatus
 {
-    public static function statusNotice(?string $license_status): string
-    {
-        if (empty($license_status)) {
-            return "<span class='ee-status-pill'></span>";
-        }
-        $class = self::statusClass($license_status);
-        $icon = self::statusIcon($license_status);
-        $notice = self::statusMessage($license_status);
-        return "<span class='ee-status-pill ee-status-bg--$class'><span class='ee-license-status'>$notice</span>&nbsp;&nbsp;<span class='dashicons dashicons-$icon'></span></span>";
-    }
+	public static function statusNotice(?string $license_status): string
+	{
+		if (empty($license_status)) {
+			return "<span class='ee-status-pill'></span>";
+		}
+		$class = self::statusClass($license_status);
+		$icon = self::statusIcon($license_status);
+		$notice = self::statusMessage($license_status);
+		return "<span class='ee-status-pill ee-status-bg--$class'><span class='ee-license-status'>$notice</span>&nbsp;&nbsp;<span class='dashicons dashicons-$icon'></span></span>";
+	}
 
-    public static function statusMessages(): array
-    {
-        return [
-            "deactivated"           => esc_html__("license key deactivated", 'event_espresso'),
-            "disabled"              => esc_html__("license key revoked", 'event_espresso'),
-            "expired"               => esc_html__("license has expired", 'event_espresso'),
-            "inactive"              => esc_html__("inactive license key", 'event_espresso'),
-            "invalid"               => esc_html__("invalid license key", 'event_espresso'),
-            "invalid_item_id"       => esc_html__("invalid item ID", 'event_espresso'),
-            "item_name_mismatch"    => esc_html__("invalid license for plugin", 'event_espresso'),
-            "key_mismatch"          => esc_html__("invalid license for plugin", 'event_espresso'),
-            "license_not_activable" => esc_html__("attempting to activate a bundle's parent license", 'event_espresso'),
-            "missing"               => esc_html__("no license found", 'event_espresso'),
-            "missing_url"           => esc_html__("site URL not found", 'event_espresso'),
-            "no_activations_left"   => esc_html__("no activations left", 'event_espresso'),
-            "site_inactive"         => esc_html__("site is not active for this license", 'event_espresso'),
-            "valid"                 => esc_html__("valid license key", 'event_espresso'),
-        ];
-    }
+	public static function statusMessages(): array
+	{
+		return [
+			"deactivated"           => esc_html__("license key deactivated", 'event_espresso'),
+			"disabled"              => esc_html__("license key revoked", 'event_espresso'),
+			"expired"               => esc_html__("license has expired", 'event_espresso'),
+			"inactive"              => esc_html__("inactive license key", 'event_espresso'),
+			"invalid"               => esc_html__("invalid license key", 'event_espresso'),
+			"invalid_item_id"       => esc_html__("invalid item ID", 'event_espresso'),
+			"item_name_mismatch"    => esc_html__("invalid license for plugin", 'event_espresso'),
+			"key_mismatch"          => esc_html__("invalid license for plugin", 'event_espresso'),
+			"license_not_activable" => esc_html__("attempting to activate a bundle's parent license", 'event_espresso'),
+			"missing"               => esc_html__("no license found", 'event_espresso'),
+			"missing_url"           => esc_html__("site URL not found", 'event_espresso'),
+			"no_activations_left"   => esc_html__("no activations left", 'event_espresso'),
+			"site_inactive"         => esc_html__("site is not active for this license", 'event_espresso'),
+			"valid"                 => esc_html__("valid license key", 'event_espresso'),
+		];
+	}
 
-    public static function statusMessage(?string $license_status): string
-    {
-        $license_statuses = self::statusMessages();
-        return $license_statuses[ $license_status ] ?? esc_html__("An unknown error occurred", 'event_espresso');
-    }
+	public static function statusMessage(?string $license_status): string
+	{
+		$license_statuses = self::statusMessages();
+		return $license_statuses[ $license_status ] ?? esc_html__("An unknown error occurred", 'event_espresso');
+	}
 
 
-    public static function statusClass(?string $license_status): string
-    {
-        switch ( $license_status ) {
-            case "valid":
-                return "active";
+	public static function statusClass(?string $license_status): string
+	{
+		switch ( $license_status ) {
+			case "valid":
+				return "active";
 
-            case "deactivated":
-            case "expired":
-            case "inactive":
-            case "site_inactive":
-                return "inactive";
+			case "deactivated":
+			case "expired":
+			case "inactive":
+			case "site_inactive":
+				return "inactive";
 
-            case "no_activations_left":
-                return "attention";
+			case "no_activations_left":
+				return "attention";
 
-            case "disabled":
-            case "revoked":
-            case "invalid":
-            case "item_name_mismatch":
-            case "missing":
-            default:
-                return "error";
-        }
-    }
+			case "disabled":
+			case "revoked":
+			case "invalid":
+			case "item_name_mismatch":
+			case "missing":
+			default:
+				return "error";
+		}
+	}
 
 
-    public static function statusIcon(?string $license_status): string
-    {
-        switch ( $license_status ) {
-            case "valid":
-                return "yes-alt";
+	public static function statusIcon(?string $license_status): string
+	{
+		switch ( $license_status ) {
+			case "valid":
+				return "yes-alt";
 
-            case "expired":
-            case "site_inactive":
-                return "clock";
+			case "expired":
+			case "site_inactive":
+				return "clock";
 
-            case "disabled":
-            case "revoked":
-            case "invalid":
-            case "item_name_mismatch":
-            case "no_activations_left":
-                return "warning";
+			case "disabled":
+			case "revoked":
+			case "invalid":
+			case "item_name_mismatch":
+			case "no_activations_left":
+				return "warning";
 
-            case "missing":
-            default:
-                return "editor-help";
-        }
-    }
+			case "missing":
+			default:
+				return "editor-help";
+		}
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -38,13 +38,13 @@ 
     public static function statusMessage(?string $license_status): string
     {
         $license_statuses = self::statusMessages();
-        return $license_statuses[ $license_status ] ?? esc_html__("An unknown error occurred", 'event_espresso');
+        return $license_statuses[$license_status] ?? esc_html__("An unknown error occurred", 'event_espresso');
     }
 
 
     public static function statusClass(?string $license_status): string
     {
-        switch ( $license_status ) {
+        switch ($license_status) {
             case "valid":
                 return "active";
 
@@ -70,7 +70,7 @@ 
 
     public static function statusIcon(?string $license_status): string
     {
-        switch ( $license_status ) {
+        switch ($license_status) {
             case "valid":
                 return "yes-alt";