ISSUE-345: return self after set

Author: tatevikg1

config/repositories.yml

@@ -11,10 +11,10 @@ services:
         arguments:
             - PhpList\Core\Domain\Model\Identity\AdministratorToken
 
-    PhpList\Core\Domain\Repository\Messaging\SubscriberListRepository:
+    PhpList\Core\Domain\Repository\Subscription\SubscriberListRepository:
         parent: PhpList\Core\Domain\Repository
         arguments:
-            - PhpList\Core\Domain\Model\Messaging\SubscriberList
+            - PhpList\Core\Domain\Model\Subscription\SubscriberList
 
     PhpList\Core\Domain\Repository\Subscription\SubscriberRepository:
         parent: PhpList\Core\Domain\Repository

src/Domain/Model/Identity/AdminAttribute.php

@@ -86,33 +86,45 @@ public function getTableName(): ?string
         return $this->tableName;
     }
 
-    public function setName(string $name): void
+    public function setName(string $name): self
     {
         $this->name = $name;
+
+        return $this;
     }
 
-    public function setType(?string $type): void
+    public function setType(?string $type): self
     {
         $this->type = $type;
+
+        return $this;
     }
 
-    public function setListOrder(?int $listOrder): void
+    public function setListOrder(?int $listOrder): self
     {
         $this->listOrder = $listOrder;
+
+        return $this;
     }
 
-    public function setDefaultValue(?string $defaultValue): void
+    public function setDefaultValue(?string $defaultValue): self
     {
         $this->defaultValue = $defaultValue;
+
+        return $this;
     }
 
-    public function setRequired(?bool $required): void
+    public function setRequired(?bool $required): self
     {
         $this->required = $required;
+
+        return $this;
     }
 
-    public function setTableName(?string $tableName): void
+    public function setTableName(?string $tableName): self
     {
         $this->tableName = $tableName;
+
+        return $this;
     }
 }

src/Domain/Model/Identity/AdminAttributeRelation.php

@@ -46,8 +46,10 @@ public function getValue(): ?string
         return $this->value;
     }
 
-    public function setValue(?string $value): void
+    public function setValue(?string $value): self
     {
         $this->value = $value;
+
+        return $this;
     }
 }

src/Domain/Model/Identity/Administrator.php

@@ -20,6 +20,7 @@
  * selected lists (as the owner), send campaigns to these lists and edit subscribers.
  *
  * @author Oliver Klee <[email protected]>
+ * @author Tatevik Grigoryan <[email protected]>
  */
 #[ORM\Entity(repositoryClass: AdministratorRepository::class)]
 #[ORM\Table(name: 'phplist_admin')]
@@ -76,75 +77,91 @@ public function getLoginName(): string
         return $this->loginName;
     }
 
-    public function setLoginName(string $loginName): void
+    public function setLoginName(string $loginName): self
     {
         $this->loginName = $loginName;
+
+        return $this;
     }
 
     public function getEmailAddress(): string
     {
         return $this->emailAddress;
     }
 
-    public function setEmailAddress(string $emailAddress): void
+    public function setEmailAddress(string $emailAddress): self
     {
         $this->emailAddress = $emailAddress;
+
+        return $this;
     }
 
     public function getPasswordHash(): string
     {
         return $this->passwordHash;
     }
 
-    public function setPasswordHash(string $passwordHash): void
+    public function setPasswordHash(string $passwordHash): self
     {
         $this->passwordHash = $passwordHash;
         $this->setPasswordChangeDate(new DateTime());
+
+        return $this;
     }
 
     public function getPasswordChangeDate(): ?DateTime
     {
         return $this->passwordChangeDate;
     }
 
-    private function setPasswordChangeDate(DateTime $changeDate): void
+    private function setPasswordChangeDate(DateTime $changeDate): self
     {
         $this->passwordChangeDate = $changeDate;
+
+        return $this;
     }
 
     public function isDisabled(): bool
     {
         return $this->disabled;
     }
 
-    public function setDisabled(bool $disabled): void
+    public function setDisabled(bool $disabled): self
     {
         $this->disabled = $disabled;
+
+        return $this;
     }
 
     public function isSuperUser(): bool
     {
         return $this->superUser;
     }
 
-    public function setSuperUser(bool $superUser): void
+    public function setSuperUser(bool $superUser): self
     {
         $this->superUser = $superUser;
+
+        return $this;
     }
 
