revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-25 00:02:40 +01:00
Code Issues Releases Wiki Activity

Fix call to non-existing ArcanistAliasEffect::EFFECT_CONFIGURATION in ArcanistAliasEngine

Browse source 
Summary:
`ArcanistAliasEffect::EFFECT_CONFIGURATION` does not exist.
Probably `ArcanistAliasEffect::EFFECT_MISCONFIGURATION` was meant, given the message it sets.

Test Plan: Carefully read the code.

Reviewers: O1 Blessed Committers, avivey

Reviewed By: O1 Blessed Committers, avivey

Subscribers: tobiaswiese, valerio.bozzolan, Matthew, Cigaryno

Differential Revision: https://we.phorge.it/D25652
This commit is contained in:
Andre Klapper 2024-05-16 17:28:47 +02:00
parent 0af89f7d32
commit 21fbc806e4
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/toolset/ArcanistAliasEngine.php
View file

@ -67,7 +67,8 @@ final class ArcanistAliasEngine
// This alias is not defined properly, so we're going to ignore it. // This alias is not defined properly, so we're going to ignore it.
unset($aliases[$key]); unset($aliases[$key]);
$results[] = $this->newEffect(ArcanistAliasEffect::EFFECT_CONFIGURATION) $results[] =
$this->newEffect(ArcanistAliasEffect::EFFECT_MISCONFIGURATION)
->setMessage( ->setMessage(
pht( pht(
'Configuration source ("%s") defines an invalid alias, which '. 'Configuration source ("%s") defines an invalid alias, which '.