eventespresso / event-espresso-core

core/libraries/batch/JobHandlers/PreviewEventDeletion.php

Spacing +3 added lines, -3 removed lines

@@ -209,7 +209,7 @@ 
             if ($units_processed >= $batch_size) {
                 break;
             }
-            if (!$root_node instanceof ModelObjNode) {
+            if ( ! $root_node instanceof ModelObjNode) {
                 throw new InvalidClassException('ModelObjNode');
             }
             if ($root_node->isComplete()) {
@@ -220,7 +220,7 @@ 
         $job_parameters->mark_processed($units_processed);
         // If the most-recently processed root node is complete, we must be all done because we're doing them
         // sequentially.
-        if (! isset($root_node) || ($root_node instanceof ModelObjNode && $root_node->isComplete())) {
+        if ( ! isset($root_node) || ($root_node instanceof ModelObjNode && $root_node->isComplete())) {
             $job_parameters->set_status(JobParameters::status_complete);
             // Show a full progress bar.
             $job_parameters->set_units_processed($job_parameters->job_size());
@@ -263,7 +263,7 @@ 
                         'event_espresso'
                     ),
                     '<span id="ee-redirect-timer">10</span>',
-                    '<a href="' . $job_parameters->request_datum('return_url') . '">',
+                    '<a href="'.$job_parameters->request_datum('return_url').'">',
                     '</a>'
                 )
             )

Indentation +212 added lines, -212 removed lines

@@ -33,141 +33,141 @@ 
  */
 class PreviewEventDeletion extends JobHandler
 {
-    protected NodeGroupDao $model_obj_node_group_persister;
+	protected NodeGroupDao $model_obj_node_group_persister;
 
-    public function __construct(NodeGroupDao $model_obj_node_group_persister)
-    {
-        $this->model_obj_node_group_persister = $model_obj_node_group_persister;
-    }
+	public function __construct(NodeGroupDao $model_obj_node_group_persister)
+	{
+		$this->model_obj_node_group_persister = $model_obj_node_group_persister;
+	}
 
-    // phpcs:disable PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	// phpcs:disable PSR1.Methods.CamelCapsMethodName.NotCamelCaps
 
-    /**
-     *
-     * @param JobParameters $job_parameters
-     * @return JobStepResponse
-     * @throws EE_Error
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     * @throws InvalidArgumentException
-     * @throws ReflectionException
-     */
-    public function create_job(JobParameters $job_parameters): JobStepResponse
-    {
-        // Set the "root" model objects we will want to delete (record their ID and model)
-        $event_ids = $job_parameters->request_datum('EVT_IDs', array());
-        // Find all the root nodes to delete (this isn't just events, because there's other data, like related tickets,
-        // prices, message templates, etc, whose model definition doesn't make them dependent on events. But,
-        // we have no UI to access them independent of events, so they may as well get deleted too.)
-        $roots = [];
-        foreach ($event_ids as $event_id) {
-            $roots[] = new ModelObjNode(
-                $event_id,
-                EEM_Event::instance()
-            );
-            // Also, we want to delete their related, non-global, tickets, prices and message templates
-            $related_non_global_tickets = EEM_Ticket::instance()->get_all_deleted_and_undeleted(
-                [
-                    [
-                        'TKT_is_default' => false,
-                        'Datetime.EVT_ID' => $event_id
-                    ]
-                ]
-            );
-            foreach ($related_non_global_tickets as $ticket) {
-                $roots[] = new ModelObjNode(
-                    $ticket->ID(),
-                    $ticket->get_model(),
-                    ['Registration']
-                );
-            }
-            $related_non_global_prices = EEM_Price::instance()->get_all_deleted_and_undeleted(
-                [
-                    [
-                        'PRC_is_default' => false,
-                        'Ticket.Datetime.EVT_ID' => $event_id
-                    ]
-                ]
-            );
-            foreach ($related_non_global_prices as $price) {
-                $roots[] = new ModelObjNode(
-                    $price->ID(),
-                    $price->get_model()
-                );
-            }
-        }
-        $transactions_ids = $this->getTransactionsToDelete($event_ids);
-        foreach ($transactions_ids as $transaction_id) {
-            $roots[] = new ModelObjNode(
-                $transaction_id,
-                EEM_Transaction::instance(),
-                ['Registration']
-            );
-        }
-        $job_parameters->add_extra_data('roots', $roots);
-        // Set an estimate of how long this will take (we're discovering as we go, so it seems impossible to give
-        // an accurate count.)
-        $estimated_work_per_model_obj = 10;
-        $count_regs = EEM_Registration::instance()->count(
-            [
-                [
-                    'EVT_ID' => ['IN', $event_ids]
-                ]
-            ]
-        );
-        $job_parameters->set_job_size((count($roots) + $count_regs) * $estimated_work_per_model_obj);
-        $this->updateTextHeader(esc_html__('Generating preview of data to be deleted...', 'event_espresso'));
-        return new JobStepResponse($job_parameters, $this->feedback);
-    }
+	/**
+	 *
+	 * @param JobParameters $job_parameters
+	 * @return JobStepResponse
+	 * @throws EE_Error
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidArgumentException
+	 * @throws ReflectionException
+	 */
+	public function create_job(JobParameters $job_parameters): JobStepResponse
+	{
+		// Set the "root" model objects we will want to delete (record their ID and model)
+		$event_ids = $job_parameters->request_datum('EVT_IDs', array());
+		// Find all the root nodes to delete (this isn't just events, because there's other data, like related tickets,
+		// prices, message templates, etc, whose model definition doesn't make them dependent on events. But,
+		// we have no UI to access them independent of events, so they may as well get deleted too.)
+		$roots = [];
+		foreach ($event_ids as $event_id) {
+			$roots[] = new ModelObjNode(
+				$event_id,
+				EEM_Event::instance()
+			);
+			// Also, we want to delete their related, non-global, tickets, prices and message templates
+			$related_non_global_tickets = EEM_Ticket::instance()->get_all_deleted_and_undeleted(
+				[
+					[
+						'TKT_is_default' => false,
+						'Datetime.EVT_ID' => $event_id
+					]
+				]
+			);
+			foreach ($related_non_global_tickets as $ticket) {
+				$roots[] = new ModelObjNode(
+					$ticket->ID(),
+					$ticket->get_model(),
+					['Registration']
+				);
+			}
+			$related_non_global_prices = EEM_Price::instance()->get_all_deleted_and_undeleted(
+				[
+					[
+						'PRC_is_default' => false,
+						'Ticket.Datetime.EVT_ID' => $event_id
+					]
+				]
+			);
+			foreach ($related_non_global_prices as $price) {
+				$roots[] = new ModelObjNode(
+					$price->ID(),
+					$price->get_model()
+				);
+			}
+		}
+		$transactions_ids = $this->getTransactionsToDelete($event_ids);
+		foreach ($transactions_ids as $transaction_id) {
+			$roots[] = new ModelObjNode(
+				$transaction_id,
+				EEM_Transaction::instance(),
+				['Registration']
+			);
+		}
+		$job_parameters->add_extra_data('roots', $roots);
+		// Set an estimate of how long this will take (we're discovering as we go, so it seems impossible to give
+		// an accurate count.)
+		$estimated_work_per_model_obj = 10;
+		$count_regs = EEM_Registration::instance()->count(
+			[
+				[
+					'EVT_ID' => ['IN', $event_ids]
+				]
+			]
+		);
+		$job_parameters->set_job_size((count($roots) + $count_regs) * $estimated_work_per_model_obj);
+		$this->updateTextHeader(esc_html__('Generating preview of data to be deleted...', 'event_espresso'));
+		return new JobStepResponse($job_parameters, $this->feedback);
+	}
 
-    /**
-     * @param EE_Base_Class[] $model_objs
-     * @param array           $dont_traverse_models
-     * @return array
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     * @throws ReflectionException
-     * @since 4.10.12.p
-     */
-    protected function createModelObjNodes(array $model_objs, array $dont_traverse_models = []): array
-    {
-        $nodes = [];
-        foreach ($model_objs as $model_obj) {
-            $nodes[] = new ModelObjNode(
-                $model_obj->ID(),
-                $model_obj->get_model(),
-                $dont_traverse_models
-            );
-        }
-        return $nodes;
-    }
+	/**
+	 * @param EE_Base_Class[] $model_objs
+	 * @param array           $dont_traverse_models
+	 * @return array
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 * @throws ReflectionException
+	 * @since 4.10.12.p
+	 */
+	protected function createModelObjNodes(array $model_objs, array $dont_traverse_models = []): array
+	{
+		$nodes = [];
+		foreach ($model_objs as $model_obj) {
+			$nodes[] = new ModelObjNode(
+				$model_obj->ID(),
+				$model_obj->get_model(),
+				$dont_traverse_models
+			);
+		}
+		return $nodes;
+	}
 
-    /**
-     * Gets all the transactions related to these events that aren't related to other events. They'll be deleted too.
-     * (Ones that are related to other events can stay around until those other events are deleted too.)
-     * @since 4.10.12.p
-     * @param array $event_ids
-     * @return array of transaction IDs
-     */
-    protected function getTransactionsToDelete(array $event_ids): array
-    {
-        if (empty($event_ids)) {
-            return [];
-        }
-        global $wpdb;
-        $event_ids = array_map('absint', $event_ids);
-        $imploded_sanitized_event_ids = implode(',', $event_ids);
-        // Select transactions with registrations for the events $event_ids which also don't have registrations
-        // for any events NOT in $event_ids.
-        // Notice the outer query searched for transactions whose registrations ARE in $event_ids,
-        // whereas the inner query checks if the outer query's transaction has any registrations that are
-        // NOT IN $event_ids (ie, don't have registrations for events we're not just about to delete.)
-        return array_map(
-            'absint',
-            $wpdb->get_col(
-                "SELECT
+	/**
+	 * Gets all the transactions related to these events that aren't related to other events. They'll be deleted too.
+	 * (Ones that are related to other events can stay around until those other events are deleted too.)
+	 * @since 4.10.12.p
+	 * @param array $event_ids
+	 * @return array of transaction IDs
+	 */
+	protected function getTransactionsToDelete(array $event_ids): array
+	{
+		if (empty($event_ids)) {
+			return [];
+		}
+		global $wpdb;
+		$event_ids = array_map('absint', $event_ids);
+		$imploded_sanitized_event_ids = implode(',', $event_ids);
+		// Select transactions with registrations for the events $event_ids which also don't have registrations
+		// for any events NOT in $event_ids.
+		// Notice the outer query searched for transactions whose registrations ARE in $event_ids,
+		// whereas the inner query checks if the outer query's transaction has any registrations that are
+		// NOT IN $event_ids (ie, don't have registrations for events we're not just about to delete.)
+		return array_map(
+			'absint',
+			$wpdb->get_col(
+				"SELECT
                       DISTINCT t.TXN_ID
                     FROM
                       {$wpdb->prefix}esp_transaction t INNER JOIN
@@ -185,90 +185,90 @@ 
                            tsub.TXN_ID=t.TXN_ID AND
                            rsub.EVT_ID NOT IN ($imploded_sanitized_event_ids)
                        )"
-            )
-        );
-    }
+			)
+		);
+	}
 
-    /**
-     * Performs another step of the job
-     * @param JobParameters $job_parameters
-     * @param int $batch_size
-     * @return JobStepResponse
-     */
-    public function continue_job(JobParameters $job_parameters, int $batch_size = 50): JobStepResponse
-    {
-        // Serializing and unserializing is what really makes this drag on (eg on localhost, the ajax requests took
-        // about 4 seconds when the batch size was 250, but 3 seconds when the batch size was 50. So like
-        // 50% of the request is just serializing and unserializing.) So, make the batches much bigger.
-        $batch_size *= 3;
-        $units_processed = 0;
-        foreach ($job_parameters->extra_datum('roots', array()) as $root_node) {
-            if ($units_processed >= $batch_size) {
-                break;
-            }
-            if (!$root_node instanceof ModelObjNode) {
-                throw new InvalidClassException('ModelObjNode');
-            }
-            if ($root_node->isComplete()) {
-                continue;
-            }
-            $units_processed += $root_node->visit($batch_size - $units_processed);
-        }
-        $job_parameters->mark_processed($units_processed);
-        // If the most-recently processed root node is complete, we must be all done because we're doing them
-        // sequentially.
-        if (! isset($root_node) || ($root_node instanceof ModelObjNode && $root_node->isComplete())) {
-            $job_parameters->set_status(JobParameters::status_complete);
-            // Show a full progress bar.
-            $job_parameters->set_units_processed($job_parameters->job_size());
-            $deletion_job_code = $job_parameters->request_datum('deletion_job_code');
-            $this->model_obj_node_group_persister->persistModelObjNodesGroup(
-                $job_parameters->extra_datum('roots'),
-                $deletion_job_code
-            );
-            return new JobStepResponse($job_parameters, $this->feedback);
-        }
-        // Because the job size was a guess, it may have likely been proven wrong.
-        // We don't want to show more work done than we originally said there would be.
-        // So adjust the estimate.
-        if (($job_parameters->units_processed() / $job_parameters->job_size()) > .8) {
-            $job_parameters->set_job_size($job_parameters->job_size() * 2);
-        }
-        $this->displayJobStepResults(
-            $units_processed,
-            esc_html__('Identified up to %d potential items for deletion.', 'event_espresso')
-        );
-        return new JobStepResponse($job_parameters, $this->feedback);
-    }
+	/**
+	 * Performs another step of the job
+	 * @param JobParameters $job_parameters
+	 * @param int $batch_size
+	 * @return JobStepResponse
+	 */
+	public function continue_job(JobParameters $job_parameters, int $batch_size = 50): JobStepResponse
+	{
+		// Serializing and unserializing is what really makes this drag on (eg on localhost, the ajax requests took
+		// about 4 seconds when the batch size was 250, but 3 seconds when the batch size was 50. So like
+		// 50% of the request is just serializing and unserializing.) So, make the batches much bigger.
+		$batch_size *= 3;
+		$units_processed = 0;
+		foreach ($job_parameters->extra_datum('roots', array()) as $root_node) {
+			if ($units_processed >= $batch_size) {
+				break;
+			}
+			if (!$root_node instanceof ModelObjNode) {
+				throw new InvalidClassException('ModelObjNode');
+			}
+			if ($root_node->isComplete()) {
+				continue;
+			}
+			$units_processed += $root_node->visit($batch_size - $units_processed);
+		}
+		$job_parameters->mark_processed($units_processed);
+		// If the most-recently processed root node is complete, we must be all done because we're doing them
+		// sequentially.
+		if (! isset($root_node) || ($root_node instanceof ModelObjNode && $root_node->isComplete())) {
+			$job_parameters->set_status(JobParameters::status_complete);
+			// Show a full progress bar.
+			$job_parameters->set_units_processed($job_parameters->job_size());
+			$deletion_job_code = $job_parameters->request_datum('deletion_job_code');
+			$this->model_obj_node_group_persister->persistModelObjNodesGroup(
+				$job_parameters->extra_datum('roots'),
+				$deletion_job_code
+			);
+			return new JobStepResponse($job_parameters, $this->feedback);
+		}
+		// Because the job size was a guess, it may have likely been proven wrong.
+		// We don't want to show more work done than we originally said there would be.
+		// So adjust the estimate.
+		if (($job_parameters->units_processed() / $job_parameters->job_size()) > .8) {
+			$job_parameters->set_job_size($job_parameters->job_size() * 2);
+		}
+		$this->displayJobStepResults(
+			$units_processed,
+			esc_html__('Identified up to %d potential items for deletion.', 'event_espresso')
+		);
+		return new JobStepResponse($job_parameters, $this->feedback);
+	}
 
-    /**
-     * Performs any clean-up logic when we know the job is completed
-     * @param JobParameters $job_parameters
-     * @return JobStepResponse
-     */
-    public function cleanup_job(JobParameters $job_parameters): JobStepResponse
-    {
-        $this->displayJobFinalResults(
-            $job_parameters,
-            esc_html__('found %d potential items for deletion.', 'event_espresso')
-        );
-        $this->updateText(
-            $this->infoWrapper(
-                sprintf(
-                    esc_html__(
-                        'If not automatically redirected in %1$s seconds, click here to %2$scontinue to the confirmation step%3$s',
-                        'event_espresso'
-                    ),
-                    '<span id="ee-redirect-timer">10</span>',
-                    '<a href="' . $job_parameters->request_datum('return_url') . '">',
-                    '</a>'
-                )
-            )
-        );
-        // Nothing much to do.
-        // We can't delete the option with the built tree because we may need it in a moment for the deletion
-        return new JobStepResponse($job_parameters, $this->feedback);
-    }
+	/**
+	 * Performs any clean-up logic when we know the job is completed
+	 * @param JobParameters $job_parameters
+	 * @return JobStepResponse
+	 */
+	public function cleanup_job(JobParameters $job_parameters): JobStepResponse
+	{
+		$this->displayJobFinalResults(
+			$job_parameters,
+			esc_html__('found %d potential items for deletion.', 'event_espresso')
+		);
+		$this->updateText(
+			$this->infoWrapper(
+				sprintf(
+					esc_html__(
+						'If not automatically redirected in %1$s seconds, click here to %2$scontinue to the confirmation step%3$s',
+						'event_espresso'
+					),
+					'<span id="ee-redirect-timer">10</span>',
+					'<a href="' . $job_parameters->request_datum('return_url') . '">',
+					'</a>'
+				)
+			)
+		);
+		// Nothing much to do.
+		// We can't delete the option with the built tree because we may need it in a moment for the deletion
+		return new JobStepResponse($job_parameters, $this->feedback);
+	}
 }
 // End of file EventDeletion.php
 // Location: EventEspresso\core\libraries\batch\JobHandlers/EventDeletion.php

