revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-21 22:32:41 +01:00
Code Issues Releases Wiki Activity

Fix PhabricatorEmptyQueryException handling in DiffusionCommitQuery::loadPage()

Browse source 
Summary: Add an additional check that `$empty_exception` is not `null` before try to `throw` it.

Test Plan: Read the previous code; run static code analysis.

Reviewers: O1 Blessed Committers, valerio.bozzolan

Reviewed By: O1 Blessed Committers, valerio.bozzolan

Subscribers: tobiaswiese, valerio.bozzolan, Matthew, Cigaryno

Differential Revision: https://we.phorge.it/D25781
This commit is contained in:
Andre Klapper 2024-08-15 16:02:33 +02:00
parent b9ea6f1ce8
commit edb2175a9b
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/diffusion/query/DiffusionCommitQuery.php
View file

@ -252,7 +252,7 @@ final class DiffusionCommitQuery
$table->getTableName());
}
if (!$subqueries) {
if (!$subqueries && $empty_exception) {
throw $empty_exception;
}