Merge pull request #26064 from nextcloud/techdept/psalm/type_oauth_db

Add psalm annotation to the DB handlers of OAuth2
2021-03-11 16:46:08 +07:00 · 2021-03-11 16:46:08 +07:00 · a3b1c1cea3
parent 69c4578b5b c2f7c417b2
commit a3b1c1cea3
2 changed files with 7 additions and 1 deletions
--- a/apps/oauth2/lib/Db/AccessTokenMapper.php
+++ b/apps/oauth2/lib/Db/AccessTokenMapper.php
@ -34,6 +34,9 @@ use OCP\AppFramework\Db\QBMapper;
 use OCP\DB\QueryBuilder\IQueryBuilder;
 use OCP\IDBConnection;

+/**
+ * @template-extends QBMapper<AccessToken>
+ */
 class AccessTokenMapper extends QBMapper {

 	/**
@ -74,6 +77,6 @@ class AccessTokenMapper extends QBMapper {
 		$qb
 			->delete($this->tableName)
 			->where($qb->expr()->eq('client_id', $qb->createNamedParameter($id, IQueryBuilder::PARAM_INT)));
-		$qb->execute();
+		$qb->executeUpdate();
 	}
 }
--- a/apps/oauth2/lib/Db/ClientMapper.php
+++ b/apps/oauth2/lib/Db/ClientMapper.php
@ -34,6 +34,9 @@ use OCP\AppFramework\Db\QBMapper;
 use OCP\DB\QueryBuilder\IQueryBuilder;
 use OCP\IDBConnection;

+/**
+ * @template-extends QBMapper<Client>
+ */
 class ClientMapper extends QBMapper {

 	/**