core/libraries/batch/JobHandlerBaseClasses/JobHandler.php

Spacing +4 added lines, -4 removed lines

@@ -132,7 +132,7 @@ 
      */
     public function getRequestData($key)
     {
-        return $this->request_data[ $key ] ?? null;
+        return $this->request_data[$key] ?? null;
     }
 
 
@@ -186,7 +186,7 @@ 
                         : esc_html__('processed this batch: %d', 'event_espresso'),
                 $processed
             )
-        ) . '
+        ).'
 			</div>';
     }
 
@@ -203,8 +203,8 @@ 
                             : esc_html__('total units processed: %d', 'event_espresso'),
                     $job_parameters->units_processed()
                 )
-            ) . '
-				' . $this->jobStatusNotice($job_parameters) . '
+            ).'
+				' . $this->jobStatusNotice($job_parameters).'
 			</div>';
         }
     }

Indentation +231 added lines, -231 removed lines

@@ -17,264 +17,264 @@
  */
 abstract class JobHandler implements JobHandlerInterface
 {
-    protected array $feedback = [];
-
-    protected array $request_data = [];
-
-    /**
-     * Extra data to include as part of the response, literally gets set as JobStepResponse::$_extra_data
-     *
-     * @var array
-     */
-    protected array $response_data = [];
-
-
-    /**
-     * utilized in newer batch job implementations, but forwarding to existing methods for now.
-     * Performs any necessary setup for starting the job. This is also a good
-     * place to setup the $job_arguments which will be used for subsequent HTTP requests
-     * when continue_job will be called
-     *
-     * @param JobParameters $job_parameters
-     * @return JobStepResponse
-     * @throws BatchRequestException
-     * @since 5.0.0.p
-     */
-    public function createJob(JobParameters $job_parameters): JobStepResponse
-    {
-        return $this->create_job($job_parameters);
-    }
-
-
-    /**
-     * utilized in newer batch job implementations, but forwarding to existing methods for now.
-     * Performs another step of the job
-     *
-     * @param JobParameters $job_parameters
-     * @param int           $batch_size
-     * @return JobStepResponse
-     * @throws BatchRequestException
-     * @since 5.0.0.p
-     */
-    public function continueJob(JobParameters $job_parameters, int $batch_size = 50): JobStepResponse
-    {
-        return $this->continue_job($job_parameters, $batch_size);
-    }
-
-
-    /**
-     * utilized in newer batch job implementations, but forwarding to existing methods for now.
-     * used to advance from one batch job to another
-     * primarily used for executing a job assessment phase where an accurate count of items to update can be made,
-     * followed by the actual update job.
-     *
-     * @param JobParameters $job_parameters
-     * @return JobStepResponse
-     * @since 5.0.0.p
-     */
-    public function advanceJob(JobParameters $job_parameters): JobStepResponse
-    {
-        return $this->advance_job($job_parameters);
-    }
-
-    // phpcs:disable PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    /**
-     * used to advance from one batch job to another
-     * primarily used for executing a job assessment phase where an accurate count of items to update can be made,
-     * followed by the actual update job.
-     * By default, this function won't do anything until overridden in a chile class.
-     *
-     * @param JobParameters $job_parameters
-     * @return JobStepResponse
-     * @since 5.0.0.p
-     */
-    public function advance_job(JobParameters $job_parameters): JobStepResponse
-    {
-        $job_parameters->set_status(JobParameters::status_continue);
-        return new JobStepResponse($job_parameters, $this->feedback);
-    }
-
-
-    /**
-     * utilized in newer batch job implementations, but forwarding to existing methods for now.
-     * Performs any clean-up logic when we know the job is completed
-     *
-     * @param JobParameters $job_parameters
-     * @return JobStepResponse
-     * @throws BatchRequestException
-     * @since 5.0.0.p
-     */
-    public function cleanupJob(JobParameters $job_parameters): JobStepResponse
-    {
-        return $this->cleanup_job($job_parameters);
-    }
-
-
-    /**
-     * @return array
-     */
-    public function requestData(): array
-    {
-        return $this->request_data;
-    }
-
-
-    /**
-     * @param string|int $key
-     * @return mixed
-     */
-    public function getRequestData($key)
-    {
-        return $this->request_data[ $key ] ?? null;
-    }
-
-
-    /**
-     * @param array $request_data
-     */
-    public function setRequestData(array $request_data): void
-    {
-        $this->request_data = $request_data;
-    }
-
-
-    /**
-     * @return array
-     */
-    public function responseData(): array
-    {
-        return $this->response_data;
-    }
-
-
-    /**
-     * @param array $response_data
-     */
-    public function addResponseData(array $response_data): void
-    {
-        $this->response_data += $response_data;
-    }
-
-
-
-    /**
-     * @param array $response_data
-     */
-    public function setResponseData(array $response_data): void
-    {
-        $this->response_data = $response_data;
-    }
-
-
-
-
-    protected function displayJobStepResults(int $processed, string $custom_message = '')
-    {
-        $this->feedback[] = '
+	protected array $feedback = [];
+
+	protected array $request_data = [];
+
+	/**
+	 * Extra data to include as part of the response, literally gets set as JobStepResponse::$_extra_data
+	 *
+	 * @var array
+	 */
+	protected array $response_data = [];
+
+
+	/**
+	 * utilized in newer batch job implementations, but forwarding to existing methods for now.
+	 * Performs any necessary setup for starting the job. This is also a good
+	 * place to setup the $job_arguments which will be used for subsequent HTTP requests
+	 * when continue_job will be called
+	 *
+	 * @param JobParameters $job_parameters
+	 * @return JobStepResponse
+	 * @throws BatchRequestException
+	 * @since 5.0.0.p
+	 */
+	public function createJob(JobParameters $job_parameters): JobStepResponse
+	{
+		return $this->create_job($job_parameters);
+	}
+
+
+	/**
+	 * utilized in newer batch job implementations, but forwarding to existing methods for now.
+	 * Performs another step of the job
+	 *
+	 * @param JobParameters $job_parameters
+	 * @param int           $batch_size
+	 * @return JobStepResponse
+	 * @throws BatchRequestException
+	 * @since 5.0.0.p
+	 */
+	public function continueJob(JobParameters $job_parameters, int $batch_size = 50): JobStepResponse
+	{
+		return $this->continue_job($job_parameters, $batch_size);
+	}
+
+
+	/**
+	 * utilized in newer batch job implementations, but forwarding to existing methods for now.
+	 * used to advance from one batch job to another
+	 * primarily used for executing a job assessment phase where an accurate count of items to update can be made,
+	 * followed by the actual update job.
+	 *
+	 * @param JobParameters $job_parameters
+	 * @return JobStepResponse
+	 * @since 5.0.0.p
+	 */
+	public function advanceJob(JobParameters $job_parameters): JobStepResponse
+	{
+		return $this->advance_job($job_parameters);
+	}
+
+	// phpcs:disable PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	/**
+	 * used to advance from one batch job to another
+	 * primarily used for executing a job assessment phase where an accurate count of items to update can be made,
+	 * followed by the actual update job.
+	 * By default, this function won't do anything until overridden in a chile class.
+	 *
+	 * @param JobParameters $job_parameters
+	 * @return JobStepResponse
+	 * @since 5.0.0.p
+	 */
+	public function advance_job(JobParameters $job_parameters): JobStepResponse
+	{
+		$job_parameters->set_status(JobParameters::status_continue);
+		return new JobStepResponse($job_parameters, $this->feedback);
+	}
+
+
+	/**
+	 * utilized in newer batch job implementations, but forwarding to existing methods for now.
+	 * Performs any clean-up logic when we know the job is completed
+	 *
+	 * @param JobParameters $job_parameters
+	 * @return JobStepResponse
+	 * @throws BatchRequestException
+	 * @since 5.0.0.p
+	 */
+	public function cleanupJob(JobParameters $job_parameters): JobStepResponse
+	{
+		return $this->cleanup_job($job_parameters);
+	}
+
+
+	/**
+	 * @return array
+	 */
+	public function requestData(): array
+	{
+		return $this->request_data;
+	}
+
+
+	/**
+	 * @param string|int $key
+	 * @return mixed
+	 */
+	public function getRequestData($key)
+	{
+		return $this->request_data[ $key ] ?? null;
+	}
+
+
+	/**
+	 * @param array $request_data
+	 */
+	public function setRequestData(array $request_data): void
+	{
+		$this->request_data = $request_data;
+	}
+
+
+	/**
+	 * @return array
+	 */
+	public function responseData(): array
+	{
+		return $this->response_data;
+	}
+
+
+	/**
+	 * @param array $response_data
+	 */
+	public function addResponseData(array $response_data): void
+	{
+		$this->response_data += $response_data;
+	}
+
+
+
+	/**
+	 * @param array $response_data
+	 */
+	public function setResponseData(array $response_data): void
+	{
+		$this->response_data = $response_data;
+	}
+
+
+
+
+	protected function displayJobStepResults(int $processed, string $custom_message = '')
+	{
+		$this->feedback[] = '
 			<div class="ee-batch-job-step-results">
 			' . $this->infoWrapper(
-            sprintf(
-                $custom_message !== ''
-                        ? $custom_message
-                        : esc_html__('processed this batch: %d', 'event_espresso'),
-                $processed
-            )
-        ) . '
+			sprintf(
+				$custom_message !== ''
+						? $custom_message
+						: esc_html__('processed this batch: %d', 'event_espresso'),
+				$processed
+			)
+		) . '
 			</div>';
-    }
+	}
 
 
-    protected function displayJobFinalResults(JobParameters $job_parameters, string $custom_message = '')
-    {
-        if ($job_parameters->status() === JobParameters::status_complete) {
-            $this->feedback[] = '
+	protected function displayJobFinalResults(JobParameters $job_parameters, string $custom_message = '')
+	{
+		if ($job_parameters->status() === JobParameters::status_complete) {
+			$this->feedback[] = '
 			<div class="ee-batch-job-final-results">
 				' . $this->okWrapper(
-                sprintf(
-                    $custom_message !== ''
-                            ? $custom_message
-                            : esc_html__('total units processed: %d', 'event_espresso'),
-                    $job_parameters->units_processed()
-                )
-            ) . '
+				sprintf(
+					$custom_message !== ''
+							? $custom_message
+							: esc_html__('total units processed: %d', 'event_espresso'),
+					$job_parameters->units_processed()
+				)
+			) . '
 				' . $this->jobStatusNotice($job_parameters) . '
 			</div>';
-        }
-    }
+		}
+	}
 
 
-    protected function jobStatusNotice(JobParameters $job_parameters): string
-    {
-        switch ($job_parameters->status()) {
-            case JobParameters::status_cleaned_up:
-            case JobParameters::status_continue:
-            case JobParameters::status_pause:
-                break;
-            case JobParameters::status_complete:
-                return $this->successWrapper('job status: COMPLETE');
-            case JobParameters::status_error:
-                return $this->errorWrapper('job status: ERROR');
-        }
-        return '';
-    }
+	protected function jobStatusNotice(JobParameters $job_parameters): string
+	{
+		switch ($job_parameters->status()) {
+			case JobParameters::status_cleaned_up:
+			case JobParameters::status_continue:
+			case JobParameters::status_pause:
+				break;
+			case JobParameters::status_complete:
+				return $this->successWrapper('job status: COMPLETE');
+			case JobParameters::status_error:
+				return $this->errorWrapper('job status: ERROR');
+		}
+		return '';
+	}
 
 
-    /**
-     * @param string $update_text
-     */
-    protected function updateText(string $update_text)
-    {
-        $this->feedback[] = "<p class='ee-batch-job-update'>$update_text</p>";
-    }
+	/**
+	 * @param string $update_text
+	 */
+	protected function updateText(string $update_text)
+	{
+		$this->feedback[] = "<p class='ee-batch-job-update'>$update_text</p>";
+	}
 
 
-    protected function updateTextHeader(string $update_text)
-    {
-        $this->feedback[] = "<h4 class='ee-batch-job-update-heading'>$update_text</h4>";
-    }
+	protected function updateTextHeader(string $update_text)
+	{
+		$this->feedback[] = "<h4 class='ee-batch-job-update-heading'>$update_text</h4>";
+	}
 
 
-    protected function attentionWrapper(string $update_text): string
-    {
-        return "<span class='ee-status-outline ee-status-bg--attention'>$update_text</span>";
-    }
+	protected function attentionWrapper(string $update_text): string
+	{
+		return "<span class='ee-status-outline ee-status-bg--attention'>$update_text</span>";
+	}
 
 
-    protected function errorWrapper(string $update_text): string
-    {
-        return "<span class='ee-status-outline ee-status-bg--error'>$update_text</span>";
-    }
+	protected function errorWrapper(string $update_text): string
+	{
+		return "<span class='ee-status-outline ee-status-bg--error'>$update_text</span>";
+	}
 
 
-    protected function infoWrapper(string $update_text): string
-    {
-        return "<span class='ee-status-outline ee-status-bg--info'>$update_text</span>";
-    }
+	protected function infoWrapper(string $update_text): string
+	{
+		return "<span class='ee-status-outline ee-status-bg--info'>$update_text</span>";
+	}
 
 
-    protected function okWrapper(string $update_text): string
-    {
-        return "<span class='ee-status-outline ee-status-bg--ok'>$update_text</span>";
-    }
+	protected function okWrapper(string $update_text): string
+	{
+		return "<span class='ee-status-outline ee-status-bg--ok'>$update_text</span>";
+	}
 
 
-    protected function successWrapper(string $update_text): string
-    {
-        return "<span class='ee-status-outline ee-status-bg--success'>$update_text</span>";
-    }
+	protected function successWrapper(string $update_text): string
+	{
+		return "<span class='ee-status-outline ee-status-bg--success'>$update_text</span>";
+	}
 
 
-    protected function warningWrapper(string $update_text): string
-    {
-        return "<span class='ee-status-outline ee-status-bg--warning'>$update_text</span>";
-    }
+	protected function warningWrapper(string $update_text): string
+	{
+		return "<span class='ee-status-outline ee-status-bg--warning'>$update_text</span>";
+	}
 
 
-    /**
-     * @return string
-     */
-    protected function spinner(): string
-    {
-        return '<span class="spinner"></span>';
-    }
+	/**
+	 * @return string
+	 */
+	protected function spinner(): string
+	{
+		return '<span class="spinner"></span>';
+	}
 }

