fix(share): do not re-accept rejected shares

ArtificialOwl · ArtificialOwl · commit 265556cb62cd · 2026-07-21T17:46:38.000-01:00
Signed-off-by: Maxence Lange &lt;maxence@artificial-owl.com&gt;
diff --git a/apps/files_sharing/lib/Migration/SetAcceptedStatus.php b/apps/files_sharing/lib/Migration/SetAcceptedStatus.php
@@ -48,7 +48,8 @@ public function run(IOutput $output): void {
 		$query
 			->update('share')
 			->set('accepted', $query->createNamedParameter(IShare::STATUS_ACCEPTED))
-			->where($query->expr()->in('share_type', $query->createNamedParameter([IShare::TYPE_USER, IShare::TYPE_GROUP, IShare::TYPE_USERGROUP], IQueryBuilder::PARAM_INT_ARRAY)));
+			->where($query->expr()->in('share_type', $query->createNamedParameter([IShare::TYPE_USER, IShare::TYPE_GROUP, IShare::TYPE_USERGROUP], IQueryBuilder::PARAM_INT_ARRAY)))
+			->andWhere($query->expr()->neq('accepted', $query->createNamedParameter(IShare::STATUS_REJECTED, IQueryBuilder::PARAM_INT)));
 		$query->executeStatement();
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,8 @@ public function run(IOutput $output): void {`
`48`	`48`	`$query`
`49`	`49`	`->update('share')`
`50`	`50`	`->set('accepted', $query->createNamedParameter(IShare::STATUS_ACCEPTED))`
`51`		`- ->where($query->expr()->in('share_type', $query->createNamedParameter([IShare::TYPE_USER, IShare::TYPE_GROUP, IShare::TYPE_USERGROUP], IQueryBuilder::PARAM_INT_ARRAY)));`
	`51`	`+ ->where($query->expr()->in('share_type', $query->createNamedParameter([IShare::TYPE_USER, IShare::TYPE_GROUP, IShare::TYPE_USERGROUP], IQueryBuilder::PARAM_INT_ARRAY)))`
	`52`	`+ ->andWhere($query->expr()->neq('accepted', $query->createNamedParameter(IShare::STATUS_REJECTED, IQueryBuilder::PARAM_INT)));`
`52`	`53`	`$query->executeStatement();`
`53`	`54`	`}`
`54`	`55`