save

Author: tatevikg1

src/Domain/Configuration/Service/Manager/ConfigManager.php

@@ -45,8 +45,6 @@ public function update(Config $config, string $value): void
             throw new ConfigNotEditableException($config->getKey());
         }
         $config->setValue($value);
-
-        $this->configRepository->save($config);
     }
 
     public function create(string $key, string $value, bool $editable, ?string $type = null): void
@@ -57,7 +55,7 @@ public function create(string $key, string $value, bool $editable, ?string $type
             ->setEditable($editable)
             ->setType($type);
 
-        $this->configRepository->save($config);
+        $this->configRepository->persist($config);
     }
 
     public function delete(Config $config): void

src/Domain/Identity/Service/AdminAttributeDefinitionManager.php

@@ -45,7 +45,7 @@ public function create(AdminAttributeDefinitionDto $attributeDefinitionDto): Adm
             ->setDefaultValue($attributeDefinitionDto->defaultValue)
             ->setTableName($attributeDefinitionDto->tableName);
 
-        $this->definitionRepository->save($attributeDefinition);
+        $this->definitionRepository->persist($attributeDefinition);
 
         return $attributeDefinition;
     }
@@ -68,8 +68,6 @@ public function update(
             ->setDefaultValue($attributeDefinitionDto->defaultValue)
             ->setTableName($attributeDefinitionDto->tableName);
 
-        $this->definitionRepository->save($attributeDefinition);
-
         return $attributeDefinition;
     }
 

src/Domain/Identity/Service/AdminAttributeManager.php

@@ -39,7 +39,7 @@ public function createOrUpdate(
         }
 
         $adminAttribute->setValue($value);
-        $this->attributeRepository->save($adminAttribute);
+        $this->attributeRepository->persist($adminAttribute);
 
         return $adminAttribute;
     }

src/Domain/Identity/Service/PasswordManager.php

@@ -65,7 +65,7 @@ public function generatePasswordResetToken(string $email): string
         $expiryDate = new DateTime(self::TOKEN_EXPIRY);
         $passwordRequest = new AdminPasswordRequest(date: $expiryDate, admin: $administrator, keyValue: $token);
 
-        $this->passwordRequestRepository->save($passwordRequest);
+        $this->passwordRequestRepository->persist($passwordRequest);
 
         $message = new PasswordResetMessage(email: $email, token: $token);
         $this->messageBus->dispatch($message);
@@ -113,7 +113,7 @@ public function updatePasswordWithToken(string $token, string $newPassword): boo
 
         $passwordHash = $this->hashGenerator->createPasswordHash($newPassword);
         $administrator->setPasswordHash($passwordHash);
-        $this->administratorRepository->save($administrator);
+        $this->administratorRepository->persist($administrator);
 
         $passwordRequest = $this->passwordRequestRepository->findOneByToken($token);
         $this->passwordRequestRepository->remove($passwordRequest);

src/Domain/Identity/Service/SessionManager.php

@@ -51,7 +51,7 @@ public function createSession(string $loginName, string $password): Administrato
         $token->setAdministrator($administrator);
         $token->generateExpiry();
         $token->generateKey();
-        $this->tokenRepository->save($token);
+        $this->tokenRepository->persist($token);
 
         return $token;
     }

src/Domain/Messaging/Service/Manager/BounceRegexManager.php

@@ -46,8 +46,6 @@ public function createOrUpdateFromPattern(
                 ->setComment($comment ?? $existing->getComment())
                 ->setStatus($status ?? $existing->getStatus());
 
-            $this->bounceRegexRepository->save($existing);
-
             return $existing;
         }
 
@@ -62,7 +60,7 @@ public function createOrUpdateFromPattern(
             count: 0
         );
 
-        $this->bounceRegexRepository->save($bounceRegex);
+        $this->bounceRegexRepository->persist($bounceRegex);
 
         return $bounceRegex;
     }

src/Domain/Messaging/Service/Manager/MessageManager.php