core/domain/services/admin/events/data/PreviewDeletion.php

Indentation +108 added lines, -108 removed lines

@@ -30,121 +30,121 @@
  */
 class PreviewDeletion
 {
-    /**
-     * @var NodeGroupDao
-     */
-    protected $dao;
+	/**
+	 * @var NodeGroupDao
+	 */
+	protected $dao;
 
-    /**
-     * @var EEM_Event
-     */
-    protected $event_model;
+	/**
+	 * @var EEM_Event
+	 */
+	protected $event_model;
 
-    /**
-     * @var EEM_Datetime
-     */
-    protected $datetime_model;
+	/**
+	 * @var EEM_Datetime
+	 */
+	protected $datetime_model;
 
-    /**
-     * @var EEM_Registration
-     */
-    protected $registration_model;
+	/**
+	 * @var EEM_Registration
+	 */
+	protected $registration_model;
 
 
-    /**
-     * PreviewDeletion constructor.
-     *
-     * @param NodeGroupDao     $dao
-     * @param EEM_Event        $event_model
-     * @param EEM_Datetime     $datetime_model
-     * @param EEM_Registration $registration_model
-     */
-    public function __construct(
-        NodeGroupDao $dao,
-        EEM_Event $event_model,
-        EEM_Datetime $datetime_model,
-        EEM_Registration $registration_model
-    ) {
-        $this->dao                = $dao;
-        $this->event_model        = $event_model;
-        $this->datetime_model     = $datetime_model;
-        $this->registration_model = $registration_model;
-    }
+	/**
+	 * PreviewDeletion constructor.
+	 *
+	 * @param NodeGroupDao     $dao
+	 * @param EEM_Event        $event_model
+	 * @param EEM_Datetime     $datetime_model
+	 * @param EEM_Registration $registration_model
+	 */
+	public function __construct(
+		NodeGroupDao $dao,
+		EEM_Event $event_model,
+		EEM_Datetime $datetime_model,
+		EEM_Registration $registration_model
+	) {
+		$this->dao                = $dao;
+		$this->event_model        = $event_model;
+		$this->datetime_model     = $datetime_model;
+		$this->registration_model = $registration_model;
+	}
 
 
-    /**
-     * Renders the preview deletion page.
-     *
-     * @param array  $request_data
-     * @param string $admin_base_url
-     * @return array
-     * @throws UnexpectedEntityException
-     * @throws DomainException
-     * @throws EE_Error
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     * @throws InvalidArgumentException
-     * @throws ReflectionException
-     * @since 4.10.12.p
-     */
-    public function handle(array $request_data, string $admin_base_url): array
-    {
-        $deletion_job_code        = isset($request_data['deletion_job_code'])
-            ? sanitize_key($request_data['deletion_job_code'])
-            : '';
-        $models_and_ids_to_delete = $this->dao->getModelsAndIdsFromGroup($deletion_job_code);
-        $event_ids                = $models_and_ids_to_delete['Event'] ?? [];
-        if (empty($event_ids) || ! is_array($event_ids)) {
-            throw new EE_Error(
-                esc_html__('No Events were found to delete.', 'event_espresso')
-            );
-        }
-        $datetime_ids = $models_and_ids_to_delete['Datetime'] ?? [];
-        if (! is_array($datetime_ids)) {
-            throw new UnexpectedEntityException($datetime_ids, 'array');
-        }
-        $registration_ids = $models_and_ids_to_delete['Registration'] ?? [];
-        if (! is_array($registration_ids)) {
-            throw new UnexpectedEntityException($registration_ids, 'array');
-        }
-        $num_registrations_to_show = 10;
-        $reg_count                 = count($registration_ids);
-        if ($reg_count > $num_registrations_to_show) {
-            $registration_ids = array_slice($registration_ids, 0, $num_registrations_to_show);
-        }
-        $form                  = new ConfirmEventDeletionForm($event_ids);
-        $events                = $this->event_model->get_all_deleted_and_undeleted(
-            [['EVT_ID' => ['IN', $event_ids]]]
-        );
-        $datetimes             = $this->datetime_model->get_all_deleted_and_undeleted(
-            [['DTT_ID' => ['IN', $datetime_ids]]]
-        );
-        $registrations         = $this->registration_model->get_all_deleted_and_undeleted(
-            [['REG_ID' => ['IN', $registration_ids]]]
-        );
-        $confirm_deletion_args = [
-            'action'            => 'confirm_deletion',
-            'deletion_job_code' => $deletion_job_code,
-        ];
-        return [
-            'admin_page_content' => EEH_Template::display_template(
-                EVENTS_TEMPLATE_PATH . 'event_preview_deletion.template.php',
-                [
-                    'form_url'                  => EE_Admin_Page::add_query_args_and_nonce(
-                        $confirm_deletion_args,
-                        $admin_base_url
-                    ),
-                    'form'                      => $form,
-                    'events'                    => $events,
-                    'datetimes'                 => $datetimes,
-                    'registrations'             => $registrations,
-                    'reg_count'                 => $reg_count,
-                    'num_registrations_to_show' => $num_registrations_to_show,
-                ],
-                true
-            ),
-        ];
-    }
+	/**
+	 * Renders the preview deletion page.
+	 *
+	 * @param array  $request_data
+	 * @param string $admin_base_url
+	 * @return array
+	 * @throws UnexpectedEntityException
+	 * @throws DomainException
+	 * @throws EE_Error
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidArgumentException
+	 * @throws ReflectionException
+	 * @since 4.10.12.p
+	 */
+	public function handle(array $request_data, string $admin_base_url): array
+	{
+		$deletion_job_code        = isset($request_data['deletion_job_code'])
+			? sanitize_key($request_data['deletion_job_code'])
+			: '';
+		$models_and_ids_to_delete = $this->dao->getModelsAndIdsFromGroup($deletion_job_code);
+		$event_ids                = $models_and_ids_to_delete['Event'] ?? [];
+		if (empty($event_ids) || ! is_array($event_ids)) {
+			throw new EE_Error(
+				esc_html__('No Events were found to delete.', 'event_espresso')
+			);
+		}
+		$datetime_ids = $models_and_ids_to_delete['Datetime'] ?? [];
+		if (! is_array($datetime_ids)) {
+			throw new UnexpectedEntityException($datetime_ids, 'array');
+		}
+		$registration_ids = $models_and_ids_to_delete['Registration'] ?? [];
+		if (! is_array($registration_ids)) {
+			throw new UnexpectedEntityException($registration_ids, 'array');
+		}
+		$num_registrations_to_show = 10;
+		$reg_count                 = count($registration_ids);
+		if ($reg_count > $num_registrations_to_show) {
+			$registration_ids = array_slice($registration_ids, 0, $num_registrations_to_show);
+		}
+		$form                  = new ConfirmEventDeletionForm($event_ids);
+		$events                = $this->event_model->get_all_deleted_and_undeleted(
+			[['EVT_ID' => ['IN', $event_ids]]]
+		);
+		$datetimes             = $this->datetime_model->get_all_deleted_and_undeleted(
+			[['DTT_ID' => ['IN', $datetime_ids]]]
+		);
+		$registrations         = $this->registration_model->get_all_deleted_and_undeleted(
+			[['REG_ID' => ['IN', $registration_ids]]]
+		);
+		$confirm_deletion_args = [
+			'action'            => 'confirm_deletion',
+			'deletion_job_code' => $deletion_job_code,
+		];
+		return [
+			'admin_page_content' => EEH_Template::display_template(
+				EVENTS_TEMPLATE_PATH . 'event_preview_deletion.template.php',
+				[
+					'form_url'                  => EE_Admin_Page::add_query_args_and_nonce(
+						$confirm_deletion_args,
+						$admin_base_url
+					),
+					'form'                      => $form,
+					'events'                    => $events,
+					'datetimes'                 => $datetimes,
+					'registrations'             => $registrations,
+					'reg_count'                 => $reg_count,
+					'num_registrations_to_show' => $num_registrations_to_show,
+				],
+				true
+			),
+		];
+	}
 }
 // End of file PreviewDeletion.php
 // Location: EventEspresso\core\domain\services\admin\events\data/PreviewDeletion.php

