Page MenuHomePhorge

Remove call to non-existing AphrontTypeaheadTemplateView::renderToken()
ClosedPublic

Authored by aklapper on Jul 3 2024, 06:39.
Tags
None
Referenced Files
F3720260: D25721.1745704946.diff
Fri, Apr 25, 22:02
F3720136: D25721.1745697668.diff
Fri, Apr 25, 20:01
F3709314: D25721.1745647263.diff
Fri, Apr 25, 06:01
F3701793: D25721.1745607231.diff
Thu, Apr 24, 18:53
F3604772: D25721.1745228505.diff
Sun, Apr 20, 09:41
F3577046: D25721.1745077631.diff
Fri, Apr 18, 15:47
F3506112: D25721.1744800156.diff
Tue, Apr 15, 10:42
F3377252: D25721.1744337750.diff
Thu, Apr 10, 02:15

Details

Summary

AphrontTypeaheadTemplateView::renderToken() is an undefined method,
and its result is stored in an orphan variable, and no child class
extends AphrontTypeaheadTemplateView defining a renderToken().
Thus remove its call from AphrontTypeaheadTemplateView::render().

Test Plan

Read/grep the code in this class.

Diff Detail

Repository
rP Phorge
Branch
aphrontTypeaheadTemplateViewRenderToken
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1421
Build 1421: arc lint + arc unit

Event Timeline

aklapper requested review of this revision.Jul 3 2024, 06:39

lgtm

Additional consideration under control: an external class may have extended AphrontTypeaheadTemplateView and defining a renderToken() there, and making this snippet operative (I'm not saying that it's a good practice, I'm saying that it would work). But this extreme corner case is not the case.

grep -FR 'renderToken(' .
./src/applications/search/compiler/PhutilSearchQueryCompiler.php:      $result[] = $this->renderToken($token);
./src/applications/search/compiler/PhutilSearchQueryCompiler.php:      $result[] = $this->renderToken($token);
./src/applications/search/compiler/PhutilSearchQueryCompiler.php:      $result[] = $this->renderToken($token, $this->getStemmer());
./src/applications/search/compiler/PhutilSearchQueryCompiler.php:  private function renderToken(
./src/view/control/AphrontTypeaheadTemplateView.php:      $tokens[] = $this->renderToken($key, $value);

The PhutilSearchQueryCompiler is completely unrelated 👍

Moreover the result is stored in an orphan variable. It may happen that this method could have desired rendering side-effects, but this is just not the case, per above. Thanks

This revision is now accepted and ready to land.Jul 20 2024, 08:40