Skip to content

Commit 828564d

Browse files
feat(sharereview): implement deleteShare() via direct SQL with logging
Assisted-by: Claude Code:claude-sonnet-4-6 Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
1 parent 58f8796 commit 828564d

1 file changed

Lines changed: 10 additions & 1 deletion

File tree

lib/ShareReview/ShareReviewSource.php

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,16 @@ public function getShares(): array {
5959
}
6060

6161
public function deleteShare(string $shareId): bool {
62-
return false;
62+
$this->logger->info('Deck ShareReview: deleting share {id}', ['id' => $shareId]);
63+
try {
64+
$qb = $this->db->getQueryBuilder();
65+
$qb->delete(self::ACL_TABLE)
66+
->where($qb->expr()->eq('id', $qb->createNamedParameter((int)$shareId, IQueryBuilder::PARAM_INT)));
67+
return $qb->executeStatement() > 0;
68+
} catch (Exception $e) {
69+
$this->logger->error('Deck ShareReview: failed to delete share {id}: {message}', ['id' => $shareId, 'message' => $e->getMessage()]);
70+
return false;
71+
}
6372
}
6473

6574
/** @return list<array<string, mixed>> */

0 commit comments

Comments
 (0)