Spacing +3 added lines, -3 removed lines

@@ -100,11 +100,11 @@ 
             );
         }
         $datetime_ids = $models_and_ids_to_delete['Datetime'] ?? [];
-        if (! is_array($datetime_ids)) {
+        if ( ! is_array($datetime_ids)) {
             throw new UnexpectedEntityException($datetime_ids, 'array');
         }
         $registration_ids = $models_and_ids_to_delete['Registration'] ?? [];
-        if (! is_array($registration_ids)) {
+        if ( ! is_array($registration_ids)) {
             throw new UnexpectedEntityException($registration_ids, 'array');
         }
         $num_registrations_to_show = 10;
@@ -128,7 +128,7 @@ 
         ];
         return [
             'admin_page_content' => EEH_Template::display_template(
-                EVENTS_TEMPLATE_PATH . 'event_preview_deletion.template.php',
+                EVENTS_TEMPLATE_PATH.'event_preview_deletion.template.php',
                 [
                     'form_url'                  => EE_Admin_Page::add_query_args_and_nonce(
                         $confirm_deletion_args,

admin_pages/events/form_sections/ConfirmEventDeletionForm.php

Indentation +64 added lines, -64 removed lines

@@ -25,67 +25,67 @@ 
  */
 class ConfirmEventDeletionForm extends EE_Form_Section_Proper
 {
-    /**
-     * @throws ReflectionException
-     * @throws EE_Error
-     */
-    public function __construct($event_ids, $options_array = array())
-    {
-        if (! isset($options_array['subsections'])) {
-            $options_array['subsections'] = [];
-        }
-        if (
-            ! (
-                isset($options_array['subsections']['events'])
-                && $options_array['subsections']['events'] instanceof EE_Form_Section_Proper
-            )
-        ) {
-            $options_array['subsections']['events'] = new EE_Form_Section_Proper();
-        }
-        $events_subsection = $options_array['subsections']['events'];
-        $events = EEM_Event::instance()->get_all_deleted_and_undeleted(
-            [
-                [
-                    'EVT_ID' => ['IN',$event_ids]
-                ]
-            ]
-        );
-        if (! is_array($events)) {
-            throw new UnexpectedEntityException($event_ids, 'array');
-        }
-        $events_inputs = [];
-        foreach ($events as $event) {
-            if ($event instanceof EE_Event) {
-                $events_inputs[ $event->ID() ] = new EE_Checkbox_Multi_Input(
-                    [
-                        'yes' => $event->name(),
-                    ],
-                    [
-                        'html_label_text' => esc_html__('Please confirm you wish to delete:', 'event_espresso'),
-                        'required'        => true
-                    ]
-                );
-            }
-        }
-        $events_subsection->add_subsections($events_inputs);
-        $options_array['subsections']['backup'] = new EE_Checkbox_Multi_Input(
-            [
-                'yes' => esc_html__('I have backed up my database.', 'event_espresso')
-            ],
-            [
-                'html_label_text' => esc_html__('Deleting this data cannot be undone. Please confirm you have a usable database backup.', 'event_espresso'),
-                'required' => true
-            ]
-        );
-        $events_list_url = EE_Admin_Page::add_query_args_and_nonce(
-            [
-                'status' => 'trash',
-                'return' => 'default',
-            ],
-            EVENTS_ADMIN_URL
-        );
-        $options_array['subsections']['buttons'] = new EE_Form_Section_HTML(
-            "
+	/**
+	 * @throws ReflectionException
+	 * @throws EE_Error
+	 */
+	public function __construct($event_ids, $options_array = array())
+	{
+		if (! isset($options_array['subsections'])) {
+			$options_array['subsections'] = [];
+		}
+		if (
+			! (
+				isset($options_array['subsections']['events'])
+				&& $options_array['subsections']['events'] instanceof EE_Form_Section_Proper
+			)
+		) {
+			$options_array['subsections']['events'] = new EE_Form_Section_Proper();
+		}
+		$events_subsection = $options_array['subsections']['events'];
+		$events = EEM_Event::instance()->get_all_deleted_and_undeleted(
+			[
+				[
+					'EVT_ID' => ['IN',$event_ids]
+				]
+			]
+		);
+		if (! is_array($events)) {
+			throw new UnexpectedEntityException($event_ids, 'array');
+		}
+		$events_inputs = [];
+		foreach ($events as $event) {
+			if ($event instanceof EE_Event) {
+				$events_inputs[ $event->ID() ] = new EE_Checkbox_Multi_Input(
+					[
+						'yes' => $event->name(),
+					],
+					[
+						'html_label_text' => esc_html__('Please confirm you wish to delete:', 'event_espresso'),
+						'required'        => true
+					]
+				);
+			}
+		}
+		$events_subsection->add_subsections($events_inputs);
+		$options_array['subsections']['backup'] = new EE_Checkbox_Multi_Input(
+			[
+				'yes' => esc_html__('I have backed up my database.', 'event_espresso')
+			],
+			[
+				'html_label_text' => esc_html__('Deleting this data cannot be undone. Please confirm you have a usable database backup.', 'event_espresso'),
+				'required' => true
+			]
+		);
+		$events_list_url = EE_Admin_Page::add_query_args_and_nonce(
+			[
+				'status' => 'trash',
+				'return' => 'default',
+			],
+			EVENTS_ADMIN_URL
+		);
+		$options_array['subsections']['buttons'] = new EE_Form_Section_HTML(
+			"
         <div class='ee-admin-button-row ee-admin-button-row--align-start'>
             <input type='submit' 
                    class='button button--primary' 
@@ -95,9 +95,9 @@ 
                 " . esc_html__('Cancel', 'event_espresso') . "
             </a>
         </div>"
-        );
-        parent::__construct($options_array);
-    }
+		);
+		parent::__construct($options_array);
+	}
 }
 // End of file ConfirmEventDeletionForm.php
 // Location: EventEspresso\admin_pages\events\form_sections/ConfirmEventDeletionForm.php

Spacing +7 added lines, -7 removed lines

@@ -31,7 +31,7 @@ 
      */
     public function __construct($event_ids, $options_array = array())
     {
-        if (! isset($options_array['subsections'])) {
+        if ( ! isset($options_array['subsections'])) {
             $options_array['subsections'] = [];
         }
         if (
@@ -46,17 +46,17 @@ 
         $events = EEM_Event::instance()->get_all_deleted_and_undeleted(
             [
                 [
-                    'EVT_ID' => ['IN',$event_ids]
+                    'EVT_ID' => ['IN', $event_ids]
                 ]
             ]
         );
-        if (! is_array($events)) {
+        if ( ! is_array($events)) {
             throw new UnexpectedEntityException($event_ids, 'array');
         }
         $events_inputs = [];
         foreach ($events as $event) {
             if ($event instanceof EE_Event) {
-                $events_inputs[ $event->ID() ] = new EE_Checkbox_Multi_Input(
+                $events_inputs[$event->ID()] = new EE_Checkbox_Multi_Input(
                     [
                         'yes' => $event->name(),
                     ],
@@ -89,10 +89,10 @@ 
         <div class='ee-admin-button-row ee-admin-button-row--align-start'>
             <input type='submit' 
                    class='button button--primary' 
-                   value='" . esc_attr__('Confirm', 'event_espresso') . "'
+                   value='" . esc_attr__('Confirm', 'event_espresso')."'
             />
-            <a href='" . esc_url_raw($events_list_url) . ";' class='button button--secondary'>
-                " . esc_html__('Cancel', 'event_espresso') . "
+            <a href='" . esc_url_raw($events_list_url).";' class='button button--secondary'>
+                " . esc_html__('Cancel', 'event_espresso')."
             </a>
         </div>"
         );

admin_pages/events/templates/event_preview_deletion.template.php

Indentation +47 added lines, -47 removed lines

@@ -18,101 +18,101 @@
 </h2>
 <h3>
     <?php echo esc_html(
-        sprintf(
-        // translators: 1: number of events
-            _n('%1$d Event', '%1$d Events', count($events), 'event_espresso'),
-            count($events)
-        )
-    );
+		sprintf(
+		// translators: 1: number of events
+			_n('%1$d Event', '%1$d Events', count($events), 'event_espresso'),
+			count($events)
+		)
+	);
 ?>
 </h3>
 <ul>
     <?php
-    foreach ($events as $event) {
-        ?>
+	foreach ($events as $event) {
+		?>
         <li>
             <?php echo esc_html($event->name()); ?>
         </li>
         <?php
-    }
-    ?>
+	}
+	?>
 </ul>
 <h3>
     <?php echo esc_html(
-        sprintf(
-        // translators: 1: number of datetimes
-            _n('%1$d Datetime', '%1$d Datetimes', count($datetimes), 'event_espresso'),
-            count($datetimes)
-        )
-    );
+		sprintf(
+		// translators: 1: number of datetimes
+			_n('%1$d Datetime', '%1$d Datetimes', count($datetimes), 'event_espresso'),
+			count($datetimes)
+		)
+	);
 ?>
 </h3>
 <ul>
     <?php
-    foreach ($datetimes as $datetime) {
-        ?>
+	foreach ($datetimes as $datetime) {
+		?>
         <li>
             <?php echo esc_html($datetime->get_dtt_display_name(true)); ?>
         </li>
         <?php
-    }
-    ?>
+	}
+	?>
 </ul>
 <h3>
     <?php echo esc_html(
-        sprintf(
-            _n('%1$d Registration', '%1$d Registrations', $reg_count, 'event_espresso'),
-            $reg_count
-        )
-    );
+		sprintf(
+			_n('%1$d Registration', '%1$d Registrations', $reg_count, 'event_espresso'),
+			$reg_count
+		)
+	);
 ?>
 </h3>
 <?php
 if ($reg_count > count($registrations)) {
-    ?>
+	?>
     <p class="notice">
         <?php printf(
-            esc_html__('Only showing first %1$d.', 'event_espresso'),
-            count($registrations)
-        );
-        ?>
+			esc_html__('Only showing first %1$d.', 'event_espresso'),
+			count($registrations)
+		);
+		?>
     </p>
     <?php
 }
 ?>
 <?php
 if ($reg_count > 0) {
-    ?>
+	?>
     <p>
         <?php esc_html_e(
-            'Note: contacts will not be deleted, only their registrations for the enumerated events.',
-            'event_espresso'
-        ); ?>
+			'Note: contacts will not be deleted, only their registrations for the enumerated events.',
+			'event_espresso'
+		); ?>
     </p>
     <?php
 }
 ?>
 <ul>
     <?php
-    foreach ($registrations as $registration) {
-        ?>
+	foreach ($registrations as $registration) {
+		?>
         <li>
             <?php echo esc_html(
-                sprintf(
-                    _x('%1$s (%2$d of %3$d)', 'Registration name (number of count)', 'event_espresso'),
-                    $registration->attendeeName(true),
-                    $registration->count(),
-                    $registration->group_size()
-                )
-            ); ?>
+				sprintf(
+					_x('%1$s (%2$d of %3$d)', 'Registration name (number of count)', 'event_espresso'),
+					$registration->attendeeName(true),
+					$registration->count(),
+					$registration->group_size()
+				)
+			); ?>
         </li>
         <?php
-    }
-    ?>
+	}
+	?>
 </ul>
 <?php
 
 echo wp_kses(
-    $form->form_open($form_url) . $form->get_html_and_js() . $form->form_close(),
-    AllowedTags::getWithFormTags()
+	$form->form_open($form_url) . $form->get_html_and_js() . $form->form_close(),
+	AllowedTags::getWithFormTags()
 );

Spacing +1 added lines, -1 removed lines

@@ -113,6 +113,6 @@
 <?php
 
 echo wp_kses(
-    $form->form_open($form_url) . $form->get_html_and_js() . $form->form_close(),
+    $form->form_open($form_url).$form->get_html_and_js().$form->form_close(),
     AllowedTags::getWithFormTags()
 );

caffeinated/modules/events_archive_caff/templates/admin-event-list-settings.template.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
             </label>
         </th>
         <td>
-            <span class="base-url"><?php echo site_url() . '/ ';?></span>
+            <span class="base-url"><?php echo site_url().'/ '; ?></span>
             <?php echo EEH_Form_Fields::text(
                 'not_used',
                 EE_Registry::instance()->CFG->core->event_cpt_slug,

Indentation +174 added lines, -174 removed lines

@@ -23,18 +23,18 @@ 
 add_filter('FHEE__EEH_Form_Fields__label_html', '__return_empty_string');
 
 $values = EEH_Form_Fields::prep_answer_options(
-    array(
-        array('id' => 1, 'text' => esc_html__('Yes', 'event_espresso')),
-        array('id' => 0, 'text' => esc_html__('No', 'event_espresso')),
-    )
+	array(
+		array('id' => 1, 'text' => esc_html__('Yes', 'event_espresso')),
+		array('id' => 0, 'text' => esc_html__('No', 'event_espresso')),
+	)
 );
 
 $description = EEH_Form_Fields::prep_answer_options(
-    array(
-        array('id' => 0, 'text' => esc_html__('none', 'event_espresso')),
-        array('id' => 1, 'text' => esc_html__('excerpt (short desc)', 'event_espresso')),
-        array('id' => 2, 'text' => esc_html__('full description', 'event_espresso')),
-    )
+	array(
+		array('id' => 0, 'text' => esc_html__('none', 'event_espresso')),
+		array('id' => 1, 'text' => esc_html__('excerpt (short desc)', 'event_espresso')),
+		array('id' => 2, 'text' => esc_html__('full description', 'event_espresso')),
+	)
 );
 
 ?>
@@ -53,8 +53,8 @@ 
         <th>
             <label for="event_listings_url">
                 <?php esc_html_e('Events List URL', 'event_espresso'); ?><?php echo EEH_Template::get_help_tab_link(
-                    'event_listings_url_info'
-                ); ?>
+					'event_listings_url_info'
+				); ?>
             </label>
         </th>
         <td>
@@ -63,10 +63,10 @@ 
                 <span class="dashicons dashicons-external"></span>
             </a>
             <p class="description"><?php
-                esc_html_e(
-                    'This is the URL where all events will be displayed.  You can change the slug below.',
-                    'event_espresso'
-                ); ?></p>
+				esc_html_e(
+					'This is the URL where all events will be displayed.  You can change the slug below.',
+					'event_espresso'
+				); ?></p>
         </td>
     </tr>
 
@@ -79,29 +79,29 @@ 
         <td>
             <span class="base-url"><?php echo site_url() . '/ ';?></span>
             <?php echo EEH_Form_Fields::text(
-                'not_used',
-                EE_Registry::instance()->CFG->core->event_cpt_slug,
-                'event_cpt_slug',
-                'event_cpt_slug',
-                'regular'
-            ); ?>
+				'not_used',
+				EE_Registry::instance()->CFG->core->event_cpt_slug,
+				'event_cpt_slug',
+				'event_cpt_slug',
+				'regular'
+			); ?>
             <p class="description"><?php
-                esc_html_e(
-                    'This allows you to configure what slug is used for the url of all event pages.',
-                    'event_espresso'
-                ); ?></p>
+				esc_html_e(
+					'This allows you to configure what slug is used for the url of all event pages.',
+					'event_espresso'
+				); ?></p>
             <?php if (has_filter('FHEE__EE_Register_CPTs__register_CPT__rewrite')) : ?>
                 <p class="important-notice">
                     <?php
-                    sprintf(
-                        esc_html__(
-                            'Usage of the %1$s FHEE__EE_Register_CPTs__register_CPT__rewrite %2$s filter has been detected.  Please be aware that while this filter is being used, this setting has no affect.',
-                            'event_espresso'
-                        ),
-                        '<code>',
-                        '</code>'
-                    );
-                    ?>
+					sprintf(
+						esc_html__(
+							'Usage of the %1$s FHEE__EE_Register_CPTs__register_CPT__rewrite %2$s filter has been detected.  Please be aware that while this filter is being used, this setting has no affect.',
+							'event_espresso'
+						),
+						'<code>',
+						'</code>'
+					);
+					?>
                 </p>
             <?php endif; ?>
         </td>
@@ -115,20 +115,20 @@ 
         </th>
         <td>
             <?php echo wp_kses(
-                EEH_Form_Fields::select(
-                    'display_status_banner',
-                    (int) $display_status_banner,
-                    $values,
-                    'EED_Events_Archive_display_status_banner',
-                    'EED_Events_Archive_display_status_banner'
-                ),
-                AllowedTags::getWithFormTags()
-            ); ?>
+				EEH_Form_Fields::select(
+					'display_status_banner',
+					(int) $display_status_banner,
+					$values,
+					'EED_Events_Archive_display_status_banner',
+					'EED_Events_Archive_display_status_banner'
+				),
+				AllowedTags::getWithFormTags()
+			); ?>
             <p class="description"><?php
-                esc_html_e(
-                    'Whether the status banner should be shown on the events list page next to the Event title.  The status banner is the banner that shows the status of the event (e.g. "Upcoming", "Expired", etc.).',
-                    'event_espresso'
-                ); ?></p>
+				esc_html_e(
+					'Whether the status banner should be shown on the events list page next to the Event title.  The status banner is the banner that shows the status of the event (e.g. "Upcoming", "Expired", etc.).',
+					'event_espresso'
+				); ?></p>
         </td>
     </tr>
 
@@ -136,26 +136,26 @@ 
         <th>
             <label for="EED_Events_Archive_display_description">
                 <?php esc_html_e('Display Description', 'event_espresso'); ?><?php echo EEH_Template::get_help_tab_link(
-                    'display_description_info'
-                ); ?>
+					'display_description_info'
+				); ?>
             </label>
         </th>
         <td>
             <?php echo wp_kses(
-                EEH_Form_Fields::select(
-                    'description',
-                    (int) $display_description,
-                    $description,
-                    'EED_Events_Archive_display_description',
-                    'EED_Events_Archive_display_description'
-                ),
-                AllowedTags::getWithFormTags()
-            ); ?>
+				EEH_Form_Fields::select(
+					'description',
+					(int) $display_description,
+					$description,
+					'EED_Events_Archive_display_description',
+					'EED_Events_Archive_display_description'
+				),
+				AllowedTags::getWithFormTags()
+			); ?>
             <p class="description"><?php
-                esc_html_e(
-                    'Whether the event description should be shown on the events list page.  If you choose "none" then the description will not be shown.  If you choose "excerpt" then the short description will be shown.  If you choose "full description" then the full description will be shown.',
-                    'event_espresso'
-                ); ?></p>
+				esc_html_e(
+					'Whether the event description should be shown on the events list page.  If you choose "none" then the description will not be shown.  If you choose "excerpt" then the short description will be shown.  If you choose "full description" then the full description will be shown.',
+					'event_espresso'
+				); ?></p>
         </td>
     </tr>
 
@@ -163,26 +163,26 @@ 
         <th>
             <label for="EED_Events_Archive_display_ticket_selector">
                 <?php esc_html_e('Display Ticket Selector', 'event_espresso'); ?><?php echo EEH_Template::get_help_tab_link(
-                    'display_ticket_selector_info'
-                ); ?>
+					'display_ticket_selector_info'
+				); ?>
             </label>
         </th>
         <td>
             <?php echo wp_kses(
-                EEH_Form_Fields::select(
-                    'ticket_selector',
-                    (int) $display_ticket_selector,
-                    $values,
-                    'EED_Events_Archive_display_ticket_selector',
-                    'EED_Events_Archive_display_ticket_selector'
-                ),
-                AllowedTags::getWithFormTags()
-            ); ?>
+				EEH_Form_Fields::select(
+					'ticket_selector',
+					(int) $display_ticket_selector,
+					$values,
+					'EED_Events_Archive_display_ticket_selector',
+					'EED_Events_Archive_display_ticket_selector'
+				),
+				AllowedTags::getWithFormTags()
+			); ?>
             <p class="description"><?php