-    public function setNameLc(string $nameLc): void
+    public function setNameLc(string $nameLc): self
     {
         $this->namelc = $nameLc;
+
+        return $this;
     }
 
     public function getNameLc(): string
     {
         return $this->namelc;
     }
 
-    public function setPrivileges(string $privileges): void
+    public function setPrivileges(string $privileges): self
     {
         $this->privileges = $privileges;
+
+        return $this;
     }
 
     public function getPrivileges(): string

src/Domain/Model/Identity/AdministratorToken.php

@@ -63,9 +63,11 @@ public function getCreationDate(): ?DateTime
         return $date;
     }
 
-    private function setCreationDate(DateTime $creationDate): void
+    private function setCreationDate(DateTime $creationDate): self
     {
         $this->creationDate = $creationDate->getTimestamp();
+
+        return $this;
     }
 
     #[ORM\PrePersist]
@@ -79,39 +81,48 @@ public function getExpiry(): DateTime
         return $this->expiry;
     }
 
-    private function setExpiry(DateTime $expiry): void
+    private function setExpiry(DateTime $expiry): self
     {
         $this->expiry = $expiry;
+
+        return $this;
     }
 
-    public function generateExpiry(): void
+    public function generateExpiry(): self
     {
         $this->setExpiry(new DateTime(static::DEFAULT_EXPIRY));
+
+        return $this;
     }
 
     public function getKey(): string
     {
         return $this->key;
     }
 
-    public function setKey(string $key): void
+    public function setKey(string $key): self
     {
         $this->key = $key;
+
+        return $this;
     }
 
-    public function generateKey(): void
+    public function generateKey(): self
     {
         $key = md5(random_bytes(256));
         $this->setKey($key);
+
+        return $this;
     }
 
     public function getAdministrator(): Administrator|Proxy|null
     {
         return $this->administrator;
     }
 
-    public function setAdministrator(Administrator $administrator): void
+    public function setAdministrator(Administrator $administrator): self
     {
         $this->administrator = $administrator;
+        return $this;
     }
 }

src/Domain/Model/Messaging/Attachment.php

@@ -50,48 +50,53 @@ public function getFilename(): ?string
         return $this->filename;
     }
 
-    public function setFilename(?string $filename): void
+    public function setFilename(?string $filename): self
     {
         $this->filename = $filename;
+        return $this;
     }
 
     public function getRemoteFile(): ?string
     {
         return $this->remoteFile;
     }
 
-    public function setRemoteFile(?string $remoteFile): void
+    public function setRemoteFile(?string $remoteFile): self
     {
         $this->remoteFile = $remoteFile;
+        return $this;
     }
 
     public function getMimeType(): ?string
     {
         return $this->mimeType;
     }
 
-    public function setMimeType(?string $mimeType): void
+    public function setMimeType(?string $mimeType): self
     {
         $this->mimeType = $mimeType;
+        return $this;
     }
 
     public function getDescription(): ?string
     {
         return $this->description;
     }
 
-    public function setDescription(?string $description): void
+    public function setDescription(?string $description): self
     {
         $this->description = $description;
+        return $this;
     }
 
     public function getSize(): ?int
     {
         return $this->size;
     }
 
-    public function setSize(?int $size): void
+    public function setSize(?int $size): self
     {
         $this->size = $size;
+        return $this;
     }
 }

src/Domain/Model/Messaging/Bounce.php

@@ -58,48 +58,53 @@ public function getDate(): ?DateTime
         return $this->date;
     }
 
-    public function setDate(?DateTime $date): void
+    public function setDate(?DateTime $date): self
     {
         $this->date = $date;
+        return $this;
     }
 
     public function getHeader(): ?string
     {
         return $this->header;
     }
 
-    public function setHeader(?string $header): void
+    public function setHeader(?string $header): self
     {
         $this->header = $header;
+        return $this;
     }
 
     public function getData(): ?string
     {
         return $this->data;
     }
 
-    public function setData(?string $data): void
+    public function setData(?string $data): self
     {
         $this->data = $data;
+        return $this;
     }
 
     public function getStatus(): ?string
     {
         return $this->status;
     }
 
-    public function setStatus(?string $status): void
+    public function setStatus(?string $status): self
     {
         $this->status = $status;
+        return $this;
     }
 
     public function getComment(): ?string
     {
         return $this->comment;
     }
 
-    public function setComment(?string $comment): void
+    public function setComment(?string $comment): self
     {
         $this->comment = $comment;
+        return $this;
     }
 }