Fix implicitly nullable parameter declarations for PHP 8.4

Summary: Followup to rPdb61eb20 and rPf3d49f74. This patch should cover all remaining issues now that PHPStan covers it (instead of the previous trial-and-error approach). Implicitly nullable parameter declarations are deprecated in PHP 8.4: https://php.watch/versions/8.4/implicitly-marking-parameter-type-nullable-deprecated The proposed syntax was introduced in PHP 7.1 and Phorge requires PHP 7.2 now. Test Plan: Run static code analysis. Reviewers: O1 Blessed Committers, avivey Reviewed By: O1 Blessed Committers, avivey Subscribers: tobiaswiese, valerio.bozzolan, Matthew, Cigaryno Differential Revision: https://we.phorge.it/D25832
2024-11-25 08:12:40 +01:00 · 2024-10-23 12:11:53 +02:00 · 2024-10-23 12:11:53 +02:00 · 5f4587bb88
commit 5f4587bb88
parent f3d49f7407
39 changed files with 56 additions and 56 deletions
--- a/src/applications/auth/engine/PhabricatorAuthInviteEngine.php
+++ b/src/applications/auth/engine/PhabricatorAuthInviteEngine.php
@ -125,7 +125,7 @@ final class PhabricatorAuthInviteEngine extends Phobject {
  private function handleLoggedInInvite(
    PhabricatorAuthInvite $invite,
    PhabricatorUser $viewer,
-    PhabricatorUserEmail $email = null) {
+    ?PhabricatorUserEmail $email = null) {

    if ($email && ($email->getUserPHID() !== $viewer->getPHID())) {
      $other_user = $this->loadUserForEmail($email);
--- a/src/applications/base/controller/PhabricatorController.php
+++ b/src/applications/base/controller/PhabricatorController.php
@ -485,8 +485,8 @@ abstract class PhabricatorController extends AphrontController {

  protected function buildTransactionTimeline(
    PhabricatorApplicationTransactionInterface $object,
-    PhabricatorApplicationTransactionQuery $query = null,
-    PhabricatorMarkupEngine $engine = null,
+    ?PhabricatorApplicationTransactionQuery $query = null,
+    ?PhabricatorMarkupEngine $engine = null,
    $view_data = array()) {

    $request = $this->getRequest();
--- a/src/applications/calendar/import/PhabricatorCalendarImportEngine.php
+++ b/src/applications/calendar/import/PhabricatorCalendarImportEngine.php
@ -55,7 +55,7 @@ abstract class PhabricatorCalendarImportEngine
  final protected function importEventDocument(
    PhabricatorUser $viewer,
    PhabricatorCalendarImport $import,
-    PhutilCalendarRootNode $root = null) {
+    ?PhutilCalendarRootNode $root = null) {

    $event_type = PhutilCalendarEventNode::NODETYPE;

--- a/src/applications/calendar/parser/data/PhutilCalendarRecurrenceSet.php
+++ b/src/applications/calendar/parser/data/PhutilCalendarRecurrenceSet.php
@ -21,8 +21,8 @@ final class PhutilCalendarRecurrenceSet
  }

  public function getEventsBetween(
-    PhutilCalendarDateTime $start = null,
-    PhutilCalendarDateTime $end = null,
+    ?PhutilCalendarDateTime $start = null,
+    ?PhutilCalendarDateTime $end = null,
    $limit = null) {

    if ($end === null && $limit === null) {
--- a/src/applications/conduit/controller/PhabricatorConduitAPIController.php
+++ b/src/applications/conduit/controller/PhabricatorConduitAPIController.php
@ -527,9 +527,9 @@ final class PhabricatorConduitAPIController

  private function buildHumanReadableResponse(
    $method,
-    ConduitAPIRequest $request = null,
+    ?ConduitAPIRequest $request = null,
    $result = null,
-    ConduitAPIMethod $method_implementation = null) {
+    ?ConduitAPIMethod $method_implementation = null) {

    $param_rows = array();
    $param_rows[] = array('Method', $this->renderAPIValue($method));
--- a/src/applications/conduit/method/ConduitGetCertificateConduitAPIMethod.php
+++ b/src/applications/conduit/method/ConduitGetCertificateConduitAPIMethod.php
@ -80,7 +80,7 @@ final class ConduitGetCertificateConduitAPIMethod extends ConduitAPIMethod {

  private function logFailure(
    ConduitAPIRequest $request,
-    PhabricatorConduitCertificateToken $info = null) {
+    ?PhabricatorConduitCertificateToken $info = null) {

    $log = PhabricatorUserLog::initializeNewLog(
        $request->getUser(),
--- a/src/applications/config/controller/settings/PhabricatorConfigSettingsListController.php
+++ b/src/applications/config/controller/settings/PhabricatorConfigSettingsListController.php
@ -75,7 +75,7 @@ final class PhabricatorConfigSettingsListController

  private function newConfigOptionView(
    PhabricatorConfigOption $option,
-    PhabricatorConfigEntry $stored_value = null) {
+    ?PhabricatorConfigEntry $stored_value = null) {

    $summary = $option->getSummary();

--- a/src/applications/conpherence/view/ConpherenceDurableColumnView.php
+++ b/src/applications/conpherence/view/ConpherenceDurableColumnView.php
@ -32,7 +32,7 @@ final class ConpherenceDurableColumnView extends AphrontTagView {
  }

  public function setSelectedConpherence(
-    ConpherenceThread $conpherence = null) {
+    ?ConpherenceThread $conpherence = null) {
    $this->selectedConpherence = $conpherence;
    return $this;
  }
--- a/src/applications/dashboard/controller/dashboard/PhabricatorDashboardAdjustController.php
+++ b/src/applications/dashboard/controller/dashboard/PhabricatorDashboardAdjustController.php
@ -204,7 +204,7 @@ final class PhabricatorDashboardAdjustController
  private function handleMoveRequest(
    PhabricatorDashboard $dashboard,
    PhabricatorDashboardPanelRef $panel_ref,
-    PhabricatorDashboardPanelRef $after_ref = null) {
+    ?PhabricatorDashboardPanelRef $after_ref = null) {

    $request = $this->getRequest();
    $request->validateCSRF();
--- a/src/applications/dashboard/layoutconfig/PhabricatorDashboardPanelRefList.php
+++ b/src/applications/dashboard/layoutconfig/PhabricatorDashboardPanelRefList.php
@ -121,7 +121,7 @@ final class PhabricatorDashboardPanelRefList
  public function movePanelRef(
    PhabricatorDashboardPanelRef $target,
    $column_key,
-    PhabricatorDashboardPanelRef $after = null) {
+    ?PhabricatorDashboardPanelRef $after = null) {

    $target->setColumnKey($column_key);

--- a/src/applications/differential/conduit/DifferentialConduitAPIMethod.php
+++ b/src/applications/differential/conduit/DifferentialConduitAPIMethod.php
@ -20,7 +20,7 @@ abstract class DifferentialConduitAPIMethod extends ConduitAPIMethod {

  protected function buildInlineInfoDictionary(
    DifferentialInlineComment $inline,
-    DifferentialChangeset $changeset = null) {
+    ?DifferentialChangeset $changeset = null) {

    $file_path = null;
    $diff_id = null;
--- a/src/applications/differential/controller/DifferentialRevisionViewController.php
+++ b/src/applications/differential/controller/DifferentialRevisionViewController.php
@ -863,8 +863,8 @@ final class DifferentialRevisionViewController

  private function loadChangesetsAndVsMap(
    DifferentialDiff $target,
-    DifferentialDiff $diff_vs = null,
-    PhabricatorRepository $repository = null) {
+    ?DifferentialDiff $diff_vs = null,
+    ?PhabricatorRepository $repository = null) {
    $viewer = $this->getViewer();

    $load_diffs = array($target);
@ -1049,7 +1049,7 @@ final class DifferentialRevisionViewController
    array $changesets,
    array $vs_changesets,
    array $vs_map,
-    PhabricatorRepository $repository = null) {
+    ?PhabricatorRepository $repository = null) {

    assert_instances_of($changesets,    'DifferentialChangeset');
    assert_instances_of($vs_changesets, 'DifferentialChangeset');
--- a/src/applications/differential/render/DifferentialChangesetRenderer.php
+++ b/src/applications/differential/render/DifferentialChangesetRenderer.php
@ -105,7 +105,7 @@ abstract class DifferentialChangesetRenderer extends Phobject {
    return $this->depthOnlyLines;
  }

-  public function attachOldFile(PhabricatorFile $old = null) {
+  public function attachOldFile(?PhabricatorFile $old = null) {
    $this->oldFile = $old;
    return $this;
  }
@ -121,7 +121,7 @@ abstract class DifferentialChangesetRenderer extends Phobject {
    return (bool)$this->oldFile;
  }

-  public function attachNewFile(PhabricatorFile $new = null) {
+  public function attachNewFile(?PhabricatorFile $new = null) {
    $this->newFile = $new;
    return $this;
  }
--- a/src/applications/diffusion/controller/DiffusionLastModifiedController.php
+++ b/src/applications/diffusion/controller/DiffusionLastModifiedController.php
@ -84,7 +84,7 @@ final class DiffusionLastModifiedController extends DiffusionController {

  private function renderColumns(
    DiffusionRequest $drequest,
-    PhabricatorRepositoryCommit $commit = null,
+    ?PhabricatorRepositoryCommit $commit = null,
    $lint = null) {
    $viewer = $this->getViewer();

--- a/src/applications/diffusion/data/DiffusionLocalRepositoryFilter.php
+++ b/src/applications/diffusion/data/DiffusionLocalRepositoryFilter.php
@ -20,7 +20,7 @@ final class DiffusionLocalRepositoryFilter extends Phobject {
    return $this->viewer;
  }

-  public function setDevice(AlmanacDevice $device = null) {
+  public function setDevice(?AlmanacDevice $device = null) {
    $this->device = $device;
    return $this;
  }
--- a/src/applications/diffusion/document/DiffusionDocumentRenderingEngine.php
+++ b/src/applications/diffusion/document/DiffusionDocumentRenderingEngine.php
@ -65,7 +65,7 @@ final class DiffusionDocumentRenderingEngine

  protected function addApplicationCrumbs(
    PHUICrumbsView $crumbs,
-    PhabricatorDocumentRef $ref = null) {
+    ?PhabricatorDocumentRef $ref = null) {
    return;
  }

--- a/src/applications/diffusion/query/DiffusionCommitRevisionQuery.php
+++ b/src/applications/diffusion/query/DiffusionCommitRevisionQuery.php
@ -68,7 +68,7 @@ final class DiffusionCommitRevisionQuery
    PhabricatorUser $viewer,
    $source_object,
    array $object_names,
-    PhabricatorRepository $repository_scope = null) {
+    ?PhabricatorRepository $repository_scope = null) {

    // Fetch commits first, since we need to load data on commits in order
    // to identify associated revisions later on.
--- a/src/applications/files/diff/PhabricatorDocumentEngineBlocks.php
+++ b/src/applications/files/diff/PhabricatorDocumentEngineBlocks.php
@ -39,7 +39,7 @@ final class PhabricatorDocumentEngineBlocks
  }

  public function addBlockList(
-    PhabricatorDocumentRef $ref = null,
+    ?PhabricatorDocumentRef $ref = null,
    array $blocks = array()) {

    assert_instances_of($blocks, 'PhabricatorDocumentEngineBlock');
--- a/src/applications/files/document/PhabricatorDocumentEngine.php
+++ b/src/applications/files/document/PhabricatorDocumentEngine.php
@ -32,8 +32,8 @@ abstract class PhabricatorDocumentEngine
  }

  public function canDiffDocuments(
-    PhabricatorDocumentRef $uref = null,
-    PhabricatorDocumentRef $vref = null) {
+    ?PhabricatorDocumentRef $uref = null,
+    ?PhabricatorDocumentRef $vref = null) {
    return false;
  }

--- a/src/applications/files/document/PhabricatorImageDocumentEngine.php
+++ b/src/applications/files/document/PhabricatorImageDocumentEngine.php
@ -18,8 +18,8 @@ final class PhabricatorImageDocumentEngine
  }

  public function canDiffDocuments(
-    PhabricatorDocumentRef $uref = null,
-    PhabricatorDocumentRef $vref = null) {
+    ?PhabricatorDocumentRef $uref = null,
+    ?PhabricatorDocumentRef $vref = null) {

    // For now, we can only render a rich image diff if the documents have
    // their data stored in Files already.
@ -36,8 +36,8 @@ final class PhabricatorImageDocumentEngine
  }

  public function newEngineBlocks(
-    PhabricatorDocumentRef $uref = null,
-    PhabricatorDocumentRef $vref = null) {
+    ?PhabricatorDocumentRef $uref = null,
+    ?PhabricatorDocumentRef $vref = null) {

    if ($uref) {
      $u_blocks = $this->newDiffBlocks($uref);
--- a/src/applications/files/document/PhabricatorJupyterDocumentEngine.php
+++ b/src/applications/files/document/PhabricatorJupyterDocumentEngine.php
@ -36,14 +36,14 @@ final class PhabricatorJupyterDocumentEngine
  }

  public function canDiffDocuments(
-    PhabricatorDocumentRef $uref = null,
-    PhabricatorDocumentRef $vref = null) {
+    ?PhabricatorDocumentRef $uref = null,
+    ?PhabricatorDocumentRef $vref = null) {
    return true;
  }

  public function newEngineBlocks(
-    PhabricatorDocumentRef $uref = null,
-    PhabricatorDocumentRef $vref = null) {
+    ?PhabricatorDocumentRef $uref = null,
+    ?PhabricatorDocumentRef $vref = null) {

    $blocks = new PhabricatorDocumentEngineBlocks();

--- a/src/applications/files/document/render/PhabricatorDocumentRenderingEngine.php
+++ b/src/applications/files/document/render/PhabricatorDocumentRenderingEngine.php
@ -329,7 +329,7 @@ abstract class PhabricatorDocumentRenderingEngine

  protected function addApplicationCrumbs(
    PHUICrumbsView $crumbs,
-    PhabricatorDocumentRef $ref = null) {
+    ?PhabricatorDocumentRef $ref = null) {
    return;
  }

--- a/src/applications/files/document/render/PhabricatorFileDocumentRenderingEngine.php
+++ b/src/applications/files/document/render/PhabricatorFileDocumentRenderingEngine.php
@ -35,7 +35,7 @@ final class PhabricatorFileDocumentRenderingEngine

  protected function addApplicationCrumbs(
    PHUICrumbsView $crumbs,
-    PhabricatorDocumentRef $ref = null) {
+    ?PhabricatorDocumentRef $ref = null) {

    if ($ref) {
      $file = $ref->getFile();
--- a/src/applications/project/tests/PhabricatorProjectCoreTestCase.php
+++ b/src/applications/project/tests/PhabricatorProjectCoreTestCase.php
@ -1534,7 +1534,7 @@ final class PhabricatorProjectCoreTestCase extends PhabricatorTestCase {

  private function createProject(
    PhabricatorUser $user,
-    PhabricatorProject $parent = null,
+    ?PhabricatorProject $parent = null,
    $is_milestone = false) {

    $project = PhabricatorProject::initializeNewProject($user, $parent);
--- a/src/applications/project/view/ProjectBoardTaskCard.php
+++ b/src/applications/project/view/ProjectBoardTaskCard.php
@ -54,7 +54,7 @@ final class ProjectBoardTaskCard extends Phobject {
    return $this->task;
  }

-  public function setOwner(PhabricatorObjectHandle $owner = null) {
+  public function setOwner(?PhabricatorObjectHandle $owner = null) {
    $this->owner = $owner;
    return $this;
  }
--- a/src/applications/repository/daemon/PhabricatorRepositoryPullLocalDaemon.php
+++ b/src/applications/repository/daemon/PhabricatorRepositoryPullLocalDaemon.php
@ -405,7 +405,7 @@ final class PhabricatorRepositoryPullLocalDaemon
  private function loadPullableRepositories(
    array $include,
    array $exclude,
-    AlmanacDevice $device = null) {
+    ?AlmanacDevice $device = null) {

    $query = id(new PhabricatorRepositoryQuery())
      ->setViewer($this->getViewer());
--- a/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
+++ b/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
@ -172,7 +172,7 @@ final class PhabricatorRepositoryPullEngine
    $this->donePull();
  }

-  private function abortPull($message, Exception $ex = null) {
+  private function abortPull($message, ?Exception $ex = null) {
    $code_error = PhabricatorRepositoryStatusMessage::CODE_ERROR;
    $this->updateRepositoryInitStatus($code_error, $message);
    if ($ex) {
--- a/src/applications/search/compiler/PhutilSearchQueryCompiler.php
+++ b/src/applications/search/compiler/PhutilSearchQueryCompiler.php
@ -437,7 +437,7 @@ final class PhutilSearchQueryCompiler

  private function renderToken(
    PhutilSearchQueryToken $token,
-    PhutilSearchStemmer $stemmer = null) {
+    ?PhutilSearchStemmer $stemmer = null) {
    $value = $token->getValue();

    if ($stemmer) {
--- a/src/applications/search/compiler/tests/PhutilSearchQueryCompilerTestCase.php
+++ b/src/applications/search/compiler/tests/PhutilSearchQueryCompilerTestCase.php
@ -227,7 +227,7 @@ final class PhutilSearchQueryCompilerTestCase
  private function assertCompileQueries(
    array $tests,
    $operators = null,
-    PhutilSearchStemmer $stemmer = null) {
+    ?PhutilSearchStemmer $stemmer = null) {
    foreach ($tests as $input => $expect) {
      $caught = null;

--- a/src/applications/search/fulltextstorage/PhabricatorElasticFulltextStorageEngine.php
+++ b/src/applications/search/fulltextstorage/PhabricatorElasticFulltextStorageEngine.php
@ -263,7 +263,7 @@ class PhabricatorElasticFulltextStorageEngine
      $exceptions);
  }

-  public function indexExists(PhabricatorElasticsearchHost $host = null) {
+  public function indexExists(?PhabricatorElasticsearchHost $host = null) {
    if (!$host) {
      $host = $this->getHostForRead();
    }
@ -397,7 +397,7 @@ class PhabricatorElasticFulltextStorageEngine
    return $data;
  }

-  public function indexIsSane(PhabricatorElasticsearchHost $host = null) {
+  public function indexIsSane(?PhabricatorElasticsearchHost $host = null) {
    if (!$host) {
      $host = $this->getHostForRead();
    }
@ -476,7 +476,7 @@ class PhabricatorElasticFulltextStorageEngine
    $this->executeRequest($host, '/', $data, 'PUT');
  }

-  public function getIndexStats(PhabricatorElasticsearchHost $host = null) {
+  public function getIndexStats(?PhabricatorElasticsearchHost $host = null) {
    if ($this->version < 2) {
      return false;
    }
--- a/src/applications/search/menuitem/PhabricatorDashboardProfileMenuItem.php
+++ b/src/applications/search/menuitem/PhabricatorDashboardProfileMenuItem.php
@ -27,7 +27,7 @@ final class PhabricatorDashboardProfileMenuItem
    return true;
  }

-  private function attachDashboard(PhabricatorDashboard $dashboard = null) {
+  private function attachDashboard(?PhabricatorDashboard $dashboard = null) {
    $this->dashboard = $dashboard;
    return $this;
  }
--- a/src/applications/search/query/PhabricatorSearchApplicationSearchEngine.php
+++ b/src/applications/search/query/PhabricatorSearchApplicationSearchEngine.php
@ -201,7 +201,7 @@ final class PhabricatorSearchApplicationSearchEngine
  }

  public static function getIndexableDocumentTypes(
-    PhabricatorUser $viewer = null) {
+    ?PhabricatorUser $viewer = null) {

    // TODO: This is inelegant and not very efficient, but gets us reasonable
    // results. It would be nice to do this more elegantly.
--- a/src/applications/settings/panel/PhabricatorEmailPreferencesSettingsPanel.php
+++ b/src/applications/settings/panel/PhabricatorEmailPreferencesSettingsPanel.php
@ -146,7 +146,7 @@ final class PhabricatorEmailPreferencesSettingsPanel
    return $form_box;
  }

-  private function getAllEditorsWithTags(PhabricatorUser $user = null) {
+  private function getAllEditorsWithTags(?PhabricatorUser $user = null) {
    $editors = id(new PhutilClassMapQuery())
      ->setAncestorClass('PhabricatorApplicationTransactionEditor')
      ->setFilterMethod('getMailTagsMap')
@ -165,7 +165,7 @@ final class PhabricatorEmailPreferencesSettingsPanel
    return $editors;
  }

-  private function getAllTags(PhabricatorUser $user = null) {
+  private function getAllTags(?PhabricatorUser $user = null) {
    $tags = array();
    foreach ($this->getAllEditorsWithTags($user) as $editor) {
      $tags += $editor->getMailTagsMap();
--- a/src/applications/settings/setting/PhabricatorSetting.php
+++ b/src/applications/settings/setting/PhabricatorSetting.php
@ -4,7 +4,7 @@ abstract class PhabricatorSetting extends Phobject {

  private $viewer = false;

-  public function setViewer(PhabricatorUser $viewer = null) {
+  public function setViewer(?PhabricatorUser $viewer = null) {
    $this->viewer = $viewer;
    return $this;
  }
--- a/src/applications/transactions/draft/PhabricatorDraftEngine.php
+++ b/src/applications/transactions/draft/PhabricatorDraftEngine.php
@ -27,7 +27,7 @@ abstract class PhabricatorDraftEngine
  }

  final public function setVersionedDraft(
-    PhabricatorVersionedDraft $draft = null) {
+    ?PhabricatorVersionedDraft $draft = null) {
    $this->hasVersionedDraft = true;
    $this->versionedDraft = $draft;
    return $this;
--- a/src/infrastructure/diff/interface/PhabricatorInlineComment.php
+++ b/src/infrastructure/diff/interface/PhabricatorInlineComment.php
@ -288,7 +288,7 @@ abstract class PhabricatorInlineComment

  public function attachVersionedDraftForViewer(
    PhabricatorUser $viewer,
-    PhabricatorVersionedDraft $draft = null) {
+    ?PhabricatorVersionedDraft $draft = null) {

    $key = $viewer->getCacheFragment();
    $this->versionedDrafts[$key] = $draft;
@ -415,7 +415,7 @@ abstract class PhabricatorInlineComment

  private function getWireContentStateMap(
    $is_edit,
-    PhabricatorUser $viewer = null) {
+    ?PhabricatorUser $viewer = null) {

    $initial_state = $this->getInitialContentState();
    $committed_state = $this->getCommittedContentState();
--- a/src/infrastructure/util/tests/PhabricatorGlobalLockTestCase.php
+++ b/src/infrastructure/util/tests/PhabricatorGlobalLockTestCase.php
@ -205,7 +205,7 @@ final class PhabricatorGlobalLockTestCase

  private function tryHeldLock(
    $lock_name,
-    AphrontDatabaseConnection $conn = null) {
+    ?AphrontDatabaseConnection $conn = null) {

    $lock = PhabricatorGlobalLock::newLock($lock_name);

--- a/src/view/AphrontDialogView.php
+++ b/src/view/AphrontDialogView.php
@ -273,7 +273,7 @@ final class AphrontDialogView
  }

  public function setValidationException(
-    PhabricatorApplicationTransactionValidationException $ex = null) {
+    ?PhabricatorApplicationTransactionValidationException $ex = null) {
    $this->validationException = $ex;
    return $this;
  }
--- a/src/view/phui/PHUIObjectBoxView.php
+++ b/src/view/phui/PHUIObjectBoxView.php
@ -135,7 +135,7 @@ final class PHUIObjectBoxView extends AphrontTagView {
  }

  public function setValidationException(
-    PhabricatorApplicationTransactionValidationException $ex = null) {
+    ?PhabricatorApplicationTransactionValidationException $ex = null) {
    $this->validationException = $ex;
    return $this;
  }