diff options
author | Côme Chilliet <91878298+come-nc@users.noreply.github.com> | 2023-02-21 17:37:45 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-02-21 17:37:45 +0100 |
commit | 18c4761ab00429f11355bb3f42772a9142577ab5 (patch) | |
tree | 5304d7f990550afc8ac388bc107cf8b18aeb0f02 /apps | |
parent | a7892881a3fd10ff234d14ea878e3104a81d77e8 (diff) | |
parent | 74811813e7218f5c24d199086f5b56f8c476ef45 (diff) | |
download | nextcloud-server-18c4761ab00429f11355bb3f42772a9142577ab5.tar.gz nextcloud-server-18c4761ab00429f11355bb3f42772a9142577ab5.zip |
Merge pull request #36795 from nextcloud/fix/remove-erronous-parentheses-in-in-clause
Avoid extra parenthesis around SQL in IN expressions
Diffstat (limited to 'apps')
-rw-r--r-- | apps/files_sharing/lib/External/Manager.php | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/apps/files_sharing/lib/External/Manager.php b/apps/files_sharing/lib/External/Manager.php index 68bc6801e46..d2e113c8bb3 100644 --- a/apps/files_sharing/lib/External/Manager.php +++ b/apps/files_sharing/lib/External/Manager.php @@ -660,7 +660,7 @@ class Manager { $query->delete('federated_reshares') - ->where($query->expr()->in('share_id', $query->createFunction('(' . $select . ')'))); + ->where($query->expr()->in('share_id', $query->createFunction($select))); $query->execute(); $deleteReShares = $this->connection->getQueryBuilder(); @@ -730,10 +730,10 @@ class Manager { // delete group share entry and matching sub-entries $qb->delete('share_external') ->where( - $qb->expr()->orX( - $qb->expr()->eq('id', $qb->createParameter('share_id')), - $qb->expr()->eq('parent', $qb->createParameter('share_parent_id')) - ) + $qb->expr()->orX( + $qb->expr()->eq('id', $qb->createParameter('share_id')), + $qb->expr()->eq('parent', $qb->createParameter('share_parent_id')) + ) ); foreach ($shares as $share) { |