D9679.id23236.diff
No OneTemporary
Actions

Size

934 B

Referenced Files

None

Subscribers

None

D9679.id23236.diff
View Options

	diff --git a/src/lint/engine/ArcanistConfigurationDrivenLintEngine.php b/src/lint/engine/ArcanistConfigurationDrivenLintEngine.php
	--- a/src/lint/engine/ArcanistConfigurationDrivenLintEngine.php
	+++ b/src/lint/engine/ArcanistConfigurationDrivenLintEngine.php
	@@ -13,11 +13,16 @@
	}

	$data = Filesystem::readFile($config_path);
	- $config = json_decode($data, true);
	- if (!is_array($config)) {
	- throw new Exception(
	- "Expected '.arclint' file to be a valid JSON file, but failed to ".
	- "decode it: {$config_path}");
	+ $config = null;
	+ try {
	+ $config = phutil_json_decode($data);
	+ } catch (PhutilJSONParserException $ex) {
	+ throw new PhutilProxyException(
	+ pht(
	+ "Expected '.arclint' file to be a valid JSON file, but failed to ".
	+ "decode it: %s",
	+ $config_path),
	+ $ex);
	}

	$linters = $this->loadAvailableLinters();