-                esc_html_e(
-                    'Whether the ticket selector should be shown on the events list page.',
-                    'event_espresso'
-                ); ?></p>
+				esc_html_e(
+					'Whether the ticket selector should be shown on the events list page.',
+					'event_espresso'
+				); ?></p>
         </td>
     </tr>
 
@@ -190,26 +190,26 @@ 
         <th>
             <label for="EED_Events_Archive_display_datetimes">
                 <?php esc_html_e('Display Datetimes', 'event_espresso'); ?><?php echo EEH_Template::get_help_tab_link(
-                    'display_datetimes_info'
-                ); ?>
+					'display_datetimes_info'
+				); ?>
             </label>
         </th>
         <td>
             <?php echo wp_kses(
-                EEH_Form_Fields::select(
-                    'venue_details',
-                    (int) $display_datetimes,
-                    $values,
-                    'EED_Events_Archive_display_datetimes',
-                    'EED_Events_Archive_display_datetimes'
-                ),
-                AllowedTags::getWithFormTags()
-            ); ?>
+				EEH_Form_Fields::select(
+					'venue_details',
+					(int) $display_datetimes,
+					$values,
+					'EED_Events_Archive_display_datetimes',
+					'EED_Events_Archive_display_datetimes'
+				),
+				AllowedTags::getWithFormTags()
+			); ?>
             <p class="description"><?php