@@ -26,7 +26,7 @@ public function createMessage(MessageDtoInterface $createMessageDto, Administrat
     {
         $context = new MessageContext($authUser);
         $message = $this->messageBuilder->build($createMessageDto, $context);
-        $this->messageRepository->save($message);
+        $this->messageRepository->persist($message);
 
         return $message;
     }
@@ -37,16 +37,12 @@ public function updateMessage(
         Administrator $authUser
     ): Message {
         $context = new MessageContext($authUser, $message);
-        $message = $this->messageBuilder->build($updateMessageDto, $context);
-        $this->messageRepository->save($message);
-
-        return $message;
+        return $this->messageBuilder->build($updateMessageDto, $context);
     }
 
     public function updateStatus(Message $message, Message\MessageStatus $status): Message
     {
         $message->getMetadata()->setStatus($status);
-        $this->messageRepository->save($message);
 
         return $message;
     }

src/Domain/Subscription/Service/Manager/AttributeDefinitionManager.php

@@ -46,7 +46,7 @@ public function create(AttributeDefinitionDto $attributeDefinitionDto): Subscrib
             ->setDefaultValue($attributeDefinitionDto->defaultValue)
             ->setTableName($attributeDefinitionDto->tableName);
 
-        $this->definitionRepository->save($attributeDefinition);
+        $this->definitionRepository->persist($attributeDefinition);
 
         return $attributeDefinition;
     }
@@ -72,8 +72,6 @@ public function update(
             ->setDefaultValue($attributeDefinitionDto->defaultValue)
             ->setTableName($attributeDefinitionDto->tableName);
 
-        $this->definitionRepository->save($attributeDefinition);
-
         return $attributeDefinition;
     }
 

src/Domain/Subscription/Service/Manager/SubscriberManager.php

@@ -48,7 +48,7 @@ public function createSubscriber(CreateSubscriberDto $subscriberDto): Subscriber
         $subscriber->setHtmlEmail((bool)$subscriberDto->htmlEmail);
         $subscriber->setDisabled(false);
 
-        $this->subscriberRepository->save($subscriber);
+        $this->subscriberRepository->persist($subscriber);
 
         return $subscriber;
     }

tests/Unit/Domain/Configuration/Service/Manager/ConfigManagerTest.php

@@ -4,6 +4,7 @@
 
 namespace PhpList\Core\Tests\Unit\Domain\Configuration\Service\Manager;
 
+use PhpList\Core\Domain\Configuration\Exception\ConfigNotEditableException;
 use PhpList\Core\Domain\Configuration\Model\Config;
 use PhpList\Core\Domain\Configuration\Repository\ConfigRepository;
 use PhpList\Core\Domain\Configuration\Service\Manager\ConfigManager;
@@ -61,30 +62,13 @@ public function testGetAllReturnsAllConfigsFromRepository(): void
         $this->assertSame('value2', $result[1]->getValue());
     }
 
-    public function testUpdateSavesConfigToRepository(): void
-    {
-        $configRepository = $this->createMock(ConfigRepository::class);
-        $manager = new ConfigManager($configRepository);
-
-        $config = new Config();
-        $config->setKey('test_item');
-        $config->setValue('test_value');
-        $config->setEditable(true);
-
-        $configRepository->expects($this->once())
-            ->method('save')
-            ->with($config);
-
-        $manager->update($config, 'new_value');
-    }
-
     public function testCreateSavesNewConfigToRepository(): void
     {
         $configRepository = $this->createMock(ConfigRepository::class);
         $manager = new ConfigManager($configRepository);
 
         $configRepository->expects($this->once())
-            ->method('save')
+            ->method('persist')
             ->with($this->callback(function (Config $config) {
                 return $config->getKey() === 'test_key' &&
                     $config->getValue() === 'test_value' &&
@@ -119,10 +103,7 @@ public function testUpdateThrowsExceptionWhenConfigIsNotEditable(): void
         $config->setValue('test_value');
         $config->setEditable(false);
 
-        $configRepository->expects($this->never())
-            ->method('save');
-
-        $this->expectException(\PhpList\Core\Domain\Configuration\Exception\ConfigNotEditableException::class);
+        $this->expectException(ConfigNotEditableException::class);
         $this->expectExceptionMessage('Configuration item "test_item" is not editable.');
 
         $manager->update($config, 'new_value');