From 64eeb5c88e8e75820b497a12e46eaf1b165688aa Mon Sep 17 00:00:00 2001
From: Manuel Aranda <maranda@qindel.com>
Date: Mon, 2 Jun 2025 00:17:42 +0200
Subject: [PATCH] refs #2133. CHanged PXE template

---
 src/EventSubscriber/ClientSubscriber.php           | 14 ++++++++++++--
 .../OrganizationalUnitMulticastPortValidator.php   | 13 +++++++++----
 2 files changed, 21 insertions(+), 6 deletions(-)

diff --git a/src/EventSubscriber/ClientSubscriber.php b/src/EventSubscriber/ClientSubscriber.php
index 7c652c2..ed08088 100644
--- a/src/EventSubscriber/ClientSubscriber.php
+++ b/src/EventSubscriber/ClientSubscriber.php
@@ -4,8 +4,11 @@ namespace App\EventSubscriber;
 
 use ApiPlatform\Symfony\EventListener\EventPriorities;
 use App\Controller\OgBoot\PxeBootFile\PostAction;
+use App\Controller\OgDhcp\Subnet\PutHostAction;
 use App\Dto\Output\ClientOutput;
 use App\Entity\Client;
+use App\Entity\PxeTemplate;
+use Doctrine\ORM\EntityManagerInterface;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpKernel\Event\ViewEvent;
@@ -18,7 +21,9 @@ use Symfony\Contracts\HttpClient\Exception\TransportExceptionInterface;
 final readonly class ClientSubscriber implements EventSubscriberInterface
 {
     public function __construct(
+        private readonly EntityManagerInterface $entityManager,
         private PostAction $postAction,
+        private PutHostAction $putHostAction,
     )
     {
 
@@ -49,14 +54,19 @@ final readonly class ClientSubscriber implements EventSubscriberInterface
         /** @var Client $client */
         $client = $clientOutput->getEntity();
 
-        $template = !$client->getTemplate() ?
+        $template = $client->getTemplate() ?
             $client->getTemplate() :
             $client->getOrganizationalUnit()?->getNetworkSettings()?->getPxeTemplate();
 
         if ($template === null) {
-            return;
+            $template = $this->entityManager->getRepository(PxeTemplate::class)->findOneBy(['isDefault' => true]);
+
+            if ($template === null) {
+                return;
+            }
         }
 
         $this->postAction->__invoke($client, $template);
+        $this->putHostAction->__invoke($client->getMac(), $client);
     }
 }
\ No newline at end of file
diff --git a/src/Validator/Constraints/OrganizationalUnitMulticastPortValidator.php b/src/Validator/Constraints/OrganizationalUnitMulticastPortValidator.php
index 6baabda..38ef7ab 100644
--- a/src/Validator/Constraints/OrganizationalUnitMulticastPortValidator.php
+++ b/src/Validator/Constraints/OrganizationalUnitMulticastPortValidator.php
@@ -9,8 +9,8 @@ use Symfony\Component\Validator\ConstraintValidator;
 
 class OrganizationalUnitMulticastPortValidator extends ConstraintValidator
 {
-    CONST int minPort = 9000;
-    CONST int maxPort = 9050;
+    public const int minPort = 9000;
+    public const int maxPort = 9098;
 
     public function validate($value, Constraint $constraint): void
     {
@@ -18,9 +18,14 @@ class OrganizationalUnitMulticastPortValidator extends ConstraintValidator
             return;
         }
 
-        if (!(self::minPort <= $value) && ($value <= self::maxPort)) {
+        if ($value % 2 !== 0) {
+            $this->context->buildViolation('El puerto debe ser un número par y encontrarse entre el 9000 y el 9098.')->addViolation();
+            return;
+        }
+
+        if ($value < self::minPort || $value > self::maxPort) {
             $this->context->buildViolation($constraint->message)->addViolation();
             return;
         }
     }
-}
\ No newline at end of file
+}