From 60481a742ed0ce570f9a687432bb3b2c265d8324 Mon Sep 17 00:00:00 2001
From: Anna Larch <anna@nextcloud.com>
Date: Wed, 26 May 2021 15:12:41 +0200
Subject: [PATCH] Add DB exception throws tag to QBMapper PHPDoc

Signed-off-by: Anna Larch <anna@nextcloud.com>
---
 lib/public/AppFramework/Db/QBMapper.php | 24 +++++++++++++++++-------
 1 file changed, 17 insertions(+), 7 deletions(-)

diff --git a/lib/public/AppFramework/Db/QBMapper.php b/lib/public/AppFramework/Db/QBMapper.php
index fb5215895a28d..42b21f8f4b846 100644
--- a/lib/public/AppFramework/Db/QBMapper.php
+++ b/lib/public/AppFramework/Db/QBMapper.php
@@ -86,10 +86,12 @@ public function getTableName(): string {
 
 	/**
 	 * Deletes an entity from the table
+	 *
 	 * @param Entity $entity the entity that should be deleted
 	 * @psalm-param T $entity the entity that should be deleted
 	 * @return Entity the deleted entity
 	 * @psalm-return T the deleted entity
+	 * @throws Exception
 	 * @since 14.0.0
 	 */
 	public function delete(Entity $entity): Entity {
@@ -108,10 +110,12 @@ public function delete(Entity $entity): Entity {
 
 	/**
 	 * Creates a new entry in the db from an entity
+	 *
 	 * @param Entity $entity the entity that should be created
 	 * @psalm-param T $entity the entity that should be created
 	 * @return Entity the saved entity with the set id
 	 * @psalm-return T the saved entity with the set id
+	 * @throws Exception
 	 * @since 14.0.0
 	 */
 	public function insert(Entity $entity): Entity {
@@ -151,6 +155,7 @@ public function insert(Entity $entity): Entity {
 	 * @psalm-param T $entity the entity that should be created/updated
 	 * @return Entity the saved entity with the (new) id
 	 * @psalm-return T the saved entity with the (new) id
+	 * @throws Exception
 	 * @throws \InvalidArgumentException if entity has no id
 	 * @since 15.0.0
 	 */
@@ -167,11 +172,13 @@ public function insertOrUpdate(Entity $entity): Entity {
 
 	/**
 	 * Updates an entry in the db from an entity
-	 * @throws \InvalidArgumentException if entity has no id
+	 *
 	 * @param Entity $entity the entity that should be created
 	 * @psalm-param T $entity the entity that should be created
 	 * @return Entity the saved entity with the set id
 	 * @psalm-return T the saved entity with the set id
+	 * @throws Exception
+	 * @throws \InvalidArgumentException if entity has no id
 	 * @since 14.0.0
 	 */
 	public function update(Entity $entity): Entity {
@@ -253,12 +260,13 @@ protected function getParameterTypeForProperty(Entity $entity, string $property)
 	 * Returns an db result and throws exceptions when there are more or less
 	 * results
 	 *
-	 * @see findEntity
-	 *
 	 * @param IQueryBuilder $query
-	 * @throws DoesNotExistException if the item does not exist
-	 * @throws MultipleObjectsReturnedException if more than one item exist
 	 * @return array the result as row
+	 * @throws Exception
+	 * @throws MultipleObjectsReturnedException if more than one item exist
+	 * @throws DoesNotExistException if the item does not exist
+	 * @see findEntity
+	 *
 	 * @since 14.0.0
 	 */
 	protected function findOneQuery(IQueryBuilder $query): array {
@@ -317,6 +325,7 @@ protected function mapRowToEntity(array $row): Entity {
 	 * @param IQueryBuilder $query
 	 * @return Entity[] all fetched entities
 	 * @psalm-return T[] all fetched entities
+	 * @throws Exception
 	 * @since 14.0.0
 	 */
 	protected function findEntities(IQueryBuilder $query): array {
@@ -339,10 +348,11 @@ protected function findEntities(IQueryBuilder $query): array {
 	 * results
 	 *
 	 * @param IQueryBuilder $query
-	 * @throws DoesNotExistException if the item does not exist
-	 * @throws MultipleObjectsReturnedException if more than one item exist
 	 * @return Entity the entity
 	 * @psalm-return T the entity
+	 * @throws Exception
+	 * @throws MultipleObjectsReturnedException if more than one item exist
+	 * @throws DoesNotExistException if the item does not exist
 	 * @since 14.0.0
 	 */
 	protected function findEntity(IQueryBuilder $query): Entity {