-                esc_html_e(
-                    'Whether the date and time details should be shown on the events list page.',
-                    'event_espresso'
-                ); ?></p>
+				esc_html_e(
+					'Whether the date and time details should be shown on the events list page.',
+					'event_espresso'
+				); ?></p>
         </td>
     </tr>
 
@@ -217,26 +217,26 @@ 
         <th>
             <label for="EED_Events_Archive_display_venue">
                 <?php esc_html_e('Display Venue Details', 'event_espresso'); ?><?php echo EEH_Template::get_help_tab_link(
-                    'display_venue_details_info'
-                ); ?>
+					'display_venue_details_info'
+				); ?>
             </label>
         </th>
         <td>
             <?php echo wp_kses(
-                EEH_Form_Fields::select(
-                    'display_venue',
-                    (int) $display_venue,
-                    $values,
-                    'EED_Events_Archive_display_venue',
-                    'EED_Events_Archive_display_venue'
-                ),
-                AllowedTags::getWithFormTags()
-            ); ?>
+				EEH_Form_Fields::select(
+					'display_venue',
+					(int) $display_venue,
+					$values,
+					'EED_Events_Archive_display_venue',
+					'EED_Events_Archive_display_venue'
+				),
+				AllowedTags::getWithFormTags()
+			); ?>
             <p class="description"><?php
