D10539.id25320.diff
No OneTemporary
Actions

Size

1 KB

Referenced Files

None

Subscribers

None

D10539.id25320.diff
View Options

	diff --git a/src/lint/linter/xhpast/ArcanistXHPASTLintNamingHook.php b/src/lint/linter/xhpast/ArcanistXHPASTLintNamingHook.php
	--- a/src/lint/linter/xhpast/ArcanistXHPASTLintNamingHook.php
	+++ b/src/lint/linter/xhpast/ArcanistXHPASTLintNamingHook.php
	@@ -113,9 +113,29 @@
	* @task util
	*/
	public static function stripPHPFunction($symbol) {
	- // Allow initial "__" for magic methods like __construct; we could also
	- // enumerate these explicitly.
	- return preg_replace('/^__/', '', $symbol);
	+ switch ($symbol) {
	+ case '__assign_concat':
	+ case '__call':
	+ case '__callStatic':
	+ case '__clone':
	+ case '__concat':
	+ case '__construct':
	+ case '__debugInfo':
	+ case '__destruct':
	+ case '__get':
	+ case '__invoke':
	+ case '__isset':
	+ case '__set':
	+ case '__set_state':
	+ case '__sleep':
	+ case '__toString':
	+ case '__unset':
	+ case '__wakeup':
	+ return preg_replace('/^__/', '', $symbol);
	+
	+ default:
	+ return $symbol;
	+ }
	}

	/**