Fix code review issues: typos, docblock consistency, and null checks

Co-authored-by: jbtronics <[email protected]>

Author: Copilot

src/Entity/Base/AttachmentsTrait.php

@@ -86,7 +86,7 @@ public function removeAttachment(Attachment $attachment): self
         $this->attachments->removeElement($attachment);
 
         //Check if this is the master attachment -> remove it from master attachment too, or it can not be deleted from DB...
-        if (isset($this->master_picture_attachment) && $attachment === $this->master_picture_attachment) {
+        if ($this->master_picture_attachment !== null && $attachment === $this->master_picture_attachment) {
             $this->setMasterPictureAttachment(null);
         }
 
@@ -104,7 +104,7 @@ protected function cloneAttachments(): void
             //Set master attachment is needed
             foreach ($attachments as $attachment) {
                 $clone = clone $attachment;
-                if (isset($this->master_picture_attachment) && $attachment === $this->master_picture_attachment) {
+                if ($this->master_picture_attachment !== null && $attachment === $this->master_picture_attachment) {
                     $this->setMasterPictureAttachment($clone);
                 }
                 $this->addAttachment($clone);

src/Entity/Base/CompanyTrait.php

@@ -94,7 +94,7 @@ public function getAddress(): string
     }
 
     /**
-     * Set the addres.
+     * Set the address.
      *
      * @param string $new_address the new address (with "\n" as line break)
      *

src/Entity/Base/StructuralElementTrait.php

@@ -92,7 +92,7 @@ trait StructuralElementTrait
      * Alternative names (semicolon-separated) for this element, which can be used for searching (especially for info provider system)
      */
     #[ORM\Column(type: Types::TEXT, nullable: true, options: ['default' => null])]
-    private ?string $alternative_names = "";
+    private ?string $alternative_names = null;
 
     /**
      * Initialize structural element collections.
@@ -153,7 +153,7 @@ public function isRoot(): bool
     /**
      * Get the parent of this element.
      *
-     * @return static|null The parent element. Null if this element, does not have a parent.
+     * @return static|null The parent element. Null if this element does not have a parent.
      */
     public function getParent(): ?self
     {
@@ -163,7 +163,7 @@ public function getParent(): ?self
     /**
      *  Get the comment of the element as markdown encoded string.
      *
-     * @return string the comment
+     * @return string|null the comment
      */
     public function getComment(): ?string
     {