-                esc_html_e(
-                    'Whether the venue details should be shown on the events list page.',
-                    'event_espresso'
-                ); ?></p>
+				esc_html_e(
+					'Whether the venue details should be shown on the events list page.',
+					'event_espresso'
+				); ?></p>
         </td>
     </tr>
 
@@ -244,26 +244,26 @@ 
         <th>
             <label for="EED_Events_Archive_display_expired_events">
                 <?php esc_html_e('Display Expired Events', 'event_espresso'); ?><?php echo EEH_Template::get_help_tab_link(
-                    'display_expired_events_info'
-                ); ?>
+					'display_expired_events_info'
+				); ?>
             </label>
         </th>
         <td>
             <?php echo wp_kses(
-                EEH_Form_Fields::select(
-                    'expired_events',
-                    (int) $display_expired_events,
-                    $values,
-                    'EED_Events_Archive_display_expired_events',
-                    'EED_Events_Archive_display_expired_events'
-                ),
-                AllowedTags::getWithFormTags()
-            ); ?>
+				EEH_Form_Fields::select(
+					'expired_events',
+					(int) $display_expired_events,
+					$values,
+					'EED_Events_Archive_display_expired_events',
+					'EED_Events_Archive_display_expired_events'
+				),
+				AllowedTags::getWithFormTags()
+			); ?>
             <p class="description"><?php
-                esc_html_e(
-                    'Whether expired events should be shown on the events list page.',
-                    'event_espresso'
-                ); ?></p>
+				esc_html_e(
+					'Whether expired events should be shown on the events list page.',
+					'event_espresso'
+				); ?></p>
         </td>
     </tr>
 
@@ -275,20 +275,20 @@ 
         </th>
         <td>
             <?php echo wp_kses(
-                EEH_Form_Fields::select(
-                    'expired_tickets',
-                    (int) $display_events_with_expired_tickets,
-                    $values,
-                    'EED_Events_Archive_display_events_with_expired_tickets',
-                    'EED_Events_Archive_display_events_with_expired_tickets'
-                ),
-                AllowedTags::getWithFormTags()
-            ); ?>
+				EEH_Form_Fields::select(
+					'expired_tickets',
+					(int) $display_events_with_expired_tickets,
+					$values,
+					'EED_Events_Archive_display_events_with_expired_tickets',
+					'EED_Events_Archive_display_events_with_expired_tickets'
+				),
+				AllowedTags::getWithFormTags()
+			); ?>
             <p class="description">
                 <?php esc_html_e(
-                    'Whether events where ALL tickets are expired should be shown on the events list page.',
-                    'event_espresso'
-                ); ?>
+					'Whether events where ALL tickets are expired should be shown on the events list page.',
+					'event_espresso'
+				); ?>
             </p>
             <span class="ee-feature-highlight-2024-notice">
                 ✨ <?php esc_html_e('NEW', 'event_espresso'); ?>
@@ -300,51 +300,51 @@ 
         <th>
             <label for="EED_Events_Archive_use_sortable_display_order">
                 <?php esc_html_e(
-                    'Use Custom Display Order?',
-                    'event_espresso'
-                ); ?><?php // echo EEH_Template::get_help_tab_link('use_sortable_display_order_info');?>
+					'Use Custom Display Order?',
+					'event_espresso'
+				); ?><?php // echo EEH_Template::get_help_tab_link('use_sortable_display_order_info');?>
             </label>
         </th>
         <td>
             <?php echo wp_kses(
-                EEH_Form_Fields::select(
-                    'use_sortable_display_order',
-                    (int) $use_sortable_display_order,
-                    $values,
-                    'EED_Events_Archive_use_sortable_display_order',
-                    'EED_Events_Archive_use_sortable_display_order'
-                ),
-                AllowedTags::getWithFormTags()
-            ); ?>
+				EEH_Form_Fields::select(
+					'use_sortable_display_order',
+					(int) $use_sortable_display_order,
+					$values,
+					'EED_Events_Archive_use_sortable_display_order',
+					'EED_Events_Archive_use_sortable_display_order'
+				),
+				AllowedTags::getWithFormTags()
+			); ?>
             <p class="description"><?php
-                esc_html_e(
-                    'Selecting "Yes" will allow you to drag and drop the order of the Event Description, Date and Times, Ticket Selector, and Venue Information on the event archive page.',
-                    'event_espresso'
-                ); ?></p>
+				esc_html_e(
+					'Selecting "Yes" will allow you to drag and drop the order of the Event Description, Date and Times, Ticket Selector, and Venue Information on the event archive page.',
+					'event_espresso'
+				); ?></p>
         </td>
     </tr>
 
     <tr>
         <th>
             <?php esc_html_e(
-                'Display Order',
-                'event_espresso'
-            ); ?><?php // echo EEH_Template::get_help_tab_link( 'event_archive_order_info' ); ?>
+				'Display Order',
+				'event_espresso'
+			); ?><?php // echo EEH_Template::get_help_tab_link( 'event_archive_order_info' ); ?>
         </th>
         <td>
 
             <?php wp_nonce_field(
-                'espresso_update_event_archive_order',
-                'espresso_update_event_archive_order_nonce',
-                false
-            ); ?>
+				'espresso_update_event_archive_order',
+				'espresso_update_event_archive_order_nonce',
+				false
+			); ?>
             <?php echo wp_kses($event_archive_display_order, AllowedTags::getWithFormTags()); ?>
 
             <p class="description"><?php
-                esc_html_e(
-                    'Drag and Drop the above to determine the display order of the Event Description, Date and Times, Ticket Selector, and Venue Information on the event archive page.',
-                    'event_espresso'
-                ); ?></p>
+				esc_html_e(
+					'Drag and Drop the above to determine the display order of the Event Description, Date and Times, Ticket Selector, and Venue Information on the event archive page.',
+					'event_espresso'
+				); ?></p>
 
         </td>
     </tr>
