fix(signup): admin (un)subscribe possible before sign-up list opens

module/Activity/src/Controller/ActivityController.php

@@ -85,7 +85,7 @@
                 [
                     'action' => 'viewSignupList',
                     'id' => $activityId,
                     'signupList' => $activity->getSignupLists()->first()->getId(),
                 ],
             );
         }
@@ -251,7 +251,7 @@
 
             // Ensure that the action is within the subscription window
             if (
-                !$this->signupService->isInSubscriptionWindow($signupList->getOpenDate(), $signupList->getCloseDate())
+                !$signupList->isOpen()
                 || ActivityModel::STATUS_APPROVED !== $signupList->getActivity()->getStatus()
             ) {
                 $error = $this->translator->translate('You cannot subscribe to this activity at this moment in time');
@@ -356,7 +356,7 @@
 
             // Ensure that the action is within the subscription window
             if (
-                !$this->signupService->isInSubscriptionWindow($signupList->getOpenDate(), $signupList->getCloseDate())
+                !$signupList->isOpen()
                 || ActivityModel::STATUS_APPROVED !== $signupList->getActivity()->getStatus()
             ) {
                 $error = $this->translator->translate('You cannot subscribe to this activity at this moment in time');
@@ -417,7 +417,7 @@
 
             // Ensure that the action is within the subscription window
             if (
-                !$this->signupService->isInSubscriptionWindow($signupList->getOpenDate(), $signupList->getCloseDate())
+                !$signupList->isOpen()
                 || ActivityModel::STATUS_APPROVED !== $signupList->getActivity()->getStatus()
             ) {
                 $error = $this->translator->translate(

module/Activity/src/Controller/AdminController.php

@@ -148,7 +148,7 @@
 
         if (0 !== $updateProposal->count()) {
             // If there already is an update proposal for this activity, show that instead of the original activity.
             $activity = $updateProposal->first()->getNew();
         }
 
         $activityData = $activity->toArray();
@@ -305,6 +305,18 @@
             throw new NotAllowedException($this->translator->translate('You are not allowed to use this form'));
         }
 
+        if (
+            !$signupList->isOpen()
+            || ActivityModel::STATUS_APPROVED !== $signupList->getActivity()->getStatus()
+        ) {
+            return $this->redirectActivityAdminRequest(
+                $activityId,
+                $signupListId,
+                false,
+                $this->translator->translate('Cannot subscribe external participant at this point in time'),
+            );
+        }
+
         /** @var Request $request */
         $request = $this->getRequest();
         $activityAdminSession = new SessionContainer('activityAdminRequest');
@@ -400,6 +412,18 @@
             throw new NotAllowedException($this->translator->translate('You are not allowed to use this form'));
         }
 
+        if (
+            !$signupList->isOpen()
+            || ActivityModel::STATUS_APPROVED !== $signupList->getActivity()->getStatus()
+        ) {
+            return $this->redirectActivityAdminRequest(
+                $signupList->getActivity()->getId(),
+                $signupList->getId(),
+                false,
+                $this->translator->translate('Cannot unsubscribe external participant at this point in time'),
+            );
+        }
+
         /** @var Request $request */
         $request = $this->getRequest();
 

module/Activity/src/Service/Signup.php

@@ -14,7 +14,6 @@
 use Activity\Model\SignupList as SignupListModel;
 use Activity\Model\SignupOption as SignupOptionModel;
 use Activity\Model\UserSignup as UserSignupModel;
-use DateTime;
 use Doctrine\ORM\EntityManager;
 use Doctrine\ORM\Exception\ORMException;
 use Doctrine\ORM\OptimisticLockException;
@@ -370,15 +369,6 @@ public function deleteOldSignups(): void
         }
     }
 
-    public static function isInSubscriptionWindow(
-        DateTime $openDate,
-        DateTime $closeDate,
-    ): bool {
-        $currentTime = new DateTime();
-
-        return $openDate < $currentTime && $currentTime < $closeDate;
-    }
-
     /**
      * Is the currently logged-in user allowed to signup.
      */