Remove some cascade removes (#1445)

MbinOrg · Feb 9, 2025 · 95338ff · 95338ff
1 parent af62c08
commit 95338ff
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 9 deletions.
diff --git a/src/Entity/MessageThread.php b/src/Entity/MessageThread.php
@@ -34,7 +34,7 @@ class MessageThread
             new JoinColumn(name: 'user_id', referencedColumnName: 'id', onDelete: 'CASCADE'),
         ]
     )]
-    #[ManyToMany(targetEntity: User::class, cascade: ['persist', 'remove'], orphanRemoval: true)]
+    #[ManyToMany(targetEntity: User::class, cascade: ['persist'], orphanRemoval: true)]
     public Collection $participants;
     #[OneToMany(mappedBy: 'thread', targetEntity: Message::class, cascade: ['persist', 'remove'], orphanRemoval: true)]
     #[OrderBy(['createdAt' => 'ASC'])]

diff --git a/src/Entity/NewSignupNotification.php b/src/Entity/NewSignupNotification.php
@@ -14,7 +14,7 @@
 #[Entity]
 class NewSignupNotification extends Notification
 {
-    #[ManyToOne(targetEntity: User::class, cascade: ['remove'])]
+    #[ManyToOne(targetEntity: User::class)]
     #[JoinColumn(nullable: true, onDelete: 'CASCADE')]
     public ?User $newUser;
 

diff --git a/src/Entity/NotificationSettings.php b/src/Entity/NotificationSettings.php
@@ -20,23 +20,23 @@ class NotificationSettings
     #[Id, GeneratedValue, Column(type: 'integer')]
     private int $id;
 
-    #[ManyToOne(targetEntity: User::class, cascade: ['remove'])]
+    #[ManyToOne(targetEntity: User::class)]
     #[JoinColumn(nullable: false, onDelete: 'CASCADE')]
     public User $user;
 
-    #[ManyToOne(targetEntity: Entry::class, cascade: ['remove'])]
+    #[ManyToOne(targetEntity: Entry::class)]
     #[JoinColumn(nullable: true, onDelete: 'CASCADE')]
     public ?Entry $entry = null;
 
-    #[ManyToOne(targetEntity: Post::class, cascade: ['remove'])]
+    #[ManyToOne(targetEntity: Post::class)]
     #[JoinColumn(nullable: true, onDelete: 'CASCADE')]
     public ?Post $post = null;
 
-    #[ManyToOne(targetEntity: Magazine::class, cascade: ['remove'])]
+    #[ManyToOne(targetEntity: Magazine::class)]
     #[JoinColumn(nullable: true, onDelete: 'CASCADE')]
     public ?Magazine $magazine = null;
 
-    #[ManyToOne(targetEntity: User::class, cascade: ['remove'])]
+    #[ManyToOne(targetEntity: User::class)]
     #[JoinColumn(nullable: true, onDelete: 'CASCADE')]
     public ?User $targetUser = null;
 

diff --git a/src/Entity/UserPushSubscription.php b/src/Entity/UserPushSubscription.php
@@ -18,7 +18,7 @@ class UserPushSubscription
     #[Id, GeneratedValue, Column(type: 'integer')]
     public int $id;
 
-    #[ManyToOne(targetEntity: User::class, cascade: ['persist', 'remove'], inversedBy: 'pushSubscriptions')]
+    #[ManyToOne(targetEntity: User::class, cascade: ['persist'], inversedBy: 'pushSubscriptions')]
     public User $user;
 
     #[Column(type: 'text')]
@@ -30,7 +30,7 @@ class UserPushSubscription
     /**
      * @var AccessToken|null this is only null for the web interface push messages
      */
-    #[ManyToOne(targetEntity: AccessToken::class, cascade: ['persist', 'remove'])]
+    #[ManyToOne(targetEntity: AccessToken::class, cascade: ['persist'])]
     #[JoinColumn(name: 'api_token', referencedColumnName: 'identifier', unique: true, nullable: true, onDelete: 'CASCADE')]
     public ?AccessToken $apiToken = null;