@@ -357,20 +357,20 @@ 
         </th>
         <td>
             <?php echo wp_kses(
-                EEH_Form_Fields::select(
-                    'reset_event_list_settings',
-                    0,
-                    $values,
-                    'EED_Events_Archive_reset_event_list_settings',
-                    'EED_Events_Archive_reset_event_list_settings'
-                ),
-                AllowedTags::getWithFormTags()
-            ); ?>
+				EEH_Form_Fields::select(
+					'reset_event_list_settings',
+					0,
+					$values,
+					'EED_Events_Archive_reset_event_list_settings',
+					'EED_Events_Archive_reset_event_list_settings'
+				),
+				AllowedTags::getWithFormTags()
+			); ?>
             <p class="description"><?php
-                esc_html_e(
-                    'Selecting "Yes" will reset the Event List Settings to their default values.',
-                    'event_espresso'
-                ); ?></p>
+				esc_html_e(
+					'Selecting "Yes" will reset the Event List Settings to their default values.',
+					'event_espresso'
+				); ?></p>
         </td>
     </tr>
 

core/domain/services/commands/transaction/CreateTransactionCommandHandler.php

Indentation +43 added lines, -43 removed lines

@@ -25,49 +25,49 @@
  */
 class CreateTransactionCommandHandler extends CommandHandler
 {
-    public function __construct()
-    {
-        defined('EVENT_ESPRESSO_VERSION') || exit;
-    }
+	public function __construct()
+	{
+		defined('EVENT_ESPRESSO_VERSION') || exit;
+	}
 
 
-    /**
-     * @param CommandInterface|CreateTransactionCommand $command
-     * @return mixed
-     * @throws EE_Error
-     * @throws InvalidEntityException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     * @throws InvalidArgumentException
-     * @throws ReflectionException
-     * @throws RuntimeException
-     */
-    public function handle(CommandInterface $command)
-    {
-        $transaction_details = $command->transactionDetails();
-        $cart_total          = null;
-        if ($command->checkout() instanceof EE_Checkout) {
-            // ensure cart totals have been calculated
-            $command->checkout()->cart->get_grand_total()->recalculate_total_including_taxes();
-            // grab the cart grand total
-            $cart_total                           = $command->checkout()->cart->get_cart_grand_total();
-            $transaction_details['TXN_reg_steps'] = $command->checkout()->initialize_txn_reg_steps_array();
-            $transaction_details['TXN_total']     = $cart_total > 0 ? $cart_total : 0;
-        }
-        // create new TXN and save it so it has an ID
-        $transaction = EE_Transaction::new_instance($transaction_details);
-        if (! $transaction instanceof EE_Transaction) {
-            throw new InvalidEntityException(get_class($transaction), 'EE_Transaction');
-        }
-        $transaction->save();
-        // ensure grand total line item created
-        $cart_total = $cart_total instanceof EE_Line_Item
-            ? $cart_total
-            : EEH_Line_Item::create_total_line_item($transaction);
-        if (! $cart_total instanceof EE_Line_Item) {
-            throw new InvalidEntityException(get_class($cart_total), 'EE_Line_Item');
-        }
-        $cart_total->save_this_and_descendants_to_txn($transaction->ID());
-        return $transaction;
-    }
+	/**
+	 * @param CommandInterface|CreateTransactionCommand $command
+	 * @return mixed
+	 * @throws EE_Error
+	 * @throws InvalidEntityException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidArgumentException
+	 * @throws ReflectionException
+	 * @throws RuntimeException
+	 */
+	public function handle(CommandInterface $command)
+	{
+		$transaction_details = $command->transactionDetails();
+		$cart_total          = null;
+		if ($command->checkout() instanceof EE_Checkout) {
+			// ensure cart totals have been calculated
+			$command->checkout()->cart->get_grand_total()->recalculate_total_including_taxes();
+			// grab the cart grand total
+			$cart_total                           = $command->checkout()->cart->get_cart_grand_total();
+			$transaction_details['TXN_reg_steps'] = $command->checkout()->initialize_txn_reg_steps_array();
+			$transaction_details['TXN_total']     = $cart_total > 0 ? $cart_total : 0;
+		}
+		// create new TXN and save it so it has an ID
+		$transaction = EE_Transaction::new_instance($transaction_details);
+		if (! $transaction instanceof EE_Transaction) {
+			throw new InvalidEntityException(get_class($transaction), 'EE_Transaction');
+		}
+		$transaction->save();
+		// ensure grand total line item created
+		$cart_total = $cart_total instanceof EE_Line_Item
+			? $cart_total
+			: EEH_Line_Item::create_total_line_item($transaction);
+		if (! $cart_total instanceof EE_Line_Item) {
+			throw new InvalidEntityException(get_class($cart_total), 'EE_Line_Item');
+		}
+		$cart_total->save_this_and_descendants_to_txn($transaction->ID());
+		return $transaction;
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
         }
         // create new TXN and save it so it has an ID
         $transaction = EE_Transaction::new_instance($transaction_details);
-        if (! $transaction instanceof EE_Transaction) {
+        if ( ! $transaction instanceof EE_Transaction) {
             throw new InvalidEntityException(get_class($transaction), 'EE_Transaction');
         }
         $transaction->save();
@@ -59,7 +59,7 @@ 
         $cart_total = $cart_total instanceof EE_Line_Item
             ? $cart_total
             : EEH_Line_Item::create_total_line_item($transaction);
-        if (! $cart_total instanceof EE_Line_Item) {
+        if ( ! $cart_total instanceof EE_Line_Item) {
             throw new InvalidEntityException(get_class($cart_total), 'EE_Line_Item');
         }
         $cart_total->save_this_and_descendants_to_txn($transaction->ID());

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

Indentation +28 added lines, -28 removed lines

@@ -21,36 +21,36 @@
  */
 class CopyRegistrationPaymentsCommandHandler extends CommandHandler
 {
-    /**
-     * @var CopyRegistrationService $copy_registration_service
-     */
-    private $copy_registration_service;
+	/**
+	 * @var CopyRegistrationService $copy_registration_service
+	 */
+	private $copy_registration_service;
 
 
-    /**
-     * Command constructor
-     *
-     * @param CopyRegistrationService $copy_registration_service
-     */
-    public function __construct(CopyRegistrationService $copy_registration_service)
-    {
-        defined('EVENT_ESPRESSO_VERSION') || exit;
-        $this->copy_registration_service = $copy_registration_service;
-    }
+	/**
+	 * Command constructor
+	 *
+	 * @param CopyRegistrationService $copy_registration_service
+	 */
+	public function __construct(CopyRegistrationService $copy_registration_service)
+	{
+		defined('EVENT_ESPRESSO_VERSION') || exit;
+		$this->copy_registration_service = $copy_registration_service;
+	}
 
 
-    /**
-     * @param CommandInterface|CopyRegistrationPaymentsCommand $command
-     * @return boolean
-     * @throws EE_Error
-     * @throws UnexpectedEntityException
-     * @throws RuntimeException
-     */
-    public function handle(CommandInterface $command)
-    {
-        return $this->copy_registration_service->copyPaymentDetails(
-            $command->targetRegistration(),
-            $command->registrationToCopy()
-        );
-    }
+	/**
+	 * @param CommandInterface|CopyRegistrationPaymentsCommand $command
+	 * @return boolean
+	 * @throws EE_Error
+	 * @throws UnexpectedEntityException
+	 * @throws RuntimeException
+	 */
+	public function handle(CommandInterface $command)
+	{
+		return $this->copy_registration_service->copyPaymentDetails(
+			$command->targetRegistration(),
+			$command->registrationToCopy()
+		);
+	}
 }

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

Indentation +42 added lines, -42 removed lines

@@ -15,47 +15,47 @@
  */
 class CopyRegistrationDetailsCommand extends Command
 {
-    /**
-     * @var EE_Registration $target_registration
-     */
-    private $target_registration;
-
-    /**
-     * @var EE_Registration $registration_to_copy
-     */
-    private $registration_to_copy;
-
-
-    /**
-     * CopyRegistrationDetailsCommand constructor.
-     *
-     * @param EE_Registration $target_registration
-     * @param EE_Registration $registration_to_copy
+	/**
+	 * @var EE_Registration $target_registration
+	 */
+	private $target_registration;
+
+	/**
+	 * @var EE_Registration $registration_to_copy
+	 */
+	private $registration_to_copy;
+
+
+	/**
+	 * CopyRegistrationDetailsCommand constructor.
+	 *
+	 * @param EE_Registration $target_registration
+	 * @param EE_Registration $registration_to_copy
     v
-     */
-    public function __construct(
-        EE_Registration $target_registration,
-        EE_Registration $registration_to_copy
-    ) {
-        $this->target_registration  = $target_registration;
-        $this->registration_to_copy = $registration_to_copy;
-    }
-
-
-    /**
-     * @return EE_Registration
-     */
-    public function targetRegistration()
-    {
-        return $this->target_registration;
-    }
-
-
-    /**
-     * @return EE_Registration
-     */
-    public function registrationToCopy()
-    {
-        return $this->registration_to_copy;
-    }
+	 */
+	public function __construct(
+		EE_Registration $target_registration,
+		EE_Registration $registration_to_copy
+	) {
+		$this->target_registration  = $target_registration;
+		$this->registration_to_copy = $registration_to_copy;
+	}
+
+
+	/**
+	 * @return EE_Registration
+	 */
+	public function targetRegistration()
+	{
+		return $this->target_registration;
+	}
+
+
+	/**
+	 * @return EE_Registration
+	 */
+	public function registrationToCopy()
+	{
+		return $this->registration_to_copy;
+	}
 }