Merge pull request #45122 from nextcloud/techdebt/noid/remove-temporary-executeUpdate

fix(querybuilder): Remove temporary internal method executeUpdate()
pull/45149/head
Joas Schilling 2 weeks ago committed by GitHub
commit e5f5f06b1b
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

@ -302,21 +302,6 @@ class QueryBuilder implements IQueryBuilder {
throw new \RuntimeException('Invalid return type for query');
}
/**
* Monkey-patched compatibility layer for apps that were adapted for Nextcloud 22 before
* the first beta, where executeStatement was named executeUpdate.
*
* Static analysis should catch those misuses, but until then let's try to keep things
* running.
*
* @internal
* @deprecated
* @todo drop ASAP
*/
public function executeUpdate(): int {
return $this->executeStatement();
}
public function executeStatement(): int {
if ($this->getType() === \Doctrine\DBAL\Query\QueryBuilder::SELECT) {
throw new \RuntimeException('Invalid query type, expected INSERT, DELETE or UPDATE statement');

Loading…
Cancel
Save