Display trainings on a venue page again

This was a regression introduced in #143 in commit f0fc546 when I've converted included pages to a component and forgot there's no variable anymore. They were right, empty() shouldn't be considered harmless.

site/app/Training/DateList/UpcomingTrainingDatesList.php

@@ -3,13 +3,19 @@
 
 namespace MichalSpacekCz\Training\DateList;
 
+use Countable;
 use MichalSpacekCz\Application\UiControl;
+use MichalSpacekCz\Training\Dates\UpcomingTraining;
 use MichalSpacekCz\Training\Dates\UpcomingTrainingDates;
 use MichalSpacekCz\Training\FreeSeats;
 
-class UpcomingTrainingDatesList extends UiControl
+class UpcomingTrainingDatesList extends UiControl implements Countable
 {
 
+	/** @var array<string, UpcomingTraining>|null */
+	private ?array $upcomingTrainings = null;
+
+
 	public function __construct(
 		private readonly UpcomingTrainingDates $upcomingTrainingDates,
 		private readonly FreeSeats $freeSeats,
@@ -22,13 +28,31 @@ public function __construct(
 
 	public function render(): void
 	{
-		$upcomingTrainings = $this->venueId ? $this->upcomingTrainingDates->getPublicUpcomingAtVenue($this->venueId) : $this->upcomingTrainingDates->getPublicUpcoming();
-		if ($this->excludeTraining) {
-			unset($upcomingTrainings[$this->excludeTraining]);
-		}
+		$upcomingTrainings = $this->getUpcomingTrainingDates();
 		$this->template->lastFreeSeats = $this->showLastFreeSeats && $this->freeSeats->lastFreeSeatsAnyTraining($upcomingTrainings);
 		$this->template->upcomingTrainings = $upcomingTrainings;
 		$this->template->render(__DIR__ . '/upcomingTrainingDatesList.latte');
 	}
 
+
+	/**
+	 * @return array<string, UpcomingTraining>
+	 */
+	private function getUpcomingTrainingDates(): array
+	{
+		if ($this->upcomingTrainings === null) {
+			$this->upcomingTrainings = $this->venueId ? $this->upcomingTrainingDates->getPublicUpcomingAtVenue($this->venueId) : $this->upcomingTrainingDates->getPublicUpcoming();
+			if ($this->excludeTraining) {
+				unset($this->upcomingTrainings[$this->excludeTraining]);
+			}
+		}
+		return $this->upcomingTrainings;
+	}
+
+
+	public function count(): int
+	{
+		return count($this->getUpcomingTrainingDates());
+	}
+
 }

site/app/Www/Presenters/VenuesPresenter.php

@@ -14,6 +14,7 @@ class VenuesPresenter extends BasePresenter
 {
 
 	private int $venueId;
+	private UpcomingTrainingDatesList $upcomingTrainingDatesList;
 
 
 	public function __construct(
@@ -33,15 +34,17 @@ public function actionVenue(string $name): void
 			throw new BadRequestException("Where in the world is {$name}?", previous: $e);
 		}
 		$this->venueId = $venue->getId();
+		$this->upcomingTrainingDatesList = $this->upcomingTrainingDatesListFactory->createForVenue($this->venueId);
 
 		$this->template->pageTitle = $this->texyFormatter->translate('messages.title.venue', [$venue->getName()]);
 		$this->template->venue = $venue;
+		$this->template->hasUpcomingTrainings = count($this->upcomingTrainingDatesList) > 0;
 	}
 
 
 	protected function createComponentUpcomingDatesList(): UpcomingTrainingDatesList
 	{
-		return $this->upcomingTrainingDatesListFactory->createForVenue($this->venueId);
+		return $this->upcomingTrainingDatesList;
 	}
 
 }

site/app/Www/Presenters/templates/Venues/venue.latte

@@ -17,7 +17,7 @@
 </p>
 
 <h3>{_messages.venues.trainingshere}</h3>
-{if empty($upcomingTrainings)}
+{if !$hasUpcomingTrainings}
 <p>{_messages.venues.notraininghere|format:"link:Www:Contact:"}</p>
 {else}
 {control upcomingDatesList}